The game cannot be displayed in full screen

What happened?
Entering the game with the client, cannot be full screen, there are large black surfaces on both sides, and the field of view is very small
Steps to reproduce
After entering the game through the client normally, the display interface cannot be displayed in full screen, and there are large areas of black on both sides.
Operating System
Windows 10  (10.0.17763) 64bit
Wallet
Metamask
CPU
AMD Ryzen 5 3600 6-Core Processor
GPU
NVIDIA GeForce RTX 3060 12143MB
RAM
49101MB
Kernel Version
1.0.0-1918065042.commit-710b300
Unity Version
desktop-main.commit-7272035
Realm
dg
NameTag
aalenst#3cd1

Can you share a screenshot?

Duplicated https://github.com/decentraland/explorer-desktop/issues/121
Please re-open if I'm missclassifying

At 2022-03-03 00:02:18, "Mateo Miccino" @.***> wrote:
Can you share a screenshot?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


doc: Proof-read documentation, checked it with the Google Docs spell che...

...cker and fixed typos.
@enisoc

LGTM


Assertion fails when dump constant table.

I used the following code to dump a constant table just right after a Func instance is created, but got an assertion failure Assert(varConst != nullptr); when feeding CH with bigfib.cpp.js.
 --- a/lib/Backend/Func.cpp
+++ b/lib/Backend/Func.cpp

@@ -295,6 +295,7 @@ Func::Codegen(JitArenaAllocator *alloc, JITTimeWorkItem * workItem,
         {
             func.TryCodegen();
             rejit = false;
+            Js::FunctionBody *fbody = (Js::FunctionBody*)(func.GetJITFunctionBody()->GetAddr());
+            Js::ByteCodeDumper::DumpConstantTable(fbody);
         }

How can i fix this problem? Thanks!

Clarifying: it looks like you modified ChakraCore source code (diff above), built ch, and ran ch bigfib.cpp.js, and then encountered this assertion failure.
Which commit did you base your change on?

I'm assuming that is asm.js function? In that case, the issue is that DumpConstantTable only supports dumping normal js functions, and doesn't support dumping constant table for asm.js.
Unfortunately it doesn't look like there is any API right now that will dump asm.js constant tables at JIT time AsmJsByteCodeDumper::DumpConstants would work only during bytecode gen. If you don't care about asm.js functions, you can wrap that call in an if(!fbody->GetIsAsmjsMode()) and it will work for everything else.

@dilijev
My modification is based on branch "release/2.0", and the latest commit is as following:
commit 039006b8f4b62eb9d4812241377cf6a38e3c073e
Merge: 34c3ab1 9f7430d
Author: Oguz Bastemur<EMAIL_ADDRESS>Date:   Sun May 21 04:30:40 2017 -0700
@MikeHolman
Yes, it is asm.js function because both fields "m_isAsmjsMode" and "m_isAsmJsFunction" are true.
It works when I use the statement "if(!fbody->GetIsAsmjsMode())" before dumping constants.
Thanks all :)


Add China-specific deploy instructions

The default golang proxy is unavailable in that geography: goproxy/goproxy.cn.
This is no doubt familiar to veteran Gophers, but adding the note should make others’ life easier – particularly given it’s a country where a distributed encryption protocol like Matrix is highly in demand.
Pull Request Checklist

[x] Pull request includes a sign off

I'm reluctant to include anything at this stage which is specific to any particular user's deployment, because there are endless combinations which can viably be included which then pollute the core of the installation instructions. This includes adjustments to GOPROXY.
As a general rule, this encompasses things like:

reverse proxies (e.g nginx/apache/caddy/haproxy)
specific installation in certain OSes/environments/architectures (e.g raspi/cloud providers/plan9/solaris/windows)
service management (e.g k8s/ansible/systemd)
extraneous supporting services (e.g pgbouncer)

Aside from it adding noise to the installation instructions, it also becomes a maintenance burden for myself because these instructions end up getting out-of-date quickly, and are typically not tested. I'd rather not add that burden at all.


Add option in Encode to ignore omitempty tags

Currently there isn't any way to marshal a struct and include empty fields that are tagged with omitempty. Creating a second struct with the same fields but without omitempty tags accomplishes the same thing, but this is a tedious process.
There might be a better way to expose this functionality, but I figured I'd put in a PR for one possible option.

CLA received. Cleared for merge from a legal perspective.

Thanks, @joshwget. I don't see this kind of option on other Golang encoders that respect the omitempty annotation, such as those in the standard library encoding/json or encoding/xml packages, and it seems unusual for an encoder to circumvent its own struct tags with configuration. Could you explain in more detail what problems this change helps you resolve and why it's the optimal solution?
Thanks,
Eric, Cloud Foundry Runtime Diego PM

Having the ability the override omitempty is needed when using marshaling and then unmarshaling as a technique to convert between types. For example, if we have an instance of struct Config, we can convert it to another type such as map[string]interface{}  by marshaling the instance and then unmarshaling it to map[string]interface{}.
To see a code example of this, check out https://github.com/docker/libcompose/blob/master/utils/util.go#L60.
omitempty tags on the original struct break this process because "empty" attributes won't be present in the resulting type. However, we still need the omitempty tags for other uses with the original struct, so we can't remove them. If we had the option to selectively ignore the tags, we could ignore them just for this use case.

Hi Josh,
I don't think we can accept this. While I understand what you're using it for, marshalling and unmarshalling seems like a computationally expensive and strangely indirect way to do a type conversion, and apart from your technique, I can't think of a reason that an explicitly omitemptyied field should ever be ignored.
Sorry,
Connor & Joachim, CF-Diego developers


could not parser lppe.asn

Parser failed when meet the following string:
OMA-LPPe-Uri ::= VisibleString (FROM ( "a".."z" | "A".."Z" | "0".."9" | ":" | "/" | "?" | "#" | "[" |
"]" | "@" | "!" | "$" | "&" | "'" | "(" | ")" | "*" | "+" | "," | ";" | "=" | "-" | "." | "_" | "~"))
Error message is following:
ERROR: Symbol '"' at line 488 is prohibited by ASN.1:1994 and ASN.1:1997
Best Regards

The problem is caused by "'".
Can workaround this by delete "'" temporarily.

Use "'" instead of "'" also can pass the compile.


add example values for block storage

What this PR does:
Provides example helm values to configure Cortex for blocks storage on AWS S3.
Checklist

[ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@kd7lxl we could also use this https://github.com/cortexproject/cortex-helm-chart/wiki
That might actually be easier, no idea if you can create PR's for the Wiki

@kd7lxl we could also use this https://github.com/cortexproject/cortex-helm-chart/wiki
That might actually be easier, no idea if you can create PR's for the Wiki

The thing I don't like about the wiki is you don't end up with a file you can use directly with helm install -f conf/provider/...

@kd7lxl we could also use this https://github.com/cortexproject/cortex-helm-chart/wiki
That might actually be easier, no idea if you can create PR's for the Wiki

The thing I don't like about the wiki is you don't end up with a file you can cp or use with helm install -f conf/provider/...

true but I don't think anybody does that

@kd7lxl we could also use this https://github.com/cortexproject/cortex-helm-chart/wiki
That might actually be easier, no idea if you can create PR's for the Wiki

The thing I don't like about the wiki is you don't end up with a file you can cp or use with helm install -f conf/provider/...

true but I don't think anybody does that

I do.
Feel free to take this content as the basis for a wiki article and close the PR if you prefer.

Added example config in the newly launched cortex wiki


🚀 - Add Shade of Purple theme

Which @codeimage/* package(s) are relevant/releated to the feature request?
highlight
Description
This task consist of creating a new theme for CodeImage editor using CodeMirror API.
https://github.com/ahmadawais/shades-of-purple-vscode

Instructions for contributions

Follow the CONTRIBUTING.md instructions to prepare your environment.

NOTE If your intention is to only develop a new theme, you don't need to use docker or install everything. You will work only on @codeimage/highlight package

Move to packages/highlight. You will see a How to create a new theme section in the README.md file .
Run the pnpm generate:theme script and put the shadeOfPurple theme name
Open the theme playground with pnpm dev (from the highlight package) or pnpm dev --filter=@codeimage/highlight (from the root folder)

For further help you can ask for help in the comments below

Hey @riccardoperra Can I work on this?

@riccardoperra When I run pnpm dev in the highlight folder, I get this error:

Can you please help with this?

Hey @SIMRAN88, have you launched the pnpm libs:build command from the root folder? It seems that it doesn't find the @codeimage/vanilla-extract dist folder.
Otherwise you can launch these two commands
cd packages/ui
pnpm build

cd packages/vanilla-extract
pnpm build

Hey @riccardoperra , some error when I try to build the vanilla-extract

The ui build has happened successfully

Hey @riccardoperra Thank you, it worked

This seems to look like this but the shadesOfPurple theme in vscode looks a little different.
something like this:

Any thoughts on this?

Hey, the generated theme is just a placeholder 😅 it doesn’t automatically generate the theme from vscode.
there should be a vscode theme .json file with all the tokens which should be added In the CodeImage theme

Hey, the generated theme is just a placeholder 😅 it doesn’t automatically generate the theme from vscode.
there should be a vscode theme .json file with all the tokens which should be added In the CodeImage theme
https://github.com/ahmadawais/shades-of-purple-vscode/blob/master/themes/shades-of-purple-color-theme.json
Obviously you don't have to port all the 300 tokens but just the most important ones present in the defineEditorTheme function (there are also other properties you can pass but they are optional)

Oops, my bad! 😅  I will make sure to follow the reference for the same. Thank you @riccardoperra

resolved #370


Download a specific file by name

Anyway, I can update a file by its name?
I found that if a file is updated, the file ID will point to the old file instead of the update the new file with the same name. Any way to fix this problem?

I think if you upload from browser at google.drive.com, by default it updates the same-named file (ie. updates its version).
You can specifically do this by right-clicking the file and select "create new version" button.
For further questions, probably this is not the right place to ask.


Ad Hoc build: Failed to verify bitcode in Apptentive.framework/Apptentive

Steps:

Archive a project that includes the Apptentive Framework
Select Export
Select Ad Hoc

You will see this error which seems to be a result of the bitcode compilation differences between normal builds and archived builds. More details about the differences between the two can be found in this thread: https://github.com/Carthage/Carthage/issues/535

Thank you for the report and sorry for the radio silence on my end.
I've built the framework for the 5.0.0 version using archive for the iphoneos destination, as suggested in the thread you linked to.
We're doing some testing on our end as well, but if you have a sec please let me know if it works for you.

I fell victim to this as well, but it looks like you have to follow the considerably-more-involved instructions for an iOS, watchOS or tvOS project or bitcode will not be built correctly. This involves not using the "Embedded Binaries" section, but instead running a custom Run Script build script phase to your app target.

I'm closing this for now, but please let me know if you run into any further issues.


Add support language to Text Extractor

Description of the new feature / enhancement
I just want you guys to add Vietnamese to Text Extractor feature.
Scenario when this would be used?
I want to use it with my mother language and English.
Supporting information
No response

Please refer to our documentation on how to install a new OCR language.
If vietnamese isn't available please suggest adding it in the /feedbackhub


Update graphql.js

this is a followup on the question i asked on slack. my understanding is that graphql gets confused when we give it $ or .. Since any field that uses $ in their name will also use ., I thought we could just amend the rule so that not only Arrays get skipped, but also Objects.

Hmm, I don't know if it's a good idea to do it this way. If field names cannot contain a . I think we should have a separate test that throws an error for that, rather than just ignoring the field.


Update pyo3_template.py

Fixes error[E0658]: trait bounds other than Sized on const fn parameters are unstable

Closing; seems it needs to go directly to Pyo3 for 2021 compatibility.


add mongodb service support

Changes

Added the docstore mongodb service support to tekton chains so it can be used with mongodb
Sometimes people are looking to integrate chains but need to keep (some of) their build systems on-prem.
MongoDB is a common database that organizations can setup and integration into chains is very simple via the docstore library used.
I've added a bit of documentation around the subject.

One shortcoming of this setup is the URI is just for the collection and you need to provide tekton-chains-controller with a mongodb connection URI via MONGO_SERVER_URL
people might also need to mount root certificate files
https://github.com/google/go-cloud/blob/master/docstore/mongodocstore/urls.go#L44
I'm open to other suggestions. Maybe we can pull in config and then setEnv from within the controller? But that feels a bit brittle
With the prospect of chains being added to the operator I'd imagine it could take config and patch the deployment as appropriate so shouldn't be too many problems here
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[X] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Release Notes

Added mongodb storage driver (see docs for integration guidance)

Generally LGTM!
If we want to make this more configurable within chains itself (i.e. without needing to modify the deployment / env vars), we could probably replicate what the mongodb driver is doing from docdb.NewStorageBackend and pull the server URL from a Chains config value.
That said, I'm fine with this PR as is (we can always add this later).

@priyawadhwa Do we have e2e tests for docdb integrations, or are we just relying on gocloud for integration coverage here?

yup we just rely on gocloud, no testing support at the moment

/ok-to-test

Rebased

/lgtm
Vendor diff makes me sad, but I don't think it's worth blocking this feature for.
@chuangw6 @ykass This may be another reason to split up the writer - pluggable attestation writer implementations 🤔

@wlynch Is this an alternative to use Grafeas ?

@wlynch Is this an alternative to use Grafeas ?

Kinda? It's adding another driver to the existing storage.docdb support. Grafeas is just another storage location for attestations (along with GCS, OCI, DocDB, etc.)

/lgtm


LC-220: Created-new-Accessibility-page-and-appropriate-footer-link

Hi Matt, Can you check this out please, I cannot run it at moment but I think it's pretty much there regarding content.

Looks good, will review on tuesday

OK, I'll check the related branches in tomorrow too so you have all the related pieces.

From: m24murray<EMAIL_ADDRESS>Sent: 29 July 2019 17:01
To: Civil-Service-Human-Resources/lpg-services<EMAIL_ADDRESS>Cc: PortobelloRain<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Civil-Service-Human-Resources/lpg-services]  LC-220: Created-new-Accessibility-page-and-appropriate-footer-link (#587)
Looks good, will review on tuesday
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Civil-Service-Human-Resources/lpg-services/pull/587?email_source=notifications&email_token=AMQPUZCU36O3D3VT3REMKFDQB4H5XA5CNFSM4IHS62RKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BFTCQ#issuecomment-516053386, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQPUZCULVF4WIUZ7OYEYJTQB4H5XANCNFSM4IHS62RA.


Incorrect links for Design-Docs in wiki

Bug description
Wiki points to same link for "Simpler Istio, Istiod" and "Istio Authentication rev2 - JWT" point to same document.
Expected behavior
"Simpler Istio, Istiod" should point to https://docs.google.com/document/d/1v8BxI07u-mby5f5rCruwF7odSXgb9G8-C9W5hQtSIAg.
Steps to reproduce the bug
Open the wiki link (https://github.com/istio/istio/wiki/Design-Doc-Links) and click on "Simpler Istio, Istiod". It opens up the document for JWT authentication
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
Master
How was Istio installed?
No installation required
Environment where bug was observed (cloud vendor, OS, etc)
Wiki

thanks! I have fixed this


Create ML-based nowcast plugin for pysteps

Include a deep learning based nowcast function that has the same / comparable interface as the other nowcast functions of pysteps.

For this, the pysteps/nowcasts/interface.py needs to include a new function to discover the new nowcast methods from plugins - similar to importers currently, and postprocessors (cfr ongoing work on pysteps-precip-type).

Hi @Loickemajou , very exciting stuff!

I would greatly appreciate any guidance or suggestions on where to add this preprocessing function within the pySTEPS structure.

in my opinion (but feel free to disagree), the preprocessing you described should not be part of your plugin: the function will require the input in the correct format and it'll be up to the user to load the data and prepare it. You just need to make sure that the requirements are well documented (in the docstrings the type hints), the input validated and, if necessary, that a kind error message is raised.

Thank you very much @dnerini for your explanation.
Will try to make it as clear as possible.

Good morning everyone,
I hope you are all doing well.
I have presently completed the DGMR plugin. I would greatly appreciate your guidance on the steps required to register this plugin with pysteps in order to test its functionality.
Additionally, the DGMR model weights are currently not publicly available and must be downloaded using the Google Cloud Shell terminal. You can do this with the following command:
gsutil -m cp -r "gs://dm-nowcasting-example-data/tfhub_snapshots ."
Thank you in advance for your assistance!

I have presently completed the DGMR plugin. I would greatly appreciate your guidance on the steps required to register this plugin with pysteps in order to test its functionality.

hi @Loickemajou very nicely done!
Plugins use the entry-point specifications, which is something worth reading about in depth before implementing it.
I suggest that you open a PR on the pysteps repo following the same structure as done by @joeycasey87 in  #405 and . This follows the pattern for io plugins, ie discover plugins, which is then called in the init . We also need to document the plugin (Note though that we don't need to provide a cookiecutter in a first phase).
I also suggest that you and @joeycasey87 work together to implement the new plugins to make sure that you do it consistently.

Additionally, the DGMR model weights are currently not publicly available and must be downloaded using the Google Cloud Shell terminal. You can do this with the following command:
gsutil -m cp -r "gs://dm-nowcasting-example-data/tfhub_snapshots ."

What do you mean with "not publicly available"?
Anyway, I wonder if your plugin should not include the DGMR weights packaged directly with it?

Thanks @dnerini  for the detailed procedure.
Will work with @joeycasey87 for a smooth and consistent implimentation of the new plugins.
Concerning the DGMR weights, Will try to work on and figure out if there are possibilities,  though the folder is quite heavy (1 GB).

Concerning the DGMR weights, Will try to work on and figure out if there are possibilities, though the folder is quite heavy (1 GB).

Uh right, good point! Then we need to find an alternative solution.
What about an integration with hugginface? we could upload the DGMR weights into a new pysteps account on huggingface (or reuse an existing one like https://huggingface.co/openclimatefix/dgmr) then use the hf_hub_download function to download the files when needed.
We should also automatically download the model weights the first time the package is launched and then cache the file for future use.
For the cache directory, we could either try to follow the same logic as for the pysteps config file (that is, $HOME/.pysteps/pystepscacheon unix/macos or %USERPROFILE%\pysteps\pystepscache on windows) or directly use https://github.com/platformdirs/platformdirs.

Thank you very much @dnerini  for your suggestions! I am currently exploring the different options you provided and am in the process of implementing them for testing.
I appreciate your guidance and support, and I will keep you updated on my progress.

Good afternoon

Concerning the DGMR weights, Will try to work on and figure out if there are possibilities, though the folder is quite heavy (1 GB).

Uh right, good point! Then we need to find an alternative solution.
What about an integration with hugginface? we could upload the DGMR weights into a new pysteps account on huggingface (or reuse an existing one like https://huggingface.co/openclimatefix/dgmr) then use the hf_hub_download function to download the files when needed.
We should also automatically download the model weights the first time the package is launched and then cache the file for future use.
For the cache directory, we could either try to follow the same logic as for the pysteps config file (that is, $HOME/.pysteps/pystepscacheon unix/macos or %USERPROFILE%\pysteps\pystepscache on windows) or directly use https://github.com/platformdirs/platformdirs.
edit: for reference, ECMWF's ai-models needs to deal with a similar challenge, see https://github.com/ecmwf-lab/ai-models?tab=readme-ov-file#assets. It might be worth having a look at their approach too.

Good afternoon @dnerini,
I hope this message finds you well.
I wanted to share that I have successfully tested the integration of model weights with Hugging Face by creating a personal account. The model can be downloaded directly for the first time and is subsequently cached for future use. It is stored in the cache directory, which is located at $HOME/.pysteps/pystepscache on Unix/macOS or %USERPROFILE%\pysteps\pystepscache on Windows (in my own case).
Regarding the new PySTEPS account, would it be advisable to transfer  ownership to PySTEPS?


Bump version number for September 2024 develop release

Bump release version:
This updates from 0.1.1a5 to 0.1.1a6 for the September 2024 monthly release of the develop code.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.01%. Comparing base (150e8bf) to head (08de592).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #569   +/-   ##
========================================
  Coverage    95.01%   95.01%
========================================
  Files           74       74
  Lines         4235     4235
========================================
  Hits          4024     4024
  Misses         211      211

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Tab margin-top needs to be lower.

Great Theme!
Here is a screenshot comparing the tabs in ST3 OneDarkMaterial (left) and Atoms Theme (right):

When in small sizes Atom removes the top margin from the tabs and when it is larger it uses smaller top-margin than in the OneDarkMaterial Theme:

Also the Atom tabs make use of a little bit of inset shadow.
What do you think about adjusting this?

I'll think about that. But for now, I can't promise you anything since I have more complex projects that need my attention. You can always do a pull request ;)
Thanks

Closed due to:
https://github.com/ptraceur/OneDarkMaterial/blob/master/README.md


Partial bindings for Function

Introduces a Function.using() method that creates a partial binding of the function parameters as an ExprTemplateFunction. If f is an n-ary Function, then
f.using(a=..., b=..., ...)
is an (n-k)-ary function whose parameters are all the parameters of f not among a, b, ....
This is useful for specifying indices without having to create explicit @expr_udfs.
TODO: Is using what we want to call it?
TODO: Add tests for validation errors

This change is


prevent gazelle to generate wrong go_library rules.

bin/bazel_to_go.py (and bin/linters.sh which runs this) creates
symbolic links to auto generated pb.go files and gen.go files.
This PR removes such symbolic links before gazelle runs, so that
gazelle doesn't create incorrect rules anymore. This allows to
remove some gazelle:ignore markers.

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins job mixer/e2e-smoketest passed


awkward marine labels

[x] improve low zooms
[x] add placement: line styles

done 8f01f1d245659aedb230288db7f96c4ccbb19080


HIVE-21270: A UDTF to show schema (column names and types) of given q…

This UDTF helps to retrieve each column name and it's data type that would be produced in resultset when query is executed.
This UDTF can be executed as SELECT GET_SCHEMA("<Some_Query>")

Hi
One entry is required in show_functions.q.out check in #532 wherein I put entry for the UDF.
Since the udtf ends with e 2 entries are required, else pre commit tests will fail

@rmsmani thanks, I added it to show_functions.q.out.
I am not sure I understand this:

Since the udtf ends with e 2 entries are required

Hi
Test cases are defined in
ql/src/test/queries/clientpositive/show_functions.q
Where one test case is written to show the functions ending with e ie like operator.

Hi
Since this function gives the schema for the SQL query its better to have the function name as GET_SQL_SCHEMA

+1

@pvary  please merge the code with master

Hi @ashutosh-bapat, @sankarh
Can you merge the code

Source committed by @sankarh. Thanks
@ShubhamChaurasia close the PR

@ShubhamChaurasia
As the sources merged, can you close this PR.

@rmsmani thanks. Closed.


How to dynamically layer .line() methods against layer_tooltips()?

I am trying to make my lets-plot plots more interactive by using Streamlit's widgets to dynamically change the generated plots upon widget interaction.
One such widget was the st.multiselect() option that is supposed to dynamically change the kind/number of variables to be shown on a plot's tooltips via the .line() method. My first attempt was to initialize a layer_tooltips() variable and then chain a bunch of .line() methods against it in a for loop depending on how many variables were selected to be displayed, but I was unsuccessful (I tried to chain strings of .line() in list comprehension and then use eval()).
Any idea on how can I accomplish this?

Hi.
I made a simple snippet to show how you to config a plot programmatically. The code doesn't use Streamlit, but I hope the code still gives you an idea of how it can be done.
In brief - layers_tooltip() returns a builder, meaning you have to create a builder object and configure it as you want. This object should then be passed to the layer creation function as the tooltips argument. You should create a separate builder for each layer.
If this won't work, please, let us see some code so we can help you.
from lets_plot import *
LetsPlot.setup_html()

# input
d = {
    'x': [1, 2, 3, 4, 5, 6],
    'y': [1, 2, 3, 4, 5, 6],
    'c': ['a', 'b', 'c', 'd', 'e', 'f'],
    'g': ['1', '2', '3', '4', '5', '6'],
    's': ['q', 'w', 'e', 'r', 't', 'y']
}

# Enter variables you want to see in a tooltip
vars_for_tooltip = ['c', 'g']

# configuring tooltips and layer
# layer_tooltips() returns a builder object. Each layer needs its own builder.
point_layer_tooltips = layer_tooltips()
for var_name in vars_for_tooltip:
    point_layer_tooltips.line("{}: @{}".format(var_name, var_name))

point_layer = geom_point(tooltips=point_layer_tooltips)

# plot object
p = ggplot(d, aes(x='x', y='y'))

# display a plot
p + point_layer


store: support more cron-like features in the job queue

We currently support running jobs in an interval. This is useful when jobs need to run periodically. But we should probably also support cron like features like running every day at 6h.
await addRecurringJob(sql, {
  name: "my.cron.job",
  priority: 3, // default 4?
  options: {
    cron: "*/3 * * * * *", // Use 'cron-parser'
    timeZone: "Europe/Amsteram", // optional
    allowedWorkerDelay: { seconds: 1, minutes: 3, hours: 2, days: 4, months: 3 },
  },
});

