Importing a jupyter notebook is not generating correct code

Environment data

VS Code version:  1.31.1
Extension version (available under the Extensions sidebar): 2019.3.6558
OS and version: windows10
Python version (& distribution if applicable, e.g. Anaconda): 3.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda

Expected behaviour

Actual behaviour

Steps to reproduce:

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
XXX

Can you paste the code that's causing the problem? Something in the imported code is invalid python.

Or even better if you could share the original notebook?

Or even better if you could share the original notebook?

yes of course. the file is here:
http://cdn.yoqi.me/direct/2019-05-21/1_hello_tensorflow.ipynb
I can give you any information you need.

The import os change working directory stuff we put at the top of the file is messing up the
from __future__ import print_function

You can just delete the code that looks like this:
#%% Change working directory from the workspace root to the ipynb file location. Turn this addition off with the DataScience.changeDirOnImportExport setting
import os
try:
	os.chdir(os.path.join(os.getcwd(), '..\Import'))
	print(os.getcwd())
except:
	pass

That code is added to every import if you have this checked:

Otherwise I'm having trouble getting tensorflow to run. I keep getting this:

Yeah it looks like tensorflow doesn't have session anymore. I must have installed too new of a tensorflow
https://github.com/tensorflow/tensorflow/issues/26816

Oh but you don't click on the tensorflow cell. So maybe my advice didn't matter. You're clicking on this cell in your gif:

And that is failing, possibly because you're using Python 2?
However in the video you posted, you mentioned the entire file wasn't there? Were you hoping it would open up the file as a jupyter notebook inside of VS Code?
Unfortunately the product doesn't work that way at the moment. Currently we expect you to work with '.py' files. The import notebook command merely extracts your python code.
We do have another open issue on inline notebooks though.
https://github.com/microsoft/vscode-python/issues/4575

Something like this:
https://atom.io/packages/hydrogen

Yeah it looks like tensorflow doesn't have session anymore. I must have installed too new of a tensorflow
tensorflow/tensorflow#26816

The function I want is just to preview the file when I open a jupyter file.  and can run code.

I don't believe we have what you want (yet). Importing a notebook just turns it into the python code in your notebook. You then have to run each cell to view the output in the other pane.

If the hydrogen thing meets your needs, feel free to up vote the issue we have that I linked:
#4575

In the video you mention the 'right cannot display the html'. Can you elaborate on that comment? What HTML did you want? (Just wondering what we should do when we implement the hydrogen support)

In the video you mention the 'right cannot display the html'. Can you elaborate on that comment? What HTML did you want? (Just wondering what we should do when we implement the hydrogen support)

thank you, I want preview the whole jupyter file. I hope that we can get this feature out quickly.

when can I preview a jupyter file code ? I read Introduction of our plugin.  it show have the function.

It does this now. All you have to do is click on the 'Run Below' command at the very top. That will run all 'cells' in the imported notebook. That's what the docs are showing anyway.
You probably want to uncheck this setting before you do the import though.

the hydrogen thing meets mine needs!
I have set it up according to your suggestion.

I often need to look at the contents of the jupyter file and then consider whether I want to execute it.

Oh you want to see the current output. The last run you mean?

Our current implementation requires a rerun on every open. It's more like a terminal than a notebook viewer.

Oh you want to see the current output. The last run you mean?

yes, you are right.

Hmm, that might be possible sooner (before we do the hydrogen thing). On import, just fill the interactive window with the current output.

the plugin has the function. but can not run code.
https://marketplace.visualstudio.com/items?itemName=jithurjacob.nbpreviewer

Thanks. That's super helpful.

I think if vscode can preview jupyter file ,and can run it. it may be better than the web page of  jupyter notebook.


Update Truth and other dependencies

Changes

Updated Gradle wrapper to 4.9
Updated Android Gradle plugin to 3.2.0
Updated Support Library to 28.0.0 (had to remove subjects for deprecated classes: android.graphics.Rasterizer, android.support.v4.media.* that no longer exist in Android framework or Support Library)
Updated Truth to 0.42 (mostly search/replace due to API changes):

Replaced SubjectFactory with Subject.Factory
Replaced FailureStrategy with FailureMetadata
Replaced name of factory method getSubject with createSubject
Replaced usage of withFailureMessage with withMessage
Replaced calls FluentIterable.of(CharSequence[]) with FluentIterable.from(CharSequence[]).
Guava 20 changed declaration of former method to take vararg arguments. As a result it doesn't
work as expected when we pass array to it. Infered type become FluentIterable<CharSequence[]> which doesn't match with signature of mapping function mapToString.
Updated code generation in generate-assertions-java.py to use new types

Removed deprecated Checkstyle check RedundantThrows that no longer exists in newer Checkstyle versions and causes checkstyle tasks to fail
Added some Lint warnings to ignore list since they are expected.

@alapshin thanks! We'll get these changes published soon

@MariusVolkhart
No problem and thank you for very useful library.
Considering that Google started using Truth in testing support library
androidx.test.ext:truth and the fact that assertj-android has been deprecated for
a year some people may be interested to migrating from AssertJ to Truth.
At least I was and was very glad to find out that I don't have to rewrite tests
that use assertj-android and could simply replace one library with another.

Just published to maven central. Took longer than expected. Thanks again for your contribution!


About the experiment

Hi and thanks for the great work. I have two questions about the experiment. 1. What is the division ratio of Sin and Sreal when calculating FID? Whether the three datasets randomly select 30 reference images for each unseen category, and then generate 128 images. 2. For the experimental part "Augmentation for Classification" , is the generated image using the image in Dtrain as reference images?

Hi, sorry for the late reply.

The ratio is 0.5 as specified in the config file. Yes, you can find the code in main_metric.py.
Yes, the source images to do augmentation is from the training set.


regression: bind-mounting onto a symlink fails with "Error: openat2 localtime: No such file or directory: OCI runtime command not found error"

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

Bind-mounting a file that happens to be a symlink in the container ("to" path) fails. This works in older versions.
Steps to reproduce the issue:

podman run --rm  -v /etc/localtime:/etc/localtime:ro -it docker.io/fedora:32

Describe the results you received:
Error: openat2 `localtime`: No such file or directory: OCI runtime command not found error
Describe the results you expected:
Successful container start, with the file properly mounted.
Additional information you deem important (e.g. issue happens only occasionally):
This is a regression from podman 1.9.3.
Output of podman version:
podman version 2.0.1

Output of podman info --debug:
host:
  arch: amd64
  buildahVersion: 1.15.0
  cgroupVersion: v2
  conmon:
    package: conmon-2.0.18-1.fc32.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.18, commit: 6e8799f576f11f902cd8a8d8b45b2b2caf636a85'
  cpus: 12
  distribution:
    distribution: fedora
    version: "32"
  eventLogger: file
  hostname: avi
  idMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  kernel: 5.6.10-300.fc32.x86_64
  linkmode: dynamic
  memFree:<PHONE_NUMBER>
  memTotal:<PHONE_NUMBER>6
  ociRuntime:
    name: crun
    package: crun-0.14-2.fc32.x86_64
    path: /usr/bin/crun
    version: |-
      crun version 0.14
      commit: ebc56fc9bcce4b3208bb0079636c80545122bf58
      spec: 1.0.0
      +SYSTEMD +SELINUX +APPARMOR +CAP +SECCOMP +EBPF +YAJL
  os: linux
  remoteSocket:
    path: /run/user/1000/podman/podman.sock
  rootless: true
  slirp4netns:
    executable: /usr/bin/slirp4netns
    package: slirp4netns-1.1.1-1.fc32.x86_64
    version: |-
      slirp4netns version 1.1.1
      commit: bbf27c5acd4356edb97fa639b4e15e0cd56a39d5
      libslirp: 4.2.0
      SLIRP_CONFIG_VERSION_MAX: 2
  swapFree:<PHONE_NUMBER>
  swapTotal:<PHONE_NUMBER>2
  uptime: 1314h 53m 11.98s (Approximately 54.75 days)
registries:
  search:
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - docker.io
store:
  configFile: /home/avi/.config/containers/storage.conf
  containerStore:
    number: 0
    paused: 0
    running: 0
    stopped: 0
  graphDriverName: overlay
  graphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-1.1.1-1.fc32.x86_64
      Version: |-
        fusermount3 version: 3.9.1
        fuse-overlayfs: version 1.1.0
        FUSE library version 3.9.1
        using FUSE kernel interface version 7.31
  graphRoot: /home/avi/.local/share/containers/storage
  graphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  imageStore:
    number: 2
  runRoot: /run/user/1000/containers
  volumePath: /home/avi/.local/share/containers/storage/volumes
version:
  APIVersion: 1
  Built: 0
  BuiltTime: Thu Jan  1 02:00:00 1970
  GitCommit: ""
  GoVersion: go1.14.3
  OsArch: linux/amd64
  Version: 2.0.1

Package info (e.g. output of rpm -q podman or apt list podman):
podman-2.0.1-1.fc32.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical, plain Fedora 32.

Downgrading podman did not help, but downgrading crun (0.14 -> 0.13) did.


Coordinate rounding with larger models results in object offsets.

I believe since gltfjsx parses coordinates out to two decimals, (10.00) it causes objects to move in models with larger scales. Normally these minute movements wouldn't be noticeable, but in my case it causes architectural elements, like sidewalks or site maps to be offset by a few inches, which results in gaps in my model.
Images attached:
Don Mccurdy GLTF Viewer

GLTF-JSX

oh, yes, and you can change it:
 --precision, -p     Number of fractional digits         [number ] [default: 2]

Thanks! I must've read right past that in the docs. 😅
The first screen capture was using Don Mccurdy's GLTF viewer which has an orange town/sunset hdri for reflections. I'm using a different hdri in the second screen capture, along with baking all the geometry in 3ds Max/Vray so I don't have to rely on Three's realtime shadowing. There's also a bit of gamma control and fog


JIT Provisioning with option Prompt for username, password and consent is not working

Environment
IS 5.10.0 RC2
Embedded LDAP with H2
Steps to Reproduce

Create another IS as a secondary IDP
Create a user as a federated user in the secondary IDP created in step1
Enable JIT Provisioning with the option Prompt for username, password and consent
Try to login to the SP travelocity with the federated user created in secondary IDP.
Give a username on the UI to proceed with self-registering

Observation
Once the username is entered and click on proceed it will say as the tenant domain is invalid and cannot proceed to the self registration page

