Building and verifying "Arabic"

Note from Amit from when we tried Arabic before setting it aside for a while:
"We are also lagging the feedback audio's as well for ftm_arabic!"

I know we've put a lot of effort into this language already, and we have been up to a 5 for effort already (!), but I'm curious what the additional would be given where we are now?
Where we are:

Levelgen completely rebuilt.
Web content check shows we have all the files.
Build the app.
Technical QA check-- spot check: letters show right-to-left and generally look like the same words / characters in the levelgen? Gameplay has letters go from right-to-left?
If passing technical QA check, we can move it on to the local language expert QA.

Issues in the Arabic FTM,

The feedback audio and the text are in English.
In one of the word level (level 10) there was 1 answer option missing.

@Ashish1998 OK looks like we are going to have to tackle these from a few different fronts...
The feedback text and audio has been sent over to the Content group to fix that.
The incorrect letter options MAY be a font thing that we will have to fix on the technical end. What font are we using for this language?

CC: @amitsinghsutara

hey @burrage ,
We are using Noto-Sans Regular font for this language!

@amitsinghsutara Okie dokie... Arabic isn't a supported language for Noto-Sans-- ahhhhhhh!!! So this is making sense...
Do we still have our Adobe Arabic Bold font? Seems like it could be the go-to for this language in particular (it's even got the language name in the font!)

hey @burrage ,
hope you are doing well ;)
Regarding In one of the word level (level 10) some answer options are missing,
The issue here is that the prompt text is of 10 character length but unfortunately we only have 10 stone positions available in our gameplay due to which we were not able to get all the options!
To tackle this issue Ben we can either add two more positions or from the content end we can have two or three characters in brackets i.e. "[]" which can be helpful in reducing the number of foilstones!

Woop! Time out!!! I think you are touching on the crux of the issue here, @amitsinghsutara !
Just to make sure I'm getting this right... let's take this Arabic word from level 10 for example, "حَــسَــمَ".
Does the code in FTM JS say this is 10 characters in length?
If so, I'll need to revert to Stephanie because, you're right, if we wanted to skip how Arabic combines characters, we would need to start using [ ]s around certain characters. I'm not sure the pedagogy intent here so I'm going to start the convo with Stephanie now regardless.
Thanks for surfacing this!!!

Hey @burrage ,
Yup that's the case the code in FTM JS say this is 10 characters in length!

Thanks for understanding 🙌🙌

Thanks Ben for thoughtful insights 🙌

I've confirmed with Stephanie that from the pedagogy perspective, we do NOT want them having to drag 10 stones to make that word.
So per your recommendation Amit, the Content group will revise the levelgen and figure out what characters they want to put the [ ]s around so we get some combined characters on stones. That way there's less than 10 stones to drag and instead like 2 or 3 depending on what we want to teach.
So... while that happens, let's go ahead and make the font change. I'm anticipating we'll probably have a few trial and error sessions with the Content group and an Arabic expert since this is combined character stuff is tricky-- especially when we don't know the language! Ahhhhhh!!! 😆

Yes Ben totally,
As per the font changes we already have an opened pr for that  ;)
I'll revise that pr having only the font changes!
best Regards
Amit

Oh and a heads up, Stephanie also went ahead and made multiple feedback phrases and linked the audio on Drive in the levelgen too.

hey @burrage ,
Hope you are doing well ;)
Thanks a lot for the updates, we have updated the ftmpwa_arabic build with the levelgen contents!
Best Regards
Amit

Woo!
Keep me posted when we've done a tech QA run and hopefully this is ready to be moved on to the local language expert 😄

@burrage Hiii Ben, in Arabic level 10 for one of the word we have same 2 options for a particular letter of a word, here one option is combined and the other is the same letter is split into 2 options.
(you can observe the behavior in the below video)
Please give your feedback on this.
https://github.com/curiouslearning/FeedTheMonsterJS/assets/99384703/e2632be0-5a33-420d-b0df-6e0dc3f8d68b

Other than this we don't have any tech related issues Ben.

Awesome @Ashish1998 and @amitsinghsutara ! Yeah I think we're really down to Content QA at this point so I'm going to pass this over to Stephanie and see if she wants to go ahead with the local language expert QA to catch issues like the one in the video.
Thanks guys!

Hey question... not sure what's happening here, but I cleared all app data, uninstalled, and I downloaded and installed the latest Dev APK (May 14th) from the development server and I'm not seeing Arabic in my dropdown of languages? Am I missing something? Clearly Ashish has it!!!

Can I also confirm that we are using the manifest from the "testing" folder on Dev Wordpress and not the "dev" folder?

Hey @burrage ,
here's the link to testing apk which Ashish M is using to test the development build changes!
link

Ben regarding the dev folder manifest,
Actually ben we are using that manifest file for the local qa only,
And we are using the testing manifest file to test our latest changes to those languages which are not in local qa!

@amitsinghsutara Ah gotcha!
Can we actually update the APK that's in the /APK/ folder on the development S3 bucket please? We have that link hardcoded in a bunch of documentation that Stephanie sends out when we ask the local language experts to QA. Can you also make sure that Arabic is selectable in that dropdown?
I chatted with Stephanie today and she agrees that we should move forward with the language expert QA because we'll be scratching our heads about what's supposed to be combined and what's not otherwise 😂
Thanks Amit!

Hey @burrage ,
For sure ;)
Here's the link!
dev apk


Attempt at patching ATM for 1.0

These files are confusing to me but I think these should work, let's see what Jenkins says.

Colliding since both mods don't work together and try to overwrite. This seems to fix it though so I'm merging :)


Add Court Data Adaptor system context diagram

For future reviewers
I am leaving this organisation with this PR unmerged.
At this moment there are no other Architects in LAA to review/approve this work. So this is a "switching the lights off" PR.
Future architects joining will need to review/modify/merge this diagram themselves as I won't be here to do so.
What does this pull request do?
This diagram displays the system context of the new Court Data Adaptor
service, built for connecting LAA systems with HMCTS Common Platform
system.
Why make these changes?
We're building the connection between HMCTS Common Platform API and LAA services in order to replace (at some point) the previous LIBRA & XHIBIT integrations.
What is the result?

@richrace I think this is one for you.

This needs updating to include View Court Data.


Hi, i want to know how you split the train and validation set?

You can find it here:
https://github.com/simula/hyper-kvasir/tree/master/official_splits

You can find it here:
https://github.com/simula/hyper-kvasir/tree/master/official_splits


Pulling Singularity image doesn’t work if SLURM node has no internet

On the computer cluster I am using (Goethe HLR), only the login node has internet access, not the computing nodes. Therefore, building or pulling a Singularity image on the computing node does not work.
I run the workflow like this:
popper run --engine singularity --config config.yml --file wf.yml

The details of the workflow are not relevant, but what’s important is that Popper (version 2020.09.01) now tries to execute singularity pull through SLURM, using srun. That happens in src/popper/runner_slurm.py. Apparently, this behavior was introduced in pull request #912.
I don’t have a good suggestion. It seems like some people want their Singularity images built on the computing node, and others (like me) on the login node.

Thanks for opening this issue @wtraylor. This helps us to understand different Slurm environments better. We can change to building images on the login node by default and give an option to use the compute node (if singularity is not present on the login node). What do you think @wtraylor and @ivotron?

we could also make use of the --skip-pull flag to indicate the slurm runner to not pull images when it executes.

My hunch is that it is more common to prepare everything for an experiment on the login node, while the actual execution happens on the computing nodes. For example, I would compile my application on the login node. So from my perspective it would be more intuitive to also prepare the container image on the login node.
But I am not a very seasoned HPC user.

My hunch is that it is more common to prepare everything for an experiment on the login node, while the actual execution happens on the computing nodes. For example, I would compile my application on the login node. So from my perspective it would be more intuitive to also prepare the container image on the login node.
But I am not a very seasoned HPC user.

another pattern I've heard from people working in HPC scenarios is that they don't have network connectivity at all to the outside world from the slurm cluster, not even on the login node, so they need to scp images to the login node and then run from there. In these scenarios, doing --skip-pull and --skip-clone would help. Maybe we also need a --skip-build?
So for this issue, do we agree that we can do this:

We can change to building images on the login node by default and give an option to use the compute node (if singularity is not present on the login node). What do you think @wtraylor and @ivotron?

given what @wtraylor mentioned above, this would address this issue, right?

Maybe we also need a --skip-build?
Since building an image typically involves installing software within that image, I think skipping the build would be required, too.

sorry, I didn't explain well. What I had in mind was running the entire workflow first on the frontend node once, so it builds containers, but in single-node mode (i.e. just doing popper run with no -r flag). Then subsequently running popper run --skip-pull -r slurm. But I agree that it would be better to do that by default, and control the behavior via config file.

Yeah, building redundantly on multiple nodes is wrong. We can make changes to have 2 modes controlled through the config : i) build on login node (like in local without srun) ii) build on a single compute node (controlled through srun). What do you think @ivotron  ?

yeah, that sounds good. I'd go further and say to not implement ii) until users request it


Changing some services from beta to live

A couple of these flood related services are now live - Flood Information service and the Rivers and Sea Levels. My changes reflect these updates from Sept 2016.

LGTM, though I question the Travis CI check passing as I can't see it running. This repo will need an update.


⚠ Bofrost.nl has degraded performance

In 817fd98, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Bofrost.nl performance has improved in e9ea009 after 9 minutes.


fix shellscripts with shellcheck

update shell scripts with shellcheck
I think we can adopt on github actions

As you brought up in #223, I think the best move here is to convert the shell scripts to Python functions. Then all testing work is done in Python, and it should be cross-platform as well.
I think that addresses the incompatibility of different shell environments across OSes, and different terminal environments that people on the same OS have as well.
How does that sound?

@ehmatthes I agree with that.
also want to address
we should do running and shooting same time. little by little

I'm not sure what you mean by "running and shooting"?

@ehmatthes I can't sure about origins about that expression.
but we should do both "running and shooting" in the war.
if we do running only, will loose
if we do shooting only, we can't move
for this PR. I have two think both time.

we need to convert to python
we need to run test done well.

so if I were you, I'll merge and document this first. if testing is working.
because it's done and working and this PR is not changing django-simple-deploy itself.
after that, I'll try to convert shell script to python. step by step.

Removing line 10 in call_simple_deploy.sh breaks unit tests on my system. Maybe shellcheck is configured for your system?
I don't think it's worth troubleshooting this. I'm going to start replacing these shell scripts, and then we can troubleshoot any issues in Python.

