Rocket.Chat CI container for SDK tests

SDK tests require a parallel instance of Rocket.Chat (provisioned by the utils/setup.ts script) to run test interactions.
Currently it can only pass tests in local development, because it requires a manually provisioned instance to be running on localhost. Would be best if it can be scripted to create it's own temporary docker instance of Rocket.Chat, so tests can run in the CI environment.
The instance would need a default admin user/pass, then the test setup utils can use the API to create/edit/remove any other users or rooms required before/after each test.

@timkinnane Did you want me to sort this out for you? Would take me a hour or two

@JSzaszvari Amazing. No hurry though, kinda just flagging it so I don't forget.
It just needs these env vars for defaults ADMIN_USERNAME=admin and ADMIN_PASS=pass
Would this be something you'd set up within Circle CI as some kind of pre-test requirement, or could it run from command line script so it can be used locally too. The latter would be nice for anyone wanting to do development, so they don't have to wait for CI to tell them they've broken something, but maybe a nice-to-have if that complicates things.

@timkinnane I would set it up so that it spins everything up locally. Using a external CI service, like you said would just be too slow for a Dev environment

@JSzaszvari Cool, but I guess that would allow the same process to be automated in Circle CI anyway right? That would be nice so we can get green lights on PRs.

It should (In theory) I'll give it a try and make sure it works. Haven't used Circle in a while (I use GitLab CI and Jenkins) but I'm sure it won't be a issue

Would like to fix this.


checksum: prepare further behavior fix with a rework

Now that #6782, #6793 and #6815 have landed, it is time to achieve the rework of checksum.rs started in #6654.
This will allow us to fix #6572, #6614 and #6653.
This PR mainly tries to split the massive perform_checksum_verification in smaller functions, and to get rid of the &mut ChecksumResult that is passed everywhere, which makes it very difficult to understand where the global state of the program is modified.
Also, it adds ignored tests for future features (it will prevent too many conflicts when rebasing future PRs on each other).
Next PRs will focus on reworking the algorithm detection behavior, in particular for blake2b size guessing,
Regex detection behavior which for now prevents us from encountering both hexa and base64 digits in the same file.

Depending on the feedback on this discussion, I might split the checksum.rs file into several files (a bit like the initial PR #6654).
I'd like to move Error structs to checksum/error.rs, everything that's Algorithm-related to checksum/algo.rs, etc...
What do you think ?

I'd like to move Error structs to checksum/error.rs, everything that's Algorithm-related to checksum/algo.rs, etc...

What do you think ?

It sounds good to me.

I've finally decided to keep the splitting in several files for later, because it's a nightmare to mix changes and file architecture when you have to rebase to keep a sound change history.

I'm not sure why the macos test failed. Is it just flakey or have I touched something ?

flaky:
---- test_stat::test_stdin_pipe_fifo1 stdout ----
run: /Users/runner/work/coreutils/coreutils/target/aarch64-apple-darwin/debug/coreutils stat -
run: /Users/runner/work/coreutils/coreutils/target/aarch64-apple-darwin/debug/coreutils stat -L -
thread 'test_stat::test_stdin_pipe_fifo1' panicked at ''  File: -
  Size: 0         	Blocks: 0          IO Block: 512    weird file
Device: 4ef1bb9ch/1324465052d	Inode: 500453238   Links: 0
Access: (0660/?rw-rw----)  Uid: (  501/  runner)   Gid: (   20/   staff)
Access: 2024-10-25 15:48:53.191317902 +0000
Modify: 2024-10-25 15:48:53.191317902 +0000
Change: 2024-10-25 15:48:53.191317902 +0000
 Birth: 1970-01-01 00:00:00.000000000 +0000
' does not contain 'fifo'', tests/by-util/test_stat.rs:288:10

@cakebaker Both of your comments address the details of implementation I came to when working on it.
If you come up with better alternatives, I'll gladly hear them ^^

Up @cakebaker
Once this is done, I will move on and fix the tests I added :)

This PR might have been forgotten, just pinging  maintainers for final review/merge @cakebaker  @sylvestre ^^

thanks for the ping :)


CreateIndex: Move Display fmt to struct

Move the Display trait fmt implementation to the CreateIndex struct to to allow for display!
The struct was created in a0f511cb21efd9e992fb4c6ef273ef41802d308c with the CreateTable fmt being moved, but the CreateIndex was not.
Follows up from #1291

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

25 of 27 (92.59%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 88.882%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/dml.rs
24
26
92.31%

Files with Coverage Reduction
New Missed Lines
%

src/ast/mod.rs
1
81.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
26254

Relevant Lines:
29538

💛  - Coveralls


laravel 5 artisan tinker namespace issue

I have a class called SendEmail as a command bus. Following this example here
http://laravel.com/docs/5.0/queues#configuration
When I go into artisan tinker and I try to instantiate the SendEmail, it's not found.
Psy Shell v0.4.4 (PHP 5.4.36 — cli) by Justin Hileman
>>> use App\Commands;
=> false
>>> $email = new SendEmail();
PHP Fatal error:  Class 'SendEmail' not found in eval()'d code on line 1

pourquoi?

you need use App\Commands\SendEmail

fixed

yes, thanks bro..

you need use App\Commands\SendEmail

so, we also use this when make using create method in tinker? => 'use App\Commands\SendEmail::create([])' ?
bcs i got same error when using create mehod just using the class name.


User editable keys

Instead of adding different keys it would be better if they were user editable and we just save the state in localStorage or supabase. This will solve all other keyboard layouts as well.
Originally posted by @njm222 in https://github.com/pmndrs/racing-game/issues/165#issuecomment-873335801

I think localStorage is the way to go so we don't create unnecessary entries in supabase for users who play only once.
The controls should read localStorage > configControls or use default controls if none are present in localStorage


Document Relations

Can you please document the expected behaviour of relations which is referenced in many places, e.g. here?
I had naively assumed a DP reader would follow foreign keys and return the end of the relation in a nested object inline.
Thanks!

That should be detailed in https://github.com/frictionlessdata/specs which is what we are trying to implement. If that project doesn't have it, the issue should be reported there first

Closing as this is a documentation issue for the specifications, not the Java port.

FWIW I think this is a poor response. This is a Java API with public methods that have next to no documentation - linking to the relevant section of the official documentation would be appropriate.

Sorry to hear that but it is what it is.
The Java port for ages has had basically no contributors except for me and I am an unpaid volunteer. I tried to improve the quality of code and docs where I use the lib, but I don't need or use document relations. If you want to contribute better docs and/or code for this part, I am more than happy to accept pull requests.
If you want to work on it, I can re-open the issue.


Background image disappearing on zoom

Describe the bug
JPG background image of a child element of panzoom will disappear when zooming in and will not reappear when reloading the page .. it will only appear again when the page is hot reloads. It works perfectly in Mozilla but in chrome it disappears.
Your environment

Version of panzoom: 4.4.3
Browser and browser version: Chrome Version 95.0.4638.69 (Official Build) (64-bit)

Expected behavior
Background image should not disappear when zooming in
Actual behavior
Background image disappears when zooming in and will not reappear until the page hot reloads
Steps to reproduce
html
<!DOCTYPE html>

		<meta charset="UTF-8" />
		<meta http-equiv="X-UA-Compatible" content="IE=edge" />
		<meta name="viewport" content="width=device-width, initial-scale=1.0" />
		<title>Document</title>
		<link rel="stylesheet" href="style.css" />

css
#panzoom {
  width: 100vw;
  height: 100vh;
  background-color: #000000;

}

#app {

  background-image: url('../img/bg.jpg'); //any bg image as long as it is jpg
  background-position: center center;
  background-repeat: no-repeat;
  background-size: contain;
 width: 100vw;
  height: 100vh;

}

Thanks for opening an issue. It's hard to say exactly, but it sounds like there's a lot going on. It would be helpful if you put together a small working example on https://jsfiddle.net, https://jsbin.com,  or https://codepen.io. I can help debug, but I don't suspect this will require a change to Panzoom. Will reopen if I'm wrong.


Arnold Setup

The documentation instructs to use bin/build but it's not present.
How to setup Arnold ?

The documentation is not up to date with the latest Arnold release. From what I've tested the command has changed to:
bin/arnold bootstrap -c CUSTOMER -e ENVIRONMENT
I have also problems with setting up Arnold. because the kubernetes 12.0.0 API has a bug in it. It connects to host only on port 80 and it can t connect on local Openshift cluster on port 8443. A workaround is to downgradekubernetes to version 11.0.0 and openshift 0.11.2. I have tested locally with command ansilbe-playbook bootstrap.yml -e development -c arnold and it worked, but I am having problems running bin/arnold bootstrap -c CUSTOMER -e ENVIRONMENT.
Arnold is an awesome opensource project which started a revolution in online learning.:D. Could the developers write a new documentation for running a demo Open EDX with other services as Richie, Marsha, Ashley, Joanie on locally Openshift cluster :D.  I am willing to HELP

@aarajbhattarai we are sorry for the lack of / outdated documentation. I wrote a quick start guide in the following issue https://github.com/openfun/arnold/issues/662#issuecomment-842294879
Closing this as duplicate.


Fix some compiler warnings in sdks/java/core

This clears compiler warnings about SuppressFBWarnings for dependent libaries using the latest version:

warning: Cannot find annotation method 'value()' in type 'SuppressWarnings':
class file for edu.umd.cs.findbugs.annotations.SuppressWarnings not found

Manually fixed a bunch of ErrorProne compiler warnings in sdks/java/core.

The project is not yet ErrorProne-clean, but if we can get it there I recommend adding the -Werror compiler flag to treat warnings as errors.

Run Dataflow ValidatesRunner

Yes, please. My favorite compiler flag :-)

Java pre-commits are failing; seems that this change has somehow triggered the full wrath of ErrorProne: https://scans.gradle.com/s/whoprvzcdtcya/console-log?task=:beam-sdks-java-core:compileJava#L106
Not sure why these are being treated as failures, but let's see how much work it is to just fix the rest..

Ok, so I went and fixed all ErrorProne issues across SDK core such that we can enable -Werror. I'd love to see more of this across additional Java components.
@kennknowles PTAL

Wow, that's amazing! I never thought we'd get there.

retest this please

run java precommit

The failure seems unlikely to be related, but curious anyhow.

retest this please

retest this please

run python precommit

Run Python PreCommit

All green! @kennknowles PTAL

Great! Would you mind curating the commits now that it is g2g?


3scale backend plugin - Only first spec gets imported

