ERROR: (0 , n8n_workflow_1.deepCopy) is not a function

Hello. I am trying to use your plugin and am having issues with receiving an error. This happens with any image I load. The original Edit Image plugin does not throw this error. I do have GraphicsMagick installed. Anything I can try to fix this?

Bug confirmed. I will fix soon as possible.

Amazing work @unilogica  - wanted to ask - was the bug fixed and/or is help needed? Seems like regression issue due to changes in n8n-core and not graphicsmagik?


i need help

Describe your bug here.
my game crashed when i lost due to a bullet note
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

Show the damn script

PsychEngine_2022-06-29_13'41'22.txt
PsychEngine_2022-06-29_13'30'04.txt
PsychEngine_2022-06-29_13'37'45.txt

those are logs, not scripts
but from the looks of it, your death character is missing animations


chore: sets correct link for the travis build status

Codecov Report

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

@@          Coverage Diff           @@
##           master     #14   +/-   ##
======================================
  Coverage    90.7%   90.7%
======================================
  Files          40      40
  Lines         882     882
======================================
  Hits          800     800
  Misses         82      82

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 4c13988...134f9d8. Read the comment docs.


Has anyone been able to install on a version of PHP 7.2+?

I try to install on a version of PHP 7.2, but as I understand it is not compatible with this version of PHP

I try to install on a version of PHP 7.2, but as I understand it is not compatible with this version of PHP

switch php version


Automated cherry pick of #6509

Cherry pick of #6509 on release-5.28.
/cc  @sudheerDev

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: ecc095e9d323cdd0702cf68d993dc125203dfc44


chore: introduces .editorconfig

Short description of what this resolves:
Ensures consistent formatting of different resources
Changes proposed in this pull request:

.editorconfig file with go, yaml and Make conventions
updates in README

I was looking for automated formatting fixes as part of the build, but so far none of the tools [1] [2] give satisfying results. I might give 2 a spin in the future. Ideally integrated as Git app.


Suggested Features

I'm leaving these here as examples of things I wanted while building ParetoSmooth.jl:

Some kind of AbstractModel type that I can dispatch on.
A log_pdf function.
A way to instantiate a model on some counterfactual dataset -- either by calling some AbstractModel object on a dataset or by having condition/decondition functions.

There exists AbstractPPL.AbstractProbabilistivProgram <: AbstractMCMC.AbstractModel. DynamicPPL.Model (ie Turing models) are a subtype of it.
There exists AbstractPPL.logdensity and loglikelihood, logprior, and logprior for Turing models.
There exists AbstractPPL.condition and AbstractPPL.decondition, and it's also implemented for Turing models (with some restriction for backward compatibility, as explained in the docstring).

Yeah, this is basically all described in the interface draft.  (The abstract supertype is not, I think, but already exists.)
Note though that we are still in the process of refining this.  condition/decondition work in DynamicPPL, but only with named tuples.  As described in the draft, there needs to come some more flexible syntax.  And for counterfactuals -- I'm not sure what exactly you mean, but I mention the possibility of having a do-operator in the prob-syntax at some place below.
Feel free to reach out to me with question, I'm always keen on refining my understanding of what downstream packages need from the interface (are you in the Turing Slack?)


Remove redundant autobuild script.

Autobuild script already in root folder.
Signed-off-by: Li Xiaoming<EMAIL_ADDRESS>

Sorry for latency. Good catch !


#924 replace moment with new date formatting module

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

issue #924

Error message should be updated to MM/DD/YYYY because single digit date and month aren't valid anymore.

If I enter the date as 1/5/2021 in the table and save. When I open the table again, date is changed to 2021-00-05 -

This is a minor change but dates in tooltips no longer match with the dates in fields -

Timestamp field in table - Day missing.

Error message should be updated to MM/DD/YYYY because single digit date and month aren't valid anymore.
single digit dates are valid.

Same for Error message in Timefield control. H:m:s no longer valid -
Same as above.  1 isn't valid since you still need m and s.

If I enter the date as 1/5/2021 in the table and save. When I open the table again, date is changed to 2021-00-05 -
Good catch.  Fixed.

If I enter time as 1:5:10 and save. Time is changed to 01:05:10:-08:00
Good catch.  Fixed

This is a minor change but dates in tooltips no longer match with the dates in fields -
This is the same as what's in the code today.  We don't format the date/time in tooltips.  We can create a new issue to track that.

Timestamp field in table - Day missing.
This is expected.  It was changed in table cells to reduce the string length.


Speed of Bot and esp32

Is there a way to increase at which the bot responds to the esp32? With the app and phone it's pretty fast but with home assistant, there is a couple of seconds delay. Is there somewhere in the code I could modify the time it takes? Thank you!

it all depends on the bluetooth signal between the bot and the esp32. the esp32 should be close to the bot if you want it to work instantly
what is the rssi value you get in the attributes?
my main bot is very close to the esp32 and it works basically instantly

Yes, my bot and the esp 32 are right next to my room in the bathroom. It is not as fast as the app. How do I find the RSSI value?

it is published in mqtt attributes for the device.
/bot//attributes
main github page has that info
it may depend on the esp32. I can say that mine work almost instantly. there are some very minor delays in the code for stability, but if you are experiencing seconds delays it means it is just taking a long time to connect
where are you standing when using the app? I just tried with the app (using an S9 samsung) while standing in my dining room, bot in the kitchen, it did 3/4 of a circle....so its not instant with the app for me

I am in home assistant. Do I run something here to listen to the device?

are u using MQTT discovery? if so open the device. it will be listed as LinkQuality

Just this. I don't quite understand most of this stuff honestly. I just followed your guide to upload the software from Arduino IDE and ran the program.

in HA main dashboard, go to Configurations, then devices. The switchbot device will be listed if you are using mqtt discovery
open that and you should see LinkQuality with an rssi number.
I cant send screenshots atm

The device is not here. I only have an entity for the device. Am I supposed to do something in the config.yaml folders?

you should have to do zero config changes in HA. no yaml files to create/edit when using mqtt discovery. the devices automatically show up
rebooting the esp32 resends the data for HA to automatically find it

Did you manually create that device? that must be one of the other switchbot integrations? I am assume you tried to get those working.  Are you sure you are even calling the esp32?
These are the HA steps to follow (if you setup the ESP32 correctly)

So I just reset my whole home assistant. This is my flashed setup in IDE. Is this setup correctly for the esp32?
/****************** CONFIGURATIONS TO CHANGE *******************/
/********** REQUIRED SETTINGS TO CHANGE **********/
/* Wifi Settings /
static const char host = "esp32switchbothub";                          //  Unique name for ESP32. The name detected by your router and MQTT. If you are using more then 1 ESPs to control different switchbots be sure to use unique hostnames. Host is the MQTT Client name and is used in MQTT topics
static const char* ssid = "Anderson 2.4G";                           //  WIFI SSID
static const char* password = "mypassword";                   //  WIFI Password
/* MQTT Settings /
/ MQTT Client name is set to WIFI host from Wifi Settings*/
static const char* mqtt_host = "<IP_ADDRESS>";                       //  MQTT Broker server ip
static const char* mqtt_user = "mqtt";                         //  MQTT Broker username. If empty or NULL, no authentication will be used
static const char* mqtt_pass = "mqtt";                         //  MQTT Broker password
static const int mqtt_port = 1883;                                  //  MQTT Port
static std::string mqtt_main_topic = "switchbot";                   //  MQTT main topic
/* Switchbot Bot Settings /
static std::map<std::string, std::string> allBots = {
/{ "switchbotone", "d3:15:04:54:18:c0" },
{ "switchbottwo", "yY:yY:yY:yY:yY:yY" }*/
};
I am not seeing any autodiscovery happening in mosquitto broker.

you have a typo here.....
/* Switchbot Bot Settings /
static std::map<std::string, std::string> allBots = {
/{ "switchbotone", "d3:15:04:54:18:c0" },
{ "switchbottwo", "yY:yY:yY:yY:yY:yY" }*/
};

