Issues with Vagrant + CentOS 6.7 + Oracle <IP_ADDRESS>

Basically, I'm trying to follow this guide https://oravirt.wordpress.com/2014/10/01/getting-started-with-ansible-oracle/ with ansible-oracle from master branch.

Look at the log file "/u01/app/oracle/cfgtoollogs/dbca/orcl/orcl4.log" for further details.

FATAL: all hosts have already failed -- aborting

PLAY RECAP ********************************************************************
           to retry, use: --limit @/home/vagrant/my-single-instance-install.retry

localhost                  : ok=65   changed=9    unreachable=0    failed=1

real	1m47.695s
user	0m5.501s
sys	0m4.417s

All other useful files like:
— /u01/app/oracle/cfgtoollogs/dbca/orcl/orcl4.log;
— /u01/app/oracle/cfgtoollogs/dbca/orcl/trace.log;
— my-single-instance-install.yml
are in attachment.
orcl4.txt
trace.txt
my-single-instance-install.txt

Hi,
I cant remember I've seen that error before. I just did a clean install on a Centos 6.7 box (box-cutter/centos67) as well and that worked.
Do you have a Vagrantfile I can test with? Have you done any changes to the defaults, like setup group_vars/mygroup or something like that?

I tried to use box-cutter/centos67 today and got the same error. Below I'll try to explain all my steps sequentially. I also need to mention here, that I run ansible from inside my vagrant box (manually, without provisioners), so basically ansible just has to ssh to localhost under user vagrant.
— Start new VM from box-cutter/centos67 with default settings, changed just amount of RAM for VM (increased to 2048 mb). Vagrantfile is in attachment.
— Enable EPEL repo installing epel-release-latest-6.noarch.rpm.
— Install vim, git, ansible (1.9.4).
— Clone git repo.
— Generate ssh-key for user vagrant and add pub-key into authorized_hosts for user vagrant to allow ssh to localhost with a key under user vagrant.
— echo "<IP_ADDRESS> oradb03" > /etc/hosts
— Create inventory/hosts and my-single-instance-install.yml in compliance with https://oravirt.wordpress.com/2014/10/01/getting-started-with-ansible-oracle/
[mygroup]
oradb03

---
 - name: Host configuration
   hosts: mygroup
   user: vagrant
   sudo: yes
   roles:
    - common
    - orahost
 - name: Oracle Database server installation & Database Creation
   hosts: mygroup
   user: vagrant
   sudo: yes
   sudo_user: oracle
   roles:
    - oraswdb-install
    - oradb-create

— Create group_vars/mygroup.
---
  is_sw_source_local: true
  install_from_nfs: false

— Put linuxamd64_12102_database_1of2.zip and linuxamd64_12102_database_2of2.zip into /tmp/.
— Run ansible under user vagrant like time ansible-playbook my-single-instance-install.yml -i inventory/hosts.
First after this run I get such error:
GPG key retrieval failed: [Errno 14] Could not open/read file:///etc/pki/rpm-gpg/RPM-GPG-KEY-oracle
Don't have full trace here, but pretty sure this raises in common role.
Have to fix it manually:
rpm --import http://oss.oracle.com/ol6/RPM-GPG-KEY-oracle
rpm -q gpg-pubkey-ec551f03-4c2d256
After next run I get another error:
msg: Destination /etc not writable.
This also raises in common role. Seems like issues in template module with sudo.
So to fix this issue, I have to switch to root user, and modify my my-single-instance-install.yml this way (also add -k to ansible run):
---
 - name: Host configuration
   hosts: mygroup
   user: root
   sudo: no
   roles:
    - common
    - orahost
 - name: Oracle Database server installation & Database Creation
   hosts: mygroup
   user: root
   sudo: no
   sudo_user: oracle
   roles:
    - oraswdb-install
    - oradb-create

After this it works fine till oradb-create | Create database(s) step.
Look at the log file "/u01/app/oracle/cfgtoollogs/dbca/orcl/orcl.log" for further details.

FATAL: all hosts have already failed -- aborting

orcl.log you can find in my previous comment.
I'm a very beginner with ansible, so probably I just do something wrong. All what I want here is to install Oracle DB, running ansible from inside my vagrant box.
Vagrantfile.txt

Hi,
Ok, I've changed a few things in the way the roles work so there are few things that I need to change in the blogpost as well. I'll fix that as soon as possible.
There are also a few things that I haven't pushed to the repo yet (like only adding the Oracle Linux
public yum repo if you're actually running on Oracle Linux), so that should solve the key issue.
In the meantime, you can try this and hopefully that'll work
In your group_vars/mygroup add the following line:
configure_public_yum_repo: false

That will just skip the step of adding the repo to the host and then you don't have to add the keys manually
You should also change your playbook to remove the sudo_user directive, so it should look like this.
---
 - name: Host configuration
   hosts: mygroup
   user: root
   sudo: yes
   roles:
    - common
    - orahost
 - name: Oracle Database server installation & Database Creation
   hosts: mygroup
   user: root
   sudo: yes
   roles:
    - oraswdb-install
    - oradb-create

Try that and see if it works. I haven't had time to test this again but that should hopefully do the trick.
Let me know how it goes!

Hi,
Thank you for your help.
I've tried this way today (added configure_public_yum_repo option and updated my-single-instance-install.yml).

Look at the log file "/u01/app/oracle/cfgtoollogs/dbca/orcl/orcl.log" for further details.

FATAL: all hosts have already failed -- aborting

Hi,
Ok, great.
I think the dbca issue maybe is a resource issue.
I dont have anywhere to test this at the moment, but try to raise the amount of RAM on the VM to say 3GB and run the playbook again. I'll see if I can test this tonight.

Hi,
I've tried to increase RAM to 6GB and this didn't work.

Did you re-run the entire playbook (orahost etc)? Not only try to create the database. I base some kernel-parameters on the amount of RAM etc so it might be a good idea to run that as well

Yep, I tried to start with new VM and 6GB of RAM, but this didn't work.

I'm sorry, but I can't reproduce this. I get the error when I set the RAM to 2G, but if I set it to 3GB it works. Did you also try to recreate the VM with 3G (as in vagrant destroy, vagrant up)
Other than that, I'm not sure what else I can do.

Sorry, for late reply. Still can't get over this error. So decided to start my own playbook from scratch.
Anyway thank you for your support and time.

Ok, sorry to hear it didn't work out


Fixes for updating Graph colors

Another PR laying groundwork for #2138 and #2143, in this case fixing some issues handling updates to Graph colors.

This will get exercised by TriMesh and Chord unit tests so I'll merge once tests pass.

PR build passing.


Image mistake

Current Implementation

The asset Illu_Onboarding_Datenschutz under cwa-app-ios/src/xcode/ENA/ENA/Resources/Assets/Assets.xcassets has a mistake in the shape. On the overlap between the woman's left arm and her body is a white stripe.

Internal Tracking ID: EXPOSUREAPP-1873

The arm should have a (white) outline to distinguish the arm (foreground) from body (background).
"Outline" of torso should NOT "cut" the arm in 2 pieces.
This applies to all instances of that woman (other files affected).

Hi christian-weiss, thanks for your contribution and this finding! Your issue is being forwarded.
Best regards
MC
Your Corona Warn-App Open Source Team

I found this 😅

talking about image mistakes, isn't it a mistake that all shown people in images don't wear masks? :) At least some mask reference would be great...

This was fixed in CWA version 1.5

Corona-Warn-App Open Source Team


[Question] Limits/filters for scraper.tweets?

When using scraper.tweets, is it possible to grab fewer tweets?
Context: I'm using twitter-api-client to build a sort of Twitter rss feed generator. Which means I want to be frequently checking the tweet history for a large number of users, but I only want tweets from the past day or just the most recent X (eg. 20) tweets. Currently if I try to scrape more than two or three users, I'm getting an error which I assume means I've hit a rate limit.

Twitter doesn't consistently return a fixed number of tweets per request, even when specified in the gql query (due to ads, recommendations, and other data being mixed into the response). You can grab "roughly" 20 tweets by using the count param (which limits the # of tweets returned per request) along with limit (limits total number of tweets returned).
E.g.
scraper.tweets([123,456,789], count=20, limit=20)


Inner Loop Weight Update Fix

Affected Baseline Models

VGGMAMLFewShotClassifier
FineTuneFromPretrainedFewShotClassifier
FineTuneFromScratchFewShotClassifier

Problem
In the above models, the operation which makes a copy of the latest classifier weights resides inside the support set loop, rather than the task loop.
Expected
for task_id in ....
    names_weights_copy = self.get_inner_loop_parameter_dict(...)

    for sub_task_id ....
          update names_weights_copy on support set

    measure target set using names_weights_copy

Actual
for task_id in ....
    for sub_task_id ....
          names_weights_copy = self.get_inner_loop_parameter_dict(...)
          update names_weights_copy on support set

    measure target set using names_weights_copy

Given the code above, the names_weights_copy in the affected models will be overriden every support set, then target set will be evaluated only on the weight updates of the last support set.
Experimental Results
I have done some experiments (before and after fixing) using the Pretrain+Tune baseline model (due to time/resource constraints). The configuration used was: omniglot_variant_default_5_way_1_vgg-fine-tune-pretrained_shot__False_4_2_LSLR_conditioned_0_few_shot.sh 0
Results Before Fix (Original Code)
....
train_seed 100, val_seed: 100, at start time
[ 5  7  9 10  6]
[0.0777     0.07678333 0.07676667 0.07675    0.07673333]

val_phase 250 -> loss: 4.0350, accuracy: 0.1000, : 100%|███████████| 600/600 [00:53<00:00, 11.18it/s]
val_phase 250 -> loss: 4.0764, accuracy: 0.0800, : 100%|███████████| 600/600 [00:54<00:00, 11.09it/s]
val_phase 250 -> loss: 4.1607, accuracy: 0.0700, : 100%|███████████| 600/600 [00:54<00:00, 10.97it/s]
val_phase 250 -> loss: 4.2079, accuracy: 0.0800, : 100%|███████████| 600/600 [00:54<00:00, 11.04it/s]
val_phase 250 -> loss: 4.0507, accuracy: 0.1100, : 100%|███████████| 600/600 [00:54<00:00, 10.93it/s]