Describe the bug
When you create a product in 3scale (ProductA) and you add a first spec (i.e., products > ProductA > ActiveDocs > create new spec) SpecA and you add a second spec (SpecB), both with an Open API endpoint definition, then, when imported by backstage, only the APIs mentioned in the first spec are available.
Expected Behavior
I would expect that both specs are available either via a concatenation in the backstage API list as '_' or otherwise as productName (i.e., as is) in the listing, but that you have both the specs available in the "definition" section
What are the steps to reproduce this bug?
see bug description
Versions of software used and environment
RH Developer Hub Version: 1.1.2
Backstage Version: 1.23.4
On OpenShift

We merged fix: https://github.com/backstage/community-plugins/pull/1249


S3 tomlparser temporary fix

Description

This PR adds a simple conditional branch to use TOML configuration with S3 Client. Two files are changed:
I modify luigi/contrib/s3.py.
I add examples/foo_s3.py to check the validity of my modification.
Motivation and Context

It is pointed out that S3 Client doesn't work with TOML configuration #2698 .
Fixing the issue would help a lot of users because TOML configuration and S3 Client are useful.
Now I propose to fix the issue by a simple modification.
Have you tested this? If so, how?

I do not add any unit tests because there is not test of S3Client._get_s3_config method.
Instead of unit tests, I add an example file (examples/foo_s3.py) to check the validity of my modification.
I confirm that my modification worked with S3 Client.
Please note that I have not confirmed that my modification does not cause other unknown errors.
I would be grateful that someone could review this PR.

Instead of #2767 , I propose to fix S3Client.
Although this PR might not be the best solution as @orsinium said, it would be a better solution because we can avoid the parsing error temporarily.
Details of tests
********:luigi ************$ LUIGI_CONFIG_PARSER=toml LUIGI_CONFIG_PATH=examples/config.toml PYTHONPATH=. python3 -m luigi --module examples.foo_s3 examples.Foo --local-scheduler
INFO     2019-08-21 15:56:19 setup_logging:86 logging configured via config section
INFO     2019-08-21 15:56:23 worker:595 Informed scheduler that task   examples.Foo__99914b932b   has status   PENDING
INFO     2019-08-21 15:56:23 interface:172 Done scheduling tasks
INFO     2019-08-21 15:56:23 worker:1182 Running Worker with 1 processes
INFO     2019-08-21 15:56:23 worker:165 [pid 61882] Worker Worker(salt=792552536, workers=1, host=********.local, username=************, pid=61882) running   examples.Foo()
Running Foo that outputs S3Target
INFO     2019-08-21 15:56:23 worker:217 [pid 61882] Worker Worker(salt=792552536, workers=1, host=********.local, username=************, pid=61882) done      examples.Foo()
INFO     2019-08-21 15:56:23 worker:595 Informed scheduler that task   examples.Foo__99914b932b   has status   DONE
INFO     2019-08-21 15:56:23 worker:493 Worker Worker(salt=792552536, workers=1, host=********.local, username=************, pid=61882) was stopped. Shutting down Keep-Alive thread
INFO     2019-08-21 15:56:23 interface:175
===== Luigi Execution Summary =====

Scheduled 1 tasks of which:
* 1 ran successfully:
    - 1 examples.Foo()

This progress looks :) because there were no failed tasks or missing dependencies

===== Luigi Execution Summary =====

@dlstadther
Could you tell me an example where _get_s3_config method does not return an empty dict?
I'm sorry that I cannot follow the context of this method completely.
Then I'll try to support a default section in TOML parser.

Let's suppose you have the following in your INI cfg:
[DEFAULT]
home_path: /opt/luigi
env: production

[s3]
aws_access_key_id: <your-key>
aws_secret_access_key: <your-secret>
calling_format: boto.s3.connection.OrdinaryCallingFormat
# ^ this was needed back before luigi upgraded to boto3, but the idea that additional boto3 kwargs could be supplied still stands

s3.S3Client()'s .__init__() will set options = self._get_s3_config() and later supply any kwarg which weren't used for authentication to boto3.resource(..., **options)
So prior to the "removing default section vars from s3 section", boto3.resource would blow up when it received an unknown kwarg home_path (going back to the example ini above).
Here was my original PR which added this functionality.

I hope that provides a more clear explanation for why that code exists.


Unauthorized in Page Deletion

When I click the "X" to delete a page from the page list the page simply refreshes and I get an "Unathorized" remark at the top of the page (see attachment).
"Delete" from the actual page works fine though.

This is a missing nonce security thing I think.  We got most of them, but clearly a couple slipped through

BTW this is an admin bug.. can you create the same issue there????

No sure what you mean, I am using the Admin plugin for all my GRAV sites.

I'm saying you created this issue in the "Grav" repo.
https://github.com/getgrav/grav/issues
The issue should be in the "Grav Admin Plugin" repo:
https://github.com/getgrav/grav-plugin-admin/issues

DOH! ...  yes.

Anyway thanks for reporting, fixed in https://github.com/getgrav/grav-plugin-admin/commit/05bfe20c330e3f3a1edf6138dd79ec64488ae4b1, will get in the next update!


Wax pr6 updating wax.cartodb.js - IE, gmaps and mobile issues

Updating wax.cartodb.js
This commit fixes the following issues:

#346 : Dragging the map triggers the info window
#313 : Pan/Zoom event don’t work as appropriate on touch-screen
Windows 8
#223 : Pointer events in IE11 on touch-screen Windows laptops do not
trigger popups
#139: Layer interaciton in IE10/IE11 not working when dragging a
leaflet marker

Note: The PR 334 is included here because I tested all the fixes together (because of IE 11 mobile case)

@javisantana , I've been testing locally the changes with several browsers and devices regarding to your comment here

So, should I merge and deploy this cartodb.js version?

yes, good work :es:


Problem WiFi Setup

I am an old codger from Australia. I am familiar with Arduino, less so with ESP8266. I make the odd Eagle PCB.
I have a few Arduino Mini's, although they don't look identical to your Pro Mini. I also use a MacOSX Big Sur for development.
I have a couple of Adafrruit Huzzah Feather boards. I am interested in the project primarily because I have a large MHI AirCon which is driving me mad, and it does seem to have a lot in common with the environment described.
The documentation provided is terrific, and I have made a lot of progress, but have needed to change things a bit to suit the bits I have.
Firstly Software:  The ESP JSON 6 code needed changing to compile - basically to use JsonDocument class. The changes seem to be a bit simpler than the original, due to Json improvements. I also needed to put a Silabs Driver on the Mac to upload to the Huzzah.
Next Hardware. I had to rework the 2 boards. The Huzzah includes a USB connector, and my Mini is loaded through a FTDI connection. To get access to each board for changing software, and for the connector to the MHI I needed a bit different arrangement. I will send a photo if it might be of general interest. I haven't wired the MHI bit yet, as we are in Covid lockdown.
Anyway, on to the problem. Power to the Huzzah and the Mini is via the USB connector on the Huzzah. Mini LED ON.
I search for a Network, and find one which is obviously the ESP board. But its SSID is ESP_C109E7 and unpassworded.
I don't know how to relate it to my home Wifi SSID/PSWD. I also changed the code temporarily to grab this network.
I do get a nice Wifi Manager screen Config Info Update etc etc, but am unsure on what I should be setting where.
When I upload code into the Huzzah via the SiLabs driver, I don't seem to need any Burnloader activity from the Arduino IDE.
Any comments welcome. Sorry this is a bit preliminary.
Peter Hanlon

Further to the last post, a photo of my setup. I put a 104 cap between the DTR pin on the FTDI line and the R reset pin on the Arduiano Mini. I couldn't get a voltage regulator Pololu but hard-wired an L7805 with a couple of caps. The 5V supply comes from either USB, FTDI, or L7805 Vout and is fed to both boards. Similarly GND. The two jumpers - one side from the Huzzah, and one side from Arduino on both the TX and RX lines. So removing them lets software in either to be uploaded. Underneath it looks like a birds nest or dogs breakfast.


⚠ Jenkins Plugins Artifactory has degraded performance

In 97e6564, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1211 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 55f466a after 38 minutes.


Use the futures crate instead of multiple sub crates.

Signed-off-by: Valdemar Erk<EMAIL_ADDRESS>
This PR makes it so we uses the futures crate in the whole repo
instead of using the sub crates futures_* this is to better ensure
compatability between crates and make it easier to update.
The real price to users will likely not be any problem as most will
have the futures crate somewhere in their dependency tree anyway.

@Arzte Then we should do it by changing wich features of futures we use.

Also I have thought a bit more about and we may go back on this later, when we are getting close to some kind of release


adding heartbeat feature

Closes #73

[ ] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

@wjcunningham7 do you plan to enhance covalent status in this PR?

@wjcunningham7 do you plan to enhance covalent status in this PR?

No, this is simply to provide the heartbeat, we can later decide what to do with it.  That's partly a UX question I think.

This test failure is the same reason why functional tests are failing on develop.


feat: Add Team Sync Group Mapping

This resource allows you to create and manage Identity Provider (IdP) group connections within your GitHub teams.

@sachinmalanki Thanks!


How to fix "Failed to get remote module base address"

Can't get the program work, please help.

stop injecting other dlls

@Sheathe007 - Idk if you already solved this, but I encountered it today, too. Did a bunch of uninstalling, reinstalling of both this tool and Genshin itself and no luck.
Turns out, the problem was Roblox lol. I had it open and the moment I closed it the game launched with no issue. I assumed the games' anti-cheat platforms don't play well with each other.

use always the latest version to many times that people opens again and again an old version.
Turn off every game or anti cheat while using the unlocker.
DONE.

I also encountered this, typically quit the game and then restart it will solve the problem. Not a big deal, but kind of annoying.

I've just started getting this problem. I'm not using any other DLLs nor do I have any other game anticheat running to my knowledge. I cannot get >60fps no matter what I do and get the 'Failed to get remote module base address' error shortly after launching Genshin. I'm on Windows 11 using fpsunlocker v3.3.0 on Genshin Impact 4.8.0

I think a windows security update may have botched things for some people. I'm partway through building my own launcher but for testing I use an older modified version of this to load DLLs. that part works with few modifications. I still get the remote address error but it doesnt matter because it boots. for fps unlock I use genshin utility

Download "unlockfps_nc_signed.exe" instead of the non signed version, put it in your Download folder and launch it. That's the only way I could unlock the fps limit without having that "Failed to get remote module base address" error.

Download "unlockfps_nc_signed.exe" instead of the non signed version, put it in your Download folder and launch it. That's the only way I could unlock the fps limit without having that "Failed to get remote module base address" error.

