Some arguments passed into the iex.intraday function aren't being used properly

Passing some args inside iex.intraday() does not actually get sent up as part of the query string in the request. Ex: Setting IEXOnly to False
intraday = iex.intraday(
    "twtr",
    IEXOnly=False,
    token=token,
    IEXWhenNull=False,
)

To Reproduce
Steps to reproduce the behavior:

Call the function iex.intraday() with the argument IEXOnly set to False
See that the returned results do not include market data fields such as marketOpen

Expected behavior
Should see the market data fields when setting IEXOnly to false.
Diagnosis
Inside the intraday function (pyEX/stocks/intraday.py) the params are conditionally set like so...
if IEXOnly:
    params["chartIEXOnly"] = IEXOnly

So if IEXOnly is False, it won't actually enter the if statement to set chartIEXOnly to false. And the endpoint's default behaviour for not setting that param is True. So currently we can only get iex intraday data only (no market data). This issue might applies to other fields (maybe across the project too)

the default behavior if nothing is sent is the same as sending false, you can verify directly, these should both include market fields:
https://cloud.iexapis.com/stable/stock/SPY/intraday-prices?chartIEXOnly=false&token=pk_
https://cloud.iexapis.com/stable/stock/SPY/intraday-prices?token=pk_
whereas this one should give you the reduced set of fields.
https://cloud.iexapis.com/stable/stock/SPY/intraday-prices?chartIEXOnly=true&token=pk_
I see the same thing from the python code, are you seeing something different?

Hey sorry, forgot about the issue. I just tested this again and everything seems fine. Works exactly like you mentioned, chartIEXOnly behaves as false when not present in the query string.
Not exactly sure how I was running into the issue when I first reported the bug... I just recently learned that I need to get authorized by UTP to see 15min delayed Nasdaq stocks.  My guess is that I was testing this during trading hours with some Nasdaq stocks. And since I have don't have access to see the market data, I didn't see those fields? But I do recall seeing those market fields in other endpoints as null though.
Anyways closing this now since I can't even reproduce the original issue.
Thanks for clarifying 🙏


Set up GPU-enabled CI

We need to be able to run tests on actual GPUs on all platforms as a part of the continuous integration workflow. The tests includes:

warden reftests and benchmarks
portions of the Vulkan CTS
wgpu-rs compute tests

This task is about setting the infrastructure in place that we'd be using for gfx-rs, gfx-portability, and wgpu-rs.

Hi @kvark  I am the creator of Cirun.io, "GPU" and "CI" caught my eye.
FWIW I'll share my two cents. I created a service for problems like these, which is basically running custom machines (including GPUs) in GitHub Actions: https://cirun.io/
It is used in multiple open source projects needing GPU support like the following:
https://github.com/pystatgen/sgkit/
https://github.com/qutip/qutip-cupy
It is fairly simple to setup, all you need is a cloud account (AWS or GCP) and a simple yaml file describing what kind of machines you need and Cirun will spin up ephemeral machines on your cloud for GitHub Actions to run. It's native to GitHub ecosystem, which mean you can see logs/trigger in the Github's interface itself, just like any Github Action run.
Also, note that Cirun is free for Open source projects. (You only pay to your cloud provider for machine usage)


CLI v1.1.0+ doesn't read tsconfig.json when --project is not specified

Current behavior
Initialize my repository for reproduce:
$ git clone https://github.com/heguro/20241216-tsr-test
$ cd 20241216-tsr-test
$ npm i

When running tsr without --project: Incorrect output, doesn't read tsconfig.json
$ npx tsr 'main\.ts$'
tsconfig using default options
Project has 3 files. Found 1 entrypoint file
file   src/app/app.ts
file   src/fn/util.ts
✖ delete 2 files

When running tsr with --project: Correct output
$ npx tsr -p tsconfig.json 'main\.ts$'
tsconfig tsconfig.json
Project has 3 files. Found 1 entrypoint file
export src/fn/util.ts:4:0     'unusedFunc'
✖ remove 1 export

Root Cause

In the new CLI implementation released in v1.1.0, --project's default value is '', not undefined or 'tsconfig.json'.

https://github.com/line/tsr/blob/83a7f3bb9c6e7ec9720dad00bca0d0e1024d1349/lib/cli.ts#L8-L15

While 'tsconfig.json' is set as the default value for the configFile parameter in the tsr function, it is not used when '' is passed.

https://github.com/line/tsr/blob/83a7f3bb9c6e7ec9720dad00bca0d0e1024d1349/lib/cli.ts#L104
https://github.com/line/tsr/blob/83a7f3bb9c6e7ec9720dad00bca0d0e1024d1349/lib/tsr.ts#L41

This generates an incorrect configPath, causing ts.readConfigFile() to fail.

@heguro thank you very much for the detailed report! I've fixed this issue in https://github.com/line/tsr/releases/tag/v1.3.1

added test cases to prevent future degrade in https://github.com/line/tsr/pull/107


Make Tensorflow lite android project using tflite of ssd_mobilenet_v1_fpn_shared_box_predictor_640x640_coco14_sync_2018_07_03 model

Hello, everyone.
Recently I try to make android project of tensorflow lite object detection.
I have successfully converted to tflite file from http://download.tensorflow.org/models/object_detection/ssd_mobilenet_v1_fpn_shared_box_predictor_640x640_coco14_sync_2018_07_03.tar.gz, which uses 640640 input image size.
But I have memory overflow error on running my project on the android mobile.
In my previous test, I can run 300300, 224224 tflite model on my android mobile.
I use libtensorflowlite_gpu_jni.so and libtensorflowlite_jni.so distributed.
Could anyone tell me the maximum value of tensorflow lite input image size?
If we can run 640640 tflite model, what should I fix the android code?
Thanks in advance.

My guess is that on your mobile device, you don't have enough memory to run this model.
We don't have a formula to calculate the runtime memory usage required for each model, but it should be easy to find it out - e.g. checking the ram usage of a process?

@Wolf20199
Looks like issue is addressed.I am closing this issue. Please, feel free to reopen if the issue still persists.Thanks!


Compiler API is not reporting all the errors .

Consider the following code
// Dummy file for testing purpose
public class Demo{

      public static void main(String[] args)
      {
           System.out.println("hello world")
           int array = new int[10];
      }
}

Compiler API first reports only semicolon in first try. After than it will handle the array [] missing .

like javac , java compiler API first reports all first level errors then it reports all other errors . @tauseefansari @Afif79


[Discussion] Runtime exposes database client based on config

Currently the two db layers, runtime-knex and runtime-mongo are separate packages. This causes a problem in templates which have runtime-knex by default.
Idea
We can pass in the db name to runtime which will dynamically import the needed package and export it. This is being done while executing plugins in code generation. In practice this will be run only once to initialize the variables. And templates won't have knex hardcoded anymore
Thoughts @wtrocki @craicoverflow ?

Idea was that there is no problem as we can tell people to switch manually. Generally we would want to avoid importing all database packages in production build. Switch should happen at graphback config time - it can be both by dropping preconfigured code that can be connected manually by user. I think this issue could be follow up for graphback config issue..We might ask for language and drop template with some code

We can have some micro templates created by config that will connect to datasource

Micro templates seems like a good idea for now.

Since we have great docs for the datasources we should print link to the docs in the final message of the config


Give failing tests on Travis a second chance

Since we continue to see spurious failures on Travis for reasons which may be out of our control, it could help to give them a retry.

It's sad this is necessary but I guess we should do it.


added Twitter Handler to Internet Provider

All that for an @... I think this doesn't bring much value. Faker users will prefer concatenating the '@' themselves instead of learning another formatter.
Thanks for your suggestion anyway.

Same. This would have been really useful.

@GrahamCampbell Not sure I get your point. You think a twitter handler formatter would be useful or not?

I think it would have been useful to have, yeh.

I thought that it might be useful. I needed and I think that hardcode an @ was just fine, but a specific handler will be better.
j.
On Sun, Jan 4, 2015 at 9:06 PM, Graham Campbell<EMAIL_ADDRESS>wrote:

I think it would have been useful to have, yeh.
Reply to this email directly or view it on GitHub:
https://github.com/fzaninotto/Faker/pull/480#issuecomment-68656068

But then we need a formatter for Google + which adds a "+", and a formatter for Facebook which just returns username, and I don't talk about snapchat or Pinterest... It doesn't make sense in my opinion.

aha, you have a point.
perhaps separating the Internet Handlers and creating a Social Media
Handlers?
j.
On Tue, Jan 6, 2015 at 12:29 PM, Francois Zaninotto <
<EMAIL_ADDRESS>wrote:

But then we need a formatter for Google + which adds a "+", and a
formatter for Facebook which just returns username, and I don't talk
about snapchat or Pinterest... It doesn't make sense in my opinion.
—
Reply to this email directly or view it on GitHub
https://github.com/fzaninotto/Faker/pull/480#issuecomment-68879641.


Conditional minification

A simple way to conditionally apply minification (e.g. if webpack is invoked with the -p flag) would be great :)

Cool idea I'll add it to my list for the next major release right now this would break the backwards compatibility so it will take a while :)


Eventhandler of different packages are not informed

If event handler in more packages exist, only 1 event handler is informed.
It's not clear if that happens only if event handler for the same event are registered, or if just classes of more packages are registered