Reopening the issue as the master PR has been reverted with https://github.com/wso2/identity-apps/pull/634 due to a regression in the self-registration flow.
We will need to fix the place where tenantDomain param is set in the backend.
For JIT provisioning with username we use MultitenantConstants.TENANT_DOMAIN_HEADER_NAME[1] variable as the key to the tenant domain param. It resolves to "TenantDomain"[2]
[1] - https://github.com/wso2/carbon-identity-framework/blob/master/components/authentication-framework/org.wso2.carbon.identity.application.authentication.framework/src/main/java/org/wso2/carbon/identity/application/authentication/framework/handler/request/impl/JITProvisioningPostAuthenticationHandler.java#L527
[2] - https://github.com/wso2/carbon-kernel/blob/4.6.x/core/org.wso2.carbon.utils/src/main/java/org/wso2/carbon/utils/multitenancy/MultitenantConstants.java#L38

The issue cannot be reproduced with the given steps in IS-5.11 m35. Possibly fixed with https://github.com/wso2/identity-apps/pull/835


Flutter is laggy 😢

Flutter is so laggy and I want the issue to be addressed. The app runs smoothly after the first run. But each time you enter the app it lags a lot. Doesn’t stay at 60FPS. I have tried it first on my iPhone 6. But then when I tried it on an iPhone 11 Pro Max the app still lags a bit at every startup. It is frustrating. Flutter promises 60FPS but I don’t see it. I need help. Btw the problem is worse on iOS than on Android.
I can’t share source code but I can give you app links to download and try it.

Have you tested the suggestions here? Flutter Best Practices
There are many factors that affect performance. For example: large images, too many items, etc.
You can send the application link so that we can make a review.

iOS: https://apps.apple.com/sa/app/split-it-split-your-stories/id1525916578
Android: https://play.google.com/store/apps/details?id=dev.alkhalaf.split_it

I think @AliAnilKocak is making a good point,
performance are very much dependent from your implementation and optimization,
you may want to make sure that

the installed flutter version is the latest stable or above

and provide

flutter doctor -v
flutter run -v (or flutter run -d chrome -v for web etc)
pubspec.yaml
MOST IMPORTANTLY a minimal reproducible code sample (not a snippet or production code!)

Hi @YazeedAlKhalaf
Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample that reproduces this issue?
Thank you

I think the cause is using the FFmpeg library.
I am closing this for now,


Clean up gatherlogs.automate2.elasticsearch_primary_shard_unavailable…

… control

ES is stylized  as Elasticsearch 😅

I don't know that the output of curl -XGET localhost:10141/_cat/shards?h=index,shard,prirep,state,unassigned.reason| grep UNASSIGNED is all that helpful given that even in healthy A2, plenty of healthy index shards look like this:

comp-3-profiles             2 r UNASSIGNED CLUSTER_RECOVERED

Is there a different token we can grep for?

Always ask for pretty-print

Finally, it looks like we copypasta'd straight from https://discuss.elastic.co/t/allocate-stale-primary-primary-is-already-assigned/142995 for the flush instructions; stats_new needs to be the relevant index name for each shard that gave the 'already assigned' error. I think 🤷♂

I'll change over 10141 to 10144, but before I do, is there also some other way we can differentiate between the problematic UNASSIGNED shards with the ones that are not? I was not able to break my A2 ES in this specific way to see what, if anything, changes about that output. If you have some ideas, maybe we could add some kind of diagnostic back in


Add telemetry

https://code.visualstudio.com/docs/getstarted/telemetry
We will need to stop bundling the server-side code, as the @vscode/extension-telemetry no longer supports it.

@marwan-at-work is looking at clientmetrics to see if we can use that.


Support for right-to-left (rtl) languages

I arranged the RTD stylesheet to allow support for rtl (right-to-left) languages.
A few points:

Text direction is determined by the page content, not the language declared. Therefore, only pages that are actually written in (or have at least the first heading translated to) a rtlscript will appear as rtl.
The entire page is mirrored, including navigation, etc.
This works pretty much impeccably well on Firefox including with responsive design, which supports rtl detection.
For browsers that do not have support for rtl detection (:dir(rtl) attribute) yet, it works passably well (a few odd paragraph alignments and borders, and the navigation is still on the left side). But at least rtl text is readable, even on these browsers (the current RTD theme displays RTL text in the wrong order). And it is expected that other browsers will add rtl detection support soon, so this should be a temporary problem.

In summary, this pull request:

Appears to work perfectly for Firefox.
Works better than the current theme on other browsers.
Has no impact on left-to-right pages, no matter what browser is used.

For an example, see our partially translated Urdu site in which I overrode the default RTD theme with this one:
http://tinamit.readthedocs.io/ur/latest/docu/Conectado.html

Thanks for the contribution. However, these changes have been made to our CSS files, not our actual base SASS templates. They should be made there to be integrated into the theme properly:
https://github.com/rtfd/sphinx_rtd_theme/tree/master/sass
More docs on how to contribute and build things are here: https://github.com/rtfd/sphinx_rtd_theme#contributing-or-modifying-the-theme

Ah, thank you! I will try to learn SASS and get back to you then.

To me, it appears that the css file changes are not needed at all. Also, I would rather set this up as a theme option that should  default to auto or ltr

Yes, I will try to get around to it soon!

