Add RuntimeEnv::try_new and deprecate RuntimeEnv::new

Which issue does this PR close?

Closes #12554
What changes are included in this PR?

Move existing functionality of RuntimeEnv::new to RuntimeEnv::try_new
Replace all calls to RuntimeEnv::new in code and documentation to use RuntimeEnv::try_new

Are these changes tested?

No extra tests required. Ensured that compilation, existing tests, and doc tests pass.
Are there any user-facing changes?
None.

Thanks again @OussamaSaoudi

Thanks for the clear and beginner friendly issues @alamb! This was a great way to get introduced to the code, and I'm looking forward to contributing more :D

Thanks for the clear and beginner friendly issues @alamb! This was a great way to get introduced to the code, and I'm looking forward to contributing more :D

You are welcome -- and I am glad the tickets fulfilled their intended purpose


Linking TBB to DLIB

Hi! I am not an expert in CMake and Build systems. I am currently trying to test the performance of DLIB's face detection with/without TBB. My initial code has taken inspiration from the following file scan_fhog_pyramid.h. Now, the tricky part is linking TBB with the program. I added TBB to the external directory and added the following lines of code into dlib/dlib/CMakeLists.txt file.
# TBB support
# Try to find TBB
include_directories(external/tbb/include)
set(dlib_needed_libraries ${dlib_needed_libraries} external/tbb/lib/tbb.dll)
set(REQUIRES_LIBS " tbb")

I was roughly testing and I might not be doing it right. Please comment if this is wrong.
Now, I build the examples of dlib and the first phase of building dlib is going fine. The second phase of building project face_detection_ex is getting the following error initially.
Project: face_detection_ex
File: scan_fhog_pyramid.h
Line: 13
C1083: Cannot open include file: 'tbb/tbb.h': No such file or directory

With simple tweaking of Project properties of face_detection_ex inside Visual Studio, I included the directory to TBB include files into my current project. I am able to suppress the error I mentioned but now I am getting the strange error related to TBB support with Windows version
1>------ Build started: Project: face_detection_ex, Configuration: Release x64 ------
1>face_detection_ex.cpp
1>c:\users\srira\dlib\dlib\external\tbb\include\tbb\internal\_tbb_windef.h(28): fatal error C1189: #error:  TBB is unable to run on old Windows versions; _WIN32_WINNT must be 0x0501 or greater.
1>Done building project "face_detection_ex.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 2 up-to-date, 0 skipped ==========

This doesn't make sense to me because I am using Windows 10 OS and the 0x0501 represents Windows XP. I am pretty sure I might be doing something wrong with respect to building the program. I tried testing integrating TBB (not with dlib) into a different file and everything works fine without errors. I am getting error only here.

Hello,
I have ran into similar problem when trying to build dlib with cpprestsdk. The problem for me was in

dlib\windows_magic.h

on lines:
#undef _WIN32_WINNT
#define _WIN32_WINNT 0x0500
#endif

After I commented out those lines, I was able to build and I haven't ran into an issue yet ... However, similarly to you I am using Windows 10 so I have no idea why those  lines were even executed.

Thank you @xbukovcikova ! It works now 👍  After a bit of research online, I figured that version 0x0500 refers to Windows Vista and TBB only supports from Windows XP (0x05001) and above. Also by changing the above code to this will also work.
#if _WIN32_WINNT < 0x0500
#undef _WIN32_WINNT
#define _WIN32_WINNT 0x0501
#endif


Filter conditions need to be according to numeric for Rate column in Tax Rates

Fixed.

Framework Version: 0.1.6

Fixed


Add a parameter to tell dev-server which network interface take the address from

What problem does this feature solve?
Under Windows and having Docker installed the dev-server takes the network address to initialize from the Docker's interface (DockerNAT), which doesn't works.
  App running at:
  - Local:   http://localhost:8082/
  - Network: http://<IP_ADDRESS>:8082/

What does the proposed API look like?
Tracked the issue to<EMAIL_ADDRESS>this line:
// This can only return an IPv4 address
lanUrlForConfig = address.ip()

it's returning <IP_ADDRESS> wich is the address for DockerNAT
Need to pass the interface's name to address.ip()
serve.js
<EMAIL_ADDRESS>line ~18
options: {
  '--open': `open browser on server start`,
  '--copy': `copy url to clipboard on server start`,
  '--mode': `specify env mode (default: development)`,
  '--host': `specify host (default: ${defaults.host})`,
  '--port': `specify port (default: ${defaults.port})`,
  '--interface': `specify network interface`,			// <== Added this option
  '--https': `use https (default: ${defaults.https})`
}

line ~72
// Add this line under the other inicializations
const networkInterface = args.interface || process.env.INTERFACE || projectDevServerOptions.interface

line ~74:
const urls = prepareURLs(
	  protocol,
	  host,
	  port,
	  options.baseUrl,
	  networkInterface		// <== added this parameter
)

