Use logger for helm output

My previous PR was auto-closed: https://github.com/helmfile/helmfile/pull/1386
This PR updates all helm invocations, so the output is written to stderr via the logger only.
Only when templating, we write directly to stdout.
This PR aims to improve the experience when using helmfile as a go library.

@mumoshu WDYT?

@inteon Thank you very much for your effort!
This looks almost good. Only two things:

Can we also make the logging output configurable https://github.com/helmfile/helmfile/blob/442e8647a7d20ec71c54b9523c0a614fdc7681bc/cmd/root.go#L63
so that anyone wants logs to flow into stdout as before can configure to do so for some form of backward-compatibility
Can we have test outputs for the combo of stdout/stderr, for example, named apply-combined and apply-live-combined and make tests to check those? The rationale is that, by moving some messages from stdout to stderr, we lose some test coverage. I hope we can keep the same degree of test coverage even after this change.

@inteon Thank you very much for your effort!
This looks almost good. Only two things:

Can we also make the logging output configurable https://github.com/helmfile/helmfile/blob/442e8647a7d20ec71c54b9523c0a614fdc7681bc/cmd/root.go#L63
so that anyone wants logs to flow into stdout as before can configure to do so for some form of backward-compatibility
Can we have test outputs for the combo of stdout/stderr, for example, named apply-combined and apply-live-combined and make tests to check those? The rationale is that, by moving some messages from stdout to stderr, we lose some test coverage. I hope we can keep the same degree of test coverage even after this change.

@inteon Thank you very much for your effort!
This looks almost good. Only two things:

Can we also make the logging output configurable https://github.com/helmfile/helmfile/blob/442e8647a7d20ec71c54b9523c0a614fdc7681bc/cmd/root.go#L63
so that anyone wants logs to flow into stdout as before can configure to do so for some form of backward-compatibility
Can we have test outputs for the combo of stdout/stderr, for example, named apply-combined and apply-live-combined and make tests to check those? The rationale is that, by moving some messages from stdout to stderr, we lose some test coverage. I hope we can keep the same degree of test coverage even after this change.

@mumoshu I applied both suggested changes.

@mumoshu


213 an organization can edit update their profile

🔗 Issue
213
✍ Description

Adds a new table, organization_profile, which connects organization to location. On top of that, the new table contains two new fields email and phone_number.
A new controller and two new actions were introduced, along with the respective views, so that the user can edit and update this new relation.
I didnt add any integration test as it seems like we are skipping all of them. If I should add them anyways, just comment, and Ill create a follow up PR, as this is already big enough
📷 Screenshots/Demos

Errors:

@Ptrboro thanks for the nice comments! Made the changes you mentioned. About the form, Ill follow this PR up with another one adding bootstrap_form, as this PR is already too large

Just testing this works
@all-contributors
please add @yuricarvalhop  for code.
please add @Ptrboro  for code review.

I'm going to merge it now as it's in good shape and will help unblock other work. 🚀
@edwinthinks, if you spot anything for revision, lets address it separately.


Add Conda install and register package with anaconda.

Once we are installable using pip we can create a meta.yaml file that tells conda how to build the rotary package. We can then register it with the Anaconda package repository.

@jmtsuji I assume you would be handling the registering and uploading the file to Conda? I can create a meta. yaml file for you.

@LeeBergstrand I'm OK either way -- I can handle registration and uploading to conda, but if it simplifies things for you, you are also welcome to do this. Let me know your thoughts. Either way, that would be great if you could make the meta.yaml file -- thanks!

@jmtsuji So once I got the meta.yaml setup I give you a walkthrough of how to setup the upload to conda. The build process is a bit complicated.

OK, sounds good -- thanks


Add missing bits for undercloud-process-count validation

Added a line to the README file documenting all available validations
and added the validations to the stages used for API PoC.

Nahh, I merged too soon. I should have caught it during code review.


Support django 1.7+ Appconfig.ready()

Django 1.7 has a new AppConfig object. One of the good things in there is the ready() method for actions that should be taken once Django has loaded. See https://docs.djangoproject.com/en/1.7/ref/applications/#django.apps.AppConfig.ready
This would be the perfect place for the djcelery.setup_loader() code that you now have to place in your settings file.
Or are there technical reasons why this wouldn't be possible?
Otherwise I could look if I could make a pull request for it.

i think it's already fixed


Unable to register a global handler for all unhandled rejected promises

One good thing for debugging is that all unhandled rejections are printer to the console in current browsers. But for this polyfill it seems this is not happening/there is no way to register a global handler which should be called if nothing handles a rejection?

@mitar Thanks for the suggestion. I just released 4.0.1 with this ability. Please let me know if this fixes your issue.
https://github.com/taylorhakes/promise-polyfill/releases/tag/4.0.1

I have made a followup pull request: https://github.com/taylorhakes/promise-polyfill/pull/26


Gasmeter not logging

Hi All,
I'm struggling to get my gasmeter working, energy consumption is working correctly.
Output from smartmeter:
---BEGIN--
0DA4
/XMX5LGBBFG1009264202

1-3:0.2.8(42)
0-0:1.0.0(160206093352W)
0-0:96.1.1(4530303331303033313732393931323135)
1-0:1.8.1(000022.940*kWh)
1-0:1.8.2(000035.495*kWh)
1-0:2.8.1(000000.000*kWh)
1-0:2.8.2(000000.000*kWh)
0-0:96.14.0(0001)
1-0:1.7.0(00.349*kW)
1-0:2.7.0(00.000*kW)
0-0:96.7.21(00001)
0-0:96.7.9(00000)
1-0:99.97.0(0)(0-0:96.7.19)
1-0:32.32.0(00000)
1-0:32.36.0(00000)
0-0:96.13.1()
0-0:96.13.0()
1-0:31.7.0(002*A)
1-0:21.7.0(00.349*kW)
1-0:22.7.0(00.000*kW)
0-1:24.1.0(003)
0-1:96.1.0(4730303235303033333236333939303135)
0-1:24.2.1(160206090000W)(00060.913*m3)
---END----

In p1meterdata.js I have this regex;
   var gasTotalUsage = returnRegExResult(data, /^0-1:24\.2\.1\(\d+[SW]\)\((\d+\.\d+)\*m3\)/m);
As far I can test the regex is working: https://regex101.com/r/kY1pE6/1
The status of gas usage is saying "Unknown" , any suggestions were to look ? I think i'm missing something.

you need to edit p1meterdata.js to :
var gasTotalUsage = returnRegExResult(data, /^0-1:24\.3\.0(.*)\(m3\)[\r]?[\n]?\(0+(\d+\.\d+)\)/m, 2);

I did get it working with this regex;
var gasTotalUsage = returnRegExResult(data, /^0-1:24.2.1(?:.*)(0+(\d+.\d+)/m, 1);
Thanks


How does 'intrinsicsize' work with 'object-fit'

This concern was bought up here. Images get distorted when using object-fit.
@dvoytenko proposed a feasible solution could be forcing object-fit:contain instead of the default object-fit:fill when 'intrinsicsize' is specified.

Squishing images is problematic for photographs, but it's not that crazy for vector graphics. So it's not clear to me that this would be Web-compatible.

@fantasai to clarify:

The squishing would happen only if the actual size is different from declared size.
Hopefully, it's a default at the lowest level of specificity and can be easily changed in the stylesheet.
It's there to avoid stretching or otherwise deforming the image, which is arguably worse.

@dvoytenko What I'm saying is that this behavior of images, that they deform when resized to a different aspect ratio, has been built into how the Web renders since the mid 1990s. While on some pages it may be an unintentional error, on others it's probably intentional. I don't think we can change this default behavior.

@fantasai I think your comment might be more relevant for #4. If you agree, please repost there. The explainer proposes to add a new attribute intrinsicsize and derive the new behavior and default styling from it. It's defined this way to explicitly avoid backward compatibility issues. The #4 makes a case to change the current behavior and that debate is still ongoing there.

@dvoytenko I'm not convinced changing object-fit based on the presence of intrinsicsize is a good idea. It's an unexpected interaction, and one more thing for authors to puzzle over...


Fix delay instruction error

Summary
Recent PR https://github.com/Qiskit/qiskit-ibm-provider/pull/346 causing integration tests to fail with
AttributeError: Instruction delay is already in the target
https://github.com/Qiskit/qiskit-ibm-provider/runs/6336245239?check_suite_focus=true#step:5:662
Do we need to add a check to see if the delay instruction is already present?
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.3%) to 29.881%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_provider/utils/backend_converter.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_provider/utils/backend_converter.py
1
15.22%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
1653

Relevant Lines:
5532

💛  - Coveralls


Add a design document for the QUIC (and generic) Record Layer

This is a preliminary design document for the QUIC (and generic) Record Layer implementation options.
Checklist

[x] documentation is added or updated

Although the general idea was implemented IMO this design document is rather misleading. I am closing this.


capture-error-trace doesn't log application errors

Bug Description
It seems to only capture mirrord's errors, but if mirrord behaves 'normally' but causes the application to malfunction, we miss it. It also doesn't seem to show the log with the github link.
Steps to Reproduce
Run mirrord with capture-error-trace, e.g.:
RUST_LOG=mirrord=trace mirrord exec --target deployment/busybox --steal --no-fs --capture-error-trace -x=*  -- npx ts-node src/repro.ts
Backtrace
No response
Relevant Logs
No response
Your operating system and version
MacOS Ventura
Local process
npx
Local process version
No response
Additional Info
No response

not sure if we can do that tbh.. the app writes to stdout/err and making it work sounds complicated.

I'm not sure it's worthwhile to have the feature otherwise - might lead people to just send us unhelpful logs (rather than just running with RUST_LOG=trace and copy-pasting).


Does not "next_epoch_itr(shuffle=True)" shuffle within each batch nor within the entire dataset?