દ્વારા: Aaron Carlisle<EMAIL_ADDRESS>મોકલ્યું: 16 ડિસેમ્બર 2017 00:10:59
પ્રતિ: rtfd/sphinx_rtd_theme
Cc: Julien Malard; Author
વિષય: Re: [rtfd/sphinx_rtd_theme] Support for right-to-left (rtl) languages (#498)
@Blendify requested changes on this pull request.
See my above comment
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rtfd/sphinx_rtd_theme/pull/498#pullrequestreview-83966780, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKMb62xx58VLKbvmVwNKBeU4CGa7Rc4Rks5tA1DigaJpZM4Q_XtQ.

I'm not familiar with SASS so won't be much help unfortunately.... RTD sets the language correctly in the HTML tag though, so maybe you could try and detect direction based on that instead? My link above also modifies the CSS to move the navigation to the right of the page, but some more work is needed there to get this functioning properly on small screens.

Hello,
Thanks! The problem I ran into with language tags is that there are a ton of languages that use different writing systems, and I did not fancy having to make a list of all possible languages that use each of the RTL scripts (arabic, divehi, hebrew, nko and probably a few others I have overlooked!), especially since there are quite a few languages that use arabic variants...
Out of interest, what differences are there between your implementation and mine (https://tinamit.readthedocs.io/ur/latest/docu/Modelo.html), navigation bar aside? I will try to keep the best of both for the final SASS version.
Will keep you posted,
Julien

I think yours focuses more on text styles, while the main thing I tried to do was play with the navigation. I also integrated the CSS in a different way from conf.py, and use a different style of language selector. I might try and get the toc-expand toggles properly aligned and fix the breadcrumbs and GitHub link over the next few days, keep an eye on this file to see if anything useful is in there for you: https://github.com/dashpay/docs/blob/master/_static/style.css

I've added conditional CSS to support RTL languages on small screens (mobile) as well. A language switcher was also added, although this does not exist in the parent theme, so that CSS would need to be removed if this was to be integrated into the Sphinx RTD theme one day.
CSS: https://github.com/dashpay/docs/blob/master/_static/style.css
Result: https://docs.dash.org/ar/latest/

Interesting! I love the language switcher; much easier to find than on the original RTD theme.

Hello,
I got it to work with SASS! At least mostly. If anyone could get the little arrows on the next and previous buttons to work it would be appreciated. :)
Please let me know if this looks better now.
Regards,
Julien Malard

Hello,
Any news?
-Julien Malard

Hello,
Any news? Please let me know if I can contribute anything.

This is updated in https://github.com/readthedocs/sphinx_rtd_theme/pull/986 please follow the discussion there instead


Add basic Fate Core layout

I'm not sure about adding the export * from "./fateCore"; to the index file, please tell me to throw it out if necessary :)

I'm not sure about adding the export * from "./fateCore"; to the index file, please tell me to throw it out if necessary :)

No worries. I only put that in there for the 5e layout so I could be lazy.


JSON serialzation

How to properly turn pydantic schemas into json?
The problem is that the BaseModel.dict() method returns dict, it's okay but it doesn't convert some default types into string (datetime.date, Enum and etc.).
It seems there is no other way to translate pydantic object into dict and serialize those types which is strange as pydantic can deserialize them and even force specific format in some way (datetime_parse.date_re for example which is hardcoded and hardly changeable).

pydantic doesn't take care of serialisation because:

people do it in lots of different ways - for example you might know that your data doesn't contain any complex types and therefore want to use ujson or similar, or you might want to use a completely different serialisation protocol like msgpack.
it's very simple - use a custom encoder like this and json.dumps(m.dict(), cls=MyEncoder)

datetime_parse.date_re for example which is hardcoded and hardly changeable

This is unrelated to this issue, if you have a request for a change and a suggestion of how it might work, please create a separate issue. Hint: it is changeable, just add your own validator.

@samuelcolvin thanks for an answer. I know about the way to change it with my own validator (as I reported some bug with it recently).
I'm just trying to understand why pydantic take care of serialization and doesn't take care of serialisation when your arguments applicable for both of them.

I guess we should add a json() method that serialises using a default and best effort encoder class but allows and optional argument of a different encoder.

@samuelcolvin yeah, sounds good.

great, PR welcome.
Otherwise, I'll work on it in a couple of weeks. The link about should be a good starting point, the only thing I can thing of to add would be enums.

@samuelcolvin I'm not sure that I'll have time for this but if I will, I would be happy to help.

-1.
In my opinion almost every other issue in the github tracker is more important than this (as of May 2018). Serialization depends on intended usage and it can be done independently of pydantic. Getting the other parts right is much more important and urgent.

By the way, the method should not be called "json()", unless it returns a string. Since it returns a dictionary, it should be called "to_dict()". JSON is another step after we have the dictionary.

Yes, this is what I was saying.
We already have the dict() method which returns a dict.
The problem is that (quite correctly) dict() doesn't return a dict which can necessarily be serialised straight to a dict without a custom serialiser.

There are a lot of types that will be serialized in the same way from project to project. The point is that nobody wants to implement the same encoder every time and duplicate code.
And It's just strange that pydantic can decode string into date but can't encode date into string. It just doesn't make sense. As if you could read but not write. My opinion is that If you strongly against encoding for user, decoding should be removed too.

+1 to @xunto's reasoning here. Serialization that handles aliases would be extremely nice.

@samuelcolvin I have simple project (code base mostly taken from Django plus added support for enums) with extended JSONEncoder which can be possible used to solve this issue. What do you think?

I think it would be simplest to keep this code inside harrier. Moreover, I disagree with some of django's decisions, eg. it's generally best to encode timedelta as seconds rather than P1DT02H00M03.400000S

I would welcome a non-intrusive default in pydantic that allows json.dumps(some_model) to just work, without blocking customization. I'm new to Python so take my opinion with grain of salt.

It won't quite be json.dumps(some_model) it'll be more like some_model.json() or equivalently json.dumps(some_model, cls=PydanticEncoder).
I'm going to work on this soon.

@samuelcolvin just noticed, cool! But what about Enums?

https://github.com/samuelcolvin/pydantic/blob/master/pydantic/json.py#L32

@samuelcolvin oh, sorry. I didn't noticed. Cool! Good job!

Thoughts on adding this to dataclasses?

pydantic enums are as close to vanilla standard library enums as possible. That's not going to change.
However I'd be keen on a bunch of utility functions for processing datacalsses, eg.:

dataclass_json()
dataclass_schema()
etc.


Get input data on output listener method

I am using pty.js in a nodejs application to emulate a interactive terminal. But the I got the input data in the output listener method. Is there a way to get the output data only?
Below is my source code:
var pty = require('pty.js');

var term = pty.spawn('bash', [], {
  name: 'xterm-color',
  cols: 80,
  rows: 30,
  cwd: process.env.HOME,
  env: process.env
});

term.on('data', function(data) {
  console.log(data);
});

term.write('echo hello\n');

when running above code, I can see the output:
$ node official.js
echo hello

bash-3.2$ echo hello

hello
bash-3.2$

from the above output you can observe that the input command echo hello got printed twice in term.on('data' ...) function.  Is there a way to get ride of the input data from the output listener method?

I'm looking for this same solution. Currently, the only thing I could think to do is to capture the input manually in a variable and compare the output with the last known input on each data event, choosing to ignore the said output if they are equal.
let lastInput = '';

term.on('data', (data) => {
  if(lastInput !== data) {
    process.stdout.write(data);  //  instead of console.log to avoid extra newline characters
  }
}

process.stdin.on('readable', () => {
  const chunk = process.stdin.read();
  if (chunk !== null) {
    lastInput = chunk;
    term.write(chunk);
  }
}

The problem with this is that while it seems straight forward, data and chunk are of different types. Then once you get past that with everything converted to what it needs to be for comparison, you end up with them having different newline characters. My attempt had lastInput possessing a trailing \n and data possessing \r\n.
Finally, on top of all of that, stdout doesn't guarantee that the entire string is handed to you on the data event. I would often get something like this.
t
est
bash-3.2$

My next step is to possibly wait for a newline character before parsing data. I'm just afraid that there will be scenarios where all of this will cause unintended behavior. It would be much better if the echo could be disabled with a configuration option or something.

Solution
Found this explanation in the node-pty repo.
process.stdin.setRawMode(true)

I would expect it to work with pty.js as well. On that note, this repo hasn't had a commit in a year, so I would consider switching to node-pty if you're doing anything crucial. I've only recently discovered them, and don't know much about them, but it seems like the community has switched boats. One might be the continuation of the other. I don't know.

@euroclydon37 node-pty was forked from pty.js about a year ago, here are the main changes:

Better Windows support
Better support for recent Node versions
Converted to TS, added some tests

Checkout the releases for more specifics, 0.4.1 was the first release after forking, 0.6.0 removed pty.js' modification to winpty and updated it to latest.

Loving it so far.


Proper BPF support?

@tharvik in case that's in line with the C4DT's priorities, having Klint work on BPF programs directly (not the hacky stuff from the paper) might be a good showcase.
A student had almost finished BPF ISA support for angr in a semester project with me, but some issues cropped up in the actual PR and I guess there wasn't time after the semester to finish it. Maybe it can be resurrected? https://github.com/angr/angr-platforms/pull/46

BPF programs might be a good showcase.

Good idea, I'll check with the hierachy :)
Hoo, a bunch of existing programs to try klint on: https://github.com/zoidbergwill/awesome-ebpf#examples
Correct me if I'm wrong, there is (or was) support for eBPF Maps in code?

There was indeed, IIRC the main difference from Klint's map_* stuff is that BPF maps copy keys and values rather than partially owning key poiners.

quick update: the hierarchy validated :partying_face:

support for eBPF in angr is in progress at https://github.com/angr/angr-platforms/pull/55

@tharvik any news on this?

upstream isn't really proactive, I bumped them, hopefully we'll have some news this week.
anyway, the PR is working (minus relocation & some specials instructions), feel free to use it, I'm happy to help with/fix any issue you might encounter.

I see the PR was merged 🎉 I guess angr-platforms should become a dependency of klint now? I don't know how the packaging of it works (so that we can point Klint to an ebpf binary and it "just works")


Flutter app failed to build or run on android

My flutter app runs well on an i.o.s simulator but does not run when I use an android emulator, also, when I try to create an android build, it runs into the same errors. When I ran flutter build apk, it initializes with no errors, resolves dependencies with no errors, but fails to run the task Gradle task 'assembleRelease' and ends with Daemon:  AAPT2 aapt2-3.2.1-4818971-osx Daemon #0. Below is a more detailed view of the error logs.
  Daemon:  AAPT2 aapt2-3.2.1-4818971-osx Daemon #0

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s

Gradle task 'assembleDebug'... Done                          6.6s
Gradle task assembleDebug failed with exit code 1

I tried changing the compileSdkVersion from 27 to 28 in android/app/build.gradle but it still throws the same error. What can I do?

Can you please post the complete log output from the flutter -v run -v

Probably a duplicate of #27106

Thanks, @zoechi, I will post the Logs of flutter -v run -v on #27106.


Incorrect example in the page

The doc topic is "How to: Programmatically search within a specific folder" but the content and the example describes "How to search a mail having a specific subject". So, the content is misleading.

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

ID: b53cb97d-2c1f-8b19-1496-d91cf408e6d2
Version Independent ID: c4f16638-df03-4440-4dca-763dbdc3fdbe
Content: How to: Programmatically search within a specific folder - Visual Studio
Content Source: docs/vsto/how-to-programmatically-search-within-a-specific-folder.md
Product: visual-studio-windows
Technology: office-development
GitHub Login: @John-Hart
Microsoft Alias: johnhart

Thanks for the feedback.  I think the article does have a sample that shows how to search in the inbox which is an Outlook.Folder.


Better math logic for border position and radius

Take 2.
Applied a 1.75 modifier to the border radius, also applied the border offset to the width and height as well as the x and y.

Actually, the 1.75 modifier might not be needed with my other change.
Please play with it before merging and remove it if you feel it does not help.

hmm it still seems a bit off for me. Doesn't have the same smooth feeling as it currently does.
Do we need to change how works?

I suppose not, I just figured supporting wider border widths would be nice. Right now they start to encroach inside the window for wider borders.
keep this open for now and I'll try and refine it some more.

Ok, I've played with it a lot, and this feels the best. It also solves this case:
Before this PR:

After this PR:

I tested down to a width of 2 all the way to a width of 16 and this feels pretty good.

Looks pretty good now, better than before I would say as well. Great :>

Ultimately I think that this should just be another config setting:
kwmc config prefix-border radius 4
It would allow for the radius to look exactly as desired, however I think what's here now is pretty good for the time being.

That could be implemented very quickly from my part in Kwm at least, if you want to just get it over with right now.
Also on an unrelated note, regarding the "random" segfaults, I have not had this happen to me in the last 3days, so the debugging process is very slow. You wouldn't happen to have gdb installed and be able to run a debug build of Kwm in there? If the crashes appear more frequently for you that is.

I'll tackle the radius option in kwm-overlay next weekend.
I actually haven't been having any seg faults either, but I've been keeping pretty up to date so I'm not sure exactly when they stopped.
If they start to appear again I will do that though.

Aight sounds good, really appreciate it.

I went ahead and did the required changes for Kwm.
I pass the radius setting at the end (after the width), as: rad: floating-point
The entirety looks something like:id r:1 g:0 b:0 a:1 s:4 rad:4

Maybe stroke offset should be a setting in-sted. This would allow borders outside the Window.
It might be an idea to have kwm-overlay read its own settings to reduce the coupling between the apps. This would allow the interface between kwm and kwm-overlay to be only about border types and positions, and let kwm-overlay focus on how those borders should look.


Enrich rules of detecting PostgreSQL

Enrich rules of detecting PostgreSQL.

Hi, @saragluna , @haoozhang . Please help to review this PR.


Fix cleanup logic to first check config_settings

Currently the logic is broken if running pdm install or pdm build as the environment variable is ignored unless config_setting is set. The logic is confusing.
The code has been changed to first check config_setting and evaluate the given value. If unset, check the environment variable. If nothing is set, cleanup by default.

Superseded by #177 Thanks


fix(core): tracer should never throw exception

Fixes @todo
https://github.com/open-telemetry/opentelemetry-js/blob/a15ccb9d25193367dafb4dee9bac2bcbc20952fb/packages/opentelemetry-core/src/trace/NoopTracer.ts#L41

Codecov Report

Merging #147 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
+ Coverage   97.68%   97.68%   +<.01%
==========================================
  Files          32       32
  Lines        2245     2247       +2
  Branches      264      264
==========================================
+ Hits         2193     2195       +2
  Misses         52       52

Impacted Files
Coverage Δ

...s/opentelemetry-core/test/trace/NoopTracer.test.ts
100% <100%> (ø)
:arrow_up:

...ackages/opentelemetry-core/src/trace/NoopTracer.ts
100% <100%> (ø)
:arrow_up:


move example dapp to github pages

from duckdns. Nick can do that and @JamesSmartCell will be across

done.


First boundary skipped

me again, about this change :
https://github.com/cybergeek94/multipart/blob/master/src/server/mod.rs#L61
The first boundary that comes from hyper won't have the \r\n, since the multipart is the http content body, the two previous CRLF are part of the header. It's a real pain, since it removes a simple way to strip the extras CRLF around the boundary.
I'll try to figure out a sensible solution and submit a PR later

Closed in #16, thanks!.


feature request: move components with keyboard

Please provide an option to move components using keyboard. Currently keyboard moves focus but given a typical somewhat irregular location of components, practical merits of this feature are limited and it is impossible to get out of nesting with keyboard at all, so I never use it. On the other hand, I find precise component positioning using a mouse or, worse still, touchpad or trackpoint, somewhat tricky even with visual aids and snap enabled, and feel the need for precise keyboard positioning many times a day.

The GEF (Graphical Eclipse Framework) provides the ability to use a keyboard to move and resize components using a keyboard. See the User Guide, page 19:

Using the Keyboard Instead of the Mouse in a View
It is possible to move and resize selected objects in a View by using the computer keyboard instead of a mouse. To move an object, press the period key (".") once to reveal the MOVE cursor. Then use the Arrow keys, followed by the ENTER key to commit the move. To resize the object, press the period key (".") until the RESIZE cursor appears at the desired resize handle. Press the ENTER key to commit the resize.

[1] https://www.archimatetool.com/downloads/Archi User Guide.pdf

Missed that - sorry. But when snap is on it requires multiple and unpredictable number of keystrokes e.g. with grid size set to 6 I select a component, press "." and start pressing up arrow: component first moves after 4 keystrokes, then after 5, then after 7 and continues to follow 5-7-5-7 pattern. Weird and difficult to judge. I'd prefer one keystroke to move one grid step when snap mode is on and one pixel when it is off. Or is it an Eclipse feature?

This is part of the Eclipse GEF, so not something I can change. If you use the Modelio tool [1], it has the same behaviour. To implement a different approach would not be easy and would require a deep understanding of GEF and ways to hack it. Perhaps we can tag this as "helpwanted" in case someone cares to look at it.
[1] https://www.modelio.org/downloads/download-modelio.html

I'll close this as it's not something I can change. If anyone cares to look deeper into GEF please feel free to do so.


Implement various improvements to the "console" lexer, closes issues #2496-#2498

ShellSessionBaseLexer: Support heredoc input
ShellSessionBaseLexer: Only match prompt once
tests: Support arbitrary Lexer options in snippets
ShellSessionBaseLexer: Support whitespace in virtualenv prompt
Closes #2496
Closes #2497
Closes #2498

This is a PR to close issues #2496, #2497, #2498, #2499.
Looking forward to any feedback you may have, happy to also split this into smaller changes, if you think these modifications need more discussion.

@jeanas

Yes, please at least separate the HTML formatter change from console lexer changes.

ACK, I have forced push into this branch so we focus this PR on the lexer changes.
I will submit the HTML formatter change separately.
I have edited my comments above to reflect this.


sound is not same after decode

When i done Wave to AMR Encoding and then i decode this same encoded file but the  the sound is not same.

For more details of test.

take one sound.wav file add in bundel then direct encode this file.
decode the file. after decode sounds are not matching.


Nx 11

Update to Nx v11

Still a lot to do:

[ ] Fix playground scripts
[ ] Update and verify Vue, Nuxt, and Docusaurus deps and check for src changes
[ ] E2e tests are using npm to install rather than yarn


Allow Passing Model Keyword Arguments Through the parallelize Interface

Allow passing model keyword arguments through parallelize to AutoConfig.from_pretrained.
This primarily came about because AutoConfig.from_pretrained needs to be passed trust_remote_code=True when a model repo has custom code, otherwise the config loading will fail. This will simply allow passing any custom params to the AutoConfig interface.

lgtm


Add support for Kind IPV6 clusters deployment

Run 'make clusters using=ipv6-stack' to deploy IPV6 Kind clusters.
This capability will be useful for verifying Submariner's IPv6 support.

Should we add "ipv6_stack" to the Deployment GHA matrix?

Should we add "ipv6_stack" to the Deployment GHA matrix?

Do you mean to update this ?
Does it make sense to add it to GHA matrix before V6 is supported on Submariner side?

Does it make sense to add it to GHA matrix before V6 is supported on Submariner side?

yeah I guess not.


Migrate dotnet/templating to dotnet/sdk

Migrate the following directories from https://github.com/GangWang01/templating/tree/main to dotnet/sdk with histories.

src/dotnet-new3/
src/Microsoft.TemplateEngine.Cli/
template_feed/
test/dotnet-new3.UnitTests/
test/Microsoft.TemplateEngine.Cli.IntegrationTests/
test/Microsoft.TemplateEngine.Cli.UnitTests/
test/Microsoft.TemplateEngine.EndToEndTestHarness/

Note that they are put under newly created folder src/dotnet-new for now. We may concern where they will be located in dotnet/sdk respectively. Please give your suggestion on this.
Pending to do:

[ ] Resolve references of projects migrated to dotnet/sdk
[ ] Consider how Microsoft.TemplateEngine.Cli from dotnet/sdk is referred by Microsoft.TemplateSearch.TemplateDiscovery in dotnet/templating once migration is done.

Associated test templates were not moved here.


Subnet broadcasts

I previously opened this before, but I think it got lost the repo ownership changes.
Currently smoltcp only responds to physical layer broadcasts (<IP_ADDRESS>), this PR enables the interface to respond to subnet broadcasts, as discussed in https://tools.ietf.org/html/rfc917 chapter 8.
Questions

Should this check be applied to all ip_addrs associated with the iface?

Seems like the travis errors are unrelated, as they fail on current master too?

No idea what happened with tests, shouldn't that be impossible with protected branches, anyway?

Yes, master is broken, with both stable and nightly. What's weird is that commits that Travis previously said passed tests are now not passing.

Should probably be rebased, looks reasonable otherwise.

Looks great now that it checks all our addrs!
Commits got somehow duplicated when merging, can you rebase on top of master?

Will do, some tests are still failing so I'll fix those first and then squash.

Looks great now that it checks all our addrs!
Commits got somehow duplicated when merging, can you rebase on top of master?

Will do, some tests are still failing so I'll fix those first and then squash.

 Should be fixed now, sorry about that :)

Whoops, must have missed that! Should be fixed now, sorry about that :)

Fairly minor change, but I realised only ipv4 has the concept of broadcasts, so I removed a pointless match, and instead just pass the ip directly.

Fairly minor change, but I realised only ipv4 has the concept of broadcasts, so I removed a pointless match, and instead just pass the ip directly.

Awesome! Thank you!

Awesome! Thank you!


Ability to initialize new react-native project with past versions of react-native

I am trying to create a new react native project which should utilize an older version of react-native.
The result I would like would be to do something like: react-native init MyProject -v 0.13.2 and have the version of react-native it uses be 0.13.2 with all of the proper generated templates put into place in xCode (and android for later versions)
However, there doesn't seem to be any options with react-native-cli for initializing with old versions of react-native.
Currently there is an upgrade command to help install these new templates moving up versions, but no downgrade for the reversal of these templates.

We don't support versions older than the very latest and master, but my recommendation would be to manually set up a project and npm install an older version of RN. To set up the templates you could go back to one of the release branches in this repo and see how the template is generated.

Honestly, I'm unsure why you'd want to do this...keeping up with the latest version of RN as much as possible is definitely the recommended approach, given the fact that it changes as fast as it does. If you get even 3 versions behind (which can happen in just 6 weeks, if you're going by stable branches), you run the risk of having a large number of breaking changes when you upgrade.
If you have a particular use case for using an older version, you should maintain your own fork (we've done that internally at my company, and it's worked well to keep from having to upgrade constantly).
In light of the fact that we're not supporting older versions of RN, I'm going to close this issue.

I recommend we reopen this issue. There is definitely a need to init a new project with older versions of RN. One example (there are many others), is that I'm a publisher of a React-Native component.
When developers open up issues against my GH repo and state the component doesn't work on RN version 0.XX then I need to be able to access that older version of a RN project and test within it. RN is moving very fast and components can/do break between versions.
This example alone, makes me feel we should support resolving issue.

Please reopen

For temporary solution (manual way) see http://stackoverflow.com/a/34213517:

If you install the version of react-native that you want in your project, you can then navigate to (project)/node_modules/react-native/local-cli. Here you will see three generator folders which enclose the exact files RN would have used to initiate a project.

Another approach that might work, and might be easiest, perhaps we should document it, is to navigate through GitHub to the tag/release you want to use for React Native version.  Then install the CLI for that version specifically, e.g. for RN 0.17, you'd install npm install -g<EMAIL_ADDRESS>per https://github.com/facebook/react-native/blob/0.17-stable/react-native-cli/package.json.  And then make your new project that uses RN 0.17 with react-native init MySadDeprecatedProjectLol

Actually that doesn't seem to work, it installs the latest RN version, scratch my previous comment :facepunch:

My next best solution is to git clone this repository, then checkout to the branch git checkout 0.17-stable, then cd react-native-cli, edit the index.js to explicitly say install<EMAIL_ADDRESS>(for all occurrences) and then npm install from inside that folder, and then run ./index.js init MyProject, and then mv MyProject ~/elsewhere.

This seems worth re-opening

As per this post on stackoverflow: http://stackoverflow.com/questions/34211131/new-react-native-project-with-old-version-of-react-native/37623531#37623531 there is now a npm package that has been started to solve this issue https://github.com/jacobbubu/rninit .

It looks like this is now possible:
react-native init --version="0.36.0" MyNewApp

react-native init --version="0.36.0" MyNewApp

It seems this is not documented (at least when trying to get the options at the terminal).


Quick-start instructions working?

In the quick-start, the last (fourth) instruction
python3 bin/gen_exploit.py 0x1234 0x1000 +1000

does not work. Teether currently produces a message:
Usage: bin/gen_exploit.py [flags] <code> <target-address> <shellcode-address> <target_amount> [savefile] [initial-storage file] [initial-balance]

It seems usage instruction described in the page may be changed.
I tested on Ubuntu 18.04.1 .

Hi, thanks for reporting!
Yes, gen_exploit.py needs the contract code as the first argument. I have fixed the readme accordingly!


[BUG][Dashboard] Unable to install on EKS

Describe the bug
Dashboard chart container gets into a restart loop on EKS. Unable to reach dashboard login via port forwarding. Was hoping to use this chart to get dashboard access to opensearch service domain that has been created in a VPC shared by EKS.
To Reproduce
Steps to reproduce the behavior:

Create opensearch domain in VPC that is shared by EKS cluster
Install the chart on an EKS cluster
Try port forwarding to the pod and make a request
Do pod log <pod name> (see output below)

Expected behavior
Pod should start up without a fatal error and stay running.
Chart Name
Dashboard
Screenshots
N/A
Host/Environment

Helm Version: 3.7.0
Kubernetes Version: 1.21

Additional context
Logs from the pod:
{"type":"log","@timestamp":"2022-04-29T21:49:20Z","tags":["info","plugins-service"],"pid":1,"message":"Plugin \"visTypeXy\" is disabled."}
{"type":"log","@timestamp":"2022-04-29T21:49:21Z","tags":["warning","config","deprecation"],"pid":1,"message":"\"cpu.cgroup.path.override\" is deprecated and has been replaced by \"ops.cGroupOverrides.cpuPath\""}
{"type":"log","@timestamp":"2022-04-29T21:49:21Z","tags":["warning","config","deprecation"],"pid":1,"message":"\"cpuacct.cgroup.path.override\" is deprecated and has been replaced by \"ops.cGroupOverrides.cpuAcctPath\""}
{"type":"log","@timestamp":"2022-04-29T21:49:22Z","tags":["fatal","root"],"pid":1,"message":"{ Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:5601\n    at Server.setupListenHandle [as _listen2] (net.js:1263:19)\n    at listenInCluster (net.js:1328:12)\n    at GetAddrInfoReqWrap.doListen (net.js:1461:7)\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:61:10)\n  code: 'EADDRNOTAVAIL',\n  errno: 'EADDRNOTAVAIL',\n  syscall: 'listen',\n  address: '<IP_ADDRESS>',\n  port: 5601 }"}

 FATAL  Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:5601

This is not a crash loop backoff. The container is in the Running state but restarts every 5 minutes and provides the above log. Have tried in both ClusterIp (the default) and LoadBalancer for the service type.
My source code is here: https://github.com/culibraries/folio/blob/issue-169-deploy-kiwi/pulumi/folio/search.ts.
AWS docs here for what I'm trying to do.
Note the stack trace in the error above. From googling on the message, this is commonly caused by the node app not having the port or and the host set in the env (it needs these items to start), which in this case would be in the app's container.

@steveellis have we checked whether the port is available and is not getting used by some other process?

@TheAlgo If I grep on the port in the services I only see one entry, for the service.
kubectl -n <my ns> get service -o wide | grep 5601
opensearch-dashboards               ClusterIP      <IP_ADDRESS>   <none>                                                                    5601/TCP                     2d21h   app=opensearch-dashboards,release=opensearch-dashboards

The ip address that is in the error message on the opensearch dashboard pod changes each time the pod restarts. However that ip address does not exist in the cluster, although it is in the CIDR range. So for example the error is:
FATAL  Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:5601

But if I do k get pods -o wide | grep  <IP_ADDRESS> nothing is found.
In my cluster's security group I have opened 5601 to the cluster's CIDR block <IP_ADDRESS>/16. I have also tried <IP_ADDRESS>/0.

Hey @steveellis can you try to ping your OPENSEARCH_HOSTS url inside dashboard pod?

Hey @steveellis were you able to install the cluster on EKS ?

I'm unable. kube-probe startup issue on EKS. Pod get on an infinite restart loop.

Could you share your values.yaml?


breaking: remove client prop, get client from hook instead

Don't pass client as prop to PrintForm, use useSHOGunAPIClient() instead.
Please review @terrestris/devs

:tada: This PR is included in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


espcoredump: add log verbosity option (IDFGH-9191)

Adds an option to override LOG_LOCAL_LEVEL in the espcoredump component. With this, the amount of logs generated when reading the coredump information from the partition (e.g. by esp_core_dump_get_summary) can be reduced.
This PR addresses and solves #10573.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

We found that an option to fully disable logs for the coredump component was merged with https://github.com/espressif/esp-idf/commit/962621e5d76699bde807c1cd9f3eea35c4352cec. Since this is enough for our use-case, I am going to close this PR.


Keptn 1.0 release

This issue is for discussing the timeline and the scope for the 1.0 release. Right now there are no confirmed plans to release the 1.0 version.
Problem statement

Lack of 1.0 could be considered as lack of stability and therefore there was a statement "do we have any signs it won't be abandoned at any moment" - Dominik Jeziorski

1.0 plans
For the 1.0 release, there are no plans for that until the project passes through incubation process in the CNCF, and then formal security audit and adoption by considerable number of  end users. The scope for 1.0 release is to be based on the feedback by these adopters, I wouldn't say it's finalized at the moment. And yes it's a great time to contribute :blush:
To answer the question regarding potential abandonment, it is also within the scope of the CNCF incubation process and subsequent steps like Core Infrastructure Initiative certification. Many items in this checklist are intended to ensure long-term sustainability of the project and independence from a particular vendor, including open governance, release processes and project infrastructure.
References

Question from Dominik Jeziorski in #help: https://keptn.slack.com/archives/CNRCGFU3U/p1638809656318200

User feedback: "Lack of 1.0 could be considered as lack of stability and therefore there was a statement "do we have any signs it won't be abandoned at any moment. Do we know what scope would have to be delivered to get to this "magic number"?

I am currently working on the proposal, not ready to publishing though. Thanks to @grabnerandi @johannes-b @thisthat @AloisReitbauer for the initial feedback.

FTR https://github.com/keptn/enhancement-proposals/pull/86


Fixed boundary chars at fragment's ends

This allows scrolling to fragments which have boundary characters at the beginning or end of the fragment.

@googlebot I consent.


Showing from scratch let button inactive

Scenario:

cc.run() with doc sample config. "accept all"/"decline all" buttons work fine + "let me choose" works fine, probably because cc.run() called _addDataButtonListeners()
reload the page. The popup doesn't show up because cookies were accepted.

Calling cc.show(0, false)  doesn't show the popup
Calling cc.show(0, true)  show the popup,  "accept all"/"decline all" buttons work fine but "let me choose" does nothing

Could this be because _addDataButtonListeners() is not called in the case of show() but only in case of .run or language change ?

Should be fixed in v2.8.9.

Fixed indeed 2.8.9, thanks!


neovide isn't executable in the macos.app bundle

Download app and unzip. Then:
$ open ~/Downloads/Neovide\ 2.app/
LSOpenURLsWithRole() failed with error -10810 for the file /Users/oliverdain/Downloads/Neovide 2.app.

To fix mark the neovide executable as executable:
$ chmod a+x ~/Downloads/Neovide\ 2.app/Contents/MacOS/neovide
$ open ~/Downloads/Neovide\ 2.app

success!

Closing in favor of https://github.com/Kethku/neovide/issues/294


Night mode color contrast fix

Description (required)
Fixes #5301
What changes did you make and why?
Tests performed (required)
Fixes app level dark mode text color contrast.
Tested {prodDebug} on {Pixel 5 emulator} with API level {api 30}.
Screenshots (for UI changes only)

Need help? See https://support.google.com/android/answer/9075928

Note: Please ensure that you have read CONTRIBUTING.md if this is your first pull request.

@sivaraam this fixes the app level dark mode settings, please review when you get time. Thanks!


Autoloader

Hello. Can I somehow run it on shared hosting? Is there for example .htaccess and autoloader?

Can I somehow run it on shared hosting?

Yes

Is there for example .htaccess and autoloader?

Here: https://www.slimframework.com/docs/v4/start/web-servers.html#apache-configuration

Can I somehow run it on shared hosting?

Yes

Is there for example .htaccess and autoloader?

Here: https://www.slimframework.com/docs/v4/start/web-servers.html#apache-configuration

Closing as resolved

Closing as resolved


Skip cleanup or sleep if files in tmp/ exist

echo 10 >  tmp/after_each_sleep
touch tmp/skip_cleanup
make test-acceptance

We should add the repo tmp dir or the two files to to .gitignore, besides that lgtm.

It's already in there.


Update the screenshot given under running the playground application (without PKCE) to match with current version

Suggested Labels
Type/Docs
Priority/High
Affected/5.5.0-Alpha3
Description
Refer to doc link [1]. In step 2 Under Running the application (without PKCE) please update the given screenshot as [2] to match with the current version.
[1]. https://docs.wso2.com/display/IS550/Try+Authorization+Code+Grant
[2]. playground-new.png

Duplicates
https://github.com/wso2/product-is/issues/2973


[CLI] Import wallet private key

Description
The Sui CLI keytool import command now accepts either a MNEMONIC or a private key (from the Sui Wallet). The user exports a private key associated with an account in the Wallet, and uses that private key in the keytool import command in the CLI to import the account in the CLI.
Test Plan
There is a new function to test the new private key code (test_private_keys_ed25519)

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[x] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Before landing this PR, please update the "Release Notes" section in the summary.

Searched through a bunch of PRs to learn what this release notes should contain - what kind of information should go in here? (If I could find a previous PR with this section filled, I'd probably figure it out. Will keep searching)

Thanks @amnn and @joyqvq for feedback!


Don't return TextEdits if formatting changed nothing

This prevents editors marking open files as changed once the format command was issued but nothing actually changed on document formatting.

Seems like something editors should do as well, but can't hurt I guess.

Seems like something editors should do as well, but can't hurt I guess.

Probably. However I checked with some other language severs and none returns a text edit event if nothing changed.


[IRBuilder] Allow implicit allocate for DeclBuffer

Will merge it in after rebased

seems it's already merged by @cyx-6 in his impl


SignIn Post Request Completed

Also if a PR is still under development then just put  at the start of the PR name to avoid merging of incomplete PRs.


Overlapping textbox and Reset/Submit button for mobile view

In mobile view, textbox and buttons are getting overlapped.

I am interested to fix it.

Hello , I want to contribute on this issue.

@Gauravkain this issue is already assigned to @anjali481 you can look for some other issues, thanks

If someone is not working. Can I work on this issue.

If someone is not working. Can I work on this issue.

This issue is already assigned.


QUESTION Request and response body logs

Summary
Is it possible to view the logs for Request body sent over Kong API? I tried to view it in syslog but nothing recorded about body request and response.
Thanks,
Steps To Reproduce

Additional Details & Logs

Kong version ($ kong version)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (the output of a GET request to Kong's Admin port - see
https://docs.konghq.com/latest/admin-api/#retrieve-node-information)
Operating system

It is true that Kong (and the underlying NGINX) are very careful not to require the complete bodies of requests or responses in order to work.
This is because bodies they have unbounded in size, and some of them tend to be fairly large. A 2GB video response could clog a worker trying to write it to the logs. Big bodies also require writing files on the host server, which is antithetical to the asyncrhonous treatment of I/O.
That said, it is perfectly possible to get the complete request/response body on a plugin, and then log it. It is just not done by default in order to avoid the efficienty problems I mentioned above.
If you are interested in building such a plugin, you could use kong.request.get_raw_body to get the request body, and kong.service.response:get_raw_body to get the response body, and then log them with kong.log

see #5968


proposal: "dapr init" add  "--image-repository"

Describe the proposal
dapr init should have param of "--image-repository" to set image repository
He is very helpful to many people who cannot access the direct external network
Release Note

RELEASE NOTE: "dapr init" ADD  "--image-repository"

related to #640

/assign

Does this issue allow download of 3rd party images too? Like redis and zipkin. Do we still have those hardcoded?

@artursouza  Yes, the images are still hardcoded. If the private/custom registry mirrors the dockerhub exactly then this command would be able to download the 3rd party images too along with placement. The current solution has this limitation for now.
The fact that cli needs different images which can be under different org/path in private registries, the complete url to pull from could be different for each. We need more robust solution along with default hardcoded images(either from docker/ghcr). So that dapr init remain as simple as it is today without the need for multiple flags even for private registries.


Adding Hacktoberfest

Don't you want to add hacktoberfest in topic section?

@AliReza1083 No not this repo


Rc/v0.8.0

Features:

Baseline JIT (#29)

Refactors:

use bits operations to round up/down (#31)
Use Pin in Rust 1.33 to simplify code  (#32)
use [ e; n ] to init array (#33)
prefer NonNull to store raw pointer (#34)
Make JIT into a feature that can be enabled or disabled (#36)

v0.8.0 should already have been released.


Mark the GCR integration as deprecated in the admiral UI

https://github.com/Shippable/pm/issues/9158

this is done


Performance improvement for multiday enabled calendars with single day events

This is a performance improvement I've implemented on a local version of clndr.js
The check to see if a multi day event is on the current day takes significantly longer than it's single day counterpart. As the number of events increase you get a visible lag on the initial load and when changing months. This change checks if the event is more than one day before doing the multiday check and falls back to the single day check if not.
I've added a test for this that has 2 multiday events and 280 single day events. Before this change it took about .7 seconds to render, after it's about .3 second. The savings stay around 50% as you increase/decrease the number of single day events.

:+1: :+1: :+1:
This is rad, thanks for adding in a test!
If you haven't seen, we're working towards a new version of clndr. Keep an eye on that thread if you're interested in contributing!

Nice! I'm happy to help out where I can. I'll keep an eye out for updates


Update README.md

some grammar changes

THANK YOU! All changes merged 🥳
Please reach out to me on our Discord server if you would like me to send you some cool Appwrite stickers and magnets! Please note that shipping might take longer times because of COVID19.
As a way of saying thank you, we have just invited you to join the Appwrite organization on GitHub. You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.

Thanks a lot.


Skyline MSFragger DIA search and tutorial

fixed MSFragger search to set default data_type correctly depending on workflow type (PRM -> data_type=1, DDA -> data_type=0; and user can override the data_type in Additional Settings)
added TestDiaSearchStellarTutorial (Stellar GPF) and TestDiaSearchQeTutorial (QE wide window DIA, same dataset as EncyclopeDIA tutorial but with less aggressively subsetted files)
increased overlap tolerance in Edit Isolation Scheme dialog from 0.0001 to 0.001
added logging of commands used to launch MSFragger and Crux

Possible todos:

[ ] Add workflow option for non-DIA Umpire DIA that sets isolation scheme properly
[ ] Add workflow option for GPF (or checkbox when DIA workflow is selected) that sets data_type=2 automatically and imports files as multi-injection replicate runs
[ ] When above workflows are selected, limit search engine selection to MSFragger
[ ] Add tolerance when visualizing overlaps in DiaIsolationSchemeGraphForm
[ ] Add on-demand demultiplexing when above workflows are used, like EncyclopeDIA search mode?
[ ] Add a way to filter out alignment DIA scans from search and extraction: e.g. allow user to delete rows from isolation scheme, and if rows were deleted, automatically run files through msconvert's new "isolationWidth" filter (or perhaps another new one that requires fully specifying the absolute m/z isolation windows to include/exclude)

@brendanx67 I've added the todo list here in lieu of detailing these considerations in an issue. I will let Mike test what's already done while we discuss how best to accomplish the workflow options.

@brendanx67 This PR currently doesn't have any visible UI changes. I think it'd be a good one to cherry pick to 24.1 for the next patch. Then I can do a follow-up PR to change the workflow options to make DIA-Umpire optional. Sound good?

/squash

Sure.


[c++] Create a cmake rule for nanoarrow

Following on from #2362:

nanoarrow.c and nanoarrow.h are vendored
They contain a bit of sprintf with destination named out which is, from the compiler's point of view, possibly null
Different compilers handle this differently:

No problem in our main CI
Error in our wheels-build CI, fixed on #2398
Another build failure on Centos 8 with GCC 8.5: 2402

As found on #2402, adding -D TILEDBSOMA_ENABLE_WERROR=OFF at scripts/bld did not result in -Werror being avoided at nanoarrow.c

The bigger issue:

nanoarrow.c -- and only nanoarrow.c -- needs to be compiled without -Werror, and by default
This will take some cmake work

@nguyenv is our TileDB-SOMA cmake expert: yet I want to free up as much of her time as possible toward closing out #1632, upon which much depends.
I'm not a cmake expert by any means but am willing to attempt to factor out a cmake file for nanoarrow.c, and will ask for help as needed.

Note that there already is some support upstream:

https://github.com/apache/arrow-nanoarrow/tree/main/examples/cmake-scenarios (more featureful)
https://github.com/apache/arrow-nanoarrow/tree/main/examples/cmake-minimal (more minimal)

and there is some 'moar' cmake support by other projects in a recent PR / like to be in 0.5.0

https://github.com/apache/arrow-nanoarrow/issues/350

What may be the real desired outcome here is a solid FindNanoarrow_EP.cmake in libtiledbsoma/cmake/Modules/ where we could also pin the release we want (we are currently at 0.4.0).


Expose replace hook

Expose a hook on the MustacheReplacer and SanitizeExecutor that allows programmatic access to the value and path during the replacing process.
Please let me know if I can/should do a version bump as part of this PR. If you accept the change I'd love to start using it right away! 😸

@bitops Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Changes look safe enough to try.  tests are green.  good luck credhub

also sign the silly CLA it will make the bots happy.  happy robots don't rise up against us (usually)


Run a git diff on a POR repository

As one of the first steps of interpretting Git diffs, we need to actually run a git diff on a repository and and receive the output from that diff.
For this task, nothing needs to be done with the output, just get it so that we can do something with it later.

This works now, and all is well, closed by #102


Update BINDINGS.md

Raylib-J currently targets 3,7 and is still in active development

@CreedVI Thanks! Waiting for the 4.0 update! :D

Was just able to download it today! Excited to get started once I get everything pushed out!


rpt down

broken on ott

Check the website, it says that the current program isn't available online.


feat(ui): refactor informational popover

What type of PR is this? (check all applicable)

[x] Refactor
[ ] Feature
[x] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Description
feat(ui): refactor informational popover

Change translations to use arrays of paragraphs instead of a single paragraph.
Change component to accept a feature prop to identify the feature which the popover describes.
Add optional wrapperProps: passed to the wrapper element, allowing more flexibility when using the popover
Add optional popoverProps: passed to the <Popover /> component, allowing for overriding individual instances of the popover's props
Move definitions of features and popover settings to invokeai/frontend/web/src/common/components/IAIInformationalPopover/constants.ts

Add some type safety to the feature prop
Edit POPOVER_DATA to provide image, href, buttonLabel, and any popover props. The popover props are applied to all instances of the popover for the given feature. Note that the component prop popoverProps will override settings here.

Remove the popover's arrow. Because the popover is wrapping groups of components, sometimes the error ends up pointing to nothing, which looks kinda janky. I've just removed the arrow entirely, but feel free to add it back if you think it looks better.
Use a link variant button with external link icon to better communicate that clicking the button will open a new tab.
Default the link button label to "Learn More" (if a label is provided, that will be used instead)
Make default position top, but set manually set some to right - namely, anything with a dropdown. This prevents the popovers from obscuring or being obscured by the dropdowns.
Do a bit more restructuring of the Popover component itself, and how it is integrated with other components
More ref forwarding
Make the open delay 1s
Set the popovers to use lazy mounting (eg do not mount until the user opens the thing)
Update the verbiage for many popover items and add missing dynamic prompts stuff

QA Instructions, Screenshots, Recordings

Review the changes to the popover text

Moving the data for all the popovers to a single file (except of course the translations themselves) makes maintenance of this feature simpler. You should never need to edit any file other than the popover component or its constants.ts to make changes to the popover across the application.

Supersedes #4647


Themes service API cleanup

This is a follow-up PR to https://github.com/TryGhost/Ghost/pull/10818
There was a long waiting TODO item in themes service to make the API exposed to other modules cleaner, which this PR does. Next up on the list to cleanup in this service is probably the init method as it does too much and accesses SettingsCache directly, but that's a topic for another PR :smiley:
@allouis in the previous PR you had a proposal to rename settings into storage. Did a summary about why I think settings should stay as a name in this comment happy to continue the discussion here :wink:

Yo @gargol thanks for the ping and ref to last comment :)
Continuing that discussion about settings vs. storage

...[Using storage] gives the impression that the service might be implementing StorageAdapter

I get this concern and I thought about it too - personally I think it's fine, but it could deffo get confusing - thought this is a reason not to use storage, rather than a reason to use settings

The settings naming tries to follow the naming of the redirect's settings and the one in routing

As for this, both of the redirects and routing "settings" - though they are files, are in fact settings - both of them use YAML syntax which is specifically for configuration/settings.
The theme service on the other hand is primarily focused on the storing/extraction of zip files. IMO we should make that clear in the code - maybe storage is the wrong word, and given that we interface with the filesystem directly - maybe we could use something to reflect that, e.g. themeService.fs?
I don't think this is blocking - but whilst we're refactoring this stuff we may as well take a bit of time to nail down some of the superficial things like naming.

Having a step back and looking at what's exposed from /themes/index.js:

init
getActive
getApiVersion
activate
settings

get - gets JSON representation of the theme for API response
getZip - gets zip file from storage
setFromZip - saves zip to starage/loads it from storage/activates
destroy - removes theme files from storate, removes from theme list cache

middleware

After having this list in front of my eyes the storage naming starts to make more sense, but under the condition the settings.get is moved out to the /themes/index.js (renamed to getJSON?).
Think a sensible move would be having following /themes/index.js signature:

init
getActive
getApiVersion
activate
getJSON - moved from settings.get
storage - renamed from settings it's methods stay as is
middleware

Wdyt @allouis?

That list makes it much nicer to see :relaxed:
You're deffo right about moving the get method out - getJSON makes sense to me :+1:
The rest of what you've written all looks good to me :dancer:
The only other thing I would say, is that setFromZip probably shouldn't activate the theme, and that should happen in the upload controller (or wherever calls it) - just to keep each method doing one thing. Another option would be adding a top level method like setFromZipAndActivate - what do you think about that @gargol ?

@allouis agree that setFromZip does a bit too much, but would keep the changeset here to a minimum as there are loads of other comments around the themes module to reorganize responsibilities, so would rather do these changes when that comes on a table.
For now, the main goal - calling fewer things from "server" and making the API minimal is achieved :tada:
FYI, also had to rename Storage to ThemeStorage to avoid conflict in naming.
Think it's good to go, will merge after the release is done :rocket:

so would rather do these changes when that comes on a table.

Reasonable to me :dancer:
Woop woop :tada:


The argument "support_email" is required, but no definition was found

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Version
1.0.0-alpha.19
ZITADEL Version
No response
Describe the problem caused by this bug
│
│ Error: Missing required argument
│
│   on auth.tf line 18, in resource "zitadel_privacy_policy" "internal":
│   18: resource "zitadel_privacy_policy" "internal" {
│
│ The argument "support_email" is required, but no definition was found.
│

Why is this a required attribute? When changing this through the web UI, no support email address must be added.
To reproduce

Add the resource:resource "zitadel_privacy_policy" "internal" {
  org_id       = zitadel_org.internal.id
  tos_link     = "https://bla.ch/agb"
  privacy_link = "https://bla.ch/datenschutz"
  help_link    = "https://bla.ch/faq"
}

terraform validate

Screenshots
No response
Expected behavior
I would expect the required fields of the Zitadel Terraform provider to match the required fields of the web UI.
Relevant Configuration
No response
Additional Context
No response

I also think the field should be optional :+1:


Ensure accepted iterables are iterated eagerly and stored as lists

Is your feature request related to a problem? Please describe.
We accept iterables but sometimes store the iterables which makes little sense for ones that can only iterate once.
Describe the solution you'd like
Convert all iterables into lists eagerly when accepted.

Rather, just accept Sequence instead of Iterable everywhere. This is what other Python libraries seem to do.


Adjust sre-login to allow arguments  [WAS: Fixes sre-login to use OCM_USER var]

Currently, the sre-login script ignores if the $OCM_USER variable is set.  This change adjusts the sre-login script to accept multiple arguments:
usage: sre-login [-h] [-d] -c CLUSTER [-i OCM_IDENTITY]

required arguments:
  -c      the name or ID of the cluster to log into

optional arguments:
  -h      show this help message and exit
  -d      debug (bash xtrace)
  -i      OCM Identity to use for cluster login (usually your Kerberos ID)

The added help message and debug will assist new users or folks having issues tracking down where their login problems are occuring, and the OCM identiy argument will allow setting of any ID needed to login to the cluster.
OCM_IDENTITY can also be set via ENV VAR, so it doesn't need to be specified manually.
This also sets the stage to add other arguments if necessary, both for use in this script or as options passed to cluster-login.
This comes at the expense of needed to pass -c cluster rather than just the bare cluster name, but I think this is more clear usage, and less prone to issues.
I'm happy to adjust to accept the old usage as well, or to add a deprecation message, if that's desired.
Currently, the sre-login script ignores if the $OCM_USER variable is set.  This change modifies the script to pass the -e<EMAIL_ADDRESS>flag when calling cluster-login if the $OCM_USER variable is set.
Signed-off-by: Christopher Collins<EMAIL_ADDRESS>

@georgettica I took this a bit further and added real argument handling so any ID could be passed as an argument, and added a help message/usage message and debug.

Confirmed working with automatic/INITIAL_CLUSTER_LOGIN as well, using ENV VARS.

❤ I will test this today to verify

for some reason it doesn't want to work for me now (with your change or without)
if you can pull someone else to verify it works for them (macos user preferably) that would be super! as I know it's an issue you have and can be solved easily with this simple change

merged a ton of PR's and this had merge conflicts
waiting for someone to review this, so I can merge it

@clcollins is this still required?

With the backplane stuff, this is no longer necessary/useful.


Minor cmake cleanup

This PR makes some changes to the cmake C++ build:

When available, it will prefer the OpenSSL targets OpenSSL::OpenSSL and OpenSSL::Crypto over the older libraries variable.

It removes the use of a number of cmake macros that have basically no effect (instead they obscure what will happen), like TARGET_INCLUDE_DIRECTORIES_THRIFT, TARGET_LINK_LIBRARIES_THRIFT, LINK_AGAINST_THRIFT_LIBRARY and TARGET_LINK_LIBRARIES_THRIFT_AGAINST_THRIFT_LIBRARY.

[ ] Did you create an Apache Jira ticket?  (not required for trivial changes)

[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?

[x] Did you squash your changes to a single commit?  (not required, but preferred)

[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?

[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

Build errors on Travis are unrelated. Merging.


TypeError: fetch is not a function

This issue may occur if you are using this library with nodejs. If you are getting this error, it is because you don't have fetch api installed.
You can install undici or node-fetch v2.x and youtube-sr automatically finds and uses one. If you have node-fetch installed and youtube-sr is still throwing this error, it could be due to node-fetch version 3 being installed on your project. Run npm install<EMAIL_ADDRESS>for example to fix this issue.

NOTE: undici is given priority over node-fetch.

discussions may go on...


pattern: simple overview

Wonil-Suh1 created the following on Aug 06:
User story
As a user of IBM.com, I need a way to get a high level overview of the content/product presented on the page that I am looking at, and understand the value for my goal.
Example from STUF design exploration

In the above example, the link should not be there.

Original issue: https://github.ibm.com/webstandards/digital-design/issues/1384

Closing as the pattern strategy has been changed with the new ontology.


Error in seq.default(rawTime[1], rawTime[length(rawTime)], step) : 'to' must be of length 1

Describe the bug
My GGIR script will not progress past Part 1 as the program seems to think that there is a problem with the timestamp data. It goes up by 1 second each row, is in the correct format and that format is specified inside my params_rawdata list. I also created a test .csv file with the same timestamp measurements but random accelerometer data to test the issue. I underwent multiple debugging options including testing timestamp parsing outside of GGIR as well as debugging seq.default outside of GGIR and every time the data showed an increase by 1 second and the functions worked with no isssue. There must be a problem with either my code or the GGIR shell function.
To Reproduce
Attached is the test .csv file and my GGIR script. It's my first time using GGIR so the code might look sloppy but all of the parameters should be correct. Run the test file through the code and you'll get the error outlined in the title: Error in seq.default(rawTime[1], rawTime[length(rawTime)], step) : 'to' must be of length 1

Sensor brand: GENEActiv
Data format: .csv
Approximate recording duration 7'days
Are you using a sleep diary to guide the sleep detection: No
Copy of R command used: GGIR()
Have you tried processing your data based on GGIR's default argument values? Does the issue you report still appear? YES

R Code
dataFolder <- "/Users/georgeannas/Desktop/test"
outputFolder <- "/Users/georgeannas/Desktop/output"
# Load GGIR library if (!requireNamespace("GGIR", quietly = TRUE)) {   install.packages("GGIR") } library(GGIR)
# Check if output folder exists; if not, create it if (!dir.exists(outputFolder)) {   dir.create(outputFolder, recursive = TRUE) }
# Run GGIR GGIR(   datadir = dataFolder,            # Input data folder   outputdir = outputFolder,        # Output folder   mode = 1:5,                      # Run all five GGIR modes   overwrite = TRUE,                # Overwrite existing output   do.imp = TRUE,                   # Imputation of non-wear time   data_Format = 'csv',             # Specifies type of file   chunksize = 1,                   # Number of days processed at a time   params_rawdata = list(     minimumFileSizeMB = 1,     rmc.firstrow.acc = 2,     rmc.firstrow.header = 1,     rmc.header.length = 1,     rmc.col.acc = c(2,3,4),     rmc.col.temp = 5,     rmc.col.time = 1,     rmc.unit.time = "character",     rmc.sf = 1,     sep = ",",     extEpochData_timeformat = "%Y-%m-%d %H:%M:%S",     desiredtz = "VUT",     deviceName = "GENEActiv"     ),
# Mode 1 (Pre-processing and calibration)   do.cal = TRUE,                   # Perform auto-calibration   do.enmo = TRUE,                  # Calculate ENMO (metric of acceleration)   do.anglex = TRUE,                # Calculate angle relative to gravity   windowsizes = c(1, 900, 3600),   # Indicates the short epoch, long epoch, and non-wear detection   printsummary = TRUE,             # Print summary to console   verbose = TRUE,
 # Mode 2 (Non-wear time and sleep detection)   do.part2 = TRUE,                 # Enable mode 2   strategy = 2,                    # Sleep detection strategy (default is 2)   hrs.del.start = 0,               # Ignore data before midnight   hrs.del.end = 0,                 # Ignore data after midnight
 # Mode 3 (Physical activity metrics)   do.part3 = TRUE,                 # Enable mode 3   threshold.lig = c(30, 100),      # Light activity thresholds (mg)   threshold.mod = c(100, 400),     # Moderate activity thresholds (mg)   threshold.vig = c(400, 1000),    # Vigorous activity thresholds (mg)
 # Mode 4 (Individual reports)   do.part4 = TRUE,                 # Enable mode 4   do.visual = TRUE,                # Generate sleep and activity plots
 # Mode 5 (Group summaries)   do.part5 = TRUE                  # Enable mode 5 )
# Print message when done cat("GGIR processing complete! Check the output folder for results.\n")
Expected behavior
I expect GGIR to work through all 6 parts and produce me summaries and visualisation of the activity and sleep data recorded from the GENEActiv.
Desktop (please complete the following information):

OS: macOS 15.2 Beta (24C5079e)
GGIR Version: 3.1.5

synthetic_data.csv

Here is the code publicly available so you can download and test for yourself!

A 1 second time resolution not raw data based on GGIR's definition.
GGIR is designed to process data with multiple values per second, typical use cases involve 30 and 100 Hertz.
Such a high resolution is critical for separating movement from orientation component.
Has your data been exported from some other software with an option to increase the sampling rate?

I see in the file paths George Annas, is this the same question as in https://groups.google.com/g/RpackageGGIR/c/2RTCUyby704/m/a9dCq4loCgAJ ??
Please try to avoid double posting. I am replying voluntarily. If you need my time urgently then just reach out with a paid consultancy request, I'd be happy to do that as I also need to pay my monthly bills.


Add resource limiting to the Wasmtime API.

This PR adds a ResourceLimiter trait to the Wasmtime API.
When used in conjunction with Store::new_with_limiter, this can be used to
monitor and prevent WebAssembly code from growing linear memories and tables.
This is particularly useful when hosts need to take into account host resource
usage to determine if WebAssembly code can consume more resources.
A simple Limiter is also included with these changes that will
simply limit the size of linear memories or tables for all instances created in
the store based on static values.

I'm marking this as a draft PR as I'd like feedback on this approach before I create a formal RFC for the API changes (I find it useful to discuss code rather than markdown).

So I've seen this CI failure and I think it's a GitHub actions things.  As far as I can tell, we're using ubuntu-latest for all of the Linux CI, but for some reason we're occasionally getting Ubuntu 20.04 instead of what should be the current latest, which is 18.04 (according to this).
Although the GitHub actions virtual environment for Ubuntu 20.04 says Clang 9 is installed, perhaps lldb-9 is not and thus the expected LLDB path set for the tests (/usr/bin/lldb-9) doesn't work.

So it appears that 20.04 is now ubuntu-latest by design, so we were caught in an upgrade to GitHub actions.

Only the last commit is new; the rest were from your last review prior to my rebasing this branch on main.


23 September error :     AttributeError: 'Config' object has no attribute 'jax_experimental_name_stack'

Hi
tried mono or mulit, same issue
Thank's
FH

Yeah, same problem.

This is an upgrade from google, the newest version of colab by default takes up jax 0.3.25 (https://colab.research.google.com/notebooks/relnotes.ipynb#scrollTo=iOWrh41ocOca) which has no support for 'jax_experimental_name_stack', I have solved this issue by downgrading colab jax to 0.3.17, you can do it by adding this command: !pip install --upgrade "jax[cuda]==0.3.17" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html and then restarting the runtime

I tried it, jax goes back to 0.3.17 but it comes with other problems. May I ask which line you put in this commands?


Invalidate the block on call completion

When calling a method on an Actor from a normal Thread, if you invoke a block after the Call has completed, you will encounter a deadlock.
This is due to the fact the sending Thread is no longer waiting for the block invocations.
gist
If you are calling from an Actor, this behaviour works.
gist
This change invalidates the block for both scenarios.
We can allow block invocations in the actor case by only invalidating if the sender is a normal Thread.

[ ] Specs
[ ] Remove extra FIXMEs
[ ] Consider async/future + blocks with respect to invalidation

@niamster do you feel like this could be part of the same thinking behind resolving #182?

This is woefully out of date. Closing.


Add xquad task

We implemented a new task, xquad, which is a multilingual extractive QA benchmark published in the referred paper. Since xquad contains the subtask xquad_es, we removed the xquad_es yaml from the spanish_bench.
We update the task README to include the xquad task.

thank you! LGTM if you could run the pre-commit!


test:add signer,verifer,provider,algorithm unit tests

Add more signer and verifier tests (test signer/verifier from file)
Update algorithm according to new spec
Add provider test

go test ./signature -cover -count=1
ok      github.com/notaryproject/notation-go/signature  6.153s  coverage: 87.4% of statements

Signed-off-by: zaihaoyin<EMAIL_ADDRESS>

do we need to increase coverage to >90% ?

do we need to increase coverage to >90% ?

https://github.com/notaryproject/notation-go/blob/main/.github/.codecov.yml
I think we only need an 80% of coverage now. We can discuss it when we merge cose branch into main.


Error: _wfopen ??? failed ← when call waifu2x outside exe folder

Tried to run any image through waifu2x, but get this error:
>waifu2x-ncnn-vulkan -i i.jpg -o o.png -n 2 -s 2
[0 GeForce GTX 1060 6GB]  queueC=2[8]  queueT=1[2]  memU=4294967295  memDL=7  memHV=9
[0 GeForce GTX 1060 6GB]  fp16p=1  fp16s=1  fp16a=0  int8s=1  int8a=1
_wfopen ??? failed

Does I need any external requirements?
CPU: Intel(R) Core(TM) i7-4820K CPU @ 3.70GHz
GPU: GeForce GTX 1060 6GB
NVIDIA Driver version: 436.02
OS: Windows 10.0.17134 x64
EDIT: with help of gnattu I figured out, what I have that error if call waifu2x outside of exe folder (this folder in PATH variable), this issue only when console in different catalog, than waifu2x-ncnn-vulkan.exe.

me too
waifu2x-ncnn-vulkan -i ./src -o ./dst -s 2
[0 GeForce GTX 1060 6GB]  queueC=2[8]  queueT=1[2]  memU=4294967295  memDL=7  memHV=9
[0 GeForce GTX 1060 6GB]  fp16p=1  fp16s=1  fp16a=0  int8s=1  int8a=1
_wfopen ??? failed
GPU: GeForce GTX 1060 6GB
NVIDIA Driver version: 436.30

I had a similar issue, but explicitly specifying the model directory with the -m flag fixed it. I'm getting garbled image outputs though.
My old issue:
fopen models-cunet/noise2_scale2.0x_model.param failed
fopen models-cunet/noise2_scale2.0x_model.bin failed
Segmentation fault

Tried adding -m, but issue don't change. Commands are:
>waifu2x-ncnn-vulkan -i i.jpg -o o.png -n 2 -s 2 -m models-cunet
>waifu2x-ncnn-vulkan -i i.jpg -o o.png -n 2 -s 2 -m models-upconv_7_anime_style_art_rgb

I encountered similar situation, and I fount that I have to cd to the path of the program to make it work under windows(Windows Server 2019 DC). And everything is fine under Linux.

Wow, that really fix issue, it work for me if I call command in same folder with exe.

Same here, but I'd prefer call the .exe in another path, hope author could fix that soon... @nihui

This still happens to me, on Linux.
fopen models-upconv_7_photo/noise1_scale2.0x_model.param failed
fopen models-upconv_7_photo/noise1_scale2.0x_model.bin failed
When I cd in the directory containing models, instead, everything works.
This issue does not concern cunet's bins and params, which can be opened from anywhere.
I am using release 20200606.

Can confirm this still happens in Windows on 20200606 as well.

try the latest artifact build in CI

I tried the artifact from https://github.com/nihui/waifu2x-ncnn-vulkan/runs/909102954 and it seems to be working!
Here's the windows-2019 build: https://github.com/nihui/waifu2x-ncnn-vulkan/suites/962691968/artifacts/12130511

Still an issue for me with latest master on Linux:
waifu2x-ncnn-vulkan -i '/home/alex/waifu2x/i' -o '/home/alex/waifu2x/o' -m models-upconv_7_photo
[0 GeForce GTX 1070]  queueC=2[8]  queueG=0[16]  queueT=1[2]
[0 GeForce GTX 1070]  bugsbn1=0  buglbia=0  bugcopc=0  bugihfa=0
[0 GeForce GTX 1070]  fp16p=1  fp16s=1  fp16a=0  int8s=1  int8a=1
fopen /usr/bin/models-upconv_7_photo/noise0_scale2.0x_model.param failed
fopen /usr/bin/models-upconv_7_photo/noise0_scale2.0x_model.bin failed
find_blob_index_by_name Input1 failed
find_blob_index_by_name Eltwise4 failed


Add "prev_ruby_yjit" config to compare yjit 3.3 with yjit dev

Not pretty, but functional:

This installs ruby 3.3.1 via ruby-build and adds it to the default list of configs that run the benchmarks.

Very happy to see this in action, and it's already useful to spot benchmarks where we might have regressions.
One thing I would like to change is to remove the bars at 1.0 and add a horizontal line at 1.0 instead. It will be more space-efficient that way. Can be another PR if you prefer.

Nitpick but, the green box with YJIT should say YJIT 3.4 or YJIT 3.4dev

I updated the legend:

I'd like to merge this as is to start getting the data.
I'll investigate redoing the graphs next.


fix(35/*) fix all navigation 404s

Resolves missing page errors from navigation links

Do we need any callouts for Sessions in these docs?

@levfishbluefish negative, insofar as they don't mention enforcing RBAC / starting config, Sessions are already set by the time we get here.


Consider using the stable compose BOM instead of the alpha one from Chris Banes

I noticed that in the latest update of the library there was a bump in the compose BOM and that's when I noticed that the alpha compose bom is used https://github.com/kiwicom/navigation-compose-typed/blob/61a53d63a50364fe37014dec4da5bdd0df6f7d1e/gradle/libs.versions.toml#L12
Is there some particular reason why that is the case? It'd be nice not to bring in the alpha dependencies by accident by using this library. I haven't encountered any specific problem in my case, but it makes me a bit reluctant to know that I'll always be bringing in all the alphas without realizing. What is your opinion about this?

It should not bring any alpha dependency to the project, as we depend only on stable navigation compose, which is not part of the bom. The lib also uses compose compiler, which is versioned explicitly. This is for demo, but I'll fix it anyway, thanks.

Ah I should've looked into where it's actually used to figure that this is the case. Sorry about that, and thanks for the reply! 🙏


Feature/salary calculator

New feature - Salary Calculator:
allows users to select any category they previously created and create salary calculation settings for it. the feature also adds edit pages for the settings and view pages for the calculations. The calculation will be done for the chosen month and will be based on all events that are labeled with the corresponding category. Multiple salary settings are allowed, but only 1 for each user and category combination.
Further changes implemented:
-moved contents of file "app.database.database" to "app.database" for easier access.
-moved function "app.database,database.get_db" to "app.dependencies.get_db".
-merged file "tests.utils" into "app.internal.utils" to prevent code duplication.
-fixed imports and type hints where necessary.

In order to pass the tests you need the files to be ok with flake8, you can choose flake8 as a linter, and it will show you where are your mistakes.

In order to pass the tests you need the files to be ok with flake8, you can choose flake8 as a linter, and it will show you where are your mistakes.

All linting errors, code conflicts and failing tests have been fixed and resolved.

All linting errors, code conflicts and failing tests have been fixed and resolved.

Codecov Report

Merging #156 (a8bf216) into develop (53c1b08) will increase coverage by 0.28%.
The diff coverage is 99.73%.

@@             Coverage Diff             @@
##           develop     #156      +/-   ##
===========================================
+ Coverage    98.42%   98.71%   +0.28%
===========================================
  Files           32       35       +3
  Lines         1145     1473     +328
===========================================
+ Hits          1127     1454     +327
- Misses          18       19       +1

Impacted Files
Coverage Δ

app/database/models.py
94.56% <97.50%> (+0.53%)
:arrow_up:

app/database/__init__.py
100.00% <100.00%> (ø)

app/dependencies.py
100.00% <100.00%> (ø)

app/internal/import_file.py
100.00% <100.00%> (ø)

app/internal/search.py
100.00% <100.00%> (ø)

app/internal/utils.py
100.00% <100.00%> (ø)

app/internal/weather_forecast.py
100.00% <100.00%> (ø)

app/main.py
97.22% <100.00%> (+0.07%)
:arrow_up:

app/routers/agenda.py
100.00% <100.00%> (ø)

app/routers/dayview.py
100.00% <100.00%> (ø)

... and 17 more

Continue to review full report at Codecov.

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

While trying to solve conflicts, I've seen there is a minor problem with test_events.py. Can you please take a look? :)

While trying to solve conflicts, I've seen there is a minor problem with test_events.py. Can you please take a look? :)

There was a missing import, now fixed.


Use general purpose SSD storage by default instead of magnetic for RDS instances

RDS suggests all new database instances be created with general purpose SSD storage for better performance and access to additional features like storage auto-scaling. CloudFormation's default is to use magnetic storage.
Use gp2 for the RDS Instance storage type.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

See discussion #102