ok. we can go with new python testing utils.


using ipywidgets

Most of the things I tried with ipywidgets work well, but here I don't understand what's wrong.
/%python
import ipywidgets as widgets
from IPython.display import display

out = widgets.Output()
def foo(x):
    with out:
        print(x)

b = widgets.Button(description='Button')
b.on_click(foo)

display(b)
display(out)

When calling foo('bar') directly it works well : 'bar' is printed in the Output widget.
When pressing the button it doesn't.
Doing exactly the same thing with a python kernel works.

Hi,
Could you let me know what the expected output is here when you push the button? With the latest version of jupyterq I see the same behaviour on clicking the button as with a python 3 kernel.
JupyterQ kernel after 2 button clicks

Python kernel after 2 button clicks.

Do you see something different with a python kernel?

The expected output is what you see.
In fact I just realized that it does work in jupyter notebook, but it doesn't work in jupyter lab.
So to sum up :
jupyter notebook + python kernel -> something is printed when button pressed
jupyter notebook + q kernel -> something is printed when button pressed
jupyter lab+ python kernel -> something is printed when button pressed
jupyter lab+ q kernel -> nothing happens when button pressed

Yes, widgets are a bit experimental and we've not done any testing with jupyterlab really as it's only in beta (moved from alpha this year I believe). Will look at adding support for widgets in jupyterlab in a future release, if you've any ideas as to what might be needed to support jupyterlab do post them here.


Fix some unassigned tests

Extract fields from tests using both formats, i.e. [Field: value] and field/value.
Assign tls tests to Test Framework
"olm" operator to belong to OLM component

/lgtm
/hold in case you're waiting for someone else.  Release at your discretion.

/lgtm

/hold cancel
Thanks!


Add ci to repo

current rust #[test]s are for docs etc, no e2e tests yet

@elmattic Could you please provide a description with tasks to do within this issue? I imagine we should start with a smallest CI possible, then iterate from there.

The kind of tests we want to add to the CI are basically those covered by cargo test:
cargo test --manifest-path ./rust/Cargo.toml

e2e tests would be the ones run by the test-suite tool and/or openrpc validate of a capture.

@ansermino @elmattic given https://github.com/ChainSafe/filecoin-common-node-api/pull/42 and https://github.com/ChainSafe/filecoin-common-node-api/pull/40, do we consider this issue closed?

I think so.


[Note?] Add .extension to valid suffixes