prepareURLs.js
<EMAIL_ADDRESS>line ~13:
// added parameter networkInterface
module.exports = function prepareUrls (protocol, host, port, pathname = '/', networkInterface) {

line ~35:
// here finally we can especify wich network attatch to
lanUrlForConfig = address.ip(networkInterface)

optionsSchema.json
node_modules\webpack-dev-server\lib\optionsSchema.json

Only necessary if we put interface under devServer at vue.config.js, if passed as command line parameter or as environment variable you don't need this.

Add this somewhere
    "interface": {
      "description": "The interface the server attatch to.",
      "type": "string"
    },

To avoid webpack-dev-server complain about this:
 ERROR  WebpackDevServerOptionsValidationError: Invalid configuration object. webpack-dev-server has been initialised using a configuration object that does not match the API schema.
 - configuration has an unknown property 'interface'. These properties are valid:

After that, I can tell webpack from wich interface needs to obtain it's address to initialize the server
vue.config.js
...
devServer: {
  host: '<IP_ADDRESS>',
  port: 8082,
  interface: 'Ethernet'
}

command line/package.json
"scripts": {
  "serve": "vue-cli-service serve --progress --interface Ethernet",
  "build": "vue-cli-service build"
},

After that dev-server listens at the correct address:
App running at:
- Local:   http://localhost:8082/
- Network: http://<IP_ADDRESS>:8082/

Can you turn this into a PR please?

Done, PR #1909

https://github.com/vuejs/vue-cli/pull/1909#issuecomment-407440082


Feature Request: Castlevania 30th anniversary edition emulator

What device are you using?
RG351P
What version of ArkOS are you on?
1.5
Wondering if there is a way to port the 30th anniversary edition rom hack of Castlevania to ArkOS shown here: https://m.youtube.com/watch?v=mO4j6Z3_YlQ
From my understanding, there are two versions of this hack. One uses a modified version of NEStopia to inject the graphics into the game. The developer of the mod, Kya then teamed with the developer of MESEN to create a HD texture pack for MESEN without having to use a hacked version of an emulator.
Because MESEN supports windows and Linux, I'm wondering if it would be possible to add it and the relevant files to the RG351 in a way such as how the PSP emulator files are structured, to play the 30th anniversary hack on the device.

Per a similar request here for MESEN, MESEN is to slow on this platform at this time.  Perhaps the situation will improve in the future.  I will not be modifying nestopia at this time.

Per a similar request here for MESEN, MESEN is to slow on this platform at this time.  Perhaps the situation will improve in the future.  I will not be modifying nestopia at this time.


Have a way to include description for smart contracts

On smart contracts page, show description of smart contract. We may want to survey how other explorer does it.

For example display the module name

module name displayed, closing this item


[BUG] If an endpoint has negotiated MIDI 1.0 protocol, we do not downscale MIDI 2.0 protocol messages sent to it.

Describe the bug
Currently, the protocol downscaler is only invoked if the target is a MIDI 1.0 byte format device. It should also be invoked for a UMP-native device which has negotiated only MIDI 1.0 protocol during initial (or subsequent) protocol negotiation.
The challenge there is that the downscaler is added when an endpoint is first created, and currently cannot be inserted afterwards. To support renegotiating protocol, (and also changing the protocol during protocol negotiation), we need to finish the ability to dynamically add/remove transforms in a midi pipe, or we need to insert the protocol downscaler in all pipes and set it to listen for updates to the device and use the negotiated value.

Closed with release of DP7


validateWhereClauseStrict throws error for nested, populated blueprint actions

Sails version: 0.12.9
Node version: 5.5.0
NPM version: 3.3.12
Operating system: OSX

Previously, I was able to do something like this:
GET /endpoint?populate=[item]&where={"item":{"name":{"contains":"searchstring"}}}
But it looks like validateWhereClauseStrict now prohibits this. When it checks the rhs and finds that the 'name' key is not a valid SUB_ATTR_MODIFIER, it throws an error.
Is it still possible to accomplish this via blueprints? Was this ever intended to be allowed by blueprints in the first place? It was a breaking change for us in production.
Apologies in advance if this has been documented and I just didn't see it, but I couldn't find anything in the changelog.
Thanks!

Ok, Fixed!

@sschwa12 Sorry for the inconvenience!  Hopefully it wasn't too much of a pain to get sorted.
The documented query language is here: http://sailsjs.org/documentation/concepts/models-and-orm/query-language
And the docs for the find where blueprint action are here: http://sailsjs.org/documentation/reference/blueprint-api/find-where
So at first glance, I thought you were talking about doing intra-document lookups in Mongo-- but I think I was missing your point there.  Still, I'm including the answer I wrote for that first, since it covers most of the things:

I'm assuming you're using Mongo, since this would be a WHERE subquery in a SQL database (which wouldn't work).  Realistically, this never should have worked w/ Mongo either in the first place, but in some places it was allowed by the adapter (i.e. it just passed it straight through), and unfortunately, we weren't doing as much validation as I would have liked in the past.
If you guys badly need something from Sails 0.12.8 or 0.12.9, I can do a hotfix that adds a route option you can enable to skip the validateWhereClauseStrict() check temporarily.  But that's not an ideal solution for Sails or for Sails apps.  Instead, I think the best move here is to pin your app's sails dependency's semver range to 0.12.7 (e.g. "sails": "0.12.7").
In Sails v1/Waterline 0.13, we are planning to continue to enhance validations and assertions to help app-level code to avoid relying on non-standard / adapter-specific structures in criteria, mainly just because it can cause unintended consequences, and can lead to hard-to-detect security holes.
There's nothing necessarily wrong with doing these kinds of queries-- and going forward, you'll still be able to perform Mongo native queries that are outside the bounds of what is supported in WLQL criteria syntax, but only in custom actions-- not via the blueprint API.
That's not to say that we couldn't add support for something like this in the future (e.g. custom, nonstandard sub-attribute modifiers) - but that's a separate topic.

--
But anyways, like I mentioned above, I don't think any of that's actually what you were getting at, I was just misunderstanding you.
It looks like the ability to use a "populate" parameter to the "Find" blueprint action was added in 2014 as an experimental feature -- and then documented in 2015.  What's confusing me is the behavior when combined with where-- what you pasted above is not something I would expect to work with the blueprint API (and it is not documented afaik).  But just to be sure, I took that for a spin w/ 0.12.x prior to the validateWhereClauseStrict() change.  I couldn't get it to do anything-- it always just fails to match any records.  Do you think that it was working before for some other reason?  I could also be missing something here, so let me know if that's the case.
Thanks!

Weird, yeah we are using mySQL...
I removed my node_modules and rolled back to 0.12.4 and it definitely worked. in 0.12.9 i get:
Error: Could not parse the provided where clause.
which makes sense. This is the exact example, maybe it'll be helpful:
http://localhost:1337/job?populate=[lineItem,member]&where={"type":["appnexus","filter","job_creator"],"member":{"name":{"contains":"search"}}}
In 0.12.4 this returns all lineitems where member.name contains 'search'.
I feel like when we wrote this, we took a guess, 'hey I wonder if this works like this' and then it did... I haven't done a lot of digging into the sails code to explore why exactly this works.
Anyway, it's not a huge deal as we can just write a custom endpoint to handle this. Just thought it was good to bring to your attention in case other people are using it in this way (it is a nice feature, intentional or not!).

@sschwa12 definitely appreciate the heads up!

I'm not sure if @mikermcneil  got the issue from @sschwa12  -  I'm having a similar problem with a query that used to work too in v0.12.4
My question is - is there a way to query and filter by association attributes?
/api/people?where={"or":[{"firstname":{"startsWith":"peter"}},{"lastname":{"startsWith":"makoni"}},{"address":{"name":{"contains":"peter"}}}]}&limit=100&sort=createdAt
Hoping reverting back is not the only option


xquad, mlqa and mlsum tasks are not correctly implemented

@katrinklug
When running the tasks mlqa_en,mlsum_en, xquad_en and xquad_en I get the following error message:
Traceback (most recent call last):
  File "./tasks/eval_harness/evaluate.py", line 446, in <module>
    main()
  File "./tasks/eval_harness/evaluate.py", line 429, in main
    results = evaluator.evaluate(adaptor, {task_name: task}, False, 0, None, bootstrap_iters=args.bootstrap_iters)
  File "/lm-evaluation-harness/lm_eval/utils.py", line 162, in _wrapper
    return fn(*args, **kwargs)
  File "/lm-evaluation-harness/lm_eval/evaluator.py", line 253, in evaluate
    resps = getattr(lm, reqtype)([req.args for req in reqs])
  File "/lm-evaluation-harness/lm_eval/base.py", line 343, in greedy_until
    re_ord = utils.Reorderer(requests, _collate)
  File "/lm-evaluation-harness/lm_eval/utils.py", line 125, in __init__
    arr = group(arr, lambda x: fn(x[1]))
  File "/lm-evaluation-harness/lm_eval/utils.py", line 59, in group
    res[fn(ob)].append(ob)
  File "/lm-evaluation-harness/lm_eval/utils.py", line 125, in <lambda>
    arr = group(arr, lambda x: fn(x[1]))
  File "/lm-evaluation-harness/lm_eval/base.py", line 340, in _collate
    toks = self.tok_encode(x[0])
  File "/lm-evaluation-harness/lm_eval/models/gpt2.py", line 122, in tok_encode
    return self.tokenizer.encode(string, add_special_tokens=False)
AttributeError: '_GPT2BPETokenizer' object has no attribute 'encode'

Performed evaluation on Taurus using scripts from evaluation repository: apptainer/juwels_german-evalds.sbatch

Any perplexity tasks fail, same goes for the German ones. The issue is that the EvalHarnessAdaptor in Megatron-DeepSpeed/tasks/eval_harness/evaluate.py does not implement the greedy_until method, which perplexity tasks require.

Happened also for germanquad and squad2

New images ...
Taurus: /projects/p025/p_gptx/apptainer_images/obmd-lmeval-21.12_100423-py3.sif
Juwels: /p/scratch/opengptx-elm/shared/apptainer_images/obmd-lmeval-21.12_100423-py3.sif


Older versions (0.2x) removed from pypi

Hi, we are currently using hatch v0.23.0 in some company python projects. Now all our CI builds are failing because that version was removed from pypi during holidays. There is a reason behind this removal?
Removing packages from pypi all of a sudden when people and businesses rely on them is not cool.
Can you please add again the older versions? Thanks

Hmm, I thought yanking first prevented this. I'll do this now.

It won't allow me to replace the old artifacts, here's a new version: https://pypi.org/project/hatch/0.23.1/
I was trying to preserve the pinning support but I messed up, sorry for the disruption.

Thank you very much for the quick turnaround!


Environment Interpolation

In influxdb & telegraf, you can substitute in environment variables with the $VAR syntax. This throws an error in kapacitor. My test case is changing this line in kapacitor.conf:
bind-address = ":$PORT"
Which then errors out on start (With the PORT env variable set)

Since both telegraf and influxdb allow for the use of environment variables I expected kapacitor to behave in the same way. I was disappointed to realize it does not. I am trying to avoid putting my influx admin user credentials in the config file directly.
I like my config files to be globally readable because that makes quickly checking the configuration easy. Of course that means I have to not include any sensitive credentials in the file.
This seems like it should be very easy to add and config behavior across the TICK stack really should be consistent.


Documentation: Supervisor uses unsupported ubuntu version (13.04)

Output of docker version:
Docker version 1.10.2, build c3959b1

Steps to reproduce the issue:

Create the Dockerfile as specified in docs/admin/using_supervisord.md and the supervisors.conf
Run docker build  --no-cache  -t /supervisord .

Describe the results you received:
Failed to fetch http://archive.ubuntu.com/ubuntu/dists/raring/main/source/Sources  404  Not Found [IP: <IP_ADDRESS> 80]
Describe the results you expected:
Expected the container to be built successfully
Additional information you deem important (e.g. issue happens only occasionally):
Looks as though ubuntu:13.04 repositories are no longer available. Changing the container to ubuntu:15.04 allowed the docker build to run successfully

Opened a pull request to update those docs: https://github.com/docker/docker/pull/22542
thanks for reporting!


prod provision: Add CentOS support part 2.

Pending:

[x] closure-compiler
[ ] scripts/lib/install <- this is the remaining major work
[ ] whether postgres_common.pp doesn't require postgresql client package. yes/no?

non-blocker pending:

[ ] Figure out postgresql cache dir on CentOS (not really a blocker)
[ ] whether IUS package requirement should be removed then mod_wsgi is to be installed via pip (on CentOS only). yes/no?

Added closure-compiler stuff.

Probably need to add JRE to the puppet dependencies. Haven't checked.

Rebased.

@timabbott rebased again.

For closure-compiler, it's no longer a dependency at all.  So you can drop those commits.

I merged the postgres config commits.  I had a thought on reading those -- if I recall correctly, the main block at the bottom is the same in all of our postgres config files; can we read that from a common file shared across all platforms rather than copying it everywhere?
I still don't think the postgres-client commit is correct.  Because Zulip supports splitting up a server installation, we need a postgres client package on the app_frontend_base config as well as a postgres server in the postgres_common ruleset, and your changes don't achieve that.

Because Zulip supports splitting up a server installation, we need a postgres client package on the app_frontend_base config as well as a postgres server in the postgres_common ruleset, and your changes don't achieve that.

My finding was that the client is part of the metapackage postgresql-common.

If @RikaDenia could confirm that it is the case that postgresql-common already includes the client.

@andersk what are your thoughts on this approach?  I'm not sure that extracting functions that are just an OS conditional is actually helpful; in my view the total complexity of our install script isn't that high and it might be more readable without adding that indirection of logic.

@tabbott My intuition agrees with yours.
I had another thought that might make everything simpler. Using OS-specific conditionals to detect whether and how to install the supposedly OS-independent lsb_release defeats the whole point of lsb_release. Can we replace it entirely with /etc/os-release? That exists everywhere we care about (as far back as Ubuntu 12.04, Debian 7, RHEL 7, and CentOS 7), and doesn’t require any package installation.

Can we replace it entirely with /etc/os-release?

Caveat: the file exists only in systemd-based distributions. Though I don't think anyone would install Zulip on a fresh BSD-like VM or even macOS server.

No, although it’s required by systemd, it’s not part of systemd. Ubuntu 12.04 and 14.04 ran Upstart; Debian 7 ran SysVinit. Also lsb_release isn’t a thing on BSD or macOS either (it stands for “Linux Standard Base”, after all).

@andersk see parse_lsb_release in zulip_tools.py; we already have code in Python for parsing the main files there (an unfortunate workaround for lsb_release itself being super slow for the provision use case).  So we could certainly expose that to the shell scripts in some reasonable way and not need to use lsb_release itself in the early installer.

That’s another absurd block of code that would be drastically simplified if we used /etc/os-release. (Are you agreeing or disagreeing?)

Assuming /etc/os-release is available on a sufficiently wide range of platforms, I'd be happy for us to switch to using it.  So no disagreement with doing using that.  (I had not been aware of this alternative to /etc/lsb-release`

@rht can you rebase this now that the /etc/os-release stuff is done?

I need to remake the last 4 commits from scratch because the contextual lines surrounding them have changed.

@timabbott I have rebased and reworked the prod install script. The commits with [WIP] are not ready to be merged, but the rest are.

@timabbott can some of the commits be merged?

This is great, merged, thanks @rht and @andersk!  I definitely appreciate all the work to make us consistently use /etc/os-release as our source of truth.
Does the installer now run to completion on CentOS/RHEL?

Does the installer now run to completion on CentOS/RHEL?

Not yet but almost, at near the end of scripts/lib/install. Stuck at this error: https://chat.zulip.org/#narrow/stream/3-backend/topic/prod.20env.20on.20CentOS.207/near/803685 .


Question about OOD detection

Hi,
thank you for this nice extension for nnU-Net. I want to run the OOD detection for my tasks. To make it work on my computer I had to change the following parts of the code:
Line 181 inper_subject_evaluate.py to
patch_size = [112, 112, 192]   # this is for my task
uncertainty = mahalanobis_uncertainty(features_path, base_name, feature_key, patch_size=patch_size, norm=norm)

Line 181 inper_subject_evaluate.py to
def evaluate_subject(prediction_path, target_path, label=1, merge_labels=None):

Line 63 in model_restore.py to
search_in = join(list(nnunet_ext.__path__)[0], "nnunet", "training", "network_training")

Then I was able to run the code. However, the (mahalanobis) uncertainties I am receiving are quite different from what I would expect. A dataset with a slightly different modality from the training data received a higher uncertainty than a dataset with the same modality.
So I was wondering if I am using it correctly. I do the following (my model is a 3d_fullres model):
dataset = "modality_arteriell"
task = "119"
fold = "0"

feature_paths = ['conv_blocks_context.4.blocks.1.conv']

nnUNet_extract_features(str(Path(os.environ["nnUNet_raw_data_base"]) / "nnUNet_raw_data" /
                            get_full_task_name(int(task)) / "imagesTr"),
                        "training_data", feature_paths, task)

nnUNet_extract_outputs(data_dir, dataset, task_id)
nnUNet_extract_non_softmaxed_outputs(data_dir, dataset, task_id)

for idx in [10,20,30,40,50]:
    nnUNet_extract_MCDO_outputs(data_dir, dataset, idx, task_id)

nnUNet_extract_features(data_dir, dataset, feature_paths, task_id)
nnUNet_estimate_gaussian(task_id, fold, ["training_data"], [dataset], feature_paths)

nnUNet_extract_uncertainties(dataset, task_id, fold, feature_paths,
                                                label_dir, label=2, nr_labels=3, temperatures=[10, 100])

My task has 3 classes: 0->background, 1->class A, 2-> class B. I want to get the uncertainty for class B.

Hi wasserth,
thank you for trying out our method and clarifying which changes were necessary in your case. These look good, I will later modify the code so that the 'patch_size' argument can be set earlier on.
Some small comments regarding the later part of your code:

Is your 'task' that you set as task = '119' the same as the 'task_id' you use as an argument?
Please note that when you set 'label=1' on 'evaluate_subject', you are calculating the metrics for your label 1 (in your case I assume class A instead of class B).
Just to clarify, the 'mcdo_ix' arguemnt for 'nnUNet_extract_MCDO_outputs' is the index for a specific output. In this case you are extracting 5 outputs with indexes [10,20,30,40,50] and calculating the divergence between these.

I did not find any obvious error with your code. As some basic sanity checks:

how are the uncertainties (distances) for the training data? I would suggest calculating these by adding the "training_data" dataset to the 'store_ds_names' argument of 'nnUNet_estimate_gaussian'. Intuitively, these should be considerably lower than the uncertainties for in-distribution unseen data.
What ranges do the uncertainties occupy?

Thank you for the tips. I found the problem: For the training gaussian fit (train_ds_names) I used the entire dataset. This then resulted in the validation data having very low uncertainties. By only using the training data for train_ds_name the uncertainties for the validation data are more reasonable. Now uncertainties for the training data is really low (roughly 3000), for the validation data it is medium (roughly 3.000.000) and for perturbated data it is clearly higher (roughly 10.000.000).


add spark batch log service rdd

What changes were proposed in this pull request?
add spark batch log service rdd
How was this patch tested?
manual tests in com.aliyun.emr.examples.TestBatchLoghub

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


Incomplete multi-character sanitization on utils.js getQueryParam function

Our company enforced a CodeQL scan against our built bundle. And an incomplete multi-character sanitization error is reported from this line
https://github.com/mixpanel/mixpanel-js/blob/93df8eb5d7ada011aedb39ab723a8f415e2e5782/src/utils.js#L965
This should be easily fixed by adding a 'g' in regex
param = param.replace(/[[]/g, '\\[').replace(/[\]]/g, '\\]');

Thanks for the heads up. This should be fixed in release v2.55.1.


onSaved callback in the InternationalPhoneNumberInput is not synchronous

Describe the bug
When the form is saved in flutter with this._formKey.currentState.save(), the internal call for saving the phone number is using a future and so the value is not available right after the call to form save.
Package version
Your current version of the package [0.6.0]
Flutter version
Your current version of the Flutter SDK [1.22.6]
To Reproduce
void _signUp(BuildContext context) async {
    var _isValid = this._formKey.currentState.validate();
    if (_isValid) {
      this._formKey.currentState.save();
      print("Form is valid");
      final phoneAuthProvider =
          Provider.of<PhoneAuthProvider>(context, listen: false);
      print("In _signUp ${number.phoneNumber}");
      phoneAuthProvider.setValues(
          phoneNumber: number,
          onCodeSent: () {
            Navigator.of(context).pushNamed(VerifyPhoneScreen.ROUTE_NAME);
          });
    }
  }

The value inside the phoneAuthProvider is null for the phonenumber as it is not saved by the time the values are sent to the provider. It prints as null inside the provider.
Expected behavior
The save must be synchronous and the onSave must be called synchronous.
Screenshots
None.
** Targeted platforms (please complete the following information):**

OS [Android]
Web [❌]

Additional context
None.

Same problem.

Hi, I have replaced the asynchronous call with a simple regexp replace non-digit on any value TextFieldController.Text has.


Custom dialog for "Navigate to"

Advantages:

an ability to add more powerful search modes, like

by upper case letters (AB => [AaBbbb; ABCcc])

fuzzy (abc => AbFfCc or something like this)

save previous patterns and use the last used by default and allow the user select from history
select in what file types to search symbols

implement good caching with storing result on disk to prevent reanalyzing assemblies each time VS restarts
put it at the center of VS window, which is much nicer, then position of the current dialog
Issues:
we have to use Roslyn to analyze C# assemblies / projects. However, as we are going to use it to enable Find all refs for C# projects, it looks ok.

No way to implement it in any near future. I'm closing it.

Even with recently implemented caching and storing it in a file, Navigate To is uncomfortably slow and limited to be really useful on real world solutions compared to the equivalent features in R# and IDEA.
So, I'm reopening it.
The main show stopper is getting all searchable symbols from all C# and VB projects in a solution. If we solve it, the rest is clearly achievable.

Again, there's no way it will be implemented in any future.


Add documentation on Azure ADLS support

As usual adding @samansmink for the review

Hey @quentingodeau, thanks a lot again! I added some minor touches here and there, feel free to change them up.
Great idea btw to add an example on the performance difference, it makes things very clear!

That a really good idea I will try to run some benchmark in an env with a lots of files :)

Thanks @quentingodeau!


Call to a member function prepare() on null

Laravel-mongodb Version: 3.8.2
PHP Version: 7.4.13
Database Driver & Version: 1.9.0

Description:
Adding a mysql relation to a mongodb model causes the given error when fetching the relation.
{
  "exception": "Error"
  "file": "/var/www/project/vendor/laravel/framework/src/Illuminate/Database/Connection.php"
  "line": 338
  "message": "Call to a member function prepare() on null"
}

My model class
use App\Models\User;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Jenssegers\Mongodb\Eloquent\Model;

class Draft extends Model
{
    protected $collection = 'survey_drafts';

    protected $connection = 'mongodb';

    protected $attributes = [
        'questions' => [],
    ];

    public function owner(): BelongsTo
    {
        return $this->belongsTo(User::class, 'owner_id', 'id');
    }
}

The User class uses the default MySQL connection.
Upon investigation further, I saw that Connection::getPdoForSelect is being called on the MongoDB connection instance, where the $pdo is NULL. Therefore throwing the error when trying to retrieve the PDO class for some reason.
The cause for this is that the MySQL User model does not have a $connection property defined as it should pick the default mysql connection when it's not specifically defined otherwise (as its done with the mongodb model).
This is being done in HasRelationships::newRelatedInstance(). When being used as a relation to a mongodb model the $this->connection  is a mongodb connection.
The only way to fix this currently is to add a
$connection = 'mysql'

to all MySQL models that are being used as relations to a mongodb model. In the given example, to the User model

The drawback to this "fix" is that if you're using the memory database for tests, they will start to fail as the models will be using the mysql connection instead of the memory one, as it's hardcoded into them.

same issue

I have the same issue too
Cart Class
`
use Jenssegers\Mongodb\Eloquent\Model;
class Cart extends Model
{
protected $connection = 'mongodb';
protected $guarded = [];

protected $with = ['items'];

public function user()
{
    return $this->belongsTo(User::class);
}

public function items()
{
    return $this->hasMany(CartItem::class);
}

}`
User model is default mysql connection and when I call like this:
$cart->user
I got same error: Call to a member function prepare() on null
But in reverse works well with HybridRelations
$user->carts

I have the same issue without any relation with a simple model
class ActivityLog extends Model
{
    protected $connection = 'mongodb';
    protected $fillable = [
        'action'
    ];
}

Replace Illuminate\Database\Eloquent\Model with MongoDB\Laravel\Eloquent\Model inside your models.
Example:
namespace App\Models;

use Illuminate\Database\Eloquent\Model;    // instead of Illuminate\Database\Eloquent\Model

class Comment extends Model
{

}

Laravel-mongodb Version: 3.8.2
PHP Version: 7.4.13
Database Driver & Version: 1.9.0

Description:
Adding a mysql relation to a mongodb model causes the given error when fetching the relation.
{
  "exception": "Error"
  "file": "/var/www/project/vendor/laravel/framework/src/Illuminate/Database/Connection.php"
  "line": 338
  "message": "Call to a member function prepare() on null"
}

My model class
use App\Models\User;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Jenssegers\Mongodb\Eloquent\Model;

class Draft extends Model
{
    protected $collection = 'survey_drafts';

    protected $connection = 'mongodb';

    protected $attributes = [
        'questions' => [],
    ];

    public function owner(): BelongsTo
    {
        return $this->belongsTo(User::class, 'owner_id', 'id');
    }
}

The User class uses the default MySQL connection.
Upon investigation further, I saw that Connection::getPdoForSelect is being called on the MongoDB connection instance, where the $pdo is NULL. Therefore throwing the error when trying to retrieve the PDO class for some reason.
The cause for this is that the MySQL User model does not have a $connection property defined as it should pick the default mysql connection when it's not specifically defined otherwise (as its done with the mongodb model).
This is being done in HasRelationships::newRelatedInstance(). When being used as a relation to a mongodb model the $this->connection is a mongodb connection.
The only way to fix this currently is to add a
$connection = 'mysql'

to all MySQL models that are being used as relations to a mongodb model. In the given example, to the User model

I think this is a logical fix. Only issue I have here is if you have 100s of models then it eliminates your model portability. Another solution could be to store connection names as constants and use those in your models so you can mass assign later in the future.

I would suggest you to run php artisan queue:listen or If running with supervisor php artisan queue:work --once
This will bootstrap the application everytime a new job is processed, :work will cache some things between each jobs.

The safest option is to define $connection for each model explicitly.


Filter does not work for all list view entries

e.g. motion list view:
you can not filter for submitter ("Max Mustermann") which is displayed in motions table.
We have to update the angularjs filter for "related" properties like the submitter (user full name).
@ostcar Any idea?

one example is in the linked pull request.


Strange behavior of PgSTAC Catalog when querying using PySTAC client.

Hi!
During the last tests, I noticed strange behavior with my catalog when querying using the PySTAC client. Specifically, regardless of the query, it always returns a maximum of 10 items.
import pystac_client
import ipyleaflet
import geogif
import stackstac
import os
import pystac_client

lon, lat = 21, 52

# Main endpoint for STAC
URL='https://stac-cdse.eu/'
catalog = pystac_client.Client.open(URL)
catalog.add_conforms_to("ITEM_SEARCH")

items = catalog.search(
    collections=['sentinel-2-l2a'],
    intersects=dict(type="Point", coordinates=[lon, lat]),
    datetime="2020-09-15/2024-09-16").item_collection()

Increasing max_items parameter does nothing :/
While using pgstac.demo.cloudferro.com with analogical query there was no such problem.

stac-cdse.eu pgstac database deployment:

#!/bin/bash
# Update the package list
apt-get update

# Install nano
apt install -y nano

# Install required packages with automatic yes
apt-get install -y ca-certificates curl

# Create the directory for Docker keyrings
install -y -m 0755 -d /etc/apt/keyrings

# Download the Docker GPG key
curl -fsSL https://download.docker.com/linux/ubuntu/gpg -o /etc/apt/keyrings/docker.asc

# Ensure the keyring file is readable
chmod a+r /etc/apt/keyrings/docker.asc

# Add Docker repository to APT sources
echo "deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/docker.asc] https://download.docker.com/linux/ubuntu \
$(. /etc/os-release && echo "$VERSION_CODENAME") stable" | tee /etc/apt/sources.list.d/docker.list > /dev/null

# Update the package list again
apt-get -y update

# Install Docker components with automatic yes
apt-get install -y docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin

# Install pypgstac
python3 -m pip install pypgstac[psycopg]

# Export initial env variables:
# for pypgstsac
export POSTGRES_USER=
export POSTGRES_PASSWORD=
export POSTGRES_DB=
export PGUSER=
export PGPASSWORD=
export PGDATABASE=

export PGHOST=<IP_ADDRESS>
export PGPORT=5432

# for docker
export postgres_ram="200g"
export postgres_cpus=62
export POSTGRES_IMAGE="postgis/postgis:16-master"

# let's make sure that we only need to export those only once
echo "export POSTGRES_USER=${POSTGRES_USER}" >> /home/eouser/.bashrc
echo "export POSTGRES_PASSWORD=${POSTGRES_PASSWORD}" >> /home/eouser/.bashrc
echo "export POSTGRES_DB=${POSTGRES_DB}" >> /home/eouser/.bashrc
echo "export PGUSER=${PGUSER}" >> /home/eouser/.bashrc
echo "export PGPASSWORD=${PGPASSWORD}" >> /home/eouser/.bashrc
echo "export PGDATABASE=${PGDATABASE}" >> /home/eouser/.bashrc
echo "export PGHOST=${PGHOST}" >> /home/eouser/.bashrc
echo "export PGPORT=${PGPORT}" >> /home/eouser/.bashrc

# Run pgstac via installed docker:
docker run -d \
  --name stac-db \
  --restart unless-stopped \
  --memory="${postgres_ram}" \
  --cpus="${postgres_cpus}" \
  -e POSTGRES_USER=${POSTGRES_USER} \
  -e POSTGRES_PASSWORD=${POSTGRES_PASSWORD} \
  -e POSTGRES_DB=${POSTGRES_DB} \
  -e PGUSER=${PGUSER} \
  -e PGPASSWORD=${PGPASSWORD} \
  -e PGDATABASE=${PGDATABASE} \
  -p 5432:5432 \
  -v stac-db-data:/var/lib/postgresql/data \
  ${POSTGRES_IMAGE}

# Base migrations install PgSTAC into a database with no current PgSTAC installation
pypgstac migrate

Populating Catalogues:

pypgstac load collections https://s3.fra1-2.cloudferro.com/swift/v1/poland-stac/collection-sentinel-2-l2a.json
pypgstac load items https://s3.fra1-2.cloudferro.com/swift/v1/poland-stac/poland-data.json --method upsert
pypgstac load items https://s3.fra1-2.cloudferro.com/swift/v1/marcin-kluczek/chunk_aa1.json
pypgstac load items https://s3.fra1-2.cloudferro.com/swift/v1/marcin-kluczek/chunk_aa0.json

Deployment stac-fastAPI:
behavior was tested with 3 different versions of FastAPI
main
3.0.0
2.4.11 (the one used for pgstac.demo.cloudferro.com)

with the same result for each.
Used postgres versions:
postgis/postgis:16-master - stac-cdse.eu
ghcr.io/stac-utils/pgstac:v0.8.4 - pgstac.demo.cloudferro.com
If this issue is related to the image being used, I would kindly ask for information or a guide on deploying PgSTAC on images other than the original ones.
Best,
Marcin

@MathewNWSH the datetime do not follow the spec can you try with '1990-12-31T23:59:61Z' format
Or maybe I'm not understanding the issue, are you complaining about the number of items being return or the fact that it the same 10 items?
if it's about the number of items, there is a limit parameter which default to 10, so you can adjust this by passing limit=100

Hey,
Yes, I completely meant that it's still returning only 10 products :)
I'm familiar with the limit parameter, but it was kind of weird for me since I didn't need to provide it for my other catalog, and it was still returning the data (even if it exceeds the number of 10).
Is there an environment variable increasing the default limit in ghcr.io/stac-utils/pgstac:v0.8.4?
Anyway, a big thank you!

the default limits comes from https://github.com/stac-utils/stac-fastapi/blob/753341f18d6c3d0454efef52d0ff841cfa3215f7/stac_fastapi/types/stac_fastapi/types/search.py#L176-L190
10 is the default defined by the specification

limit The limit parameter follows the same semantics of the OAFeat Item resource limit parameter. The value is a suggestion to the server as to the maximum number of Item objects the client would prefer in the response. The server may return fewer Item objects, but must not return more. The OpenAPI specification defines the default and maximum values for this parameter. The base specifications define these with a default of 10 and a maximum of 10000, but implementers may choose other values to advertise through their service-desc endpoint. If the limit parameter value is greater than the advertised maximum limit, the server must act as if the request were for the maximum and not return an error.

ref: https://github.com/radiantearth/stac-api-spec/tree/release/v1.0.0/item-search#query-parameter-table
You can built your own stac-fastapi-pgstac application with other default than 10 (ref: https://github.com/stac-utils/stac-fastapi/discussions/737)

@MathewNWSH are we good to close this?

@MathewNWSH are we good to close this?

Yes, thanks again


connecting SDN-WISE to Floodlight controller

Hello.
Is it possible to connect SDN-WISE to Floodlight controller? If so, would you please prepare a tutorial for this?

Hi,
Is it possible? Yes it is.
It has been done? No it hasn't
If you want to start it as a side project, feel free.
I'm not working full time on this project but I can help you a little bit if you start working on it

Thank you. Would you please tell me what specifically steps have you done to configure and connect to ONOS controller?

You can refer to this commit to have an idea of all the code that has been added to onos: https://github.com/sdnwiselab/onos/commit/f4101400c06ed069010f88d1a3f05d48cfd5f797

Thanks, by the way i wanted to know if we do the same thing for the floodlight controller, should sdnwise team implement the code (in order to make floodlight work with sdnwise) ?

Hi,
you are talking right now with all the sdnwise team, which is just me at the moment. So the answer is no, I can just help from time to time.


Hector, oks, can you add your mail next to your name, please?

ok. I did it.

Great!


MiniPlayer song position slider is inaccessible beneath the playback controls

Probably a z ordering issue...

Fixed in working branch

Fixed in working branch


Modify AddInvokeTransaction with BroadcastedInvokeTxn

Need updating:

Account test files
Simple invoke example

Check if it needs to be updated somewhere else

No need to do this as Broadcasted Invoke transaction is just a wrapper for Invoke transaction


CONUS404 dimension documentation

Some of the CONUS404 variables have a z-axis spatial dimension. I haven't seen these documented anywhere, but I'm happy to be corrected if they are. I would love to see this built into the metadata of the zarr. I also think it could be helpful to create another spreadsheet to document these dimensions for the data release in general. I'm wondering

if anyone on our team has this information, or if we need to go the NCAR for this
if there is a way to add descriptive metadata about a dataset dimension to the zarrs we have

@pnorton-usgs , what say ye?   Do you know or do we need to ask NCAR?

NCAR has a technical report (A Description of the Advanced Research WRF Version 3; http://dx.doi.org/10.5065/D68S4MVH) that goes into the specifics of WRF. In section 3.2 (Spatial Discretization) describes the grid discretization. The non-staggered dimensions are located at the cell center points (mass points) while the staggered dimensions are located at the cell boundaries.
I've never seen specified descriptions of the dimensions themselves. There is discussion in the WRF documentation about how to unstagger the wind fields onto the mass points. Note that the x_stag and y_stag dimensions in our zarr datasets are west_east_stag and south_north_stag, respectively, in the wrfout files.
To add descriptive information to the staggered variables we would need to create coordinate variables for them. I would check with NCAR and see if they have some further documentation about the staggered dimensions and how to work with the variables that use them.

I am closing this out - I was looking for descriptions of the dims to add in to STAC, but I'll just leave the descriptions out since it'll be a bit of lift and it isn't a required piece of metadata.


fix #21 - Android browser: Object.assign is not a function

Coverage remained the same at 100.0% when pulling 3d4d2f2dcc2d9b042f58fb349f586b67011d4995 on smcatala:master into 04c094864c549cd51abb2758b0c00586e3ddee1a on ZenyWay:master.

Coverage remained the same at 100.0% when pulling b3313fdca85f5f2e96b8bbcc74d89a7f0a0725dd on smcatala:master into 04c094864c549cd51abb2758b0c00586e3ddee1a on ZenyWay:master.

squash merged with commit a45030e5ba3133b188eb18b05c0c35010e28eadd


PvP Punishment not being applied.

Enable PvP Punishment = true
Punish Level Difference = -10
Offense Limit = 1
Offense Cooldown = 300
Debuff Duration = 1800

When a level 75 kills a level 30 the level 75 does not receive the debuff values. The debuff can be applied manually with code. Does this have something to do with the 'Offense Limit' being 1?

Did you figure out what the issue here was? I'm having the same problem as.


Feature Request: Search from browser urlbar / search field

Hello. The project is a amazing, thank you.
It would be handy to have an ability to search from url bar (search bar) right from the browser. This is very popular scenario.
For now Firefox, for example, doesn't recognize app as a search engine and doesn't allow to add it to search field or url bar.

Just added in #28!


Support assemblyPath property in host.json

For precompiled functions VS support, we need a way to specify assembly name at the Function App level.
Propose a property assemblyPath in host.json. Example:
"assemblyPath": "bin\\MyAwesomeAssembly.dll"

This assembly can be overridden using the scriptFile property in function.json.

One reason I think we don't need this urgently is that in the VS compiled support, all the function.json files are generated and not user written. Hence, the duplication of the assembly in each of them does not hurt the user experience. i.e. if they need to rename the assembly, it all gets fixed up by just republishing.
So while the idea makes sense, I don't think we need to prioritize this.

@davidebbo Ah, good point!  This may not change the prioritization, but I think this feature would be useful for customers who are migrating from CSX and haven't yet fully adopted the new model (where function.json is generated). A property in host.json would mean you just have to add the assembly path in one place, as opposed to every function.json.
That said, you do still have to touch every function.json to add entryPoint, so maybe it's still not that worthwhile.

I am getting an error regarding assemblyPath, completely out of the blue starting today. My functions were working fine day before yesterday. Here is my function.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Attach to Python Functions",
      "type": "python",
      "request": "attach",
      "port": 9091,
      "preLaunchTask": "func: host start"
    }
  ]
}