this must be an issue with your windows configuration. Disabling secure boot and/or defender features/exploit protection would probably resolve it

Download "unlockfps_nc_signed.exe" instead of the non signed version, put it in your Download folder and launch it. That's the only way I could unlock the fps limit without having that "Failed to get remote module base address" error.

this must be an issue with your windows configuration. Disabling secure boot and/or defender features/exploit protection would probably resolve it

Secure Boot was already disabled, and Defender has never seen the unlocker as a threat, maybe something went wrong while I updated Genshin with the new hoyo launcher.


Replace GAE SDK with GCLOUD SDK

Browsers and versions affected
Description
Steps to reproduce
Expected results
Actual results

Fixed


Mousewheel zoom on the stage

The IAEA Interactive Chart of Nuclides affords the user standard scroll-to-zoom/drag-to-move capabilities. The nuclides layout should do this too, probably by way of a clip path for the stage.

The concept of layouts has given way to the concept of questions, but this is still a viable enhancement. To abstract it perhaps any given question could be "zoomable" with captions and specifics being defined with respect to a zoom level.


Update Dataset Creation Form For Dynamic Fields

The dataset management GUI view (still under review #197) has a component view and form for creating new datasets. At the moment, the dynamic field for an upload input is partially broke (although that may need to be dynamic). This needs to be corrected.
It also does not yet have anything for defining the DataDomain of the dataset. There should be some dynamic fields for setting these during creation. They should depend on the DataFormat selection, as this will determine the indices applicable within the domain.

@robertbartel, looking at the source for createDatasetForm.js, I see that NETCDF_FORCING_CANONICAL should have an upload input option, what other data formats, if any, should have an upload input option?

I think that particular input needs to be changed to apply to all the formats.  It may be worth having a checkbox or something to toggle whether it is shown.  The one exception could be OUTPUT, but I think I am going to need to change that to be something more specific, and even then we may want it possible to upload already created results.
The things that really should be more dynamic are inputs to control the continuous and discrete restriction values; e.g., time range, the set of catchments, etc.  Exactly what those is specific to each format.

The one exception could be OUTPUT, but I think I am going to need to change that to be something more specific, and even then we may want it possible to upload already created results.

Thanks for clarifying. I am leaning towards leaving upload open to all data formats at the moment. My thinking is one person's output is or could be another person's input. It might make more sense for me to leave output off for now and just leave a simple way to change this in the future in the form of comments when we fully support such a feature.

The things that really should be more dynamic are inputs to control the continuous and discrete restriction values; e.g., time range, the set of catchments, etc. Exactly what those is specific to each format.

I am not seeing those fields currently. Should I be?

I think dynamic field removal issue is because of how we are removing children elements of #dataset-create-form-div-dynamic-inputs:  dy_div.childNodes.forEach(child => child.remove());. forEach iterates and mutates that underlying vector, so when the first child is removed that is causing issues. Changing the code in createDatasetForm.js to the following should fix this issue:
// remove all dynamic input child elements
while (dy_div.firstChild) {
  dy_div.removeChild(dy_div.firstChild);
}

My thinking is one person's output is or could be another person's input.

Exactly.   This is why I am strongly considering removing the category OUTPUT, likely replacing it with several new categories (e.g., NGEN_RESULTS, etc.).  It becomes even more necessary if we have MaaS Job subtype(s) for deriving a forcing dataset from other existing forcing data.

I am not seeing those fields currently. Should I be?

No, not yet.  I got stuck at getting the dynamic logic working at all for something, and didn't start the others.  Feel free to set those up however seems most appropriate.

@robertbartel, I think I am moving in the right direction, I still need to iron out the details of handling datetime objects, but before I move ahead below is what I have working at the moment. Ill get this into a PR this morning, but for discussion sake thought it made sense to add it here first.

First, there doesn't need to be an input for discrete versus continuous. The indices for which we want to input values are themselves intrinsically one or the other. This should inform the type of input component that gets used.
Second, I don't think those are the indices for which we want values input. Values within the dataset, perhaps. But not domain defining properties. I.e., the user won't need it determined whether datasets they need are available by looking at the range of rain rate values. The times and catchments for which data is available are more likely the things to check for something like forcings. But those things are explicitly part of each DataFormat enum value.

Okay, that makes sense. And sorry for what in retrospect are pretty naive assumptions, I thought it was a little odd to include a toggle for discrete vs continuous given we typically know the difference a priori. In reading the commented out sections of dataset_management.html in your e21/dataset_mgr_gui/main, it wasn't clear to me if there should or shouldn't be a toggle between discrete and continuous.
Okay, thanks for clarifying regarding the input values. I think I was thrown off by the use of parameter in dataset_management.html. I took that to mean a DataFormat's data_fields. Admittedly, I thought it was a little odd to get min max values for each data field, but alas here we are.
Luckily these shouldn't be huge changes and I think we are on the same page and moving in the right direction now.

Now closed by #240.

The original associated PR for this was closed without merging, with (some, but not all of the) various pieces of work separated into individual PRs.  As such, this needs to be moved back to another status.
For now, I am moving to Requested, since it is realistic we are going to re-implement the entire dataset management GUI view.


Rango for Safari

Wrap Rango as a Safari web extension. Implements clipboard support in the native app.
Update CONTRIBUTING.md with Safari instructions.
Here's the app's main window:

I did some file reorganization and renaming. Now clipboard.ts is in src/background/utils/clipboard.ts. What's the best way to solve the conflict, you do a rebase?

I was trying to resolve the merge conflicts myself and I realized that you have to request "nativeMessaging" permission in Safari. I think we are going to need to create a different manifest file for Safari, because requesting permissions that you're not using could be problematic

I was trying to resolve the merge conflicts myself and I realized that you have to request "nativeMessaging" permission in Safari. I think we are going to need to create a different manifest file for Safari, because requesting permissions that you're not using could be problematic

OK, whatever works, just let me know if you want me to do anything. Should have some time this weekend again.

I'll create another manifest and commands for building to dist-safari. I just need to know if there's anything else that needs to be changed from your code to reflect this change

I'll create another manifest and commands for building to dist-safari. I just need to know if there's anything else that needs to be changed from your code to reflect this change

Would just need to change the Xcode project file to point at the new manifest (and build folder if appropriate). Should be a 5 min change once you have the new manifest/build command done.

FYI, I am holding off on revising this PR until you let me know you're ready!

Yes, sorry. I'll do it this week so you can do your part on the weekend

I created the separate manifest and build scripts for Safari.
Btw, I bought an air m1 because the Linux machine I was using was very slow and was getting on my nerves. So next time I can try to package it myself for Safari

I've updated (finally) to the new manifest and undid my changes from the old one. May want to add a debugging guide, but here's some things to be aware of:

Don't ever have more than one copy of Rango.app on your system or results may be unpredictable (use Product > Clean Build Folder in Xcode if needed).
Some things that can help if Rango doesn't appear in Safari: uncheck/check in Safari Extensions Preferences; restart Safari.
Messages logged by the extension do not appear in Xcode but are logged to the system log (stream with /usr/bin/log stream --predicate 'subsystem == "net.sabi.Rango.extension"' --info).


ci: skip more tasks if only markdown files changed

No longer just skips the unit test and e2e tasks if only markdown files changed. Now everything will be skipped, since there is nothing that tests against the .md files.
Prints messages for the deploy tasks that run inside of Pull Requests but end up in a NOOP.

Shouldn't this PR be skipping unit tests? Looks like they're still running on it

Every task will run in this PR because .js files have been changed here.

Duh. My brain is not functioning today.


The program has an error message: ValueError: high <= 0

Very great work! It is a pleasure to communicate with you.
I got the following error when I used make_spect_f0.py and make_metadata.py to train the model after processing my own dataset and then using main.py. Have you encountered it and how can I fix it?
Traceback (most recent call last):
  File "/home/ubuntu/work/vc/solver.py", line 135, in train
    x_real_org, _, f0_org, len_org = next(data_iter)
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 521, in __next__
    data = self._next_data()
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 561, in _next_data
    data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
  File "/home/ubuntu/.local/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 52, in fetch
    return self.collate_fn(data)
  File "/home/ubuntu/work/vc/data_loader.py", line 100, in __call__
    left = np.random.randint(0, len(aa) - len_crop[0], size=2)
  File "mtrand.pyx", line 748, in numpy.random.mtrand.RandomState.randint
  File "_bounded_integers.pyx", line 1247, in numpy.random._bounded_integers._rand_int64
ValueError: high <= 0

high <= 0 means len(aa) - len_crop[0] <=0, which probably means aa is too short

Thank you very much for your answer, I have successfully solved the problem


Feature provider

Look good to me.
src/scripts/app.js : 106 : authenticate() seems to be added at commit: 6330393c391f272efe01c4ea0e7c987086107937  and is a leftover that can be removed.

goob jod


Saleor storefront stuck on loading with 3 dots

What I'm trying to achieve
Set up a local test version of Saleor. Followed steps to set up API and then Storefront. Things seem okay but when I go to http://localhost:3000/ the page is stuck with the 3 loading dots in the middle. Login seems to be working, but no dashboard is available.
Separately, I think the docs are out of date. They still reference build-assets and don't include a mention of the separate storefront.
Steps to reproduce the problem

Install Saleor API
Install Saleor storefront
Load http://localhost:3000/

System information
Operating system: Windows 10

Should be fixed by https://github.com/mirumee/saleor-storefront/pull/551


Fixes #4162

The ChemicalReaction copy constructor and the ReactionPickler _depickler  create ROMols, while constructMolFromString creates RWMols which are then cast to ROMols. This is why the dynamic_cast performed by RxnOps::sanitizeRxn failed with the copy constructor and the depickler, while it succeeded on the reaction created directly from SMARTS.
This PR sets the copy constructor and the depickler to create RWMols and adds relevant C++ and Python tests.

This is a good catch, I think most of the code assumes that ROMols are RWMols in disguise, we should probably never allow the constructor on the ROMol to be called.  I wonder what his would break :)?

I also find it confusing trying to understand when it is safe to cast from an RWMol to an ROMol.  I wrote an email about it recently. It was a bit vague and didn't get any replies, perhaps a github discussion would have been better.
So an object that is created as a RWMol and then cast to an ROMol can be dynamically cast back to an RWMol, but an object that starts life as an ROMol will fail that dynamic cast, is that correct?  Do you know how that works?
And what about static casting from an ROMol to an RWMol, will that ever fail?  The python wrapper for sanitizeMol does that, and makes no check on the validity of the RWMol before sanitizing it.