Allowed worker delay, gives the system N time to catch up on previous cron jobs. And will scheduled the next job after the previous finished. Note that the allowed worker delay may prevent a job from retrying

Probably want to refactor recurring jobs in a 'static' definition, that is required if the job queue worker can handle recurring jobs (ie not excluding the name). This makes it easier to remove 'old' recurring jobs as well.


Separated piece selection code from selectSquare method

No difference in functionality but could come in handy for checking for Check

Need to refactor based on new changes

Fixed previous issues


Finlandselection has re-appeared, perhaps as part of the github mishap

Fixed in commit e22ef7f

@ralfstrandell can we not do back-and-forth updates? If you have a reason to add something, tell me. If I have a reason to remove something, I'll tell you. But it is not productive was us to keep removing and adding the same thing. The reason that finlandselection was removed was that it was taking up space in the cramped little windows. It would be fine to add it on the filter popup, but now it is coming up on the little window.
If we want the finland selection, then we need to do it properly, make it always appear on the filter popup. And probably not as the first thing.
(Stretch goal would be to make it more logical in the UI. Now it is kind of a double not filter. Not finland is not included...)

Here are the bugs on this that I can currently see:
1/ The filter takes up space on the upper left corner small box. Yet, it is not the most important thing to add if we had space. (Solution: move to the filter popup).
2/ On mobile version, the finland filter actually does come in the filter popup. But it doesn't work, it is initially set to do the filtering, even if the checkmark is not on. Solution: code needs restructuring to make the initial filter on init match the filter that is created when some checkbox is clicked. Currently that's two separate pieces of that do not agree. This was previously not a problem, but now that we added the filter popup and re-enabled the finland selection it has become an issue.
3/ Style: The "only" style looks a bit hackish. If we do it properly, it could perhaps be "[x] Finland [x] Other" or similar.

And
4/ May grow popup in mobile too large

Should be fixed in the latest update & cleanup.

So, this issue is resolved in part (it now works) but in part not (the filter should really have finland and other countries checkboxes). Leaving the issue open for that.

Good points.
Not all changes were based on tracked issues, and tasks were not assigned. This caused us to work on same problems, sometimes even doing opposite things. The process itself is important.
This looks good (it needs the word countries though, to look nice):
Country:
[x] Finland [x] Other countries
I can see the list growing in the coming years with one...

Yes, exactly. But now that this isn't buggy, we can take our time improving it.

Closing this and creating other issue for the nicer finland+other filter (#135)


Path Printing Bug

During small refactoring in previous commit, did not turn List.forall into List.exists correctly. This caused a bug in path printing.

Looks good! I'm going to merge this right away since it's a small change.


Tag read write additions

Added support for additional tag references
-Array Elements
-Integer Bits (Bit Index) (writing a bit index required addition of READ_MODIFY_WRITE_TAG service)
Added support for program name prefix for tags (in additional to current method of passing program name separately)
Updated tagname validation
Fixed bug for reading BOOL
Description, Motivation, and Context

Some atomic tag names were not able used
-Array Elements
-Integer Bits (Bit Index)
Program name tag prefixes allows the creation of program scope tags with a single string
Prevous tag name validation wouldn't pass new tag names and would pass other invalid tag names
BOOL tags always returned false the value was checked against 0x01 instead of 0xff
How Has This Been Tested?

Tested reading/writing
-tested different read/write methods
-individual
-group
-scan
-tested tags:
-BOOL, SINT, INT, DINT, REAL
-INT[x], SINT[x], DINT[x], DINT[x,x], DINT[x,x,x]
-SINT.0-7, INT.0-15, DINT.0-31
-DINT[x].x
-Tag.DINTmember[x]
-Tag.DINTmember.x
-Tag.DINTmember[x].x
-visually inspected IOI paths for various tags and checked them against the manuals
Tested reading/writing tags with both methods of creating program tags
-pass program as value
-prefix program to tag name
Tested new tag name validation
-tested new regex extensively regex101.com as well as adding new tagname tests to tag.spec.js
-tested new tag/member length validation and added tests to tagspec.js
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] This is a work in progress, and I want some feedback (If yes, please mark it in the title -> e.g. [WIP] Some awesome PR title)

Related Issue

Pull Request Test Coverage Report for Build 72

30 of 93 (32.26%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-2.8%) to 47.382%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/controller/index.js
0
6
0.0%

src/enip/cip/epath/segments/data/index.js
5
21
23.81%

src/tag/index.js
25
66
37.88%

Files with Coverage Reduction
New Missed Lines
%

src/controller/index.js
1
10.45%

src/tag/index.js
1
43.82%

Totals

Change from base Build 70:
-2.8%

Covered Lines:
438

Relevant Lines:
874

💛  - Coveralls

A couple of notes:

reading the base tag of an array returns the value of the 0'th position
reading a BOOL[x] tag results in Unrecognized Type Passed Read from Controller
I'm not sure if it was failing before, but tagname validation fails for Local tags like Local:1:I.Data.3

Reading array elements of REAL, DINT, and INT are all working for me. Reading bit elements of DINT and INT are working as well

@patrickjmcd These are the notes you made from testing of the PR, correct? If so, lets get those things resolved and I will pull this.  Thanks again for the contributions @jhenson29!

yes, testing the PR

I fixed bool arrays and module defined tags.
The only issue with bool arrays is that you HAVE to pass in BIT_STRING for the data type. The actual read returns 0xd3 (BIT_STRING) and 4 bytes of data. The array index you request is in 4 byte increments, then you have to treat the offset as a bit index. e.g. If you have a data type BOOL[64] and you want to read tag[44], it has to be done as tag[1].12. The module does this internally, (e.g the user still just sets it as tag[44]) but the module has to know to do it by passing in BIT_STRING. If you try to read tag[44], it will error because it's trying to access an index (tag[44*32]) that doesn't exist.

I finally have some down time! I am checking this out now locally.  Will provide feedback by end of day! Sorry about the delay.

Thanks again for the help. Everything looks good. I updated the semver version to denote a feature addition. I am going to release these changes to v1.2.0


List of transformers/mappers

Which transformers/mappers we need?

Observing and reading the documentation we can initialy say that the primary transformations we can implement are are below.

Rename an event
Rename a property or trait
Update a property value
Property Transformations

Casing functions (lowercase, uppercase, snakecase, kebabcase, or titlecase.)