{'test_accuracy_mean': 0.07596666666666667, 'test_accuracy_std': 0.2649447720228166}

The mean test accuracy of the top-5 models ensemble is 7.596666667%, which is within range to the 7.91% result in the paper.
Results After Fix
The fix involved is minor. Simply moving the names_weights_copy = self.get_inner_loop_parameter_dict(...) copy operation to the task outer loop.
The following results are using the same configuration as above, just with the fix applied.
train_seed 100, val_seed: 100, at start time
[8 9 5 7 6]

[0.10061667 0.1006     0.09981667 0.0991     0.0991    ]

val_phase 10 -> loss: 3.2573, accuracy: 0.1200, : 100%|████████████| 600/600 [00:52<00:00, 11.41it/s]
val_phase 10 -> loss: 3.1858, accuracy: 0.1100, : 100%|████████████| 600/600 [00:52<00:00, 11.35it/s]
val_phase 10 -> loss: 3.3038, accuracy: 0.1200, : 100%|████████████| 600/600 [00:53<00:00, 11.31it/s]
val_phase 10 -> loss: 3.2920, accuracy: 0.1100, : 100%|████████████| 600/600 [00:53<00:00, 11.13it/s]
val_phase 10 -> loss: 3.2781, accuracy: 0.1300, : 100%|████████████| 600/600 [00:54<00:00, 10.97it/s]

{'test_accuracy_mean': 0.1071, 'test_accuracy_std': 0.30924034342239376}

The mean test accuracy of the top-5 models ensemble is boosted from 7.59% to 10.71%.

I also have some experimental results on Pretrain+Tune model with data processing modifications in https://github.com/AntreasAntoniou/FewShotContinualLearning/pull/1.
[ 8  6 10 11  7]
[0.34863333 0.34558333 0.34523333 0.34363333 0.34315   ]
val_phase 12 -> loss: 1.8140, accuracy: 0.4300, : 100%|████████████| 600/600 [00:55<00:00, 10.78it/s]
val_phase 12 -> loss: 1.8378, accuracy: 0.3800, : 100%|████████████| 600/600 [00:55<00:00, 10.73it/s]
val_phase 12 -> loss: 1.8819, accuracy: 0.3900, : 100%|████████████| 600/600 [00:56<00:00, 10.69it/s]
val_phase 12 -> loss: 1.8285, accuracy: 0.3800, : 100%|████████████| 600/600 [00:56<00:00, 10.62it/s]
val_phase 12 -> loss: 1.8665, accuracy: 0.3900, : 100%|████████████| 600/600 [00:56<00:00, 10.66it/s]
{'test_accuracy_mean': 0.37533333333333335, 'test_accuracy_std': 0.48420886218885156}

Checked, everything looks fine to me. Merged.


Added the SciHubEVA as a brew cask formula

Hi, @womeimingzi11, thanks for the PR and adding it to brew cask. I also made some changes for this PR.

It is my honor! @leovan


merge from arlac77/template-esm-only,arlac77/template-github

.travis.yml

chore(travis): (jobs.include)
chore(travis): add 14.0.0 remove 13.13.0 (jobs.include.node_js)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Display Author in Problem Description

It occurred to me that it would be nice to include an author field in problem.json that could be displayed in the problem description. Attribution and identification of problem authors are a nice-to-have.

Sounds reasonable. Let's discuss some of the details. What's the argument for adding a new field to the JSON description versus displaying it in the description field perhaps with specialized styling?
Would this be a required field? If so, do we support backwards comparability with problems that do not have an author field?

I think it is something that could be added to problem's JSON definition and should not be required. I think the tricky party would actually be displaying it somewhere that would consistently look good as not all problems would have the field.
I like the idea of putting it somewhere in the problem description with specialized styling.

The functionality to include challenge author exists in the new picoCTF-shell-manager packaging.
UI enhancement issue ported to picoCTF-Web the actively maintained version.


Format dict key to print

I am using it to print this list:
1: Valtteri Bottas   1:53.581
2: Kimi Räikkönen    1:53.596
3: Max Verstappen    1:54.807
4: Felipe Massa      1:55.591
5: Daniel Ricciardo  1:55.626
6: Sergio Perez      1:55.676
7: Nico Hulkenberg   1:55.623
8: Daniil Kvyat      1:56.183
9: Carlos Sainz Jr.  1:56.151
10: Lance Stroll      1:56.397
11: Kevin Magnussen   1:56.808
12: Esteban Ocon      1:57.530
13: Romain Grosjean   1:57.588
14: Fernando Alonso   1:57.852
15: Jolyon Palmer     1:58.786
16: Pascal Wehrlein   1:58.776
17: Marcus Ericsson   1:58.702
18: Stoffel Vandoorne 1:58.586
19: Sebastian Vettel  2:02.790
20: Lewis Hamilton    1:53.520
but when the position printed is 10 and beyond, it breaks the align with the timings.
How can this be fixed?
I would like to be able to print
01: Valtteri Bottas   1:53.581
02: Kimi Räikkönen    1:53.596
03: Max Verstappen    1:54.807
04: Felipe Massa      1:55.591
05: Daniel Ricciardo  1:55.626
06: Sergio Perez      1:55.676
07: Nico Hulkenberg   1:55.623
08: Daniil Kvyat      1:56.183
09: Carlos Sainz Jr.  1:56.151
10: Lance Stroll      1:56.397
11: Kevin Magnussen   1:56.808
12: Esteban Ocon      1:57.530
13: Romain Grosjean   1:57.588
14: Fernando Alonso   1:57.852
15: Jolyon Palmer     1:58.786
16: Pascal Wehrlein   1:58.776
17: Marcus Ericsson   1:58.702
18: Stoffel Vandoorne 1:58.586
19: Sebastian Vettel  2:02.790
20: Lewis Hamilton    1:53.520
I added just now by hand a leading space to format the driver standings, while the timings are in correct alignement in my output, but here github is breaking the alignement :)
EDIT 1:
I added a leading 0 because the space isn't shown :)
Great library, very useful

May a auto padding function for the key helps? Which may automatically pad the all the keys with spaces to help you keep the alignment.
It's easy to add that function into reprint package :)

Oh I think i didn't get your point just now, you mean the output sorted by the key as strint but not digits.
In your case, you can use a custom sort_key function to make the output sorted properly.
Here is a sample code
# -*- coding: utf8 -*-
from reprint import output

with output(output_type="dict", interval=0, sort_key=lambda x: int(x[0])) as output:
	output['1'] = 'Valtteri Bottas 1:53.581'
	output['2'] = 'Kimi Räikkönen 1:53.596'
	output['10'] = 'asdKimi Räikkönen 1:53.596'


I train under the ubuntu, only four minigames can run ??

Dear Inorry,
Thanks your sharing, I can learn a lot. Now I have trained four minigames and they are consistent with your results. But the other three minigames can not run. the error is id 1/id 17 unknown. I use gtx1080 ti , ubuntu16.04. I wonder if it has something to do with it?

It's definitely not related to your GPU or OS. Did you install pysc2 from master branch?

yes，I install pysc2 from master branch.  When I run the program using map of FindAndDefeatZerglings, ValueError: Unknown ability_id: 1, type: cmd_quick. Likely a bug. When i run the program using map of CollectMineralsAndGas and BuildMarines, ValueError: Unknown ability_id: 17, type: cmd_quick. Likely a bug.

What SC2 version do you use?

SC2 version is 3.16.1，maybe this reason?

Yes, Reaver requires SC2 version 4.1+.

Thank u very much!


Plugin execution not covered by lifecycle configuration

I have a project that won't build in eclipse.  I am getting the error

Plugin execution not covered by lifecycle configuration: io.github.deweyjose:graphqlcodegen-maven-plugin:1.8:generate (execution: default, phase: generate-sources)

I have the following the section configuration of the pom based on the readme

        <groupId>io.github.deweyjose</groupId>
        <artifactId>graphqlcodegen-maven-plugin</artifactId>
        <version>1.8</version>
        <executions>
            <execution>
                <goals>
                    <goal>generate</goal>
                </goals>
            </execution>
        </executions>
        <configuration>
            <typeMapping>
                <PositiveDecimal>java.math.BigDecimal</PositiveDecimal>
                <PositiveInt>java.lang.Integer</PositiveInt>
            </typeMapping>
            <schemaPaths>
                <param>src/main/resources/schema/schema.graphqls</param>
            </schemaPaths>
            <packageName>org.example.graphql.generated</packageName>
        </configuration>
    </plugin>
    <plugin>
        <groupId>org.codehaus.mojo</groupId>
        <artifactId>build-helper-maven-plugin</artifactId>
        <executions>
            <execution>
                <phase>generate-sources</phase>
                <goals>
                    <goal>add-source</goal>
                </goals>
                <configuration>
                    <sources>
                        <source>${project.build.directory}/generated-sources</source>
                    </sources>
                </configuration>
            </execution>
        </executions>
    </plugin>

Additional Note: I also want to add the the original developer usee Intelj and doesn't seem to run into this issue.  However even when I try to build on the maven command line I still get errors above.

Is this plugin actively being maintained?

Sorry @AbstractAlao I use IntelliJ extensively, I won't be much help with Eclipse these days.


Resolved Popup Content issue and made Scroll to the top button only if we start scrolling

This PR is a continuation of #1264 I resolved all the conflicts  and removed one more issue with the Scroll to the top button/icon , @PriyaGhosal can you please take a look at it and let me know if something is required, but don't merge others PRs before this as this will lead to MERGE CONFLICTS again and this doesn't effect any other PR so please give priority to this.

@pa1narendra When I am clicking on privacy policy it is not taking me privacy page. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. please resolve it

@PriyaGhosal I pulled the latest code and then I made changes to the code can you please verify whether they are working or not in the main code , please let me know if I have to make any changes


TypeError: Cannot read properties of null (reading 'trim')