https://github.com/pytorch/fairseq/blob/b5f41f828b0ec9b67fa60aceb0778073d1b368b2/fairseq/data/iterators.py#L177
The comment above seems "next_epoch_itr(shuffle=True)" shuffles only between batches.
I just want to make sure that it doesn't shuffle data within each batch nor within the entire dataset.
Thank you in advance.

@mjeensung Yes that only shuffles batches.
In fairseq, we shuffle dataset and create batches only once at initialization. And then at the start of each epoch the batches are shuffled.
This was done for performance considerations.

@mjeensung what if I need to create batches each epoch, and set shuffle to False for the dataset and the iterator, how can I reach this?? I want at the beginning of each epoch, a new cal to ordered_indices of my dataset is called. However, I do not know how to achieve this.


Is that dxcore_interface.h and dxcapi.h can turn into idl files, so that they can be used by both c/c++ codes

There's no plans to convert those to IDL, no.

I have created dxcore.idl for mesa but not in mingw-w64 repo right now. The main hurdle is to declare the method overloads in an interface. Some suggested to use cpp_quote in interface declaration but I have not tested it yet (some doubts).

Do you have any project that uses dxcapi.h?


🌱 Upgrade golangci-lint to v1.47.2 and fix its issues

What this PR does / why we need it:
The PR upgrade golangci-lint to the latest and fixes the issues found in it
https://github.com/golangci/golangci-lint/pull/2999
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #291
Fixes https://github.com/kubernetes-sigs/cluster-api-provider-nested/pull/290#discussion_r925997669 and adds the goheader linter to catch Copyright issues

/lgtm
/approve


Strange VT related crash on embedded ESP32-S3 application

Describe the bug
I used the current main branch with the 'vt-version3-object-pool' and removed the [] { return true; } part which was introduced in this commit: https://github.com/ad3154/ISO11783-CAN-Stack/commit/046461bfb2276d9fb3a107de2d3076d862666301 but apparently blocks the application from reaching the watchdog timer and results in the watchdog timer raising exceptions.
When running the slightly adapted 'vt-version3-object-pool' example:

It runs fine when you don't initialize the VT, so the vt doesn't update but the other stuff such as address claiming still works
With the VT initialized it runs until state=13 (WaitForGetVersionsResponse) in the update() function and somewhere immediately after that the application crashes and reboots without any usable information.

Supporting Documentation
The latest branch that I was able to get it all working fine was at 5b159f69596254c1a879b79388474ff45f204739. Somewhere since then we must have introduced something which badly corrupts memory I think?
Environment (please complete the following information):

OS: ESP-IDF
Compiler: PlatformIO
CAN Driver: TWAIPlugin

Out of curiosity, does it work if you bypass the versioning by removing the version from the set_object_pool ?

Uhmmmm... very vague stuff. It fails now anywhere from 5 seconds to 30 seconds from start of uploading the pool and it's taking ages to upload a pool compared to that "stable" commit https://github.com/ad3154/ISO11783-CAN-Stack/commit/5b159f69596254c1a879b79388474ff45f204739. Like not even 10% uploading in 30 seconds compared to the whole pool uploading within 20 seconds before. Any ideas?

Hmmmmmm....  I would maybe suggest 0eeba9fa18454f43981e82020739395bc0c5dfa4 as I have seen some interesting things happening since #163 with some of the mutexes... maybe you are getting deadlocked.

For example, on Windows with PEAK, if the bus is down when I call start it murders the mutex while locked which causes a nasty exception deep in the Windows files....

Another quick update; when I add 1 extra logging statement in VirtualTerminalClient::process_rx_message it crashes already after reaching state=2 (SendWorkingSetMasterMessage), these are the changes I made btw: https://github.com/ad3154/ISO11783-CAN-Stack/commit/c20da6fdc0ed1f1d11c7cf05a3dbbf42dd7c3cf8

Good tip! I'll try some branches in between while I am at it

https://github.com/ad3154/ISO11783-CAN-Stack/commit/0eeba9fa18454f43981e82020739395bc0c5dfa4 still same issue, but instead of crashing it freezes now
https://github.com/ad3154/ISO11783-CAN-Stack/commit/46324fcdc93964c916c260d3bac6b6f3bca8f682 no crashes but still extremely slow upload speed, so slow that we even receive aborts from the VT
https://github.com/ad3154/ISO11783-CAN-Stack/commit/c9a4ef439de7c64a69a7ab9cf4b2f53e88b8b97f still slow but no aborts and it was able to upload to 100% after ~8 mins but with an error in the object pool?
https://github.com/ad3154/ISO11783-CAN-Stack/commit/5b159f69596254c1a879b79388474ff45f204739 which was the stable branch is still super slow lol?

I then started testing on my personal project, where it was able to upload the same pool in approx 1 min, which is still pretty slow and it had to retry a few times because it received aborts. I guess our TWAIPlugin has never been really reliable nor fast.
So I guess I will do some more testing/bugfixing tomorrow:

[ ] Compare the current vt_version3_object_pool example with the previous version/my project and try to find the bottle neck because it is now about 8x slower.
[ ] Try to fix the unreliable upload with the TWAIPlugin.

Still the origin of the crash is somewhere introduced between https://github.com/ad3154/ISO11783-CAN-Stack/commit/0eeba9fa18454f43981e82020739395bc0c5dfa4 and https://github.com/ad3154/ISO11783-CAN-Stack/commit/46324fcdc93964c916c260d3bac6b6f3bca8f682. To compare: https://github.com/ad3154/ISO11783-CAN-Stack/compare/46324fcdc93964c916c260d3bac6b6f3bca8f682...0eeba9fa18454f43981e82020739395bc0c5dfa4

Bruh... found why it's so slow 😂
CANHardwareInterface::set_can_driver_update_period(100);

I forgot that I had set it so slow haha, I changed it when it was crashing to check if that caused the crashes, but forgot to change it back.

Hahaha, yeah that would do it! 😂

Fixed:
The issue was, as far as I understand, that the main update thread of the HardwareIntegration part of this stack did not have enough memory available to do all it's tasks and so it crashed, therefore it was needed to increase the CONFIG_PTHREAD_TASK_STACK_SIZE_DEFAULT value of the ESP32 in the menuconfig. (I increased it from 3072 to 6144).
We could add this to the FAQ one day if there are more people having this issue... Cus I found it damn hard to find the cause of the crash


enable HTTP events by default

Closes #556.

Tried testing the changes using the http integration test and some of the checks are failing as expected but the HTTP/1.1 tests are unexpectedly failing. They're running scoped cURL configured to connect via SCOPE_CRIBL to a local instance of LogStream which is configured to send everything to JSON files in /tmp/.
When I manually test with SCOPE_CRIBL pointing to a local TCPserver or my Cribl.Could instance, I'm getting the HTTP/1.1 events as well as the http-metrics events.  Curious...

We tagged this branch v0.7.4-rc2 to publish the image but we're not going to merge it as the changes made here are already being incorporated into the work on #543.


Scraper still working ?

I am using the scraper since a few weeks for a project. The program was working fine, however since yesterday I am not able to retrieve data from Twitter anymore...
For instance when I am using the example : twitterscraper Trump --limit 1000 --output=tweets.json I got 0 tweets.
Same when using query_user_info, I got now NoneType object.
Did Twitter fix something about scraping with BeautifulSoup ?
Thanks for your answer and help.
Best regards,
Alex Bietrix

I ran into the same issue.

Maybe is the same issue reported and solved here:
https://github.com/taspinar/twitterscraper/issues/296

Maybe is the same issue reported and solved here:
#296

Yes thank you ! It works now to retrieve the tweets. However I get this error when I want to have user info : local variable 'user_info' referenced before assignment

Maybe is the same issue reported and solved here:
#296

I tried this solution today. It does not seem to work anymore

#296 does not work anymore. I tried today and keep receiving the same thing "0 tweets"

Yea, #296  stopped working. Would be great if someone can help!
Thanks!

Please try this and let me know how it works for you https://github.com/taspinar/twitterscraper/pull/302

Duplicate of https://github.com/taspinar/twitterscraper/issues/296

Maybe is the same issue reported and solved here:
#296

Yes thank you ! It works now to retrieve the tweets. However I get this error when I want to have user info, using query_user_info : local variable 'user_info' referenced before assignment

have you solved the problem?


[Bug]: Inconsistencies between v5/types.json and typespec

Describe the bug
By utilizing the emitter typespec-autorest, we have the capability to transform typespec into swagger 2. In the case of the generated swaggers, they will incorporate certain shared definitions from v5/types.json. However, there exist discrepancies between types.json and typespec, encompassing aspects such as naming conventions, readonly properties, required fields, descriptions, and more. For a comprehensive understanding of these inconsistencies, please refer to the following link: https://github.com/Azure/azure-rest-api-specs/pull/28717. This pull request outlines the necessary changes to ensure the consistency of types.json with typespec.
Except for above, there is another one, which is in typespec, there is an empty model https://github.com/Azure/typespec-azure/blob/main/packages/typespec-azure-resource-manager/lib/arm.foundations.tsp#L40, while in swagger, we do not have this model.
And we will need to fix those inconsistencies either in typespec or in swagger.
Reproduction
To find those inconsistencies, you just need to refer to my drafted pr to find those typespec types in https://github.com/Azure/typespec-azure/tree/main/packages/typespec-azure-resource-manager/lib.
Note: If the decorator @friendlyName is specified, the frendlyName will be used.
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Closing in favor of the issues above


support chunking for MITgcm Field dataset

currently, the NetCDF dimensions of fields from MITgcm are not registered in the _name_map of NetcdfFileBuffer. This needs to be added.

Same thing would be good for MOM data

Hi Angus,
Thanks for the info. I am still re-thinking this process, in some shape or form - we already have the discussion somewhere.
Now, how pressing is the issue for the MOM data ? If you need that support immediately, then it would be nice if you open a PR and add your dimension names for MOM to the name_maps (l. 1660 - l. 1664). Ideally, you would also design a test similar to the ones in example_dask_chunk_OCMs.py. Erik and me are right today brainstorming how to ideally cluster those tests and examples where data are not freely distributed, but which still demonstrate data format usage etc. - so we'd be happy to also add that script there.
Otherwise, if it still has a week or two time, then you, Erik and me will have figured out a better way how to do this name mapping more versatile.
Cheers,
Christian