Gantt chart implementation

Hi, it is the Gantt chart planned to be implemented or is possible support it?

It's not currently planned. When a horizontal bar chart is implemented you might be able to somewhat fake it using that. But currently only a vertical bar chart is implemented.

Okay understood, thank you!


[MWPW-138525][Block enhancement] - Enable full width on list icon block

Resolves: MWPW-138525
Description:

Add a 'fullwidth' variant to enable authors to make the block full width and match the XD

Test URLs:

Before: https://main--express--adobecom.hlx.page/drafts/esallee/edu-icon-list?martech=off
After: https://mwpw-138525--express--adobecom.hlx.page/drafts/sandra/edu-icon-list?martech=off

I see there's a test failing with cta-carousel but I can't seem to replicate it locally. If you still see this happening after updating your code, could you please copy over the code change related to cta-carousel.js and cta-carousel.test.js in https://github.com/adobecom/express/pull/476/files#diff-3eb0d32d28a1d353eeded4ebd75498205e22c603c3ba849be24d27186db6ff09 and see if that fixes it? Thanks!


fix(package.json): edit main, exports and publishConfig to improve DX

Edit main, exports and publishConfig to improve DX
Cannot find module directly, if contributors didn't build packages.

Though there is built packages, Contributor want to access source itself, not built package.
Solution I thought
How about making vscode access to workspaces source code directly, not built package. if package.json using publishConfig, making built package and developing package in workspaces for DX separately is available.
{
  "main": "src/index.ts",
  "exports": {
    ".": {
      "require": "./src/index.ts",
      "import": "./src/index.ts"
    },
    "./package.json": "./package.json"
  },
  "publishConfig": {
    "access": "public",
    "exports": {
      ".": {
        "require": "./dist/index.js",
        "import": "./esm/index.mjs",
        "types": "./dist/index.d.ts"
      },
      "./package.json": "./package.json"
    },
    "import": "./esm/index.mjs",
    "main": "./dist/index.js",
    "types": "dist/index.d.ts",
    "module": "esm/index.mjs"
  },
}

Thanks for Contribute.
I would appreciate it if you could change the order of the exports fields(types first) by referring to below information. 👇 microsoft/TypeScript#46334
cc @JaeYeopHan

after trying to reproduce problem situation. but actually, I couldn't reproduce it what you worried.
Anyway, I changed order of exports as your link reference's advice, but I couldn't get confidence at this implementation. If my change is wrong, could you make some code for me? I just want to know what you want exactly.

I don't know how you tried it, but as written in the issue above, in an environment using (ts)module: next and wrong order of the exports map - types are not described first, the type evaluation can match the wrong file.
btw, i'll investigate broken netlify issues...  I don't think it's because of this change 🤔, but can you wait a moment? 🙏  🙏

@manudeli netlify issue is solved in #231. I'll rebase this PR and publish beta release for testing.

@manudeli sorry i cant rebase this pr (this pr is created through a forked repo 😅)
can u do it?

@JaeYeopHan @SoYoung210 Rebasing done


Add alert window on Android

from https://godotengine.org/qa/10266/os-alert-not-working-on-android
OS.alert("Alert message test!!", "Title !!")

Which Android version did you use to test this? The dialog above looks like a Android 2.3 dialog window, it should be Holo-themed (Android 4.x) or Material Design (Android 5.0 and later) if the user is running those Android versions.

@Calinou I tested it with Android 7.0, Nexus 6p.

Hey question is it possible to change the looks of the alert, since it looks like the theme it's from an Android 2 or older to me. Thanks

Hey question is it possible to change the looks of the alert, since it looks like the theme it's from an Android 2 or older to me. Thanks

This is due to (presumably) outdated API usage, so please open a new issue for this.

https://github.com/godotengine/godot/issues/85110#issue-2001074979


Advertised broker address

Expected behavior
Connecting through the discovery to the broker in the cluster setup should redirect to the public address of the broker. Preferably there should be an option to set an advertised hostname, this should come together with an option to set a bind host for the broker (however, this might not be necessary).
Actual behavior
Broker advertises the hostname as the address for the client to connect to. On EC2, it results in the client trying to connect to something like ip-xx-xxx-xxx-xxx which is not resolvable from outside of the same region. The error I see on my local machine while trying to publish messages to Pulsar is the following:
2016-09-19 17:45:38,592 - ERROR - [main:CmdProduce@186] - java.net.UnknownHostException: ip-10-239-182-4: unknown error
com.yahoo.pulsar.client.api.PulsarClientException: java.net.UnknownHostException: ip-10-239-182-4: unknown error
	at com.yahoo.pulsar.client.impl.HttpClient$1.onThrowable(HttpClient.java:139)
	at com.ning.http.client.providers.netty.future.NettyResponseFuture.abort(NettyResponseFuture.java:238)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.abort(NettyRequestSender.java:422)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.sendRequestWithNewChannel(NettyRequestSender.java:290)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.sendRequestWithCertainForceConnect(NettyRequestSender.java:142)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.sendRequest(NettyRequestSender.java:117)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.sendNextRequest(NettyRequestSender.java:493)
	at com.ning.http.client.providers.netty.handler.Protocol.exitAfterHandlingRedirect(Protocol.java:189)
	at com.ning.http.client.providers.netty.handler.HttpProtocol.handleHttpResponse(HttpProtocol.java:427)
	at com.ning.http.client.providers.netty.handler.HttpProtocol.handle(HttpProtocol.java:470)
	at com.ning.http.client.providers.netty.handler.Processor.messageReceived(Processor.java:88)
	at org.jboss.netty.channel.SimpleChannelUpstreamHandler.handleUpstream(SimpleChannelUpstreamHandler.java:70)
	at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
	at org.jboss.netty.channel.DefaultChannelPipeline$DefaultChannelHandlerContext.sendUpstream(DefaultChannelPipeline.java:791)
	at org.jboss.netty.handler.stream.ChunkedWriteHandler.handleUpstream(ChunkedWriteHandler.java:142)
	at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
	at org.jboss.netty.channel.DefaultChannelPipeline$DefaultChannelHandlerContext.sendUpstream(DefaultChannelPipeline.java:791)
	at org.jboss.netty.handler.codec.http.HttpContentDecoder.messageReceived(HttpContentDecoder.java:108)
	at org.jboss.netty.channel.SimpleChannelUpstreamHandler.handleUpstream(SimpleChannelUpstreamHandler.java:70)
	at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
	at org.jboss.netty.channel.DefaultChannelPipeline$DefaultChannelHandlerContext.sendUpstream(DefaultChannelPipeline.java:791)
	at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:296)
	at org.jboss.netty.handler.codec.frame.FrameDecoder.unfoldAndFireMessageReceived(FrameDecoder.java:459)
	at org.jboss.netty.handler.codec.replay.ReplayingDecoder.callDecode(ReplayingDecoder.java:536)
	at org.jboss.netty.handler.codec.replay.ReplayingDecoder.messageReceived(ReplayingDecoder.java:435)
	at org.jboss.netty.channel.SimpleChannelUpstreamHandler.handleUpstream(SimpleChannelUpstreamHandler.java:70)
	at org.jboss.netty.handler.codec.http.HttpClientCodec.handleUpstream(HttpClientCodec.java:92)
	at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
	at org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:559)
	at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:268)
	at org.jboss.netty.channel.Channels.fireMessageReceived(Channels.java:255)
	at org.jboss.netty.channel.socket.nio.NioWorker.read(NioWorker.java:88)
	at org.jboss.netty.channel.socket.nio.AbstractNioWorker.process(AbstractNioWorker.java:108)
	at org.jboss.netty.channel.socket.nio.AbstractNioSelector.run(AbstractNioSelector.java:337)
	at org.jboss.netty.channel.socket.nio.AbstractNioWorker.run(AbstractNioWorker.java:89)
	at org.jboss.netty.channel.socket.nio.NioWorker.run(NioWorker.java:178)
	at org.jboss.netty.util.ThreadRenamingRunnable.run(ThreadRenamingRunnable.java:108)
	at org.jboss.netty.util.internal.DeadLockProofWorker$1.run(DeadLockProofWorker.java:42)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.net.UnknownHostException: ip-10-239-182-4: unknown error
	at java.net.Inet6AddressImpl.lookupAllHostAddr(Native Method)
	at java.net.InetAddress$2.lookupAllHostAddr(InetAddress.java:928)
	at java.net.InetAddress.getAddressesFromNameService(InetAddress.java:1323)
	at java.net.InetAddress.getAllByName0(InetAddress.java:1276)
	at java.net.InetAddress.getAllByName(InetAddress.java:1192)
	at java.net.InetAddress.getAllByName(InetAddress.java:1126)
	at java.net.InetAddress.getByName(InetAddress.java:1076)
	at com.ning.http.client.NameResolver$JdkNameResolver.resolve(NameResolver.java:28)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.remoteAddress(NettyRequestSender.java:358)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.connect(NettyRequestSender.java:369)
	at com.ning.http.client.providers.netty.request.NettyRequestSender.sendRequestWithNewChannel(NettyRequestSender.java:283)
	... 37 more

I could set the hostname on the machine to a public IP address / public hostname but it is an ugly workaround. I do not have any DNS other than EC2 defaults available right now.
Steps to reproduce
Launch the cluster setup in EC2 (without any DNS other than EC2 defaults) and try producing messages.
System configuration
Pulsar version: 1.14
If the suggestions sound okay, I can prepare a PR.

I'm not a big fan of setting the hostname to bind on the config file, since it would require to have a different config file on each host.
What about a setting to advertise the broker address with just the IP ?
Eg: something like :
# Control whether the broker should advertise itself by using the hostname
# or its own IP address.
advertiseBrokerWithHostname=false

This will not work on EC2, the public IP address is not available via ifconfig. Broker would always advertise private 10.x.x.x address. What about doing it similar to how Kafka does it?
# If set, broker will advertise to discovery using this address;
# if empty, broker will advertise using the hostname, unless bindOnLocalhost
advertiseBrokerAddress=<ip address or hostname>

Yet to figure out this will work if the broker binds on the hostname. It might be a good idea to allow people set the bind address / host. For example:
# to bind on all interfaces, set to <IP_ADDRESS>
brokerBindAddress=<ip or hostname>

What do you think?

Oh, I see. Then your proposal sounds good.

Shall are target for brokerBindAddress and advertiseBrokerAddress?

Yes, having both gives the most flexibility, and please go ahead in removing bindOnLocalhost.

@merlimat Please see attached PR. I went for brokerAddress and advertisedAddress as these settings relate to broker.conf file.