Trying to convert a specific (zoho) api.
https://drive.google.com/file/d/1mHIiGEkppfonKr_zTrxSESn43-IXidfv/view?usp=sharing
Command used:
P2o Zoho_CRM_REST_APIs.postman_collection.json -f ZOHO_CRM_OPENAPI.YML -o options.json
With options.json blank - {}

Fixed in version 4.2.1, the response body with value null was not supported but is part of the specs so now is added.
Take into account that the collection has incorrect json bodies that will still fail but is not a library issue, is a postman issue.


feat: add C4GHStorage on Url and S3 storage backends

Changes

Enables the C4GHStorage backend to use S3Storage and UrlStorage in configuration and in htsget-storage.
Bump github action versions.

<EMAIL_ADDRESS>-><EMAIL_ADDRESS>docker/build-push-action@v5 -> docker/build-push-action@v6

Addresses the first point in https://github.com/umccr/htsget-rs/pull/262#issuecomment-2345039202.

Updates

Renamed type to backend.
Clarify experimental feature flag comments.
Removed ObjectType in favour of flattened keys inside the storage structs /cc @brainstorm.
Make the overflow handling logic more robust when converting encrypted/unencrypted byte ranges.
Bump noodles and tower dependencies. This should close https://github.com/umccr/htsget-rs/security/dependabot/21


⚠ Bofrost.at has degraded performance

In c4e9c58, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1530 ms

Resolved: Bofrost.at performance has improved in 31d3339 after 18 minutes.


Update package.json

For a better representation on npmjs and the future rescript-lang.org package index
Here is an example on how it looks like right now (defaulting to the less ideal README file):

Thanks! will do that for all package of the org tomorrow & will cut releases.

Should be good, I finishing to publish all packages right now.


Add a readme for import validator

Codecov Report

Merging #811 (86d2da4) into develop (0f3254b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #811   +/-   ##
========================================
  Coverage    59.17%   59.17%
========================================
  Files           32       32
  Lines         4108     4108
========================================
  Hits          2431     2431
  Misses        1379     1379
  Partials       298      298

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 0f3254b...86d2da4. Read the comment docs.


Document all endpoints created by the BE

Create API documentation for Super Admin API: Using tools like Swagger/OpenAPI, Postman create a well detailed Documentation on all endpoints in the team project.This documentation outlines essential tasks that super admins can perform, including retrieving vendors information, updating profiles, deleting and restoring vendors profile, Banning and unbanning, deleting and restoring products.

assign me
my boss


First class save / load support

What problem does this solve or what need does it fill?
Saving and loading game state is a core functionality that will be reused across almost every game made in Bevy, but implementing this functionality requires advanced Bevy knowledge and there isn't consensus on how you might effectively accomplish it.
What solution would you like?
A pair of Commands to save and load a game state, probably in combination with some Resources to set global config of this functionality.
@TheRawMeatball whipped up the following prototype:

struct SaveCommand;
struct LastSaveData(Option<SomeSerializedDataType>)
trait SaveCommandExt {
  fn save(&mut self);
}

impl Command for SaveCommand {
  fn write(self: Box<Self>, world: &mut World, resources: &mut Resources){
    let data = /*serialize stuff*/;
    resources.get_mut::<LastSaveData>().unwrap().0 = Some(data);
  }
}

impl SaveCommandExt for Commands {
  fn save(&mut self) {
    self.add_command(SaveCommand)
  }
}

Usage:
fn regular_system(commands: &mut Commands) {
  commands.save();
}

What alternative(s) have you considered?
Clearly document an idiomatic way to do this.
Additional context
Here's an example of a new user trying to implement this on their own and stumbling.
This issue relates directly to #255.

A big thing to be taken into consideration is that not everything should be saved or can even be saved. In addition how should changes to the game be handled that for example add or remove ui elements. Loading a game should use the new set of ui elements. Not the saved set. This issue has overlap with how to handle scene saving in the editor.

Chiming in on what @bjorn3 mentions here. I'm implementing load/save, also taking inspiration from #166 .
I'm using the TypeRegistry to register which components I want to be saved, and before saving, I filter out the empty entities:
pub fn drop_empty(scene: DynamicScene) -> DynamicScene {
    let new_entities = scene.entities
        .into_iter()
        .filter(|entity| {
            entity.components.len() > 0
        });

    DynamicScene {
        entities: new_entities.collect()
    }
}

And here's the system saving:
fn save_scene(world: &mut World) {
    // let type_registry = world.get_resource::<TypeRegistryArc>().unwrap();

    let type_registry = TypeRegistry::default();
    type_registry.write().register::<Health>();
    type_registry.write().register::<Player>();
    type_registry.write().register::<Enemy>();

    let scene = DynamicScene::from_world(&world, &type_registry);
    let scene = helpers::drop_empty(scene);

    let mut file = File::create("assets/scenes/start_scene.scn.ron").unwrap();
    file.write_all(
        scene
            .serialize_ron(&type_registry)
            .unwrap()
            .as_bytes(),
    )
    .unwrap();
}

I ran into this, and instead of using bevy_reflect I write "hard coded" save logic to walk all entities/resources and serialize them.
As someone who had to ask around on discord for help on getting started, I figured an example might be useful for other people who came across this issue - https://gist.github.com/chamons/37e8c6f8753e63eaef08bef36686c2e2

A nice third party library experimenting in this space: https://github.com/hankjordan/bevy_save

Have to do the same in my game and also found this lib: https://github.com/Zeenobit/bevy_atomic_save
Haven't decided yet how I will go about myself. I guess the options today are:

Doing it like @chamons and looping over and saving everything yourself
use Dynamic scenes yourself (Havent fully understood this yet)
use third party bevy_save or bevy_atomic_save

Here's another alternative I wrote a while back, using a macro to save specified components:
https://github.com/bbarker/bevy_serde_macros:
https://github.com/bbarker/bevy_serde_macros/blob/main/src/lib.rs#L206-L234
there's still a small amount of boilerplate, but only in one location (late last year - 2023).


Rename ToastNotificationsDemoApp to AppNotificationsDemoApp

Rename references to ToastNotifications to AppNotifications

/azp run

How was this tested? I've rebuilt the solution and I've run all the tests and didn't notice anything wrong.
How is the demo app used? always as a stand-alone app or does it get called by other components?


Upgrade to Quarkus latest release (1.7.0 as of this ticket)

Is your feature request related to a problem? Please describe.
Blueprint should be on the latest
Describe the solution you'd like
Blueprint generates a 1.7 quarkus project
Describe alternatives you've considered
N/A
Additional context
https://github.com/quarkusio/quarkus/releases

I'll work this one but can't assign it to myself. If it can be marked I'll own it.

Never mind. Already PR'd. Can we merge?
https://github.com/jhipster/jhipster-quarkus/pull/65

Fixed


The keylogger seems broken on my new laptop

I reinstalled uLogMe recently on my newest laptop.
Everything works fine, but I just observed that the keylogger seems broken:

It's quite weird. I added two echo "blabla" comments in the scripts/keylog.sh script, and... now it works.

It's quite weird. I added two echo "blabla" comments in the scripts/keylog.sh script, and... now it works.


Feature/gmm model native

Feature introduces the Gaussian Mixture Model structure that will serve as return type generated by GMM-family algorithms.

 Native GmmModel


Import database in memory and use it as graph

This is full issue: http://stackoverflow.com/questions/35391496/orientdb-import-database-in-memory-and-use-it-as-graph

hi @urosjarc
do you have a simple database to reproduce it?

hi @urosjarc
have you solved the issue?

Yes... Thanx


Not able to import TrackletParser

Describe the bug
Hi,
I am trying to implement tracklet_parser on Google Colab. I cloned the repository and ran the setup.py file. For the next step as shown on in https://github.com/holtvogt/tracklet_parser, I am trying to import the TrackletParser. But it is not able to import it and shows error.
To Reproduce
Steps to reproduce the behavior:

Go to Google Colab Notebook.
Clone the repo using !git clone https://github.com/holtvogt/tracklet_parser
Set the current directory using %cd /content/tracklet_parser
Run the setup.py file using !python setup.py install
Import TrackletParser using from tracklet_parser import TrackletParser
See error.

Expected behavior
The TrackletParser should be imported. However, I see error on importing.
Screenshots
The screenshot of the error is attached below.

Please let me know how to handle this.

There is an issue with the package structure. I will have a look and push a fix today. Thanks for letting me know!


Mention RHEL and Debian too, for completeness

(Not actually tested on RHEL)
Description
Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)

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

Just a quick note, since we do not test on Ubuntu and RHEL we should add a disclaimer somewhere that says "might work, but we didn't check". I'm happy to have both here, as the package names that this applies to should work on all mentioned systems.


add todoist2