Here is the error I'm getting.

[8/12/20 6:10:45 PM] Loading startup extension 'AzureStorage'
[8/12/20 6:10:45 PM] A host error has occurred during startup operation '3596fd77-72f0-4e50-9888-0a2c9858f1b1'.
[8/12/20 6:10:45 PM] System.Private.CoreLib: Absolute path information is required.
[8/12/20 6:10:45 PM] Parameter name: assemblyPath.
Value cannot be null.
Parameter name: provider
Application is shutting down...

I don't know if this relates, but I'm posting here because of the discussion of assemblyPath. Do you have any suggestions?
Thank you!
Dylan


Optimize .fasta sequence IO

Background
Once we're done with the end-to-end process for assigning clades, there are some optimizations we should do to better work with the sequence data (which is in .fasta format)
Definition of done

[ ] The ZSTD-compressed version sequence data is less than half the size of the lzma-compressed file we're currently using. Adding the ability to read ZSTD files in cladetime will shorten the download time (keep the lzma handling, because some of nextstrain's sample/subset data is not provided in ZSTD format)
[ ] Use a generator when using biopython to filter sequence data (to prevent memory issues)

I did a few experiments for different ways to optimize the cladetime process that downloads a full sequence .fasta file and passes a subset of metadata to sequence.filter to write a second, smaller sequence file to use for clade assignment.