[Backport 1.x] Add utf-8 encoding settings in gradle settings

Description
[Backport 1.x] Add utf-8 encoding settings in gradle settings
Issues Resolved
#244
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #247 (b98e04d) into 1.x (d3c79f7) will increase coverage by 0.43%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.x     #247      +/-   ##
============================================
+ Coverage     78.50%   78.93%   +0.43%
- Complexity     3715     3741      +26
============================================
  Files           275      275
  Lines         15569    15569
  Branches       1624     1624
============================================
+ Hits          12223    12290      +67
+ Misses         2631     2573      -58
+ Partials        715      706       -9

Flag
Coverage Δ

plugin
78.93% <ø> (+0.43%)
:arrow_up:

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

Impacted Files
Coverage Δ

...opensearch/ad/indices/AnomalyDetectionIndices.java
80.14% <0.00%> (-0.24%)
:arrow_down:

...java/org/opensearch/ad/task/ADBatchTaskRunner.java
83.09% <0.00%> (+0.31%)
:arrow_up:

...ain/java/org/opensearch/ad/task/ADTaskManager.java
77.53% <0.00%> (+0.46%)
:arrow_up:

...rch/ad/transport/AnomalyResultTransportAction.java
80.69% <0.00%> (+0.69%)
:arrow_up:

.../java/org/opensearch/ad/util/RestHandlerUtils.java
86.44% <0.00%> (+1.69%)
:arrow_up:

...in/java/org/opensearch/ad/model/ADTaskProfile.java
51.69% <0.00%> (+2.96%)
:arrow_up:

.../main/java/org/opensearch/ad/cluster/HashRing.java
84.48% <0.00%> (+3.26%)
:arrow_up:

...rch/ad/transport/ForwardADTaskTransportAction.java
97.45% <0.00%> (+3.38%)
:arrow_up:

...va/org/opensearch/ad/feature/SearchFeatureDao.java
81.47% <0.00%> (+4.11%)
:arrow_up:

...c/main/java/org/opensearch/ad/util/ParseUtils.java
67.06% <0.00%> (+4.36%)
:arrow_up:

... and 3 more


YASL_pop should return void

It always succeeds, so returning an error code is useless.

on v0.11.0 branch


tests: kernel: Add test to verify k_thread_user_mode_enter()

Test to verify k_thread_user_mode_enter() when usermode is
not enabled or supported by architecture. The thread which calls
k_thread_user_mode_enter() with CONFIG_USERSPACE disabled or
architecture doesn't support usermode, should be marked
as user thread and if it is essential, then it has to be cleared.
Signed-off-by: Spoorthi K<EMAIL_ADDRESS>

@AdithyaBaglody @andrewboie: This test cases fails on qemu_x86, minnowboard and freedom_k64f, please refer the console log below.
Would you please review the test case and let me know if I need to raise a issue for these crashes?
On qemu_x86:
starting test - test_user_mode
***** CPU Page Fault (error code 0x00000005)
User thread read address 0x00401248
PDE: 0x027 Present, Writable, User, Execute Enabled
PTE: 0x80000000263 Present, Writable, Supervisor, Execute Disable
Current thread ID = 0x004181e0
eax: 0x000012ad, ebx: 0x004181e0, ecx: 0x00000000, edx: 0x004181e0
esi: 0x00000000, edi: 0x0040efec, ebp: 0x0040efd0, esp: 0x0040dff8
eflags: 0x00000246 cs: 0x002b
call trace:
eip: 0x00006b01
     0x000027bd (0x4181e0)
Fatal fault in thread 0x004181e0! Aborting.

On freedom_k64f:
starting test - test_user_mode
***** BUS FAULT *****
  Executing thread ID (thread): 0x20002870
  Faulting instruction address:  0x6366
  Precise data bus error
  Address: 0x200004a4
  NXP MPU error, port 3
    Mode: User, Data Address: 0x200004a4
    Type: Read, Master: 0, Regions: 0x8800
Fatal fault in thread 0x20002870! Aborting.

On minnowboard:
starting test - test_user_mode
***** CPU Page Fault (error code 0x00000005)
User thread read address 0x0010c248
PDE: 0x027 Present, Writable, User, Execute Enabled
PTE: 0x80000000263 Present, Writable, Supervisor, Execute Disable
Current thread ID = 0x001231c0
eax: 0x0010028d, ebx: 0x001231c0, ecx: 0x00000000, edx: 0x001231c0
esi: 0x00000000, edi: 0x00119fec, ebp: 0x00119fd0, esp: 0x00118ff8
eflags: 0x00210246 cs: 0x002b
eip: 0x001057b0
Fatal fault in thread 0x001231c0! Aborting.

And passes on qemu_cortex_m3, qemu_nios2, qemu_xtensa, arduino_101, quark_d2000_crb and quark_se_c1000_devboard.

I am wondering @spoorthik : is the test passing for any board having the ARM MPU?

@ioannisg : I have checked on freedom_k64f and on sam_e70_xplained. And it fails on both the boards.
To be specific, call to _is_thread_essential() after entering user mode is causing the crash.
Console log on sam_e70_xplained
starting test - test_user_mode
***** MPU FAULT *****
  Executing thread ID (thread): 0x2040323c
  Faulting instruction address:  0x40518e
  Data Access Violation
  Address: 0x2040047c
Fatal fault in thread 0x2040323c! Aborting.

And passes on qemu_cortex_m3, qemu_nios2, qemu_xtensa, arduino_101, quark_d2000_crb and quark_se_c1000_devboard.

That's because none of those boards enable user mode.

@andrewboie what is usage of k_thread_user_mode_enter() for kernel thread (if user space is not enabled)?
does it make sense to move the function under #ifdef CONFIG_USERSPACE? if not how can we test this API in kernel mode?

@andrewboie what is usage of k_thread_user_mode_enter() for kernel thread (if user space is not enabled)?

There isn't any use-case

@andrewboie: The intention of writing the test case is to improve the code coverage of k_thread_user_mode_enter() on native posix. Please help me understand how can we do that.

@andrewboie: The intention of writing this test case is to improve the code coverage of k_thread_user_mode_enter() on native posix. Please help me understand how can we do that.

native_posix doesn't support user mode so I don't see this as a particularly productive use of our time.

@andrewboie so how we should approach this from the testing point of view, should we leave this function as dead code in case non-userspace enabled platforms?

@andrewboie

There isn't any use-case other than very crude portability to targets that don't support user mode

then

native_posix doesn't support user mode so I don't see this as a particularly productive use of our time.

so the API can be called in native_posix, and should address the coverage problem, but placing it in this test project will not work they way you are doing it, the test itself should be run on non-userspace platforms only, i.e only native_posix in this case.
This qualifies as a test, it goes into branches that have not been tested/verified yet.

@nashif: I have modified the test so that the test can be called by the non-userspace platforms only. Is this what you were expecting? Can you please review.

Shippable is failing due to unrelated build error, it has nothing to do with the current patch:
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h: In function ‘LPUART_GetDataRegisterAddress’:
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h:500:30: error: ‘LPUART_Type {aka struct <anonymous>}’ has no member named ‘data’
     return (uint32_t) & (base->DATA);
                              ^~
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h: In function ‘LPUART_WriteByte’:
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h:602:9: error: ‘LPUART_Type {aka struct <anonymous>}’ has no member named ‘data’
     base->DATA = data;
         ^~
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h: In function ‘LPUART_ReadByte’:
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h:624:21: error: ‘LPUART_Type {aka struct <anonymous>}’ has no member named ‘data’
         return (base->DATA & 0x7F);
                     ^~
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h:628:20: error: ‘LPUART_Type {aka struct <anonymous>}’ has no member named ‘data’
         return base->DATA;
                    ^~
/home/buildslave/src/github.com/zephyrproject-rtos/zephyr/ext/hal/nxp/mcux/drivers/imx/./fsl_lpuart.h:633:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^
cc1: all warnings being treated as errors

@andrewboie @nashif: Would you please review the changes and let me know what modifications are required for the test?

recheck

recheck


flow --colors

That'd be nice to be able to do something like:
flow --colors | less

Because right now, piping cause the colors to be disabled (as expected from well-behaved UNIX tools):
flow | less

added in 0.12!
flow --color=always (or never, or auto)
the flag is missing from some commands in 0.12, like check, but that will be fixed in 0.13.
(btw, for your example you probably want less -R so it shows the colors instead of the ANSI escape characters)


Resource Types