we created a new repository for those user-contributed and not necessarily 'main branch'-scripts at https://github.com/OceanParcels/parcels_contributions, where I put the MITgcm test.


pkg/obfuscate: improve formatting and string parsing in the SQL obfuscator

The SQL obfuscator inserts spaces between parsed tokens. This is often
fine, but it makes some queries very difficult to read. For example, the
query 'select [user].[name] from [users];' becomes 'select [ user ] . [
name ] from [ users ] ;'.

The obfuscator has also been removing the quotation characters (' " `) from
strings and identifiers. These quotation characters are necessary for users
to understand and investigate their queries. Without them, necessary
context can be lost. For example, the query 'select `user`.`full.name` from
`public.users`;' becomes 'select user.full.name from public.users;'. The
query is now ambiguous, since we can't tell whether the dots are part of
the identifier.

This commit fixes both of these issues. It eliminates some unnecessary
spaces when writing out an obfuscated query, and it also preserves the
quotation characters associated with strings and identifiers.

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
The SQL obfuscator is heavily unit-tested with many queries. I have reviewed all queries which have had their outputs changed and made sure that they are an improvement.
To test, we should run sample queries through a traced application and ensure that they appear correctly in the UI.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

Can you fill out the Possible Drawbacks / Trade-offs section in the commit message so we have that context?
Is this a high risk change in any way? I'm wondering if this is something that should be behind a feature flag or something similar.


nbytes does not return the true size for sparse variables

This wasn't entirely surprising to me, but nbytes currently doesn't return the right value for sparse data -- at least, I think nbytes should show the actual size in memory?
Since it uses size here:
https://github.com/pydata/xarray/blob/a0c71c1508f34345ad7eef244cdbbe224e031c1b/xarray/core/variable.py#L349
Rather than something like data.nnz, which of course only exists for sparse arrays...
I'm not sure if there's a sparse flag or something, or whether you'd have to do a typecheck?
Minimal Complete Verifiable Example:
import pandas as pd
import numpy as np
import xarray as xr

df = pd.DataFrame()
df["x"] = np.repeat(np.random.rand(10_000), 10)
df["y"] = np.repeat(np.random.rand(10_000), 10)
df["time"] = np.tile(pd.date_range("2000-01-01", "2000-03-10", freq="W"), 10_000)
df["rate"] = 10.0
df = df.set_index(["time", "y", "x"])

sparse_ds = xr.Dataset.from_dataframe(df, sparse=True)
print(sparse_ds["rate"].nbytes)

8000000000

Anything else we need to know?:
Environment:
Output of xr.show_versions()
INSTALLED VERSIONS
------------------
commit: None
python: 3.7.9 (default, Aug 31 2020, 17:10:11) [MSC v.1916 64 bit (AMD64)]
python-bits: 64
OS: Windows
OS-release: 10
machine: AMD64
processor: Intel64 Family 6 Model 158 Stepping 9, GenuineIntel
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: None.None
libhdf5: 1.10.5
libnetcdf: 4.7.3

xarray: 0.16.1
pandas: 1.1.2
numpy: 1.19.1
scipy: 1.5.2
netCDF4: 1.5.3
pydap: None
h5netcdf: 0.8.0
h5py: 2.10.0
Nio: None
zarr: 2.4.0
cftime: 1.2.1
nc_time_axis: None
PseudoNetCDF: None
rasterio: 1.1.2
cfgrib: None
iris: None
bottleneck: 1.3.2
dask: 2.27.0
distributed: 2.30.1
matplotlib: 3.3.1
cartopy: None
seaborn: 0.11.0
numbagg: None
pint: None
setuptools: 49.6.0.post20201009
pip: 20.3.3
conda: None
pytest: 6.1.0
IPython: 7.19.0
sphinx: 3.2.1

I guess it could be possible to do a typecheck for this case. It's tricky to have a smart nbytes that works in all cases, though. For example, it's not really possible to get this information for dask arrays with sparse chunks (https://github.com/dask/dask/issues/5313).

I guess it could be possible to do a typecheck for this case. It's tricky to have a smart nbytes that works in all cases, though. For example, it's not really possible to get this information for dask arrays with sparse chunks (https://github.com/dask/dask/issues/5313).

We should probably be pulling xarray's nbytes from nbytes attribute on arrays. Then at least we can leave this calculation up to the backend array.

We should probably be pulling xarray's nbytes from nbytes attribute on arrays. Then at least we can leave this calculation up to the backend array.


Fixed: Cleanup of metadata for Linux services

With this PR we clean up the metadata for Linux services. The name is the service name
and not the executable path, unless if the service name is not provided.

Codecov Report

Merging #167 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   55.73%   55.73%
=======================================
  Files          30       30
  Lines        3913     3913
=======================================
  Hits         2181     2181
  Misses       1579     1579
  Partials      153      153

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 b73822f...ba8dcb6. Read the comment docs.


Add SQL store option to token store

This does a couple of few things:

Makes the token store honor the kind of session store that's been configured in the global settings
Adds support for a SQL session store, if that's been configured
Updates some existing tests to use both memory and sql to ensure good integration
Updates the purger to respect the ttl of the token, not the updated_on value

This is kind of a naive implementation that might have some issues. The most glaring is perhaps that we purge the Sessions based on a global ttl, meaning that this won't respect the ttl that is configured in the API/elsewhere, and that may not be easy to change. Opening this as a WIP to aid discussion
@miq-bot add-label api, core, enhancement
@miq-bot assign @gtanzillo
Fixes https://github.com/ManageIQ/manageiq/issues/14882
Fixes https://www.pivotaltracker.com/story/show/130490379
/cc @martinpovolny

@miq-bot rm-label wip

@gtanzillo @abellotti I've fleshed this out into something more workable. LMK what you think. Probably best reviewed ignoring whitespace.
There were quite a few opportunities to refactor along the way with this one but I resisted as it is already quite large. You may see some further duplication added (e.g. the knowledge of how we serialize/deserialize data in sessions) along the way....because this PR is already quite large I think it would be best if I tackled that in a follow up.

Adds support for a SQL session store, if that's been configured

Unless I'm mistaken, isn't this what https://github.com/rails/activerecord-session_store does? Can't you utilize that instead?

(we already use it in the project, I mean)

@chrisarcand we do.....but that doesn't have an abstraction layer at the level we need. It just provides an adapter for action dispatch which interacts directly with the model. The token store code is coupled to this interface, which the other adapters provide, so I'm adding an adapter for that here.

@gtanzillo @abellotti any feedback on this? It's currently red only because brakeman doesn't like the Marshal.load - that's somewhat unavoidable but in theory harmless because we what goes into that marshalled data is controlled by us. So we could whitelist that - I can go ahead and do that if the approach outlined above is agreeable.

@miq-bot add-label fine/yes

@imtayadeway Is there a BZ for this?  Can you please create if it doesn't exist?

@simaishi here's the BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1459674 (hope I interpreted this correctly)

Fine backport details:
$ git log -1
commit 9503a9a3ac59fa77f3a2cf72e5a9fcc13735bfeb
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Tue May 30 18:09:58 2017 -0400

    Merge pull request #14947 from imtayadeway/api/add-sql-store-to-token-store

    Add SQL store option to token store
    (cherry picked from commit ffd89722751ebc3fb0498a24a288b90411eea2ac)

    https://bugzilla.redhat.com/show_bug.cgi?id=1460348


Fixed typo in nl.toml

Kubernates >> Kubernetes

@sftim pleasure, please do let me know if i can contribute more in any way possible 🙂

Hi @abhigyadufare thanks for the enthusiasm.
You can find more reference and guidelines for contribution from https://www.kubernetes.dev/docs/guide/
and especially https://www.kubernetes.dev/docs/guide/#advanced-topics

/approve


Windows Toolchain Build

This adds a complete build of the Swift Windows toolchain that is currently distributed.  This is intended to help provide signal that a particular tag is able to build the complete toolchain.  This is a temporary snapshot point, with the assumption that this will grow further.  However, even in its current state it would help provide additional signal to help ensure that we keep the Windows toolchain in a stable state.

@swift-ci please smoke test

CC: @shahmishal

@swift-ci please test Windows platform

@swift-ci please smoke test

@swift-ci please smoke test Linux platform

@swift-ci please smoke test Linux platform

@compnerd This looks good! Let's merge it.


CFY-4393 Delay fetching events after an execution ended

this is obsoleted by #303 which resoled CFY-4393 in a different way


Made 4x faster than 'naive' version

This version uses an UnsafeMutableBufferPointer to get rid of all retains and releases, and a struct for nodes for giggles. This might be considered cheating, but it’s what I’d do if I were writing production code that needed to be super-fast.

@wilshipley Nice! Thank you for your contribution! I will merge this implementation on the next round (I am going to do it once I have some spare time in the next months)

I had re-run the whole benchmark and added your solution to the scoreboard! https://github.com/frol/completely-unscientific-benchmarks/commit/54637029f81a1286682125a14cf5463b2d2e1431
Thanks again and sorry for taking it for this long...


fix: added elasticsearch_index and elasticsearch_cluster_settings resource block to basic example

Description

added elasticsearch_index and elasticsearch_cluster_settings resource block to basic example
remove elasticsearch_index and elasticsearch_cluster_settings resource block from complete example

issue: https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/issues/237
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 1.16.1 :tada:
The release is available on:

GitHub release
v1.16.1

Your semantic-release bot :package::rocket:


Schedule design

We need to create a design we'll use for the final schedule inc workshops.
Design deadline: 2019-05-19 (a week before schedule deadline)
Implementation deadlines: 2019-05-25 (schedule going public)
Cc: @encukou

design is done


chore(alert): fix inline alert examples

Adds the inline modifier back to the inline alert examples.

PatternFly-Next preview: https://patternfly-next-pr-2354.surge.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Uncaught Error: unreachable

Not really sure what's going on, but in a fresh ember app, I'm getting the following error:

my project is simply:
$ ember -v
ember-cli: 3.3.0
node: 8.11.3
os: darwin x64

$ ember new scratch
...

$ cd scratch
$ ember install ember-collapsible-panel
...
[replace application.hbs with "basic usage" example from the docs site]

I can give you a repo if you like. I have a hunch it has to do with the module unification/pod component structure.

@noslouch replace all block uses of |p| with |panel| and you should be good to go.

Yeah, the gotcha here is caused by angle bracket invocation in Ember.
When you say this:
{{#cp-panel as |p|}}
  {{#p.toggle}}

Click me!

  {{/p.toggle}}

The
 tags are actually trying to invoke the p variable as if it was a component!
If you're using the default ember-cli-template-lint, it will warn you about this problem.


About displaying menu by clicking button

Hello . Is it possible to display the menu by clicking on the button?

please see #1

What about hideing the menu? Do I need to use releaseItemView() function?

Usually no need.


Fix Concourse and Google Cloud sdk links

Thanks, @RochesterinNYC!

I manually merged this into the develop branch. Thanks again.


Change api host, https://translate.google.cn/

Thanks for the great repo, it is very useful.
I have one suggestion:
In some countries, we cannot access https://translate.google.com
It will be helpful if we support use other api host

It is a create idea. I would love to implement this new feature in the next release.
Thank you.

any update?

Hi @yuanoook, version 2.0.0 can support customized host now. Please read more here: https://github.com/zlargon/google-tts#options-all-options-are-optional

Hello zlargon
Its pretty amazing what you have made. I have a small question as im not that smart in this coding stuff. But where can I change the language? Do I change it in the app im using, or can I change it somewhere in your google-tts files?
-Rasmus

Hi @MrRasmus, here is an example to change the language to Chinese.
https://github.com/zlargon/google-tts/blob/master/example/chinese.js
You can change the language by setting the option lang with language codes.
Hope it's helpful. 😁

Seems like this can be closed?


New Unkown Issue

Describe the bug
This Bug Happens When Specified Courses are Being Enrolled
Error Code
Traceback (most recent call last):
File "urllib3\connectionpool.py", line 699, in urlopen
File "urllib3\connectionpool.py", line 445, in _make_request
File "", line 3, in raise_from
File "urllib3\connectionpool.py", line 440, in _make_request
File "http\client.py", line 1344, in getresponse
File "http\client.py", line 307, in begin
File "http\client.py", line 268, in _read_status
File "socket.py", line 669, in readinto
File "ssl.py", line 1241, in recv_into
File "ssl.py", line 1099, in read
TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "requests\adapters.py", line 439, in send
File "urllib3\connectionpool.py", line 755, in urlopen
File "urllib3\util\retry.py", line 532, in increment
File "urllib3\packages\six.py", line 734, in reraise
File "urllib3\connectionpool.py", line 699, in urlopen
File "urllib3\connectionpool.py", line 445, in _make_request
File "", line 3, in raise_from
File "urllib3\connectionpool.py", line 440, in _make_request
File "http\client.py", line 1344, in getresponse
File "http\client.py", line 307, in begin
File "http\client.py", line 268, in _read_status
File "socket.py", line 669, in readinto
File "ssl.py", line 1241, in recv_into
File "ssl.py", line 1099, in read
urllib3.exceptions.ProtocolError: ('Connection aborted.', TimeoutError(10060, 'A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond', None, 10060, None))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "udemy.py", line 713, in main1
File "udemy.py", line 546, in auto
File "udemy.py", line 361, in get_course_id
File "requests\sessions.py", line 555, in get
File "requests\sessions.py", line 542, in request
File "requests\sessions.py", line 655, in send
File "requests\adapters.py", line 498, in send
requests.exceptions.ConnectionError: ('Connection aborted.', TimeoutError(10060, 'A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond', None, 10060, None))
** The Last Course Enrolled**
129 Buy “The Complete MERN Stack Development course 2021” https://www.udemy.com/course/the-complete-mern-stack-development-course/?couponCode=989444919C1790D763E9
System Info:

OS: win 10
d Chrome
Version 4.0

I am really sorry I won't be getting much time until my exams are over. I will try my best to solve it.
Try using https://github.com/techtanic/DUCE-CLI see if the error still exists.
I will start accepting pull requests

Hey ! No Problem ! I am just posting issues to let you know of these issues


Use playlist items for playlist_index if specified (fix #10591)

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fix #10591 by making use of playlistitems which has correct playlist indices

Add a test.

I'm not sure how to test passing flags like --playlist-items.
The way I tested manually is with something like
youtube-dl --playlist-items 4,3 --get-filename -o '%(playlist_index)s %(title)s' https://www.youtube.com/playlist?list=PL2qgrgXsNUG5ig9cat4ohreBjYLAPC0J5
which returns on fixed branch

4 Small Scale Baler and Braiding Rugs
3 Volunteer and Exhibitor Social

and returns on main branch

1 Small Scale Baler and Braiding Rugs
2 Volunteer and Exhibitor Social

You don't need to test passing flags you must test at API level. See test_YoutubeDL.py.

It looks like playlist_index from ydl.downloaded_info_dicts is correct but not playlist_index from process_ie_result in YoutubeDL.py. I am not sure what the discrepancy is.

The indexing is correct here https://github.com/ytdl-org/youtube-dl/blob/fffc618c519d10a7335eb5b06ab13d56ecea8561/youtube_dl/YoutubeDL.py#L929-L933

Something weird is happening with references or cache or something.
For example, if I reorder the tests so that result = get_ids({'playlist_items': '2,4'}) comes first, like this
        result = get_ids({'playlist_items': '2,4'})
        self.assertEqual(result, [2, 4])

        result = get_ids({})
        self.assertEqual(result, [1, 2, 3, 4])

        result = get_ids({'playlistend': 2})
        self.assertEqual(result, [1, 2])

        result = get_ids({'playliststart': 10})
        self.assertEqual(result, [])

        result = get_ids({'playliststart': 2})
        self.assertEqual(result, [2, 3, 4])

        result = get_ids({'playlist_items': '2-4'})
        self.assertEqual(result, [2, 3, 4])

        result = get_ids({'playlist_items': '10'})
        self.assertEqual(result, [])

        result = get_ids({'playlist_items': '3-10'})
        self.assertEqual(result, [3, 4])

        result = get_ids({'playlist_items': '2-4,3-4,3'})
        self.assertEqual(result, [2, 3, 4])

then printing playlist indices with print([v["playlist_index"] for v in ydl.downloaded_info_dicts])
produces different results
[1, 2]
[1, 1, 3, 2]
[1, 1]
[]
[1, 3, 2]
[1, 3, 2]
[]
[3, 2]
[1, 3, 2]

So the answer was deepcopy?
Anyway for future reference the commit is here https://github.com/ytdl-org/youtube-dl/commit/de1121d749089ea62d17aadb43493c00492bb37a


KOGITO-6890 Code Start test is picking up AssertJ version from the environment

use of old assertJ API. I think we are erroneously depending on the AssertJ version that is shipping with Quarkus somehow
2022-03-18T06:00:41.7516065Z [ERROR] Tests run: 2, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 29.376 s <<< FAILURE! - in org.kie.kogito.quarkus.dmn.KogitoDMNCodeCodestartIT
2022-03-18T06:00:41.7519047Z [ERROR] org.kie.kogito.quarkus.dmn.KogitoDMNCodeCodestartIT.testContent  Time elapsed: 0.104 s  <<< ERROR!
2022-03-18T06:00:41.7520044Z java.lang.NoSuchMethodError: 'org.assertj.core.api.AbstractAssert org.assertj.core.api.AbstractPathAssert.satisfies(java.util.function.Consumer[])'
2022-03-18T06:00:41.7521037Z 	at org.kie.kogito.quarkus.dmn.KogitoDMNCodeCodestartIT.testContent(KogitoDMNCodeCodestartIT.java:53)
Failing test is in /quarkus/extensions/kogito-quarkus-decisions-extension/kogito-quarkus-decisions-integration-test/

The simplest way to fix is to add an override at src/test/java/org/kie/kogito/quarkus/dmn/KogitoDMNCodeCodestartIT.java:
        .extension(ArtifactCoords.fromString("org.assertj:assertj-core:" + assertjVersion()))

where assertjVersion is 3.22.0
disable this override when the version will be globally updated. We cannot do that right now, because in Quarkus 2.7.3 a bump to assertJ 3.22.0 (current 3.21.0) will cause the quick start framework to error out at the same point. The issue is a subtle API change
java.util.function.Consumer vs java.util.function.Consumer[]

(tests) - kogito-runtimes job #1266 was: FAILURE
Possible explanation: Pipeline failure or project build failure
Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/1266/display/redirect or see console log:

Console Logs
[2022-03-21T08:22:57.372Z] du: cannot access ‘/home/jenkins/workspace/KIE/kogito/1.19.x/nightly/kogito-quarkus-2.7_ws-cleanup_1647850965541/kogito-apps/data-index/data-index-service/data-index-service-mongodb/target/jib/com/github/benmanes/caffeine/cache/FWAMS.class’: No such file or directory[2022-03-21T08:22:57.372Z] du: cannot access ‘/home/jenkins/workspace/KIE/kogito/1.19.x/nightly/kogito-quarkus-2.7_ws-cleanup_1647850965541/kogito-apps/data-index/data-index-service/data-index-service-mongodb/target/jib/com/github/benmanes/caffeine/cache/FWAMW.class’: No such file or directory[2022-03-21T08:22:57.372Z] du: cannot access ‘/home/jenkins/workspace/KIE/kogito/1.19.x/nightly/kogito-quarkus-2.7_ws-cleanup_1647850965541/kogito-apps/data-index/data-index-service/data-index-service-mongodb/target/jib/com/github/benmanes/caffeine/cache/FWAR.class’: No such file or directory[2022-03-21T08:22:57.372Z] du: cannot access ‘/home/jenkins/workspace/KIE/kogito/1.19.x/nightly/kogito-quarkus-2.7_ws-cleanup_1647850965541/kogito-apps/data-index/data-index-service/data-index-service-mongodb/target/jib/com/github/benmanes/caffeine/cache/FWARMS.class’: No such file or directory[2022-03-21T08:22:57.372Z] du: cannot access ‘/home/jenkins/workspace/KIE/kogito/1.19.x/nightly/kogito-quarkus-2.7_ws-cleanup_1647850965541/kogito-apps/data-index/data-index-service/data-index-service-mongodb/target/jib/com/github/benmanes/caffeine/cache/FWARMW.class’: No such file or directory[2022-03-21T08:22:58.297Z] 14G	/home/jenkins/workspace[2022-03-21T08:22:58.297Z] 28K	/home/jenkins/.pki[2022-03-21T08:22:58.297Z] 0	/home/jenkins/caches[2022-03-21T08:22:58.866Z] 1.1G	/home/jenkins/.gradle[2022-03-21T08:22:58.866Z] 8.0K	/home/jenkins/.java[2022-03-21T08:22:58.866Z] 2.8M	/home/jenkins/.npm[2022-03-21T08:22:58.866Z] 0	/home/jenkins/.yarn[2022-03-21T08:22:58.866Z] 81M	/home/jenkins/tools[2022-03-21T08:22:58.866Z] 19G	/home/jenkins[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // withEnv[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Install build-chain tool)Stage "Install build-chain tool" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Build projects)Stage "Build projects" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Sonar analysis)Stage "Sonar analysis" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (check space after build)Stage "check space after build" skipped due to earlier failure(s)[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh[2022-03-21T08:22:59.726Z] + wget --no-check-certificate -qO - https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/1266/api/json[Pipeline] readJSON[Pipeline] sh

yes, it is a problem with how integration is currently done. We can't align to Quarkus 999-SNAPSHOT where AssertJ has been already upgraded. See also https://github.com/kiegroup/kogito-runtimes/pull/2091
Let's see if we can get away with this temporary fix. When we update to Quarkus 2.8 we should be able to remove the override entirely


Array chain rec

Addition of two implementations of chainRec (depth-first and breadth-first) for ReadonlyArray and Array.
In ChainRec instance by default the depth-first method is selected, but there is access to breadth-first via BreadthFirstChainRec instances.
As for the optimization of each of the two algorithms, breadth-first avoids initial array copy by having two loops that do the same thing. In depth-first I couldn't avoid copy of the initial array.

Friendly bump: is this useful?

@qlonik I'm sorry, somehow I missed this PR, I apologize.
Following the current naming convention (for example SemigroupSum, SemigroupProduct), I would rename:

ChainRec to ChainRecDepthFirst
BreadthFirstChainRec to ChainRecBreadthFirst

(I'll retarget this PR to the 2.11 branch which is dedicated to the next release)

Done. I also rebased it for clean merge.
In the Array.ts module, I'm reusing implementations from ReadonlyArray.ts. It used to be like that for other non-pipeable functions, which is why I implemented like this. However, it seems that now they are implemented independently between two modules. So, should I copy implementation from readonly array to regular array?

Thanks @qlonik


Connection Bad Request Designer VS Code

Describe the Bug with repro steps
Existing connection

On existing logic apps, when going to designer if a connection is needed for the action following message appears: Invalid connection, please update your connection to load complete details.
When clicking on Change Connection an error appears with Code 404 Bad request.

Creating new connection.

Add trigger or action that uses a connection.
Stays on loading forever.
Clic on anything.
Clic on update connection.

The error is only on designer, when running the Logic App the connection works.

The connections are authenticated using the authentication key that gets created on a different process using a BICEP file.
What type of Logic App Is this happening in?
Standard (VSCode)
Are you using Preview Designer or GA Designer
Preview
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
All
Additional context
No response
AB#24354545

Hi @JorgeCadilloCalpine which vscode extension version are you using? Is this happening also when creating the workflow from zero? Or is this when you update a workflow with actions on it?

Hi @ccastrotrejo , first example is when updating an existing one. Second is when creating a new logic app and trying to add any action that uses a connection.
The version I'm using is: v2.24.7

Hi @ccastrotrejo, could this be related to permission issues?
We has a lot of things restricted for our accounts on Azure

Hi @JorgeCadilloCalpine, it might be. It is an odd behavior as I have been trying but I haven't been able to repro. The error isn't a 403 so the resource isn't refusing the access, I feel that it is more about not been able to find the resource. If it is completely restricted that might be the case.

I will close this issue, got access to a Sandbox environment where we have all permissions and it worked as expected.
The DevOps team is working on getting it fixed on the dev environment.
Thank you for the help @ccastrotrejo


⚠ Text Filter API Endpoint has degraded performance

In f0dd0d9, Text Filter API Endpoint (http://textfilter.roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 3063 ms

Resolved: Text Filter API Endpoint performance has improved in b95da5a.


Multiple calls to defaultProps merges them into a single object?

Hi,
Looking at the source for defaultProps, it looks to overwrite defaultProps for every invocation. For my app I've been separating concerns for a single HoC out into separate files. Each file, e.g. withFeatureX, has the appropriate withState, withPropsOnChange, withHandlers etc.  Was hoping to also put the specific defaultProps per file as well and have them all merge together.  It's easy enough to achieve outside of Recompose, just wondered if it's something you've considered?

I havent got the question. Recompose does not overwrite defaultProps of existing component. It creates a new one as a HOC and adds default props to it. Its expected.


”BABADAG” is listed in the drop-down menu

Description:
"BABADAG" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Ațel" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oraș BABADAG, TULCEA (BABADAG)".


Won't build on MacOSX

nake default                                                                                [12:39:05] Hint: used config file '/Users/.choosenim/toolchains/nim-#devel/config/nim.cfg' [Conf] Hint: used config file '/Users/.choosenim/toolchains/nim-#devel/config/config.nims' [Conf] ................................................................................... /Users/.nimble/pkgs/async_http_request-0.1.2/async_http_request.nim(125, 34) template/generic instantiation of requestfrom here /Users/.choosenim/toolchains/nim-#devel/lib/pure/httpclient.nim(1058, 14) Warning: Deprecated since v1.5; use HttpMethod enum instead; string parameter httpMethod is deprecated [User] .................. /Users/.nimble/pkgs/ttf-0.2.11/ttf.nim(934, 19) Error: type mismatch: got <int> but expected one of:  proc<(x, y: int): bool   first type mismatch at position: 2   missing parameter: y proc <`(x, y: int64): bool
first type mismatch at position: 2
missing parameter: y
19 other mismatching symbols have been suppressed; compile with --showAllMismatches:on to see them
expression: <numTables`

Sorry wrong repo


Problems with code suggestions in VSCode on Flutter 1.3.8

I recently updated to Flutter version 1.3.8, after updating the auto completion on VSCode stopped working. However, this was working before the update.

My flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v1.3.8, on Mac OS X 10.14.2 18C54, locale en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.4)
[✓] VS Code (version 1.32.3)
[!] Connected device
! No devices available

This is being investigated at https://github.com/Dart-Code/Dart-Code/issues/1565.


Table overflows don't work right on mobile

Description
For some reason, the tables, when loaded with content, overflows in a way that pushes the viewport out to the right when scrolled to the right.
Expected Behaviour
Viewport should scroll to the right when table has content.
Additional Information

Reopening because fix still doesn't work on Safari Mobile.


Consider moving away from legacy tokens

Slack is encouraging developers to build secure Slack apps, and to move away from using the legacy API tokens. As community admins, we have concerns with these requests:
"(user) is requesting a legacy API test token which provides access to your team’s data and their private data. A developer on your team might want this type of token for testing an app or integration, but you should be cautious about who you grant these to. Instead of using legacy tokens, we encourage creating internal integrations which have more controlled access to data.

For clarity - emphasis in comment was provided by the Slackbot notification, and was not added by myself.

Bump - legacy tokens have been deprecated, breaking any new set ups


UDP broadcast storm on network

Hi,
I've just noticed that when I open aurora it starts sending a lot of UDP packets all over the network.
I'm running Aurora 0.6.0b Beta on Windows 10.
I have attached a wireshark capture of the phenomenon.
Please let me know if I can provide some more information.
What are you experiencing an issue with?

[x] Aurora
[x] Aurora Updater

What kind of an issue is it?

[x] Other, Network issue

Expected behavior:
The application should not broadcast continuously on the network.
Actual behavior:
Explain here what happened instead.
How often does this problem occur?
This problem occurs...every time the software is running, even when it is docked.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Open up aurora.

What is the version of Aurora you are experiencing this issue with?
My version is... beta v0.6.0b
Does the previous version of Aurora have this issue?
The previous version, 0.5.1d, did not have this issue.
Please include any logs, screenshots, or files that are related to this issue:
This is a wireshark capture of the packet sent by aurora.
auroracapture.zip

Already fixed at develop branch, see #423 #395 #342

Thank you for the report, but like @brainbug89 already said, it's been fixed on dev. Should be fixed next version.


issue with ReadElementAttribute keyword

ReadElementAttribute keyword with "disabled" attribute is always returning "true" whether locator passed is disabled or enabled. Is there any issue with the keyword?

There's not much room for error, since this keyword is really simple. Here's the source code.
One thing to look out for is that "disabled" is a boolean attribute, meaning that any value you assign to it will be interpreted as true. Basically, if the "disabled" attribute exists at all (regardless of its value), then the element is disabled. My understanding is that all three of the elements below will be disabled:
<button type="button" disabled>Click Me!</button>
<button type="button" disabled="true">Click Me!</button>
<button type="button" disabled="false">Click Me!</button>

If you want an element to be enabled, you must remove the "disabled" attribute altogether.

Okay.. thank you!


Support WalletOpts in cast wallet derive-private-key command

Motivation
Currently derive-private-key implements its own wallet init code with hardcoded derivation path. Replacing it with existing WalletOpts allows to reduce code duplication and also:

specify custom derivation path
use encrypted keystore
use the same mnemonic flags as in other cast commands improving consistency

The command still supports explicitly specifying mnemonic and index as arguments.
Output format was changed to print private key only, allowing it to be used in shell scripts.
Breaking!! The command was renamed from derive-private-key to private-key and assigned pk alias. The old name is too verbose. The renaming is done in a separate commit, so it can be easily excluded from this PR if maintainers decide so.

2 additional questions came up during review:

Can I add some magic and make the second arg mnemonic_index_or_derivation_path_override then try to cast it to a number, if success then it's index, otherwise path? In theory a raw number can be used as derivation path but it's highly unlikely, and in this rare case --mnemonic-derivation-path will still work.
decrypt-keystore is now obsolete since keystore can be specified with options to private-key command

@mattsse do I need to make any additional changes before this can be merged?


GLTFDocument.append_from_scene executed on non-ui thread cost tool much time compared to on ui thread.

Tested versions
Reproducible in Godot v4.0.4.stable
System information
Mac & Android & iOS - Godot v4.0.4.stable
Issue description
try to execute GLTFDocument.append_from_file on ui thread and non-ui thread respectively, the time cost on non-ui thread is much higher than on ui thread. for example, in the minimal reproduction project,  append on ui thread time cost: 3436ms,
but append on other thread time cost: 117287ms
Hope to see your replay as soon as possible! Thanks so much!
Steps to reproduce
run the MRP Main.tscn to check result
Minimal reproduction project (MRP)
MinimalProject.zip

Can you try later versions of Godot Engine to see if the time changes?

Can you try later versions of Godot Engine to see if the time changes?

Thx for your reply, just tested on godot 4.3
raised this error:
E 0:00:03:0670   Main.gd:35 @ appendFromScene(): This function in this node (/root/Node3D/@Node3D@2) can only be accessed from either the main thread or a thread group. Use call_deferred() instead.
<C++ Error>    Condition "!is_readable_from_caller_thread()" is true. Returning: (false)
<C++ Source>   scene/3d/node_3d.cpp:821 @ is_visible()
 Main.gd:35 @ appendFromScene()
Main.gd:26 @ ()


chore: update readme contributors link

Proposed changes
Update the readme link so it properly points to the contributors page
Types of changes

[x] Other... Please describe: chore

Checklist

[x] I have read the CONTRIBUTING documentation
[x] Lint and unit tests pass locally with my changes

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   97.61%   97.61%
=======================================
  Files          19       19
  Lines         839      839
=======================================
  Hits          819      819
  Misses         20       20

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 f537d42...dbfbf66. Read the comment docs.


User/evannawfal/22 add lookup table mappings to ros config

Resolves #22

@evannawfal Can you print the output from your terminal to confirm that the configuration is read in properly? The variable names that have been received should be printed out when launching in debug mode.


pkg_resources.DistributionNotFound: The 'CellProfiler==3.1.9' distribution was not found and is required by the application

After full installation completes, the app cant launch and I have no idea why? Everything else seems to go right
Successfully installed botocore-1.20.88
./CellProfiler_Install.sh: line 179: cd: /home/mark/Cellprofiler/plugins: No such file or directory
Cellprofiler should now launch - may take a few moments for initialization...
Don't forget to submit bugs via the git page and make adjustments/pulls as you see fit, generally I'll push them
relatively swiftly... Cheers.
if cellprofiler starts - I HIGHLY recommend a restart before you dive in.
Traceback (most recent call last):
File "/usr/local/bin/cellprofiler", line 6, in
from pkg_resources import load_entry_point
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 3242, in
@_call_aside
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 3226, in _call_aside
f(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 3255, in _initialize_master_working_set
working_set = WorkingSet._build_master()
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 568, in _build_master
ws.require(requires)
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 886, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/local/lib/python3.6/dist-packages/pkg_resources/init.py", line 772, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'CellProfiler==3.1.9' distribution was not found and is required by the application

Hey! Weird.
I'm trying to replicate your issue presently. In the meantime can you provide the following outputs:
$ sudo pip freeze
$ cat /etc/*release
$ python3 -V
$ sudo sed -n '770,780p' /usr/local/lib/python3.6/dist-packages/pkg_resources/init.py
Thanks!

as a follow up inquiry - I note:  ./CellProfiler_Install.sh: line 179: cd: /home/mark/Cellprofiler/plugins: No such file or directory
did you run the installer with sudo priviliges? Might have failed somewhere in the pipeline without permission to properly clone the baseline setup or build a directory. You should be able to rerun: $ sudo ./CellProfiler_Install.sh to ensure things are where they should be if you didn't run with adequate permissions first.

I observed that the acquisition step doesn't correctly swap branches to v3.1.9 the way it should. I'll patch this shortly so future installs won't hang, In the meantime you should be able to correct this by running the following:
$ cd ~/CellProfiler
$ git checkout v3.1.9
$ sudo python3.6 -m pip install --editable .
$ cd ~/CellProfiler/plugins && git clone https://github.com/CellProfiler/Cellprofiler-Plugins
$ mv ~/CellProfiler/plugins/Cellprofiler-Plugins/* ~/CellProfiler/plugins/
$ cd ~/CellProfiler/plugins && cp classifypixelsunet.py ~/CellProfiler/cellprofiler/modules
$ cp /tmp/cellprofilerstuff/* ~/CellProfiler/cellprofiler/modules/
$ cd ~/Cellprofiler/plugins && sudo pip install requirements.txt
then launch with
$ sudo cellprofiler
I'll need a little longer to validate my changes (the VM I'm running on is potato speed compiling WXPython, so I may update again later with another fix but let me know if this doesn't resolve). Next release candidate should do all this automatically once I validate this but figure this way you can give it a shot while I work on the overall issue.

well the good news is that I've validated your bug. The bad news is I've opened a whole host of other issues that now need to be addressed to actually get this back into production status.
Keep an eye here for when there's next an update - will be working on this for a bit. Thanks for bringing it to my attention. There's a number of requirement modifications that will need to be made to the baseline script to be compatible with some of the changes that were implemented with the cellprofiler branch awhile ago, and more than a few variances.

Thanks you very much for the quick response and so extensive!

Sorry for the delay on this issue - I appreciate your patience! I'm rebuilding a new branch under develop which will be merged with main after I resolve the issues you are experiencing here. I have discovered a chain of issues that initiate from a number of inconsistencies. I've let this project lapse for awhile so it's a bit more out of date than I thought. I'm hoping to have a resolution for you shortly, as this impacts all new users at present. Hopefully in the next few days. Thanks!

Good morning - still broken on my end. I'm going to refer you to the official repository's conda installation path solution which may resolve your issues faster than I'll be able to patch this build: https://github.com/CellProfiler/CellProfiler/wiki/Conda-Installation
I will still update here when available but if you have a project and need something running now, this should resolve ^
thanks!

This conda installation did eventually work! thanks for the help

Thanks for the update! Glad to hear it.


Simd 118: replace calculation-blocks config value with constant

Problem
See https://github.com/anza-xyz/agave/pull/1159#discussion_r1591793159
SIMD-0118 specifies that rewards calculation be done in one block (the first block of the epoch). But there is configuration plumbing in the code for >1 calculation blocks. Since any changes to calculation duration would need to be proposed in a new SIMD first, this is a potential footgun.
Summary of Changes

Replace PartitionedEpochRewardsConfig::reward_calculation_num_blocks with a constant and use in partitioned_epoch_rewards module
Add logic to preserve force_partition_rewards_in_first_block_of_epoch, which was manhandling the reward_calculation_num_blocks field
Also fix an incorrect use of the Slot type alias nearby

Sorry for the extra pushes. I originally put the constant by the config struct/field being replaced (in accounts-db), but realized there is no current reason for that.
Anyway, I'm done churning now!

Codecov Report
Attention: Patch coverage is 88.88889% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 82.1%. Comparing base (7181011) to head (328f74b).
Report is 11 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1251   +/-   ##
=======================================
  Coverage    82.1%    82.1%
=======================================
  Files         886      893    +7
  Lines      236577   236589   +12
=======================================
+ Hits       194390   194406   +16
+ Misses      42187    42183    -4

Only changes in two new commits; rebased to ensure no conflicts.


Forecast / storm sensitivity

Right now one can set the sensitivity to low/medium/high, but one does not have any idea what gradient is actually used in the back end. I would suggest to allow some numerical input here, like it's done at my usual weather station. Something like nn hPa/3h or mm hPa/4h. I do understand that this might require a bit more knowledge at the users end.

I'll consider making it numerical, but for now, I can display the x hPa/Xh in the summary of the preference (below the title)

Inspirational(?) notes on numbers from the manual of my other hardware (just as I finally happen to have it at hand, and it works quite well for some years now):

The change of weather forecast icon is in accord to the relationship between current relative pressure and the pressure change since the last 12 hours. If the weather is changing, [some indicator]. And after the next 3 hours  if weather conditions become stable [other indicator].

(They use 4 symbols sunny, partly cloudy, cloudy, rainy and animated arrows or just arrows between them to indicate the change from to. This is very similar to Trail Sense :)

The pressure threshold can be set to suit the user's requirement for weather forecasting from 2-4 hPa (default 3 hPa). Areas that experience frequent changes in air pressure require a higher setting compared to an area where the air pressure is stagnant. [...] e.g. if 4 hPa is selected there must be a fall or rise in air pressure of at lest 4 hPa before the weather station will register this as a change in weather.

=> I would guess this is low/medium/high in the forecast sensitivity.

The storm threshold can be set to suit the users requirement for storm forecasting from 5-9 hPa (default 6 hPa). When there isa fall over pressure threshold _within 3 hours, the storm forecasting will be activated [...] for 3 hours.

(I found there seem to be two schools of thought, one relying on 3h intervals others on 4h.)


Implementing a camera confinement option for the PhantomCamera2D

Project Type
2D
Feature Description
Limit the camera’s position to a volume or area defined by a CollisionPolygon2D
Should work like the Camera Confiner of Cinemachine in Unity : https://www.youtube.com/watch?v=CvJaHAvp6dA
Use Cases
As part of the design of a level, I'd like to confine the camera to several complex polygonal zones to easily limit its movement in various level design situations.
(Optional) Proposed Solution
I would like to add a field bellow the Limit target property of the Limit seciont of the PhantomCamera2D called "Limit confiner area" Which accept only a CollisionPolygon2D that will define that the camera have to stay in this area

As the plugin is written in GDSCRIPT i am abble to implement this myself into the code

How would this be different from Limit Target, which can be pointed to a collision shape or tilemap and stay bounded to the object according to the Phantom Camera website ?

Actually the limit target is only limiting on a rectangle area
For more advanced level design i would need that the camera saty confined inside a polygonal Area like this one :

This has been brought up before, and the advice I generally give at the minute is to have multiple CollisionShape2Ds within different Area2D nodes, and then switch the Limit Target between them during runtime.
Adding a free form collider limit like this is exponentially more complicated than the current system, both in terms of implementation, but also getting the expected behavior right.
The addon's current Limit system leverages Godot's built-in one, so adding this would be its own thing with its own logic and behavior.
Not against adding support for this but, again, it's a fair amount of work.
If you would like to take a stab at it, feel free!

I would like to add a field bellow the Limit target property of the Limit seciont of the PhantomCamera2D called "Limit confiner area"

For this, I would rather have it be part of the existing Limit Target property. It just needs to identify what type of shape it is. Adding a second option will start getting confusing, as the user now has to know which one to use, which is a question the user shouldn't have to answer.

Not against adding support for this but, again, it's a fair amount of work.
If you would like to take a stab at it, feel free!

Currently i've worked on my side to implement this system and it's not finished yet
So my will with this issue is to implement myself this to your PhantoMCamera project beceaus i think it is the best place to put in and make everyone benifit it ! (And also i really appreciated your work on this really thank you for that <3)

For this, I would rather have it be part of the existing Limit Target property. It just needs to identify what type of shape it is. Adding a second option will start getting confusing, as the user now has to know which one to use, which is a question the user shouldn't have to answer.

Ok so i will consider using this field instead of what i thought i agree with you
Can you give me your green light to start working on that ?

You're welcome to give it a go!
However, I think it's important to mention a couple of things when it comes to working on PRs, big or small.
There's never a guarantee that it will be merged into the repository. Ultimately, it comes down to the maintainer (me) to promote the feature, maintain the addition, fix any future bugs it may have etc. Once a contributor has made a PR and submitted it, there's no assurance that they will help resolve any future changes their work requires. So while the idea here is good, if it's implemented in a very complex, unmaintainable or non-flexible way, it comes down to me maintaining it, which can be no small job.
The reason I'm mentioning this is that I don't like rejecting other people's work. Yet, when the choice comes down to choosing between maintaining an addition to a project that I don't align with and in turn struggle with working on, or rejecting someone's hard work, it will always the latter. As much as I dislike throwing people's time and effort away. One or two of the closed PRs are examples of that, and it's one of the most common reasons why PRs don't get merged.
So if you want to give it a go still, I would highly recommend not working on it in secret for 2 months and then comes back with a finished solution. But instead make a Draft PR with a written down plan on how you're thinking of implementing it, potentially with a basic but functional version, before sharing anything.
This is just to make sure that there's no fundamental disagreements with the given solution from a design or tech perspective. The more communicative the work is early on, the more likely it is to succeed or at least not be a waste of time for the person making it.

Thank you for your time putting clarification here !
Ok so i'll proceed the way you recommended by starting with a draft PR to put documentation and plan on how i see implementing this, i'll see doing this in the next 2 weeks
Thank you for your answer !


Rename this project to emerald-testrpc

Sputnik-vm-dev is a confusing name. Its a test Rpc server, lets just name it that.
@shanejonas

That may be a good idea


Using browser.screenshot(name = '/var/aa', suffer = '.jpg') filename turn out to 'aaxiHXq8.jpg'

Using browser.screenshot(name = '/var/aa', suffer = '.jpg') filename turn out to 'aaxiHXq8.jpg' I don't know why? need some help

IIUC the string xiHXq8 is a randomly generated string which gets appended to the filename aa you provide in browser.screenshot(name, suffer). I assume this is done to avoid to replace an already existing file with the same name. Just a guess.

@andrewsmedina browser.screenshot uses tempfile.mkstemp to get a filename with a random portion to the filename. Is this the desired behaviour? In the event of the provided filename already existing, should we write a new file (current behaviour, due to random string) or throw an exception? Either way, overwriting the existing screenshot is definitely something to avoid.

This is very confused. Can you add a switch to use my real filename or add random postfix?

Fixed by #949


Error icon incorrect for notification

The icon used by the notification is visually jarring. It doesn't look like a symbol you'd associate with an error (looking akin to a rotated no entry sign). It also doesn't match the error icon in the iconography library
Error icon:
http://carbondesignsystem.com/style/iconography/library
Notication page:
http://carbondesignsystem.com/components/notification/code

@CptPilot did you have any other suggestions for what this icon could look like instead?

Closing due to inactivity. feel free to re-open if you have icon designs/suggestions.


Setup CI/CD

This can be closed. We have travis ci/cd setup. But it has no tests. That will be handled in #1


Warning When Using Different HuggingFace Datasets

Hello,
Any idea if this warning will impact the training of the model when using alternative datasets? Or can it be ignored? I understand that PaLM needs concatenated input sequences of length 2048.
Warning thrown:
Token indices sequence length is longer than the specified maximum sequence length for this model (1366 > 1024). Running this sequence through the model will result in indexing errors.
Contained Example:
dataset = load_dataset("the_pile", 'enron_emails')

print(dataset)

tokenizer = GPT2TokenizerFast.from_pretrained("gpt2")

seq_len = 2048

def tokenize(examples):
    seq_length = seq_len
    examples = tokenizer(examples["text"])
    concatenated_examples = {k: list(chain(*examples[k])) for k in examples.keys()}
    total_length = len(concatenated_examples[list(examples.keys())[0]])
    if total_length >= seq_length:
        total_length = (total_length // seq_length) * seq_length

    result = {
        k: [t[i : i + seq_len] for i in range(0, total_length, seq_length)]
        for k, t in concatenated_examples.items()
    }

    result["labels"] = copy.deepcopy(result["input_ids"])

    return result

tokenized_dataset = dataset.map(
    tokenize, batched=True, num_proc=16, keep_in_memory=True, remove_columns= ['text', 'meta']
)

Thank you,
Enrico

In my humble option, the reason is that the max seq len of the embedding of your pretrained model is limited to 1024. Maybe you need to modified the embedding logic. It is common in Bert pertaining using longer sequences. Because max len of bert embedding is only 512. The GPT2 may fall into the same cases.

Hi @feifeibear ,
Thank you for the response.
Is the GPT2Tokenizer in the build_data_from_wikitext() function pre-trained on the wiki dataset with a specific sequence length of 2048 tokens first? I noticed you set the sequence length to 512 when constructing the dataset: seq_len: int = 512. The warning I provided is being thrown before I input any tokenized data into your PaLM model.
# Vocab in the paper is sentencepiece. GPT2Tokenizer uses tokenizers like sentencepiece.
tokenizer = GPT2Tokenizer(vocab_file=tokenizer_path + "/vocab.json", merges_file=tokenizer_path + "/merges.txt")

Thank you again,
Enrico


Use Jupyter Notebook with SAP HANA, express edition

Repo suggested for python packages 'https://download.opensuse.org/repositories/devel:/languages:/python/SLE_12_SP2/devel:languages:python.repo is no longer available. I have tried using https://download.opensuse.org/repositories/devel:/languages:/python/SLE_12_SP3/devel:languages:python.repo repo, but it looks like it is missing python-devel package, so I cannot proceed forward to install Jupyter on my HANA XS. Would appreciate update on the tutorial docs or at least hints to solve this problem.

Hi @sarahESL,
My initial goal was to make this tutorial as easy as possible and avoid downloading the SDK and the associated patches or registering your system.
But apparently the https://download.opensuse.org site is not keeping historical versions.
Therefore your only option, if you haven't found one yet, is probably to:

download the SUSE SDK DVD locally from https://download.suse.com/index.jsp
then add it as a local repo with a command like
zypper addrepo iso:/?iso=/path/to/iso1.iso disk1
install python-devel
apply the relevant patches python-devel from https://download.suse.com/patch/finder/

I'll try to update the SUSE part in the coming weeks.
As another alternative, you can also install it on your local PC and remotely connect to your HXE instance as well.

Thanks for your response. Although the suggested command for adding the iso as repo does not work (error: URI or .repo required). I used "zypper ar -c -t yast "iso:/?iso=/path/to/image/SLE-12-SP2-Server-DVD-x86_64-GM-DVD1.iso" "repo_name" ``` instead and it worked. However, it still says "Package found python-devel not found"!

I solved the issue: the image I was using was not the SDK! SDKs can be downloaded from https://download.suse.com/Download?buildid=g3e7P21X6Lw~

fixed in commit : https://github.com/SAPDocuments/Tutorials/commit/8d797a1ad7cb09ccb3eced6d2d5c91f2fe04c077


Example in ResponseBody of /v1/list-applications

The indentation should be reduced by one level so that the example refers to the array level rather than the object level.
(The issue has low priority if the implementers do not complain.)

Changed as proposed.


[Suggestion] Scale PEX, performance boost for large server

I've been using PEX for 3 years on many large servers, and I've always been preferring flatfile over MySQL. It is so simple and easy to use. However, there's a performance issue with PEX when both of these things happen:

When there are lots of players currently playing. (Around 150 to 300)
When there are lots of player entries inside permissions.yml file (Usually 15,000 or more)

The server starts to slow down at that time, and when there are PEX command being excecuted, either automatic (vote rewards) or by admin, the server pauses for about 0.5 to 1.5 second. It is mostly due to lots of entries inside the permissions.yml file.
My suggestion is to have an option for seperated UUID.yml file for each player in the uuid folder. This idea is not new, it's being used by Essentials and it can handle millions of players just fine. It might not matter for small server, but for big server, it is a great thing and will surely boost server performance.
Thank you for reading this.

Hopefully some of the performance issues should be resolved with recent commits. Look at #1824 to track the status of this


[Get WB Score on each domain] How to get WB Score on  Info Seek/Creative/Code & Debug etc

When running
bash evaluation/run_all_eval_batch.sh model_pretty_name
python src/openai_batch_eval/check_batch_status_with_model_name.py model_pretty_name
bash leaderboard/show_eval.sh

The WB_Elo score output, such as,

BUT the WB Score for Info Seek/Creative/Code & Debug did not output. We would like to evaluate the model privately. Is it free or does it require payment? If payment is required, how do we pay?

When running scripts locally like:
bash evaluation/run_all_eval_batch.sh model_pretty_name
python src/openai_batch_eval/check_batch_status_with_model_name.py model_pretty_name
bash leaderboard/show_eval.sh

The WB_Elo score output, such as,
BUT the WB Score for Info Seek/Creative/Code & Debug do not output as the LeaderBoard（https://huggingface.co/spaces/allenai/WildBench）. Is it free or does it require payment? If payment is required, how do we pay?

solved，thx


Replace falkon.mmv_ops with homemade mmv_ops

Hi again!
Thanks again for the help last time.
This time, I'd like to replace the falkon.mmv_ops in the InCoreFalkon solver with a homemade mmv_ops for a research project.
Wondering what is the "cleanest" and simplest way to do this?
Thank you!
Best regards,
Robert

Hi!
There are a few moving parts for the mmv operations, and it's a bit more complex than it should be.
Essentially, if you look at the fmmv_cuda.py file, and in particular the generic_fmmv function you have a generic way of running kernel-vector multiplications. That function is responsible for splitting the data matrices and computing the matrix-vector product, but not for computing the kernel itself.
The kernel is a class which defines methods _prepare, _compute and _finalize which get called to actually compute the kernel matrix (examples can be found in the falkon.kernels submodule!)
If I understand what you want to do, you should define a new generic_fmmv function, and possibly also the function which splits among multiple GPUs and then calls generic_fmmv (which is in the same file and is called fmmv_cuda). There are other versions of these functions: the distk variants do some tiny optimizations for the Gaussian kernel, the fdmmv variants run a "double" kernel-vector product. They could be substituted by two fmmv calls at the cost of running more computations.
Get in touch if you want to expand on what your idea is, or if you have other doubts on the implementation!
Best,
Giacomo

Dear Giacomo,
Thank you for your reply. This is very helpful. Upon fiddling around yesterday, I might be close to achieving what I want. I'l let you know if it doesn't work out.
Thank you!
Best regards,
Robert


Dont make idemptotent queue set calls

Coverage increased (+2.8%) to 80.283% when pulling 5b5d6fd590470f19bdf3a0d63c6edefe57ebbcbe on dont-make-idemptotent-queue-set-calls into a845c8c41eda24a52ae34943d6c08589f368e1f5 on main.


Backmerge release 3.33.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.008%) to 31.721%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
12935

Relevant Lines:
14638

💛  - Coveralls


SAK-48661 Trinity: Restore double column layout for Overview, etc.

A screenshot of the expected two column layout which results from this proposed fix is attached to SAK-48661.
I removed the CSS styling for "#{$namespace}multipleTools .#{$namespace}sakai-iframe" to prevent one column from being higher than another when an iframe tool is presented as one of the columns (like we use at our institution's custom Gateway page).

Hello @hornersa, Could you kindly share the link to the specific JIRA you're referring to? Thank you!

@kunaljaykam - Here's the link: https://sakaiproject.atlassian.net/browse/SAK-48661

Thanks for pointing out my mistake, @hornersa. I appreciate your help in identifying the issue with the div structure. As you suggested, it seems like we'll need to create a separate div. However, I was wondering if we could streamline the code by using a ternary operator. How about something like this?
<div class="${pageTwoColumn ? 'd-flex' : ''}">
What do you think?


manage users link shown only to certain privledged users

resolves #121

We have this same problem with several of the admin menu links; The main problem is that in order to correctly determine whether or not a user should have access to the link, we need to know the press.
When the user is on the main catalog page, or the root page, there is no press in scope, but when the user is on a press_catalog page, for example, we do know the press, and we should be able to pass it to the Ability class so that cancan can properly decide the abilities.
In the alexandria code base, we had a similar requirement:   in order to determine certain abilities, we needed to know if a user's IP address corresponded to an 'on-campus' location.
I wonder if we could do something like this for heliotrope?  i.e. Have the base application_controller set the press (if there is a press in scope), and then pass it to the ability class.
Here's the example code from alexandria:
https://github.com/curationexperts/alexandria-v2/blob/master/app/models/ability.rb#L9-L12
and here:
https://github.com/curationexperts/alexandria-v2/blob/master/app/controllers/application_controller.rb#L26-L28


Observing error on API:/v1/regard-controller-object-deletion when device connection changing to connecting

When device connection status is changing from connected to connecting, observing below error
[05:29:47.321] DEBUG (1): sending subscriber notification to: HTTP://x.x.x.x:xxxx/v1/regard-controller-attribute-value-change with content: {"notification-proxy-1-0:attribute-value-changed-notification":{"counter":1,"timestamp":"2023-12-14T05:29:47.312732642Z","resource":"/core-model-1-4:network-control-domain=live/control-construct=OpenDayLight1/logical-termination-point=513250004/layer-protocol=0/mount-point-1-0:mount-point-pac/mount-point-configuration","attribute-name":"connection-status","new-value":"connecting"}}
[05:29:47.361] DEBUG (1): sending subscriber notification to: HTTP://x.x.x.x:xxxx/v1/regard-controller-object-deletion with content: {"notification-proxy-1-0:object-deletion-notification":{"counter":1,"timestamp":"2023-12-14T05:29:47.312732642Z","resource":"/core-model-1-4:network-control-domain=live/control-construct=OpenDayLight1/logical-termination-point=513250004/layer-protocol=0/mount-point-1-0:mount-point-pac/mount-point-configuration/session-id"}}
[05:29:47.397] ERROR (1): error during subscriber-notification for /v1/notify-controller-object-deletions
    err: {
      "message": "Request failed with status code 404",
      "name": "AxiosError",
      "stack":
          AxiosError: Request failed with status code 404
              at settle (/home/openbackhaul/NotificationProxy/node_modules/axios/dist/node/axios.cjs:1913:12)
              at IncomingMessage.handleStreamEnd (/home/openbackhaul/NotificationProxy/node_modules/axios/dist/node/axios.cjs:2995:11)
              at IncomingMessage.emit (node:events:525:35)
              at endReadableNT (node:internal/streams/readable:1358:12)
              at processTicksAndRejections (node:internal/process/task_queues:83:21)
      "config": {
        "transitional": {
          "silentJSONParsing": true,
          "forcedJSONParsing": true,
          "clarifyTimeoutError": false
        },
        "adapter": [
          "xhr",
          "http"
        ],
        "transformRequest": [
          null
        ],
        "transformResponse": [
          null
        ],
        "timeout": 0,
        "xsrfCookieName": "XSRF-TOKEN",
        "xsrfHeaderName": "X-XSRF-TOKEN",
        "maxContentLength": -1,
        "maxBodyLength": -1,
        "env": {
          "Blob": null
        },
        "headers": {
...
...
        },
        "method": "post",
        "url": "HTTP://x.x.x.x:xxxx/v1/regard-controller-object-deletion",
        "data": "{\"notification-proxy-1-0:object-deletion-notification\":{\"counter\":1,\"timestamp\":\"2023-12-14T05:29:47.312732642Z\",\"resource\":\"/core-model-1-4:network-control-domain=live/control-construct=OpenDayLight1/logical-termination-point=513250004/layer-protocol=0/mount-point-1-0:mount-point-pac/mount-point-configuration/session-id\"}}"
      },
      "code": "ERR_BAD_REQUEST",
      "status": 404
    }

Hello Zia,
the 404-error can mean two things: The target url is not available or the resource path was not found in the MWDI.
Regarding the target url: In the current MWDI there does not exist an endpoint for "regard-controller-object-deletion" yet or do you work with another version of MWDI where it was already added?

Fixed as part of v1.0.0_test-4, hence closing


Install capi operator

this is built on top of the other PRs
mostly working..
oc get all
NAME                                                    READY   STATUS             RESTARTS   AGE
pod/capi-operator-controller-manager-65d9865c69-8rbjq   1/2     ImagePullBackOff   0          9m9s
pod/cluster-capi-operator-669dc6f697-c4pk7              1/1     Running            0          11m

NAME                                                       TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
service/capi-operator-controller-manager-metrics-service   ClusterIP   <IP_ADDRESS>   <none>        8443/TCP   76m

NAME                                               READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/capi-operator-controller-manager   0/1     1            0           9m9s
deployment.apps/cluster-capi-operator              1/1     1            1           11m

NAME                                                          DESIRED   CURRENT   READY   AGE
replicaset.apps/capi-operator-controller-manager-65d9865c69   1         1         0       9m9s
replicaset.apps/cluster-capi-operator-669dc6f697              1         1         1       11m

/retest

Let's get this moving, I think this is good enough for now, we all seem to be on the same page re the RBAC so we can follow up on that later. Thanks for all your work on this @asalkeld and @alexander-demichev
/lgtm