Keep using the existing, much larger .xz format (sequences.fasta.xz), but use multiple threads to download it in chunks. Result: roughly the same amount of wall time as existing code, 887s
Add third-party ZSTD reader, switch to using smaller .zst format (sequences.fasta.zst),keep the download single-threaded (chunk ZSTD data is more involved, since you have to maintain order). Result: : 55% reduction (394s)
Add third-party ZSTD reader, but stream the download instead of writing to disk first. Filter against metadata on the way in, only writing the matched sequences to disk. Result: 52% reduction (427s)

While it might be possible to further optimize the first and third approaches (e.g., try different chunk sizes), I'm going to proceed with option 2:

It adds a significant performance boost with the least amount of complexity (no managing parallel threads or stream chunk sizes)
Even if using multiple threads made an impact on the overall timing, there would be diminishing returns for people using machines with fewer cores

Note that I removed the "generator" to do from this ticket, since we don't need a generator when reading the sequence file line by line.


Recent update to Gradle is cursed

Builds are taking a minute or more, redownloading everything on every run. It's awful.
Manually reverting PR #4 solved all issues.

cursed? more like spooky


How to search by choices?

For example, school grade 3. School table stores min and max grades. Elementary has K-5, Middle has 6-8, and High has 9-12. I have stored the grades as K, 1st, 2nd, etc.