@jasondbiggs that's because of the assumption that you need an RWMol to make a ROMol which is true in all (most?) cases except pickling apparently.

pinging @greglandrum to loop him in

The intention is that an RWMol and an ROMol are exactly the same except that an RWMol has the extra public methods required to change the molecule graph (i.e. add/remove/replace atoms/bonds).
It should always be safe to cast an RWMol to an ROMol.
Casting an ROMol to an RWMol and then editing the molecule is more questionable since it is semantically equivalent to casting away const-ness - the calling code could theoretically be relying upon the fact that the graph doesn't change.
However sometimes, hopefully really rarely, you need to hold you nose and do the ROMol -> RWMol cast, just like sometimes you need to do cast away constness. This should theoretically be safe except for the fact that somewhere very early in the development process the RWMol picked up a data member - d_partialBonds. This makes it technically unsafe to make the ROMol -> RWMol cast (and will lead to justified warnings/errors on some compiler settings. The partial bond mechanism is only used in the SMILES parser, which has a real RWMol to work with, so it shouldn't really be a problem as long as you don't end up calling the delete() or operator=() method on one of these fake RWMols.
So in general: casting an ROMol to an RWMol is something you probably shouldn't do and, if you do, you should be really careful with what happens with the resulting RWMol.
I wouldn't be terribly surprised if there were some odd lurking Heisenbugs in the code which result from this stuff.

FWIW: we could fix the evil hidden possible memory corruption problem without breaking any API by either moving the partialBond data member to ROMol or (better) modifying the code to use the property mechanism instead of a dedicated data member.

Another possible option is to have the ROMol constructors private so you HAVE to use an RWMol to make an ROMol.

@greglandrum I believe what you are saying is:
static_cast<RWMol>(romol)

should always be safe and:
dynamic_cast<RWMol(romol)

may fail.


Add mhrama file

Created a branch, created a file and made a commit, and opened a pull request

Nice work!


Job cattle-node-cleanup fails due to restrictive standard PSP

Rancher Server Setup

Rancher version: 2.6.3

Information about the Cluster

Kubernetes version: v1.21.7
Cluster Type: Downstream
Custom cluster / Provider: RKE1

Describe the bug
There is the periodic event in the cluster event log:
FailedCreate 	Job cattle-node-cleanup-xxx Error creating: pods "cattle-node-cleanup-xxx-" is forbidden: PodSecurityPolicy: unable to admit pod: [spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used]
To Reproduce

Remove a node from a downstream cluster
Watch the job cattle-node-cleanup-xxx
--> Recent events:
Error creating: pods "cattle-node-cleanup-xxx" is forbidden: PodSecurityPolicy: unable to admit pod: [spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used]

Result
Error creating: pods "cattle-node-cleanup-xxx" is forbidden: PodSecurityPolicy: unable to admit pod: [spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used]
Expected Result

Job cattle-node-cleanup-xxx should run without errors
The job should not run in the default namespace, because it violates the CIS policy

Additional context
I use a more restrictive standard PSP in the cluster which prohibits the usage of a hostPath volume.
The YAML file of the job "cattle-node-cleanup-8nx54" shows, that no specific service account is used. So the default service account is used which is bound to my default PSP for the cluster.
The job should use a specific service account to which any unrestricted PSP is bound.

Addition: And the job should not be performed in the Default namespace, because this is against the CIS scan rules.

Workaround: Remove the failing job. Then the periodic event in the cluster log vanishes. If the node is removed anyway, it possibly doesn't matter, if the cleaning job is not performed.

Workaround: Remove the failing job. Then the periodic event in the cluster log vanishes. If the node is removed anyway, it possibly doesn't matter, if the cleaning job is not performed.

I am not sure if this is a valid workaround as the cleanup might not only delete the node object but also other things?

Workaround: Remove the failing job. Then the periodic event in the cluster log vanishes. If the node is removed anyway, it possibly doesn't matter, if the cleaning job is not performed.

I am not sure if this is a valid workaround as the cleanup might not only delete the node object but also other things?

Yes, but if the underlying server (at cloud level) is removed anyway, then it should not matter. But of course not quite sure...

Do we know if the cleanup only deletes stuff on the "local to the deleted server"? Probably there are also some other k8s objects that might need to be deleted?
I believe for the workaround we should find out which serviceaccount is used in the job and then create a role/rolebinding that would allow usage of system-unrestricted-psp for the used serviceaccounts.
I.e.
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
  name: cattle-cleanup-role
  namespace: default
rules:
- apiGroups:
  - policy
  resourceNames:
  - system-unrestricted-psp
  resources:
  - podsecuritypolicies
  verbs:
  - use
---
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: cattle-cleanup-rolebinding
  namespace: default
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: cattle-cleanup-role
subjects:
- kind: ServiceAccount
  name: cattle-cleanup-c22lc
  namespace: default
- kind: ServiceAccount
  name: cattle-cleanup-f5v84
  namespace: default

Do we know if the cleanup only deletes stuff on the "local to the deleted server"? Probably there are also some other k8s objects that might need to be deleted?

Unfortunately not

I believe for the workaround we should find out which serviceaccount is used in the job and then create a role/rolebinding that would allow usage of system-unrestricted-psp for the used serviceaccounts.

The job currently uses the default service account. Therefore my strict PSP applies. And it's deployed in the default namespace.
And yes, the creation of appropriate SA, Role, Rolebinding would be the right way forward. But I wouldn't know where the manifests are stored or which instance triggers the creation of the job.
And the namespace should changed to cattle-system

I have no idea how to fix this in general (I am not a developer  ;-)) - but the job should run in case the role/roleassignment would be done with the above yaml (adjusted to default service account) and then delete the role/rolebinding after the job completed.

Only Rancher can and should fix this. To be honest, it's a pitty, that the vendor doesn't consider such basic things like service accounts and namespaces for such cleaning jobs.
But thanks for your proposal. It seems to be a cumbersome but valid workaround. I will document it and apply it the next time...

Only Rancher can and should fix this. To be honest, it's a pitty, that the vendor doesn't consider such basic things like service accounts and namespaces for such cleaning jobs. But thanks for your proposal. It seems to be a cumbersome but valid workaround. I will document it and apply it the next time...

True - a real fix should be implemented in Rancher.
Could you create a Support Case so that this gets tracked properly?

Sorry, I am not under maintenance. Therefore I created this Github issue...

This issue is still relevant and not solved at all. Please leave it open...

@rosskirkpat has requested some additional information from the customer via the internal support issue.
@luthermonson states All our cleanup jobs across all of rancher (cluster/node) are in the default namespace as it's the only thing guaranteed to be there. This was a design decision made quite some time ago.

The default namespace is not the problem. The problem is, that the job doesn't start due to my restricted PSP. The creation and assignment of a SA with appropriate permissions should be sufficient.

With the recent team changes, I'd like this re-triaged and re-prioritized in the team to see when and who should pick it up

Moving to team 1 as the root cause is related to the PSP assigned to the cleanup job.

Can we keep this targeted for 2.6? We have a lot of clients hitting this and it will be a while before they can update to 2.7.x
Thanks!

This doesn't appear to deal at all with the Rancher object/chart itself, or its PSPs. This is the cattle-node-cleanup pod, which needs a service-account+PSP attached to it in order to function in a restricted environment: https://github.com/rancher/rancher/blob/release/v2.7/pkg/controllers/management/node/cleanup.go#L279-L280
Or as mentioned above it could run in the system namespace, which is probably more appropriate. I don't understand why the cattle system namespaces wouldn't exist, even in rke1 standalone ?

/backport 2023-Q2-v2.6x

@pwurbs I am working on reproducing this issue. Could you please share the yaml file for your restricted standard PSP? Or at the structure with redacted data? Thank you

@annablender
Here it is:
spec:
  allowPrivilegeEscalation: false
  defaultAddCapabilities:
    - CHOWN
  defaultAllowPrivilegeEscalation: false
  fsGroup:
    rule: RunAsAny
  runAsGroup:
    rule: RunAsAny
  runAsUser:
    rule: RunAsAny
  seLinux:
    rule: RunAsAny
  supplementalGroups:
    rule: RunAsAny
  volumes:
    - emptyDir
    - gitRepo
    - secret
    - persistentVolumeClaim
    - downwardAPI
    - configMap
    - projected

@pwurbs I was unable to reproduce this issue on 2.6.3.
Repro steps

Run rancher instance v2.6.3
Create rke1 downstream cluster 1 cp,etcd 2 worker nodes with PSP enabled and set to restricted-test using the config file above
Remove a downstream worker node
Node is removed immediately. Cattle-node-cleanup-xxx does not get stuck and the node is removed too fast for me to see any pod logs

Are you still seeing this issue?

@annablender You are right. The issue must have been solved by one of the latest updates. In 2.6.10 (and maybe former versions) I am not facing this issue.
But apart from that, would you like to check, in which namespace the cleanup pod runs now? It was my main concern that it runs in default namespace, which was also against the CIS rules.

@pwurbs According to @luthermonson https://github.com/rancher/rancher/issues/36006#issuecomment-1095244459 all cleanup jobs including cattle-node-cleanup-xxx run in the default namespace. They are guaranteed to be there. I know there's a Rancher 2.6 and 2.7 hardening guide for CIS but could you link me the docs stating jobs/cleanup jobs should run in a non-default namespace?

@annablender Here is the link: https://ranchermanager.docs.rancher.com/v2.6/reference-guides/rancher-security/rancher-v2.6-hardening-guides/rke1-self-assessment-guide-with-cis-v1.6-benchmark#574-the-default-namespace-should-not-be-used-automated

@pwurbs Thank you! I will discuss this with my team. Otherwise, QA can test this using repro steps https://github.com/rancher/rancher/issues/36006#issuecomment-1431649029.

@pwurbs Thanks for the link. From the discussion, the cattle-node-cleanup-xxx jobs run in the default ns because that is a ns that is most accessible to rancher, and will not be removed during the cleanup process. Rancher cleans up the cattle-system ns and others on custom clusters and it would not complete the process successfully if rancher removed the ns the cleanup job was in.
I also will try again to reproduce this on a custom cluster, not a node provisioned one since the cleanup jobs only run on the former. That's my mistake. This is still in review.

@sowmyav27 This is ready to test.
I further investigated this issue and was able to reproduce it on rancher 2.6.2 but was unable to reproduce it on rancher 2.6.10. As per this comment https://github.com/rancher/rancher/issues/36006#issuecomment-1434247590, I believe this was fixed somewhere in between those versions.
Please have QA verify this is fixed on the latest rancher release v2.6.11.
Testing template
Root cause
What was fixed, or what changes have occurred
Areas or cases that should be tested
Test steps