As the title suggest, adding the .extension suffix as a valid suffix (along with .portable and .install).
Or perhaps change the validation message to something like The .extension` suffix have been used, please make sure that the package is an extension of chocolatey or another package
It's nothing major, it's just that it is annoying as it currently is.

Yes, there are a few more valid suffixes that are known. We should definitely be sure all of them are allowed.

what are the other suffixes?
I found one other .template (excluding .app and .commandline as they shouldn't be used anymore).


[Enhancement] Set Hive string type to a larger length and set null for parse fails column

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #12382
In this pr:

Keep the same behavior as Trino, set a column to null when its parse fails.
Set a larger length for string type in the Hive.

Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[x] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 4 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/connector/ColumnTypeConverter.java
1
1
100.00%
[]

run starrocks_admit_test


Error on iOS xCode not compile

After make a fresh install i get this error on xcode
Environment
React 16.3.1
RN 0.55.20
RN-maps : 0.21.0
OS Mac
Xcode  9.3
SMCalloutView.o
    /Users/dantecervantes/Library/Developer/Xcode/DerivedData/TaxiApp-fwvamjecghqwakegtupmpgaqhrjx/Build/Products/Debug-iphonesimulator/react-native-maps/libreact-native-maps.a(SMCalloutView.o)
ld: 376 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

1 .- AirGoogleMaps and AirMaps folders are inside my project folder

2 .- I have make some code changes to use google maps in AppDelegate.m
/**
 * Copyright (c) 2015-present, Facebook, Inc.
 *
 * This source code is licensed under the MIT license found in the
 * LICENSE file in the root directory of this source tree.
 */
#import "AppDelegate.h"

#import <React/RCTBundleURLProvider.h>
#import <React/RCTRootView.h>

// **********************************************
// *** DON'T MISS: THE NEXT LINE IS IMPORTANT ***
// **********************************************
#import "RCCManager.h"

//#import <GoogleMaps/GoogleMaps.h>
@import GoogleMaps; //add this line if you want to use Google Maps

@implementation AppDelegate

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{

  //use google maps
  [GMSServices provideAPIKey:@"AIzaSyDN8-IXffH6N03tBmwm1YFcO1AULPzq8SA"];

  NSURL *jsCodeLocation;
  #ifdef DEBUG
  jsCodeLocation = [[RCTBundleURLProvider sharedSettings] jsBundleURLForBundleRoot:@"index" fallbackResource:nil];
  #else
    jsCodeLocation = [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];
  #endif

  /*RCTRootView *rootView = [[RCTRootView alloc] initWithBundleURL:jsCodeLocation
                                                      moduleName:@"TaxiApp"
                                               initialProperties:nil
                                                   launchOptions:launchOptions];
  rootView.backgroundColor = [[UIColor alloc] initWithRed:1.0f green:1.0f blue:1.0f alpha:1];

  self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];
  UIViewController *rootViewController = [UIViewController new];
  rootViewController.view = rootView;
  self.window.rootViewController = rootViewController;
  [self.window makeKeyAndVisible];*/

  // **********************************************
  // *** DON'T MISS: THIS IS HOW WE BOOTSTRAP *****
  // **********************************************
  self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];
  self.window.backgroundColor = [UIColor whiteColor];
  [[RCCManager sharedInstance] initBridgeWithBundleURL:jsCodeLocation launchOptions:launchOptions];

  return YES;
}

@end

3 .- i haved cleaned and re-build the project, and readed this solutions #718 and #2159, nothing works
any solution for this?

same problem, I lost a full afternoon trying to make it work but it never worked, following the instructions to the letter

Its already a week now I have been trying to solve this issue . Feeling demotivated now  :(

Same problem! stuck for a few days now.

 Can someone tell me the version number of this package where they DIDN'T make  developers manually install google maps?
This is EXTREMELY frustrating. This is the only known library that I have in my RN project that is not compiling. The only ERROR.
I tried manually installing Google Maps for React Native 0.51, React 16.0.0, it still did not work.
Please help resolve this or point me in the right direction.

#2396  was merged, please test with the latest git version.
Please keep in mind that you might have to setup the module again as now it works with react-native link
Read the install manual carefully

Add pod 'GoogleMaps', '~> 2.7.0' and then pod install solves the problem


Specify the versions for node, yarn and webpack module bundler, please.

It is hard to guess what is workable versions.

hi, for what is worth, on my mac, it took me some time to figure out all the dependencies, as the criptical error of " did webpack created the blabla.json?" was not very helpfull.
Finally, what I did was to:
1 - Ensure webpack was installed
2 - Run webpack
3 - Read the error of the module it could not find,
4 - run npm install 'module it could not find'
5 - check if some dependencies are needed
6 - run npm install dependencies
7 - go to step 4 until all are installed
8 - go to step 3  until all are installed

HI, thanks @juanflorez .
I used the next and it works:

node: v6.11.2
yarn:  v0.27.5
webpack: v1.14.0

@juanflorez Do you have a list of modules you were unable to install? It's supposed to work out of the box with a single yarn install command.


Run clippy for db

Codecov Report

Merging #242 into develop will decrease coverage by 0.05%.
The diff coverage is 90.19%.

@@             Coverage Diff             @@
##           develop     #242      +/-   ##
===========================================
- Coverage    49.15%   49.09%   -0.06%
===========================================
  Files          116      116
  Lines        17300    17288      -12
===========================================
- Hits          8504     8488      -16
- Misses        8796     8800       +4

Impacted Files
Coverage Δ

db/src/trie/mod.rs
81.25% <ø> (ø)
:arrow_up:

db/src/kvdb/in_memory.rs
46.15% <0%> (ø)
:arrow_up:

db/src/trie/node.rs
94% <100%> (-2%)
:arrow_down:

db/src/trie/fatdb.rs
89.36% <100%> (-2.13%)
:arrow_down:

db/src/kvdb/mod.rs
89.47% <100%> (-3.03%)
:arrow_down:

db/src/trie/recorder.rs
98.79% <100%> (ø)
:arrow_up:

db/src/journaldb/archivedb.rs
98.2% <100%> (-0.36%)
:arrow_down:

db/src/journaldb/earlymergedb.rs
93.21% <100%> (+0.12%)
:arrow_up:

db/src/trie/fatdbmut.rs
96.15% <100%> (-3.85%)
:arrow_down:

db/src/journaldb/mod.rs
97.22% <100%> (ø)
:arrow_up:

... and 21 more

Continue to review full report at Codecov.

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

Please update #177 and #224 .


No video only sound

When playing rtsp stream sometimes shows black screen with sound, player.reset() doesnt help

Have you solved your problem? I have the same problem when playing RTMP.

Я просто нашёл другой фреймворк)


Glass box fixes + Construction & Stealthy deconstruction

About the PR

Fix not being able to put Antique Laser back in glass box.
Fix not being able to close opened glass boxes prototypes.
Fix glass box damage overlay textures.
Fix glass box's fixture. Now only lasers can shoot through it.
Add glass box frame. You can climb on it, just like on table.
Add alarm sound on glass box destruction.
Add glass box's construction graph. Now can be constructed and deconstructed. Glass Box Deconstruction takes 20 seconds.
Broken glass box now can be repaired.
Glass box now can be repaired with welder.
Glass box now has locked and unlocked sprite states.

Still, there's a lot of work to do. Damage visuals fix required.
I put some TODOs for further fixes and upates.
Could be better if we had alarm system or smth, but it's okay for now.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/63782763/82030b96-3cbc-4e6c-8e25-fb0e425a338b
Changelog

:cl:

add: Glass Box now can be constructed and deconstructed!
tweak: Glass Box now makes an alarm sound on destruction.
fix:  Now Antique Laser can be put back in the Glass Box.

I don't think stealthy deconstruction is a good idea if it uses the default tools, stealthy deconstruction should require unique tools provided by the syndicate on roundstart if you have the steal laser objective like how 13 does it.

@MilenVolf what'd you change with warning_buzzer.ogg

@MilenVolf what'd you change with warning_buzzer.ogg

Removed the hum in the background.

If needed I'll revert the change and will make it in another pr

If needed I'll revert the change and will make it in another pr

There was no comment on it so not sure what happened there.


consider adding commands to flutter_tools to switch between channels

Not sure how common it'll be, but we may want to add the ability to easily switch flutter channels:
flutter channel alpha

or
flutter channel master

Agreed.  Our current situation of telling folks the magical git syntax is confusing.  @sethladd ran into this this morning.


Support for more linting types than error/warning

Previously we had an 'info' error level. Handy for things which aren't necessarily broken but perhaps unneeded or not best-practice.

You can pass whatever you want as a type. Error and Warning are just the ones colored by core.

You might want to look at: https://github.com/AtomLinter/Linter/wiki/Styling-Messages

Ah great, was thrown off by this:
type: string, // "Error" or "Warning"

Don't worry. I'll fix it.


Add conference file for ICLR

Which conference does this PR update?

-ICLR 2022
Related URL

-https://iclr.cc/Conferences/2022

It seems that the ICLR is not included in the CCF recommendation list.

You can add it to https://github.com/paperswithcode/ai-deadlines


Producing "latest" training data potentially includes invalid ground truth dates

Description
Running a big icenet_dataset_create to cache the tfrecords. The available data is up to 25/12/2023, so the end date is configured as such. In running the process scripts with that as the end date, an invalid SIC selection is happening:
Traceback (most recent call last):
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loaders/dask.py", line 408, in generate_sample
    sample_output = var_ds.siconca_abs.sel(time=forecast_dts)
  File "/home/USER/.conda/envs/icenet/lib/python3.9/site-packages/xarray/core/dataarray.py", line 1536, in sel
    ds = self._to_temp_dataset().sel(
  File "/home/USER/.conda/envs/icenet/lib/python3.9/site-packages/xarray/core/dataset.py", line 2573, in sel
    query_results = map_index_queries(
  File "/home/USER/.conda/envs/icenet/lib/python3.9/site-packages/xarray/core/indexing.py", line 188, in map_index_queries
    results.append(index.sel(labels, **options))
  File "/home/USER/.conda/envs/icenet/lib/python3.9/site-packages/xarray/core/indexes.py", line 489, in sel
    raise KeyError(f"not all values found in index {coord_name!r}")
KeyError: "not all values found in index 'time'"

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/USER/.conda/envs/icenet/bin/icenet_dataset_create", line 33, in <module>
    sys.exit(load_entry_point('icenet', 'console_scripts', 'icenet_dataset_create')())
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loader.py", line 126, in create
    dl.generate()
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loaders/dask.py", line 78, in generate
    self.client_generate(client,
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loaders/dask.py", line 218, in client_generate
    in client.gather(futures):
  File "/home/USER/.conda/envs/icenet/lib/python3.9/site-packages/distributed/client.py", line 2372, in gather
    return self.sync(
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loaders/dask.py", line 340, in generate_and_write
    x, y, sample_weights = generate_sample(date, var_ds, var_files,
  File "/rds/user/USER/hpc-work/icenet/icenet/icenet/data/loaders/dask.py", line 414, in generate_sample
    raise RuntimeError(sic_ex)
RuntimeError: "not all values found in index 'time'"

The location of this looks like it's in the ground truth select, meaning the generate_sample is maybe selecting dates past the range of the available training data. The icenet_process commands do not limit training date ranges based on number of days forecast, so we need to ensure the forecast window is correctly accounted for when creating samples.
$ cat loader.full_train_south.json | jq '.sources.osisaf.dates.train' | grep '2023_12_25'
  "2023_12_25"

This is likely only being observed as this training configuration is introducing data at the END of the full data window: the test and validation sets are pre-2023.

The earliest 93 day window we can train from with that end date is 23/09/2023, so skimming 09/23 onwards from the loader dates for the moment


Integration Test - Release - 2019/04/25 02:01:19

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#backlog'

#pri2

#backlog

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


No file type to pass to another program

struct Bkmd5sum {
// define the value
savedevice: ?????,
}
// initialize the value.
self.savedrive = ????;
for path in cd_drives() {
// Print the path
match Device::open(path.clone()){
Ok(drive) => {
// save the value
self.savedrive = drive;
}
Err(e) => {
println!("error {} in opening dvd", e);
}
}
}
If I try to open the device in another  sub-program given the /dev/sr0, I get an error.
need to save the device so I can do a self.savedevice.eject.

I'm not sure if I understand the problem but feel free to clarify. It would be helpful to have a copy of the error message and more details about what you're trying to do
The type that's missing in your first ???? would be Device or eject::device::Device
Now about this line
// initialize the value.
self.savedrive = ????;

There's no way to initialize a Device variable with an empty value like you would do in many other languages, this is just the way most types in rust work. There's two options, either use an Option<Device> type and initialize it with None or if your function is a constructor for the Bkmd5sum struct, the usual thing is to create the struct at the very end once you have all the required values. Something like this works:
struct Bkmd5sum {
    // define the value
    savedevice: Device,
}

impl Bkmd5sum {
    fn new() -> Option<Bkmd5sum> {
        // initialize the value.
        for path in cd_drives() {
            // Print the path
            match Device::open(&path) {
                Ok(drive) => {
                    // return a Bkmd5sum with the first drive found
                    return Some(Bkmd5sum { savedevice: drive });
                }
                Err(e) => {
                    println!("error {} in opening dvd", e);
                }
            }
        }
        // if there are no cd drives we need to return something
        // like None or an Error
        return None;
    }
}


Issue building Index

First of all thanks for the time spent on building the compose. Good Work.
However i'm not able to build the index with that command:
docker run -dit --log-driver=gelf --log-opt gelf-address=udp://<logstash-host>:12201 alpine ping <IP_ADDRESS>
i do also try:
docker run -dit --log-driver=gelf --log-opt gelf-address=udp://<logstash-host>:12201 alpine logger error
after i telnet on-to the logstach on port 10514 and send some strange keys, i was able to build the index.
any ideas?

Hi @Berndinox, thanks for the input!
I might have had a typo in that command, I just updated it without the -d flag since using -d with -it won't work.
I just confirmed it worked on my machine. You should see something like this in the logstash logs while the ping is running.
{
       "source_host" => "<IP_ADDRESS>",
             "level" => 6,
           "created" => "2017-02-27T22:41:01.670076264Z",
           "message" => "64 bytes from <IP_ADDRESS>: seq=0 ttl=61 time=24.529 ms",
           "version" => "1.1",
           "command" => "ping <IP_ADDRESS>",
        "image_name" => "alpine",
        "@timestamp" => 2017-02-27T22:41:02.020Z,
    "container_name" => "eager_kowalevski",
              "host" => "manager",
          "@version" => "1",
               "tag" => "b3bc46393fea",
          "image_id" => "sha256:88e169ea8f46ff0d0df784b1b254a15ecfaf045aee1856dca1ec242fdd231ddd",
      "container_id" => "b3bc46393fea562d0847d2f5cd577b625a4b521ea108fc5f44becdea28ddf41e"
}

Let me know if you're still running into issues.

Thanks! Looks good


ConfigurationManager no longer throws if _configRetriever fails but _currentConfiguration is not null

Fixes #1111.

                throw LogHelper.LogExceptionMessage(new InvalidOperationException(LogHelper.FormatInvariant(LogMessages.IDX20803, (_metadataAddress ?? "null"))));

perhaps this should be the only place where _config == null should throw?

Refers to: src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs:214 in b828af1. [](commit_id = b828af1b65f066f29d8c2694a715a2513290d489, deletion_comment = False)

I updated this PR so that a more specific exception type is thrown, and so that we log error instead of warning when we can't obtain an updated configuration (but the current configuration is not null).
I also added in a test case to check that the correct exception is thrown when we can't connect to the provided metadata endpoint. I think that the sequence of inner exceptions we provide does give enough information to understand why obtaining the configuration failed, but I can also work on providing more  information from the HTTP response in the exception thrown if we agree that's necessary.
Though I do believe that throwing a more specific exception (rather than InvalidOperationException) may be a better choice, I'm not sure if this is a change we want to make as it is a breaking change.

:shipit:


OCS returns Content-Type: text/plain; charset=utf-8 when JSON is requested

curl -k https://localhost:9200/ocs/v1.php/config?format=json -uadmin:admin -v
or
curl -k https://localhost:9200/ocs/v1.php/cloud/capabilities?format=json -uadmin:admin -v
the content-type in the response is text/plain; charset=utf-8 but should be application/json; charset=utf-8

Seems to have been fixed.
❯ curl -k 'https://localhost:9200/ocs/v1.php/config?format=json' -uadmin:admin -i
HTTP/1.1 200 OK
Content-Length: 180
Content-Type: application/json; charset=utf-8
Date: Thu, 21 Oct 2021 08:24:51 GMT
Ocs-Api-Version: 1
Vary: Origin

❯ curl -k 'https://localhost:9200/ocs/v1.php/cloud/capabilities?format=json' -uadmin:admin -i
HTTP/1.1 200 OK
Content-Length: 1775
Content-Type: application/json; charset=utf-8
Date: Thu, 21 Oct 2021 08:24:47 GMT
Ocs-Api-Version: 1
Vary: Origin


Feature-Request: Investigate Super-Linter configuration

Linter, while great, has some very strong opinions in some areas which can/are blocking PRs. We should hence investigate if we can configure exceptions for rules we deem too intense.
For example:

PSAvoidUsingPositionalParameters for Join-Path cmdlet usage
PSUseOutputTypeCorrectly as it tends to have false-positives
MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading as this doesn't make any sense for 'include' files where this has been a repeated issue

For include files just include the markdown lint rule disable on next line for MD41. This can be in each file.

Of the file isn't generated, I'd agree. But this is now the 2nd time we'd need a workaround for a Fundamentally flawed rule.
The file in question is generated which in turn means we'd need to hardcode a rule of a 3rd party tool into the script. Maybe it's just me, but this feels fundamentally wrong as it will just become a problem again later for the next item 🙄

Of the file isn't generated, I'd agree. But this is now the 2nd time we'd need a workaround for a Fundamentally flawed rule.
The file in question is generated which in turn means we'd need to hardcode a rule of a 3rd party tool into the script. Maybe it's just me, but this feels fundamentally wrong as it will just become a problem again later for the next item 🙄
For the previous case we only got around it by actually storing the markdown as a txt file. The snippet in question is directly pulled into the readme generation and would hence become a part of each module Readme

Can we include the markdownlint exclusion in the generated content?
Failing that we can exclude the rule based on file path I'm sure.

clude the markdownlint exclusion in the generated content?

We can. I'd need to code it into the script generating the content in the first place. But that's what I'm saying - it's working around an issue which already caused quite some issues for a previous matter. If we can exclude the rule, it would be much better.
Let's have a word on it some time to discuss :)


[Compaction] Resolve the -230 error by saving latest read version of a tablet

Fix #3859
This CL mainly changes:

add a new field _latest_read_version for tablet on BE to save the largest version which has
been read by query.
Modify the cumulative compaction logic. When select rowset to merge, consider both
cumulative_compaction_skip_window_seconds and _latest_read_version.
Add some docs for config
Remove unused config cumulative_compaction_budgeted_bytes

why close ?


Allow user to overwrite output dir

When testing a new video & subtitle file, I run substudy a couple times. I have to manually delete the output dir each time, or else substudy quits. It would be nice if it could prompt me to overwrite if I want to continue anyway. Or maybe a --force option, if you want to conform to Unix-y expectations and avoid interactive prompts.

Yeah, even though I kinda hate options, but -f and --force is standard enough that users will expect it to be there.
Note that we're using docopt to parse our arguments, with a lot of subcommands. This breaks down quickly as we add flags, and becomes an ugly mess. The solution is to switch to clap. You might prefer to have me tackle this switch—I can try to get around to it sometime soon. See #18 for the tracking issue.

Merged into #58.


Add support for auto-tenant-activation

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

Project coverage is 94.64%. Comparing base (9fa261f) to head (e77f368).
Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1082      +/-   ##
==========================================
+ Coverage   94.56%   94.64%   +0.08%
==========================================
  Files         181      182       +1
  Lines       18825    18874      +49
==========================================
+ Hits        17802    17864      +62
+ Misses       1023     1010      -13

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


Not works with Alfred 4

The path for workflow was changed, but here is hardcoded for Alfred 3.

Hi Vlad, can you give an example?
Thank you,
Jarek

Oh, sorry, I didn't say that today Alfred 4 was released and workflows have another path

Okay, now it works. Strange.

Alfred 4 just released and these vars not actual
TZWD="$HOME/Library/Caches/com.runningwithcrayons.Alfred-3/Workflow Data/carlosnz.timezones" TZPREFS="$HOME/Library/Application Support/Alfred 3/Workflow Data/carlosnz.timezones"

Check v2.7 please.
Closing this ticket for now, will reopen if needed.
Thank you,
Jarek

timezone edit still refers to ~/Library/Application Support/Alfred 3/Workflow Data/carlosnz.timezones/timezones.txt
Also, timezone move does move the file but then doesn't refer to it and cities go back to default.

Hi @Amparose
Well spotted, fixed in v2.9

Hey @jaroslawhartman, the edit file command now works but moving the file (via the command) still doesn't make it read from that file location, which I'm pretty sure the previous version did.

Hi,
Created a new ticket for this, see #23 .
Best regards,
Jarek


Visualizing results

Hi,
how to visualize results? I tried to use open3d library, but results are not such spectacular as yours. Moreover, there are two means_lod files in the results directory. Which should I use?

Hi Stekoaa,
Sorry for the late response. We used blender to visualize the point clouds. Basically we gave a radius to the points and render from the camera we want. The two LoDs represent the two levels with gaussian in the hash table. For visualization purposes you might want to use the mean_lods_15 version as it represents the final hash level.
With regards,
Shrisudhan


what should I change if I want to use a data set with images of 750* 184

Hi, thanks for sharing the code, I am using a dataset that can be converted to images of size 750* 184, I was wondering what should I change in this code ?
thanks in advance

Hi @Oussamab21, The first layer of the model is a convolution layer hence you don't need to change the model for processing images with different image dimensions. You need to create a YAML file with your desired image dimension.


Simplify auto-discovery process

@rmccue notes that appending ?rest_route=/ to a WordPress URL will yield the root response JSON (e.g. https://make.wordpress.org/core/?rest_route=/)—this is simpler than trusting to the presence of a header or an HTML <link> element, and we can use the self relation for the root API route to determine that base URL.

Here's how I'm doing discovery FWIW.


feat: sort-classes: Add static-block and accessor-property to default groups options

Default groups options currently do not handle static-block and accessor-property. Those elements get marked as unknown and end up at the bottom of the class.
This PR changes the default groups options by adding static-block at the very top, and grouping properties and accessor-properties together.
What is the purpose of this pull request?

[x] New Feature

Should we add tests for this?

I will add some

Test added and documentation updated.


Add --quiet/-q option and --logfile

from #1097:

We could add a -q, --quiet option to silence the StreamHandler and a --logfile option for writing a logfile to every tool.

For now this draft PR only adds the -q/--quiet option, which sets logLevel = 60, so nothing gets logged.

Actually, when writing that issue, I didn't realize that traitlets.Application already gives us already a --log_level option.
I also think quiet should only apply to the stream handler, not all handlers (you probably want full file log even if you use -q)

Yes this is a question I just stumbled upon:
if you use --logfile do you want to have terminal output or not?

if you use --logfile do you want to have terminal output or not?

By default: yes. But I would say -q and --logfile is a pretty natural combination.

I could not change the colored output in the logfile. For me it seems that the log_format gets overwritten somewhere (internally?) with the ColoredFormatter format, thus using self.log_format yields the colored format

You could make "--quiet" only set the log level for the console logger, and not the file logger.  Or in more general, we could add a --file-log-level option, so you have more control (the uyser can then set --log-level=WARNING --file-log-level=DEBUG
That gives the ability to set both independently.  In principle that can be achieved with a logging config file (As supported in the standard library), but not sure we properly support that yet - with one of those, you can have fine-grained control of all loggers, even specific module-level ones., so can for example turn the debug logging on only for hillas parameterization, or something.

One question: what is the advantage of this in the current implementation over using --log-level=WARNING? (or higher?).  I would guess we should never go above WARNING, otherwise important info might be lost.

@kosack I think when I proposed -q and -v in the issue, I didn't know about --log-level.
@nbiederbeck Could you update this, to implement a general way to configure logging, using e.g. the logging configuration files as described here:
https://docs.python.org/3/howto/logging.html#configuring-logging

You could make "--quiet" only set the log level for the console logger, and not the file logger

This is what happens, it set the "default" logger loglevel to 60, so it logs nothing. Afterwards I add the FileHandler if necessary with given loglevel.
This way -q is short for --log-level=60.

https://docs.python.org/3/howto/logging.html#configuring-logging

I'll look into that

Colored logfiles are okay with less -R and tail, but annoying in text editors

+1

Yes, I think we need to revisit this color logger in general, it prevented me from changing the format of the level.

Yes, I think we need to revisit this color logger in general, it prevented me from changing the format of the level.

There are fancier systems like "rich" (https://github.com/willmcgugan/rich), but I think we can just fix the color logger we use if there is a problem. It's nice to be able to identify warnings visually, and if logged to a file, we don't use it anyhow.

I added --log-level and  --log-file-level, which both take the level numbers and names (eg. 10/DEBUG)
If supplied --log-file the console output is untouched and additional logging to file happens

Setup logging via file with --log-config, and a configuration schema in yaml based on https://docs.python.org/3/library/logging.config.html#logging-config-dictschema
or
Use default logging and potentially set --log-level, --log-file, --log-file-level

I added a simple test for the setting of the log level.
There is a pytest fixture caplog for capturing logging output, which would be really nice for test.
I can't get this to work, maybe someone has experience with it?

@maxnoe I think I adressed all your points.
I marked this PR as draft because I still get failing tests and thought travis would not check drafts

Failing tests:

test_stage_1
test_muon_reconstruction

both fail here:
Caught unexpected exception: unsupported operand type(s) for +: 'PosixPath' and 'str'

File "/home/travis/build/cta-observatory/ctapipe/ctapipe/io/hdf5tableio.py", line 118, in __init__
    self.log.debug("h5file: %s", self._h5file)

where self._h5file is the file opened by tables. I checked, and this can be substituted by filename. I don't get why this happens only in this PR and not eg. on main branch.
Should I include this fix?

This is a bug in hdf5 I fixed some time ago: https://github.com/PyTables/PyTables/pull/809
Does travis somehow install an old version of pytables?

To save one click, this is what travis does 🤷♂:

@nbiederbeck sorry, the latest release is from October last year. So the fix is not yet included.

I have no idea why this only errors in the conda build then.

Can you add a pip freeze as last command to the install section? That will be helpful for any future debugging as well.

Ah, wait. I guess it is because you changed the log level of something to debug? Before it was probably info, so the message probably was never actually rendered.

so ..

pytables version is in all 4 travis runs 3.6.1
I'll add pip freeze, this would have made my search faster
maybe it's because of a different logging level now, but I wonder why this did'nt pop up earlier (somewhere..) I'll look into it

No it doesn't, this still weirds me out. Also, at some point this must have come up, otherwise I wouldn't have opened the PR at pytables.

changing the level of the root logger "fixes" this problem

Quick overview of the new functionality in this PR
For tools:

Add a default ctapipe logger with level WARN
Add -q to not log anything onto command line
Change the ctapipe logger level with --log-level
Log to a file with --log-file [--log-file-level] , or --log or -l (regardless of --log-level or -q)
Configure logging in-depth with our config files (--config)

Here we support loggers from other libraries, see example below

{
    "CameraDemo": {
        "log_config": {
            "loggers": {
                "matplotlib": {"level": "DEBUG", "handlers": ["console"]},
                "ctapipe.utils": {"level": "ERROR"},
            }
        }
    }
}


Cluster Autoscaler integration

This PR integrates Cluster Autoscaler into the Claudie, by implementing autoscaler-adapter, which is just an implementation of external gRPC provider.
Things that are implemented:

Autoscaler adapter
Autoscaler deployment procedure
Autoscaler testing in CI
Docs updation

Things which need to be done:

Add configuration option for CA?
Add kubernetes.io/arch label implementation
Dynamically change Autoscaler versions based on cluster version
possibly other features (expanders, etc.)

The diff might seem large, however, a lot of files were changed only due to renaming Berops to berops. Please, review carefully.

Things which need to be done:
Add configuration option for CA?
Add kubernetes.io/arch label implementation
Dynamically change Autoscaler versions based on cluster version
possibly other features (expanders, etc.)

I see them as separate issues. Are all of them must-haves?

I see them as separate issues. Are all of them must-haves?

I also see them as separate issues/PR and no, they are not must-haves. :)

Thank you. As you have the context, you're the best person IMO to open issues for those you believe are must-haves.

Hi folks,
is this still relevant? If so, have we opened the follow-up issues? Just to make sure it won't be forgotten.

Things which need to be done:
Add configuration option for CA?
Add kubernetes.io/arch label implementation
Dynamically change Autoscaler versions based on cluster version
possibly other features (expanders, etc.)

I believe kubernetes.io/arch labels are covered (@MiroslavRepka was able to provision a cluster using mixed-arch nodepools) but I don't know about the rest...

Hey @wULLSnpAXbWZGYDYyhWTKKspEQoaYxXyhoisqHf, the kubernetes.io/arch I mentioned here is regarding the node templates in node_manager. Essentially, the Cluster Autoscaler requests a template of a potential new node, which includes all labels it will have. Currently, this template does not include kubernetes.io/arch label, as not all cloud libraries included information about architecture type, thus I skipped it for now. The issue is still relevant.

I dig this :rocket:


README passes Ataru tests

finished ataru tests started in 56dabc9f. Fixed some syntax error and
deprecation warnings.

Changes Unknown when pulling 3d85ed716975f3a0800b38c68b6449bf5f2cf45e on mifix:update_readme into ** on plexus:master**.


Can't add package to Flutter

I tried adding the package to my pubspec.yaml file, after loading for several minutes it will show an error that "can't connect to flutter socket"

I don't think this is an error from the package. Post the whole error you're getting here.


Cherry Pick: Set seconds as the value of MaxAge instead of Duration.String (#11447)

/approve
/lgtm

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

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS [GregHanson,ymesika]

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

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
7ed6165e6997e86496d6c21c6550d0866c8008af
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
7ed6165e6997e86496d6c21c6550d0866c8008af
link
/test istio-pilot-multicluster-e2e

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


add travis

Hi
I just added a travis.yml and the build badge
Before merging, you just need to log in travis-ci.com and
enable this repo
closes #129

Thanks, now it's up and running. A weird thing is that some of the older node-versions didn't manage to build properly, for example node 0.12.9. I've removed them from .travis.yml for now, because the errors were very undeterministic and hard to track down. Almost to the point that I'm suspecting it to be related with some older v8-bugs, although I know that it should probably not be the case. cc @mzgol

0.11 are pre-releases to 0.12, it doesn't make sense to test on the latest 0.11 Otherwise you should test on earlier 0.11s as well, after all they all introduced breaking changes so if someone doesn't update from 0.11.13 to 0.12 then someone else doesn't update from 0.11.12 to 0.11.13.
I wouldn't test on 0.6 but whatever you want.

@mzgol check out https://github.com/olov/ng-annotate/blob/master/.travis.yml - I've only enabled 0.10 of the pre 1.0 releases. I'm still puzzled why 0.12 is problematic but I'm leaving that for now because there are so many other things to do (such as releasing ng-annotate 1.1)

BTW, this might be useful to determine the testing strategy: https://docs.google.com/spreadsheets/d/1AY1GbB1WGix4CZXY6L-6QEFZlArN1C_Ew3jMMWQ1XpQ/htmlview#gid=0&vpid=A2
npm stats showing popularity of various Node/iojs version. tl;dr: no one uses Node<0.10, almost no one uses iojs. In other words, almost no one uses unsupported versions which is good!

As for versions 4 & 5, only latest 4.x.y & 5.x.y are supported by upstream. 4.2.y is the current latest LTS.

@mzgol thanks, that's useful!


MP, QA: Move errors/validations to inside the Modal

When a user is working on MP data and QA Test Data and a screen validation is displayed, the modal needs to be closed to review the error or in some cases is closed automatically
User loses the data since it is not saved at this point
Resolution: Display the validations inside the create/view/edit modals

Bug not resolved
Use Cases:
MP
For the Monitoring Plan, when Adding or Editing data, leave the End hour or Start Hour empty after entering the Date. Then you should see the validations since we need to enter both the Date and the Time together in order to be able to save

System displays the validation on the screen and not in the modal
QA - works as expected

When saving a QA record, if the user omits a test type code the system does not display an error message at all. @vishnunavuluri will track this issue seperatly

Bug resolved
MP

QA


Update --profiling flag in kmesh yaml

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/retest

/lgtm


[Lumen] Documentation doesn't have text search

I wasn't sure where to put this, as the lumen repo doesn't allow issues. When 5.1 was released, all documentation was updated with full text search, however the Lumen documentation was not. It might be good to update the site with that as well to keep it consistent throughout.

Lumen's docs are small enough you can easily find things though?

Ping @taylorotwell.


The 8.2 alpha version and Trailers?

Mike;
I cannot see the issue post I made on Trailers. I don't see a data match with json in your trailers object in the 8.2 alpha. .
Was this the additional "coding" you referred to earlier today?
Joe

Yes. I can make an 8.2.1-alpha if you'd like?
For reference: #79

Never mind. I see you closed it!
Joe

Ok
Please do and i'll test it! Will let you know the results
joe.

@joebob49 it is now here https://www.nuget.org/packages/TMDbLib/<IP_ADDRESS>-alpha
Sorry about the delay. An emergency came up at work.

Thanks Mike;
Must be Monday there. We're still on Sunday here on the sunny west Coast of Canada.
Anyway I'll update and get back to you when I've tested....
Joe

Humm..
Not seeing any changes. Checked version and its 8.2.1
Using:
Dim Movie_Trailers As TMDbLib.Objects.Movies.Trailers = URLResponse.GetMovieTrailers(Movie_id)
I was expecting something to containing this:
{
"id": 78,
"results": [
{
"id": "533ec651c3a3685448000089",
"iso_639_1": "en",
"key": "xP4WvJaMfj8",
"name": "Trailer 1",
"site": "YouTube",
"size": 480,
"type": "Trailer"
},
{
"id": "533ec651c3a368544800008a",
"iso_639_1": "en",
"key": "PSIiGE105iA",
"name": "Harrison Ford On Blade Runner",
"site": "YouTube",
"size": 480,
"type": "Featurette"
},
Not having the "id" is causing me a uniqueness problem in my DB table. All I see in the wrapper is; Name, Size, Key per trailer.
Pls advise...
Joe

Found your code:
namespace TMDbLib.Objects.Movies
{
public class Youtube
{
public string Name { get; set; }
public string Size { get; set; }
public string Source { get; set; }
}
}
To:
public string Id { get; set; }
public string ISO-639-1  { get; set; }
public string Site { get; set; }
public string Name { get; set; }
public string Size { get; set; }
public string Source { get; set; }
public string Type  { get; set; }
Though "type" as a var name I never liked! Too generic for starters. I rather prefer your "source" to Travis's "key". Your "Standard" or "HD" seems enough and more understandable that numeric sizes.
Joe

It didn't change?
Must've messed up. :-o
I'll take a look at it when I get back home this evening.

Ah. I see now. I completely messed up when I did the two pull requests last night.

I can also see now that the /trailers has been deprecated in favor of /videos.

Arh screw it. We haven't made a full release since 0.7.7, so we can break stuff.

@joebob49 I'll try to make an <IP_ADDRESS>-alpha.

@joebob49 an <IP_ADDRESS>-alpha should be out.
https://www.nuget.org/packages/TMDbLib/<IP_ADDRESS>-alpha

Mike;
Something isn't right! I expected:
  Dim Movie_Trailers As TMDbLib.Objects.General.Video = URLResponse.GetMovieVideos(Movie_id)

But the method and the container are not agreeing. I looked at your code and see:
namespace TMDbLib.Objects.General
{
public class Video
{
public string Id { get; set; }
public string Iso_639_1 { get; set; }
public string Key { get; set; }
public string Name { get; set; }
public string Site { get; set; }
public int Size { get; set; }
public string Type { get; set; }
}
}
Which looks good btw.
Hah! Got it....
  public ResultContainer<Video> Videos { get; set; }

Right? So:
Dim Movie_Trailers As TMDbLib.Objects.General.ResultContainer(Of TMDbLib.Objects.General.Video) = URLResponse.GetMovieVideos(Movie_id)
Joe

Wow, VB looks verbose :P
Did it work out for you?

I'll be testing in a bit. have to alter my table to match new model. May take a bit as it will take three passes at least if I am to retain any fragment of my current data. ~250,000 records.
I'll let you know the results.
Joe

Mike;
Looks just fine thanks!
Joe

Cool. Remember, quite a few methods will shift places and change names over the next days and weeks, until we can get 0.8.3 out. I'll continue making 0.8.2.x builds until then.

Thanks I'll check daily...
Joe


Adding text to an existing page removes original page content

Hi,
I want to load an existing pdf document with a bunch of things on it, created with an external tool and then just add text and an image to it with PdfBox-Android.
I'm doing this (this is in Kotlin):
Log.d("Load blanks.pdf file from assets")
 val document = PDDocument.load(assetManager?.open("blanks.pdf"))
Log.d("Get the first page from the pdf")
val page = document.getPage(0)

Then I add to the page like this:
Log.d("Start the contentStream")
                val contentStream: PDPageContentStream
// Define a content stream for adding to the PDF
                    contentStream = PDPageContentStream(document, page)

                    Log.d("Add value")
                    contentStream.beginText()
                    contentStream.setNonStrokingColor(15, 38, 192)
                    contentStream.setFont(font, 12F)
                    contentStream.newLineAtOffset(100F, 700F)
                    contentStream.showText(myObjectContainingValues.value[0])
                    contentStream.endText()

Log.d("Draw the image")
                    val alphaXimage = LosslessFactory.createFromImage(document, bitmapToPrint)
                    contentStream.drawImage(alphaXimage, 20F, 20F)
Log.d("Close the content stream")
                    contentStream.close()

Log.d("Save the final PDF document to a file: pop.pdf")
                    val filePdf = File((view as Context).filesDir, "theFinalPdf.pdf")
                    if (!filePdf.exists()) {
                        try {
                            Log.d("File did not exist so create it")
                            val fos = view.openFileOutput("theFinalPdf.pdf", Context.MODE_PRIVATE)
                            fos.close()
                        } catch (e: IOException) {
                            e.printStackTrace()
                            Log.e("Could not create theFinalPdf.pdf. ", e)
                        }
                    }
                    Log.d("Saving...")
                    document.save(filePdf)
                    Log.d("Close the document")
                    document.close()

But when I display the final PDF, the page is blank with the added text and image. The original content is gone.
So my question is: When I get the first page of the document, should I do it in a different way so that I can add text and images to the content of the existing pdf?

Resolved: I didn't realise that there were other constructors for getting a PDPageContentStream. I'm now using this one and setting appendContent to true.
public PDPageContentStream(PDDocument document, PDPage sourcePage, boolean appendContent, boolean compress)

Update
Use: val pdfContentStream = PDPageContentStream(pdfDocumentToStamp, pdfFirstPage, PDPageContentStream.AppendMode.APPEND, true)


CI: Add GCC 4.8 entry to travis build matrix

Reference issue
Ref gh-12579
What does this implement/fix?
This makes one of the CI entries use the same compilers as the manylinux1 image.
I expect the first build to fail as I based this off of a commit from before gh-12586

Okay, lets try the other way.  I would like to make sure the CC and CXX flags are working correctly.

It looks like either distutils or numpy.distutils is not happy with calling ccache directly. More worryingly, the CI passed even though the build failed. I'm hoping set -e will get it to fail.

Okay, after a bit of work the failure is being reproduced here:
https://travis-ci.org/github/scipy/scipy/jobs/710118562#L10818

I've tried restarting the single travis CI matrix entry that failed with mv: cannot stat '*.gcov': No such file or directory

Normally I'd just ignore something like that, but given the modification of Travis CI code let's be sure first.

It looks like that mv command has always been failing, for example I found this in gh-12516:
https://travis-ci.org/github/scipy/scipy/jobs/710121153#L1461
It is just only now causing the build to fail because I added set -e to check for failed commands.

Alright, as a bonus feature runtests.py --gcov now works with CC and CXX. Coverage reports for native code should also be fixed now.

Great, CI is all green now. I will squash-merge to facilitate backporting and hopefully be in a position to retry 1.5.2 in the next week or so.


feat: implementation of command send process

Description

This PR adds a process for nodes with services that require sending commands to facilities.
See NODE-GRAPH(COMPANY INTERNAL LINK) for messages between nodes. See Cargo Loading Service Module(COMPANY INTERNAL LINK) for the design of modules containing nodes.
Also, the following remarks remained in the previous pull request have been corrected.

https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107921356
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107922027
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107924613
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107925149
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107925850
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107938391
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107943175
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107944728
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107945292
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107951148
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107952195
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107953611
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107954015
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107954815
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107955597
https://github.com/eve-autonomy/cargo_loading_service/pull/1#discussion_r1107959533

TODO:

The part where state and status are different in naming will be discussed and corrected after the 2nd make's PR has run its course.
Optional tag value check process not yet implemented.

Related links

Message Design

NODE-GRAPH(COMPANY INTERNAL LINK)

Module Design

Cargo Loading Service Module(COMPANY INTERNAL LINK)

Jira ticket

Ticket(COMPANY INTERNAL LINK)

Tests performed

The tests described below were conducted.

Verified that the package can be built without Error/Warning by colcon.
We started the node and waited for more than 1 minute to confirm that it did not CoreDump.
Verified the input/output values of the nodes in the description of Cargo Loading Service Module(COMPANY INTERNAL LINK).

Confirmation details are listed in Cargo Loading Service Checkpoint at the bottom.
The input/output values to be checked are listed in Cargo Loading Service Test Target at the bottom.
Details of the test can be found in ". /test/cargo_loading_service_test.md" for details.

Verified that the node can communicate for the description of NODE-GRAPH(COMPANY INTERNAL LINK).

The details of the checks, the target to be checked, and the test are the same as for the check on the Cargo Loading Service Module(COMPANY INTERNAL LINK) one above.

See Cargo Loading Service Checkpoint and Cargo Loading Service Test Target and
". /test/cargo_loading_service_test.md".

We have confirmed the startup of the node using the launch file.

Cargo Loading Service Checkpoint

Check Point No.
Description

1
The service /parking/cargo_loading returns success or failure as the response value.

2
Service behavior changes depending on the value of aw_state received at /in_parking/state.

3
Service behavior changes depending on the value of approval received in /infrastructure_status.

4
Service behavior changes depending on the value of id received in /infrastructure_status.

Node Communication Test Target

Test Target No.
Test Target
Check Point No.

1
Return SUCCESS(=1)
1

2
Return FAIL(=2)
1

3
aw_state is NONE(=0)
2

4
aw_state is AW_EMERGENCY(=1)
2

5
aw_state is AW_OUT_OF_PARKING(=2)
2

6
aw_state is AW_ARRIVED_PARKING(=3)
2

7
aw_state is AW_INVALID_DEPARTURE_INSTRUCTION(=4)
2

8
aw_state is AW_WAIT_DEPARTURE_INSTRUCTION(=5)
2

9
aw_state is AW_UNAVAILABLE(=6)
2

10
approval is true
3

11
approval is false
3

12
Match with ID requested by service
4

13
Mismatch with ID requested by service
4

Notes for reviewers

The following outstanding items are included. Additional PR will be added before this feature is released.

The function that a node should have.
Description of README.md.
Correction of remarks by Formatter and Linter.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

No
テストケース
想定挙動
結果

1
サービスが来ない
command: publishしない

2
サービスが来た- aw_stateがNONE
command: publishしないサービス: FAILを返す

3
サービスが来た-  設備連携完了でない- aw_stateがAW_EMERGENCY
command: publishサービス: 何も返さず継続

4
サービスが来た-  設備連携完了でない- aw_stateがAW_OUT_OF_PARKING- aw_stateがAW_UNAVAILABLE
command: SEND_ZEROをpublishサービス: FAILを返す

5
サービスが来た-  設備連携完了でない- aw_stateがAW_ARRIVED_PARKING- aw_stateがAW_STILL_IN_PARKING
command: REQUESTINGをpublishサービス: SUCCCESSを返すサービス: 何も返さず継続

6
サービスが来た-  設備連携完了
command: SEND_ZEROをn秒間publishサービス: SUCCESSを返す


json5 0.9.11

Recipe directory diff between current master and this update:
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
index 3dbfb1b..320812e 100644
--- a/recipe/meta.yaml
+++ b/recipe/meta.yaml
@@ -1,5 +1,5 @@
 {% set name = "json5" %}
-{% set version = "0.9.6" %}
+{% set version = "0.9.11" %}

 package:
   name: {{ name|lower }}
@@ -7,7 +7,7 @@ package:

 source:
   url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/{{ name }}-{{ version }}.tar.gz
-  sha256: 9175ad1bc248e22bb8d95a8e8d765958bf0008fef2fe8abab5bc04e0f1ac8302
+  sha256: 4f1e196acc55b83985a51318489f345963c7ba84aa37607e49073066c562e99b

 build:
   number: 0

Jira ticket:  (-)
The upstream data:
Actions:

Notes:
*
Additional info:

Package's statistics

Priority C | effort: easy | Category: anaconda | subcategory:  | pkg_type: python
Outdated platfroms: 'linux-64', 'linux-aarch64', 'linux-ppc64le', 'osx-64', 'osx-arm64', 'win-64'
Latest version: 0.9.11
Release on PyPi:

version: 0.9.11
date: 2023-01-03T02:55:41

PyPi history
Popularity:

3 months downloads: 562163.0
All time:  4102681 downloads -  json5  0.9.5  A Python implementation of the JSON5 data format

Other checks:

[x] https://github.com/dpranke/pyjson5/tree/v0.9.11 exists

[ ] The upstream url error:
https://github.com/dpranke/pyjson5

[ ] Check the pinnings

[ ] Changelog url error:
https://github.com/dpranke/pyjson5/tree/v0.9.11
200

[ ] Additional research
https://github.com/dpranke/pyjson5/issues

[ ] dev_url error:
https://github.com/dpranke/pyjson5

[ ] doc_url error:

[ ] Verify that the build_number is correct

[x] has setuptools

[x] has wheel

[x] pip in test

[ ] Verify the test section

[ ] Verify if the package is architecture specific

[ ] Verify that private modules are not mentioned in the recipe For example: (_private_module)

[x] license_file: LICENSE is present

[x] license_family Apache is present

[x] License: Apache-2.0
[x] License is spdx compliant

Check if the license identifier has correct name from the SPDX License List

Identifier

Apache-2.0

Check dependency issues:

../aggregate/json5-feedstock/recipe/meta.yaml
Dependencies: ['pip', 'python', 'setuptools', 'wheel']
noarch:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-ppc64le:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-s390x:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-arm64:

py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

win-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

Package Build Score: 17

Links:

Anaconda Recipes feedstock
Update branch
conda-forge recipe
PyPI
Diff between upstream and feature branch
Diff between origin and feature branch
The last merged PRs

Updating the recipe:
If the recipe needs additional modification the update branch can be modified. Note that the PR diffs are not updated with these changes.
git clone -b 0.9.11 git@github.com:AnacondaRecipes/json5-feedstock.git

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2cm8snl290/clone/recipe/meta.yaml:47: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_2cm8snl290/clone/recipe/meta.yaml:47: missing_description: The recipe is missing a description
Errors were found


[objc][docs] Small update for formatting and mentioning tvOS and watchOS support will come later

Build and tests passed.


Mark native engine as supporting MSAA

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14292/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14292/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14292/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14292/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14292/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14292/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14292/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14292/merge#BCU1XR#0

Does it not need to be on a min version of Native to not break @bghgary ???

Does it not need to be on a min version of Native to not break @bghgary ???

No, this doesn't do anything if the native side isn't updated. I ran the visual tests locally and passed with just this change.


unable to find executable for codelldb

OS:  22621.1555
VSCode version:  1.77.3
CodeLLDB version: 0.2.0
Compiler: SwiftForWindows-2.0.exe
Debuggee: Swift
Attempted to debug simple code to check debugger then an error came up  from VSCode stating "Unable to find executable for" followed by file name.

You need to replace <executable file> with the path of the actual executable you are debugging.


Bug: Extra underscore when encounter the first space character when cretaing drop-down items

When adding options for the drop-down, the first space is automatically replaced by an underscore. In this example "Department of Health" becomes "Department_of Health".
The workaround is for the user to enable raw OSM tags, and remove the underscore manually.

Woops 🤔
Good catch!

Fixed in 1.13.2, deployment scheduled today :)

That's what you call a quick fix. :wink:. Thank you!


python3Packages.python-picnic-api: init at 1.1.0

Motivation for this change
Python wrapper for the Picnic API
https://github.com/MikeBrink/python-picnic-api
This will become a Home Assistant dependency with the next release.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120469 run on x86_64-linux 1

2 packages built:

python38Packages.python-picnic-api
python39Packages.python-picnic-api


Update Docker build

Closes #28.
Currently we are basing the Docker build on Firedrake as of February. This needs updating.

Okay this now works. Note that I've also updated the jwallwork23/firedrake-parmmg PETSc branch.

Thanks @jwallwork23! Just out of interest, what was the issue in the end?

Didn't really investigate tbh. Firedrake's PETSc branch just got updated so I rebased the recently merged PETSc metric changes on top of that branch. Seems to work nicely now because the PETSc rebuild is minimal.


GIPP prombles

I am trying to run MAJA 3.3.0 without CAMS data for the test region around RIO, but am blocked by the following :

S2A_MSIL1C_20160802T130252_N0204_R095_T23KPQ_20160802T130656.SAFE.log
The GIPP directory used is GIPP_S2_MAJA_3.3_TM,downloaded from http://tully.ups-tlse.fr/olivier/gipp_maja/tree/master/GIPP_S2_MAJA_3.3_TM.The GIPP file was in the directory of start-maja.py .Here is the folder.txt that I configure:
folders.txt
Can you help me? I really need your help. Thank you!

Dear Xu Rui,
According to the screenshot, it seems like you point to GIPP with CAMS, not without ("../GIPP_S2AS2B_CAMS"). Did you renamed and/or modified the GIPP folders ? Are you pointing to the content of "GIPP_S2_MAJA_3.3_TM" ?
Please attach your log file to your answer.
Thanks,
Jerome

Hi Jerome,
Thank you for your reply.Last time,I renamed the GIPP_S2_MAJA_3.3_TM folders as GIPP_S2AS2B_CAMS.But this time ,I did not rename the GIPP_S2_MAJA_3.3_TM folders,the same problem was arised,

The fellowing is my log file and folders.txt :
S2A_MSIL1C_20160802T130252_N0204_R095_T23KPQ_20160802T130656.SAFE.log
folders.txt
Thanks ,
Xu Rui

Hi Xu Rui,
Sorry for replying late, that's an holiday period, here.
the message you get is related to the absence of a DTM (" AUX_REFDE2").

Did you make it (with TMPCreation) for your specific tile 23KPQ ?
did you follow the instructions regarding the DTM ?
Best regards,
Olivier

Hi there,
I am also trying to run MAJA 3.3.0 without CAMS data for a test area around Kolding, Denmark (tile 32UNG).
I have 5 L1C files, I have downloaded the SRTM and SWBD files as explained in the tutorial provided on github. I have created the DTM required with the "DTMCreation-tool" using a L1 product from the Start-MAJA github tutorial.
I have the same error as the one described here.
See below my folders.txt and log files :
folders.txt
S2A_MSIL1C_20190609T103021_N0207_R108_T32UNG_20190609T124633.SAFE.log
I hope you'll be able to help me,
Best regards,
Marion

Hi Marion,
We are developing a new version of Start_MAJA, as several users have difficulties, I hope it will improve soon.
The "AUX_REFDE"2 file is the DTM. The DTM should be stored in
repCode+"/DTM/, where repCode=/media/mhadmin/OS/Start-MAJA in your case.
Could you please list the contents of this folder anc check it contains a DTM folder, with the DTM folder for 32UNG in it  ?
Thanks,
Olivier

Hi Olivier,
Thank you for your quick response.
Content of the repostiory "repCode=/media/mhadmin/OS/Start-MAJA":
cams_download
LICENSE
SRTM
Common
LUT_MAJA_S2A_S2B_CAMS_H2ONew_20190410.tgz
start_maja.py
DTM
LUT_S2AS2B
userconf
folders.txt
prepare_dtm
Water
GIPP_S2_MAJA_3.3_TM
Readme.md
In the DTM folder there is a folder called 32UNG, and inside a folder called "S2__TEST_AUX_REFDE2_T32UNG_0001". In this one there are a folder called "S2__TEST_AUX_REFDE2_T32UNG_0001.DBL.DIR", and a file: S2__TEST_AUX_REFDE2_T32UNG_0001.HDR
Best regards,
Marion

Thanks for the detailed content
S2__TEST_AUX_REFDE2_T32UNG_0001 should be directly in the DTM folder, and 32UNG folder should be removed.
I hope it will solve the issue, please tell us !
Olivier

Hi Olivier,
I have put S2__TEST_AUX_REFDE2_T32UNG_0001in the DTM folder, removed 32UNG folder and it ran!
Thanks!
Marion

Good news !
I am closing the issue (having no news from @xurui96125
Please feel free to reopen it.
Olivier

Thank you very much. Under your guidance, I have successfully run MAJA.
At 2019-08-20 17:47:52, "Olivier Hagolle"<EMAIL_ADDRESS>wrote:
Good news !
I am closing the issue (having no news from @xurui96125
Please feel free to reopen it.
Olivier
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Create Email Notification when new Lead is generated

Hi
I am trying to create Email notification on Lead generation. Previously It was done through salesforce workflow now I need to implement AssignmentRuleHeader to send email Motification. Do we have this feature in this library.

If this is still an issue (see latest updates to code) please re-open with more specifics.


[JENKINS-66326] Prepare SCM Sync Configuration for core Guava upgrade

Downstream of #70. See JENKINS-66326 and JENKINS-65988. Jenkins core is using Guava 11.0.1, which was released on January 9, 2012. Jenkins core would like to upgrade to Guava 30.1.1, which was released on March 19, 2021. Plugins must be prepared to be compatible with both Guava 11.0.1 and Guava 30.1.1 in advance of this core transition.
In particular, this plugin has been identified as using the following methods which have been removed in the latest version of Guava:

com/google/common/io/ByteStreams#getChecksum
com/google/common/io/ByteStreams#newInputStreamSupplier
com/google/common/io/Files#getChecksum

These methods existed in Guava 11.0.1 but have been removed in recent versions.
To facilitate the Jenkins core transition, this plugin must be prepared and released such that it works with both Guava 11.0.1 and latest. This PR rewrites the relevant code to avoid the use of Guava and use native Java Platform functionality instead.
CC @guipal @rodrigc

@guipal @rodrigc Would it be possible to take a look at this PR?


Issues with sorting in sap.ui.table.Column

OpenUI5 version: 1.120.1
URL (minimal example if possible): https://github.com/AnastasyaNN/openui5-isssue-with-sorting
We have faced several issues with sorting (prepared project emulates behavior that we have on our project):
I. 'Descending' is switched to 'Ascending' after refreshing the model (works ok on OpenUI5 1.119.1)
Steps to reproduce the problem:

Clone repository https://github.com/AnastasyaNN/openui5-isssue-with-sorting
Install dependencies npm install
Serve resources npm run start
Go to http://localhost:8080/index.html
Check sorting of Date column (it should be Descending but actually it's Ascending)
Also there is an information in dev console.

What is the expected result?
Screenshot is for OpenUI5 1.119.1 (where it works ok and items are sorted Descending):

What happens instead?
Items are sorted Ascending:

II. Uncaught TypeError: Cannot read properties of undefined (reading 'fnCompare') - it's intermittent problem, reproduced from time to time on OpenUI5 1.119.1 and 1.120.1.
Steps to reproduce the problem:

Clone repository https://github.com/AnastasyaNN/openui5-isssue-with-sorting
Install dependencies npm install
Serve resources npm run start
Go to http://localhost:8080/index.html
Sort by several columns Descending, e.g. Date and Dummy*, Title and Dummy*, Creator and Dummy* (where * is 1, 2,3 or 4). I don't know if it's important, but anyway: Title, Creator and Date columns have name property defined, other columns don't have it.

What is the expected result?
Items are sorted without any issues.
What happens instead?
Uncaught TypeError: Cannot read properties of undefined (reading 'fnCompare') in dev console:

Hi AnastasyaNN, I am sorry but to proceed your issue report I need a running URL. Maybe binding maybe table maybe core colleagues need to involve in this incident and we cannot expect from all parties to clone and install. Please see the bullet point 5 and 7 of "Requirements for a bug report"

Hi aborjinik, I have copied the resources to a plunk https://plnkr.co/edit/4DbqeKeocUfRiB5c?preview. You can access running URL here https://run.plnkr.co/preview/clpi09qli001h356w49rk0p2m/
I was able to reproduce issues using the URL above:

I get an "Oh dear, something didn't go quite right. Internal Server Error" message from the specified URLs I will try again later...

Thanks! I have created an internal incident:<PHONE_NUMBER>. We will inform you again!


Add new QuickFont project

Added a link to the QuickFont project I maintain at https://github.com/opcon/QuickFont and deprecated the old, unmaintained one.

Thanks for the PR, @opcon!
I'm not familiar with the project, but from what I can tell, yours seems like the active fork. I'd say go ahead and remove the old row entirely and replace it with yours.

Sounds good to me, I've updated the resource file to reflect this

Thanks for the PR!


Fixes #122

Codecov Report
Base: 71.83% // Head: 71.83% // No change to project coverage :thumbsup:

Coverage data is based on head (ca856f4) compared to base (90fa254).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #123   +/-   ##
=========================================
  Coverage     71.83%   71.83%
  Complexity      482      482
=========================================
  Files            83       83
  Lines          1548     1548
  Branches        169      169
=========================================
  Hits           1112     1112
  Misses          380      380
  Partials         56       56

Impacted Files
Coverage Δ

src/main/java/tuthub/commons/core/Messages.java
0.00% <ø> (ø)

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

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


which version of transformers should I use?

when I run your code and load your pkl model file, I got error like this:
AttributeError: 'T5DenseReluDense' object has no attribute 'relu_act'
which version of transformers should I use?

Thank you for your attention! We will add requirements.txt later.
python==3.8
transformers==4.25.1
torch==1.10.0

3Q for your reply, I update the transformers to 4.25.1, but got this error:

File "D:\ProgramData\Anaconda3\lib\site-packages\torch\serialization.py", line 1042, in find_class
    return super().find_class(mod_name, name)
AttributeError: Can't get attribute 'T5DenseReluDense' on <module 'transformers.models.t5.modeling_t5'

Thank you for your attention! We will add requirements.txt later. python==3.8 transformers==4.25.1 torch==1.10.0

I remove this 4.25.1 version and down to version 4.8.2, then it works~  Thank you~


Testing hb 1.7.1

Testing nixos/nixpkgs#362647

Failure is unrealated

nixpkgs PR merged, let's close


SOLR not indexing 856 in DPUL and links are not visible in item level metadata

Summary
Discovered in Performance Archive materials. Links that have been provided within MARC record 856 fields that connect to concert programs for the sound recording are not visible within the item level metadata in DPUL.
Impact
Please include hard deadlines, if the exhibit is part of an event, the issue is stopping work, etc.
While this is not hindering work, it would solve an issue we are having with needing to connect Audio files to scanned resources in some way.
Priority recommendation

[ ] asap
[ ] within the next 3 weeks
[x] PO will prioritize

Sudden Priority Justification
Required if "asap" or "within the next 3 weeks" is checked.  Add "Sudden Priority" and "Maintenance/Research labels
Expected behaviour
When I go to an item  in the performance archive, I expect to see a link to the Concert Program, which is available via the 856 in the Marc (see catalog record)
Actual behaviour
When I go to an item  in the performance archive,  I do not see any link to the Concert Program.  The description mentions that a complete program is available online, but in a different field (this is a 500 field).
Steps to reproduce behaviour
Go to the DPUL Performance archive https://dpul.princeton.edu/performances/catalog/e8e51703-43e9-48c7-bb31-a0700526a8f0
confirm in OL record that there is an 856.
look to see if any link is represented within DPUL for that item.
Screenshots

Usually the 856es have arks, which would often just link back to DPUL.

Close in favor of pulibrary/figgy#5076


Bare minimum code for InternVL-Chat?

Hi,
do you have any example available to run InternVL-Chat without gradio? It would be very helpful if you could share some bare minimum python script to load the model from HF and interact.
Thanks!

Thanks for your suggestion, we will provide bare minimum python script as soon as possible.

Hi, you can run InternVL-Chat with transformers like this code:
import torch
from PIL import Image
from transformers import AutoModel, CLIPImageProcessor
from transformers import AutoTokenizer

path = "OpenGVLab/InternVL-Chat-Chinese-V1-1"
model = AutoModel.from_pretrained(
    path,
    torch_dtype=torch.bfloat16,
    low_cpu_mem_usage=True,
    trust_remote_code=True,
    device_map='auto').eval()

tokenizer = AutoTokenizer.from_pretrained(path)
image = Image.open('./examples/image2.jpg').convert('RGB')
image = image.resize((448, 448))
image_processor = CLIPImageProcessor.from_pretrained(path)

pixel_values = image_processor(images=image, return_tensors='pt').pixel_values
pixel_values = pixel_values.to(torch.bfloat16).cuda()

generation_config = dict(
    num_beams=1,
    max_new_tokens=512,
    do_sample=False,
)

question = "Describe this image in detail."
response = model.chat(tokenizer, pixel_values, question, generation_config)

Thank you for providing this! I think there are some wrong import checks. I get an error when loading the model from HF: ImportError: This modeling file requires the following packages that were not found in your environment: flash_attn. Run "pip install flash_attn". However, I have this already installed.
Before that I also got the error ImportError: This modeling file requires the following packages that were not found in your environment: fastchat. Run "pip install fastchat". Fastchat does not exists in pip, but I could fix it by installing fschat

Thanks for your feedback. The implementation of flash attention in InternViT-6B is compatible with both old and new versions. After my testing, it runs normally from v0.2.8 to v2.3.6. There may be some interface changes in v2.5.2, and I am currently verifying this. Additionally, fastchat was not actually used, and I have already removed it from the code.

Thank you for bringing this to our attention. The issue has been addressed and closed. Feel free to reach out if you have any more concerns. Thanks again!

Hi, It seems that recents commots reintroduced fastchat is back as a dependency for all models. Could you kindly check?

Hi, It seems that recents commots reintroduced fastchat is back as a dependency for all models. Could you kindly check?

I just fixed this issue, thanks for feedback!

Hi, It seems that recents commots reintroduced fastchat is back as a dependency for all models. Could you kindly check?

I just fixed this issue, thanks for feedback!

How did you fix that?


NAT classification server is offline

We need to bring up a new server and if readily practicable, come up with some basic way to monitor it.

From the pull request referenced above, it looks like the new IP of the NAT classification server is <IP_ADDRESS>. However, it seems to be down right now -- at least, I can't ping it.
@jpevarnek can you take a look?

$ ssh <IP_ADDRESS>
The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is SHA256:dhGPYWWjU8WmOFP/7p7SO3bU0BSzie5kiPgD4qJNKuY.
Are you sure you want to continue connecting (yes/no)?

Pings may be blocked.

Oooh, okay. Good catch. Blocking pings is...annoying.

Okay, should be listening again!  Not sure why the screen session died, I'll try looking into that more later.

Great. Two things we overlooked last time:

monitoring
document how to spin up a server from scratch

I think monitoring is slightly higher priority.

We do have the information for how to spin it up still (in Readme)
On Wed, Jan 27, 2016 at 4:10 PM, trevj<EMAIL_ADDRESS>wrote:

Great. Two things we overlooked last time:

monitoring
document how to spin up a server from scratch

I think monitoring is slightly higher priority.
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/2157#issuecomment-175854478.

Do you mean this, which I just noticed:
https://github.com/uProxy/uproxy-probe/blob/master/server_deploy_instruction.txt

Exactly.
On Wed, Jan 27, 2016 at 4:19 PM, trevj<EMAIL_ADDRESS>wrote:

Do you mean this, which I just noticed:
https://github.com/uProxy/uproxy-probe/blob/master/server_deploy_instruction.txt
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/2157#issuecomment-175858625.

Weird to have it in a .txt. Github will highlight and render the README.md.

How about this? https://github.com/uProxy/uproxy-probe/pull/14

@gwpenn this is no longer broken, correct?

Yeah. I believe this is outdated.


Bad Image update test for BMC

This case is to check the BMC should not get updated with bad image retain the last good image.

[ ] Bad MANIFEST
[ ] Missing image
[ ] Invalid Key

https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/24737/


feat: add option to use a pre-registered Runner

Description
GitLab announced then Next GitLab Runner Token Architecture. Runners have to be registered manually.
This PR adds a new import parameter runner_gitlab.preregistered_runner_token_ssm_parameter_name holding the name of a SSM parameter (type: SecuredString). This parameter contains the GitLab Runner token obtained from GitLab. All other registration methods will still work, but have been marked as deprecated and will be removed with v8.0.0 end of the year.
This also solves the problems with Runners removed from GitLab at shutdown, resulting in new Runners not able to start.
Closes #1074 and #1109
Verification

[x] deployed the module using the new registration version. Runner is online.
[ ] deployed the module using the old authentication schema. Runner is online.

We already have several options to configure the Runner registration. I used a new parameter to be able to remove all the old methods easily with v8.0.0

@woz5999 @lpsm-nuageit Can you please have a quick look here? Are there any better names for the new variable coming into your mind?

Still to do: update examples to new registration method, update docs and describe new registration method

@kayman-mk I believe we can use the same config nomenclatures that gitlab runner uses in the helm chart version.
But since we're in a different stream, I think the name preregistered_runner_token_ssm_parameter_name makes more sense and avoids ambiguity.

This all looks reasonable to me

Are there any plans to support AWS Secret Manager as secret store for Gitlab tokens?

The docs are still unclear to me (new to project) preregistered_runner_token_ssm_parameter_name gets created so it can't exist beforehand. Does that mean I need to pass the token into the resource after it's created?


Killswitch changes

Change it to allow users to continue if they want to.
Also allows developers to add message without pausing the bot.
However, the ability to totally kill the bot is retained.

👍


Inconsistent refresh behavior with Gateway resource's image1url field

upstream: https://github.com/Juniper/terraform-provider-mist/issues/53

fixed in latest release


Consider adding a tool to extract fundamental data from web pages

There is a lot of free fundamental data available on web pages. We already have experience with a web scrapper:
162c94410c603a488efedf407451939db3be676c
The code above was written especially for Investing.com.
Can we have something more generic to parse table-like data?
The requirements are to be able to parse table elements, but also eventually pseudo-tables made of div span constructs.

Here is a sample HTML fragment from Yahoo! Finance:

Cash Flow Statement

Operating Cash Flow (ttm)	17.13M

Levered Free Cash Flow (ttm)	-210.33M

Stopped for now.


Should this role make use of community.crypto.x509_certificate_info

For the openssl operations should this role make use of the collection 'community.crypto.x509_certificate_info'?
What do people think, i am happy to do the work if others agree.

I didn't know https://galaxy.ansible.com/community/crypto was a thing.
It would certainly clean a bunch of the openssl code up.
If you want to work on it, I can review PRs again.


Add check box for fahrenheit

The default is still Celsius
checked={isCelcius}
conditional rendering for the math and F or C
excuse the extra changed files due to linting still working on fixing that I only really changed like 8 lines

The default is still Celsius
checked={isCelcius}
conditional rendering for the math and F or C
excuse the extra changed files due to linting still working on fixing that I only really changed like 8 lines

Looks pretty good   , Thank you @DavidKozdra for valuable contribution , Merging it please give star to repository


BugReport

https://github.com/Niemeyer-Research-Group/pyMARS/blob/01a67ba2f95ad1469eeb38f7bbca1f1018153ffc/pymars/soln2ck.py#L136
if len(equation_object.products) == 1:
This line should be changed into if sum(equation_object.products.values()) == 1.
Considering the following reactions "H2O2 (+M) <=> 2 OH (+M)". There are actually two products.

Thank you for the report.  Feel free to submit a pull request with your suggestion as well if you would like.

Hi, I have submitted a pull request along with a typo fix of the doi in the ReadMe.md.

This should be resolved in the recent big refactoring.


Use Sandboxed IFRAME for fetch + parsing

https://www.html5rocks.com/en/tutorials/security/sandboxed-iframes/
Right now, MIDIConvert has the same permissions as the rest of the background page.  Since the background page doesn't have many permissions, this is probably OK.  Still, it would be safer if the MIDI parsing happened in a sandboxed IFRAME, so we knew a malformed MIDI couldn't takeover the background page.

@tambien This was some advice I got from Devlin on the Chrome Extensions team when I asked for tips on making my extension ready to ship.   Think it would be a good addition to MIDIConvert?

@appsforartists great article. but doesn't really seem necessary to add sandboxing in the code itself. I'd rather keep it as simple as possible and let users decide what sort of security measures they want to apply

@tambien Sounds good to me.  The only reason I proposed adding a parseInSandbox method is I imagine most users wouldn't even think to do it otherwise.