This adds more kinds of materials. It is part of the Economy v2 initiative (#32). This closes #30. This PR also squashes the previous migrations, in accordance with #33. It also removes legacy properties that are going to be majorly changed later on, like population and army units.

[x] Update schema
[x] Squash migrations
[x] Update query files
[x] Update server API
[x] Update frontend UI
Any other final changes

[x] Distribute income message
[x] Member page

Looks good! :D


Error message when not in Deep Dungeon

Mob Id
13429 - Ousted Asterodia
Content Type
DeepDungeon
What threat level sould the mob have
None
What agro type does the mob have
None
Vulnerabilities for the enemy

[ ] Stun
[ ] Heavy
[ ] Slow
[ ] Sleep
[ ] Bind
[ ] Undead

Additional mob text guide
These mobs are ion Heritage found, so this should not be displayed outside of DD

If you don't want to see these, configure the plugin.


Create OS landing zone management group

To be consistent with Soveriegn Landing Zone, have a specific LZ mgmt group

Added policy to example "OS" management group- Only approved VM extensions should be installed
Added initiative for Configure machines to automatically install the Azure Monitor and Azure Security agents on virtual machines


Revert "Add support to create realtime segment in local (#10433)"

This reverts commit 16cae5d9c630dd247fb037d4c0e7a18a8374178c.
Instructions:

The PR has to be tagged with at least one of the following labels (*):

feature
bugfix
performance
ui
backward-incompat
release-notes (**)

Remove these instructions before publishing the PR.

(*) Other labels to consider:

testing
dependencies
docker
kubernetes
observability
security
code-style
extension-point
refactor
cleanup

(**) Use release-notes label for scenarios like:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

reason for revert?

https://github.com/apache/pinot/pull/10433#issuecomment-1472457463


Missing System Center Components

Hello,
My organization is working on PS DSC configurations over at @d9micromgmt i'd like to know if there are existing code bases for the missing system center dsc resources, such as config manager?

Hi @flyboarder. I'm also interested in ensuring the remaining System Center components have Resources.
In particular Orchestrator currently looks like an omission @kwirkykat do you have any info on this?
Perhaps we can all work together on this. Thank you.
Jonathan

@jpritchard @flyboarder I don't have any more System Center resources other than what is already posted (xSCDPM, xSCOM, xSCSMA, xSCSPF, xSCSR, xSCVMM). But if you'd like to submit a new module, I would be glad to add more System Center resources to the DSC Resource Kit.

Thank you Katie. Good to know, just didn't want to duplicate work. I'll start looking into it. Not done anything like it before though :-)
-----Original Message-----
From: "Katie Keim"<EMAIL_ADDRESS>Sent: 19/09/2016 23:48
To: "PowerShell/DscResources"<EMAIL_ADDRESS>Cc: "jpritchard"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Subject: Re: [PowerShell/DscResources] Missing System Center Components (#30)
@jpritchard @flyboarder I don't have any more System Center resources other than what is already posted (xSCDPM, xSCOM, xSCSMA, xSCSPF, xSCSR, xSCVMM). But if you'd like to submit a new module, I would be glad to add more System Center resources to the DSC Resource Kit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I suggest we close this issue since any new resource modules should be submission-ed to this repository if a resource module owner wants it added to the DSC Resource Kit.

Closing this as solved as per author comment and my previous comment.


Proposal: go keyword returns <-chan struct{} that closes when goroutine ends

I would like the calling code to have the optional ability to determine when the called goroutine is finished. For example, here the main function does some preparation, giving part of the tasks to the goroutines. To determine the moment of their completion, you need to create a channel and pass it to the goroutine:
package main

import (
	"fmt"
)

func main() {
	done := make(chan struct{})
	go fn(done)
	// do something
	<- done
	fmt.Println("fn is done!")
}

func fn(done chan<- struct{}) {
	// do something
	close(done)
}

I would like the go keyword to be able to return the simplest channel with an empty structure, which will close when the goroutine finishes:
package main

import (
	"fmt"
)

func main() {
	done := go fn()
	// do something
	<- done
	fmt.Println("fn is done!")
}

func fn() {
	// do something
}

For backward compatibility, please add this syntax element to the language, and leave the old one as it is. The new approach simply won't compile with the 1.x compiler.

Please fill out https://github.com/golang/proposal/blob/master/go2-language-changes.md when proposing language changes

Duplicate of #26287 #217


Apply BCROSS_DIODE_SKY aka HV phase corrections

This is also known as "Phase Two of Polarisation Calibration". It does both Phase 2A (BCROSS_DIODE: HV phase derived from noise diode) and 2B (BCROSS_DIODE_SKY: noise diode HV phase transformed to sky).
Fire the noise diodes as part of phase-up (both initially and during verification). This triggers the cal pipeline to produce KCROSS_DIODE and BCROSS_DIODE_SKY solutions. Get those solutions from telstate and combine them with the K and B solutions, respectively.
Also do a few tweaks to the script logger output to synchronise this script with calibrate_delays, and remove somewhat spurious logs.
This addresses JIRA tickets SR-1940 and SR-1980.

Yes!
The one thing to look out for: the noise diode will now also fire during phase-up, resulting in larger noise and higher signal levels when solving for gains. I'm interested whether this has any negative impact on the beamformer.

I have now rolled BCROSS_DIODE_SKY support into this PR, since it is only a few extra lines of code (most of the action has shifted to the cal pipeline). This will simplify deployment, since we have to roll out a triple whammy of new pipeline (SDP, ska-sa/katsdpcal#2) + new capture session (CAM, ska-sa/katcorelib#339) + new script (OBS, this PR)...
Please have a look at the last two commits:

The merge with master added narrowband support (capture-start all streams and not just correlator).
The final commit adds the BCROSS_DIODE_SKY bit.

Sure, what's the use case? And does that imply not firing the noise diode?

@ludwigschwardt, no we keep the noise diode firing et al. we just do not apply the corrections. With this option it is going to be easier for me to do verification tests.

So let me summarise the suggested changes:

Add a --disable-hv-correction script option that still fires the noise diode to get CROSS_DIODE solutions in telstate, but don't apply them to F-engine gains
If no BCROSS_DIODE_SKY is found (like on current UHF / S-band), fall back to BCROSS_DIODE instead of doing no HV correction.

Does that sound OK?

Yes, exactly that.

@mserylak, have a look at the latest commit to check that it does what you want.


Can't be used in tests

I currently get this error: Uncaught Error: Compile Error: import is not a helper

Sounds like the build time transform isn't running - the addon should transform the import syntax into a {{component invocation, which means import should never appear in the compiled template.
What's the source file that has the import in it?

Something like:
import { module, test } from 'qunit';
import { setupRenderingTest } from 'ember-qunit';
import { render } from '@ember/test-helpers';
import hbs from 'htmlbars-inline-precompile';

module('Integration | Component | profile/intercom', function(hooks) {
  setupRenderingTest(hooks);

  test('it renders', async function(assert) {
    // Set any properties with this.set('myProperty', 'value');
    // Handle any actions with this.set('myAction', function(val) { ... });

    await render(hbs`
      {{import Intercom from "profile/intercom"}}
      <Intercom />
    `);

    assert.equal(this.element.textContent.trim(), '');
  });
});

Ah, that makes sense. Yea, inline hbs won't be preprocessed I think? Not sure of an elegant way to solve this one to be honest. If the inline precompiler skips the CLI preprocessor registry, we'll probably have to manually search test files for the hbs tag and do a find/replace that way.


how to solve problem of EncryptedExoPlayerDemo with more than 2GB files ?

Hi
Recently I used EncryptedExoPlayerDemo to play encrypted files it has no problem in playing files with less than 2GB volume but unfortunately, I couldn't play more to 2GB files.
Is there any solution for solving this problem.
Thanks so much.
error :
Source error.
com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, FlvExtractor, OggExtractor, PsExtractor, WavExtractor) could read the stream.

@Navidhp I saw on the exoplayer hub that you solved the issue - do you mind posting your solution?
Thanks!

@hardcodet Sorry for the delay, please rewrite this repository on your project and follow this :
1- find this method computeBytesRemaining
2- If you pay attention to this method, you will notice that it calculates the remaining bytes, but the amount that it can finally calculate is equal to (Integer.MAX_VALUE), That's why it can't read larger files and I changed the inside of the method
private void computeBytesRemaining(DataSpec dataSpec) throws IOException {
        File file = new File(dataSpec.uri.getPath());
        if (dataSpec.length != C.LENGTH_UNSET) {
            mBytesRemaining = dataSpec.length;
        } else {
            mBytesRemaining = file.length() > Integer.MAX_VALUE ? file.length() : mInputStream.available();
            if (mBytesRemaining == Integer.MAX_VALUE) {
                mBytesRemaining = C.LENGTH_UNSET;
            }
        }
    }

And now it works


pkg/option: add metrics-label-filter flags

The commit is to add the missing metrics-label-filter flag.
fix: #1667

@lambdanis
Checking the documentation, I can't seem to find any information related to filter.
Will it be added later?

Checking the documentation, I can't seem to find any information related to filter.
Will it be added later?

@Jack-R-lantern I'm afraid the search in the docs might not work properly. This option is documented in the Helm chart docs, but only there. Do you have suggestions on how to document it better?

Checking the documentation, I can't seem to find any information related to filter.
Will it be added later?

@Jack-R-lantern I'm afraid the search in the docs might not work properly. This option is documented in the Helm chart docs, but only there. Do you have suggestions on how to document it better?

I think the values.yaml file in the helm chart is sufficient, too
Even for 'standalone', I think "--help" is sufficient.


Album art and progress do not show up

For some reason, the album and progress bar did not show up in my computer event I try to restart it a couple of time.

Can confirm. Progress bar and album art do not appear. However, this has been the case for the past couple of releases and has been acknowledged by the developer.

That is correct. It should be fixed once I get around to fixing spotify issues. The only things that work right now are the song title and playback controls

Idk if it was already fixed or not but my album art isn't showing, neither progress bar. Maybe something changed about spotify?

Spotify is still being worked on

Should now be working, fixed it in #32. Still some issues, I've posted a discussion here #39


How can I remove cluster in the config file and take effect immediately without restarting clickhouse ？

Hi, I come across some troubles recently. I removed one cluster from the config file, but the cluster was still available for the distributed table. When I restarted clickhouse-server, everything back to normal. The same situation occurred while removing the dictionary file. As the official documentation says Clusters from config are updated on the fly, it does not require server restart， adding cluster and dictionary can work well without restarting server while removing was not ok. what should I do to solve the problem ? 😄

External dictionaries reload will be fixed in this pr https://github.com/yandex/ClickHouse/pull/2779.

Added removing clusters.


[BUG] Cython doesn't understand inplace build mechanics when searching for __init__.pyd

Describe the bug
See also numpygh-25135 at least for in-place builds with a hook, Cython cannot find the __init__.pyd.
Now, to some degree, that is not surprising and maybe just expected, because sys.path doesn't include the module unless it is imported first.
(The issue creator says there is some other path a wrong __init__.pyd might be found, but I am not sure.)
Solution(?)
I suspect, the solution has to be to add an explicit import (e.g. import numpy) if a development/inplace install is hooked (i.e. when a <top-level-module>-editable.pth is found).
(Or whatever is exactly the normal import machineries behavior here.)
Additional context
This isn't high priority at least for me; I taped it over in the NumPy tests.  And I doubt there are many who use in-place builds of NumPy to then build a different project.
(Maybe our in-place build setup could also be improved to help Cython.)

I just realized that there is also a way to install zips of a package loaded via zipimport.
Not dealing with that (if that is the case, I have not checked) could be a bigger issue in practice than the editable install issue.

Copying a comment from elsewhere, The heuristic used to find include files is:

user provided include_directories
sys.path
Cython/Includes

There are situations like editable installs where a project's top-level __init__.pxd is not on sys.path, and the only reliable place to find it is via import <project>; <project>.__file__

One way to change this from a potentially confusing error (where the duplicate files get out of sync) to a clear error would be to drop the Cython include files for NumPy. The Numpy ones have been pretty stable since NumPy 1.20 or so (released in Jan 2021), which was the first release with the Cython3.0-specific __init__.cython-30.pxd. Going forward (cython3.1) maybe we could set a minimum numpy requirement to 1.20 ?


Enhancement - Validation Rule/Code Check make sure Relationship Criteria fields is populated when Relationship Criteria isn't blank

Maybe a validation rule that checks if you have Relationship Criteria to make sure you fill in your Relationship Criteria Fields.
AND(
NOT(ISBLANK(RelationshipCriteria__c)),
ISBLANK(RelationshipCriteriaFields__c)
)
Error message top of page
If you have Relationship Criteria please put the fields in that criteria into the Relationship Criteria Fields.

Great idea thanks!

I'm not convinced this is always a good idea to enforce.
What if the Relationship Criteria Fields may not actually be necessary in every single criteria is, for instance, related to the parent record and not the child (I know it is a bad idea and could really only be kept in sync via scheduled calculated)
I'll leave this open for discussion but I suspect we won't be able to guarantee this is always necessary and people would have to put in redundant values like the Relationship Field, which is already included in the field set.

I'm going to close this. I still don't think it would be ideal to mandate Relationship Criteria Fields in this case.
We will be making some improvements in the future to make that field more automatic and less error prone.


The leaderboard is missing in https://paperswithcode.com/sota/speech-recognition-on-common-voice-8-0-sakha

We found the leaderboard is removed by the officials; thus, we decide to close this issue for now.


cant see if random music button is enabled

Did you read the README?

[X] I have read the README, the theme is installed/configured correctly and the "Troubleshooting" section contains no information related to my problem

Do you have the latest versions of Spicetify, Spotify and Bloom?

[X] I'm using the latest versions of Spicetify, Spotify and Bloom

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Is your problem definitely caused by the theme?

[X] I have checked the app behavior without the theme and it is different

Environment & Computer Info
- Spotify version: newst avaible with spicetiy
- Installed from: (Spotify site, )
- Spicetify version: newest
- Color scheme:
- Extensions:
- Custom apps:
- Bloom installation method: (Spicetify Marketplace)

Description
cant see if random music button is enabled when its clicked
Screenshots

fixed by @sanoojes ! Thanks!!


Complete "Outreach" form

Work Completed

Added the remaining "responseDetails" and "clientWork" elements to the "Outreach form.
Elements were added in the same manner as the existing elements to create a longer form.
Headings were also added to separate between contact, response, and client work.

Ideas for further development (Updated 10/08/2023)

Add a check for whether the client was contacted before adding response or work details.
ResponseYes is now currently a checkbox.  Consider the response to be a drop down of "Yes", "No", and "Maybe Later".  This might give the user a better indication of whether they should reach out to the client later.

It looks like we both made PRs at the exact same time, so there are still (different) conflicts unfortunately. If you wouldn't mind resolving again, I'll go ahead and merge afterward! I really appreciate your help!

Sounds good. I'll take another look later today when I have some time.


Remote notification completion handler

Motivation
To allow handling of remote notifications using the preferred delegate application:didReceiveRemoteNotification:fetchCompletionHandler.
React native right now is setup to use application:didReceiveRemoteNotification which doesn't give the engineer the option to call back to the app via a completion handler and inform the app that it is done with handling the push notification.
The primary reason to use the fetchCompletionHandler would be to wake up the app to fetch updates so the next time the user opens the app it will be up to date.
A new method will be exposed: PushNotificationIOS#finish which can be called for remote notifications.
Test Plan
The new flow would be something like this:
Set up the app to receive a notification and then register for _onNotification listener
PushNotification.addEventListener('notification', this._onNotification);

Trigger a remote notification and then handle it like so:
_onNotification(notification) {
  // handle notification, and then ...
  notification.finish(PushNotification.FetchResult.NewData);
}

By analyzing the blame information on this pull request, we identified @nicklockwood and @slycoder to be potential reviewers.

cc @javache

@JAStanton Will this change allow for updating the icon badge when a notification comes in and the app is in the background? I've noticed my app badge only increments upon opening the app but not when the remote notification comes in and I'm on the home screen or another app. Thanks.

@fgaleano that's not something I set out to fix. But I don't recall it being an issue before and using my app now that relies on this PR it definitely works. Sorry for the wishy washy answer it's just hard for me to test the old behavior now that i'm so far down this path :P

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @nicklockwood as a potential reviewer. Could you take a look please or cc someone with more context?

Any updates on this? @javache

@JAStanton updated the pull request.

@JAStanton updated the pull request.

Alright @javache I updated this PR with your comments

Any movement?

@javache any movement?

Looks good, just a couple of style nits

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@JAStanton updated the pull request - view changes

@javache I'm not a Flow user, I attempted to fix https://circleci.com/gh/facebook/react-native/10648 (https://github.com/facebook/react-native/pull/8040/commits/734ef9758defdcea21f9035b887bcf1f58f2b2e3) with https://github.com/facebook/react-native/pull/8040/commits/db74ec897413d6c9f227c1257464f01f46b0e258 but I ended up getting linter errors. I know it must be simple, but how do I put a typed static that wont blow up the website documentation generator?

@bestander / @hramos Can you help here?

Yeah, the website EcmaScript parser is quite outdated and we start stepping on it more and more often.
cc @lacker
Flow issue should be possible to fix by declaring a static on the class.
In the PR you are assigning static FetchResult field to a class that does not have it declared.

@bestander

Flow issue should be possible to fix by declaring a static on the class

I originally declared a static on the class: https://github.com/facebook/react-native/pull/8040/commits/734ef9758defdcea21f9035b887bcf1f58f2b2e3 but I got this error: https://circleci.com/gh/facebook/react-native/10648
Thats when I changed up the syntax a bit https://github.com/facebook/react-native/pull/8040/commits/db74ec897413d6c9f227c1257464f01f46b0e258 but i ended up getting the current failures https://circleci.com/gh/facebook/react-native/10653

There is internal task raised to update the ES Parser 12652958.
Meanwhile, @JAStanton, could you experiment with the static FetchResult?
Maybe if you split definition and assignment it will pass

The website doc generator is updated so it should work with statics on the class now. Let me know if that is not the case

Travis looks OK, it looks like that run was canceled rather than failing. So I think this should be good to go - there's a merge conflict though. Can you fix that conflict and then I will ship this?

@lacker alright updated, now lets wait a few hours to see if CI runs successfully :P

Eh, Travis is failing but it looks unrelated - there is some logging test that's failing on tvOS. @bestander what would you normally do in this situation?

Tests look green now

@lacker @bestander :shipit: ?

Import failed:
Libraries/FBReactKit/js/react-native-github/Libraries/PushNotificationIOS/RCTPushNotificationManager.m:261:17: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security]
    RCTLogError([NSString stringWithFormat:
    ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
Libraries/FBReactKit/js/react-native-github/React/Base/RCTLog.h:35:52: note: expanded from macro 'RCTLogError'
#define RCTLogError(...) _RCTLog(RCTLogLevelError, __VA_ARGS__)
                         ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
Libraries/FBReactKit/js/react-native-github/React/Base/RCTLog.h:127:74: note: expanded from macro '_RCTLog'
#define _RCTLog(lvl, ...) _RCTLogNativeInternal(lvl, __FILE__, __LINE__, __VA_ARGS__);
                                                                         ^~~~~~~~~~~
Libraries/FBReactKit/js/react-native-github/Libraries/PushNotificationIOS/RCTPushNotificationManager.m:261:17: note: treat the string as an argument to avoid this
    RCTLogError([NSString stringWithFormat:
                ^
                @"%@",
Libraries/FBReactKit/js/react-native-github/React/Base/RCTLog.h:35:52: note: expanded from macro 'RCTLogError'
#define RCTLogError(...) _RCTLog(RCTLogLevelError, __VA_ARGS__)
                                                   ^
Libraries/FBReactKit/js/react-native-github/React/Base/RCTLog.h:127:74: note: expanded from macro '_RCTLog'
#define _RCTLog(lvl, ...) _RCTLogNativeInternal(lvl, __FILE__, __LINE__, __VA_ARGS__);
                                                                         ^
1 error generated.

And so it continues.

@lacker fingers crossed.

That Travis failure looks like nonsense. I'm just gonna shipit.

@facebook-github-bot shipit

139 days later!

Thank you all so much!! You wouldn't believe the nonsense I've been trying to work around this.

:) glad to hear it!

I've just tried this on on 0.38.rc and i've kind of fallen at the 1st hurdle... My AppDelegate has this:
-(void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)notification fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler {

NSLog(@"push-notification with content body => %@", notification);

  [RCTPushNotificationManager didReceiveRemoteNotification:notification fetchCompletionHandler:completionHandler];
}

My I'm just seeing the red error screen with
ExceptionsManager.js:71 RCTJSONStringify() encountered the following error: Invalid type in JSON write (__NSMallocBlock__)
And the notification that gets delivered to my JS code is empty (all the fields have undefined values).
I can see the notification being delivered the the AppDelegate correctly.
Any idea what I might be doing wrong?

I think I found the problem - the wrong object is being sent
https://github.com/facebook/react-native/pull/10860

Hi.  Still having issues getting this to work end-to-end.  I've patched my code with #10860 and I'm calling [RCTPushNotificationManager didReceiveRemoteNotification:notification fetchCompletionHandler:completionHandler]; but I see the following in the iOS logs:
Warning: Application delegate received call to -application:didReceiveRemoteNotification:fetchCompletionHandler: but the completion handler was never called
I've implemented my JS to call the new finish method, but it's not being given a chance to run - the log message above happens immediately.
I'm not exactly sure but i think the problem might be that didReceiveRemoteNotification needs to either call the completion handler in-process (which isn't possible here because of the async call to the JS code), or it needs to start a background task in which we call the completion handler.  The background tasks keeps the app from being suspended for up to 30s.  When the JS code calls finish the task can call the completion handler and then end.
https://forums.developer.apple.com/message/96731#96731


Landing page and active menu toggle

Included new landing page section with toggle button and text responsive to the viewing window.
Added JS code to toggle when sidebar menu tabs are clicked on and active.
Added extra paragraph content in about me, education, and experience sections.

Ready for re-review!


token=null

i have problem run cezerin on centos 7 install mongodb node npm nginx and follow on install ubuntu
but can't login in admin page , when opening /admin this error show in console
plz help
app-3fbed4893b8cb0a4d0a1.js:1 WebSocket connection to 'ws://<IP_ADDRESS>:3000/ws/dashboard?token=null' failed: Connection closed before receiving a handshake response
u @ app-3fbed4893b8cb0a4d0a1.js:1
t.connectToWebSocket @ app-3fbed4893b8cb0a4d0a1.js:1
(anonymous) @ app-3fbed4893b8cb0a4d0a1.js:1
i @ app-3fbed4893b8cb0a4d0a1.js:1
n @ app-3fbed4893b8cb0a4d0a1.js:1
(anonymous) @ app-3fbed4893b8cb0a4d0a1.js:1
(anonymous) @ app-3fbed4893b8cb0a4d0a1.js:1

+1

did you set developerMode: false yet?
set this will prevent authorizing when go to admin page

+1

Can we change this type of authentication to old traditional one? Using email and password to login

Can we change this type of authentication to old traditional one? Using email and password to login

yes, I am changing it, but i face to many bug


Improve UI design

Things added:

Icons for each type of information displayed on PersonCard
Borders for grouping of each type of information
Updated display of NOK information after get /nok command

Complete #106.

Codecov Report
Base: 64.10% // Head: 63.77% // Decreases project coverage by -0.33% :warning:

Coverage data is based on head (3f77a9b) compared to base (6b6fc0d).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #132      +/-   ##
============================================
- Coverage     64.10%   63.77%   -0.34%
  Complexity      666      666
============================================
  Files           117      117
  Lines          2304     2316      +12
  Branches        289      289
============================================
  Hits           1477     1477
- Misses          717      729      +12
  Partials        110      110

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/ContactCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonViewPanel.java
0.00% <0.00%> (ø)

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

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


Rewrite - Test branch in progress

After the various discussions about streamlining and speeding up powerlevel9k - especially with precompiling as much as possible and removing tests wherever possible, I have updated the entire code base (based on #771).
This is a WIP as it does not now have a rewritten zsh-async.p9k yet, but it seems to be a whole lot faster.
@bhilburn @dritter Please give it a try and let me know what you think.

I haven't updated any of the tests, as I want to know what your comments are first before I continue with the work.

#817 - Something like this?

@onaforeignshore - Holy crap! This is a ton of work. Right off the bat, I can tell you I'm seeing a significant improvement in performance relative to the current HEAD of next, but it appears to actually be slower compared to the work in #771 on my machine:
current HEAD of next:

current HEAD of this dev branch:

553c788e8382 from #771:

You can also see there are some minor hiccups with colors and icons, but I realize this is all WIP.
So, just to make sure I understand, this branch is built on top of #771, and is basically meant to apply the discussion from #817 regarding Speed vs Flexibility to the code base, right?
I haven't reviewed the code as thoroughly as @dritter has, but everything I've looked at so far is much better organized and easier to understand.
@onaforeignshore - What kinds of performance numbers are you getting on this branch relative to #771? I wonder if something is just wonky with my configuration / machine.

@bhilburn – These are my response times:
Rewrite:
<EMAIL_ADDRESS>Code Separation:
<EMAIL_ADDRESS>We may need to get a few people to respond to this to find out an average…
From: Ben Hilburn<EMAIL_ADDRESS>Reply-To: bhilburn/powerlevel9k<EMAIL_ADDRESS>Date: Wednesday, 23 May 2018 at 17:38
To: bhilburn/powerlevel9k<EMAIL_ADDRESS>Cc: My accounts<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [bhilburn/powerlevel9k] Rewrite - Test branch in progress (#833)
@onaforeignshorehttps://github.com/onaforeignshore - Holy crap! This is a ton of work. Right off the bat, I can tell you I'm seeing a significant improvement in performance relative to the current HEAD of next, but it appears to actually be slower compared to the work in #771https://github.com/bhilburn/powerlevel9k/pull/771 on my machine:
current HEAD of next:
[screen shot 2018-05-23 at 09 27 48]https://user-images.githubusercontent.com/569571/40427429-b3888512-5e6b-11e8-9298-0f8261a023ee.png
current HEAD of this dev branch:
[screen shot 2018-05-23 at 09 27 54]https://user-images.githubusercontent.com/569571/40427447-c078dea2-5e6b-11e8-8079-7cf02e195d62.png
553c788https://github.com/bhilburn/powerlevel9k/commit/553c788e8382a222463a1594d6a99c62de19a737 from #771https://github.com/bhilburn/powerlevel9k/pull/771:
[screen shot 2018-05-23 at 09 30 56]https://user-images.githubusercontent.com/569571/40427603-2c82c324-5e6c-11e8-9fd1-5fa69f45820c.png
You can also see there are some minor hiccups with colors and icons, but I realize this is all WIP.
So, just to make sure I understand, this branch is built on top of #771https://github.com/bhilburn/powerlevel9k/pull/771, and is basically meant to apply the discussion from #817https://github.com/bhilburn/powerlevel9k/issues/817 regarding Speed vs Flexibility to the code base, right?
I haven't reviewed the code as thoroughly as @dritterhttps://github.com/dritter has, but everything I've looked at so far is much better organized and easier to understand.
@onaforeignshorehttps://github.com/onaforeignshore - What kinds of performance numbers are you getting on this branch relative to #771https://github.com/bhilburn/powerlevel9k/pull/771? I wonder if something is just wonky with my configuration / machine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bhilburn/powerlevel9k/pull/833#issuecomment-391349737, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAq_lsDhvD7Q91hz8OcriHA7iVcQ64aSks5t1WZcgaJpZM4T56UW.

I will stop pushing updates to this until @dritter has finished his merge and then I'll base this on that

Superceded by #875. Closing


Update RISCV prebuild toolchains to support manylinux_2_28_x86_64

Toolchains are built from the same source as
https://github.com/openxla/iree/commit/a7a784e1f but under a manylinux_2_28_x86_64 docker to enable more contributors.
CI riscv docker is updated with the new toolchains.
Also check in the riscv_toolchain_builder docker file as a reference.

@ElEHsiang FYI


Use checkpoint for distributed-sequence.

Use checkpoint for distributed-sequence.

Thanks! I'd merge this now.


Upgrade karma to 3.0

Related to #969

Coverage remained the same at 94.892% when pulling 53e1dd2d6718ebbcee09b06e1545fc6421d8b50f on stevenmusumeche:task/upgrade-karma into 9f7af1676a46048d6c39270b4898b486e1ebdde2 on FormidableLabs:master.


Fix hardcoded string

https://github.com/schmich/marinara/blob/master/src/options/router.js#L47

@schmich I use this plugin and want to start contributing. What do you want to do in this issue? Move it to a constant or...?

Welcome, @novito!
Marinara is localized to a bunch of different languages. You can check out the various messages.json files under the package/_locales folder.
We load localized strings through the chrome.i18n API which we wrap in the generated Messages.js file. This class gets pulled into various places as import M from '../Messages';. Then, using localized strings is just M.some_localized_string or, if it is parameterized, M.some_parameterized_string(value, param, whatever).
In this case, I think we just need to reference M.app_name_short instead of using the hardcoded "Marinara" string.

If this is still open and there's no activity going on, I'm happy to pick this up, that ok?

@ryan-nanson all yours.

Fixed with 1334c7bc961378dd97b2660dcf6ae5d867bc7369.


docs(readme.md): Update etherscan API URL and add diamond contract usage

To specify how to use a diamond.
Address potential confusion, I encountered while verifying.

thanks!


Feature/update thank you page [LEI-240]

Refs: https://openscience.atlassian.net/browse/LEI-240
Purpose
Update thank-you page:
Summary of changes

Center and enlarge buttons
Add a picture

Done making those changes @abought! 🍂

Made that change @abought! 🎃

These changes should help future refactoring; thanks.
Ultimately the decision to merge this one was yours, so since you indicated it's ready, in we go!


Dataset download link not working

The link 'http://www.timeseriesclassification.com/Downloads/Archives/Multivariate2018_ts.zip', which is provided under utils.Data_Verifier line 92, appears to be inoperable. Could you kindly provide the dataset download link from a different source?
EDIT
For manual download: 'http://www.timeseriesclassification.com/aeon-toolkit/Archives/Multivariate2018_ts.zip'

Yes, please refer this link 'http://www.timeseriesclassification.com/aeon-toolkit/Archives/Multivariate2018_ts.zip' for download the UEA dataset.


Debug - not able to get value of dereferenced variable

This may be me not knowing the gide debugger. I've tried follow ptr. If I have code like this:
for i := 2; i <= *np; i++ {
	var sum float32 = 1.0
	c[i] = math32.Cos(2.0 * math32.Pi * float32(i-1) / float32(n))
	x := (1.0 - c[i]) / 2.0
	y := x

I'd like to see the value of *np. Here is what I see in GoLand, notice value after address.

I tried to look at this case -- it should work!  anyway, looks like some other files are needed:
2020/03/04 00:54:04 open VocalTractEnglishPhones.dat: no such file or directory
bad access
File not found or error opengin file: VocalTractEnglishPhones.dat (open VocalTractEnglishPhones.dat: no such file or directory)2020/03/04 00:54:04 open VocalTractEnglishPhones.dat: no such file or directory

I checked in the data file so you can run to a breakpoint but there is a crash beyond the example code that I will fix so you might as well wait.

what is the status of this one?

Working


ablation experiment question

Thanks for your excellent work, you have tried a lot of benchmark and ablation experiment, that's very nice of you. and I have a question, I was just wonder if you train the semi- and self supervised model in the same steps (200k)? Because I notice train moco or pseudo label in a larger steps may have a good result. so I was just wondering if you have contrast the result in 200k steps, 300k steps.

Hello,
Thank you for your interest in our work :)

if you train the semi- and self supervised model in the same steps (200k)?

Yes, I did.

if you have contrast the result in 200k steps, 300k steps.

Yes, I did.
In our experiments, it was up to the experimental setting (using augmentation or semi/self-supervised learning).
Sometimes 200K is better, and sometimes 300K is better.
We thought that it does not make a significant difference that can change our conclusion.
Thus we used 200K steps for faster training.
If the number of steps makes a significant difference and insight,
and if someone can find some law or theory between iteration and performance,
it would be interesting work!
(we had tried to find some law or theory, but it was not easy to conclude)
Hope it helps :)
best.


BUG: Fix mantel bootstrap on some archs

Please complete the following checklist:

[X] I have read the guidelines in CONTRIBUTING.md.

[X] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[X] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

I believe this should fix the issues noted in #1785

Looks like this is going to work

Doesn't this add a significant overhead to the compute?

@sfiligoi, no it's the difference in 999 permutations and 1000. That's worthwhile for correctness. If you have a better way to refactor, I am happy to use that instead. But I will insist that our test suite passes. Otherwise we can stick with scipy.pearson.

Priority 1 is tests passing. Priority 2 is any addressing fixes if necessary to optimized code

@sfiligoi, no it's the difference in 999 permutations and 1000. That's worthwhile for correctness. If you have a better way to refactor, I am happy to use that instead. But I will insist that our test suite passes. Otherwise we can stick with scipy.pearson.

Ah, yes... now I understand it better.
Sure, no objections from my side.

Slightly nicer implementation in https://github.com/biocore/scikit-bio/pull/1787/commits/bb4c04b7b039106d2dc6c5ff26b8d284878deb4f

Tests have passed, ready for merge if reviewers agree.