Disregard, used widgets ChoiceField.


created onParticleRequire strategy

included require section in recipe clone

This looks good, and is well tested.
Just to confirm: slots are matched at the moment but not handles yet. Is that correct? If so, it'd be good to add a TODO that notes this to the top of the strategy.

I've added another test that makes sure the recipe doesn't resolve when the strategizer finds a pattern in the active recipe that almost matches the require section.


Importing Resource Location or Zone - Requires Uid instead of ExternalUid

Running terraform import citrix_zone.zone EXTERNAL-UID-HERE returns "Cannot import non-existent remote object" when specifying the ExternalUid of a resource location. It works fine if the Uid is used instead.
When manually creating a resource location in the Citrix Cloud web console, the ID listed there is the ExternalUid. The only way I've been able to find the Uid is via PowerShell with Get-ConfigZone. Is it possible to use the ExternalUid with this zone import instead of having to first retrieve the Uid via powershell?

Hi @amshaffer,
We are expecting a Zone ID here and not a Resource Location ID that is exposed in the Citrix Cloud UI. We have plans to expose the resource ID in the webstudio UI so it is easily available for use in terraform.
For now, another way to get the zone ID is through inspecting the network calls in the browser dev tools when loading Zones in the webstudio UI.
Also as an alternative to importing zone, you can have a matching terraform config for your existing zone resource and run apply which will adopt your existing zone resource into terraform state.
Aneesh

Makes sense, thanks very much! This will be easy enough to work through with your recommendations.


pkg.go.dev: function does not appear in table of contents

The function Expect appears on the page https://pkg.go.dev/github.com/onsi/gomega?tab=doc#Expect, but it is not listed in the table of contents.

It is shown under "type Assertion", though I also feel this is some confused.

/cc @dmitshur for input - this is also how it is currently displayed on godoc.org.

This problem has been discussed before: https://github.com/golang/go/issues/28006