axoniq is already contacted for help. A demo where the situation can  be reproduced


Update rules for ruff

What does this PR do?

This PR adds an option to auto-format list/dict comprehension with ruff as it is now done in Transformers: https://github.com/huggingface/transformers/pull/21694 and https://github.com/huggingface/transformers/commit/ee6e71e29cbe8c6df45dd42993c345001788ed9d.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Hexed crashes during launch

Mod Name
Hexed
Mod Homepage
https://modrinth.com/mod/hexed
Minecraft version
1.20.1
Describe the bug
The Hexed fabric mod crashes during launch with Sinytra Connector.
Steps to reproduce

Install these versions of the following mods in a fresh Forge 1.20.1 profile:

Connector-1.0.0-beta.42+1.20.1.jar (https://modrinth.com/mod/connector/versions)
fabric-api-0.91.0+1.20.1.jar (https://modrinth.com/mod/fabric-api/versions)
fabric-api-0.92.1+1.11.7+1.20.1.jar (https://modrinth.com/mod/forgified-fabric-api/versions)
fabric-language-kotlin-1.10.19+kotlin.1.9.23.jar (https://modrinth.com/mod/fabric-language-kotlin/versions)
hexed-0.8.1H.jar (https://modrinth.com/mod/hexed/versions)

Run the profile and obtain the following error logs:

Latest.log (https://mclo.gs/8kRgAMW)
Debug.log (https://mclo.gs/mfDtLky)

Logs
https://mclo.gs/8kRgAMW https://mclo.gs/mfDtLky
Additional context
No response

У меня он нормально работает

I'm hexed, it works.

Should be noted that this error apparently only occurs in forge versions after ~47.1.28ish.


Make --version great again!

I seem to recall that once futhark --version showed a hash for the git commit that was used to build that version.
I was just running an example on my own machine and the GPUs in the clusters, and found different results. I wanted to verify we where using the same version of the compiler, but all the information I can find is that we're using version 0.1 :cry:

You are very welcome to add it back.  The problem I had was that my
Setup.hs hack broke 'cabal sdist'.  But maybe we don't care about that.
(It's 'stack sdist' anyway now.)
--
\  Troels
/\ Henriksen

Fixed in 8791fd45409de0219097fea46a14f620694cffae.


server

hi, do i have to do anything for my server if i want to use this modpack, or i only have to install the modpack on my pc

https://github.com/Fabulously-Optimized/fabulously-optimized/wiki/Server-setup


empty VOLUME is very restricted: root:root 700

Is there really no way to set ownership and permissions of a VOLUME?
~ $ docker build -t test -
FROM ubuntu
RUN mkdir /data ; chmod 777 /data
VOLUME ["/data"]
RUN chmod 755 /data

Uploading context 2048 bytes
Step 1 : FROM ubuntu
 ---> 8dbd9e392a96
Step 2 : RUN mkdir /data ; chmod 777 /data
 ---> Using cache
 ---> 5e2d567c32ef
Step 3 : VOLUME ["/data"]
 ---> Using cache
 ---> d2832c4afd2c
Step 4 : RUN chmod 755 /data
 ---> Running in 34b05587ce60
 ---> 7c4d94ba6eed
Successfully built 7c4d94ba6eed
~ $ docker run -t -i test ls -la /data
total 8
drwx------  2 root root 4096 Nov 30 09:25 .
drwxr-xr-x 37 root root 4096 Nov 30 09:25 ..

I'm making a container which will run something that does not want to run as root, so 700 is particularly frustrating.
somewhat related to #1742 and #2088

So this was definitely fixed way back, I cannot reproduce.
Closing, but if you can provide a way to repro this (prefferably with a Dockerfile + run command), please ping here!
FROM busybox
RUN mkdir -p /data && chmod 777 /data && chown 1000:1000 /data
VOLUME /data
CMD ls -lh /

docker run -it --rm test
total 52
drwxrwxr-x    2 root     root        4.0K Feb 20 19:43 bin
drwxrwxrwx    2 default  default     4.0K Feb 20 19:43 data
drwxr-xr-x    5 root     root         380 Feb 20 19:43 dev
drwxr-xr-x    1 root     root        4.0K Feb 20 19:43 etc
drwxrwxr-x    4 root     root        4.0K Feb 20 19:43 home
drwxrwxr-x    2 root     root        4.0K Feb 20 19:43 lib
lrwxrwxrwx    1 root     root           3 May 22  2014 lib64 -> lib
lrwxrwxrwx    1 root     root          11 May 22  2014 linuxrc -> bin/busybox
drwxrwxr-x    2 root     root        4.0K Feb 27  2014 media
drwxrwxr-x    2 root     root        4.0K Feb 27  2014 mnt
drwxrwxr-x    2 root     root        4.0K Feb 27  2014 opt
dr-xr-xr-x  125 root     root           0 Feb 20 19:43 proc
drwx------    2 root     root        4.0K Feb 20 19:43 root
lrwxrwxrwx    1 root     root           3 Feb 27  2014 run -> tmp
drwxr-xr-x    2 root     root        4.0K Feb 20 19:43 sbin
dr-xr-xr-x   13 root     root           0 Feb 20 19:43 sys
drwxrwxrwt    3 root     root        4.0K Feb 20 19:43 tmp
drwxrwxr-x    6 root     root        4.0K Feb 20 19:43 usr
drwxrwxr-x    4 root     root        4.0K Feb 20 19:43 var


@type stdout not working if redirect fluentd logs to a file using -o "filepath"

Describe the bug
Hi,
I have a use case to redirect fluentd's own logs to a file which is done by the below option:
fluentd -o /path/to/log_file
I have another requirement where I need to read logs from a different file and send it to stdout. I have used below config to achieve this by having a source section to read from the file and then a match section with stdout type.
<source> @type tail path /tmp/*.log pos_file /tmp/td-agent.pos read_from_head true tag test <parse> @type json </parse> </source> <match test> @type stdout <format> @type json </format> </match>
However, I observe that even logs from this file path is also going to the redirected log file only and not coming to stdout.
Please suggest some way to achieve the above explained use case.
Thanks in advance.
To Reproduce
Redirect fluentd logs to a file during startup
/usr/sbin/fluentd -o logfile_path
and provide below configuration to fluentd
<source> @type tail path /tmp/*.log pos_file /tmp/td-agent.pos read_from_head true tag test <parse> @type json </parse> </source> <match test> @type stdout <format> @type json </format> </match>
Expected behavior
Logs read from source file should be outputted to stdout and should not go to redirected log file path.
Your Environment
- Fluentd version:1.16.2

Your Configuration
<source\> @type tail path /tmp/*.log pos_file /tmp/td-agent.pos read_from_head true tag test \<parse\> @type json \</parse\> \</source\> \<match test\> @type stdout \<format\> @type json \</format\> \</match\>

Your Error Log
There is no error. Logs read from source file are not coming on stdout and getting redirected to file

Additional context
No response

@type stdout is affected by -o option.
So, there is no wonder that matched event will be logged into redirected to file.

@type stdout is affected by -o option. So, there is no wonder that matched event will be logged into redirected to file.

So, this is the current specification of Fluentd.


copy PIC test to 1.3 and 1.4

Copy #86 changes to 1.3 and 1.4. @feynmanliang

We need to modify TestRunner as well

LGTM

Merged.


Shadow rendering at wrong distance on AMD cards

Shadows are rendering at the wrong location on AMD cards.  They render at half the distance between occluder and light source.  See images in forum post:  https://forum.defold.com/t/2d-lights-and-shadows-sample/68970/21?u=pabaucom
This was confirmed an issue on an RX 6800 on driver 23.7.1 and the previous version with all driver settings reset to factory defaults.  nVidia cards rendered as expected.
It appears I have fixed it on commit d9c6faa of a fork I was testing on.  I would like some feedback on performance impact of this and why it might be necessary on my AMD chip.  It looks like step and coord have to be slightly different and not be offset by a whole number.
For reference, commit e451d6e on a test branch also fixed AMD, but then doubled the shadow distance on nVidia.  I speculate the AMD driver is doing some optimization leading to this edge case with 1D shadow maps.  This is my first experience with shader programming, so I am guessing.

I would like some feedback on performance impact of this and why it might be necessary on my AMD chip.

Let's see if we can get some input from @Jhonnyg

In the PR, I multiplied by 1.00001 and it fixes the issue on AMD while nVidia still renders properly.  Adding a small value to coord rather than multiplying by 1+eps does not fix it.  My comment in code may be more confusing than accurate and may need to be changed.
I am probably assuming a lot about what AMD is doing to optimize that loop versus what the nVidia driver does.  I made those assumptions based on what I was seeing playing with the values of distance, step, and y and running on AMD vs nVidia.
I see the same behavior on a Windows and HTML5 debug or release build.  Testing on two computers with latest nVidia and AMD drivers.  It is consistent across several previous versions of Defold.  I have not tested on my Intel iGPU.


Need links to bios/portolios/specific projects the team wants to showcase

Since we are still building out our brigade portfolio, we would like to feature projects by individual brigade members so that we can showcase our skills for potential partners.
Could you please submit:
Your name as you'd like it to appear on the site, plus the URL you'd like your name to link to
A one sentence bio.
A link to an online representation of one project you'd like to showcase.
An image to represent that project

Kim Schaefer
www.kim-schaefer.com
Kim Schaefer is a UX Designer and civic technologist who lives for a tough creative challenge and the chance to interact with humans.
https://www.kim-schaefer.com/sparechange/
image


Add Joel as Stakeholder

Joel would like to demo the project as a example for the department. This means his input as a liaison to the department faculty and university administration is invaluable to the demo's success.

I submitted an email to Joel indicating we have identified him as stakeholder becuase he is a liaison for the administration and his input will help drive the success of delivering a strong demo.

Joel has opted to meet with the group on 2-12-16 at 10:00am. @aaroncameron21  and @benjic are going to go.


Update description of the group policy setting

In all the ADMX/ADML files I have seen, the description of the policy setting is "Allow retrieving the cloud kerberos ticket during the logon"

@Heidilohr

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Heidilohr @femila - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

@Heidilohr Heidi Lohr FTE @femila Femila Anilkumar FTE - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

It looks like this PR has a conflict. I'll need to import the changes into my own branch and merge them myself.

@Heidilohr - when you've completed your pull and merge, can you close this pr by putting #please-close in a comment?

@dknappettmsft Please review this again. You might have an older version of the ADMX files. Currently, the setting name is as indicated in the PR I created.
This is a screenshot from Windows 10 22H2, but it changed I believe even in 21H2:

Hi @SvenAelterman, thanks for the update. I think it's actually the other way around - Windows 10 21H2 and 22H2, plus Windows 11 21H2 have your version, but Windows 11 22H2 has my version. I've added both to the article, with yours coming first, as it'll be most common. The change should be visible later today.


How to get reply-to header?

Hi all. Thank you for this great library
Is there a way to get the reply-to header content?
Thank you

This library does not as much interact with the underlying email itself (and thus not with it's headers), but how that email is exposed via Gmail itself.
There might be a solution to your problem, but I don't know it from the top of my head. I suggest reading the documentation and exploring the API to see if you find anything which works for you.

Hi @josteink ,
Any update on this? My use case is to filter out the no-reply emails and read the reply-to header instead.

Like I said, this library is not really for those things, and works more in the UI layer of Gmail.
In theory you could extract the email MIME message itself, and the run that through a MIME-parser, to get the headers, but there's no direct way using GmailJS to get those header-values.


OnAfterPostGLAndCustomer symbol isn't working

There's an event OnAfterPostGLAndCustomer  in CU80 that isn't available in the compiler after downloading the symbols.
The interesting thing is that OnBeforePostGLAndCustomer is working though.

My bad :) Thanks for clarifying @AlexanderYakunin !

@StanislawStempin yes. I gave that another try now to confirm, but it persists. Strange huh?
@AlexanderYakunin it is NA.

Hello,
I found the problem. I had to generatesymbolreference from finsql


Update TaskInfo definition to match server response and move some mag…

Pull Request
What does this PR do?
Update TaskInfo definition to match server response and move some magic strings to enum

bors merge


fix: #799 - now handling both svg and png files (for "Label" icons)

New files:

ab-agriculture-biologique.74x90.svg: - optimized cache to precision O (cf. https://jakearchibald.github.io/svgomg/)
abstract_async_asset.dart: Widget with async load of asset file.
abstract_cache.dart: Widget that displays an image from network (and cache while waiting).
bio-europeen.135x90.png: cache
fairtrade-international.77x90.png: cache
null_cache.dart: Empty image cache: the url was null, there is not much we can display.
raster_async_asset.dart: Widget with async load of raster asset file (png, jpeg).
raster_cache.dart: Widget that displays a png/jpeg from network (and cache while waiting).

Impacted files:

knowledge_panel_title_card.dart: now handling both svg and png files (for "Label" icons)
svg_async_asset.dart: now extending new abstract class AbstractAsyncAsset
svg_cache.dart: now extending new abstract class AbstractCache

The issues we were seeing on some labels were related to not having a PNG caching system ?
We should be able to turn PNG white quite safely if we can't for SVGs ?
We shouldn't have render issues from network SVGs ? but the local cache was too much optimized ?

Actually for this type of icons we were only displaying svg. That means, even if the icon was a png, we called a svg renderer: the result was obviously not good.
Now we check first, and display svg as svg (potentially with an additional color), and png/jpeg as png/jpeg.
No color parameter is applied on png/jpeg: it seems less relevant than for svg. And that fixes this issue.

Some high quality code here, looks good

@M123-dev Thank you, I was afraid it would look too much Java-style!


Native Tests

Run on creation of docker build, but do not run JVM tests.
Marked TODO in Dockerfile.

Is running with commit bc0440f
==================================================================================
  native.image.path was not set, making a guess for the correct path of native image
  guessed path: /code/target/dynamic-query-api-1.0.0-SNAPSHOT-runner
==================================================================================


[Feature]: Publish to open-vsx

Feature Description
Publishing the extension to open-vsx would make it available for users of VSCodium and other VS Code forks, which do not use the official Microsoft Marketplace.
https://github.com/eclipse/openvsx/wiki/Publishing-Extensions

Please publish this extension to the Open VSX marketplace.
Context
Unfortunately, as Microsoft prohibits usages of the Microsoft marketplace by any other products or redistribution of .vsix files from it, in order to use VS Code extensions in non-Microsoft products, we kindly ask that you take ownership of the VS Code extension namespace in Open VSX and publish this extension on Open VSX.
What is Open VSX? Why does it exist?
Open VSX is a vendor neutral alternative to the MS marketplace used by most other derivatives of VS Code like VSCodium, Gitpod, OpenVSCode, Theia-based IDEs, and so on.
You can read on about Open VSX at the Eclipse Foundation's Open VSX FAQ.
How can you publish to Open VSX?
The docs to publish an extension can be found here. This process is straightforward and shouldn't take too long. Essentially, you need an authentication token and to execute the ovsx publish command to publish your extension. There's also a doc explaining the whole process with an example GitHub Action workflow.


Remove actor id

Removes the actor ID because it isn't used anywhere except to check if an actor is the root. Also if riker is used on systems where usize is 32 bit or smaller, there is a chance that the actor id wraps around and multiple actors could end up with the same id.
The old is_root checks if the actor id is 0. In case of an overflow this could happen more than once.
The new is_root function uses the actor path, because it is already guaranteed that there can't be a second actor with the same path.

If I remember, this is used to differentiate between instances of the same actor at the same path. This occurs during supervision events such as Restart.  When an actor is restarted, it occupies the same path as the failed actor but is given a new ID. This assists in debugging and also allows users to be able to make decisions on versions of the actor.

Good grief, i really like to forget sometimes, that there are — relatively common! — platforms without 64 bit atomics

I just wrote a simple test:
use riker::actors::*;

#[derive(Debug)]
struct ActorIds {
    fail_on_start: bool,
    fail_on_recv: bool,
}

impl ActorFactoryArgs<(bool, bool)> for ActorIds {
    fn create_args((fail_on_start, fail_on_recv): (bool, bool)) -> Self {
        ActorIds {
            fail_on_start,
            fail_on_recv,
        }
    }
}

// implement the Actor trait
impl Actor for ActorIds {
    type Msg = i32;

    fn pre_start(&mut self, ctx: &Context<Self::Msg>) {
        println!("Started: {:?}", ctx.myself.uri());
    }

    fn post_start(&mut self, _ctx: &Context<Self::Msg>) {
        if self.fail_on_start {
            panic!("OMG");
        }
    }

    fn recv(&mut self, _ctx: &Context<Self::Msg>, _msg: Self::Msg, _sender: Sender) {
        if self.fail_on_recv {
            panic!("OMG2");
        }
    }
}

fn main() {
    let sys = ActorSystem::new().unwrap();

    let fail_on_start = true;
    let fail_on_recv  = true;

    let a = sys.actor_of_args::<ActorIds, _>("asd", (fail_on_start, fail_on_recv)).unwrap();
    a.tell(1, None);

    while sys.user_root().has_children() {
        std::thread::sleep(std::time::Duration::from_millis(50));
    }
}

If the actor panics in post_start, it is restarted with the same actor id as the actor before (103).
If the actor panics in recv, it isn't restarted at all.
I think it would be better to log the panic when an actor crashes/restarts. This should be enough to debug what happened.
I don't know how often it is needed to differentiate on the number of restarts an actor has made. If you just need to know if an actor has restarted, we could provide a on_restart callback on the Actor trait, so that a user can decide what the actor needs to do in case of a restart.
If there is a good use-case for the number, it would probably be better to keep a number per actor, so you can see which generation this actor is and not just if the current number is different than some number the actor had before.

If there is a good use-case for the number, it would probably be better to keep a number per actor, so you can see which generation this actor is and not just if the current number is different than some number the actor had before.

That makes sense to me and I like the term "generation" to describe it too. This is how Akka and other systems have done this.
The case for checking the generation would be be used by other actors when subscribing to ActorRestarted system event, or used in clustering, but that is best addressed separately in an RFC.
Thanks for clarifying! I'll merge it.


Issue with lengthy message text.

Not able to scroll down to see all the message if the message length is more. There is no option to scroll down or up to read all text.

Yes this is the bug i will try to resolve this issue as soon as possible


Support dynamic loading

Signed-off-by: Igor Chebykin<EMAIL_ADDRESS>Signed-off-by: Mircea Gherzan<EMAIL_ADDRESS>Signed-off-by: Midhun Kodiyath<EMAIL_ADDRESS>

QA testing for VPx use case is required.

Can a commit message be added to the patch to understand who is dynamically loading cmrt?  It looked like the application that is loading cmrt has the requirement for all the renaming to add the _RT (run time) suffix to most classes and structures.
Can you provide also the use case that triggered this change?  It will help understand the scope of testing needed to happen for the Hybrid-driver.
Thanks.

Okay.I will take care of the Indentation in cm_rt.h.Thanks.
This for DPD's use cases.We need to cover our regular tests (mv_decode and other regular tests) and make sure nothing has broken.
DPD already tested their use cases and it passed.

On checking this patch, I found that there was segmentation fault at the end of the test execution even though the file generated matched the md5 value.
I did not observe the seg fault  without the patch.
Error message:
Segmentation fault (core dumped)
Commands used for test:
$ mv_decoder_adv --vp9 -i<EMAIL_ADDRESS>-o<EMAIL_ADDRESS>--i420
$ mv_decoder_adv --vp9 -i ~/input/bitstreams/park_joy_50fps_w3840h2160.ivf  -o ~/output/park_joy_50fps_w3840h2160.yuv --i420

Okay..Thanks.
Looks like decoding is complete and seg fault is happening after that at the end.That is the reason why file generated matched the md5 value.
One strange thing observed is that without the patch also I saw a seg fault at the end using your samples on my setup.I will take a look and get back on this with a fix.

Okay let me know when we need Rams to retest.

Can you please retest the latest changes?
We need to cover our regular tests (mv_decoder_adv app and other regular tests) and make sure nothing has broken.Thanks..

i did a try and found more abstract classes are needed, for example CmSurface2D.
here is the build issue with out abstract class for CmSurface2D:
undefined reference to CmSurface2D::WriteSurface(unsigned char const*, CmEvent*, unsigned long)' undefined reference to CmSurface2D::GetIndex(SurfaceIndex*&)'
undefined reference to CmSurface2D::GetIndex(SurfaceIndex*&)' undefined reference to CmSurface2D::ReadSurface(unsigned char*, CmEvent*, unsigned long)'
collect2: error: ld returned 1 exit status

May I know what test you are running?
Scope of this patch is VPx test suite and DPDs use cases.This patch is very much limited to the DPDs requirement.They tested these changes with ~100% pass rate.Now we need to test vp9 decode use cases.I think Ram/Jayesh(QA) is going  cover vpx tests.

i just write a unit test myself, with the APIs exported in cm_rt.h, thanks.

I see.Thanks Guoyenjun.
I agree with you.We need to make CmSurface2D also abstract.I think it will helpful for future usecase and it's right thing to do.

i think there is another method to support dynamic loading. It doesn’t need to change current files inside cmrt, just add two files, one .h file for the c interface, the other .cpp file for the connection with cmrt c++ class.
The C interface looks like::
extern “C” INT CmDevice_LoadProgram(CmDevice* pCmDev,
void *pCommonISACode, const UINT size,
CmProgram * &pProgram, const char *options)
{
return pCmDev->LoadProgram(pCommonISACode, size, pPgogram, options);
}
And, yes, application has to use the new c functions if it want to dlopen/dlsym.
just for your reference

Validated this patch. Looks good to me. It can be merged.

We need to hold off on merge until we complete critical changes for Google.  We will merge next week.  Feel free to provide a drop to the folks needing it.  But I'm not ready to merge until we are in the clear on current Hybrid work.

lgtm +2


[Bug] [connector-v2] #!/bin/sh need modify to #!/bin/bash

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Such as in file start-seatunnel-flink-new-connector.sh
#!/bin/sh need modify to #!/bin/bash
Because in some Linux  Distributions,sh linked to dash.
SeaTunnel Version
2.1.3-SNAPSHOT
SeaTunnel Config
no

Running Command
Run FakeSourceToConsoleIT.java

Error Exception
22/09/03 18:28:04 ERROR FlinkContainer: /tmp/flink/seatunnel/bin/start-seatunnel-flink-new-connector.sh: 46: [: 2: unexpected operator

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

pls change Line Spearator to LF

pls change Line Spearator to LF

I see.

see PR #2633
you should fix it local


Remove unnecessary entry from set_destinations list

Thank you @aaronjamt! I have just seen your MR and will look at it soon :slightly_smiling_face:

Thanks for spotting this. LGTM :+1:


Hi, I have a question about the TFLOPs of the IFRNet. For a pair of 720*1280 images, the TMACs of IFRNet-L I calculated it's 0.817, which is same as the result of thop. 0.817TMACs approximately equals to 1.634TFLOPs, it's different with the 0.79TFLOPs in your paper.

maybe there's something wrong in my calculation？


Error during DWX_ZeroMQ_Server_v2.0.1_RC8.mq4 compilation

Hi! I faced with seven errors during compile's process of DWX_ZeroMQ_Server_v2.0.1_RC8.mq4 file in MT4. I believe that these errors are related to HTML comments, but I guess those are misleading errors! Is everybody faced with this problem too?
'' - single quote needed	DWX_ZeroMQ_Server_v2.0.1_RC8.mq4	158	10

'`' - unknown symbol	DWX_ZeroMQ_Server_v2.0.1_RC8.mq4	158	11

For the god sake just write this somewhere:
Please please do not download "DWX_ZeroMQ_Server_vX.Y.Z_RCx.mq4" file with right-click on it, or something else. Open it on a new tab and then copy the raw code on your preferred editor! After that, you can save that with the original name and in Metatrader4 right-click on it on the Expert Advisor and click on modify to modify it on Meta Editor program. That's all!


feat(meroxa): update spec

PR generated automatically from push-to-fig-autocomplete-action.

@raulb does this all look good?

@janelletavares There are some tweaks I need to make so it can actually go into folders and such. I'll do that shortly.

Looking into this, I'm confused about why we (I) needed to create this fork at all https://github.com/meroxa-machine/cli-autocomplete.

PR https://github.com/meroxa/cli-fig-autocomplete/pull/22


feat: add iPhone 11 models to DeviceDescriptors

Hi, I noticed the latest iPhone models were missing. I got the width, height, deviceScaleFactor from this site: https://yesviz.com/devices.php

@googlebot I signed it!


Deploy a CAP Business Application to SAP Cloud Platform : error adding SAP HANA as a dependancy

Tutorial URL: https://developers.sap.com/tutorials/cap-service-deploy.html
In step 1, when I run npm add @sap/hana-client --save, the following error comes up:

I cannot find any resources on how to resolve this. Here is the debug log:
2020-07-27T14_25_12_670Z-debug.log

Hi,
As I can see in the log file you are using<EMAIL_ADDRESS>As we recommend in the prerequisites in the first tutorial, please use a supported version of Node.js, which is 12.18.3 LTS. Also please refer to the linked troubleshooting guide in section Check the Node.js version which is linked in the prerequisites of the first tutorial.
As an improvement, I´ll add the link to the troubleshooting guide also to the second tutorial.
Hope that helps and best regards,
Iwona

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Release Google.Cloud.Deploy.V1 version 2.2.0

Changes in this release:
New features

Enable REST transport in C# (commit 496c8ab)

Created release for Google.Cloud.Deploy.V1-2.2.0


Maintained Order

Could you let me know if I'm reading this wrong?
So what do we want?
...
maintained order (older ID < newer ID)
...

I was expecting to see the Marshaled ids to still be in maintained order.
Sample Results
Consecutive IDs
  308035 -><PHONE_NUMBER>
  308036 ->  531536956
  308037 -><PHONE_NUMBER>
  308038 -><PHONE_NUMBER>

Thx!

I should have worded it better.
What I meant is that since this obfuscation is reversible and "cheap" you don't store obfuscated IDs, just the original ones: marshaling obfuscates, unmarshaling deobfuscates. Because of that you can simply keep using consecutive numbers (such as autoincrement field in db).
By comparison if obfuscation was one way, you used random IDs etc the order would be gone.


Declaration of ServiceContainer::load() should be compatible with Container::load($file)

I got this error on fresh install

Just stumbled upon this error myself. This has actually been corrected in the most recent release.
The problem is likely that you don't have the php bz2 extension configured on your server, which was been added as a dependency in v4.6. The result is that composer finds the latest compatible release (4.5.1) and installs that instead. This bug is still present there.
Installing the bz2 extension should solve your problems.

Same error. bz2 extension installed
PHP Warning:  Declaration of JonnyW\PhantomJs\DependencyInjection\ServiceContainer::load() should be compatible with Symfony\Component\DependencyInjection\Container::load($file) in /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/DependencyInjection/ServiceContainer.php on line 20 PHP Stack trace: PHP   1. {main}() /home/develop/parsers/manyvids-parser/crawler.php:0 PHP   2. JonnyW\PhantomJs\Client::getInstance() /home/develop/parsers/manyvids-parser/crawler.php:8 PHP   3. spl_autoload_call() /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Client.php:102 PHP   4. Composer\Autoload\ClassLoader->loadClass() /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Client.php:102 PHP   5. Composer\Autoload\includeFile() /home/develop/parsers/manyvids-parser/vendor/composer/ClassLoader.php:322 PHP   6. include() /home/develop/parsers/manyvids-parser/vendor/composer/ClassLoader.php:444 PHP Fatal error:  Uncaught JonnyW\PhantomJs\Exception\ProcedureFailedException: Error when executing PhantomJs procedure - File does not exist or is not executable: bin/phantomjs in /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Procedure/Procedure.php:138 Stack trace: #0 /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Procedure/ProcedureValidator.php(84): JonnyW\PhantomJs\Procedure\Procedure->run(Object(JonnyW\PhantomJs\Procedure\Input), Object(JonnyW\PhantomJs\Procedure\Output)) #1 /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Procedure/ProcedureValidator.php(61): JonnyW\PhantomJs\Procedure\ProcedureValidator->validateSyntax('\n\n/**\n * Set up...') #2 /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Procedure/ProcedureCompiler.php(107): JonnyW\PhantomJs\Procedure\ProcedureValidator->validate('\n\n/**\n * Set up...') #3 /home/develop/parsers/manyvids-parser/vendor/jonny in /home/develop/parsers/manyvids-parser/vendor/jonnyw/php-phantomjs/src/JonnyW/PhantomJs/Procedure/Procedure.php on line 138

It sounds either way that one of the dependencies required to run v4.6 is not satisfied. In your composer.json file you could try fixing the version number eg "jonnyw/php-phantomjs": "4.6" and then running composer update. If v4.6 cannot be installed, the reason will be flagged.

听起来这两种方式都不满足运行v4.6所需的依赖项之一。在您的composer.json文件中，您可以尝试修复版本号，例如"jonnyw/php-phantomjs": "4.6"，然后运行composer update。如果无法安装v4.6，则会标出原因。

PHP Warning:  Declaration of JonnyW\PhantomJs\DependencyInjection\ServiceContainer::load() should be compatible with Symfony\Component\DependencyInjection\Container::load($file) in D:\wamp\www\wcphp\vendor\jonnyw\php-phantomjs\src\JonnyW\PhantomJs\DependencyInjection\ServiceContainer.php on line 20
Warning: Declaration of JonnyW\PhantomJs\DependencyInjection\ServiceContainer::load() should be compatible with Symfony\Component\DependencyInjection\Container::load($file) in D:\wamp\www\wcphp\vendor\jonnyw\php-phantomjs\src\JonnyW\PhantomJs\DependencyInjection\ServiceContainer.php on line 20


Multiple fields

Add support for multiple fields of 1 type. E.g: UTF8Field (string), UniField (string)

Maybe remove Packet.GetFields() in favor of Packet.Format

Use field IDs


Contributing

Hey, great job with this library, I'm just starting to use it but it seems fantastic. I'm currently developing a DAW in rust and searching a kinda "backend" for it. Since i would like to use this library with my project, I would like to know how could i contribute. Maybe in implementing libraries or other things. I'll be glad to help this library grow.

Hey that's cool! This was one of the first things I ever wrote in Rust so I'm not sure how well I followed rust best practices if at all. If you notice anything wrong or have better ideas for the design I'm all ears. I probably won't be spending much time on this code in the near future though since I'm pretty busy with work and other things.

I'm also not experienced a lot with rust, but i'll be glad to have an eye on the code.


[FancyZones] Allow to change keyboard shortcut for stretching a window across zones

📝 Provide a description of the new feature
At the moment, the only way to stretch a window across multiple zones via keyboard shortcuts is Win+Alt+Ctrl+Arrows which is quite unergonomical for something that I do quite a lot. On the other hand, Win+Shift+Arrows is currently used by Windows for moving windows between screens, which can be done better with Win+Arrows and Fancy Zones anyways.
I would love to be able to change the keys for stretching windows across zones to Win+Shift+Arrows. Ideally there should be a way to just freely assign shortcuts, but to keep this feature lean and to the point I suggest a checkbox in the FancyZones setting below "Override Windows Snap shortcut [...]" that reads "Override shortcode for stretching windows across multiple zones to Win + Shift + Arrow", is turned off by default and, when turned on, changes the keyboard shortcuts.
See #1972 for previous discussions on this.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Ideally there should be a way to just freely assign shortcuts

This. I'd very much like Win+Arrow and Win+Shift+Arrow to do what they always do in Windows (I work on a lot of different computers, learning custom tools is all nice and that but musle memory for non-standard things cripples one when working on standard machines), whereas I'd like to use e.g. Win+Alt+Arrows for moving between Fancy Zones instead of the standard Windows Zones.

There are so many issues related to shortcuts that I have created a more general issue #22186 to prompt discussion for the entire topic hoping to find a more complete solution to the shortcut issue.
I specifically mentioned the stretch across zones issue.


lua54Packages: set the alias

Description of changes
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Do you use those (aka what 's the motivation) ? I wonder if we shoiuldn't rely on lua5_4.pkgs instead. The problem with those luaXXPackages is that they need to be redefined when the original set is modifed (via an overlay for instance).

My primary motivation was consistency. The python package sets, would have a similar problem then.

yeah, my reflexion is part of a bigger infrastructure concern. Let's merge


Add order item group split toggle

What is the motivation for adding/enhancing this feature?

Add support for the order item group split toggle. This toggle is an option that is available along side the regular allow partial allocation toggle.
This toggle can only be enabled if allow partial allocation is enabled. Otherwise this option will remain hidden and toggled off.

EnumId to apply the item group split check is: IIP_SPLIT_ITEM_GROUP, enumTypeId: INV_FILTER_PRM_TYPE.
Need to add a check in the filter modal to exclude this enum from displaying on the UI.

Verified and working fine in release v1.4.0. Hence, closing this ticket.


[CWS] add a CORE support rule filter

What does this PR do?
This allows to target rule only on platform supporting CORE

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Sporadic error when re-focusing godot

Sometimes when I focus Godot again after interacting with a different program (e.g. web browser) I get these error messages in the Godot output:
scene/gui/text_edit.cpp:5194 - Condition "!has_selection(p_caret)" is true. Returning: -1
scene/gui/text_edit.cpp:5188 - Condition "!has_selection(p_caret)" is true. Returning: -1
scene/gui/text_edit.cpp:5206 - Condition "!has_selection(p_caret)" is true. Returning: -1
scene/gui/text_edit.cpp:5200 - Condition "!has_selection(p_caret)" is true. Returning: -1

Any ideas why this might be happening?

That specific listing of errors was for the version currently in the godot asset store.
When downloading the latest source from github, I get the following list of errors:

Accidentally closed...

Just chiming in to say I'm getting the error too. It seems to happen when I click on the text with my mouse, not when focus is changing. Can you check to see if this is what's actually happening for you to?

Found solution for that. You have to add condition
code_edit.get_selected_text() != ""

To the if code_edit.is_dragging_cursor() in draw_cursor function in cursor.gd.
I will make pull request with this change.


New version: OptimalTransportNetworks v0.1.7

Registering package: OptimalTransportNetworks
Repository: https://github.com/SebKrantz/OptimalTransportNetworks.jl
Created by: @SebKrantz
Version: v0.1.7
Commit: a45214c1cbcfad6c15cbe2da391d0836f0012e36
Reviewed by: @SebKrantz
Reference: https://github.com/SebKrantz/OptimalTransportNetworks.jl/issues/7
Description: Optimal Transport Networks in Spatial Equilibrium - in Julia

merge approved

You can also cancel this one. I'll delete the Tag and resubmit this as 0.1.6.


add cran section

this is adding a CRAN section to the report. If the package isn't on CRAN you will get a message about that, otherwise you will see a CRAN badge, properties from the DESCRIPTION, the authors and the number of downloads. It is using the pkginfo library

It's been a month, I'll close it for now.


nerdctl: update to v0.10.0

https://github.com/containerd/nerdctl/releases/tag/v0.10.0
Fix #79

Cirrus CI not running?


feat(circleci): Remove circleci as not needed

circleci was running danger, we have moved this into github actions now so it is no longer required
Signed-off-by: Jason Field<EMAIL_ADDRESS>Description
Describe what this change achieves
Issues Resolved
List any existing issues this PR resolves
Check List

[ ] A summary of changes made is included in the CHANGELOG under ## Unreleased
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

@Xorima can you push another small commit to test the webhooks please?

@Xorima looks like it's working now?

@damacus its not posting it's danger/danger check only the one that runs it

1 Error
🚫	Please include a CHANGELOG entry.
Generated by 🚫 Danger

@Xorima I think it's working now. The permission changes always need to be merge to main before using them. So I did that in the other PR

Released as: 2.0.1


[Bug]: Branch Diff missing in Python Wrapper

What happened?
Branch Diff (equivalent to https://pydocs.lakefs.io/v1.7/docs/BranchesApi.html#diff_branch in lakefs_client) is missing in new Python Wrapper.
I use Branch Diff to find any uncommitted changes in a branch. This is not the same as Ref Diff.
Expected behavior
No response
lakeFS version
No response
How lakeFS is installed
No response
Affected clients
python lakefs 0.2.0
Relevant log output
No response
Contact details
<EMAIL_ADDRESS>

see branch.uncommitted().

Superseded by https://github.com/treeverse/lakeFS/issues/7276


Auto-upgrade to <IP_ADDRESS>

Base PullRequest
default branch (https://github.com/http4k/http4k-connect/tree/master)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/build/src/bin

./upgrade_http4k.sh <IP_ADDRESS>

Changed files

Changed file:

gradle.properties

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Codecov Report

Merging #251 (eef64be) into master (ebe4437) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##             master     #251   +/-   ##
=========================================
  Coverage     72.32%   72.32%
  Complexity     1243     1243
=========================================
  Files           677      677
  Lines          7433     7433
  Branches        577      577
=========================================
  Hits           5376     5376
  Misses         1727     1727
  Partials        330      330

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


It is possible to get duplicate signature sessions

I’ve just installed RTVS 0.3 and I noticed something that could be improved. This is something I do at the beginning of almost all scripts I author.
Observed: 2 floating help panes
Expected: 1 floating help pane
Observed: Inside string literal when I hit Ctrl+Space I get a list of functions I cannot use because I’m in string literal.
Expected: A list of files under “D:\” path

BTW, to get list of files string has to be in R format (i.e. forward slashes) like "D:/"

Also, string must be closed. The issue here is that literal is now closed and caret technically does not belong to it (i.e. caret position is beyond the end of the open literal which by default ends at the line break and end of the line is not inclusive. This is why Ctrl+J or Ctrl+Space brings list of functions.


Show errors in the requested format

render.html -> HTML error
render.png or jpeg -> Image error
All other endpoints -> JSON error (#226)

The original issue AFAIU was about parsing and finding out what's happened. I don't think that presenting the error in image format is better than plaintext in that regard. Or is it?

I remember there was this suggestion from @kmike https://github.com/scrapinghub/splash/issues/226

My thoughts to show an image instead of text was indeed to make it easy to
see what happened in case somebody uses
Or saves the result in a file without checking the response code and
displays it somewhere.
Of course api consumers should always check the responses status code, but
if you forget or don't bother to do it the browser will only display a
broken image with no other clues of what went wrong.
On Aug 17, 2015 9:28 PM, "Pawel Miech"<EMAIL_ADDRESS>wrote:

I remember there was this suggestion from @kmike
https://github.com/kmike #226
https://github.com/scrapinghub/splash/issues/226
—
Reply to this email directly or view it on GitHub
https://github.com/scrapinghub/splash/pull/275#issuecomment-131839954.

@pawelmhm yes, @kmike's suggestion was about the fact that json is better than plaintext, which i can relate to.
@Youwotma i'd wildly guess that splash is used programmatically more often than visually and it's quite tricky to show images in, for example, spider log. Besides, images embedded as <img src="..." /> sometimes result in http errors too, and I cannot remember from the top of my head anyone serving a pretty "404 Not Found" image instead of a plain HTTP error.

You both have a point, but for backwards compatibility it is better to keep png/jpeg error responses simple by default.
In /execute endpoint one can get a screenshot or html of a page even if it returns an error. What about adding an option to do it in render.xxx endpoints, something like ignore_errors=1? If this option is active the result will have the same status code, but splash will still render it and return page's html or screenshot. It may also allow to say "I want to render as much as possible in 30s and get a result; I don't care if it is incomplete".

@Youwotma could you please add a test which will check the error result explicitly, i.e. that it is a JSON response with 'error' and 'message' fields?

Yep, will do it tomorrow

Current coverage is 82.32%

Merging #275 into master will decrease coverage by -0.30% as of 3f2c900

@@            master    #275   diff @@
======================================
  Files           37      37
  Stmts         4028    4028
  Branches       523     523
  Methods          0       0
======================================
- Hit           3328    3316    -12
- Partial        143     152     +9
- Missed         557     560     +3

Review entire Coverage Diff as of 3f2c900

Powered by Codecov. Updated on successful CI builds.

Thanks!


TypeError: Cannot read property 'development' of null

Workflow Version: 5.2.5
Workflow Plugin React Version: 5.2.5
To reproduce:

cross-env NODE_ENV=production av build

Stack Trace:
› TypeError: Cannot read property 'development' of null
(node:56639) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'development' of null
    at Object.include (<redacted>/node_modules/@availity/workflow-settings/index.js:50:44)

Possible solution. Use lodash.get here ?

configuration should never be null an this is another case of duplicate @availity/workflow-settings being present in the workspace. We create an object that stores the values rather than passing the config down to the workflow plugins so if you have duplicate instances of workflow-settings the init method may be called on one which populates configuration but in workflow-plugin-react it calls the other package which as not yet been init'ed.
@robmcguinness  I think this is a good time to probably refactor the whole process and change to have the workflow-plugins have the workflow-settings config object passed into them as args rather than importing.

i think this is related: https://github.com/Availity/availity-workflow/issues/199

Closing since it is a dupe and being handled by #211


feat(transformer/typescript): report error that do not allow namespaces

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3448  👈
#3447
#3446
#3445
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite


Reset password (resetPasswordForEmail) ignores "redirectTo" parameter

Bug report
Describe the bug
When a user requests the Password Reset option, they get a link which points to ...redirect_to=http://localhost:3200, which is my app's home url, not the one specified in the redirectTo parameter. That way, my app router cannot show the user the option to reset the password. Similar to issue https://github.com/supabase/supabase/issues/832.
To Reproduce
Request the password reset, make sure to specify a redirectTo param
const { data, error } = await<EMAIL_ADDRESS>{
   redirectTo: 'http://localhost:3200/forgot-password',
});

Expected behavior
The user should receive an email containing a link with a URL pointing to ...redirect_to=http://localhost:3200/forgot-password
Screenshots
Authentication settings are set to allow the Forgot Password URL

System information

OS: macOS
Browser chrome
Version of supabase-js: 1.20.0
Version of Node.js: 12.19.0
Front-end application using React Router

related: https://github.com/supabase/gotrue-js/issues/104

Hey @andresiggia, I'm unable to reproduce the issue you encountered and it's actually working for me. Could you send me your project ref so I can help you debug further?

@kangmingtay sure my project is studio-finder (id vsxtrjktxzcvygmqmguh) and I'm getting the following message on Chrome's console when requesting the password reset

Hmm, this is strange, while I debug this issue, have you tried using the cURL endpoint for sending password recovery / magiclinks? It would be a good sanity check :)
curl -X POST 'https://vsxtrjktxzcvygmqmguh.supabase.co/auth/v1/recover?redirect_to' \
-H "apikey: SUPABASE_KEY" \
-H "Content-Type: application/json" \
-d '{
  "email"<EMAIL_ADDRESS>}'

@kangmingtay I've cleared cache here, restarted my app and now it seems to work, sorry for that.

Awesome! Glad it's solved :)


Add benchmarks

Start of resolving #11. Ideally these wouldn't be benchmarks that we made up but this is a good start to proving that skia_canvas is as fast, if not faster (within a few milliseconds) of node alternatives.

@DjDeveloperr any chance that this could get merged?


I don't know if it's related but I can see that Composer is dwonloading vimeo/psalm dev-master 42c3a70. On my local environment it was a part of problem with Composer itself. I updated it in #40 (maybe you should remove your composer.lock and generate it again?

@mateuszchodor @krzysztofrewak any clue why build is failing on Install dependencies part?
Looks like he's been looking for Redis. We have database and php containers enabled in Github Actions


proposal: net: UnixListener: provide a way to get the unlink attribute

https://github.com/golang/go/issues/11826 prevents listeners not created by the process to be unlinked.
https://github.com/golang/go/issues/13877 allows devs to explicitly specify whether go should unlink a listener
why does go design a setter SetUnlinkOnClose but not a getter? was there any concerns?
would golang provide a getter?

Are there cases where a getter is useful?  We didn't provide one because it's hard to see a use case.

it's actually a very specific case, but I felt it could be generalized...
I came along with some lines of code which enumerates all file descriptors, and SetUnlinkOnClose(false) if listener.(UnixListener). it passes these file descriptors to a subprocess created by StartProcess. by doing this, it achieves graceful restart. on startup, the code SetUnlinkOnClose(true) on all the listeners passed to it, removing all the file descriptors the parent passed to the child.
however, the common sense is that a listener should not be unlinked if it's not created by the process. so the parent process have to tell the child which fds are to be preserved by passing a list.
without the getter, the parent process have to maintain a list containing all the fds to be preserved upon creating the listeners, this adds complexity, concurrency locks (listeners are created asynchronously). but with a getter, the parent process only have to build that list when forking the child process.


Please update docs before releasing a major version

See https://reactnavigation.org/docs/drawer-navigator/#checking-if-the-drawer-is-open which conflicts with https://reactnavigation.org/docs/upgrading-from-5.x

It's an open-source project, it's totally possible to miss parts of docs when updating. Feel free to send PRs to fix the docs or open specific issues about missing parts instead of opening an issue saying "update docs before releasing a major version"


record field names must start with a lowercase letter

And here is an SSCCE:
f x = x.Counter

main = f { }

This results in error message:
Function `f` is expecting the argument to be:

    { b | Counter : ... }

But it is:

    {}

So the compiler is printing out a "type" { b | Counter : ... } as if it were valid Elm, though it isn't.

Previous discussions: #1318, https://github.com/elm-lang/error-message-catalog/issues/98

Centralizing into #1374, follow along there

This is fixed by the new parser. Thanks for the new report!


Fixed instance of NadaParser not being optimized out

SpecifiedLengthExplicitParser was not checking for isEmpty, which was
resulting in prodcuing a NadaParser.
DAFFODIL-2062

Took a little while to track this down, but ended up being a simple fix once found.  I wouldn't be surprised if there are other issues similar to this where we don't have checks in place for Parsers that will generate NadaParsers.


How to remove view from the view stack

Version
Tell us which versions you are using:

react-native-router-flux v3.34.0
react-native v0.30.0

How do I remove a view from the view stack after for instance navigating? Couldn't find it anywhere in the docs nor in the issues, so asking it here.

Actions.pop()

I'm sorry but that's not what I meant. Navigating from let's say a pincode screen (only use once) to some activity screen would allow you to swipe or navigate back right now. PanHandlers={null} is a nice hack if it worked, but in 3.34 it doesn't. Removing it from the view stack would be much cleaner imo.
Get Outlook for iOS
On Wed, Aug 17, 2016 at 10:01 PM +0200, "John Brennan"<EMAIL_ADDRESS>wrote:
Actions.pop()
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Try ActionConst.REPLACE as a parameter like this: Actions.someAction({type: ActionConst.REPLACE})


support typeless api

https://www.elastic.co/jp/blog/moving-from-types-to-typeless-apis-in-elasticsearch-7-0

https://github.com/personium/personium-core/issues/390
already done.


Fix android keyboard input

Using a custom IInputConnection to catch all input data produced by keyboard.
Closes #2631.

there is only one bug to fix.
when i use OnImeComposition and OnImeResult to add new text into the textbox,
the cursor will go to the beginning of the textbox

I'm going to merge this in its current state since it seems to work correctly. We can build on it from here.


Add Documentation for clear() Method

This adds documentation and a unit test for the clear() method.

Closing this PR. All changed here have been merged with the 2.1 PR.


CI Unit Test Job Failing

Unit Test Job is failing with error

Same code is working on local machine

History:
Before  Renovate PR GithubActions Build Job and Unit test were failing due to android lint issue

This caused Renovate Bot PR  failure.
In new PR we added back renovate config. and Disabled Android Linting rule.
After this changes now Build job is passing but Unit tests are failing.

I'd still recommend targeting the latest.  Not sure why mockito is failing in CI though 🤔


Filter on browse all data page

User Story: 2.2 I want to filter through the data, so that I can narrow down my selection

[ ] 13 filters total, organized within 7 groups (see design)
[ ] general AND/OR logic:

AND logic between different filters (e.g., I want to see object name = ribosome AND camera manufacturer = Gatan)
OR logic for multiple values within one filter (e.g., I want to see datasets that have Ribosomes OR Membranes annotated)
exception to the above rule: multi-select under the "available files" section is AND logic (e.g., i want to see datasets where it includes raw frames AND tilt series)

[ ] when filter is applied, show as a blue tag that use can "X" out of to remove
[ ] only show the filter values that we have available in the database (i.e., we wouldn't list "Mesh" under object shape type if it doesn't exist in our metadata)
[ ] once filter is applied, show number of search results in the dataset table
note: see specific filter menu types in design

Link to design: https://www.figma.com/file/WEmbsjtlBUtRy7pzmuCCjj/CryoET-Data-Portal---Phase-2-Designs?type=design&node-id=1637-37001&mode=design&t=QGhDNstOreaiFP49-0

Screenshot from Design (for quick reference):

Following bugs were found during this testing:
Ground Truth Annotation filter is not in correct format and has no hover state #171
None of the filters remains shadowed at click and arrow doesn't turn blue as in design #172
"Results must include all selected file types:" is missing in [Available Files] dropdown filter #173
Check symbol next to text in all filter dropdowns are not centered with text #174

Passing this item to Completed as bugs opened are taking care of already, no need to track this one here, thank you!


Remove Zoo Animals API

NOTE: I am going to take down this API from Heroku, which is why I am removing it from this list of public APIs.

[ x ] My submission is formatted according to the guidelines in the contributing guide
[ x ] My addition is ordered alphabetically
[ x ] My submission has a useful description
[ x ] The description does not have more than 100 characters
[ x ] The description does not end with punctuation
[ x ] Each table column is padded with one space on either side
[ x ] I have searched the repository for any relevant issues or pull requests
[ x ] Any category I am creating has the minimum requirement of 3 items
[ x ] All changes have been squashed into a single commit

@Christiansarpong I mentioned this in the PR description, but I am the owner of the Zoo Animals API, and I am going to take down the API from Heroku. So, that's why I am removing it from this list of public APIs. 😄


How to change port number when running from docker

I'm using the docker command mentioned in https://github.com/rhasspy/larynx#docker-installation
However, when I flip the port number in the docker run command, it doesn't work the way I want it to.
May I know the right way to host larynx in a different port, if there is one

I assume you mean the -p flag, which when changed has no effect.  The HTTP server inside takes a --port <PORT> argument.

That works as expected, thank you. I'm closing this issue now.


RADARR: Add getCalendar def

Add a def for calendars
https://github.com/Radarr/Radarr/wiki/API:Calendar

Added under 820058c in dev branch


build: run tests natively on ARM, Mac, and Windows in CI

Closes #26
Closes #27
Closes #28

Will need to update our branch protection rules before I can merge this.


Is 1.1.0 actually available yet?

The README instructions have not been updated and I can't seem to pull it down from jcenter.

No, https://github.com/groovy/groovy-android-gradle-plugin/releases
See on how to use 2.2.0 with 1.0.0 of this plugin https://github.com/groovy/groovy-android-gradle-plugin/issues/121#issuecomment-248320583

Cool, thanks @pieces029 .


feat(labs): support elements with no reactive props

For each of the wrapper generators, this adds support for elements with no reactive props.
For example:
class Foo extends LitElement {
  render() {
    return html`no props!`;
  }
}

We basically conditionally output the code any reactive property related logic depends on, rather than always outputting it.
Vue
When generating vue wrappers, we produce two relevant things:

a Props interface (empty when no properties exist)
a vDefaults.created function for setting the defaults of these props

If no props exist, we end up with an empty interface ({}), which means keyof Props is now never, leading to various type problems in vDefaults.created (which tries to key into props).
For example:
      interface Props {}
      const vueProps = defineProps<Props>();
      const defaults = reactive({} as Props);
      const vDefaults = {
        created(el: any) {
          for (const p in vueProps) {
            defaults[p as keyof Props] = el[p]; // ERROR: `never` can't be used to key into `defaults`
          }
        }
      };

I have changed this to conditionally output vDefaults.created if there are props. So without props we will now get:
      interface Props {}
      const vueProps = defineProps<Props>();
      const defaults = reactive({} as Props);
      const vDefaults = {};

Angular
When generating angular wrappers, we produce a constructor and some properties like so:
private _el: FooElement;
private _ngZone: NgZone;
constructor(e: ElementRef, ngZone: NgZone) {
  this._el = e.nativeElement;
  this.ngZone = ngZone;
  // ...
}

When no props exist, this constructor is basically empty other than the assignments. Which leads to type errors in strict mode, since _el and _ngZone are unused.
I have changed this to do the following:

Only output _el if we have properties or events (existence of one of these results in code which uses _el)
Only output _ngZone if we have properties (only property setters use _ngZone)
Prefix unused parameters with _ in the constructor (e.g. constructor(_e: ElementRef))

Open questions/feedback
As you can see, this has resulted in fiddlier code. I wonder if we should slightly refactor how we build up these templates to make such conditional logic a lot easier.
Do we want to think about doing that in this PR instead of the approach i've taken?
Also, in the vue code, i don't fully understand how vDefaults gets consumed. Can we skip it entirely? is it ok to have just cut out created? etc.

@kevinpschaaf is there anything blocking merge since your last review?
This PR seems to be blocking #4147 which is blocking the MWC doc site


Dark Light Toggle Button Shifted In Footer

Fixes #373

Describe the changes you've made
Changed Position of dark and Light Toggle button
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Screenshot

Checklist

[x] I have mentioned related issue number in PR description.
[x] I have added screenshots and related demo links of my changes..
[x] The title of my pull request is a short description of the requested changes.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

I think we should do this on all pages what say @QAZIMAAZARSHAD ?

I think we should do this on all pages what say @QAZIMAAZARSHAD ?

@nlok5923 it already existed on every page but was recently disturbed by a new update in the login page.
That's why only this page.


fix type checking issues in generated code

KitQL generated code runs into a lot of type checking issues when running svelte-check with tsConfig option "strictNullChecks": true. For instance:
.../src/lib/graphql/_kitql/graphqlStores.ts:141:54
Error: Spread types may only be created from object types.
			if(type === 'cache-and-store' ) {
				set({...get(KQL_GetContactMethodByPrimaryEmail), ...updatedCacheStore});
			}

.../src/lib/graphql/_kitql/graphqlStores.ts:174:33
Error: Type 'number | undefined' is not assignable to type 'number | null'.
					const cachedData = kitQLClient.requestCache<Types.GetUserQuery, Types.GetUserQueryVariables>({
						variables, operationName, cacheMs,	browser
					});

..../src/lib/graphql/_kitql/graphqlStores.ts:255:5
Error: Type 'RequestResult<GetUserQuery, Exact<{ [key: string]: never; }>> | undefined' is not assignable to type 'undefined'.
  Type 'RequestResult<GetUserQuery, Exact<{ [key: string]: never; }>>' is not assignable to type 'undefined'.
			if(type === 'cache-only' || type === 'cache-and-store') {
				updatedCacheStore = kitQLClient.cacheUpdate<Types.GetUserQuery, Types.GetUserQueryVariables>(operationName, data, { variables });
			}

As I usually put my _kitql folder in .gitignore, I didn't see all these.
Thank you for letting me know.
I'll take care of this in the next release.

This package is now deprecated.
Check out the Announcements here


Query all offers and/or issuers for a particular asset?

Would it be useful to have a horizon endpoint that lists all of the issuers of a particular asset?  This adds a degree of network discoverability to horizon.  I can know who is participating in the market for BTC (for example).
Likewise, it might be useful to be able to query all offers for a particular asset.  Something like StellarServer.offers('BTC',null) could return all offer/issuer pairs regardless of the issuer.  This would allow easy construction of the orderbook for a particular asset.  It would then be up to the client to determine the level of trust that they have for each issuer.

done here: https://github.com/stellar/go/pull/147


Refactor JWT class location

In the security project, should the classes in the jwt package be moved into the control package?  Should the AuthenticationMechanism go there as well?  Considering the Credential class was renamed to TokenCredential, it's now kind of redundant with the package name.  Does that mean the IdentityStore needs to be renamed as well?

I don't think we need to move the AuthenticationMechanism into the control package, especially since it's annotations as an application scoped bean.


Allow quiet operation in disabled mode

With ec856996a7caa6f6ef0c21c7c03457ec1a27552c the mocked instance is logging all navigation events. While I appreciate the mocked instance, I don't really want to see the logs in development. Maybe that should be controlled by a new debug option?

Hi @rchl i can see. Would you be interested in adding option for it? Even disabled by default seems good to me :)

I could work on it when I have some time but the more I think about it, the less sure I am how this should be fixed.
So adding debug option to log pushes IMO makes sense but then it also makes sense that it works whether using normal or mock variant. So the mock instance would just nothing on calling the methods and there would be separate plugin just for handling logging when debug is enabled. Does that sound good to you?


Update gali.ts

added 2 words in hindi string

This branch has conflicts that must be resolved
Use the web editor or the  to resolve conflicts.
Conflicting files
api/gali.ts


Add a programming language

Ideally you would be able to write a game or other application on the Monotron, using the keyboard (or UART input) and without changing the contents of Flash. But what programming language?

BASIC
PicoC
Lua
MicroPython?
Monkey
Something else?

The Flash ROM should provide generic routines (like put_character_on_screen) through a Software Interrupt (SWI) system.

We don't really have the RAM to edit decent sized functions so we need a language we can tokenize / convert to byte code on a line-by-line basis, just like BASIC.
Rhai is an option, but the syntax might need a little tweaking to make it line based.
BBC Basic has some nice assembler support and plenty of examples.
We could also take BASIC and improve it a bit, with arrays and hashes, etc.

Ported TinyBasic.

Ported a 6502 emulator and Microsoft Enhanced BASIC.


can't get wikiart.zip to download

Hi. Thanks for the great tool!
I can't get the wikiart.zip to download. First I was running the script included in the package, but it would always time out. Then I went directly to , but I get a message (in Chrome) that "this site can't be reached."
Is there any other way to obtain this resource?
Thanks in advance.

Hi, thanks for reaching out! I just tried to ping the link in Chrome and the download is starting, can you try again? Let me know if it still doesn't work
Tom

It indeed appears to be downloading now. I'm quite sure it wasn't for a while. Glad it's working. And sorry for the trouble.

No problem, I also had hard time to download it completely


feat: add ConfigManager methods for finding configs

This commit adds two static methods to the ConfigManager class:

listConfigFiles() - this lists well known config files with or without a project type filter. It does not touch the file system at all.
findConfigFile() - this looks for a well known config file (with or without a project type filter) in a specific directory.

The goal of these methods is to reduce code duplication in the codebase and make it simpler to discover config files before creating a ConfigManager instance.

CI is not happy


travis: update go to 1.13

Signed-off-by: FUJITA Tomonori<EMAIL_ADDRESS>

Codecov Report

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

@@            Coverage Diff            @@
##             master     #119   +/-   ##
=========================================
  Coverage          ?   37.72%
=========================================
  Files             ?       62
  Lines             ?    30736
  Branches          ?        0
=========================================
  Hits              ?    11594
  Misses            ?    18021
  Partials          ?     1121

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 7426699...ea3e309. Read the comment docs.


update post submit container

What this PR does / why we need it:
Missed edit on my part - need to get to go 1.14
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Specify your PR reviewers:

/cc @gyliu513 @morvencao
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

given the change is only for metering, I'm trying to help during  NA day time.

/lgtm


But if I log into the docker interactively:
docker exec -it -u curtine container_name bash
the $USER environment variable you expect to be populated is.
The $USER environment variable should be populated in both cases.
Running CentOS6 guest on Ubuntu 16.04 host.
`docker version
Client:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:        Thu Aug 18 05:33:38 2016
OS/Arch:      linux/amd64
Server:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:        Thu Aug 18 05:33:38 2016
OS/Arch:      linux/amd64`
docker info Containers: 1  Running: 1  Paused: 0  Stopped: 0 Images: 102 Server Version: 1.12.1 Storage Driver: aufs  Root Dir: /var/lib/docker/aufs  Backing Filesystem: extfs  Dirs: 28  Dirperm1 Supported: true Logging Driver: json-file Cgroup Driver: cgroupfs Plugins:  Volume: local  Network: host bridge overlay null Swarm: inactive Runtimes: runc Default Runtime: runc Security Options: apparmor seccomp Kernel Version: 4.4.0-45-generic Operating System: Ubuntu 16.04.1 LTS OSType: linux Architecture: x86_64 CPUs: 4 Total Memory: 3.859 GiB Name: curtinevmu ID: WLZS:ATZR:GDFQ:7GHM:SBRF:2XHR:55LV:KJJL:LCDT:ZZJT:MOGA:SQWH Docker Root Dir: /var/lib/docker Debug Mode (client): false Debug Mode (server): false Http Proxy: http://web-proxy.corp.hp.com:8080/ Registry: https://index.docker.io/v1/ WARNING: No swap limit support Insecure Registries:  <IP_ADDRESS>/8

http://unix.stackexchange.com/a/76356
I think this behaviour is expected. It's not up to docker to set $USER. Maybe bash -l -c will do what you want?

You are 110% correct! Thanks @dnephin and apologies for logging something that wasn't an issue. You normally don't notice this when you use bash -c as USER is exported... So it looked like a bug, rather than actually being one! I guess this issue should be closed...


feat: full width docs

🚀 Deployed on https://66e307ffa022331b28a5a896--cozy-fox-0defec.netlify.app


Clear simulator cache

Is your feature request related to a problem? Please describe.
Some times to fix the "Unable to boot simulator" we need to just remove cache folder
rm -rf ~/Library/Developer/CoreSimulator/Caches/.
Would be nice to have a button to clear this cache in the settings page.
Describe the solution you'd like
button that triggers rm -rf ~/Library/Developer/CoreSimulator/Caches/
Additional context

If this make sense i would love to contribute.


Release 0.13.0

Issue #, if available: N/A
Description of changes:
Create release for aws-xray-sdk-ruby package. Since we are adding functionality, and not patching anything, we bump to version 0.13.0.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The most recent run of the release workflow failed, but this was because the CHANGELOG had an invalid version header:
https://github.com/aws/aws-xray-sdk-ruby/blob/5a228df14e8e3a1113d5bb8dcef4ce1738803024/CHANGELOG.md#L1-L4
This PR sets the CHANGELOG to have the correct version number (0.13.0) which matches the version.rb file. Therefore, we expect that merging this will result in success for the automatically triggered release workflow.


Ancient market price increase/QoL updates

With how much Yen floating around the game, increase "Money per kunai" from 10.0 -> 20.0
Add a sorting option which can view AK from lowest to highest, or vice versa
Add in a faster way to submit single kunai being sold instead of having to redo all of your inputs one by one

Reverse Auction / Yen Offer is an interesting idea.
I don't know that offering a listing increase on price is necessarily needed. I know yen is pretty abundant right now, but gain changes/etc., may be reducing the amount of yen in the game soon.
I'd personally like to see some other form of yen sink to the game, rather than continually update the price per AK to fit the yen surplus. Some idea currently thrown around are offering BL ability training / increased efficiency ability training in exchange for yen.

Reverse auction was attempted but cancelled due to flooding concerns.
Recent update has increased the price of AK.


refactor: chunk graph

Summary

Refactor chunk graph and support chunkGraph.getModuleChunksIterable()
Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench

!bench


chore: disable cw pool tests until solution is found

Closes: #XXX
What is the purpose of the change
Ref: https://github.com/osmosis-labs/osmosis/issues/5477
Having issues in CI. Disabling until we investigate and fix
Testing and Verifying
This change is a trivial rework / code cleanup without any test coverage.
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

Yeah I think Nicolas fixed these. Closing for now