add Todoist 0.4.0

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/rde-master/ioBroker.todoist2", press the first "Edit" button you will find on the page and add the description.
[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/rde-master/ioBroker.todoist2", press the first "Edit" button you will find on the page and add the description.
[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/rde-master/ioBroker.todoist2", press the first "Edit" button you will find on the page and add the description.
[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/rde-master/ioBroker.todoist2", press the first "Edit" button you will find on the page and add the description.
[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew

Please clean up your package.json ... there is stuff in that only is there when you did an npm install of that package (basically all the keys with _ at the beginning.
Please also add the info channel object in https://github.com/rde-master/ioBroker.todoist2/blob/master/io-package.json#L114
supportsCustom and the custom view is included in io-package but no custom_m defined ... so I expect it can be removed or?
why a restartSchedule is defined? https://github.com/rde-master/ioBroker.todoist2/blob/master/io-package.json#L73

all your comments are done:

package.json is cleand up
info channel object ist added
supports Custom and custom view ist deletet
schedule is deletet (it was for the Case, that the connection is lost,... i build a logic so that is no longer needed)

I have added version 0.6.0 to npm to...

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/rde-master/ioBroker.todoist2", press the first "Edit" button you will find on the page and add the description.
[ ] :heavy_exclamation_mark: [E802] No topics found in the repository. Please go to "https://github.com/rde-master/ioBroker.todoist2" and press Manage topics
[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.todoist2

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "todoist2" in latest repository

Add comment "RE-CHECK!" to start check anew


Validation of nested attributes does not work

If you try to validate nested attributes, it does not work - so even your example is wrong:
public function rules()
    {
        return [['dimensions.length', 'double', 'min' => 0.0]];
    }

Validation of nested attributes can't work due to fact, that yii\validators\Validator uses $model->$attribute instead of $model->getAttribute($attribute) in validateAttribute($model, $attribute) method:
public function validateAttribute($model, $attribute)
    {
    	$result = $this->validateValue($model->$attribute);

        if (!empty($result)) {
            $this->addError($model, $attribute, $result[0], $result[1]);
        }
    }

How to overcome this issue? We could override the validateAttribute() method of yii\validators\Validator class in our own inhered class and create a Standalone Validator for every validator type we want to use for nested dynamic attributes.
Is there a better way?

@TomasMolnar $model->$attribute uses DynamicActiveRecord::__get($name) internally, that also uses $model-> getAttribute($name), so validation is functional (just tested it locally too).
I found that we don't have unit tests for this case, we'll add them. But if validation doesn't work in your case, it must be something else.

Thanks @djagya for your quick response. You are right, the issue was caused by forgotten JsonBehavior, which was tied to the same attribute. I was playing around with the best method of saving schemaless data :)


Create setting to disable/enable parse error reporting

Until we've perfected the Bash tree-sitter grammar it would be nice to be able to disable showing the parse errors.

Should it be a configuration? Or how do you envision it?

@skovhus I think making it a configuration would be a nice solution :)

An option would be really helpful given that ShellCheck is available in other extensions for VSC (and I'm sure others) and seems like a pretty mature tool.
PS: Thank you so much for creating this, it's so fantastic to have proper symbols for Bash scripts, I very nearly gave up on VSC for shell scripts and switched back to Sublime before finding this!

This has been fixed in release bash-language-server 1.4.1 and version 1.3.2 of the vscode client.


oci_tarball & oci_push api mismatch

👋 Hey There!
Something interesting I've noted is that the api for oci_tarball and  oci_push are similar but quite different.
In oci_tarball the repotags args should include the repo name and repo tag.
In oci_push the repository name is split out, and repotags is just tags and is stampable.
Would be good to standardise 😄

agreed, we usually go over all the public APIs with a fine-toothed comb before we cut a semver-stable 1.0 RC. Want to join that meeting?

We decided to remove the repository and make oci_pull match oci_tarball.

@alexeagle I'm happy to join it.
@thesayyn Assume you mean oci_push? If so how will stamping change? I.E Does it need to be aware of the repository url(s) and tagging strategies now?
IMO, I would prefer repository names and tags to be separate. Though that preference may come from my experience of doing this with an nx plugin

Assume you mean oci_push

sorry. yes.

Does it need to be aware of the repository url(s) and tagging strategies now?

after the change you'll be able to provide fully qualified names in repotags like
oci_push(
  repotags = [
      "gchr.io/org/image:latest",
      "gchr.io/orgx/imagex:latest"
  ]
)

which is more explicit version of what you have in nx matrix.

i'd also be comfortable with matrix strategy similar to nx. we could introduce attributes such as repositories = [] tags = [] which creates a matrix of both automatically. however, since we have the desire to match the oci_tarball i think we should go with repotags approach as it is explicitly in terms of final repositories x tags matrix and matches oci_tarball.

Would that also change oci_pull's interface to include a parseable tag/digest in the image field? (or also get called this repotags attribute)
Whilst I'm questioning it, why repotags and not just images?

tag yes, digests no.

@thesayyn if you could get to this one in the next day or so, I think it unblocks us finishing up https://docs.aspect.build/guides/rules_oci_migration

@thesayyn if you could get to this one in the next day or so, I think it unblocks us finishing up https://docs.aspect.build/guides/rules_oci_migration

sure. i'll take this first.

Though that preference may come from my experience of doing this with an nx plugin

BTW, the nx syntax can be easily reproducible in starlark with
oci_push(
   repotags = [
       "{}:{}".format(repository, tag)
       for tag in ["latest", "v1.0.0"]
       for repository in ["grc.io", "ghcr.io"]
   ]
)

oh that's neat
i'm not used to these maps (not sure what they're called)

the repotags change is almost finished but I remembered why it doesn't match oci_tarball API.  oci_tarball#repotags must have a tag, while oci_push doesn't.
so it's a weird case to have, but totally valid use case.
oci_push(
  repotags = [
      "gchr.io/org/image@<digest>",  # this is computed automatically at runtime
      "gchr.io/orgx/imagex:latest"
  ]
)

so either we allow repotags to specify a tag optionally to indicate "push-by-digest" or we invent a placeholder string. I like neither.
cc @alexeagle

I reverted #169 since it made the user experience significantly more confusing.
I think the likely resolution here is now "oci_tarball and oci_push are inherently different because of what the underlying tool allows you to express, so the API difference is intentional.
I'll let @thesayyn provide any more thoughts on it and maybe re-close this.

Initially the public API for oci_tarball and oci_push was different because they inherently did different things. oci_tarball responsible for loading unto daemon and oci_push pushing to 'a' registry.
unfortunately, we had to have an attribute named repotags in both rule because bazel already occupied tags which would have been the perfect name for this.
that said I don't see any action we could take here other than having strong documentation and better error messages in case people pass something we don't expect to those attributes.

I don't understand. Why not make oci_tarball match oci_push then?

I don't understand. Why not make oci_tarball match oci_push then?

that would limit users to being able to pass only one repository to manifest.json which can take more than one. that's something I consider a bug as we are committed to following specifications and the specification for tarball says there could be more than one repotag for each tarball.

decided to have oci_tarball#repo_tags to match RepoTags which appears in the metadata.json
https://snyk.io/blog/container-image-formats/
and oci_push#remote_tags to make it clear that the tags don't get "applied" to anything in bazel-out when you bazel build, they are performed on the remote when the push target is run

that would limit users to being able to pass only one repository to manifest.json which can take more than one. that's something I consider a bug as we are committed to following specifications and the specification for tarball says there could be more than one repotag for each tarball.

Is it possible to specify multiple repositories/image for oci_push?

I think this is solved by making multiple push rules in a list comprehension. You can use rules_multirun if you need a single runnable target. @thesayyn do we have an example?


store/load frequency table to/from file

feature idea: store/load frequency table to/from a file (json, pickle, ...?)

handled by 290d19a074e97a
did it with pickling because JSON would require more post/pre-processing to get working:

the _EOF object is not JSON serializable by default
keys in JSON have to be strings/int/float, while in dahuffman code tables they can be other things too


Uncaught TypeError: atom2.read is not a function

I get the above error in the browser console when I try to run a basic vite-react-pages app. The page crashes and nothing is displayed.
Steps to Reproduce

Create a new app pnpm init vite-pages app-demo --template app
Install dependencies. Install jotai because it's not included in this project's dependencies pnpm add jotai
Start the app and open it in the browser.

Thanks for report! I am looking into it.

Install dependencies. Install jotai because it's not included in this project's dependencies pnpm add jotai

Why do you manually install jotai? vite-plugin-react-pages already install it (version ^0.15.3) in dependencies.
If you do pnpm add jotai you will install version ^1.0.0 which will break vite-plugin-react-pages.
Are you using jotai in your App? If that is the case, I will remove jotai from vite-plugin-react-pages to avoid conflict.

Hmm that's curious. If I skip step 2, I get this error described here https://github.com/vitejs/vite-plugin-react-pages/issues/37
Yet, I can see jotai 0.15.5 in the pnpm lock file. Looks like the other issue is the cause of this one then.
But yeah, I do intend to use jotai within my project, as you mentioned. Not sure if there'd be a conflict, but I guess it wouldn't hurt to remove it since you'd planned to anyway.

This should be fixed in the next version.

Awesome! Works fine after removing the lock file and reinstalling dependencies.


Improve dry mode

What does this PR do?

Dry mode displays cards that will be created.
Motivation

It is nice to see what will be really created.
Additional Notes

Replace https://github.com/DataDog/integrations-core/pull/4783
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Need to fix the tests


relative path support

Currently it's a pain to write a generic queryfor "everything in current folder", as you can't do LIST FROM this.file.folder
I think something like LIST FROM "." or LIST FROM "./subfolder"  would be nice

I've actually added relative path resolution for some DataviewJS work, so I can probably extend it to queries at this point.

This would be very useful - was it implemented? I don't seem to be able to get it working.

Don't think it's been implemented but you can use [[#]] or [[]] as "current file" I think.

Also running into this issue, support for relative paths would be great.

Any update on this?

I know the following doesn't work:
LIST FROM this.file.folder

BUT, is there an equivalent using DataviewJS?

dv.current().file.folder. In DQL, you can use a where condition where file.path = this.file.path.

That's weird. Are you sure the path is correct and doesn't have typos? What's the output of dv.current().file.folder?

Hello,
You can do that with help of the folder, not the path implicit field:
LIST
WHERE file.folder = this.file.folder

Path contains the file name, folder does not.

Thank you. I see it interprets the string, allow me to extend your example for those wanting to point to a relative dir. E.g. in the current dir we have a subfolder recipes, then we can use:
WHERE file.folder = this.file.folder + "/recipes"

Hello,
You can do that with help of the folder, not the path implicit field:
LIST
WHERE file.folder = this.file.folder

Path contains the file name, folder does not.

any news on this?

any news on this?

any news on this?

Revision History
table without id
   file.name as "Revision Date",
  author,
  comment
from #revision
	where file.folder = this.file.folder + "/revisions"

works currently

I would like to add my vote for this feature request.


Feature request: CliOption take priority to file config

Add priority to parameters entered on the command line for Boolean types.
In the current version, you can add command line parameters such as --skip-tag, --skip-commit, ...
However, they are necessarily set to True and it is impossible to change this value (set False instead).
The idea I'm proposing is to change the CommandOptionType.NoValue in the CliConfig.cs file to CommandOptionType.SingleOrNoValue, which would accept a True/False boolean.
Example:
versionize --pre-release alpha --skip-tag=False --find-release-commit-via-message=True
versionize --pre-release alpha --skip-tag=True --find-release-commit-via-message=False
versionize --pre-release alpha --skip-tag --find-release-commit-via-message=False (skipTag will be True)

Ok, is this a feature that you need,  or is it just a general feature idea?

Yes, it's general feature idea. Not a priority


[Feature]: add "interest-cohort" feature policy

Type of request
improvement
Requested change or question
I think it would be a good idea to, by default, disable the FLOC feature (Federated learning of cohorts)
This a build-in way for Google to track users using the Chrome browser.
https://www.howtogeek.com/724441/what-is-googles-floc-and-how-will-it-track-you-online/
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Treggats I've updated the default directives set, but rector and phpstan fail with 8.2. Locally everything seems to be fine :-/

@mazedlx I'll have a look this weekend to see what's going on.

at a first glance it seems that either pushMiddleware() self is not present with PHP 8.2 or that it uses something only present in PHP 8.3 and up

Got a fix kinda ready, want to do some more checking and find out why it stopped working to begin with.


Fix typo in some NUCLEO uvproj files

Hello,
I fixed typo in some NUCLEO uvproj files.
AfterMake RunUserProg1 options are now enabled.
Updated boards lists are shown below.
F042K6,F302R8,F303K8,F303RE,F334R8
Export .bin file names are now Fixed.
Updated boards lists are shown below.
F042K6,F303K8,F303RE
Regards,
Tiryoh

Can you please sign http://developer.mbed.org/contributor_agreement/ and share your nick on mbed.org?
This looks good to me.

OK, I signed it.
My username on mbed.org is "Tiryoh".


Documentation mistake?

Now
err := db.NewSelect().Model(&users).Scan(ctx)
res, err := db.NewInsert().Model(&users).Exec(ctx)
res, err := pg.NewUpdate().Model(&users).WherePK().Exec(ctx)
res, err := pg.NewDelete().Model(&users).WherePK().Exec(ctx)

Is pg. meant to be in the New section?
Is the documentation website for Bun on github at all?

Thanks, should be fixed now. Docs are not on GItHub yet (but they will eventually). For now feel free to post issues you find here or on Discord.


Custom error handling and compile error

Hi,
I'm checking the example on https://grip-framework.github.io/docs/error_handling/ and when I run the test app, I get the next error:
There was a problem expanding macro 'error'
Code in src/napake.cr:39:5
39 | error 403, ForbiddenController
^
Called macro defined in lib/grip/src/grip/dsl/macros.cr:82:7
82 | macro error(error_code, resource)
Which expanded to:

1 | @exception_handler.handlers[{{error_code}}] = {{resource}}.new
^---------
Error: undefined macro variable 'error_code'

I also tested the code(custom error pages) with my old examples from version 1.1.1 and the results are practically the same.
br, uz

I don't even know how this slipped through, I added macro tests to make sure it works next time.
Thank you 👍


test/k8sT: skip default deny egress tests for kubernetes 1.7

Since kubernetes 1.7 doesn't support default deny for egress in 1.7 we
need to skip the test for that version.
Signed-off-by: André Martins<EMAIL_ADDRESS>

test-nightly

test-nightly

test-me-please

test-nightly

Hi @aanm this can be merged. Thanks for the work


sbd payload

In response to #19. Reimplement as it was before. Since the payload does not necessarily conform with UTF-8, there is a chance that it can't be decoded as such, in which case would print an error.
@gadomski , I tried to mimic the style/approach from the original code.

Nice, thank you!


About train_dataset and val_dataset.

Hi,thanks for your great job here,and share your amazing codes here.
I am little confused about your dataset in your paper,said "We use the irregular mask from the previous method (Liu et al. 2018) for mask generation with holes up to 25% following HiFill (Yi et al. 2020)." But i can not find the setting in these  papers or their codes.Would you please share your train and test dataset details? That would really helps us a lot.
Looking forward to your reply!

Thanks for your interesting in our work.
The irregular mask (Liu et al. 2018) can be found at https://nv-adlr.github.io/publication/partialconv-inpainting. And the setting of mask ratio for high-resolution image inpainting can be found in Section 4.3 (Yi et al. 2020). As for the code, you can generate custom dataset via creating a mask-generator function using OpenCV to generate the desired mask.

total understand,thanks lot!


How to configure the team dashboards widgets based on the code committing version number

Hi All,
How to configure the team dashboard widgets (Jenkins, Sonar, Bitbucket and Version one) based on the version number which we are building in Jenkins(ex: 0.0.1-SNAPSHOT,0.0.5-RELEASE). Can some one please guide me on this.
Thanks & Reagards,
Prince Vijayakumar

@princevijayakumar Closed. You'll need to run the collectors in question and populate the data which can then be selected in the team dashboard. Refer to http://capitalone.io/Hygieia for how to get started.


[SPARK-45758][SQL] Introduce a mapper for hadoop compression codecs

What changes were proposed in this pull request?
Currently, Spark supported partial Hadoop compression codecs, but the Hadoop supported compression codecs and spark supported are not completely one-on-one due to Spark introduce two fake compression codecs none and uncompress.
There are a lot of magic strings copy from Hadoop compression codecs. This issue lead to developers need to manually maintain its consistency. It is easy to make mistakes and reduce development efficiency.
Why are the changes needed?
Let developers easy to use Hadoop compression codecs.
Does this PR introduce any user-facing change?
'No'.
Introduce a new class.
How was this patch tested?
Exists test cases.
Was this patch authored or co-authored using generative AI tooling?
'No'.

ping @dongjoon-hyun @srowen @LuciferYang

The GA failure is unrelated.

The GA failure is unrelated

Merged to master.
@LuciferYang @pan3793 Thank you!


helm: add missing env variables to cronjobs

Add missing environment variables that caused cronjobs to fail to
connect to the database when REANA is deployed with a non-default
namespace or prefix.
Closes #711

How to test:

Deploy REANA locally with non-default namespace and/or prefix
Trigger resource-quota-update cronjob

The cronjob should run successfully, instead of having database connection issues


Sample using qdrant and semantic kernel

This would enable us to show OpenAI, along with the ability for an Aspire app to pull in a containerized feature set like qdrant. I have this working in a larger-than-a-sample-sized app we could scale back for customers to see opportunities for Aspire and OpenAI.

So the demo shown in DotNet conf 2023 with Teams bot searching github issues stored in Qdrant and answers formatted in gpt-4 is not available anywhere? Would really love to get my hands onto that although I do not need teams bot necessarily, but just need the semantic kernel example. Would be fun to test Aspire too while at it.

And I would think it is better just make the code available and worry about the size and simplicity later as I'm sure people who need some stuff from there would be able to simplify it them selves. How about storing these as sample drafts or something?

@luisquintanilla Tagging you here for the Qdrant demo from .NET Conference using new Qdrant SDK C# (and hopefully Qdrant Azure Add-on)

Thanks for the tag @tawalke
We were cleaning up that sample. Here's the sample shown at .NET Conf using Aspire, Qdrant, and Teams
https://github.com/Azure-Samples/openai/tree/main/End_to_end_Solutions/GithubRepoAssistant
cc @maryamariyan


Id not set on Call resource object by Notification processor with latest beta packages

Describe the issue
We upgraded to latest Graph SDK packages and the biggest change for us is that they moved from NewtonSoft.Json to System.Text.Json. This is all fine with us, however the Call.Id property is not populated anymore from the notification requests, causing our application to be unable to handle the updates properly.
Expected behavior
Call.Id populated by the NotificationProcessor for all notifications.
Graph SDK:

Microsoft.Graph.Communications.Core <IP_ADDRESS>09 (beta)

I don't know if you're problem is related to that but I also had problems during my dependency chores and it turns out the maximum version of Microsoft.Graph that seems to be currently supported is 4.17.0

Also Graph 4.17 seems to have an issue, that the "additionalData" field is null. In Graph 4.7 this is a JsonElement (and previously it was an JObject). What is the latest supported Graph version?

The latest version works, you need to do some code changes to support it:
https://github.com/microsoftgraph/msgraph-sdk-dotnet/blob/feature/2.0/docs/upgrade-to-v4.md


Fix config name at TenantScopedModelTrait

When publishing with L5, the config file is named tenant, so it should be used to get the config, instead of multi-tenant, which returns nothing in this case.
Copied File [/vendor/aura-is-here/laravel-multi-tenant/src/config/config.php] To [/config/tenant.php]

This is right! Please merge!

Solved my issue after composer-update with the new release.


Circleci: Add workflow, and tag build

The Problem:
We need/want automatic building on tag push
The Fix:
Add "workflows" and configure as such. It is not a major win for us, but it does do these two things (hopefully)
Note that before pull this still requires:

[x] Removal of the test build step that was showing the token's existence
[x] Removal of the tag_build.sh script as it's hopefully obsolete with this.
[x] Removal of any reference to org-global, as that didn't work out.

Looks like this PR might go stale. Any chance to get reviewed/merged today?


Allow owner to define lifecycle status for an API

Add lifecycle status field to API schema and allow owner (and admin) to define lifecycle status for an API.
User story

As an API owner,
I want to add lifecycle status information for my API
so that I can inform the API users of the API status

Definition of done

[ ] API schema contains a field for lifecycle status
[ ] The allowed values are predefined: design, development, testing, production, deprecated

[ ] The values are internationalized

[ ] Owner can add the lifecycle status

[ ] in Add API form
[ ] in API profile (@Nazarah: Please consider where to add the status)

Wireframes

(in case adding a status is mandatory)
API owner can include a lifecycle status when they are adding an API.

In this case, only the options Design, Development, Testing and Production statuses are available in the selection.
Since we have 5 statuses till now, we can color code them. E.g. Design -> light blue, Development -> dark blue, Testing -> orange, Production -> green and Depricated -> light grey

This is a feedback from @kyyberi  on issue #18

User can see lifecycle status of an API on its profile page (top portion)

On clicking the status, a widget control allows API owner to change it status from the available ones. There is no restriction on maintaining chronological order of the status. I.E. API owner can switch back an forth between statuses.
(this sort of editing style has been used in Dipor, so we can reuse this)

The time a status gets changed from the profile (i.e. not the time an API was added with a lifecycle status), the information about changing date, time and from which status appears beside the label. (alt. design: showing it as a tooltip that appears on hovering over the label)

In case selecting lifecycle phase is not mandatory when adding an API, we can always keep an option about changing the life cycle status from the settings tab.
Even in this case, on saving a lifecycle here would make the lifecycle appear at the top part of API profile.
The widget based editing should also be available for quick access.

Open issue: Should lifecycle status be a required field?

@Nazarah and @brylie I confimed with @kyyberi that this is a valid list for lifecycle statuses: design, development, testing, production, deprecated


"TypeError: Cannot read property 'apply' of undefined" when handlebars version > 4.6.0

After the release of<EMAIL_ADDRESS>(2nd week of January 2020) I get a "TypeError: Cannot read property 'apply' of undefined":
{code}
TypeError: Cannot read property 'apply' of undefined
at Object.wrapper (/myProject/node_modules/handlebars/lib/handlebars/internal/wrapHelper.js:5:19)
at Object.eval (eval at createFunctionContext (/myProject/node_modules/handlebars/lib/handlebars/compiler/javascript-compiler.js:260:23), :13:196)
at main (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:230:22)
at ret (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:250:12)
at ret (/myProject/node_modules/handlebars/lib/handlebars/compiler/compiler.js:548:21)
at Object.invokePartial (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:405:12)
at Object.invokePartialWrapper [as invokePartial] (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:82:39)
at Object.eval (eval at createFunctionContext (/myProject/node_modules/handlebars/lib/handlebars/compiler/javascript-compiler.js:260:23), :15:28)
at main (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:230:22)
at ret (/myProject/node_modules/handlebars/lib/handlebars/runtime.js:250:12)
at ret (/myProject/node_modules/handlebars/lib/handlebars/compiler/compiler.js:548:21)
at ExpressHandlebars._renderTemplate (/myProject/node_modules/express-handlebars/lib/express-handlebars.js:250:12)
at ExpressHandlebars. (/myProject/node_modules/express-handlebars/lib/express-handlebars.js:173:21)
{code}
There are some breaking changes in the<EMAIL_ADDRESS>release: https://github.com/wycats/handlebars.js/commit/770d746e600eb1939501ab91d4bc81ed6b4cde94

Sorry, it's more likely an issue for express-handlebars...

Please add details.
The function in your stack trace is new. It may be a problem here.

This issue should be fixed in Handlebars 4.7.2.


Zero AP when train custom dataset

Hi, I trained my custom dataset consisting of 199 lidar, calib, and label files. I specifically train for Pedestrian detection. I have followed all guide regarding custom dataset such as x, y, z conversion, shifting center height to around 1.7 meter, check my bin generated from  python -m pcdet.datasets.kitti.kitti_dataset create_kitti_infos tools/cfgs/dataset_configs/kitti_dataset.yaml tools/cfgs/dataset_configs/kitti_dataset.yaml  , changing FOV in cfgs to False to detect all surrounding points and labels. Everything is okay. But the result of training is 0 AP. This is the last output of the training:
epochs:  99%|████████████▊| 79/80 [24:08<00:18, 18.58s/it, loss=nan, lr=3.13e-8]WARNING:root:NaN or Inf found in input tensor.8<00:00,  3.09it/s, total_it=4000]
WARNING:root:NaN or Inf found in input tensor.
WARNING:root:NaN or Inf found in input tensor.
WARNING:root:NaN or Inf found in input tensor.
WARNING:root:NaN or Inf found in input tensor.
WARNING:root:NaN or Inf found in input tensor.
train: 100%|█████████████████████| 50/50 [00:18<00:00,  2.66it/s, total_it=4000]
epochs: 100%|█████████████| 80/80 [24:08<00:00, 18.11s/it, loss=nan, lr=3.13e-8]
2021-11-10 00:19:26,591   INFO  End training kitti_models/pointpillar(default)
INFO:pcdet.utils.common_utils:End training kitti_models/pointpillar(default)
INFO:pcdet.utils.common_utils:*************** EPOCH 80 EVALUATION *****************
eval: 100%|███████████| 50/50 [00:05<00:00,  8.97it/s, recall_0.3=(0, 0) / 1173]
2021-11-10 00:20:37,044   INFO  *************** Performance of EPOCH 80 *****************
INFO:pcdet.utils.common_utils:*************** Performance of EPOCH 80 *****************
2021-11-10 00:20:37,044   INFO  Generate label finished(sec_per_example: 0.0281 second).

INFO:pcdet.utils.common_utils:Evaluation done.*
2021-11-10 00:20:37,160   INFO  Epoch 80 has been evaluated
INFO:pcdet.utils.common_utils:Epoch 80 has been evaluated
Wait 30 seconds for next check (progress: 0.0 / 0 minutes): /home/yonas/OpenPCDe2021-11-10 00:21:07,189   INFO  End evaluation kitti_models/pointpillar(default)
INFO:pcdet.utils.common_utils:End evaluation kitti_models/pointpillar(default)
Is there something wrong in my dataset? or should I train with more epocs? Any help is appreciated.
Thanks

Hello, have you found the reason?

Hi! have you solved the problem? I meet the same problem with you. Please give me some clue! Thank you!

I have the same problem, have you find a solution yet?

hello,do you find a solution?Thanks

Hi @liyn69sysu @giangdip2410 @zjr-bit @curiousboy20 @ferrari700 ,
I am facing the same issue with my custom dataset. Could you figure out the reason for this?
Thanks.

@AJAY31797 Did you find a solution?


Make Resource view nodes clickable

Resolves #2663 by making resource view nodes selectable. Ideally, we would also zoom into the selected node like we do in the graph view, but the animation there is a bit more involved, so it shouldn't be a blocker for this action that feels very basic.

@davkal ping

I played with it and it works well!


Simple removal test on own image

Hello,
I would like to test your model of shadow removal on a random image but I am a bit confused where to save the image as well as which command to type in.
For now, I saved the image under datasets/test/, but when I run
python test.py --dataset_root ../datasets/
I get the following error:
AssertionError: ../datasets/test/input is not a valid directory
If I create an input directory, then the program is asking me for a target directory and a maskdirectory. Both should contain the same number of elements as the input directory.
But if I test on a random image, I do not have any target or mask, so I do not know how to proceed. Could you please help me?
Thanks!
Guillaume

Hi! Unfortunately, the server I use is under emergency maintenance, I cannot push the codes.
The easiest workaround will be as follows:

put shadow detection results in <PRECOMP_MASK_NAME> (Note that SP+M model assumes access to pre-computed mask)
put a symbolic link from target to input and from mask to input directory. Images in each folder might be loaded but do not affect the test process, I hope.
try python test.py --dataset_root ../datasets/<DATASET_NAME> --name <IDENTIFIER> --mask_to_G <PRECOMP_MASK_NAME> --mask_to_G_thresh 0.95

Hi! Unfortunately, the server I use is under emergency maintenance, I cannot push the codes.
The easiest workaround will be as follows:

put shadow detection results in <PRECOMP_MASK_NAME> (Note that SP+M model assumes access to pre-computed mask)
put a symbolic link from target to input and from mask to input directory. Images in each folder might be loaded but do not affect the test process, I hope.
try python test.py --dataset_root ../datasets/<DATASET_NAME> --name <IDENTIFIER> --mask_to_G <PRECOMP_MASK_NAME> --mask_to_G_thresh 0.95

I have uploaded the codes. You can use --dataset_mode demo to use the simplified dataset class. For more details, please refer to #3.

I have uploaded the codes. You can use --dataset_mode demo to use the simplified dataset class. For more details, please refer to #3.


The Dashboard's chart can't export to other document(Excel, PDF ....)

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.20.4
Expected results
the charts in Dashboard can export into Excel or PDF document
Actual results
I can't find the way to complete this function
Steps to reproduce

Dup of #2995 please close


Permissions are not being set with env variables

Pigging back on to #461 and #463. Does not look it's honoring the FILES_GID and FILES_UID environmental variables, as new files are being created as root. Made sure I was using the latest image.
Docker inspect info: https://pastebin.com/ZKiy7jcD
Thanks.

Do you have the container's log output ?
What OS and docker version are you using ? Anything specific on your host like SELinux or AppArmor?

Just the standard log before all the certs info.
Info: Custom Diffie-Hellman group found, generation skipped.
Reloading nginx proxy (735fda5135b676609d52e57749c6e7879449d3263542a6a99f0183e6ddb5f7be)...
2018/11/23 08:28:32 Contents of /etc/nginx/conf.d/default.conf did not change. Skipping notification ''
2018/11/23 08:28:32 [notice] 91#91: signal process started
2018/11/23 08:28:36 Generated '/app/letsencrypt_service_data' from 18 containers
2018/11/23 08:28:36 Running '/app/signal_le_service'
2018/11/23 08:28:36 Watching docker events
2018/11/23 08:28:36 Contents of /app/letsencrypt_service_data did not change. Skipping notification '/app/signal_le_service'
```

Server: Ubuntu 16.04 LTS with 4.18.6 kernel. No SELinux or AppArmor enabled.

Debugging info about the permissions changes are in the middle of the certs info.

I dont see any such info. Even with debug env set to true.
Do still see this: Debugging tips: -v improves output verbosity. Help is available under --help..

Are you sure you are using an up to date version of the container ?

Removed the containers and images and pulled and rebuilt.
docker images:
jrcs/letsencrypt-nginx-proxy-companion   latest              56cc2f664c54        10 hours ago        84.8MB

docker inspect:
sha256:56cc2f664c54a2fb587c21969065ab053c4c95aace43ad0d9a6e195b8a036041

same issue here. thanks @desimaniac for tracking down the bug.

@techoutlooks > OS, Docker version, LE companion version, config, container logs ?
@desimaniac > my bad, you don't have debug output enabled at all. The value you have to set the DEBUG environment variable to is true, not True. I'll make this case insensitive asap.

Ok set true as debug... created a new subdomain: strm.domain.com
Debug: setting /etc/nginx/certs/strm.domain.com/.companion ownership to 1000:1000.
Debug: setting /etc/nginx/certs/strm.domain.com/cert.pem ownership to 1000:1000.
Debug: setting /etc/nginx/certs/strm.domain.com/key.pem ownership to 1000:1000.
Debug: setting /etc/nginx/certs/strm.domain.com/chain.pem ownership to 1000:1000.
Debug: setting /etc/nginx/certs/strm.domain.com/fullchain.pem ownership to 1000:1000.

It does say that it's changing ownership, but only the folder is owned by 1000, but the files are still owned by root.

Okay, I see. Are you experiencing issues because of the symlinks being owned by root:root ?

Yes, I believe so.

Can you describe more precisely the issue your are running into because of the symlink ownership ?
From the test I just did, symlinks ownership does not appear to matter : even an unprivileged user can mv or rm a symlink owned by root:root. That was tested on Alpine, Ubuntu and Manjaro.

Using Ubuntu. I am able to rm fine. I guess that was my only issue. rm ing or tarring up / untarring, etc.

Could you test it to be sure it's not impacting you in your intended usage of the files ?

Tested tarring. Not an issue. So i guess everything is working within parameters 😛 .

Sure it's working within parameter but the fact you thought this was a bug means not changing symlinks ownership might cause confusion.
@techoutlooks is your issue about symlinks too ?

Yeah that's true. Any way the permissions on symlinks can also be set ?

Yes, I'll look into this. Not that hard, but it'll require to use core utils instead of their busybox equivalent.


Improve color accessibility

Thanks for the great extension!
But I have a slight color weakness, so the red/green color scheme is a little difficult for me. It would be much better with a blue/red combination.

If I ever find the time I'll look into introducing more distinction than just switch colors. Current scheme has the same shade for both colors, this lacks contrast.

I came up with this design. Let me know what you think. I'd personally get rid of the "CORS" text and rely just on icons and their colors :)

Looking good. However:

We're not losing the text. Clean, explicit icons are always better. I actually wish we could keep the E in there (to differentiate the addon from the general concept of CORS) but eh, it's getting cramped I guess.
We need stronger brightness differentiation. Very light when disabled, much darker when enabled. Filter mode is technically enabled but a medium shade might be preferable. They should be reasonably distinguishable when completely desaturated.
I wonder if the shield could be misinterpreted as being "protected" from CORS. But I have no idea for an alternative that would basically describe that the "floodgates are open".

@spenibus Thanks for the invaluable feedback. I am going to reiterate on them a little when I have some more time. It's really hard to come up with something in such small space. Losing the text would make it much more readable, but I get your point. Let me come back with this soon.


[CHORE] Readme

Describe the chore
Complete the README with examples.
Additional context
Add any other context about the problem here.

Closed


fix: round dimensions to 4 places

This PR revives PR #1747, which has stalled. It rounds distances in emitted KaTeX HTML to 4 decimal places.

Codecov Report

Merging #2460 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2460   +/-   ##
=======================================
  Coverage   93.82%   93.83%
=======================================
  Files          84       84
  Lines        6110     6114    +4
  Branches     1254     1254
=======================================
+ Hits         5733     5737    +4
  Misses        346      346
  Partials       31       31

Impacted Files
Coverage Δ

src/buildCommon.js
97.49% <100.00%> (+0.01%)
:arrow_up:

src/buildHTML.js
96.38% <100.00%> (ø)

src/mathMLTree.js
88.46% <100.00%> (ø)

src/units.js
95.65% <100.00%> (ø)

src/utils.js
94.59% <100.00%> (+0.30%)
:arrow_up:

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 c87f9e6...34d5744. Read the comment docs.

found way more than are addressed in this PR. Those should probably be wrapped in calls to utils.round as well.

No doubt that's true, but I wasn't really trying to reach rounding perfection in this PR. I was just trying to not lose the ground gained by the original PR. I'm hoping that the other dimensions can be left to be done later.

I think we can create a util function, makeEm, which automatically calls round and appends em unit.

Thank you for the PR and sorry for the delayed review. Please find attached the screenshot diffs: diff.zip. I don't see significant changes, but requesting another set of eyes.

Thank you!

Also note that this should be labeled BREAKING CHANGE.

I don't think this would be a breaking change, since this doesn't change how KaTeX is used.

I don't think this would be a breaking change, since this doesn't change how KaTeX is used.

All inputs generate different outputs, and it's not exactly fixing a bug... I'd say it could break things for people. But too late now. 🙂


OPCT-23 - fix/waiter: unblock waiter when previous job failed

This PR prevents the blocker plugin to wait longer (until timeout) to run when the blocker plugin has been failed.
https://issues.redhat.com/browse/OPCT-23

/lgtm
/approve


Colors off and not bright at feed

Hello, love this project but I noticed color feed doesn't match what is on the screen, is off by a lot and brightens doesn't give maximum. When my lights just gets on in standby mode and waiting for app to start and I'm getting color cycle - all color looks bright and rainbow. But when software starts it looks like only 50% brightens and color is off - for example if screen image is yellow - it shoes like red or pink, if its green it looks more like blueish, if its white - looks like blue.
On soft I have all color correction at 100% but it is still doesn't give same brightens as standby mode
Do u guys have same issue.
I can do video to show.
Any suggestions?

Please upload your adrilight config file and maybe some photos where the
color shift is most obvious?

The file may be found here C:\Users\your
username\AppData\Local\adrilight\adrilight-settings.json

adrilight-settings.zip

Here is stand by mode:
https://youtu.be/WwwM72Z7kJo
Here is actual data feed to see brightnes difference and color distribution(im posting more photos as well):
https://youtu.be/3keQDS5fGYQ

Photos.zip
Here is photos white different color combinations

Thanks for helping and looking into it ;)

it looks like it uses way to much blue in all it does but the settings say, you have not touched the white balance.
Things to check:

Open the Preview-Tab in the settings window. Does the preview of your screen there look like the actual screen or is there already a color shift?
Check the code you flashed on your arduino. did you change something there?
try adjusting the white balance by reducing blue by a lot, this might be it already. the color reproduction is not that great in most led strips

regarding the brightness, i cannot really tell something is wrong actually. there is a brightness setting in the arduino code, but it affects the demo mode as well so there should not be any difference between the modes.
https://github.com/fabsenet/adrilight/blob/ef7873ecc6f344fc54607aaacc478294ab738ac4/Arduino/adrilight/adrilight.ino#L5
did you wired in more than one power input into the strips?

I have 1 power adapter to power strip and data contacts to Arduino board. But if there would be any issues with power that would effect demo mode as well, right?
I did video for performance page but I don't understand how to use that page.
https://youtu.be/WmFp92zpfM8
Can you do screen or video of explaining?
Thank you

The preview page looks fine.
please post the arduino code you flashed on the arduino and provide a url or photo or something, what led strip you bought.
this is sooo weird! at least after turning the white balance on blue to zero, the blue should be gone completely!

Hi, is there is possiblity something wrong with Arduino library - but than im thinking would be same issue with demo mode! Demo mode colors looks awesome and correct but feed too much blue like you said!
here are my code and extra info
Photos(20).zip
.

I think i have this strip:
https://www.aliexpress.com/item/32481030210.html?spm=a2g0o.90423<IP_ADDRESS>424c4dzNotgL

original project was led build from this instructions: https://www.instructables.com/DIY-Ambilight-Using-Arduino-Nano/

Hi, I figure out...my 2811 color led code wasn't build the same and I just swap code and now all works. Thanks you

https://github.com/FastLED/FastLED/wiki/Rgb-calibration
Swap GRB to BRG and that's it!


Update preview

Hello good sire!

I think you should update it!

Resolved in 1765a12. Please, check it out!

That will do! Thanks!


The H1 element text "I am blue!" changed color on the screen, but the check list says that it is not changing color

Challenge Use RGB to Mix Colors has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
The color of the font changed like it should on the screen, but the tasks are saying that it hasn't yet.
My code:

I am red!

I am orchid!

I am sienna!

I am blue!

@ShelbyF73 The color for blue is 255, not 225, if you refer the table on the left.
If you need help with challenges like these, please visit the Help Room first. Happy Coding!

@ShelbyF73 Or, if you want a more relaxed help environment - check out the forum.
Good luck! :blush:

@Manish-Giri I didn't even see that typo lol! Thanks for pointing that out. It's still weird that 225 changed the same text to same shade of blue as 255.


check if maintenance mode is enabled and suspend cron processes

fixed in https://github.com/magemojo/m2-ce-cron/commit/bd433ae292f25abccb63bae1fb21c94cdbbf2f75


import boxing

Right now imports dump everything into the global space. Perhaps we should change this to create a box and then stuff everything in that box kind of like "use" does.
This would give the advantage of sectioning off everything imported.
Need to keep in mind that we don't import things multiple times right now. Need to determine a way to cache imported boxes so upon a second import we can just give a shared pointer to the importer with access to the box in their own env

We already have a caching mechanism ... it IS env.... just do this. it will make sense

Realistically we should merge use and import into one command. Using some FS stuff we can determine if we will import a file or a directory. Then we can ensure that the directory is a pkg or not. Single files would be loaded just like the source_files in pkgs.
This is  a great idea.

After deep investigations I've decided that this is not behavior that I want to create at this point in time


import @hashgraph/cryptography issue on Vue.js

sdk version: 2.0.8
node.js version: 12.14.0
vue version: 2.5.2
I want use @hashgraph/sdk on javascript client side, just like MyHbarWallet.
I create environment Vue.js and install @hashgraph/sdk version2. (MyHbarWallet is currently version1)
After, I ran 'npm run dev' but I got error
`This dependency was not found:

@hashgraph/cryptography in<EMAIL_ADDRESS><EMAIL_ADDRESS>and 7 others

To install it, you can run: npm install --save @hashgraph/cryptography`
But @hashgraph/sdk has dependency with @hashgraph/cryptography. So that installed @hashgraph/cryptography. But the module_loader can not find the module.
What can I do?
How can I use sdk version2 on javascript client side?
Any comments?
Thank you.

It is fixed.


ansi_colours dependency introduces LGPL-3.0 code

Hi! First of all: I am not a lawyer.
This commit broke our CI, because ansi_colour is introduced as a dependency of this crate.
Problem here is that ansi_colour is LGPL-3.0 licensed. To my understanding, this crate (coloor) must be LGPL-3.0 as well now, as well as all crates that depend on coloor, because of static linking.
I filed a similar issue in the termimad repository because that's the direct dependency I found the issue with.

This has been solved a long time ago but I forgot to close the issue


Feature request: Option on whether or not to use dupes

Just wanted to put in a request for a prompt on whether or not a dupe is used to save a character. A relevant scenario would be if I have a duped Jorah with 2 betrayal tokens on him and I used Waking the Dragon to stand him during the challenge phase. At the end of the phase I actually want the Waking the Dragon to send him back to my hand. Thanks!

In fact, players will be prompted before using duplicates to save, but only if other options for saving a character exist (bodyguard, iron mines, etc.) This has been implemented by @ystros in #594 .
This request, which is totally valid, is hence something like "prompt before saving a character that can only be saved by a duplicate".
I agree it makes sense, and won't probably be much of a burden overall in terms of user experience.
If others thinks it would be, this is another good candidate for user-configuration options that would generalize what we already have for optional action windows.

It would be useful to have this implemented for situations where a duplicated character is controlled by another player (e.g. Warded) when it is being forced to leave play. Whether to save it or not is a legit question.


fix: properly expand absolute path placeholder for link calls

What type of PR is this?

Bug fix

What does this PR do? Why is it needed?
PR #4009 added a mechanism that uses a sentinel value as a placeholder for paths which need to be expanded to absolutes for compile operations. These paths end up embedded in .a files and come back to the linker during link operations. The previous PR replaces the CC tool with a pointer to builder cc which in turn substitutes the placeholder value for actual paths. Unfortunately this wasn't done for the linker, so the linker is receiving parameters like --sysroot=__GO_BAZEL_CC_PLACEHOLDER__external/_main~_repo_rules~ubuntu_20_04_sysroot/ which is not valid. When a sysroot is configured this means that internally the linker won't be able to find the sysroot in some cases.
In almost comically bad luck, this only affects the go tool link calls to linkerFlagSupported because for some reason linkerFlagSupported puts the -extldflags at the beginning of the linker command, while the main host link command has them at the end. So this means the linker detects all flags as unsupported but then proceeds to run the linker "correctly" which results in really confusing linker errors. In my case this resulted in the linker trying to run with -rdynamic incorrectly.
Which issues(s) does this PR fix?
Fixes #3886
It could be reasonable to open a bug against go tool link describing the difference between the host link and the flag checks. I suspect it's not really intended behavior and was just sort of happenstance that things ended up that way.
I also really have no idea how to write a test for this. I'm willing to give it a shot but would really appreciate a suggestion as to where/how.

Clearly this naive approach won't work on windows, I guess we could detect the OS write an OS specific script? This all feels pretty messy so i'm hoping someone out there has ideas for a better solution.

One idea for a different approach would be to use a symlink named something like builder-cc and have the builder inspect os.Argv[0] to set the command.

I've changed this to use a symlink approach instead of a script which seems to be more portable.

Could you add a test for this under tests/core/cgo? It doesn't have to be a full end-to-end test, but perhaps you could set some --copts that are turned into absolute paths and assert on the build output with -s? That way you don't have to set up an actual sysroot in the test.

Sure, I will do both.
I honestly think even if an upstream issue is fixed there is still technically a bug here, we just won't hit the bug for sysroots because the last one will take precedence but it is conceivable that other parameters with paths wouldn't work the same way and we would end up with some new subtle bug.

It looks like someone else already filed an issue relating to this:
https://github.com/golang/go/issues/69350

@fmeum I've added a test. I think the test most likely only works on platforms with gcc-ish compilers (I have no idea if this bug really even exists on windows and have no way to test unfortunately) so I added a build constraint for !windows.

actually it seems the test passes without the build constraint and fails with it so I'll leave it alone for now. I'm assuming the windows test must still use gcc

I think the build failure is in an unrelated test, maybe a fluke? Either way I'm unable to trigger a retry without pushing an update.

@fmeum I believe this may be ready. I addressed one comment slightly differently than you asked so I let it unresolved but I think the issue is fixed. Let me know if you need me to do anything else on this one.

Thanks for the very thorough execution!


Fixes for 2023-04-01-preview API version

Data Plane API - Pull Request

Fixes for new API version 2023-04-01, a continuation of this PR - https://github.com/Azure/azure-rest-api-specs/pull/23265
Renaming shapeFileType to shapeType and shapeFileResolution to shapeResolution post bugbash and discussion with PMs.

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your engagement record.

Link to engagement record issue: https://github.com/Azure/azure-rest-api-specs/issues/23007

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous Open API document (swagger) if applicable, and the root paths that have been updated.

Design Document:
Previous Open API Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Approved breaking changes as this version has not yet been make available to customers.


Rename bionano to Life Sciences

Issue https://github.com/Autodesk/molecular-simulation-tools/issues/300
Some notable things that I DID NOT change:

Link to our blog, which still seems to include bionano for the near future.
URLs to several internal services that seem to still include bionano

@dionjwa You should probably run ag bionano and just do a double check that none of those services I mentioned should have their names changed.  They all look like they'd never be exposed to the user to me, though.

Looks good. Yeah all the others are external or internal services that still are using bionano for the foreseeable future (quay.io/blog/aws internal DNS entries).


Suggest the maven wrapper if the build was executed using it

When the formatting fails, it produces the following hardcoded message:
[ERROR] There are .editorconfig violations. You may want to run
[ERROR]
[ERROR]     mvn editorconfig:format
[ERROR]
[ERROR] to fix them automagically.

This is misleading for projects that use the Maven Wrapper
What It should do instead is to suggest:

Linux/Mac/Other Unix systems:

./mvnw editorconfig:format

2)Windows

.\mvnw.cmd  editorconfig:format

Hm... I wonder whether there is a programmatic way to figure out how the Maven process was launched?


Candrabindu within consonant stacks

The cluster structure does not document support for candrabindu within a consonant stack.  This has been implemented to support the use of candrabindu to show that the first consonant of a cluster is nasalised.

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

ID: 7c1a0129-e23f-c785-89bc-25fb41b378c0
Version Independent ID: b2a39f6a-94ed-b3ff-effb-17b2cc31b9e1
Content: Developing OpenType Fonts for Devanagari Script - Typography
Content Source: typographydocs/script-development/devanagari.md
Product: typography
GitHub Login: @alib-ms
Microsoft Alias: alib

If there is no visual distinction between applying a candrabindu to the first consonant and to the last consonant, the Unicode order is to encode the candrabindu last.

What you're saying is bizarre, and doesn't gel with the contrast between, say, ल्ँला <U+0932 LA, U+094D VIRAMA, U+0901 SIGN CANDRABINDU,  U+0932, U+093E SIGN AA> ("l̐lā", one akshara in Microsoft Windows 10 renderers, though probably needing a capable font, e.g. Nirmala UI, nasalised first consonant) and ल्लाँ <U+0932, U+094D, U+0932, U+093E, U+0901> ("llā̃", nasalised vowel).  Having the nasalisation on the last consonant does not, so far as I am aware, occur.  Now, sometimes, the consonants, in particular, LA, may be stacked vertically. in which case you are saying that the encoding depends on whether the [i]font[/i] would choose to stack the consonants!  However, even with such a conjunct, "l̐lā" and llā̃ would still be visually distinct.
Now, you may have meant, "If there is no visual distinction between applying a candrabindu to the first consonant and to the [i]vowel[/i], the Unicode order is to encode the candrabindu last."  However, if the font, like Nirmala UI, does not yield a vertical stack for L.LA, then the two forms would still be distinct even if the vowel be implicit or below.
I append the Nirmala UI rendering contrasts from IE 11 on Windows 10:

The vowels are implicit, AA, E and I.

This is a supported behaviour in our Indic engine that I need to document. I've assigned to myself for action.


Incompatible manifest version prevents installation on Firefox

Firefox does not support manifest version 3, so as a result this extension can't be installed even on version 107.0.1, the latest available.

I am not willing to downgrade to V2 manifest, if you have another solution I am open.
In addition if you download the extension via the file download link on the Firefox store it will allow install.

No, actually, it won't, and I tried that before I opened this issue. Either don't ship it on Firefox at all, write a Greasemonkey script, or put up with having to ship it with manifest version on Firefox.

Several people including myself have had success with the method I described above.
I will continue to monitor the FF v3 manifest situation. For me downgrading is an inefficient use of my time.
If this is a blocker for you then my code is under the no license license and can be modified and released as you see fit.
Always happy to work on new features or bug fixes as I have time but downgrades and backwards compatibility are not priorities of mine.

Just adding my voice that the manual download approach doesn't seem to work for me either (same "appears to be corrupt" message). Perhaps you can explain which approach you used to install it on firefox?


fix musl compilation

this header is not available.

And it is absolutely not necessary.
Thanks. Merged.


derive: use private const trick instead of modules

It looks like prost generates modules to contain impls.
The best strategy that I have found (by reading serde's source) is to use a private module to contain impls. This has the best hygiene. You can see how it is done here.

Full disclosure, doing this means you must refer to all types using an absolute path as use statements don't seem to work. It is annoying, but the hygiene makes it worth it IMO.

If this sounds good to you, I can work on a PR.

Fixed in #185.


Allow Application and Bookmark to provide http[s]

"https-only" application will work only if a automatic redirect is in place (somehow a good practice).
Is there any reason to not allow users to define proper protocol? (which could even be something other than http[s])

You can read this solved isssues. Right now don't have a proper parser for URL.
https://github.com/pawelmalak/flame/issues/5

Really the same as #5
Closing as duplicated, sorry

I'm confused if this was fixed on that pull request or not
IF not, I suggest to reopen this and flag it as enhancement