Run rancher instance latest v2.6.11
Create an rke1 custom cluster with PSP restricted enabled, 3 nodes (1 cp/etcd, 2 worker)
Cordon, drain, and then delete a worker node from k8s

What you are looking for

cattle-node-cleanup-xxx job to be successfully deployed on the node without a HostPath error in the job yaml

kubectl get jobs --kubeconfig
NAME                        COMPLETIONS   DURATION   AGE
cattle-node-cleanup-mjqf2   1/1                      15h
cattle-node-cleanup-ztfk6   1/1                       17m

node removed successfully from the rancher UI

Repeat with second worker node. Verify same behavior.

What areas could experience regressions ?
RKE1 prov
Are the repro steps accurate/minimal ?
Yes.

Validated the issue on rancher v2.7.5 with k8s version v1.23 with PSP enabled and set to restricted. Issue is not active anymore.
Validations steps.
Test steps

Created a rancher v2.7.5 (this version still has PSP enabled.)
Created an rke1 custom cluster with PSP enabled and set to restricted, 3 nodes (1 cp/etcd, 2 worker)
Cordon, drain, and then delete a worker node from rancher.
cattle-node-cleanup-xxx job was deployed successfully on the cluster.
Node deletion was successful.

Issue was not encountered, hence closing the issue.


Updated ISO-urls

I've updated the 8.4 ISO url to use the archive-path in the official Debian-CD repository.
Everything prior to 8.4 is only available as jigdo in the official Debian-CD repository. I've found a taiwan mirror, that (still?) holds the ISOs, but I've only updated 8.2 and 8.3.

Thanks, but we're going to go with the jigdo option.


Installing it with yarn leads to a crash on it's initial run