Per comments above, Expect is considered to be a "factory function" for Assertion, so it's displayed underneath it.
The code for this is in go/doc (see https://github.com/golang/go/issues/39813#issuecomment-649636776). I don't see anything to do here, so let's close this. We can use #39813 as the tracking issue for considering changes to the "factory function" behavior.


Creating duplicated ALBs on AWS

Hi,
I've migrated all my pods to a new AWS eks cluster and once I deployed it it started to duplicate the ALB one for the new cluster and other alb for the deleted k8s cluster.
I've created a new ingress as well and saw it create two alb.
Why is that ?
Looking into the k8s ingress manifest I see the wrong reference, but it is created automatically :thinking:
status:
  loadBalancer:
    ingress:
    - hostname: internal-ea50810c-cops-copsingress-7a6b-1946681565.us-east-1.elb.amazonaws.com

while the new and right ALB is:
internal-fd66572f-cops-copsingress-7a6b-1966552449.us-east-1.elb.amazonaws.com

Hi, I think i didn't understand the problem correctly.
Did you create an new Ingress in new cluster? every Ingress will only create a single ALB. (The ALB name is calculated by clusterName+IngressNamespace+IngressName)

hi @M00nF1sh,
I've 3 ingress which would lead me to 3 ALBs but after my migration to a new cluster it started to happen.
So I am still using the same ingress I used to have before and for purpose of tests I just create this 3° ingress to test if a new ingress would have the same problem.
If I go inside the ALB and look at the rules they are all duplicated as well

Do you have identical Ingress in two clusters(old cluster and new cluster)? If so, it will be two ALB since it's different Ingress(same spec in two different cluster)

@M00nF1sh  I've not. I've deleted the old cluster so now I've one k8s cluster with 3 different ingress, 1 ingress per namespace.

Here are the flags I'm using to run the aws alb ingress controller.
I've upgraded to the last image and got not difference in the problem.
      - args:
        - --cluster-name=-k8s-prod
        - --ingress-class=alb
        - --aws-region=us-east-1
        - --aws-vpc-id=vpc-11223a75
        image: docker.io/amazon/aws-alb-ingress-controller:v1.1.5
        imagePullPolicy: IfNotPresent

@weslleycamilo From the aws console of ALB, you can check the Tags, where is contains corresponding Ingress's clusterName, ingressNamespace and IngressName.
Would you help paste it here? (BTW, is the vpc-id correct? you don't need to specify region and vpc-id if it's ec2metedata is not disabled)


suggestion: delete cache files

After the edge update, there will be Temp and EdgeUpdate folders under the Microsoft folder, I think these files should be deleted.

EdgeUpdate
Temp

Added.
EdgeUpdate does not always uninstall like it is supposed to. Sometimes the folder disappears a minute or so after the tool is ran.


Compile Problem

I have a problem, i download and extracted bur-rest-api on my Kali.
Then i download burp free edtion and i put in bur-rest-api/lib and i renamed like you said burpsuite_pro.jar
then ./gradlew bootRun "-Dburp.edition=free"
show "downloading https://...gradle-2.13-bin.zip"
"unzipping ... set executalbe permision /root/.gradle/wrapper/dists/gradle-2.13-bin/.../bin/gradle"
Then process end without error.
But i don't see any build directory o burp-rest-api.jar
What is it problem?

Fixed, i used openjdk 10,
when it work only with jdk 8


Simulation of low flip angles and delays

Include in simulations use of low/zero flip angles and/or delays that currently can cause divide by 0 type errors

fixed for fit_pyr_kinetics()


Add constants and cleanup

sweet! syntax highlighting is a bit broken, but we can make do! this makes it easier to add variable support i think.
thank you for your contribution!

I think the syntax highlighting was already broken from previous commit, especially the  = ans part.

I think the syntax highlighting was already broken from previous commit, especially the  = ans part.

yep you are right. ill add some commits to this pr, fix them and merge them to master.
thank you!

I am wondering if the syntax highlighting at the back could be tested, so far I am not sure how.

// To debug rustyline:
// RUST_LOG=rustyline=debug cargo run --example example 2> debug.log```

i picked this up from the example code over at the rustyline repo. it might come in handy! i am a little tied up now, but ill give it a go sometime this week.

hey @pickfire, i pulled your fork and made some changes locally and merged it to master. (some improvements to highlighting)
sorry i messed up this pr, consider it merged. its your code that runs master!


⚠ AGS_GIS- Vicmap_Parcel_Property_Aspatial_Table-MS has degraded performance

In e03abc8, AGS_GIS- Vicmap_Parcel_Property_Aspatial_Table-MS (https://vicmap.land.vic.gov.au/agsgis/rest/services/vicmap/Vicmap_Parcel_Property_Aspatial_Table/MapServer/0/query?where=1%3D1&resultRecordCount=1) experienced degraded performance:

HTTP code: 200
Response time: 162 ms

Resolved: AGS_GIS- Vicmap_Parcel_Property_Aspatial_Table-MS performance has improved in 3f99fde after 14 minutes.


Playerrank

I use my LOL code before but now I can't get it to work @DimaKudosh
I'm getting
from pydfs_lineup_optimizer.constants import PlayerRank
ImportError: cannot import name 'PlayerRank' from 'pydfs_lineup_optimizer.constants' (/usr/local/lib/python3.7/dist-packages/pydfs_lineup_optimizer/constants.py)
Use to work just fine

Code I'm using and it use to work
`from collections import Counter
from itertools import chain
from pydfs_lineup_optimizer.constants importPlayerRank
from pydfs_lineup_optimizer.stacks import PositionsStack, PlayersGroup, NestedPlayersGroup, OptimizerStack
from pydfs_lineup_optimizer.utils import get_players_grouped_by_teams, list_intersection
class PositionsStackWithStar(PositionsStack):
def build_stacks(self, players, optimizer):
players_by_teams = get_players_grouped_by_teams(players, for_teams=self.for_teams)
all_positions = tuple(set(chain.from_iterable(self.positions)))
positions_for_optimizer = Counter(self.positions)
positions_for_optimizer[all_positions] = len(self.positions)
all_groups = []
for team_name, team_players in players_by_teams.items():
groups = []
groups.append(PlayersGroup(
players=[player for player in team_players if
player.rank == PlayerRank.STAR],
min_from_group=1,
))
for positions, total in positions_for_optimizer.items():
groups.append(PlayersGroup(
players=[player for player in team_players if
list_intersection(player.positions, positions) and
player.rank == PlayerRank.REGULAR],
min_from_group=total,
))
nested_group = NestedPlayersGroup(
groups=groups,
max_exposure=self.max_exposure_per_team.get(team_name, self.max_exposure),
)
all_groups.append(nested_group)
return [OptimizerStack(groups=all_groups)]`

I believe you removed some stuff for this to work since I tried an older version and it works again

Yes, this code was removed some time ago and replaced with the creation of multiple separate players for each slot as it does in DK, you need to update your code.


feat: support strictNullChecks

Turns on strictNullChecks everywhere and refactors all of the code to make it compliant. This makes us consistent with the core Angular modules which support strictNullChecks after 4.0.
Fixes #3465.
Note: This is pretty conflict-prone and is blocked on a couple of things:

We need to bump to Angular 4.0+ to be able to compile.
@angular/forms doesn't support strictNullChecks as of 4.0 RC2, which means that our code won't be able to compile until it does.

Closing in favor of #5094.


feat: allow network to be defined

What kind of change does this PR introduce?
Simple change to allow a custom docker network name to be used instead of the default.
What is the current behavior?
Default and only available network is supabase_network_project_id (where project_id is that of the current project).
What is the new behavior?
If the configuration toml defines this at the top-level:
network = my-custom-network

The network called my-custom-network will be used instead of the default.
Additional context
This is useful for running supabase inside an existing local development network so that other docker images can access supabase resources without going through the host. Given the simple names of supabase services, like db, auth, rest etc. there is a chance of name collisions. #1510 helps reduce complications with longer aliases.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 57.11%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/utils/config.go
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
5205

Relevant Lines:
9114

💛  - Coveralls

This is useful for running supabase inside an existing local development network so that other docker images can access supabase resources without going through the host.

If supabase start errors due to unhealthy container, the current implementation will try to delete the network by id. This means any custom network may be deleted and recreated by cli if there are no resources using it. The recreated network will likely have different properties, like additional labels. Would this be a concern for you?
If so, I think a more desirable behaviour might be to leave the existing custom network alone. This can be implemented by pruning the network only if it matches a filter by label.

Thanks @sweatybridge! Normally docker wouldn't let you delete a network that is in active use, which is our case when using docker compose down, it'll just raise an error which is currently ignored as per the link you posted. Even if deleted, running docker compose up again will just re-create it from scratch.
In our use-case, this wouldn't be an issue, but I can see the potential for problems. Filtering by label for networks that were created by supabase could be a reasonable solution.

I thought about this more and find that an extra flag like supabase start --use-network <id> might be more suitable here. Reason is network id doesn't seem to be a project level config. This concept doesn't apply to remote project, it may change between different local dev machines, and doesn't need to be committed to git.
I would be happy to add the start flag if you are ok with it too.

closing due to staleness


Update Kubernetes dependencies to release kubernetes-1.16.3

Please update the Kubernetes dependencies from legacy commits closer to kubernetes-1.16.3 releases, so that this library can be consumed more easily with other Kubernetes 1.16.3 libraries.
[WARN]	Conflict: k8s.io/apimachinery rev is currently kubernetes-1.16.0, but k8s.io/client-go wants 3c7067801da2
[WARN]	Conflict: k8s.io/api rev is currently kubernetes-1.16.0, but k8s.io/client-go wants bbc9463b57e5
[ERROR]	Error scanning k8s.io/client-go/kubernetes/typed/discovery/v1beta1: cannot find package "." in:
	/Users/andrew/.glide/cache/src/https-k8s.io-client-go/kubernetes/typed/discovery/v1beta1
[ERROR]	Error scanning k8s.io/client-go/kubernetes/typed/flowcontrol/v1alpha1: cannot find package "." in:
	/Users/andrew/.glide/cache/src/https-k8s.io-client-go/kubernetes/typed/flowcontrol/v1alpha1
[WARN]	Conflict: github.com/Azure/go-autorest rev is currently release/v11.1, but k8s.io/client-go wants v0.9.0

You can use the kubernetes-1.16.3 tag, which corresponds to that release
/close


callback support for save method

I was noticing that I was sending the file before it was update and would like to have a callback fired after the file is written.

https://github.com/gitbrent/PptxGenJS/pull/41

Hi @ellisgl ,
Thanks for opening an Issue and supplying the code!
Change has been implemented (I've added the filename as a return value to the callback function) and the nodejs-demo.js file has been updated with examples.

My presentations have a lot of images, so I use plenty of async/awaits while generating a PPTX, the "filename" callback fires too early in my case. I saw that a Promise for save is coming in 3.0 https://github.com/gitbrent/PptxGenJS/blob/master/CHANGELOG.md, is there already a quick fix I can apply to the code while waiting for the full release?

My temporary fix for the saveCallback firing too early for whatever reason in Electron is saving as a node buffer, then saving the buffer as a '.pptx' in my own app.
 pptx.save(fileNameNotUsed, buffer => {  fs.writeFileSync(fileName + '.pptx, buffer)                 }, 'nodebuffer')

Version 3.0 will be released very soon and returns a Promise with every save method.


Feature: bootstrap icons

As this is bootstrap based and bootstrap released their own icon set, I believe support for that set would be great. Unfortunately I'm not good enough yet to make a PR for it...

Are you talking about Bootstrap 4? Could you link the page with the icons you mention so I can take a look, please? Thank you!

Sorry for only getting back now. GitHub doesn't show me notifications when new issues are opened.

https://icons.getbootstrap.com/

Thanks for the link!
Will check it out as soon as I have time. It could be a good to add support for this icon set as well.

I am closing this issue for now because Eonasdan (the original maintainer of tempusdominus-bootstrap) has come back and maybe this integration will happen in the upstream project. We'll see what happens.
Cheers!


Need bundle UX

 Issue by chriskacerguis
Thursday Jan 22, 2015 at 15:04 GMT
Originally opened as https://github.com/projectjellyfish/ux/issues/109

We need to create a UX for the CRUD'ing of bundles.
Core: https://github.com/booz-allen-hamilton/jellyfish-core/issues/113

Need to rethink the bundle system.  Closing for now.


Show list of images

Is your feature request related to a problem? Please describe.
In Docker Desktop, we can see a list of images.  In OrbStack, I don't see that option.
Describe the solution you'd like
There should be a menu item on the left, along with Containers and Volumes that is Images. When you click it, you see a list of your Docker images.
Describe alternatives you've considered
In order to do this in terminal, I use docker image ls.  Just take that output and put in the UI.

Would love to be able to delete unused images using this

Would love to be able to delete unused images using this

You can do that with the CLI: docker image prune -a

Added for the next version.

A preview:

Looks great!  Thanks so much!


Implements Connection and Channel messages

Description
This PR implements all the Connection and Channel messages. There are a few Todos left in the code which is out of the scope of this PR but should be addressed in a followup one.

For contributor use:

[X] Unit tests written
[ ] Added test to CI if applicable
[ ] Updated CHANGELOG_PENDING.md
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments
[X] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #86 into master will increase coverage by 1.9%.
The diff coverage is 21.7%.

@@           Coverage Diff            @@
##           master     #86     +/-   ##
========================================
+ Coverage    14.5%   16.4%   +1.9%
========================================
  Files          58      59      +1
  Lines        2219    3137    +918
  Branches      761    1128    +367
========================================
+ Hits          322     517    +195
- Misses       1598    2031    +433
- Partials      299     589    +290

Impacted Files
Coverage Δ

modules/src/ics03_connection/error.rs
33.3% <0.0%> (-9.6%)
:arrow_down:

modules/src/ics04_channel/error.rs
25.0% <0.0%> (-8.4%)
:arrow_down:

modules/src/ics04_channel/packet.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/msgs/create_client.rs
0.0% <ø> (ø)

modules/src/ics23_commitment/mod.rs
0.0% <ø> (ø)

modules/src/ics04_channel/msgs.rs
20.8% <21.4%> (-1.5%)
:arrow_down:

modules/src/ics03_connection/msgs.rs
22.1% <22.0%> (+1.0%)
:arrow_up:

modules/src/ics03_connection/connection.rs
14.0% <33.3%> (+8.6%)
:arrow_up:

relayer/relay/src/config.rs
42.8% <100.0%> (+0.6%)
:arrow_up:

modules/src/ics24_host/validate.rs
73.0% <0.0%> (-1.0%)
:arrow_down:

... and 19 more

Continue to review full report at Codecov.

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


Write up 1 submission by Peter Carlip

As far as I can tell, my code works, and both of the tests come up fine.

Grade:
Problem 1: 2/2
Problem 2: 2/2
Output of testing/test_all.sh, showing differences in incorrect output:
Testing problem1
Success testing problem1 case 1 in 0.00 seconds.
Testing problem2
Success testing problem2 case 1 in 0.01 seconds.
Success testing problem2 case 2 in 0.01 seconds.
Success testing problem2 case 3 in 0.01 seconds.
Success testing problem2 case 4 in 0.01 seconds.
Success testing problem2 case 5 in 0.00 seconds.
Success testing problem2 case 6 in 0.01 seconds.
Success testing problem2 case 7 in 0.01 seconds.


Question about connection and channel

Hello,
Many thanks for your work! It's very helpful and easy to use it.
One question, does your implementation manage itself the connections pooling and channel reuse?
Best Regards,
Jérôme.

Hi, Jérôme!
If you don't mind, I'll answer relying on the implementation of the library that applies to the 5.0.0 version (and its betas).
Both IProducingService and IConsumingService implementations are registered in a singleton mode. Each of them use separate connections and those connections, obviously, are long lived ones (as the official RabbitMQ documentation advises you to do). Each service (IProducingService and IConsumingService) gets a channel created via "own" connection. And, of course, IConsumingService gets a consumer that is created via its "own" channel.
Basically, two connection and two channels for the lifetime of the application (excluding batch message handlers). So there is no custom pooling algorithms at all.
Default values for connection configuration are
public bool AutomaticRecoveryEnabled { get; set; } = true;

public bool TopologyRecoveryEnabled { get; set; } = true;

public TimeSpan RequestedConnectionTimeout { get; set; } = TimeSpan.FromMilliseconds(60000);

public TimeSpan RequestedHeartbeat { get; set; } = TimeSpan.FromSeconds(60);

and those values are used by the connection factory, so the created connection can manage network problems itself (at least try to).
But maybe I am missing something and do not clearly understand your question.
Best regards,
Antony


On changing current password admin/customer should get mail

Bug report
Title
On changing current password admin/customer should get mail.
Preconditions
1. framework Version. Master

Steps to reproduce:
1. step1:  change current password from admin/customer profile

Expected result:
admin/customer should get mail
Actual result
No mail

fixed


Create a new Starter project, Create a new Quick start Project

From @burrsutter on June 9, 2017 14:48
https://screencast.com/t/3B8NL94gq
These two buttons appear to lead to the same place, therefore one of them is superfluous.
I believe the rightmost Quick start option is the most important and therefore should be left-most (most of our end-users will read and prioritize left-to-right).
Copied from original issue: openshiftio/openshift.io#215

duplicates #1417


Documentation generation

#32 suggests to implement an automated way for documentation. So far, it is automated to the level of each module within the package. We need to manually add an rst file for each module that is added. I have automated the arg types so that typehints are used to automatically include that information, as well as the default values. One thing to keep in mind is that the return docstring doesn't support multi value output so it has to be wrapped within a tuple.
The matplotlib dependency is also within the pull request.

All looks good to me, so once the conflict is resolved, feel free to merge from my side. @rettigl and @zainsohail04? your reviews are still open, do you agree with the changes proposed?

Since I am the author, I can not also be the reviewer of this. However, I agreed with @daviddoji and changed the version to 3.8 already.

All looks good to me, so once the conflict is resolved, feel free to merge from my side. @rettigl and @zainsohail04? your reviews are still open, do you agree with the changes proposed?

Since I am the author, I can not also be the reviewer of this. However, I agreed with @daviddoji and changed the version to 3.8 already.

I think you'll need to checkout the poetry.lock and pyproject.toml files to resolve the conflicts

I don't overlook all changes here, but sure, go ahead. One further comment: Does in addition to the poetry stuff the standard pip installation still work?

Do you mean installing the dependencies from a requirements.txt files? That can easily be done, if people find it useful.
Do you mind @rettigl to open an issue so we keep track of this request?


Release 1.2.0.Beta15

Summary

https://github.com/quarkus-qe/quarkus-test-framework/pull/601
https://github.com/quarkus-qe/quarkus-test-framework/pull/603
https://github.com/quarkus-qe/quarkus-test-framework/pull/606
https://github.com/quarkus-qe/quarkus-test-framework/pull/608

Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[ ] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[ ] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

@michalvavrik Done


Implement SMTP notifier

This would indeed be very helpful.

This is the next one on my list. :)

This is done in #97. It's now merged in mainstream.


Implements Ragged Dot API

Background
Ragged Dot is a specialized matrix multiplication operation that is commonly used in the context of Mixture of Experts (MoE) models. MoE models are a type of neural network architecture that consists of a collection of independent expert networks, each of which is responsible for processing a specific subset of the input data. In order to determine which expert should process a given input, a routing mechanism is employed. Ragged Dot plays a crucial role in this routing process.
At the linear algebra level, Ragged Dot can be defined as follows:
Given matrices, A (m x k), B (g x k x n), and G (g), where m is the number of input samples, k is the dimensionality of the input features, and g is the number of experts, the Ragged Dot operation produces a matrix C (m x n), where each row of C corresponds to the weighted sum of the corresponding row of A and the columns of B associated with the expert assigned to that input sample.
More formally, the (i, j)-th element of C is computed as follows:
C[i, j] = \sum_{k=1}^{k} A[i, k] * B[g,k, j]

where k ranges over the columns of B associated with the expert assigned to the i-th input sample.
The key characteristic of Ragged Dot is that the rows of A and slices of B in the 0th dimension (of size g) are grouped into disjoint sets, with each set corresponding to an expert. This grouping structure allows the routing mechanism to efficiently assign input samples to the appropriate experts.
Requirements
Arguments:

lhs: (m, k) shaped array with integer or floating point element type. (REQUIRED)
rhs: (g1, k, n) shaped array with integer or floating point element type, where g1 denotes the number of local groups. (REQUIRED)
group_sizes: (g2,) shaped array with integer element type, where g2 denotes number of groups. The ith element indicates the size of ith group. (REQUIRED)
precision: consistent with precision argument for jax.lax.dot. (OPTIONAL).
preferred_element_type: the element type (jnp.dtype) for the output array. Consistent with preferred_element_type argument for jax.lax.dot. (OPTIONAL).
group_offset: (1,) shaped array. Indicates the group in group_sizes to start computing from. (OPTIONAL) If not specified, defaults to [0].
existing_output: (m, n) shaped array with elements of type preferred_element_type, where the output should we written to. (OPTIONAL) defaults to None, in which case the output should be written to a newly allocated array.

Results:

(m, n) shaped array with preferred_element_type element type.

Preconditions:

group_sizes = [s_1, ..., s_g2], where s_1 + ... + s_g2 <= m

g1 <= g2 (number of local groups <= number of groups).

If g1 == g2, group_offsets must be either unspecified or explicitly set to [0].
If g1 < g2, group_offsets must contain a single value in [0, g2) where group_offsets[0] + g1 <= g2. In this case, we perform g1 dots, where irrelevant slices of the results remain unchanged.

Assigning @mattjj, who has thought a bit about how to support ragged operations more broadly in JAX.

Removed uses of numpy functions from lax.ragged_dot
Added _CompileAndCheck tests in addition to self._CheckAgainstNumpy.

PTAL.

Implemented the ragged_dot as JAX primitive.

Addressed all the comments and replaced _reduce_sum() by contracting dimensions in the dot_general() call. PTAL.

Applied Matt's suggestion. Thank you all for the feedback!

Addressed Jake's feedback, thank you.

Looks great - the last thing we need before getting this submitted is to squash the changes into a single commit.


CAMEL-7378 Adding outputType to allow user to choose between a List or a...

... DBCursor while performing a findAll operation.

Thanks for this PR. I have just merged it. Do you mind closing this.
And you are welcome to help with the docs.

I will update docs this wend.


Rest api: update container nics with put request

AYS service has the support for update but we need to expose this feature over the REST API

Only NIC changes are implemented in the process change of the container actions, should I implement other data changes if so, which.
Also concerning this  AYS service has the support for update but we need to expose this feature over the REST API isn't it already exposed as execute blueprint  isn't it enough to execute a blueprint with new data to call the process change action, or there other functionality missing .

Actually the only "dynamic" feature of a container that u can update on the fly is NICS, by adding and removing nics. Name, tags, mounts and others are not changeable once the container is started

got it thank you

Steps

make sure that ovc configuration doesn't exist
create container without nic
use put API to put vlan nic

Issue
it gives right  error in ays  but it give 201 in put request and vlan network added when u get this container:
INFO:network:
[Wed16 14:18] - Job.py              :56  :j.core.jobcontroller.job.64ab5fa3a5c9fc2147491cf751057e2d - ERROR    - Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/tmp/actions/container/5db963e8dc61f38fa3bab5f9bf22429d.py", line 275, in processChange
    update(service, job.logger, job.context['token'], args['nics'])
  File "/tmp/actions/container/5db963e8dc61f38fa3bab5f9bf22429d.py", line 250, in update
    cl.nic_add(container.id, nic_dict)
  File "/usr/local/lib/python3.5/dist-packages/zeroos/core0/client/client.py", line 1120, in nic_add
    return self._client.json('corex.nic-add', args)
  File "/usr/local/lib/python3.5/dist-packages/zeroos/core0/client/client.py", line 726, in json
    result = self.sync(command, arguments, tags=tags, id=id)
  File "/usr/local/lib/python3.5/dist-packages/zeroos/core0/client/client.py", line 716, in sync
    raise RuntimeError('invalid response: %s' % result.state, result)
RuntimeError: ('invalid response: ERROR', STATE: ERROR
STDOUT:

STDERR:

DATA:
"ovs is needed for VLAN network type"
)

Software Version
8124cc51afc8f1f6e1431f50e4e83894954e4091

@grimpy
@muhamadazmy
@Dinaamagdy
options (i implemented both as instructed):

first implementation do the ovs check using  the golang client of core0 to check on containers with tag ovs,  this returns whatever status code i want but completly ignores the ays portion as it is done before the ays blueprint

second implementation check on the status of job if error return message with response of api call to blueprint , the message is error executing blueprint for container mycontainer23 update, the testers are upset with this cause it returns a 500 , which i cannot change so choose

this requires a version 8f7faca26027f10bcbdfca859ed3e691f57adcdd

it return error now ,but the nic added to container nic
Software version
e7383bcca187748a6ac6f7e54feb4fa5433e843b

Software Version
c682e501935c720d2c55cfb61ea2d9f8f792c5c2


Country Code Values Do Not Fit Within Combobox At Certain Screen Widths

Issue Type
Bug report
Component or Pattern Affected
md-combobox
md-input
Expected Behavior
The country code values fit within the combobox and encompassed input within all supported screen widths.
Current Behavior
Country code values are not fully displaying within the combobox and encompassed input at certain supported screen widths.

Possible Solution
Steps to Reproduce (for bugs)

Login into the app as a doctor on a small device that has a screen width of minimum of 320px (Ex: iPhone SE/5) using portrait mode
Click Make a Phone call
Select the country code dropdown list

Context
The md-combobox component is being used within the Instant Webex project as part of a feature allowing dial-out to meeting participants. We are leveraging Momentum's country code library in service of this as well.
Accepted screen widths for the Instant Webex project are not uniformly defined but in general seem have a floor of 320px width, with 550px width being the breakpoint for portrait mode display.
Your Environment
Local dev using the PROD Webex services as a backend.

@jameswhitely Have you tried setting a style override for that breakpoint by using the part selector we recently added? something kinda like
md-phone-input::part(combobox) {
  font-size: .5rem;
}


Advanced Search

New Search Dialog Window

Features

Advanced item filtering using multiple dimensions
You can uncheck all box in a category by right-clicking the label

Display match count during selection

Show/Hide categories at multiple level

You can expand/collapse categorie by left-clicking the label.

Reduce categories during selection On/Off

Logic operator : Or/And

Fulltext input search (flavor & itemset included)

Query persistence : don't loose time when you track something complex

Flexible visibility : Visible elements per category

Resizable window

Tooltip result preview

Tooltip "item count" per box

Fully integrated with current result window

UI fully translated EN/FR

Technical

Logging infra goes agnostic using Microsoft.Extensions.Logging.Abstractions
Translation rework : Every database translation pass through TranslationService checking resource file override.
Cleanup resource files using Database instead

Very nice, quite the new feature!
I'll cut a release before merging that one in. Also, I wouldn't mind a second pair of eyes to look through this, @ejfudd.

@EtienneLamoureux this will please players having hard time finding very specific items in there 10 years old character collection.
Usefull also for build crafting theory.

This looks great and will be an awesome addition to TQVault.
It' a pretty big change so it might take me a little bit to look at the changes.

I solved this by introducing a bool TryTranslateXTag(string tag, out string translation) in the service. Now there is no false interpretation of a failed translation.
And for racial tag, i give TranslationService the ability to redirect a tag to another that fit better.
https://docs.google.com/spreadsheets/d/19Wx83V39Y0tcdsOAFZmtb5GkEVHye7LahKvPj09uBXE/edit?usp=sharing

Great that's another way to do it.  It's probably better to have that stuff in the Translation Service for maintainability.  This might also fix #231 as well.

I think advanced search fix issue #146.

The last thing I would consider for fa0afbb would be to add the tag back into:
if (!TranslationService.TryTranslateXTag($"racialBonusRace{races[j]}", out var finalRace))
	Log.LogDebug("missing racialBonusRace={0}", races[j]);

As it is, a failed lookup will just be logged and the finalRace will be null.  Maybe just set
finalRace = races[j]; after the logging line so that at least the tag is shown.

@ejfudd @EtienneLamoureux Done !

I'll package a release later today.

@EtienneLamoureux I see you changed documentation.
But when i said this fix #146, it's not because you can use double quote notation. It's because you don't need it anymore, you can choose directly the mastery property you want with a check box. It's a fix by obsolescence of sort.


Added missing package in README

I updated the README to show packages that are actually in the package.json (paypal-rest-sdk, multer, serve-favicon, ordrin-api) (see https://github.com/saveman71/hackathon-starter/commit/d55bb05347b481b37ee4dc78fd841844a85c4b25 for a better diff).
I also improved lisibility by removing center align on tables (let me know if you don't want that) and adding spaces in the markdown (http://wbond.net/sublime_packages/alignment ftw).
Cheers

Thanks!


The following build commands failed

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
Please provide a clear and concise description of what the bug is. Include screenshots if needed.
Please test using the latest React Native release to make sure your issue has not already been fixed: https://reactnative.dev/docs/upgrading.html
React Native version:
Run react-native info in your terminal and copy the results here.
System:
OS: macOS 11.3.1
CPU: (8) arm64 Apple M1
Memory: 192.09 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.2.0 - /opt/homebrew/bin/node
Yarn: Not Found
npm: 7.13.0 - /opt/homebrew/bin/npm
Watchman: 4.9.0 - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.10.1 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 12.5/12E262 - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.1 => 0.64.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.
1  start project avec react-native init test
2 run cd test and run react-native run-ios
Expected Results
Describe what you expected to happen.
** BUILD FAILED **
The following build commands failed:
Ld /Users/drago/Library/Developer/Xcode/DerivedData/snapp-bpmblfwizbucetawlksqsqshuafo/Build/Products/Debug-iphonesimulator/snapp.app/snapp normal
(1 failure)
Snack, code example, screenshot, or link to a repository:
Please provide a Snack (https://snack.expo.io/), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
You may provide a screenshot of the application if you think it is relevant to your bug report.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

@dulmandakh Hello , i try your topic but nothing , i can't build my app..


[Feature Request]: number instead of select for charge rate

The feature
I don't know what are the implications on this, neither why it was decided first to use a select for charge rate, but I think it would be more approriated to have the select.openevse_charge_rate entity replaced by a number.openevse_charge_rate
Using a slider on UI gives a better UX than a selector, and stays in sync with Openevse UX.
Best
Additional information
No response

Has this changed? I see that select.openevse_max_current is no longer available in the HA integration.

Has this changed? I see that select.openevse_max_current is no longer available in the HA integration.

Please review the release notes: https://github.com/firstof9/openevse/releases/tag/2.1.46

Hey
I'm testing your commit. Thanks for this.
The number slider is sluggish as it waits for update before moving.
I think there's a way to make this entity optimistic,
This way slider takes current setted value, and will update if needed when state return back instead of waiting for it.
Also if you want to stay in synch with OpenEVSE UI behavior, when the number is set back to default max current, it should remove the "charge_current" property from the overrides.

It is using the soft_max_current as the value.

The charge rate slider follows the charge rate as it shows in the OpenEVSE UI this is as intended.
Maybe you are looking for the sensor.openevse_max_current to provide the max current sensor you're looking for?

the charge_rate value shouldn't be settable then and is not what is used on openevse UI.
I know the api is a pain, i had some headache to find some good logics for simple needs :)
What you call charge_rate is the "pilot" in openevse
It follows on openevse UI on your side because you're not using the current shaper so pilot is not controlled by another service.
But when the current shaper run, or any another service with higher priority than override ( I think OCPP is too ) , they can control the Pilot too.
So if the number.charge_rate follows Pilot, then your setting slider will move alone following Pilot changed by the shaper itself not by the user.
That's why I had to do it differently on openevse UI.
Here is the logic of how I've made the OpenEVSE UI main tab slider :
Changing value set the "max_current" property as an Override.
If user manually set it back to 32A ( = "max_current_soft" from the /config ) , it then remove the override property as it is useless to keep it.
The displayed value in the slider is the "charge_current" property from the /override. If it's not present then it takes "max_current_soft" from /config

This means on your plugin

number.charge_rate perhaps should then be renamed in number.charge_current and use the logic above by taking its state on /override endpoint, charge_current prop, and if no override from /status "max_current" and not from Pilot value
pilot value should be exposed as sensor ( and not number.charge_rate ) aka sensor.openevse_pilot
sensor.openevse_max_amps and sensor.openevse_max_current seems to be the same value , are they ?
those are not the max amp value you set from number_charge rate but comes from the configuration (  max_current_soft ), perhaps they should be rename to sensor/openevse_config_max_current to claerly indicate it's origin. ( Same for configuration value if they are exposed but i don't think  so )

sensor.openevse_max_amps and sensor.openevse_max_current seems to be the same value , are they ?

No
Max Amps is the hardware max as sent by OpenEVSE.
Charge rate controls the charge rate as indicated in the Web UI:

Max Current is indicated here:

When your shaper is active does the charge rate from the first screen move around? Mine does when I have PV divert set with eco mode going.

I have made the openEVSE UI mate ;)
Yes when shaper change the available current, the slider charge rate change on HA, not on openevse UI ( because it works as I explained above ).
I know it's as confuse as the openevse API is , would you want we chat about this directly , that would be easier to explain if I was not clear.

Charge rate controls the charge rate as indicated in the Web UI:

It is not reallly.  The slide rfrom your first screen ( the max current from main tab ) , is working like detailed above. It creates /override with charge_current property.
The number.charge_rate  creates an override like on the UI, but the value displays pilot not the /override property that has been set.

Ok i get it, I have tested by simulating PV data:

On PV divert case, it's normal you see the slider following the available current, as Divert control the charge_current property too.
( Shaper control the max_current prop )
When divert as set the charge_current, it has bigger priority than override in eco mode.
Then on the OpenEVSE UI, the slider is locked, you can't set the value, as it's useless since the PV use it.
You can see the little tooltip above, mentionning this value is now controlled by Divert, and the disabled slider.
But with shaper it's different, shaper is a security feature, it set the max_current, but you still can set the charge_current.
It means until shaper keep max_current above the charge_current setted by override, charge_current is used as pilot value.
But when max_current from shaper goes under this value, the pilot can goes above. Still charge_current prop is active, but superseded by the max value.
On UI side, the charge_current slider is still usable then as the max_current set by the shaper follow the current available it calculated

The max current from the second screen shouldn't be accessible from HA anyway, as it's a one time configuration. This one is written to the flash so if people use it on their script it can kill the flash if there's lot of changes.

This is why it's not configurable via HA and it only shows up under the diagnostics area.

You can see the little tooltip just under, mentionning this value is now controlled by Divert, and the disabled slider.

I can mimic this in the integration as well.
Part of the issue is the API docs for the shaper values doesn't exist either. I'm trying to work off of what's available on https://openevse.stoplight.io/docs/openevse-wifi-v4/e3153e2367cb1-open-evse-wi-fi-api and then anything else I have to dig through the firmware code.

Don't hesitate to bump me if you need help or clarification on the API. I had to leave for a year the openevse project because of personal life changes, but I still have it in mind. Spent so much time on this.
I can make you some few scheme to summerise the logic used on UI and how it's handled on OpenEvse.
I'll need first to point the values and actions exposed on your integration with those in openevse first.
Reading what I've written here is really not clear :)

I guess I need to see if there's a way to set the shaper_max_pwr via the API.

shaper max power is a fixed value from the conf , if you meant shaper_live_pwr, you already can feed it over the api with POST on /status :
 {"shaper_live_pwr": xxx}


fix(cat-voices-fronted): dart warnings

fix a few dart warnings
fix formatting to align with project recommendations

Merged without a review, attempting to fix CI to publish the latest front end of cat-voices with updated Nginx configurations.


[BigQuery] Schema autodetection does not seem to work when loading JSON data via the Java API

When trying to import a JSON file into BigQuery from GCS using schema autodetection via
LoadJobConfiguration.builder(table, path)
      .setAutodetect(true)
      .setIgnoreUnknownValues(false)
      .setMaxBadRecords(0)
      .setFormatOptions(FormatOptions.json())
      .setCreateDisposition(CreateDisposition.CREATE_IF_NEEDED)
      .setWriteDisposition(WriteDisposition.WRITE_TRUNCATE)
      .build()

I receive the error "Schema has no fields". I would expect the schema to be autodetected (see also https://cloud.google.com/bigquery/docs/schema-detect).
Tested using version 1.37.1 of google-cloud-bigquery.

Not sure if this might be related to #3425

I wasn't able to recreate this. I ran:
    String tableName = "statesjson";
    TableId destinationTable = TableId.of("macbeth", tableName);
    LoadJobConfiguration configuration =
        LoadJobConfiguration.newBuilder(destinationTable, "https://storage.googleapis.com/cloud-samples-data/bigquery/us-states/us-states.json")
                .setAutodetect(true)
                .setIgnoreUnknownValues(false)
                .setMaxBadRecords(0)
                .setFormatOptions(FormatOptions.json())
                .setCreateDisposition(JobInfo.CreateDisposition.CREATE_IF_NEEDED)
                .setWriteDisposition(JobInfo.WriteDisposition.WRITE_TRUNCATE)
                .build();
    Job remoteLoadJob = bigquery.create(JobInfo.of(configuration));
    remoteLoadJob = remoteLoadJob.waitFor();

Which ran fine, and I verified in the console that the results were valid. Are you still experiencing this issue? Could it be related to the specific JSON file you're using?

Closing due to being unable to reproduce and inactivity on the issue, feel free to reopen if you're still experiencing this issue


Increase timeout on ReplicatorTest#testCloseReplicatorStartProducer

A good run takes around 3.5 seconds, so it doesn't take much to push
it past 5 seconds. Increased timeout to 15 seconds.

retest this please

retest this please