should be this
/* Switchbot Bot Settings /
static std::map<std::string, std::string> allBots = {
{ "switchbotone", "d3:15:04:54:18:c0" }
};

THANK THE LORD! AND THANK YOU!


java/util/zip/ZipFile.getManifestNum(J)I

Some jvm use this and it's not supported so there are issues loading the classes

Closing c9e0c8f18527be8d989c83f2b8b21324421fe641


Perhaps TF graph connectivity issues in keras-based Model containing Masking layers

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
gist
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu-20.04 on a Windows Subsystem for Linux (WSL)
TensorFlow version (use command below):
2.3.1 v2.3.0-54-gfcc4b966f1
Python version:
3.8.2 64-bit using a venv
Ray[rllib] version (RL framework):
1.0.1.post1
Text editor (development environment):
Visual Studio Code

Describe the current behavior
Get InvalidArgumentError: You must feed a value for placeholder tensor if line 73 in articleSchedulingModel.py is active, instead comment in line 74 and comment out line 73 then there is no error.
It seems to me that perhaps there might be an issue with connectivity of the TF graph similar to a former related issue mentioned in this annotation:

IMPORTANT NOTE: This model will not work with PPO due to a bug in keras
that surfaces when having more than one input placeholder (here: inputs
and is_training) AND using the make_tf_callable helper (e.g. used by
PPO), in which auto-placeholders are generated, then passed through the
tf.keras. models.Model. In this last step, the connection between 1) the
provided value in the auto-placeholder and 2) the keras is_training
Input is broken and keras complains.

My assumption is that some connection gets lost in the tf.keras.Model caused by the masking layer defined in line 64 and further connected to the concatenate layer defined in line 73 (in articleSchedulingModel.py).
Is the assumption appropriate or what else might cause this error?
Standalone code to reproduce the issue
Link to gist
Other info / logs
Traceback also in gist

Update: For now, I am quite sure that the masking is causing the issue, but I do not understand why.
The masking layer in line 58 works fine, but including further masking layers (e.g. in line 64) leads to the error above.

@klausk55
Please provide with "simple stand alone code" to replicate the issue or if possible share a colab gist with the error reported.

@Saduf2019
Additional to the shared gist from above, I share a colab gist

A question regarding masking which might cause my reported issue:
First, can a TF keras model only have one mask or is it feasible to have more different masks? If so, must all present masks have at least the same shape and otherwise connections in TF graph gets broken?

I think I know what triggers the error. For example, the computation graph of a keras model as depicted on the left-hand side works fine, but the example on the right-hand side raises the InvalidArgumentError from above.
The first concatenation of graph parts which contain connections to layers.Masking is, in my opinion, okay, but a later further concatenation of graph parts which again contain connections to layers.Masking seems to trigger the error.
I do not know the reason behind, but it seems to be not feasible or not supported yet. Any suggestions?

I am able to replicate the issue reported, please find the gist here.

Any feedback or ideas?

While trying to reproduce your issue in Tf Nightly 2.6 facing different error, please find the gist here, Thanks!

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


postgresql: default to v15 in 23.11

Description of changes
Change the default postgres version from 14 to 15 in release 23.11.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This PR should go to staging.
https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#flow-of-merged-pull-requests

@nbdd0121 please rebase and fix merge conflicts

Rebased

@nbdd0121  looks like conflict again

how about making 16 default for 23.11 ?

I think psql 16 is too new for 23.11 release. IIUC timescaledb doesn't yet support psql 16. Probably would be better to wait a bit before making it the default.

I'm sorry to come up with this now, i haven't seen this PR before it got merged. I think users will probably get confused that services.postgresql.ensureUsers/ensureDatabases will not "just" work (see #216989 for context). imo we should use postgresql 14 for less confusion (it's okay if you feel different)

We have 3 PRs to choose from to fix this: #203474 , #252071, #255961


DaysOfWeek field requires update on this documentation

Hello There,
I could see this field has been given with some predefined values, but when I query WMI it shows some other values which is not the one shared in the document. Sample given below:
MOF Instance of a task from SCCM Primary Site:
instance of SMS_SCI_SQLTask
{
BeginTime = "00000000000000.000000+";
DaysOfWeek = 127;
DeleteOlderThan = 30;
DeviceName = "";
Enabled = TRUE;
FileType = 2;
ItemName = "Delete Aged Client Download History";
ItemType = "SQL Task";
LatestBeginTime = "00000000050000.000000+";
NumRefreshDays = 1;
SiteCode = "XXX";
TaskName = "Delete Aged Client Download History";
TaskType = 3;
};
This task is enabled for all days with delete option data > 30 days, but in WMI it shows 127, how can we calculate the days if this documentation proved to be right?
~Praveen

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

ID: 2b61e290-f893-381d-2a4d-ed2fe0a46d87
Version Independent ID: 0cb1ffe5-de99-b1e5-39a0-fbc9caf6ab88
Content: SMS_SCI_SQLTask Class - Configuration Manager
Content Source: sccm/develop/reference/core/servers/configure/sms_sci_sqltask-server-wmi-class.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks to @jpcast88 for contributing content for this issue, which is why it closed. The changes will publish live tomorrow around 11:00 Pacific time.


add lang option to GeoSearchOptions (multi language support)

Hi, thanks for your nice software!
I want to get the search result in English not only in Czech.
I read Mapy.cz Suggest API's document and I found that the API support multi language output such as English(en), Czech(cs), German(de), Polish(pl), Slovak(sk), Russian(ru), Spanish(es) and French(fr).
So I added lang option to GeoSearchOptions. Users can show result in a language which they prefer.
And, this option is also optional. So this PR is not breaking change.
geoSearch.suggest("Kyoto tower", {
    lang: "en" // "en" | "cs" | "de" | "pl" | "sk" | "ru" | "es" | "fr"
}).then(result => {
    console.log(result); // returns search results in English
})

Hi, @bartholomej
Thank you for approving my changes!


Check if url has https protocol scheme

Checks scheme for https before it sets as default 80 port @andythsu

Idk if there should be a document that tells user to have url port in backend_url if it is not default ones (80/443)
maybe at https://github.com/trinodb/trino-gateway/blob/main/docs/operation.md page?
https://trinodb.slack.com/archives/CG9K9MX1V/p1696522430792409?thread_ts=1696494602.350609&cid=CG9K9MX1V

@andythsu @vishalya Hey, i am having hard time trying to write test code with Mockito, any help would be welcome.
I can't find a way so that Mockito would use mock Conn instead of trying to connect to the mocked jdbc urls.

https://github.com/trinodb/trino-gateway/blob/32cc62f6800e52058bf774e0a29fd3fe18044bdf/gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java#L77-L79

I found this https://stackoverflow.com/questions/19464975/how-to-mock-drivermanager-getconnection link, maybe you can give it a shot

@Chaho12 You can check out my PR here: https://github.com/trinodb/trino-gateway/pull/71
I basically built upon yours

I built upon yours and fixed the test. You can check out my PR here: #71

Wow! What a simple way to test! Thx for sharing the PR. I'll close this one.
@ParameterizedTest
@MethodSource("provideSchemeAndPort")


Support Bucket/Object lock operations

To Dos

Tests

[x] System
[ ] Unit

References

https://docs.google.com/document/d/1wRpiAxTHZoypR1NLVP97Cj0WCGVKx43Tivso0xXhyEA/edit?ts=5b60f543#
https://www.googleapis.com/discovery/v1/apis/storage/v1/rest

This introduces new behavior:
Buckets

bucket#lock()
Lock a previously-defined retention policy. This will prevent changes to the policy.

bucket#removeRetentionPeriod()
Remove an already-existing retention policy from this bucket.

bucket#setRetentionPeriod(durationInSeconds)
Lock all objects contained in the bucket, based on their creation time.

Files

file#getExpirationDate()
Get a Date object representing the earliest time this file will expire.

Additionally, bucket#getMetadata(), bucket#setMetadata(), file#getMetadata(), and file#setMetadata() are available if users wish to interact with the raw resource schemas as defined by the API.

@frankyn I've added the system tests to verify the hold behavior will block overwrite and delete operations on File objects. I've also removed file.hold() and file.release() and updated the docs for file.setMetadata().


Code clean-up to satisfy Credo complaints

@jeregrine I was going through the code locally to clean up a few things and decided to run credo against the project which turned up a few more things I went ahead and cleaned up.  Let me know your thoughts, we can always close this PR too 😁
If you'd like I can add a commit adding Credo, it's configuration, and a Travis hook.

@doomspork 👍 on adding credo

I'll resolve the conflict, add Credo, and ping you for a second go round 😁

This is good to go @jeregrine 👍

Rebase and conflict free again!

Thank you! ❤❤


Set Unique Slug Per Tenant

Hi,
First of all, thank you for such great package.
I have been using it for almost all of my projects.
Is it possible to set unique slug per tenant in multi-tenant app?
I have tenant_id column in each table so I need to set unique slugs only per tenant_id.
Any suggestion would be highly appreciated.
Thank you

Yup!  Take a look at https://github.com/cviebrock/eloquent-sluggable#scopewithuniqueslugconstraints for how to do it.


How to get selected option from material_select

Not a bug, but a question:
materialize creates a ul from simple select-element, how can you get the currently selected option?
<select id="myselect">
    <option value="1">A</option>
    <option value="2">B</option>
    <option value="3">C</option>
    <option value="4">D</option>
    <option value="5">E</option>
</select>

Thanks for this awesome framework!

val() on the original select
On Mar 4, 2016 11:43 AM, "Ben S."<EMAIL_ADDRESS>wrote:

Not a bug, but a question:
materialize creates a ul from simple select-element, how can you get the
currently selected option?

A
B
C
D
E

Thanks for this awesome framework!
—
Reply to this email directly or view it on GitHub
https://github.com/Dogfalo/materialize/issues/2877.

Thanks!

I tried to set the selected value with $("#id").val("value") ,but it doesnt works out. Any suggestions?

Try just .val() without any parameters.

I using $('#selectId').parent(["0"]).children()[1].value

That is a weird approach @uwong15.

So where does .val() go? If you want to capture the selected values in the HTML, are you saying you would use .val() somewhere in the component.ts to obtain them?

$('select#jsSeatSection').val()
I've used this and it works. Don't forget to select some value then try otherwise you see "null"

you should call $("#yourselect").val("2") after call$('select').material_select();ans you will see change
hope help :)

And it's sad that's it's necessary to create a new plugin instance just because the value was changed from the outside.

If you look at line 28 in /js/select.js, the new object being constructed gets assigned as an attribute of the element it's constructed from this.el.M_FormSelect = this;.  After logging that attribute to the console and navigating the object you can see the value in M_FormSelect.input.value, This works for me:
someVar = document.getElementById('my-select').M_FormSelect.input.value;


Replace salt/credit emoji with CashIcon

Replace the salt/credit emoji with the CashIcon everywhere.

@ceeriil  said he will work on this

@ceeriil did you work on this?


NIFI-5021 Moving nifi-elasticsearch-client-service-api to nifi-elasti…

…csearch-bundle and creating NAR for the API
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

+1 LGTM. I ran it and didn't notice any regression.

Thanks for reviewing, I'm going to wait til 1.6.0 release finalizes before merging this because there will likely be conflicts when the poms get updated to 1.7.0-SNAPSHOT.

Sounds good to me.

@bbende @MikeThomsen since there is another RC of 1.6.0, did you want to merge this in?

I think that would be a good idea.

Agreed, will merge


Ability to rename filenames directly in the database

We can edit text properties, but we cannot edit the filename directly. Would be nice to be able to rename files directly in the database like a textfield.

the principal problem with this is the integration of the autorenaming link of all the references inside the vault. Needs to check if could be done

I found. a way to use context-menu of obsidian


Two-column layout presentation?

In our presentation, we'd like to show some texts on the left and a picture on a right in one page.
I used <div class="columns-2"> and </div> as explained in this page
When I knit the Rmd, R studio shows them correctly like this:

But once I open the html file in a browser (which I think is what we're going to do in class tomorrow), the two columns just collapse into one :(

Does anyone know how to fix this? Just in case, the Rmd is here.
Many thanks!

Curious.
Side notes:

if you want to run the presentation off of your own laptop, that's ok

it would be good to remove the y and x axis labels and tick marks (I think there is an example in the recent lecture slides)

I am unclear how the plot (which really shows regional variation) is a good representation of the information in the bullet point (which is about a national average). I personally would just show the map and say that on average 18% of British people are interested in EVs, with support being about 7% more in souther regions. Or something like that.

Thank you so much for the quick answers!
Yeah, for this page I could just show the map. But I have another page with this 2 column layout (with more information). Maybe I'll bring my laptop tomorrow.
And thanks for the advice on plot labels!


How to run train crpits on multi-gpus in single node?

How to run train crpits on multi-gpus in single node?
@longouyang @welinder @JoshuaGross @jietang @esigler

@pokameng Did you manage to solve the issue ?

hello, have you solved this issue?


Feature: Disable Link Sharing

Environment
Host

item
version

GROWI
4.1.8

Note

In our organization, it is not allowed to share some documents with externals. Due to this security policy, we are not able to use the latest version of GROWI without modifications.

Implemented.
Please wait next v4.3.0 release.


Converted ExtractBasePathRule to use Context

test(server): Removed unnecessary tests from ExtractBasePathRuleTest
refactor(server): Converted ExtractBasePathRule to use Context
feat(server): ExtractBasePathRule violation messages differ for Swagger vs OpenAPI

Working to #834

👍

:+1:

👍

:+1:


MM-20144 - Fixed Mattermost typo in System Console -> LDAP Settings

Summary

Fixed Mattermost typo in System Console screen.
Ticket Link
Fixes MM-20144

Remove every other translation file but en.json

Even the ones that have the text translated already? like pl.json?

Yes

@enahum Done! Yeah I wasn't sure if I could modify the translated files directly, hence you as a reviewer 😉 Thanks!

Creating a new SpinWick test server using Mattermost Cloud.

Ok now we are good @enahum

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 4092b137bd7bc27678b0af6aea269e2a5e72a83b.
Access here: https://mattermost-webapp-pr-4222.test.mattermost.cloud

Test server destroyed


Update README.md

Hey Roger! Good to see you back! Sorry I was busy making games previously... btw could you add me to the Slack channel?

@PotterDai hello!
I just realized that the slack link does not work.
Here is a working link
https://join.slack.com/t/asynkron/shared_invite/zt-kekdxn11-v9wBea8K8kD33WcGP17Wrg
There however expire after 30 days, so Im trying to find some way to automate this :-)

@rogeralsing Hey Roger! I see...
We should use https://github.com/rauchg/slackin using Asynkron or Proto.Actor domain.
That's what these guys are doing:
https://github.com/elementary/website/issues/2255
translators-slack.elementary.io


Tesla Model S mileage

Hi there,
I was wondering if there is a code to get the mileage of my tesla?
I am trying to control my trips with the JSON API.

You can get the current odometer reading from the Vehicle State endpoint:
https://timdorr.docs.apiary.io/#reference/vehicles/state-and-settings/vehicle-state
The response contains: "odometer": 2171.454021,


fix build version

change iomgr master cononfile to link to sisl from 11 -> 10, bump ver 10.0.7-> 10.0.8
this will resolve the building issue of homestore master branch

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

Comparison is base (c99fd9b) 45.23% compared to head (746130a) 45.28%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
+ Coverage   45.23%   45.28%   +0.04%
==========================================
  Files          54       54
  Lines        4001     4001
  Branches      478      478
==========================================
+ Hits         1810     1812       +2
+ Misses       2061     2059       -2
  Partials      130      130

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


CLR20r3 error when launching application

Microsoft Windows 7 Ultimate x86
app cannot by started - failed to load driver:
DriverMon-0.2-alpha:
EventType clr20r3

P 01:	DriverMon.exe
P 02:	<IP_ADDRESS>
P 03:	59ff3a57
P 04:	DriverMon
P 05:	<IP_ADDRESS>
P 06:	59ff3a57
P 07:	8
P 08:	67
P 09:	System.ComponentModel.Win32
OS version:	6.1.76<IP_ADDRESS>.256.1
next info 1:	0a9e
next info 2:	0a9e372d3b4ad19135b953a78882e789
next info 3:	0a9e
next info 4:	0a9e372d3b4ad19135b953a78882e789
DriverMon-0.3-beta:
EventType clr20r3
P 01:	DriverMon.exe
P 02:	<IP_ADDRESS>
P 03:	5a0c2db1
P 04:	DriverMon
P 05:	<IP_ADDRESS>
P 06:	5a0c2db1
P 07:	cc
P 08:	0
P 09:	System.NullReferenceException
OS version:	6.1.76<IP_ADDRESS>.256.1
next info 1:	0a9e
next info 2:	0a9e372d3b4ad19135b953a78882e789
next info 3:	0a9e
next info 4:	0a9e372d3b4ad19135b953a78882e789

Currently, I compiled it for x64 only. You'll have to compile from source for x86


Verify fields passed into MyModule.new/1

I might be wrong, but from my experiments using proto3 and exprotobuf, it seems like if I have a definition:
message Foo {
  string bar = 1;
}

and I compile it with exprotobuf, this happens:
Foo.new(baz: "typo!")
#=> %Foo{bar: nil}

If I'm not doing anything wrong myself, shouldn't we check that the fields passed into Foo.new/1 actually match all the fields defined by the schema (that is, struct!/2 behaviour and not struct/2 behaviour)?
\cc @bitwalker

This is critically needed, IMO

@bitwalker ping?

Seems like activity on the repo is very low, and we've switched to something else so this is not personally an issue for us anymore. @bitwalker I still believe that, provided I'm not missing something, this is a bug and the described behavior would be the correct one, but I need to keep my issues tidy so I'll close this 😄  Please feel free to reopen in the future!


Add doc_as_upsert in bulk requests

This PR adds a function doc_as_upsert (https://www.elastic.co/guide/en/elasticsearch/reference/master/docs-bulk.html#bulk-update) to bulk update requests.
This is my first PR on a rust project, I'm not sure if that's the "rust way" to do it.
Also I don't know if the tests are enough.
Any comments appreciated!

Thanks @tobias-o! This looks good to me.


Add Pop Out Modal Support

Description
This PR added pop-out modals to Moorhen which should allow more flexibility when working with multiple monitors.
Details
For a given component, you can use createPortal() to change the DOM the component is outputted to to a new window.
This PR should be worked on to add the following things:

A collapse all popouts button from within Moorhen
Resize card when window is resized - I assume a standard eventListener could do that.
Closing the pop outs when Moorhen is closed
Allow multiple pop outs at once, currently all new pop outs go to the same new window and since they aren't draggable, appear on top of each other, when closed only the last one is returned to the appropriate place.

I am happy to work on this.

Given that this is not going to the master branch and that as you pointed out there are a few other things that need to be done, I will simply merge this now. When the points you have outlined are addressed then open a PR from this branch into master and I'll make a review.


Select kernel from CPA to CubeCL

Pull Request Template
Checklist

[ ] Confirmed that run-checks all script has been executed.
[ ] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
Provide links to relevant issues and dependent PRs.
Changes
Summarize the problem being addressed and your solution.
Testing
Describe how these changes have been tested.

For some reason this fails the CI pipeline on mac and I don't understand why this is the case.


Expose Napkin_diagnostic's toString and explain function

Exposes the Diagnostic toString and explain function as public API.
Why?

The playground needs a way to transform a single Diagnostic.t into a pretty report string (right now I can only stringify a list of Diagnostic.t with stringOfReport)
The playground needs a small msg string for hover message display (basically only the thing explain returns)

If you need other formats or diagnostics data, let me know.  The sky is the limit.


feat: add fluentbit log tail configuration for each service

Which issue(s) this PR fixes

Fixes #613
Additional comments

This PR adds a configuration step to each service in which we template a valid Fluentbit configuration file to tail the service's logs.
Example for hbase_master:
[INPUT]
    Name tail
    Path /var/log/hbase/hbase-hbase-master-master01.tdp.log

I did not variabilize the ability to disable the creation of the file since it's the user's job to include these config in a Fluentbit configuration file (can be done with https://github.com/TOSIT-IO/tdp-collection-prerequisites/pull/63). The SERVICE-COMPONENT-fluent-bit.conf do nothing on their own.
Agreements

[X] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[X] I hereby agree to grant TOSIT a copyright license to use my contributions.

Closing since https://github.com/TOSIT-IO/tdp-collection-prerequisites/pull/63 won't be merged in favor of https://github.com/TOSIT-IO/tdp-observability/pull/24


Getting started improvements: what dart build?

The mac instructions say:

Mac:
brew tap dart-lang/dart && brew install dart, or, if you don’t use homebrew, grab the latest stable channel build

...but they don't say what build. There's several options: 32bit SDK, 32 bit Dartium, 64 SDK, and API docs. We should be specific about which version the developer should get.

We also don't say what you're supposed to do with it. Do I unpack the ZIP and put it in my Applications folder? Home directory? Some directory I created for Flutter development?

"Flutter requires Dart SDK 1.12.2 or greater" is probably wrong also. Don't we need at least 1.13 now?

https://github.com/flutter/flutter/pull/1895 makes this moot. Once we rev alpha we can remove all of this.


Update SPN for unknown languages

Update note on page for usage with https://github.com/ISDCF/registries/pull/830

Addresses note for https://github.com/ISDCF/registries/issues/783


fix(connectors): fixes MGDCTRS-331, allow scaling cos-fleet-manager pods

Description
Marks reconcile done in a background thread to support multiple fleet manager instances with only one with started workers.
Verification Steps
CI should work, integration test for checksum verification should pass.
Deployment in dev environment should be able to scaled to more than one fleetmanager.
Checklist (Definition of Done)

[X] All acceptance criteria specified in JIRA have been completed
[X] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
- [ ] Documentation added for the feature
[x] CI and all relevant tests are passing
[ ] Code Review completed
- [ ] Verified independently by reviewer
- [ ] Required metrics/dashboards/alerts have been added (or PR created).
- [ ] Required Standard Operating Procedure (SOP) is added.
- [ ] JIRA has created for changes required on the client side

I'm not sure I understand this PR in relation with MGDCTRS-331 is trying to achieve.
My understanding of MGDCTRS-331 is about being able to have multiple instances of the manger running but it is not clear to me what's the reason as AFAIK, only one manager can be active at time so even if we have 3 instance up, then they are not serving any traffic, am I right ?

the manager can run multiple instances and they can all serve requests since the request processing is stateless, i.e

I recall that there were some access pattern to the db not done in a transaction an in such case, having multiple replica could lead to undefined behavior.
For my knowledge, do we have any test for scenario where we have multiple replicas of the manager ?

I recall that there were some access pattern to the db not done in a transaction an in such case, having multiple replica could lead to undefined behavior.

That's an issue with even a single instance and parallel API calls. Unless we refactor all server APIs and use transaction isolation levels everywhere, it's a problem. Right now there is an underlying assumption that changes to the same resource won't happen in parallel.

For my knowledge, do we have any test for scenario where we have multiple replicas of the manager ?

No, the integration test harness is not built for that. That's why I want to do this in dev and see if we run into any issues. Also we are using the same core framework as RHOSAK that enables running multiple server instances.

I recall that there were some access pattern to the db not done in a transaction an in such case, having multiple replica could lead to undefined behavior.

That's an issue with even a single instance and parallel API calls. Unless we refactor all server APIs and use transaction isolation levels everywhere, it's a problem. Right now there is an underlying assumption that changes to the same resource won't be requested in parallel.

Yep that's why I was saying that having multiple instances, right now, doe not seem right in absence of some tests that would validate the assumption.

For my knowledge, do we have any test for scenario where we have multiple replicas of the manager ?

No, the integration test harness is not built for that. That's why I want to do this in dev and see if we run into any issues. Also we are using the same core framework as RHOSAK that enables running multiple server instances.

I think dev is not a a good way as there's' not real contention most of the time so we should have a real test case and scenario to prove this is working as expected.

I disagree. This PR doesn't change the number of instances, it allows the manager to be run in multiple instances. To change the number of instances, we'll have to change the deployment config.
For the number of instances issue, I thought it was already agreed that we would run multiple fleet manager instances, at least according to @chirino and the long open issue for this in JIRA.
Parallel API call ambiguity behavior won't be different irrespective of the number of instances, even if we add more transaction boundaries. And we are running exactly the same framework bits as kas-fleet-manager to support worker leases. The kas-fleet-manager is running 6 replicas in staging and production.
@chirino please add your thoughts.

I disagree. This PR doesn't change the number of instances, it allows the manager to be run in multiple instances. To change the number of instances, we'll have to change the deployment config.

I'm aware of that and I'm not questioning the PR, the only think I'm asking is to be sure that we are on the safe side before misusing the functionality. I know that the same problem affects also a single instance, but multiple instances would just exacerbate the problem.

For the number of instances issue, I thought it was already agreed that we would run multiple fleet manager instances, at least according to @chirino and the long open issue for this in JIRA.

Again, I'm fine with multiple instance, but reading through the comment it seems to me, and as often happens I may be wrong, that we don't have a clear idea if that would works or not.

That's an issue with even a single instance and parallel API calls. Unless we refactor all server APIs and use transaction isolation levels everywhere, it's a problem. Right now there is an underlying assumption that changes to the same resource won't be requested in parallel.

This assumption is a big assumption specially without data from usage and/or tests to be confident there would not be problems in case the assumption dose not hold...
That said I agree that merging this PR dose not mean automatically running multiple instances of the manager, unless we also specifically configure it. I also think, though, that merging this PR will not be sufficient to define MGDCTRS-331 as done.
Regardless of the merging of this PR, personally I would prefer to start GA with just one instance of manager and gain confidence that it runs fine and stable. Troubleshooting problems would be easier.

This assumption is a big assumption specially without data from usage and/or tests to be confident there would not be problems in case the assumption dose not hold...

@valdar, can you cite a use case that shows that's a big assumption? Preview release will only support eval namespaces, which are dedicated to single users. Do you anticipate the user opening multiple browser sessions to edit the same connector?

That said I agree that merging this PR dose not mean automatically running multiple instances of the manager, unless we also specifically configure it. I also think, though, that merging this PR will not be sufficient to define MGDCTRS-331 as done.

I'm glad we agree that this PR can be merged without having to change the number of instances. Since the title MGDCTRS-331 is "Allow fleet manager to scale up", can you create another issue to decide what the number of instances should be.

Regardless of the merging of this PR, personally I would prefer to start GA with just one instance of manager and gain confidence that it runs fine and stable. Troubleshooting problems would be easier.

@valdar can you also explain how you would support rolling updates with a single instance in preview release? Isn't that in the SLO?

You guys increase the risk of the uptime SLO breaching without extra
replicas.  If I was in SRE I would not allow a service to be rolled out
without a replica.
On Wed, Apr 6, 2022 at 5:08 PM Andrea Tarocchi @.***>
wrote:

That's an issue with even a single instance and parallel API calls. Unless
we refactor all server APIs and use transaction isolation levels
everywhere, it's a problem. Right now there is an underlying assumption
that changes to the same resource won't be requested in parallel.
This assumption is a big assumption specially without data from usage
and/or tests to be confident there would not be problems in case the
assumption dose not hold...
That said I agree that merging this PR dose not mean automatically running
multiple instances of the manager, unless we also specifically configure
it. I also think, though, that merging this PR will not be sufficient to
define MGDCTRS-331 as done.
Regardless of the merging of this PR, personally I would prefer to start
GA with just one instance of manager and gain confidence that it runs fine
and stable. Troubleshooting problems would be easier.
—
Reply to this email directly, view it on GitHub
https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/899#issuecomment-1090808786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAZGV25SKYRYGN5TMPN2YDVDX4LNANCNFSM5ST7MYCA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
https://red.ht/sig
Hiram Chirino
Chief Architect, Red Hat Integration
+1.813.966.8660 <+18139668660>
@.*** @.***>
Tampa, FL https://goo.gl/maps/5mBTN9PPbfZqDSu79

Thanks Luca, that's prudent. Let's continue the conversation on that jira issue.


key not found: cause error after adding migration event

I tried to create a temporary solution for the key not found: cause error that happens in the branches with migration events (CorefHandler can only work with the cause events and broke when a different type of event---migration event---was introduced, I think). Locally, all tests pass, but the travis build doesn't pass. Any suggestions?
This was the error (and also what caused the errors @zupon mentioned here https://github.com/clulab/eidos/issues/594#issuecomment-499684549):
[info] org.clulab.wm.eidos.system.TestParallel *** ABORTED ***
[info]   java.util.NoSuchElementException: key not found: cause
[info]   at scala.collection.immutable.Map$Map1.apply(Map.scala:108)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.existsDeterminerCause(CorefHandler.scala:100)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$5(CorefHandler.scala:51)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$5$adapted(CorefHandler.scala:48)
[info]   at scala.collection.immutable.List.foreach(List.scala:389)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$3(CorefHandler.scala:48)
[info]   at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:156)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.resolveCoref(CorefHandler.scala:47)
[info]   at org.clulab.wm.eidos.EidosActions.$anonfun$globalAction$4(EidosActions.scala:48)
[info]   at scala.Option.map(Option.scala:146)
[info]   ...
[info] org.clulab.wm.eidos.system.TestSerial *** ABORTED ***
[info]   java.util.NoSuchElementException: key not found: cause
[info]   at scala.collection.immutable.Map$Map1.apply(Map.scala:108)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.existsDeterminerCause(CorefHandler.scala:100)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$5(CorefHandler.scala:51)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$5$adapted(CorefHandler.scala:48)
[info]   at scala.collection.immutable.List.foreach(List.scala:389)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.$anonfun$resolveCoref$3(CorefHandler.scala:48)
[info]   at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:156)
[info]   at org.clulab.wm.eidos.actions.CausalBasicCorefHandler.resolveCoref(CorefHandler.scala:47)
[info]   at org.clulab.wm.eidos.EidosActions.$anonfun$globalAction$4(EidosActions.scala:48)
[info]   at scala.Option.map(Option.scala:146)

Not sure who to address this to, so tagging @MihaiSurdeanu and @kwalcock.

If there are both Travis and Jenkins tests, then we're concerned with the ones from Jenkins.  Travis tests will be running out of memory and testing there will soon be eliminated.  Sorry for the confusion.  It's temporary.

Thanks, @kwalcock! For this PR, who is the best person to assign as reviewer?

Yours truly, me.

And often @BeckySharp.

Great! Thanks, @kwalcock!

As far as I can tell, this is OK, but it is also effectively the same code that I tried once before.  At that time it seemed to help in a couple of cases, but not all of them.  I can't tell what will happen when the rest of migration_schema_* is added.  Perhaps you've already tried it or maybe time will tell.  I'll merge and see what happens next.  Thanks!

@kwalcock, thank you for having a look! That makes sense. It is currently helping with that error we were getting, but it will probably need to be revisited eventually.


Using reserved words as enum member in strict mode

TypeScript Version:  2.7.0-dev.201xxxxx

Search Terms:
reserved words
Code
enum AA{
    if,
    static
}

class A {
    a: AA.if | AA.static
}

Expected behavior:
Error should not be thrown when reserved words are used as props
Actual behavior:
Class throws TS1213: Identifier expected. 'static' is a reserved word in strict mode. Class definitions are automatically in strict mode.
Function throws TS1214: Identifier expected. 'static' is a reserved word in strict mode. Modules are automatically in strict mode.
Playground Link:
http://www.typescriptlang.org/play/index.html#src=enum AA{
    if%2C
    static
}

class A {
    a%3A AA.if | AA.static
}

Related Issues:
#17452

You can use quotes for these members just like you would with regular properties
enum AA{
    'if',
    "static"
}

It seems reasonable that each enum member be a String or an IdentifierName.

@DanielRosenwasser note that this is an error, which means the bug is elsewhere:
enum AA{
    "if",
    "static"
}

class A {
    a: AA.if | AA.static
}

Can I take a crack at this?

@dhruvrajvanshi go for it!

@RyanCavanaugh So, I've been looking into it for a few hours now, and I've run into a bit of an issue.
Checking for strict mode identifiers is currently done in binder.ts (checkStrictModeIdentifier). But at this point, I don't have a symbol table, so I can't check SymbolFlags on the parent node's symbol to see if it's an enum symbol.
I can think of two ways to deal with this (I've already tried out the second one and it seems to be working with a bit of a change to an existing test case's expected errors):

Do this check in typechecking phase rather than binding phase. This seems a bit extreme to me so I wouldn't prefer this but it should theoretically work. Does checker do a full traversal of every node?
Allow types to have strict mode reserved names everywhere.
So, this would be valid after the change

type static = number

I like this idea better because types are erased after checking and there's a clear syntactic seperation between types and expressions so there shouldn't be any conflicts, and the emitted file would be strict mode compliant (unless you plan to add dependent types in the future :P).
I'm pretty sure I'm missing a very obvious thing here. I'm not really familliar with the codebase and this is my first attempt at an issue so, a bit of help would be appreciated.
Thanks

@RyanCavanaugh I've attempted to fix this in https://github.com/dhruvrajvanshi/TypeScript/commit/ac97987ee00f4f6c5eb44c3f76abc2677c99f785. Can someone review it?
Basic gist of this is allowing strict mode reserved keywords as identifiers in qualified type references. This broke another baseline that expected an error, so I removed the expected errors from that.


The word invest/ invested is not recognized as a verb

Adding it to the lexicon solves the issue but, this is something that should not have happened because other verbs are almost working fine. Any idea why?

hi Hrishikesh, thanks, i'll take a look.


macOS app release doesn't include the version in the filename

Is there a way to fix this?

Is there a way to fix this?

Only by manually uploading them since https://github.com/tauri-apps/tauri-action/issues/215 isn't implement. Also, here's the upstream issue about this: https://github.com/tauri-apps/tauri/issues/5890. We'll likely (not a definite answer) won't explicitly change this in the action, but will wait for tauri's change in v2 and also hopefully add #215 before that so that users that want it can change it in v1 already.

Ah okay, well if it's planned for v2 then I'll manually rename it until v2 or #215 is added :+1:


Remove Helm release update checker

Since Helm does not store information about repository/chart where the release is installed from, we have to remove update checker.
We can re-implement this if/after Helm will have some metadata available: https://github.com/helm/helm/issues/6464
Fixes #361

Shame.. I liked the feature.


Typo fix in sonatype-nexus/README.md

Small typos fix.
#4956 is not synced, recreate it.
@rjkernick

/assing @rjkernick

/assign @rjkernick

/ok-to-test

/lgtm


fix: make PHPStan faster with Xdebug

Description
See below image

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

$ vendor/bin/phpstan --help | grep xdebug
      --xdebug                                 Allow running with XDebug for debugging purposes

In locally, when i run vendor/bin/phpstan analyze --xdebug better faster without --xdebug. If in GH Action affect report result to Coverage, i'll remove changes test-phpstan.yml

Reading into phpstan-src's code, phpstan actually turns off xdebug if active for better performance.

My env based on README Tests to use xdebug.mode to coverage.

My env based on README Tests to use xdebug.mode to coverage.

yes, that's for testing. in this case, xdebug.mode will be turned off only when running phpstan


Crashing when trying to enter the arena as of e1.4.0

When entering the arena of any of city, the game will crash to desktop. There have been many others reporting the same issue on NexusMods.

Fixed in version 1.4.5


Mute bypass

Right now you can bypass being muted by leaving and rejoining, shouldn't be too hard to fix but I'm just making this issue so I don't forget.

fixed


Extension Host is unresponsive

Extension Name: vscode-icons
Extension Version: 7.27.0
OS Version: Darwin x64 18.0.0
VSCode version: 1.28.0

We have written the needed data into your clipboard. Please paste:

Please read https://code.visualstudio.com/updates/v1_28#_extension-host-unresponsive


Rewards are always 0

The rewards field of each account is always 0 even when rewards were received:
Concretely
$ curl -s "localhost:8980/v2/accounts/SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE" | jq

returns
{
  "account": {
    "address": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
    "amount": 882040798,
    "amount-without-pending-rewards": 882040798,
    "assets": [
      {
        "amount":<PHONE_NUMBER>218796,
        "asset-id": 2512768,
        "creator": "",
        "is-frozen": false
      }
    ],
    "created-assets": [
      {
        "index": 2512768,
        "params": {
          "clawback": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
          "creator": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
          "decimals": 6,
          "default-frozen": false,
          "freeze": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
          "manager": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
          "metadata-hash": "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=",
          "name": "YouNow Pending Props",
          "reserve": "SN3FSQVLLNWSI3I7QCMQEQN6L42OW5RP4FNM6BNOZHBRKHODSQYU4MUPKE",
          "total":<PHONE_NUMBER>370955,
          "unit-name": "xPROPS",
          "url": "https://propsproject.com/"
        }
      }
    ],
    "pending-rewards": 0,
    "reward-base": 134204,
    "rewards": 0,
    "round": 8208842,
    "sig-type": "sig",
    "status": "Offline"
  },
  "current-round": 8208842
}

Indexer has made no attempt to track 'the sum of all rewards for an account'. This can be a feature request for the future.

What is the field rewards representing?

When algod returns that field it has the sum of all rewards an account has received. Indexer was initially focused on providing new functionality (search of transaction history) and didn't recreate that statistic.

It seems like it should include at least the same data as is available from the local node.  All the indexer needs to do is update the account record with the latest value as it updates everything else.  Latest block wins.    If the indexer doesn't return equivalent data to just calling the node itself and the /v1/account/xxx route, that seems a very unusual omission.
The fact that it actually returns the rewards property and returns it as 0 (when it isn't) is even more striking (and IMO a bug).  It should either return the correct value or not return it at all.

Maybe the best short-term solution

I agree. It is a bug and will get fixed as soon as possible.
Many thanks for catching it.

This should be addressed asap, any news?

simple fix: make "rewards" field on account optional. algod will set it and algorand-indexer will not set that member of the json dict. Then there won't be the confusing zero.

Can we have rewards returning the right value for the accounts instead of getting rid of that field?

@roccomuso we're going to support rewards with indexer. It's been prioritized and we'll start working on it soon.

This issue is being handled in the current sprint. Should have a solution within a week.

This is now implemented in the develop branch.

when this will be in the prod release?

@roccomuso it looks like this isn't going to make it into the next release due to how long it takes to update historical rewards data. I need to do improve the update logic so that it can happen in the background instead of blocking indexer while the migration happens. I don't have an ETA for when it will make it into a release, but am optimistic that it wont take more than another week or two.

@winder Any update on this?

@winder can you confirm this is fixed and available in the current release?

@roccomuso confirmed.


Please update website : https://www.youku.tv/. Thanks !

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

WRITE DESCRIPTION HERE

Please update website : https://www.youku.tv/. Thanks !

How? Why?
Maybe you meant to open a broken site support issue, with the required supporting information?

FWIW:
www.youku.com is the Chinese version of youku, and there's already an IE for that, youku.py:
class YoukuIE(InfoExtractor):
    IE_NAME = 'youku'
    IE_DESC = '优酷'
    _VALID_URL = r'''(?x)
        (?:
            https?://(
                (?:v|player)\.youku\.com/(?:v_show/id_|player\.php/sid/)|
                video\.tudou\.com/v/)|
            youku:)
        (?P<id>[A-Za-z0-9]+)(?:\.html|/v\.swf|)
    '''

I can't check www.youku.com in yt-dl  and in my browser, because it's mostly geo-blocked for non-Chinese IPs...
https://github.com/ytdl-org/youtube-dl/issues?q=youku.com+is%3Aopen
and
https://github.com/ytdl-org/youtube-dl/issues?q=youku.com+is%3Aclosed
both suggest the youku.com IE might be broken (up for investigation by Chinese natives) ...
OP is asking for support of www.youku.tv, which is the international version of youku 😜 ; below URI:
https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html
plays fine (no account required, no DRM enforced) in my South European location; but there's a catch: the first time you load www.youku.tv in your browser, you're presented with a "smart captcha" challenge (please, slide [towards the right] to verify [you're not a bot]):

not sure if/how yt-dl will be able to cope with that...
FTR:
yt-dl -vF "https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html" =>

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--ffmpeg-location', '.\\FFmpeg', '--external-downloader-args', '-v 8 -stats', '-vF', 'https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html']
[debug] Encodings: locale cp1253, fs mbcs, out cp737, pref cp1253
[debug] youtube-dl version 2<IP_ADDRESS>4
[debug] Lazy loading extractors enabled
[debug] Single file build
[debug] Python 3.4.10 (CPython x86 32bit) - Windows-Vista-6.0.6003-SP2 - OpenSSL 1.0.2k  26 Jan 2017
[debug] exe versions: ffmpeg n6.1-dev-1700-N-111584-ga4e6168, ffprobe n6.1-dev-1700-N-111584-ga4e6168, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] id_XNTk3NTM3NTU4OA==: Requesting header
WARNING: Falling back on generic information extractor.
[generic] id_XNTk3NTM3NTU4OA==: Downloading webpage
[generic] id_XNTk3NTM3NTU4OA==: Extracting information
WARNING: [generic] id_XNTk3NTM3NTU4OA==: Failed to parse JSON Invalid control character at: line 12 column 1367 (char 1817)
ERROR: Unsupported URL: https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html
Traceback (most recent call last):
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\YoutubeDL.py", line 859, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\YoutubeDL.py", line 955, in __extract_info
    ie_result = ie.extract(url)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\extractor\common.py", line 565, in extract
    ie_result = self._real_extract(url)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\extractor\generic.py", line 3704, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html

meant to open a broken site support issue,

more like a Site Support Request 😜 (hopefully, a "fixed" youku.py can handle both *.youku.com and *.youku.tv URIs ) ...

FWIW:
www.youku.com is the Chinese version of youku, and there's already an IE for that, youku.py:
class YoukuIE(InfoExtractor):
    IE_NAME = 'youku'
    IE_DESC = '优酷'
    _VALID_URL = r'''(?x)
        (?:
            https?://(
                (?:v|player)\.youku\.com/(?:v_show/id_|player\.php/sid/)|
                video\.tudou\.com/v/)|
            youku:)
        (?P<id>[A-Za-z0-9]+)(?:\.html|/v\.swf|)
    '''

I can't check www.youku.com in yt-dl and in my browser, because it's mostly geo-blocked for non-Chinese IPs...
https://github.com/ytdl-org/youtube-dl/issues?q=youku.com+is%3Aopen
and
https://github.com/ytdl-org/youtube-dl/issues?q=youku.com+is%3Aclosed
both suggest the youku.com IE might be broken (up for investigation by Chinese natives) ...
OP is asking for support of www.youku.tv, which is the international version of youku 😜 ; below URI:
https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html
plays fine (no account required, no DRM enforced) in my South European location; but there's a catch: the first time you load www.youku.tv in your browser, you're presented with a "smart captcha" challenge (please, slide [towards the right] to verify [you're not a bot]):

not sure if/how yt-dl will be able to cope with that...
FTR:
yt-dl -vF "https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html" =>

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--ffmpeg-location', '.\\FFmpeg', '--external-downloader-args', '-v 8 -stats', '-vF', 'https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html']
[debug] Encodings: locale cp1253, fs mbcs, out cp737, pref cp1253
[debug] youtube-dl version 2<IP_ADDRESS>4
[debug] Lazy loading extractors enabled
[debug] Single file build
[debug] Python 3.4.10 (CPython x86 32bit) - Windows-Vista-6.0.6003-SP2 - OpenSSL 1.0.2k  26 Jan 2017
[debug] exe versions: ffmpeg n6.1-dev-1700-N-111584-ga4e6168, ffprobe n6.1-dev-1700-N-111584-ga4e6168, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] id_XNTk3NTM3NTU4OA==: Requesting header
WARNING: Falling back on generic information extractor.
[generic] id_XNTk3NTM3NTU4OA==: Downloading webpage
[generic] id_XNTk3NTM3NTU4OA==: Extracting information
WARNING: [generic] id_XNTk3NTM3NTU4OA==: Failed to parse JSON Invalid control character at: line 12 column 1367 (char 1817)
ERROR: Unsupported URL: https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html
Traceback (most recent call last):
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\YoutubeDL.py", line 859, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\YoutubeDL.py", line 955, in __extract_info
    ie_result = ie.extract(url)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\extractor\common.py", line 565, in extract
    ie_result = self._real_extract(url)
  File "C:\Python3410-32\yt-dl_pycompile\youtube_dl\extractor\generic.py", line 3704, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.youku.tv/v/v_show/id_XNTk3NTM3NTU4OA==.html

meant to open a broken site support issue,

more like a Site Support Request 😜 (hopefully, a "fixed" youku.py can handle both *.youku.com and *.youku.tv URIs ) ...

Yes please update youku.tv into [youku] extractor. Thanks so much!

@jayt1990
Please, there was no need to quote the bulk of my comment, especially since it was immediately preceding yours 😉 ...
I can also understand English not being your "forte", but, be that as it may, you should have started by first reading fully the PINNED issue #30839 and then proceeding to file a proper Site Support Request (by complying fully to its template), if you were so inclined...
There's currently only one person taking on "maintainer duties" and it's hard for him to process incomplete issue reports; I hope you do understand that 😄 ...
Regards.

See https://github.com/yt-dlp/yt-dlp/issues/8058


Refresh page when clicking on corresponding button

On Youtube, clicking Home when I'm at Home, or Subscriptions when I'm at Subscriptions refreshes the page.
I think it would work well on Holodex as well.

So technically when you move between the two pages, it will try to refresh the videos like Youtube, if there is new videos. There is an additional 60 second delay, so the mobile versions can't spam refreshes while tabbing back and forth.
Please let me know if you are having trouble getting it to refresh.

I just find it slightly annoying to tab out and the tab in again. Refreshing works fine

Oh, I misunderstood your original question. I'll see what I can do about it.

Added in latest update


Zooz temp/humidity sensors stop reporting humidity on 9.16.0

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
9.16.0
ZwaveJS version
1.37.0
Describe the bug
My ZSE40 and ZSE44 sensors stopped reporting humidity after updating to 9.16.0 from 9.15.0.
After excluding and re-including the ZSE40 to troubleshoot, the humidity was not even listed as a sensor value in the UI. Only temperature was listed.
Reverting to 9.15.0  resolved the issue. You can see when the upgrade and downgrade took place looking at these graphs.

To Reproduce
Upgrade zwave-js-ui to 9.16.0 with a ZSE44 or ZSE40 sensor.
Expected behavior
Humidity values are reported.
Additional context
No response

Please make a driver log, loglevel debug and attach it here as a file (drag & drop into the text field).


Optional Profile Image

Issue 417

In UpdateAccount component:

image: Yup.mixed().required() replaced with Yup.mixed().nullable().test(with the present test) fixed this.

Hello @devkiran. In this issue, I forgot to include a button so the user can remove photo. I have done that now and created a PR to update it.


Validate Field and Fields Parameters in Relevance Search Functions

Description
Relevance functions that query fields should act similar to how a SELECT query works. If a field is queried that does not exist, a SemanticCheckException should be thrown.
Example Queryies
SELECT * FROM stackexchange_beer WHERE query_string([invalid], 'beer');
{'reason': 'Invalid SQL query', 'details': "can't resolve Symbol(namespace=FIELD_NAME, name=invalid) in type env", 'type': 'SemanticCheckException'}

SELECT * FROM stackexchange_beer WHERE match(invalid, 'beer');
{'reason': 'Invalid SQL query', 'details': "can't resolve Symbol(namespace=FIELD_NAME, name=invalid) in type env", 'type': 'SemanticCheckException'}

Issues Resolved
Issue: 613
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

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

Codecov Report

Merging #1067 (d3700ae) into 2.x (03f30e3) will decrease coverage by 35.50%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.x    #1067       +/-   ##
=============================================
- Coverage     98.27%   62.76%   -35.51%
=============================================
  Files           327       10      -317
  Lines          8457      658     -7799
  Branches        553      119      -434
=============================================
- Hits           8311      413     -7898
- Misses          142      192       +50
- Partials          4       53       +49

Flag
Coverage Δ

query-workbench
62.76% <ø> (?)

sql-engine
?

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

Impacted Files
Coverage Δ

...ain/java/org/opensearch/sql/analysis/Analyzer.java

...h/sql/expression/function/OpenSearchFunctions.java

...rage/querybuilder/TimeRangeParametersResolver.java

.../opensearch/sql/data/model/ExprTimestampValue.java

.../org/opensearch/sql/ppl/antlr/PPLSyntaxParser.java

...org/opensearch/sql/expression/NamedExpression.java

...ensearch/sql/planner/logical/LogicalMLCommons.java

...rg/opensearch/sql/data/model/ExprMissingValue.java

...ch/planner/logical/rule/MergeLimitAndRelation.java

...rg/opensearch/sql/planner/logical/LogicalPlan.java

... and 327 more

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

@forestmvey can you make sure that we aren't reducing coverage with this PR?


Allow whitespace in Base64 encoded strings.

Fixes #100.
The JavaScript implementation already stripped whitespace.

LGTM


feat: use gas price from API

Codecov Report

Merging #164 (918a524) into master (0ce9381) will increase coverage by 0.08%.
The diff coverage is 28.57%.

@@            Coverage Diff            @@
##           master    #164      +/-   ##
=========================================
+ Coverage    3.90%   3.99%   +0.08%
=========================================
  Files         527     533       +6
  Lines        9639    9674      +35
  Branches     2751    2760       +9
=========================================
+ Hits          376     386      +10
- Misses       9263    9288      +25

Impacted Files
Coverage Δ

<EMAIL_ADDRESS>0.00% <0.00%> (ø)

<EMAIL_ADDRESS>0.00% <0.00%> (ø)

<EMAIL_ADDRESS>0.00% <0.00%> (ø)

<EMAIL_ADDRESS>0.00% <0.00%> (ø)

app/src/pages/earn/index.tsx
0.00% <ø> (ø)

<EMAIL_ADDRESS>58.33% <58.33%> (ø)

<EMAIL_ADDRESS>100.00% <100.00%> (ø)

<EMAIL_ADDRESS>100.00% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


LockAssert.makeAllAssertsPass(true) can start more than one test lock

When migrating to the latest version, we noticed (in addition to #1635) that since 5.7.0 multiple calls to LockAssert.TestHelper.makeAllAssertsPass(true) result in multiple test locks (net.javacrumbs.shedlock.core.test-lock).
In previous version one call to LockAssert.TestHelper.makeAllAssertsPass(false) was sufficient to reset the behaviour for the test configuration. Now a LockAssert.TestHelper.makeAllAssertsPass(false) is required for each LockAssert.TestHelper.makeAllAssertsPass(true).
While this can easily be solved in tests by e.g. using a try.. finally or a @BeforeEach + @AfterEach the old behaviour is preferable from our point of view (or alternatively a reset method that clears the ThreadLocal in LockAssert) .
Expected behavior
LockAssert.TestHelper.makeAllAssertsPass(true);
LockAssert.TestHelper.makeAllAssertsPass(true);

LockAssert.TestHelper.makeAllAssertsPass(false);

assertThatThrownBy(LockAssert::assertLocked).isInstanceOf(IllegalStateException.class);

Actual behavior
LockAssert.TestHelper.makeAllAssertsPass(true);
LockAssert.TestHelper.makeAllAssertsPass(true);

LockAssert.TestHelper.makeAllAssertsPass(false);
LockAssert.assertLocked(); // although makeAllAssertsPass was set to false the assertion still passes

LockAssert.TestHelper.makeAllAssertsPass(false); // only if makeAllAssertsPass is called with false for each previous call with true the assertion fails
assertThatThrownBy(LockAssert::assertLocked).isInstanceOf(IllegalStateException.class);

we will add a proposal/PR to change this in a minute


(IMAGES-1323) Adds RedHat 9 support

A companion PR to https://github.com/voxpupuli/beaker/pull/1719

I'd really be in favor of dropping codecov. It's obviously not doing anything useful and only generates noise. That's my general impression of codecov, but especially true here.

Could we get a beaker-hostgenerator gem release that contains this fix? It looks like this is handled through the github action, but I wanted to ask before kicking it.

@joshcooper I can take care of this today; the steps for releasing are documented here.


[EN DatetimeV2] 11.05.2018 recognized like 2 DatetimeV2

The text "11.05.2018" is recognized like :
datetimeV2 --> "2018"
And the text "on 11.05.2018" is recognized like :
datetimeV2 --> "11 . 05"
datetimeV2 --> "2018"
I am also interested by the support for English regional formats. Those interested can vote for it here:
https://cognitive.uservoice.com/forums/551524-language-understanding-luis/suggestions/31549864-luis-add-support-for-english-regional-formats

Support for date formats in dot-syntax is being improved in PR https://github.com/Microsoft/Recognizers-Text/pull/511, it should be merged in soon.
I'll close this issue for now.


dialoggpt model creation

What are the changes?

Who worked on the changes?

New Components/Features

Enhanced Components/Features

Other Changes/Fixes

Additional Notes

TODOs

A lot more documentation and explanation needed, please reopen if you work on this further.


Print gitlab password

this is a duplication of the verify step which gives you the password after reconfiguration. what your logic?

this breaks CI pipeline: fatal:
[gitlab-enterpriselinux-latest]: FAILED! => {"changed": false, "msg": "file not found: /etc/gitlab/initial_root_password"}


Compatibility for Thread._locks

See https://github.com/dlang/druntime/pull/2519

Hmm, this doesn't work: https://issues.dlang.org/show_bug.cgi?id=19761

The whole point of using this method was to grant us access to private members which we need in order to switch fibers not from within the Thread druntime module.
The real solution to this is to create an advertised API in Thread for switching fibers, and have D's Fiber class use that instead of accessing private members of Thread (there is no reason for Fiber and Thread to share a module aside from this).
This would allow Mecca to use public APIs for switching fibers.


chore: remove unused grafana & prometheus charts

remove the grafana & prometheus charts from the Helm stable repository, which is deprecated.
we're now using charts maintained by grafana & the prometheus community directly

Hi @vbehar. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign hferentschik
You can assign the PR to them by writing /assign @hferentschik in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vbehar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

kube
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

kube
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

kube
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test vault

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

kube
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test kube

tls
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

tls
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test tls

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

tls
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@vbehar: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

tls
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test tls

gsm-kapp
b5873d74fd3e2a2ae84ab933b64df7d2616faf41
link
/test gsm-kapp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest


Link to download SQL Stored Procedure is missing in blockchain/blockchain-workbench/iot-integration-samples/ConfigureIoTDemo.md page

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

https://github.com/Azure-Samples/blockchain/blob/master/blockchain-workbench/iot-integration-samples/SQL/IoTSprocs.sql

Link added to document.


Plugin modification to support enhanced features with Grafana 8

Features supported:

Supports Grafana8
Alerting
Multi-tenancy data metrics via multi profile or cross tenancy policy
Filter based on resource tags
Filter by group by option
Filters are more user-friendly (show names instead of ocid)
Dashboard support labels like Prometheus

Signed-off-by: Chiranjit Datta

Please don't merge.
It requires review.

Hi,
Looking for help to validate i'll post some issues that i found on this PR today. Issues that i noted:

After i update i've lost some configurations on graphs and then it stopping working. So, when i select the data again that is missed, all the other fields are erased and i should fill all again.

I'm monitoring CPU from instance machines and i have a instance pool, so my ecosystem tends to power on and off machines from time to time. When this happen the graph is mixing data from one machine to another instead of being null.

Fields on Query won't filter data properly:

If this won't help, please let me know and i stop posting.

Hi,
Looking for help to validate i'll post some issues that i found on this PR today. Issues that i noted:

After i update i've lost some configurations on graphs and then it stopping working. So, when i select the data again that is missed, all the other fields are erased and i should fill all again.
I'm monitoring CPU from instance machines and i have a instance pool, so my ecosystem tends to power on and off machines from time to time. When this happen the graph is mixing data from one machine to another instead of being null.

Fields on Query won't filter data properly:

If this won't help, please let me know and i stop posting.

I will check and reply you back

Hey guys, any plans to release this version soon? We are missing alerts and multi tenant support.

any update on the ETA for this to be released ... mostly interested in Alerting being enabled on this datasource

We are trying to configure dashboard based on multiple OCI tenancies We could add two tenancies as OCI data source. One tenancy is on which we have installed Granana and for it we can make the dashboard. However when we try to make the report for other tenancy (where granae is not installed) then we don't get the list of compartments while making the dahsboard. From the instance where we have installed grafana we can make a CLI query to list compartments and metrics of other tenancy so permissions seems to be fine.
Did we miss something or this plugin works only for the single tenancy.


Split interop into interop-cats,interop-future and interop-scalaz7x.

refs:https://github.com/scalaz/scalaz-zio/issues/394

@hepin1989 One more change and should be good to go: can you update the microsite so it contains appropriate directions for each of the different modules? They will be different JARs now, and will require separate changes to the sbt build file dependencies.

@jdegoes I have updated it.

@hepin1989 Perfect! If you can resolve the few conflicts, we'll get this merged. Thanks for your work on this!

@jdegoes Let me first fix my Akka PR:)

@jdegoes I have rebased, it's your turn now:)


Add ability to upload logo for wlan portal

Currently we have the resource for the backround image which is both mist_org_wlan_portal_image and mist_site_wlan_portal_image it would be nice to have the ability to upload the logo for the wlan portal.
It appears the way this works today via the API is the image is base64 encoded and uploaded via a portal_template call as logo, logoHeight, and logoWidth
Additional Context
Continuation of https://github.com/Juniper/terraform-provider-mist/issues/21#issuecomment-2344654012

Thanks you @zbuchheit . I'll add it in the next release

available in v0.1.34

looks to be working, Thanks!


Cannot read property 'body' of null

I made a .d.ts file that contains a class without constructor function body, the eslint plugin report the error message as title.
screenshot:

@tjx666 what happens when you validate the file in the terminal. I am pretty sure it results in the same exception since it originates from a eslint npm module.
If it occurs in the terminal can you please report this against ESLint (see https://eslint.org/)


Misleading error message

https://github.com/RTIInternational/teehr/blob/2bd727d358dc5c4c566960eb295518217e30cf58/src/teehr/loading/utils_nwm.py#L169

@samlamont please close issues that have been resolved.  I think this one has.


Add option to automatically create a topic and re-use a subscription.

This adds autoCreate to pubsub.topic:
var topic = pubsub.topic('does-not-yet-exist', {autoCreate: true});
// topic will be created on first RPC.
topic.publish({data: {hello: 'world'});

And reuseExisting to topic.subscribe:
topic.subscribe('existingSub', {reuseExisting: true}, function(err, sub){ ... }));

Combined, these two features completely eliminate any bootstrapping code/steps required for the Getting Started on GCP sample.

Fixed the lint issues and updated after the review.
Also, forgot to mention in the PR - this fixes #445.

Overall LGTM. One stylistic nitpick: we add spaces between our keys in an object.
Like this: { hello: 'world' } not like this: {hello: 'world'}

LGTM. Thanks for doing this!

Merged in 3f8f60fd404a1bc6e0837356c6af8ac2f09f1b43

I wasn't able to get this working from the example you provided. Here's my code:
const PubSub = require('@google-cloud/pubsub')

const pubsub = PubSub({
  projectId: require('./cred.json').project_id,
  credentials: require('./creds.json')
})

const topic = pubsub.topic('hi-there', { autoCreate: true })

topic.publish({
  data: 'New message!'
}, function(err) {
  console.log(err)
});

pubsub.topic doesn't actually take an 2nd options parameter

A similar example came up on StackOverflow (http://stackoverflow.com/questions/38864043/gcloud-check-if-a-topic-exist-and-ability-to-reuse-the-topic), so I'll quote my response there:
--
I believe the problem you'll run into is that if a message is published to a topic that doesn't exist, it is immediately dropped. So, it won't hang around and wait for a subscription to be created; it'll just disappear.
However, gcloud-node does have methods that will create a topic if necessary:
var topic = pubsub.topic('topic-that-maybe-exists');
topic.get({ autoCreate: true }, function(err, topic) {
  // topic.publish(...
});

In fact, almost all gcloud-node objects have the get method that will work the same way as above, i.e. a Pub/Sub subscription or a Storage bucket or a BigQuery dataset, etc.
Here's a link to the topic.get() method in the docs: https://googlecloudplatform.github.io/gcloud-node/#/docs/v0.37.0/pubsub/topic?method=get

I have error
TS2345: Argument of type '{ autoCreate: boolean; }' is not assignable to parameter of type 'PublishOptions'.   Object literal may only specify known properties, and 'autoCreate' does not exist in type 'PublishOptions'.

these are publish options
export interface PublishOptions {
    batching?: BatchPublishOptions;
    flowControlOptions?: FlowControlOptions;
    gaxOpts?: CallOptions;
    messageOrdering?: boolean;
    enableOpenTelemetryTracing?: boolean;
}


: Report Formatting

Absence of numbering for tables and equations in the report.
Missing table numbers in the final report for logistic regression results.
Missing cross-references for figures within the text.

I will look into this and get this done - I'm not too sure about the table and equation numbering, I would prioritize others changes as using glue we can only specify them as 'Fig' and they are actually Tables and Equations - so this personally I think would take away from the report.


Fix logo rendering

Converted the text to shapes to avoid rendering issues.

Yup sorry for the delay. This works for me too!

On Nov 13, 2019, at 3:44 PM, Amanda Dumi<EMAIL_ADDRESS>wrote:

This looks good on my screen now, both font and no cropping issues. Since @shivupa had a problem too, I want to wait to make sure his is okay too before merging.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


[DOI Registration] User-side Updates and Appeals via Email

[ ] Implement functionality for the owner of a resource that has been submitted for formal publication to remove the resource from the publication queue.  I suggest a button on the landing page visible only to owners, when a resource is on the pending list. This button will only be available when the resource is pending publication. Once a resource has been approved for publication by a CUAHSI reviewer, this button would no longer appear.
[ ] Resources in the publication pending queue are not editable (this is currently the case and should be preserved).  The only things that can be done to them are that a reviewer can approve or decline publication, or an owner can withdraw it from publication (enabling further work and possible resubmittal).
[ ] Resources denied formal publication by a CUAHSI-approved reviewer may be appealed via email to ??????

@dtarb @Lizabrazil  I need the text for these emails as well, or for the UI notifications

The UI notification on the resource landing page
The Email sent to the user when their resource enters the queue
The "your appeal has been received" email and the appeal email sent to admins

Alternatively, we can point them to the<EMAIL_ADDRESS>flow

For UI notification on the resource landing page I suggest "Your resource has been submitted for publication.  It will be checked for completeness by a CUAHSI staff member and once they approve it, it will receive a digital object identifier (DOI)."
It is unclear what you are asking here.  What does "enters the queue" mean.  I think that the next step after a resource being submitted for publication is that the CUAHSI staff person either approves it or declines it.  If the CUAHSI staff person approves it an email should go to the user that reads.  "Congratulations! Your resource  that you submitted for publication in HydroShare has been published."  I think that this email should only go out after the DOI has been registered, so that whey they click on the link it works.
If a resource is denied, the CUAHSI staff person should write a short explanation why.  This should say what is missing.  I do not think a general email can be crafted, because it will vary on a case by case basis.  It would be simple if the system for approving/denying publication included a text box for the staff person to write this message.  The from address in the email sent to the user should be one to which they can reply if they disagree and want to appeal.  Then any discussion continues from there.

<EMAIL_ADDRESS>does not currently exist on the CUAHSI email server.  I believe that CUAHSI support uses<EMAIL_ADDRESS>

When a resource enters the satte of under-review, will it still discoverable?

@engineSound Yes, because it is already public.

This has been indexed alongside 25 related GH Issues, and relevant information extracted into a growing design document, and reorganized beneath major topics. Areas for meeting/presentation/async viewing by stakeholders, meetings and discussion/action items are also designated.

This has been indexed alongside 25 related GH Issues, and relevant information extracted into a growing design document, and reorganized beneath major topics. Areas for meeting/presentation/async viewing by stakeholders, meetings and discussion/action items are also designated.


chore: update api compliance

Hello!
We are migrating to a new image for doing API compliance.
Check that the paths and names are correct to your repo.
If you have any questions don't hesitate to reach out.

Duplicate PR https://github.com/qlik-oss/sn-scatter-plot/pull/195


Fix wrong usage of == for strings

This bug causes the asset loader to miss textures
GearVRf-DCO-1.0-Signed-off-by<EMAIL_ADDRESS>Nola Donato

Will merge after lunch

Not yet – I haven’t fixed the issues…
From: Mihail<EMAIL_ADDRESS>Sent: Friday, July 8, 2016 9:20 AM
To: Samsung/GearVRf<EMAIL_ADDRESS>Cc: Nola Donato<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Samsung/GearVRf] Fix wrong usage of == for strings (#682)
Will merge after lunch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Samsung/GearVRf/pull/682#issuecomment-231404064, or mute the threadhttps://github.com/notifications/unsubscribe/AQw1S_N9-i6Ku7zCUtu9TeHehHQqsKDdks5qTni9gaJpZM4JHlgc.

OK to merge now I have used the clever "".equals way of comparing


add surface forcing  lfo_Nx+-   to HISTORY.rc.tmpl

Add specs for land surface forcing (lfo) collection from the predictor segment to HISTORY.rc.tmpl.  This is needed for weakly-coupled land-atmosphere assimilation.

@gmao-rreichle @rtodling    a PR to add the forcing collection from predictor segment to HISTORY.rc.tmpl  in gcm_App

@sdrabenh: This PR seems trivially 0-diff to me, but let us know if you would rather see it tested.  I suppose there's always the risk that we introduced a syntax err in HISTORY.rc.tmpl
@biljanaorescanin


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In e699686, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 6862 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in 9ac871c after 25 minutes.