Happens on the first run.
TypeError: program.prompt is not a function
    at Object.ask (/Users/furkantunali/.config/yarn/global/node_modules/cmd-jira/lib/auth.js:48:25)
    at Object.setConfig (/Users/furkantunali/.config/yarn/global/node_modules/cmd-jira/lib/auth.js:68:22)
    at Command.<anonymous> (/Users/furkantunali/.config/yarn/global/node_modules/cmd-jira/bin/jira.js:48:18)
    at Command.listener (/Users/furkantunali/.config/yarn/global/node_modules/commander/index.js:315:8)
    at Command.emit (events.js:182:13)
    at Command.parseArgs (/Users/furkantunali/.config/yarn/global/node_modules/commander/index.js:651:12)
    at Command.parse (/Users/furkantunali/.config/yarn/global/node_modules/commander/index.js:474:21)
    at /Users/furkantunali/.config/yarn/global/node_modules/cmd-jira/bin/jira.js:404:13
    at Object.execCb (/Users/furkantunali/.config/yarn/global/node_modules/requirejs/bin/r.js:1941:33)
    at Module.check (/Users/furkantunali/.config/yarn/global/node_modules/requirejs/bin/r.js:1116:51)```

Apparently, it's working fine when installed with npm but yarn

I haven't tried installing it with yarn, did you find anything useful?

@JacopKane still facing the issue?
If you are still facing the issue please feel free to reopen the issue, and we'll solve it together : ) .

@HeadFox did you get a chance to look into it?

Hey !! Sorry I was busy these days !
I tried again and it seems to work now ! Maybe the problem came from yarn itself
@palashkulsh

no problem @HeadFox , glad it works now : )


XmppWebSocketIo.write

Specify the sample to which the issue belongs (use [x]):
[x] Chat sample
[] P2P Calls sample
[] Conference Calls sample
Platform (use [x])
[] Android
[x] iOS
[] macOS
[] Windows
[] Web
[] Linux
Device info
Manufacture: Apple
Model:iPhone 12
OS version: iOS 16.6.0
Describe the bug:
When App is in background for long time, and come back to it again after few hours. These are the errors thrown at console. Although internet was stable but it still throws these errors. Logs screenshot attached.
Logs:
Fatal Exception: FlutterError
SocketException: Write failed (OS Error: Broken pipe, errno = 32), address = chat.connectycube.com, port = 61525
0
dart:io
_Socket.write
1
XmppWebsocketIo.dart - Line 47
XmppWebSocketIo.write + 47
2
Connection.dart - Line 357
Connection.write + 357
3
Connection.dart - Line 372
Connection.writeNonza + 372
4
StreamManagmentModule.dart - Line 61
StreamManagementModule.sendAckRequest + 61
5
StreamManagmentModule.dart - Line 195
StreamManagementModule.handleResumed.<fn> + 195

it means that the socket connection was broken and you should relogin into the chat again

We already doing it, when there is inactivity of 30 mints we are relogin him again. Can you tell me what is the exact we have to relogin?

when there is inactivity of 30 mints we are relogin him again

how do you do it? the iOS system kills the app in the background about after 10 minutes. do you try to relogin every 30 minutes?

what is the exact we have to relogin

I see it in a next way - after backing the app from the background (or resume) you should check if the connection state is CubeChatConnectionState.Ready and if not try to relogin into the chat. Do you do something with the chat connection when the app moves to the background?

no activity for a long time. closing...


Errors Adding Dependency

My apologies if this is not on you, but I tried adding the dependency for map_view and it's erring out when I attempt to install the packages.
Package uri has no versions that match >=0.11.1 <0.12.0 derived from:
- map_view 0.0.10 depends on version ^0.11.1
pub get failed (1)

dependencies:
  flutter:
    sdk: flutter
  fluro: ^1.2.1
  http: ^0.11.3
  scoped_model: ^0.2.0
  timeago: ^1.2.1
  map_view: 0.0.10

I've got the same problem. I followed the instructions from here with and without ''.
Running "flutter packages get" in opendata...
Package uri has no versions that match >=0.11.1 <0.12.0 derived from:
- map_view 0.0.10 depends on version ^0.11.1
pub get failed (1)
Process finished with exit code 1

This is a brand new project in IntelliJ with the following dependencies:
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^0.1.0
  map_view: ^0.0.10

I guess a solution would be to clone this repo and import the file that way. Would be nice to resolve this as it will frustrate newcomers.

@kelvinharron yes I'm already using it, but I was hoping to use the static google maps stuff that this one provides.

@jpgilchrist I'm trying to use it but unable to do so. Same installation error. Any idea how to install it.


Register xoxwawa.is-cool.dev

Added xoxwawa.is-cool.dev using the CLI.

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

1 out of 2 committers have signed the CLA.:white_check_mark: @DEV-DIBSTER:x: @arby89You can retrigger this bot by commenting recheck in this Pull Request

Please sign the CLA agreement.


How to press / tap on an ActionSheetIOS item

How to press / tap on an ActionSheetIOS item,
the only way I can think of it to get the item by text, but that does not work ...
Note that I'm able to assert that the item is visible by text, but tapping it produces an error
Test Failed: Failed expectation: TOBEVISIBLE WITH MATCHER(identifier: == “singInScreen.screen”), Another view “<RCTView: 0x7fc96cd65520>” is visible at window point “{"x":207,"y":448.71497584541066}”     TIP: To print view hierarchy on failed actions/matches, use loglevel verbose and above.

this could be a bug with detox

It doesn't look like a bug in Detox. Your visibility assertion is failing, meaning your item is not visible.

@LeoNatan the visibility assertion is passing
getting element by label && then selecting the 0th index solves the issue for me
await getByLabel("Sign Out")().atIndex(0).tap()

"Sign Out" is the label in the action sheet?
Just be careful with using atIndex(), as it might break if your interface changes, or more items appear with the same label. Sadly, UIAlertController (implementation of action sheet in iOS) does not expose its buttons and does not allow setting an identifier on them.

will do, I'm new to the whole testing thing 😕 .
thank you @LeoNatan

Cheers!

"Sign Out" is the label in the action sheet?
Just be careful with using atIndex(), as it might break if your interface changes, or more items appear with the same label. Sadly, UIAlertController (implementation of action sheet in iOS) does not expose its buttons and does not allow setting an identifier on them.

I ran into the same problem recently. My solution is to wrap the title of the option with \u2002 when setting the action sheet options. For example:
"Sign Out" -> "\u2002Sign Out\u2002"

User will not notice the space character, but for some views that contains several "Sign out" text, it's useful :)


Blocks using Fieldnumber with limits are igneored

Hi,
it try to add a block to the set a int value in the range of 0-15. I use the code below, that behaves in the blockfactory of blockly correct. Means that values outside of the limit are set to the max or min limit. It seams that limits are ignored.
Best Christian
Blockly.Blocks['matrix_setbrightness'] = {
  init: function() {
    this.appendDummyInput()
        .appendField("Set brightness to")
        .appendField(new Blockly.FieldNumber(15, 0, 15), "BRIGHTNESS");
    this.setPreviousStatement(true, null);
    this.setNextStatement(true, null);
    this.setColour(230);
    this.setTooltip(Blockly.Msg.ARD_SETBRIGHTNESS_TIP);
    this.setHelpUrl('');
  }
};

Hi @lazyzero,
Looks like the ability to set minimum and maximum values to FieldNumber was added to Blockly in June 2016 (https://github.com/google/blockly/commit/0be0cc89c7f686ed6377248f7e1caa76578669c3). The blockly version in Ardublockly is a bit older than that, so this feature is not yet supported.

Hi @carlosperate
thanks for the answer. As your project is injecting own code to the blockly code I guess it is not that easy to update the blockly?
In the meantime I now added a validator that is called onchange events of the block.


Emit scaffolding when there is a new project

Description
Fixes https://github.com/Azure/autorest.csharp/issues/3414
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

/azp run autorest.csharp - ci


question(ibc-hooks): incentives to relay wasm hooks

there are 2 undone issues to handle relayer incentives it on CW level:
https://github.com/CosmWasm/cosmwasm/issues/1491
https://github.com/CosmWasm/cosmwasm/issues/893
so incentive - why would relayer burn gas on wasm hook packet?
one solution could be packet fee takes from ics20, but that is per chain module.
so how I can within existing wasm hook incetive?
can my contract reflect (query current TX  context) on signer (get signer infos) in Cosmos chain so that my contract can bank transfer some?
gas prices spiked recently on bull market.
 If that is right, spec should be changed to same block amount as in ICS20+memo imho.

dzmitry lahoda, [19/12/2023 12.22]
I guess if that is the case of 2x block times, that is most bearish in spec and likely impl. If I have read it right?

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 12.26]
I've added in Serdar who worked on the callbacks middleware

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 12.27]
just want to check with @srdtrk that you can see the previous messages? The context is that yesterday I was speaking with Blas about why they were interested in ibc-hooks and whether the callbacks middleware could solve for their desired use case

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 12.28]
do you mean that the aciton takes place over 2 blocks rather than in one? I don't think this is the case though

dzmitry lahoda, [19/12/2023 12.28]
over 2 IBC roundtrips

dzmitry lahoda, [19/12/2023 12.29]
2 times more than ICS20+hook

dzmitry lahoda, [19/12/2023 12.29]
ICS20+ICS27 takes 2x time of ICS20+Hook

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 12.32]
ok got it, I see what you mean now

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 12.40]
but once confio implement the contract keepers for the callbacks middleware then you can use it in the same way as hooks

dzmitry lahoda, [19/12/2023 12.43]
i can do it now too

dzmitry lahoda, [19/12/2023 12.43]
but helpers are nice sure

dzmitry lahoda, [19/12/2023 12.44]
i guess keepers will not change samantics of spec? so keepers and 2x are independant things

dzmitry lahoda, [19/12/2023 12.47]
i feel that as chatty as it is now, the spec, may not fit eth or to some degree sol. when cosmos stops subcidies to relays.and does 1000x gas price. are all things ok?

dzmitry lahoda, [19/12/2023 12.53]
also hope confio at some point may use ibc rs types, because need to write wrappers all the time to extend cw with ibc rs https://github.com/cosmos/ibc-rs/pull/719

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 13.49]
yeah, I guess we just wanted to have the 'official' support from Confio as the maintainers of cosmwasm and so it is more of a default there

Susannah | IBC at Interchain away until 3rd Jan, [19/12/2023 13.49]
do you have more specific feedback there? what precisely won't fit?

dzmitry lahoda, [19/12/2023 13.52]
sure

dzmitry lahoda, [19/12/2023 18.07]
there are several kinds of incetives, with  ics20-memo hooks and high prices I feel that some incetives are more easy to do or possible at all, and high prices lead to more incetive thinking. you may ask Brainjar about incetives - so you may come up to other conclusions than me.

Hello @dzmitry-lahoda if it’s a cosmos related issue i will suggest you contact cosmos support like i did and got help on their general chain link Cosmos Live Support Agent
You can use the  ticket ID DR3486 as well for a faster response.


Android - can't compile release - Execution failed for task ':@react-native-community_netinfo:generateReleaseRFile

Platforms
Android (release build only)
Versions
RN 0.64.4
react-native-netinfo 9.3.7
react 17.0.1
Description
Compiling a debug build works without issue but release build consistently fails.
Execution failed for task ':@react-native-community_netinfo:generateReleaseRFile'.
> Could not resolve all files for configuration ':@react-native-community_netinfo:releaseCompileClasspath'.
   > Failed to transform react-native-0.71.0-rc.0-release.aar (com.facebook.react:react-native:0.71.0-rc.0)
...

I think the problem seems to be related to this issue and the following lines in the netinfo package /android/build.gradle file. If I change the following:
dependencies {
  //noinspection GradleDynamicVersion
  implementation 'com.facebook.react:react-native:+'
}

to use a specific RN version, like so:
dependencies {
  //noinspection GradleDynamicVersion
  implementation 'com.facebook.react:react-native:0.64.4'
}

the app compiles a release version without issue. RN0.64.4 is a patched version fixing the linked issue above.
I'm aware this isn't an ideal solution but could there be a more elegant way of handling this (if I am on the right track...)?

Did you find a solution for this ?

@Nishchay1571999 - I forked and then forced the dependancy to use my patched RN version for now (not ideal I know).
dependencies {
  //noinspection GradleDynamicVersion
  implementation 'com.facebook.react:react-native:0.64.4'
}


node-expat fails to compile

I have some users reporting errors when installing node-xml2json which depends on node-expat. https://github.com/buglabs/node-xml2json/issues/84#issuecomment-133740487

A PR was sent to fix this issue #123


Package hls4ml with setuptools

Create a single user-facing tool and package the project with setuptools so it can be installed with pip in the future. More details here:
https://gist.github.com/vloncar/f149c7a0dba34d51e0d9c6d3cb17b12a
(sans the model zoo part for now)
Noah Paladino (@ngpaladi) will work on this.

https://github.com/vloncar/hls4ml/tree/intel

I have created a branch with some initial support for setuptools:
https://github.com/vloncar/hls4ml/tree/packaging
Includes a major restructuring of the project packages and introduces a new tool (located in scripts/hls4ml) that, when installed, will be available from the command line. A lot of things still need to be done to support running the included hls4ml from any location. @nhanvtran @ejk43 comments on the project structure and setuptools are welcome.
Brief example on how to build and install the wheel:
python setup.py sdist bdist_wheel
pip install dist/HLS4ML-0.2.0-py3-none-any.whl

@ngpaladi If you have any questions, feel free to ask me.

Nice! Installation worked for me in a clean conda environment and I could access the hls4ml script on my path. Also, if you're in the hls4ml repo, you can just do pip install . which saves a line of typing.
I tried running hls4ml convert -c <config> in a out-of-tree test directory but had a few failures, mostly related to typos and not installing the hls-templates package_data. Some minor updates here seemed to fix stuff: https://github.com/ejk43/hls4ml/tree/packaging
I was able to create a project from the example config in a new directory and run through simulation/synthesis! Pretty cool! Very useable.
One odd thing is that the include file for the nnet_utils folder ends up referencing the files inside the conda environment (or wherever hls4ml is installed). Maybe it would make sense to optionally copy the nnet_utils library into some source code directory? Or inside the generated project?

Hi @ejk43, thanks for the fixes. I thought that include_package_data=True in setup.py should be enough, but the files weren't there.
BTW, is pip install . always reinstalling all files of a package? I had issues when using that command to switch between different (major) versions of onnx development branch. Apparently not all files got rebuilt and replaced and that led to many problems, I hope that in our project we can't get into that situation.
Regarding the nnet_utils, my preference is to bundle these files with the generated project. That is how I did it for the intel branch. I thought it was a simpler solution as it can save us from users complaining that something broke when they upgrade hls4ml (or conda).

Regarding the nnet_utils, my preference is to bundle these files with the generated project. That is how I did it for the intel branch. I thought it was a simpler solution as it can save us from users complaining that something broke when they upgrade hls4ml (or conda).

Yeah, thats a really good point. I completely agree. My branch refers to the nnet_utils in the conda env but packaging with the generated project is definitely a better approach.

BTW, is pip install . always reinstalling all files of a package? I had issues when using that command to switch between different (major) versions of onnx development branch.

Aha, gotcha, pip is sometimes weird. I usually do pip install -U . which makes it upgrade the package if you've incremented the version. If that still doesn't work: pip install -U --force-reinstall .

@ngpaladi I've merged EJ changes and added a fix for nnet_utils. You can take over from here.


Update README.md

Description
Add my zappa/flask talk @ PyConFr in README.md

Coverage decreased (-42.4%) to 30.803% when pulling 89154188a1730821b699bd40d9ca3814d1b5dd4a on freezed:freezed-readme-my_guide into 60fbb55fffa762a85e79e756f2a1373832d78320 on Miserlou:master.

Did I merge wrong branch ?

@freezed no, it was just coveralls misbehaving for few months - thanks.


Question about dependency name and the origin package name

Below are dependencies samples.
dependencies:
    myCloudant:
      location: /whisk.system/cloudant
      inputs:
        dbname: myGreatDB
   myhelloworld:
      location: github.com/daisy-ycguo/wskdeploy-test/helloworld

Now we can define two kinds of dependencies:

Binding to existing package, e.g. myCloudant in the sample.  We will create a package binding to /whisk.system/cloudant with the new package name as myCloudant.
Dependency to an external source code in git repo, e.g. myhelloworld in the sample. We will deploy the package myhelloworld using the manifest file under folder http://github.com/daisy-ycguo/wskdeploy-test/helloworld.

If the dependency name is different than the origin package name, the reference to actions under this package will look weird. For example, the origin package name is myhelloworld, the dependency name is dependency_name. When I want to refer to an action under this package, I have to use myhelloworld/helloworld, not dependency_name/helloworld. Refer to below sample.
dependencies:
   dependency_name:
      location: github.com/daisy-ycguo/wskdeploy-test/helloworld
sequences:
    mySequence:
      actions: myhelloworld/helloworld

So my question is how to handle if the dependency name is different with the origin package name. Maybe we can deploy the origin package at first and then create a binding with the dependency name. It might be a reasonable solution.
@mrutkows what's your suggestion?

@daisy-ycguo  The intent was that the  dependency's <package_name> would be treated as a local alias for the actual package name as decribed in the referenced package.  The referenced package would have its own Manifest file that would include its actual Package name (and the one that would be used by the wskdeploy tool to replace the local alias).

@daisy-ycguo this would align with our decision on how to handle 'rules' and 'sequence' name references (that is, assume their names to be package-scoped names unless otherwise specified) in https://github.com/apache/incubator-openwhisk-wskdeploy/issues/394

I verified and I think package binding can help this use case.
I made following test.

create package IntegrationTestHelloWorld by wskdeploy -p tests/src/integration/helloworld
create a binding package newpkg by wsk package bind IntegrationTestHelloWorld newpkg, and then get the newpkg by wsk package get --summary newpkg, I see all the actions are listed:

package<EMAIL_ADDRESS> action<EMAIL_ADDRESS> action<EMAIL_ADDRESS> action<EMAIL_ADDRESS> action<EMAIL_ADDRESS> action<EMAIL_ADDRESS>

create another binding package newnewpkg by wsk package bind IntegrationTestHelloWorld newnewpkg and then get the newnewpkg by wsk package get --summary newnewpkg, all the actions are listed too.
invoke an action of newpkg by wsk action invoke -r newpkg/helloJava, the correct result is returned.

{
    "greeting": "Hello Bob!"
}

So I think, to this use case, we can create the original package at first, and then create binding with the new dependency name if the dependency name is different from the original package name. What's more, users can use inputs here as a binding to the original package.

merged in PR 461.  Please see follow-on issue https://github.com/apache/incubator-openwhisk-wskdeploy/issues/464 for additional updates to this use case.


Use NodeFlags.ParameterPropertyModifier rather than NodeFlags.AccessibilityModifier to detect parameter properties

This is a continuation of #8555.

Hi @andy-ms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Andy Hanson). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:+1:


Added rounding of aa and b in _schumaker_qspline.py, see Issue #1311

[ ] Closes #1311
[ ] I have read the contributing guidelines
I'm not sure how to do the following items :
[ ] Tests added
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] Pull request is nearly complete and ready for detailed review.
[ ] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

I'll take a look at the failing test. I can add the text for the whatsnew document, and take care of assigning labels. Don't worry about the API item.

I removed the data that was causing the test failure. Those data were made up to check consistency between Matlab and python implementations of pvlib.ivtools.sde._fit_sandia_cocontent. The test still has 2 of the original 3 made-up IV curves, which pass the test criteria. So I am not concerned about dropping part of the test data, the test still meets its intent. The removed data that are removed would still produce values for the 5 IV curve parameters, but they would not be close to what is produced by the Matlab code (e.g. 16% relative difference in one parameter).
I also split the EPS parameter into EPS_slope to determine when a slope is 0, and EPS_val to determine when a number is zero. The EPS_slope value is around 10^-6, the EPS_val value is single digit precision.
Why did the removed data fail the test? The single-curve fitting function is rather sensitive to its input data. Rounding small values to 0 changes the spline fit, which affects the cocontent integral, which affects a regression, etc. This sensitivity is why the diode model fitting in pvlib.ivtools.sdm.fit_pvsyst_sandia (and similar) retains only the shunt resistance from the single IV curve fits; the other four parameters aren't reliably estimated by the cocontent method.

any objections to merging this?

no, this is all fine for me, thanks again for running the tests !

Thanks @Antoine-0

I merely noticed an issue, thanks to you for updating the tests and the great work you put into pvlib !


dpdk: fail if negative perf variation is greater than 20%

Also, allow the dpdk package to be installed from a ppa or from the native debian repo.


let replacement access all data like {url}

Now you can:
php artisan notifynder:create:category "post.comment" "{from.name} leave a comment to your post"

In this case you can simply remove the array_filter() the mixedGet() checks for this and if it returns an empty value it doesn't replace anything.
This will save a bit of logic.
After this change can you please add/extend unittests  to cover the new logic/possibilities.

@Gummibeer I think array_filter() is necessary to check, so I just add a test.

To check what? The mixedGet can't return a not existing value.

@Gummibeer I don't want to remove all {xxx}, so check if xxx is a property name first.

Ok, cause there could be a case when this is part of the real message - now I got it, thanks!
I will take a look at it and extend the unittests and will merge it.

is already implemented in v4 - cause of inactivity I will close this PR.


Playground not working for HID Keyboard

Describe the bug
I am using the playground page for HID Keyboard. I want to press the character "a" without any modifiers. The playground requires a modifier to be sent and gives an error when no modifier is specified and "Key" is clicked:

TypeError: Cannot read property 'length' of undefined
    at packArguments (command.ts:57)

I next tried to specify a modifier but when I did this all the modifier options appeared in the modifier input box:

Why do modifier values start at 0xeY?

Probably want to buidl a custom UI for it.
Sent from Outlookhttp://aka.ms/weboutlook

From: James Devine @.>
Sent: Wednesday, May 12, 2021 3:20 AM
To: microsoft/jacdac @.>
Cc: Peli de Halleux @.>; Assign @.>
Subject: Re: [microsoft/jacdac] Playground not working for HID Keyboard (#380)
Why do modifier values start at 0xeY?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fissues%2F380%23issuecomment-839656841&data=04|01|jhalleux%40microsoft.com|044ccebd4ebb45e1ba0a08d9152f9a7d|72f988bf86f141af91ab2d7cd011db47|1|0|637564116464937713|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=xPhexWS8PESGy6LnAyXiJwP72yvgEfHYNJkN%2FslMMCU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKI73CQKE6QRWUFMHRDTNJI7ZANCNFSM44YJANIA&data=04|01|jhalleux%40microsoft.com|044ccebd4ebb45e1ba0a08d9152f9a7d|72f988bf86f141af91ab2d7cd011db47|1|0|637564116464947709|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Vp8YHthQmLp%2BTjpHXt%2BLhjpODUNEPsxQykepwekNJoo%3D&reserved=0.


Post release automated changes for deviceupdate releases

Post release automated changes for azure-arm-deviceupdate

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-deviceupdate


tx_execution_profile.c reserves TX_EXECUTION_TIME_SOURCE value zero to maintain internal state

Hi,
The logic in tx_execution_profile.c seems to use a value of zero to mark a timestamp as "not present". Cortex-M CYCCNT is a free-running 32-bit counter (see also https://developer.arm.com/documentation/ddi0403/d/Debug-Architecture/ARMv7-M-Debug/The-Data-Watchpoint-and-Trace-unit/CYCCNT-cycle-counter-and-related-timers ) which overflows every few seconds, e.g. every 4.3 secs on an 1 GHz CM7. There is a (small) chance that the value is zero when the counter is read. This leads to incorrect cycle accounting in tx_execution_profile.c.
Separate variables are needed for maintaining timestamp present/not present state info.
Regards,
Udo

Thank you for reporting this. We will fix this in the next release.


Update helm-chart to 1.4 GA release

As a developer, I'd like to update the helm-chart (v0.2.0 -> v0.2.1) to 1.4 GA release of K8S-server and friends.

This has been addressed. See here.


MockTestRunner fail only for exec() but ok for execSync()

I'm writing a task. However I found an weird behavior.
When I use MockToolRunner, unit test fails when I use exec() command.
let binary = new ToolRunner(tl.which(this.binary, true));
:
 await binary.exec();

##vso[task.debug]task result: Failed
##vso[task.issue type=error;]Error: istioctl failed. spawn istioctl ENOENT
##vso[task.complete result=Failed;]Error: istioctl failed. spawn istioctl ENOENT

STDERR: (node:1992) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Failed to execute istioctl command.

However, If I changes it to execSync(), it doesn't happen. Just pass the all unit test.
I'm using ^2.0.3-preview. I don't use 2.0.5 for the other issue.
https://github.com/Microsoft/vsts-task-lib/issues/264

I'll take a look at this ...

Thank you. If you want to reproduce this, please use this repo. (Master)
https://github.com/TsuyoshiUshio/KubernetesTask
Also, if you have any question, please let me know.

Hi @bryanmacfarlane , I create a sample project for reproducing this issue. It might be much easier to investigate.
https://github.com/TsuyoshiUshio/vsts-task-lib-reproduce

I'm no longer able to reproduce - is this still an issue for you?

I'm going to close, please reopen if appropriate.


Bug Report: Problems creating new library

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[x] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Don't know, first time using this feature.
Description
I was trying to create an angular library. I followed all the commands given in the documentation but after I ran the ng generate library name command I got an error while installing packages. My guess is that the cli was trying to install a nonexistent library though I'm not sure. See below for error message
🔬 Minimal Reproduction
Run the following commands:

ng new my-lib --create-application=false
cd my-lib
ng generate library my-lib

At this point you should get an error while installing packages
🔥 Exception or Error

⠏ Installing packages (npm)...npm WARN ERESOLVE overriding peer dependency
npm WARN Found: postcss@undefined
npm WARN node_modules/ng-packagr/node_modules/postcss
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/ng-packagr/node_modules/postcss-url
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/ng-packagr
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR!      /Users/apple/.npm/_logs/2021-03-26T11_16_20_754Z-debug.log
✖ Package install failed, see above.
The Schematic workflow failed.
No

From the output log, it appears than npm 7 is in use.  Please try update to the latest version of npm 7 as earlier versions contained defects that would prevent package installation in a variety of cases.

I'm having some trouble installing v7.7.5 of npm. When I fix that I will see

Hi @SiddharthShyniben,
I am unable to replicate the above using NPM 7.10.


CBAUpdate-chrisda to Main

Learn Build status updates of commit c349a17:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


uprev curve25519-dalek

https://github.com/w3f/schnorrkel/blob/master/src/keys.rs#L200C27-L201 is how one such warning is addressed in the original repo...

https://github.com/w3f/schnorrkel/blob/master/src/keys.rs#L200C27-L201 is how one such warning is addressed in the original repo...

I think the warnings are fine for now. My understanding is they only show up when building this crate directly, and will only be seen by consumers if they do the -vv flag when building.


Twitter integration not expanding all messages

https://zulip.tabbott.net/#narrow/near/24293/stream/general/topic/twitter-bot
I see a lot of this in my errors.log
2016-10-13 21:35:17,459 WARNING  Traceback (most recent call last):
  File "/home/zulip/deployments/2016-10-01-20-58-04/zerver/lib/bugdown/__init__.py", line 451, in twitter_link
    res = fetch_tweet_data(tweet_id)
  File "/home/zulip/deployments/2016-10-01-20-58-04/zerver/lib/cache.py", line 143, in func_with_caching
    val = func(*args, **kwargs)
  File "/home/zulip/deployments/2016-10-01-20-58-04/zerver/lib/bugdown/__init__.py", line 145, in fetch_tweet_data
    tweet = timeout(3, api.GetStatus, tweet_id)
  File "/home/zulip/deployments/2016-10-01-20-58-04/zerver/lib/timeout.py", line 88, in timeout
    raise TimeoutExpired
TimeoutExpired: Function call timed out.

Zulip server version 1.4.2

Does this happen consistently when you send a link to a tweet, or does it sometimes work?  I can imagine this being a problem with firewall/networking configuration that is preventing requests from reaching the server.

I happens some of the time, but not always. There are screen snapshots in the converation at zulip.tabbott.net that I linked to above.
There is no firewall blocking outgoing connections. The server is behind a NAT configuration due to necessity. I imagine that twitter is not trying to contact my server directly, but only responding to requests being made from my end, but if there are ports that need to be opened, please let me know what I should be looking for.

If it's working sometimes, it's probably actually a timeout issue (not a request hanging forever issue).  So, I think basically we need to raise the timeout.  Can you try changing it to 10s and then restarting the server (you can just do this directly without a special deploy), and see if the problem goes away?

It's possible you have slower connectivity to Twitter's servers than zulip.tabbott.net does, and thus a short timeout that works fine for us doesn't work for you.

@timabbott where is the timeout set?

So, there are two timeouts; one is the 3s in the traceback above for the twitter rendering itself; there's also a 5s overall timeout in this part of zerver/lib/bugdown/init.py
    try:
        # Spend at most 5 seconds rendering.
        # Sometimes Python-Markdown is really slow; see
        # https://trac.zulip.net/ticket/345
        return timeout(5, _md_engine.convert, content)
    except:

Indeed, increasing the timeout seems to have solved my issue. My connection is very fast, but my Zulip server is running on a VM on an old Mac Pro. The issue might just be slow computers...
I pushed the tweet timeout to 15 and overall timeout to 20. Originally, I had increased them to 10 and 11 respectively, but I still got the occasional timeout. It seems to work with the settings as I have them now.

OK, cool.  Increasing the possible time for a message to be delivered to ~20s seems a bit crazy, so solving this properly may need to involve adding support for having the tweet rendering added to the message after it is first displayed to users.

FWIW, I am also having bugdown timeout issues when sending messages with many katex blocks (e.g. the abstract of https://arxiv.org/abs/1712.08010v1).

@punchagan I was thinking about this issue, and maybe it makes sense for the Twitter integration to use a 2-tier approach: If it gets a timeout trying to fetch the tweet the first time, it still adds the messages to the "fetch links async" system, and will try to re-fetch them with a higher timeout?  I don't think it's OK to have a 20s delay before a message is delivered because of previews which is sounds like we may want to have here, but I also think that in the common case where the tweet is fetchable in ~1s, I'd prefer to do 1s of latency than have the message arrive and then be immediately edited to add the Twitter preview.

@timabbott The 2-tier approach seems promising, and we could give that a shot.  On the other hand, how do you feel about completely dropping the special API based twitter previews that we have, in favour of just using open graph tags to show a preview?  It would also get rid of some setup for server admins - no need to setup Twitter API keys.

I guess we could add some per-domain customizations for how we embed the data. We could possibly still keep the UI as it is, right now, but the data from open graph tags.

I think the current UI is nicer, and we should potentially keep the custom styling, but I'm fine with using open graph for the data if it means we can get rid of a bunch of annoying API code (and more importantly, a setup step for folks who want Twitter previews).

I also think the current UI is nicer. I can look into re-using that with data coming from open graph tags.  Doing linkification of mentions is going to be tricky with open graph tags, though. We could use @<word> regex, but it's not fool proof.

There's a few nice things that can't be done, if we switch to using the data from the open graph metadata.

@mentions being links to the profile will be a hack, and just plain words like @all may be incorrectly marked as links.
Twitter short URLs will be displayed in tweets rather than the actual URLs put into the tweet.

The third, I think, is actually a bug which makes this method quite suboptimal.
The og:image tag is set to the profile image of the user, when the tweet does not have a picture/image in it. If it does, og:image is the link to that image, and not the profile image.  There's an additional tag that identifies whether the image is "user generated" or not, but we won't be able to show both the profile image of the user and the image in the tweet content.

vs.

Based on that, I think we should probably keep the current integration, though I can imagine something where we do open graph as a fallback if we don't have keys and (2) put some sort of notice on Twitter previews messages with this fallback advertising the nicer integration.  May not be worth spending time on, though.

Yeah, keeping the existing integration makes sense.
Currently, if the keys are missing, twitter URLs don't get any previews. So, doing what you suggest might be good.
Or we could add a link to the docs or a note, in org settings -- next to the flag for URL previews, that says twitter previews need keys. The flag doesn't control those previews, si it might not be a great place to put the note, though.

We already document it as a top recommended thing to do in server setup; I just worry the actual experience might be worse for folks skipping doing that setup.

Closing this as Twitter's API pricing changes mean the Twitter integration no longer available.


9999 - JS-comment re-grouping for ESR users

makes it even easier for ESR users to re-enable the prefs they still need

!bump @earthlng Did you not see my comments about this?

Modify it as you see fit.


MAG CDF updates

Change Summary
Updating mag to produce burst CDF files as well as norm
Overview

Updated mag CDF generation to write burst and norm data into different datasets
Updated L1A mag processing to handle production of 2 files
Addressed a bug that came up with a test file provided by mag, which broke with differing length vectors

Updated Files

cli.py/mag_l1a.py

Updated to create two files

decom_mag.py

moved xarray creation into a new method, updated to split data by norm and burst

Testing

Added some additional testing around xarray and CDF generation

I don't need this merged before I leave (although it would be nice...), but I wanted to put it up for review since it is what I deployed and will leave running for the MAG team.


Can't get it to work

I'm following exactly the steps written to install. Doesn't even start running.
PoE is in window mode.

MercuryTrade.jar

I placed the file into \documents\My Games\Path of Exile
Double clicking opens a folder with tons of sub folders and files in it. No .exe around here.

MercuryTrade.rar

I placed the file into \documents\My Games\Path of Exile
Double clicking doesn't do anything at all. It's not even right-clickable

What to do?
Please help
Thank you.
Tom

Download and Install Java: https://java.com/download

Java is already downloaded and installed

Java Plug-in <IP_ADDRESS> x86
Using JRE version 1.8.0_144-b01 Java HotSpot(TM) Client VM
User home directory = C:\Users\Felhasználó
c:   clear console window
f:   finalize objects on finalization queue
g:   garbage collect
h:   display this help message
l:   dump classloader list
m:   print memory usage
o:   trigger logging
q:   hide console
r:   reload policy configuration
s:   dump system and deployment properties
t:   dump thread list
v:   dump thread stack
x:   clear classloader cache
0-5: set trace level to
Initializing applet
Version: 1.8.0_144
Vendor: Oracle+Corporation
OS: Windows+10
OS Version: 10.0
Called setUserInfo
Ending init


Add PPP for Emily

https://nus-cs2103-ay2223s1.github.io/website/schedule/week7/project.html#4-add-a-skeletal-ppp

Hey this looks good, we can merge it in already

Ok, thanks!


"Export all projects" disabled in My projects view

I am exporting my students' work from earlier this month.
I've noticed that if I go to the projects listing from the top-level "My Projects" button, then the menu option to "Export all projects" is not available.
OTOH, if I go there by choosing Projects > My projects, then it's available.
I think this is a bug.

Dear Prof. Hal, has this issue be fixed already? I am trying to do it and I have one question. As a normal user, should I be allowed to export all projects when I am in the designer/block view?

@bxie
Yes, I think you should be able to do it from the designer view.
I'll assign the issue to you.


Make raven-go a go module

Created go.mod and go.sum files to use raven-go as a go module.

This has already been discussed and we decided against doing this because I don’t want these dependencies pinned. certifi is just root certificates, so ideally, you just grab whatever is latest. If it’s pinned, that’s potentially a security risk.
And the errors package won’t change.


feat: improve error reporting

This also adds tests for the most common error cases.

This change is part of the following stack:

#758 ◀

#759

Change managed by git-spice.


[JS] [Dropdown Menu] submenu links doesn't work

Submenu links in the dropdown menu doesn't work, clicking the links closes the dropdown and doesn't redirect to the url.
Tested using version 6.0.3, Chrome  46.0.2490.86, OSX 10.11.1

I have the same issue.
6.0.3, Chrome 49.0.2572.0 canary (64-bit), OSX 10.10.5/

A slight fix is to move the e.prevent/stop in this recent commit:
https://github.com/zurb/foundation-sites/commit/1de4ee9e14d043a6b32b16f852c7cee878949c14#diff-4cf6360d3bca32bd9c93c4f8d96ead70R184
and put it into the else block (effectively !hasClicked). However, this means the close link on the submenu will follow through, usually taking you through to #, so it's not an ultimate fix.
I don't know the code well enough, but it needs a way of determining if the link click is a submenu child link or just the submenu toggle (which should prevent default). Hope that's helped.

Thanks! Meanwhile I took DropdownMenu.js from this commit and it works fine so far.

Hi I downloaded the complete version of foundation6 from the download page on the foundation site and I only have foundation.js and foundation.min.js and then a vendor folder with jquer.min.js and what-input.min.js. How do I go about updating my files so that my dropdown works? Any assistance will be appreciated.

I found the 6.0.4 versions of the js files on github and updated my docs. dropdown working fine now.


Running "update-rc.d defaults", "update-rc.d enable" and "service xxx start" on install?

Thanks for writing cargo-deb, my daemon packages already work with systemd-based OSs, such as Ubuntu and Debian.
I currently have /etc/init.d/xxx as a resource, and it is correctly installed with dpkg -i.
For sysv-init OSs, such as Devuan, I have to manually run update-rc.d defaults, update-rc.d enable and service xxx start after installing the package.
Does cargo-deb have any sysv-init support?
How can I automate these commands?

Hi @fadedbee,
You can use the maintainer-scripts setting mentioned in the README for this.
Ximon


Imp: support batch write

Package is write one by one currently, it can be changed to batch write to improve the performance.
The key points of design:

add a []byte response for Writer.Write,  which can be aggregated to write in batch：

type Writer interface {
  // if @Session is udpGettySession, the second parameter is UDPContext.
  Write(Session, interface{}) ([]byte, error)
}

enable user to config to use batch or single write mode.
aggregate bytes of packages to write in batch, set default batch size to 10.

ref: https://github.com/dubbogo/getty/pull/26


Duplications due to mixed case "File Format" values

The "File Format" facet (file_format_ontology.text) contains duplications due to the use of mixed case:

rds vs RDS vs Rds

Acceptance criteria for the task:

standardize on "rds" (all lower case) to remove duplications

This is a difficult one. We provide with the files as is, and all 3 suffixes are observed in these files. We either change the filenames or we need to keep it like this.
This is a similar issue to the fastqs displaying "fq.gz", "fq", "fastq.gz" etc
Of course, renaming it is an option, but if that is the path we want to follow we should document it! @gabsie

@theathorn isn't this one of the ones that would be easier to do a translation layer in the data portal rather than us changing all the file names? Seems like we know all the possible versions of an rds file so a translation layer should be pretty straightforward.

That wouldn't help programmatic users and would make (UI) filtering by this facet complex, unless we add case-insensitive faceting to Azul.
Tagging @NoopDog @hannes-ucsc to comment further.

The problem is that fixing inconsistencies in the UI layer removes the incentive for wranglers to be consistent. Sure, a single inconsistency is easy to fix cosmetically but the fix has to be maintained in perpetuity and the underlying issues (lack of validation, coordination and consistency) remain, causing the UI layer fixes to accumulate over time.

@theathorn It would be great to add case-insensitive faceting to Azul.
@hannes-ucsc If we move towards contributor-led submissions one day, you can expect to have a lot more inconsistencies. There needs to be a programmatic way to handle that.

That programmatic way is known as validation. It has me seriously worried that you seem to propose that we let the inconsistencies percolate all the way to the edges of the system. I don't think your arguing in good faith here, Parisa.

It's OK for file_core.file_name to be a user-supplied free text field, but file_core.format should be ontologized (e.g. an enum) or controlled (e.g. by enforcing all lower case), otherwise it would seem to be of little value.