Error: Failed to remove private key

Description:
When attempting to deploy to a maven repository following the configuration documentation [1] the deploy action is successful but the action fails consistently on the "Post" step with output:
Post job cleanup.
Removing private key from keychain
Error: Failed to remove private key

The setup-java code where the attempt to remove the key occurs here: https://github.com/actions/setup-java/blob/v2.0.0/src/cleanup-java.ts. Note that the error is swallowed, making it harder to know why the failure is occurring.
[1] https://github.com/actions/setup-java/blob/main/docs/advanced-usage.md#Publishing-using-Apache-Maven
Task version:
actions/setup-java@v2
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Repro steps:
A description with steps to reproduce the issue. If your have a public example or repo to share, please provide the link.
Github actions workflow: https://github.com/duracloud/duracloud-db/blob/4afd1ac05398ff95c6c3ee88734b0bfa2ad2614f/.github/workflows/deploy-snapshot.yml
An example of a failed run: https://github.com/duracloud/duracloud-db/runs/3205773250
Expected behavior:
A description of what you expected to happen.
The GPG key would be removed and the job would complete successfully.
Actual behavior:
A description of what is actually happening.
An error is printed and the step and job are failed.

Hello @bbranan. Sorry for the long reply. Does it still reproduce for you ?

Hey @dmitry-shibanov, thanks for following up. I was able to resolve the issue by creating and using a new key. It's still not clear what about the old key kept it from being removed in the action, given that it worked for the code signing step. I would still recommend updating the catch() in https://github.com/actions/setup-java/blob/main/src/cleanup-java.ts to print the error being caught, as that may provide additional information about the problem when/if this issue comes up for others.

I also suffer from this problem because my project's GitHub repo has been provisioned with a GPG_PRIVATE_KEY secret which only contains a signing subkey without the primary private key. See https://wiki.debian.org/Subkeys especially the section where it discusses removing the primary private key. The issue then is that when the cleanup for the setup-java action goes to delete the key, the --delete-secret-keys command fails for the fingerprint of the installed GPG_PRIVATE_KEY as there is no secret key with that fingerprint.
Here is some example data showing the issue using a private key I created according to the Debian Subkeys article.
~ gpg --batch --import-options import-show --import subkey.key
sec#  ed25519 2021-09-13 [SC]
      098A3F32AD9899B69806F5B88D6DB9DBDC161CF2
uid                      BJ Hargrave (Signing Subkey<EMAIL_ADDRESS>ssb   cv25519 2021-09-13 [E]
ssb   ed25519 2021-09-13 [S]

gpg: To migrate 'secring.gpg', with each smartcard, run: gpg --card-status
gpg: key 8D6DB9DBDC161CF2: secret key imported
gpg: Total number processed: 1
gpg:               imported: 1
gpg:       secret keys read: 1
gpg:  secret keys unchanged: 1

The imported key has the fingerprint 098A3F32AD9899B69806F5B88D6DB9DBDC161CF2.
Notice the sec# in the output which indicates the primary private key is not present.
So when cleanup for the setup-java action later attempts to --delete-secret-keys in cleanup, it fails:
~ gpg --batch --yes --delete-secret-keys 098A3F32AD9899B69806F5B88D6DB9DBDC161CF2
gpg: key "098A3F32AD9899B69806F5B88D6DB9DBDC161CF2" not found
gpg: 098A3F32AD9899B69806F5B88D6DB9DBDC161CF2: delete key failed: Not found

Note that --delete-keys works fine and all the keys are now deleted.
~ gpg --batch --yes --delete-keys 098A3F32AD9899B69806F5B88D6DB9DBDC161CF2
~ gpg --list-keys
~ gpg --list-secret-keys

So the cleanup for the action needs to properly handle deleting a GPG_PRIVATE_KEY which only contains a signing subkey without the primary private key. This could be accomplished by ignoring any error from the --delete-secret-keys command and proceeding to the --delete-keys command.
cc: @mbarbero

Further reading led me to the option --delete-secret-and-public-key which means both the secret key and public keys can be deleted in a single operation.
~ gpg --batch --yes --delete-secret-and-public-key 098A3F32AD9899B69806F5B88D6DB9DBDC161CF2

Good catch @bjhargrave! I can confirm that the key I was using initially (that failed) was a subkey. The new key I created (that works) is a primary key. Your explanation makes sense of the failure I was seeing and the reason why the new key would work.

Having the same issue here: https://github.com/JarvisCraft/padla/runs/3640842927?check_suite_focus=true
While using the following configuration: https://github.com/JarvisCraft/padla/blob/development/.github/workflows/deploy-snapshot.yml
The suggested PR (#226) seems to be the fix for it.

Hello @bbranan @bjhargrave. Thank you for contributions. We appreciate it.
Could you please confirm that everything works as expected for you ? Could you please for testing purpose specify actions/setup-java@v2 -> actions/setup-java@main.

Hello @bbranan @bjhargrave. Thank you for contributions. We appreciate it. Could you please confirm that everything works as expected for you ? Could you please for testing purpose specify actions/setup-java@v2 -> actions/setup-java@main.

I can confirm that this fix wotks:
Successful run with main as the version
Previous broken run with v2.3.0 as the version
Now just waiting for the fix to be released under a tag!

I'm no longer in a good position to test this as my environment has changed since submitting this ticket, but given the success reported by @bjhargrave and @JarvisCraft I can only conclude that this issue has been resolved by the combination of #220 and #226.

Hello everyone. Thank you for all your responses ! We appreciate it. We released new version of the action.
For now I'm closing the issue. If you have any concerns feel free to ping us.

@dmitry-shibanov I'm still getting this error in salesforce/formula-engine, but I just started using it.  Is there a way to get a better error message than this.
Post job cleanup.
Removing private key from keychain
Error: Failed to remove private key due to: The process '/usr/bin/gpg' failed with exit code 2

Hello @steventamm. Actually we add error message to the core.setFailed command in this pull request. I think it is  full error message you can get in your case.
Could please provide public repository to reproduce the issue and investigate it deeper ?

https://github.com/salesforce/formula-engine

I am suffering from this problem, I followed this tutorial to publish the maven package.
Regardless of whether I set GPG_SIGNING_KEY to a fingerprint, or to a private key exported with gpg --armor --export-secret-keys YOUR_ID, I get the following error
Failed to remove private key due to: The process '/usr/bin/gpg' failed with exit code 2

my release.yml is
name: release
on:
  release:
    types: [created]
  workflow_dispatch:
jobs:
  publish:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - name: Set up Maven Central Repository
        uses: actions/setup-java@v4
        with:
          java-version: '8'
          distribution: 'temurin'
          server-id: central
          server-username: MAVEN_USERNAME
          server-password: MAVEN_PASSWORD
          gpg-private-key: ${{ secrets.GPG_SIGNING_KEY }}
          gpg-passphrase: MAVEN_GPG_PASSPHRASE
      - name: Publish package
        run: mvn --batch-mode deploy -DskipTests
        env:
          MAVEN_USERNAME: ${{ secrets.CENTRAL_TOKEN_USERNAME }}
          MAVEN_PASSWORD: ${{ secrets.CENTRAL_TOKEN_PASSWORD }}
          MAVEN_GPG_PASSPHRASE: ${{ secrets.GPG_SIGNING_KEY_PASSWORD }}


Add Support for SentinelOne PQ

Changes

Create backend for PQ syntax
Create pipeline for PQ field names
Fix logic error in DV backend that incorrectly used {field} IS NOT EMPTY when searching for null values

Closes #6

Closing as this was moved to it's own separate repo


config.yaml renamed to hugo.yaml

On the guide https://toha-guides.netlify.app/posts/update-v3-to-v4/#6-update-the-configyaml-file there is an instruction:

you can check the sample config.yaml

that should be renamed to

you can check the sample hugo.yaml (previously config.yaml)

I'm not 100% sure about the parenthesis though, waiting for comments.
There are probably more references like this on other posts as well

Hi @tcarreira, thank you for pointing it out. You're right, config.yaml was renamed to hugo.yaml. For me, previously config.yaml looks good.

Hi @hossainemruz, what do you think is best?

Replace all occurrences of config.yaml to hugo.yaml
Replace only the references to the file in hugo-toha.github.io

Hi @hossainemruz, what do you think is best?

Replace all occurrences of config.yaml to hugo.yaml
Replace only the references to the file in hugo-toha.github.io

Let's go with the first option. Replace all occurrences of config.yaml with hugo.yaml


Web and Management Bind Address

Right now the Web and Management Bind Address are set to <IP_ADDRESS>, Issue #34 added two variables jboss_eap_bind_web_address and jboss_eap_bind_management_address set to <IP_ADDRESS>. This needs to be set to the ansible host IP address for better security while provisioning, adhering to JBoss guide defaulting to localhost for bind

closed by #41


No way to add calendar attribute value in dashboard

Running version 8.3.2 and I cannot seem to find anyway to attach new calendar attributes to actual events for saving the attribute value.
Steps to reproduce

Create a new attribute for calendar
Add calendar block to a page. You can now select the new attribute as displayed value in the modal.
There is no way to add the actual value for the attribute. The dashboard calendar UI does not allow you to select displayed attributes for the calendars and the attribute is not visible in the calendar event edit page.

Expected result:
The attribute should be either selectable to be displayed in the calendar event details or it should be always displayed.

Figured it out that you first have to create an attribute set and add the attribute to that set. Then the new set is visible in the Add Event modal.
This is not intuitive at all and its impossible to figure it out without any help text or other information how the attribute sets and attribute types should work in calendar events.

I encountered this last night. I think help text would be useful, but I am not sure what the text should be.
Example:
"Event attributes must be added to an attribute set. The attribute set name will be displayed as a tab in the add/edit event modal."

I think that would help a lot.

This whole thing seems buggy to me.
I created an attribute set, then added an address attribute to that set.
Then, I tried adding a new calendar using the "create and link to more details page". Here, no attributes can be selected from the "calendar event attribute" dropdown - nothing shows up here.
On the front-end, adding a regular calendar block, I do see the created attribute under the lightbox attributes options. However, there's no way to edit these created attributes.

@pinecreativelabs

Here, no attributes can be selected from the "calendar event attribute" dropdown - nothing shows up here.

Which drop-down are you referring to?

@pinecreativelabs
The Calendar Event Attribute select drop-down is a list of existing Calendar Event page attributes.
Dashboard > Pages & Themes > Attributes > Choose Type > Calendar Event
After a Calendar Event page attribute is created and selected as the Calendar Event Attribute, it will be applied to each event page created. The value of the attribute will be the event and the event's calendar.


TSConfig Italian translation: added files

Added the following files

disableSizeLimit.md
disableReferencedProjectLoad.md
diagnostics.md
declarationDir.md

LGTM


Show all projects by default

When no search query is entered, show all projects. To get the full list now you'll have to type a space as search query (st  ).

Added and publish at 1.0.
https://github.com/yourtion/Alfred_SourceTree/releases/tag/1.0.0


Using new REST endpoint for SLPDB data

No longer directly call slpdb for SLP token transaction information
Call new rest endpoint, which communicates to SLPDB with a specialized query

improves #223

Pull


Policy Refresh H2 FY24

Overview/Summary
This is the H2 FY24 Policy Refresh!
Testing URLs
Azure Public

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Ensured contribution guidance is followed.
[x] Updated relevant and associated documentation.
[x] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

Hey Team, could you please update Excel (ALZ Policy Assignments v2) with new policies?
https://github.com/Azure/Enterprise-Scale/wiki/ALZ-Policies

Hey Team, could you please update Excel (ALZ Policy Assignments v2) with new policies? https://github.com/Azure/Enterprise-Scale/wiki/ALZ-Policies

@nekdima please note the Excel only includes policies we assign by default. The new policies and initiatives are not enabled by default and require selecting scopes to apply at. We'll look at an additional Excel that includes all ALZ policies/initiatives regardless of assignment in a future update.


Does it support stream video (m3u8,..)

As the title, please let me know if it support stream video.
Thanks you so much

I have think about other formate , #40 #42 , and we think exoplayer is suit for us, how do think about that?
Now the main work is custom ui, that purpose is user will not write code about video , and user also can custom ui ,user also can add other feature .

A friend online told me , android.media.MeidaPlayer support live stream m3u8 and also support RTSP , you can find here https://developer.android.com/guide/appendix/media-formats.html .
Thank you @axeon

Few days later I will put some example in demo .

Thanks so much. Hope to see that

use vitamio or ijkplayer replace sys mediaplayer


404 when clicking on "Need help?" button for connect to remote node

Description
Clicking on Get Help links results in a 404 page in the local browser.

Expected Behavior
I presume it should open up this URL: https://docs.zaphq.io/

Actual Behavior

It opens this URL: https://ln-zap.github.io/zap-tutorials/ which is a 404 page.
Possible Fix

Swap out the help URLs for the correct one.
Steps to Reproduce

From the home screen click the orange 'Setup Wallet' button
Choose a PIN
Confirm the PIN from Step 2
Click 'Connect to Remote Node'
Click 'Need help?' button.

Can also replicate by going to Settings > Need Help?
Context

Trying to figure out how to pay my first Lightning invoice and testing out a few clients.
Your Environment

Using Zap version  on Android. I

Zap version: 0.2.5-alpha Build 1
Android version: 10
Device: Google Pixel XL
LND Version: not connected

Hi sixside,
Thanks for trying Zap and your detailed report.
With the launch of the new website this link got broken. We already fixed it in the code though, so it will be working again with the next update.

Fixed in 0.2.6


clojure: use latest java LTS

Description of changes

Things done
Clojure supports java LTS versions, we should use the latest LTS (currently jdk17)

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

clojure

That's cool, thank you!


Remove custom compiler environment variables

With the PR below merged, we no longer set the CUDAHOSTCXX variable in any of our CI images. This PR cleans up some references to CUDAHOSTCXX.

https://github.com/rapidsai/gpuci-build-environment/pull/265

rerun tests

Thanks AJ! 🙏


Leverage wasi-cli and wasi-http instead of Node.js HTTP API

See related wasi-cli and wasi-http with related worlds. Depends on #3 since it requies WASI Preview2+.

You might be interested in this WASI HTTP (preview2) shim for the Node.js host: https://github.com/bytecodealliance/jco/blob/main/packages/preview2-shim/lib/http/wasi-http.js

Indeed, thanks for bringing it to my attention.

I think I will try first to leverage https://github.com/nodejs/uvwasi/issues/183, and tackle WASI HTTP as part of #3 later.


Strange decimals when doing SUM GROUP BY.... eg:<PHONE_NUMBER>0000001

Hi, i got this statement...
alasql(
    `select ESTACLI,SUM(VLRTOTAL) AS SOMA
    from csv("c:/windows/temp/audit05.csv")
    group by ESTACLI
    order by SOMA`,[],(data)=>{
console.log(data);
});

and i have getting this output....
[ { ESTACLI: 'AC', SOMA: 35843.56 },
  { ESTACLI: 'AP', SOMA: 48573.69 },
  { ESTACLI: 'RR', SOMA:<PHONE_NUMBER>0000001 },
  { ESTACLI: 'TO', SOMA:<PHONE_NUMBER>0000007 },
  { ESTACLI: 'MA', SOMA:<PHONE_NUMBER>999999 },
  { ESTACLI: 'PA', SOMA: 1067937.7699999996 },
  { ESTACLI: 'RO', SOMA: 1139423.24 },
  { ESTACLI: 'AM', SOMA: 1165764.7 }, ...

I already ensure that the VLRTOTAL field of the csv file only have 2 decimals...
csv file...
NRNOTA,DATAEMI,VLRTOTAL,CODIVENNOT,CODINAT,ROMA1,VLR1PARC,CODCLI,NOMECLI,CODIVENCLI,ESTACLI
1,"11/03/2010",5115.84,114,611 ,554292,5115.84,5680,"COFIP COMERCIAL LTDA",61,"SP"
2,"11/03/2010",1827.80,6,611 ,554264,1827.8,5684,"MAGAZINE CHOHFI LTDA EPP-FL03",6,"SP"
3,"11/03/2010",1827.80,6,611 ,554265,1827.8,5685,"MAGAZINE CHOHFI LTDA EPP-FL 04",6,"SP"
4,"11/03/2010",1827.80,6,611 ,554266,1827.8,5683,"MAGAZINE CHOHFI LTDA- F05",6,"SP"
5,"11/03/2010",1827.80,6,611 ,554267,1821.6,6588,"MAGAZINE CHOHFI LTDA EPP.",6,"SP"
...

If you do a simple 1.2-1.0 in javascript the result will be 0.19999999999999996
Its because javascript uses floating points for numbers.
As AlaSQL is based on javascript you experience the same with your numbers.
For more information: http://floating-point-gui.de/

ok, thank you, sorry for disturbing you...

Its all good - thank you for taking the time to contact us to let us know something was wrong...

Hello,
I kown it's an old post but for those who read this question i came up with a quick solution  (i work with JSON so it's not easy to use money type or https://github.com/MikeMcl/bignumber.js/  :
sum(col*100)/100
For example :
alasql(
select ESTACLI,SUM(VLRTOTAL*100)/100 AS SOMA     from csv("c:/windows/temp/audit05.csv")     group by ESTACLI     order by SOMA,[],(data)=>{
console.log(data);
});
Thank you for this amazing library !
Benoit.

Hi @Benoit75 thank you for contributing to the subject :)

Hello, I kown it's an old post but for those who read this question i came up with a quick solution (i work with JSON so it's not easy to use money type or BigNumber.js : sum(col*100)/100
For example : alasql( select ESTACLI,SUM(VLRTOTAL*100)/100 AS SOMA from csv("c:/windows/temp/audit05.csv") group by ESTACLI order by SOMA,[],(data)=>{ console.log(data); });
Thank you for this amazing library ! Benoit.

it's working, thanks a lot


Add svg use info about integration tests/etc to readme

#patch#

Visual Acceptance tests:
No changes

Approved


Feature request: Reconnect attemps

I'm using 2.0.0-beta.4. When I reload the window while socket is connected I saw this error in console in my production build:
WebSocket is already in CLOSING or CLOSED state.
I think this error is happens when client trying to send ping to server when it isn't really connected. As far as I can see there is no function in the next branch controls the readyState of the WebSocket.
https://github.com/twitch-apis/twitch-js/blob/06fb311c246503f04af16e93874d34bc6fb7723d/src/Chat/Client.js#L37
master branch is handles readyState and have reconnect attemps.

Please try 2.0.0-beta.5+ and follow-up. This release includes #104, which should help with reconnecting.

I saw this update but it isn't includes anything about my issue and feature request. I have tested 2.0.0-beta.7 and same issue happened.

What do you mean exactly by "reload the window"? Can you provide a code sample?

My build is works on browser and connect to socket after page load. I mean refresh the page. It is usually works normal after refreshing page but sometimes throwing error. My code:
// ./twitch.js
import Twitch from 'twitch-js';
import { token, username, channel } from './config';

export default class extends Twitch {
  constructor() {
    super({
      token,
      username,
    });
  }

  async init() {
    await this.chat.connect();
    await this.chat.join(channel);
    return Promise.resolve();
  }
}

// ./main.js
import Twitch from '../twitch';
import { channel } from './config';

async function asyncCall() {
  const twitch = new Twitch();
  await twitch.init();
  twitch.chat.say(config.channel, 'Hi!');
}

asyncCall();

@sibbngheid I am having difficulty reproducing this bug. Can you try 2.0.0-beta.8 and follow-up with this issue?

I will be able to test it in next few days and give you more information.

Also,  async will wrap any returned value in a promise, but if you are not interested in the value of chat.connect() or chat.join(channel), I don't think you need to  return Promise.resolve() in .init().

Sorry for late response. I have tested 2.0.0-beta.8 and it is works perfect. Thank you.

Awesome!


Can Shenyu weak admin  in the future.Because I want to use other config center,such as nacos,apollo.

Feature Request

Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is. Ex. I have an issue when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen. Add any considered drawbacks.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Please read the Contribution Guideline before submitting the PR

Now you can do this if you use Nacos as the data synchronization.

@rottenmu  hi， I suggest you use shenyu-admin to configure the routing rules
if you not use shenyu-admin， The data format will be more complex ， also you can used nacos。
you can see this  org.apache.shenyu.sync.data.nacos.NacosSyncDataService .

for details of Nacos sync data, pls see  https://shenyu.apache.org/docs/next/user-guide/use-data-sync#nacos-synchronization-config


Quickstart guide violates foreign key constraint when running seed

Hello and thanks for this project !
I followed the QuickStart guide and got this error, I use bun.
I ran all the commands described in the guide, the db and the models are all set up.
$ bun src/db/dbScript.ts seed
2176 | };
2177 | const makeSQL = (table, options) => {
2178 |   const query = table.q;
2179 |   const sql = [];
2180 |   const values = (options == null ? void 0 : options.values) || [];
2181 |   const ctx = {
                  ^
error: insert or update on table "comment" violates foreign key constraint "comment_postId_fkey"
 code: "23503"

      at /Users/alex/orchid-orm/node_modules/pqb/dist/index.mjs:2181:14
error: script "db" exited with code 1 (SIGHUP)

Am I doing something wrong ?
Thank you for your help.

Confirming, I reproduced the bug, and there are other few things to update for the quickstart

It's a bug specific to Bun, apparently AsyncLocalStorage has a bug in a specific edge case, so I filed an issue.
The bug means that transactions currently don't work with Bun.

I published pqb with a workaround, now it works fine.
The installation guide and scripts are yet to be improved, but Bun works.


Allow only lock

Hi, I would like to ask if you plan to add feature which would allow to only lock the door. O would like to be sure that because of some bug my door will not be opened remotely.
I guess it would be easiest to handle this on token level (it would have right only to lock), but I haven't seen there this option.

Is not enough, if you disable the open button in Tedee App? Or do you want it open from Tedee App, but block open function only in Homeassistant?

The second option, to be able to open in from tedee app and not from HA.

Jakub, please jump to tedee_hass_integration of  Patrik Hilker. I do not maintain this here anymore.


Tab size saved but not re-applied

Hi all,
[for Acode version 1.2.143]

I set the "Tab size" editor setting to 8 and my tab-delimited text file appears in columns as expected.
When i re-open the same file, the text not appears in columns (columns are not justified as step 1) although the "Tab size" editor setting is still 8.
To see again the text in columns as step 1, every time i open the file i have to change the "Tab size" to any other value and again change it to 8.

Best regards
Thanks

Thanks for reporting the issue. I'll check this and fix it as soon as possible.

version 1.2.149 did NOT fix the problem.

deadlyjack moved this from To-do to Done

sorry, version 1.4.163 and the problem still exist.

version 1.4.164 did NOT fix the problem.


Keras distriibuted train problem

hello, iam using tfjob to train keras model.
most of times, they work fine. But some times, it will crash  after train and savemodel.
our partial train code is here:
    mirrored_strategy = tf.distribute.MultiWorkerMirroredStrategy()

    with open(args.featmap_path, 'r', encoding="utf-8") as f:
        onehot_feature_len = len(f.readlines())

    feature_description = {
        'label': tf.io.FixedLenFeature([], tf.int64),
        'feature': tf.io.VarLenFeature(tf.int64),
    }
    # ========================== Create dataset =======================
    dataset = tfrecords_reader_dataset(data_filename, feature_description, batch_size=batch_size)
    test_dataset = tfrecords_reader_dataset(test_data_filename, feature_description, batch_size=batch_size)
    # ============================Build Model==========================
    with mirrored_strategy.scope():
        model = DeepFM(onehot_feature_len, input_length=input_length, embed_dim=embed_dim, hidden_units=hidden_units, dnn_dropout=dnn_dropout)
        model.summary()
    # ============================Compile============================
        model.compile(loss=binary_crossentropy, optimizer=Adam(learning_rate=learning_rate),metrics=[AUC()])

    model.fit(
        dataset,
        epochs=epochs,
        verbose=2

    )

    # checkpoint_path = '/'.join((args.checkpoint_path, save_day, save_hour))
    # if TASK_INDEX == 0:
    #    checkpoint_path = checkpoint_path
    # else:
        # Save to a path that is unique across workers.
    #    checkpoint_path = checkpoint_path + '/worker_tmp_' + str(TASK_INDEX)

    inputs = tf.keras.layers.Input(shape=(input_length,), dtype=tf.int64, name='input')
    outs = model(inputs)
    # Here, does mymodel need be write in  the mirroredworker scope?
    mymodel = tf.keras.Model(inputs, outs)
    mymodel.save(checkpoint_path)

    if TASK_INDEX == 0:
        # tf2onnx
        try:
            onnx_args = OnnxArgs()
            onnx_args.saved_model = checkpoint_path
            onnx_args.output = checkpoint_path + '/deepfm.onnx'
            onnx_args.tag = 'serve'

            parse2onnx(onnx_args)
            logging.info("Success")
            logging.info(onnx_args.saved_model)
        except Exception as e:
            logging.error("Failed convert")
            logging.error(str(e))

Does mymodel in the code  need to be within mirrored_scope??? I think here it is a little strange.
And the log is:

I get the coredumpe file;
it show this:

the operator log like  this :
i don't know the reason and don't know whether it is a tensorflow bug...

Any help will be appreciated... Thanks alot

I think. It is the same as https://github.com/tensorflow/tensorflow/issues/50853

Could you please provided the complete code to reproduce your issue, Thanks.


Closed Book Choices Template Fix

small nit fix in exams to ensure that various number of choices work

 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.


Add Cinamon Rolls emotes

React to messages talking about Gotoubun no Hanayome with custom emotes, or add custom emotes to bot messages like the about embed.

Added. :)


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1181 ms

Resolved: Jenkins Plugins Artifactory performance has improved in eb506f7 after 12 minutes.


Problem with @nuxt/font Usage

Environment:

Node Version: v18.20.3
Nuxt Version: ^3.13.0
@nuxt/fonts Version: ^0.8.0
Package Manager: pnpm 9.6.0

Description:
I am experiencing an issue with the font definitions generated by the @nuxt/font package in my Nuxt 3 application. The package generates CSS that improperly uses the local() function in the src property of the @font-face declarations, causing the fonts to not load correctly.
Generated CSS Snippet:
@font-face {
    font-display: swap;
    font-family: 'Archivo';
    font-weight: 300;
    src: local("url('/fonts/Archivo-Light.woff2')");
}

/* Other generated font definitions... */

Expected Behavior:
The fonts should load correctly without any issues.
Actual Behavior:
The generated CSS prevents the fonts from loading properly due to the incorrect syntax in the src property.
Nuxt Configuration:
fonts: {
    defaults: {
      styles: ["normal"],
    },
    families: [
      {
        name: "Archivo",
        src: "url('/fonts/Archivo-Light.woff2')",
        global: true,
        weight: 300
      },
      {
        name: "Archivo",
        src: "url('/fonts/Archivo-Regular.woff2')",
        global: true,
        weight: 400
      },
      {
        name: "Archivo",
        src: "url('/fonts/Archivo-Medium.woff2')",
        global: true,
        weight: 500
      },
      {
        name: "Archivo",
        src: "url('/fonts/Archivo-SemiBold.woff2')",
        global: true,
        weight: 600
      },
      {
        name: "Clash Display",
        src: "url('/fonts/ClashDisplay-Regular.woff2')",
        global: true,
        weight: 400
      },
      {
        name: "Clash Display",
        /*
          Even when i following the docs with this format (no quote)
          src: { name: 'Other Font', src: 'url(/fonts/ClashDisplay-Medium.woff2)', weight: 'bold' }
        */
        src: "url('/fonts/ClashDisplay-Medium.woff2')",
        global: true,
        weight: 500
      },
    ]
  }

Steps to Reproduce:

Install @nuxt/font using pnpm.
Configure the module in nuxt.config.js with the provided font settings.
Run the application.

Additional Context:

The paths to the font files are correct, but the current usage of local() in the generated CSS is causing loading issues.
A potential fix could be to modify the generated CSS to remove local() and use url() directly:src: url('/fonts/Archivo-Light.woff2') format('woff2');

Thank you for your assistance!
@danielroe

You should not set src: "url('/fonts/Archivo-Light.woff2')".
Instead, you set src: "/fonts/Archivo-Light.woff2".
Fixed the incorrect docs here: https://github.com/nuxt/fonts/commit/ca44e0d274518f1ab37ddba20945f9354dd669da.
Let me know if you encounter any further issues 🙏

You should not set src: "url('/fonts/Archivo-Light.woff2')".
Instead, you set src: "/fonts/Archivo-Light.woff2".
Fixed the incorrect docs here: ca44e0d.
Let me know if you encounter any further issues 🙏

Thank you for the quick response! I updated the src property to src: "/fonts/Archivo-Light.woff2" as you suggested, and it resolved the issue. The fonts are now loading correctly.
I appreciate the fix in the documentation as well. If I encounter any further issues, I'll be sure to reach out. Thanks again! 🙏


Error: VPack error: Expecting Object

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[x] 3.2

Mode:

[x] Single-Server

On this operating system:

[x] Windows, version: 10

I'm issuing AQL via:

[x] web interface with this browser:     running on this OS:

I've run this query to input the the jason web token (JWT) into the server as a header in order to return a collection.
`results = requests.put("http://<IP_ADDRESS>:8529/_db/database_name/_api/simple/all",
                      params={"collection": "collection_name", "batchSize":1000},
                      headers={"Authorization": str('Bearer ' + Jason_Web_Token)})
results.text

Sadly, I get this error:

u'{"error":true,"errorMessage":"VPack error: Expecting Object (exception location: /var/lib/otherjenkins/workspace/RELEASE__BuildPackages/arangod/GeneralServer/RestHandler.cpp:204). Please report this error to arangodb.com","code":500,"errorNum":4}'

The error message is not saying what it expects to be an object. I've looked within the browser console error messages as well to no avail. Where should I look to debug this?

Hi jonberthet,
it looks like you are working with one of the python community-drivers. Please tell me which one and which version you are using. It would be great if you could provide a minimal (not-)working example.
Regards Jan

I am using the Arango Docker environment found here (https://www.arangodb.com/download-major/).
The 'requests' python package version is 2.18.4 and documentation & examples shown here (http://docs.python-requests.org/en/master/api/). Does this help? Plz let me know if addit'l info is needed.

Hi,
if you intend to use ArangoDB and you do not want to implement another python driver, then you should use a existing one like pyarango. The request module you linked is for plain http requests. You could use that, but it would be much more complicated to use.
Have a look at our tutorial!

Hi @jonberthet ,
I understand that the error message is a bit misleading. Velocypack is also ArangoDBs JSON-Parser library; a JSON object looks like that:
{
   "foo": "bar"
}

Please have a look at the documentation of the REST-route you're invoking, either via the regular HTTP API-Documentation or in your ArangoDB Webinterface,
Support->Rest API->[Simple Queries]->_api/simple/all
for the interactive swagger.io documentation.
Both show you an example about how the request should look, the Swagger one lets you interactively build the request.
Comparing what you saw there, you can see that that PUT-request usually have a POST-Body. In the Request you're sending with that python snippet you're sending a PUT request, but the POST-Body is empty:
PUT /_db/database_name/_api/simple/all?batchSize=1000&collection=collection_name HTTP/1.1
Content-Length: 0

See the difference?
Usually its a good idea to have wireshark or ngrep at hand to inspect what some test snippets generate on the wire.


fix(formatter): incorrect exit code on files with syntax errors

Fixes #1088.
For the PR title and commit message, since I failed to find a consistent commit style across the recent Git history, I'm using Conventional Commits instead. Please let me know if there's a desired style, thanks!

This change is

For the PR title and commit message, since I failed to find a consistent commit style across the recent Git history, I'm using Conventional Commits instead. Please let me know if there's a desired style, thanks!

I do not recall any conversations about message style so in general I'd simply assume using descriptive titles 😃 your title is perfectly fine!

Thanks for the reviews! Please let me know if anything is pending from my side.

Ah I wasn't aware of that being a requirement. Rebased now :)

Wait that's auto-merged? lol

Oops, I tihnk we broke main :S weird, Ill check why CI didnt stop it

Yeah I thought the auto-merge were going through only if CI passes.

Seems it's not a mandatory check


any idea on multi batch training

same as faster rcnn, the training only allow single image per batch in training. In caffe, the parameter of "iter_size" can be adjusted to do multi batch training as weights are updated after "iter_size" iterations, i.e., images. Can this be done in TF?
Thank you.

following http://stackoverflow.com/questions/37710974/how-to-accumulate-and-appy-gradients-for-async-n-step-dqnetwork-update-in-tensor, I modified train.py in most recent codes of this project, using TensorFlow 1.0 on customized dataset. The basic idea is create a placeholder as the switch to apply the accumulated gradients over several iteration.
But after several iterations, a warning appears indicating overflow in exp function in lib/fast_rcnn/bbox_transform.py, line 49.
https://github.com/rbgirshick/py-faster-rcnn/issues/65 this thread also reports overflow in exp in bbox_transform. I followed the solutions there by trying to adjust learning_rate (as low as 0.000001), RNG_SEED with no success. The annotation of the bounding-box is also 1-based.
@smallcorgi do you have any idea in implementing multi batch training? please help, thank you.


Remove 0 change when shrinking the table

If we want to do so, this will remove the lines where the change is == 0 when the table is shrunk
Relates to https://github.com/alex/nyt-2020-election-scraper/issues/91

Meh. I don't care about mixed semicolons, personally. :) LGTM!

I'm not 100% certain that we'd want to, just since even when the change is 0, the Hurdle still is going to change based on the estimated votes remaining fluxuation

Maybe we can reduce it to just keep most recent one.


malformed tag on demo assets

i get this when packaging on 5.2
are you targeting 5.3?

UATHelper: Packaging (Android (ASTC)): LogInit: Display: -----------------------------------
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/M_OXRHand.uasset has malformed tag
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/OXR_TestMap.umap has malformed tag
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/IMC_OXRDemo.uasset has malformed tag
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/WBP_DebugHandTracking.uasset has malformed tag
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/IA_OXR_Submit.uasset has malformed tag
UATHelper: Packaging (Android (ASTC)): LogInit: Display: LogAssetRegistry: Error: Package C:/w/vrtest/Plugins/FSOpenXRHandTracking/Content/Demo/BP_OXRPawn.uasset has malformed tag

Hello,
Thanks for the report, as indicated into the README, it's only tested on 5.3. However, you can probably recreate the pawn on 5.2 using the FSInstancedHand components for left and right hands.

Which version of 5.3 is this intended for? The demo assets do not show up in the editor, and when opening them in a text editor they only contain the following text:
version https://git-lfs.github.com/spec/v1 oid sha256:3e36390e0735e6b67636776826d08cc2ddbfc7858b288e932d02742a02738a0f size 1374
Looks like these .uassets actually don't contain any content.

It's updated for 5.4, I've to update the readme. Everything is fine on my PC (5.3, 5.4) so it's an LFS issue. I'll try to fix it.


Review Outstanding Questions

We need to do a review of various outstanding questions/tasks.
Tasks

[ ] Take a look at group membership permissions - we have only one membership semantic, do we need the other two? Link.
[ ] Review federation - what's left?

This is a 'question' task. Closing it out.


How to integrate srtool to Github actions?

Examples:

https://github.com/chevdor/srtool-actions/blob/master/action.yml
https://github.com/paritytech/cumulus/blob/master/.github/workflows/srtool.yml

Related to https://github.com/substrate-developer-hub/substrate-docs/issues/407 -- maybe a duplicate? we can consolidate to one issue.


how to install

can you post some hintsfor installation. I would try it...
goeran

works, great

not an issue


pitivi: pass hicolor-theme to fix missing icons

Description of changes
Pitivi seems to not be compatible with Adwaita and requires hicolor-theme-icons at runtime.

Things done

Built on platform(s)

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

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

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

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

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

[X] Fits CONTRIBUTING.md.

This should be ready to get merged @wegank, thanks for your help

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review-december/1711/16

Yeah, the aarch64-linux failure looks irrelevant.

I noticed I made a little mistake, I added myself to the maintainers but without you know separating the commit, was that okay?

It's OK for me.


CLI describe command

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Describe changes
Added a zenml [COMPONENT] describe command (with the option to pass in the name of the component as an argument if you want something other than the current active component).
Works for artifact, container_registry, metadata, orchestrator, stack.

@bcdurak 💣 is now defused. Ready for any reviews / feedback...

Thanks for the reviews @htahir1 and @schustmi. Indeed it's much better now. Fixed everything you mentioned.

@htahir1 I added in a function as suggested.

The issue with private properties is fixed on my kubeflow branch already, where I filter the dict for all base components (so I think no need to fix it here, should be fine once both are merged)

Ok then Lets Go To Main @strickvl !


'Update App Settings in Azure' intro sentance makes no sense

This sentence makes no sense at all:
"The local bot does not use an encrypted .bot file, but the Azure portal has an that you are deploying doesn't"

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

ID: 2cfa1653-6ab2-730b-5479-81a05b9bb18e
Version Independent ID: a7c8085e-3b7a-711e-308c-7061176c0a5c
Content: Deploy your bot - Bot Service
Content Source: articles/bot-builder-deploy-az-cli.md
Service: bot-service
Product: bot-framework
GitHub Login: @ivorb
Microsoft Alias: v-ivorb

Thanks for your feedback! You'll see the corrected sentence the next time this article is published.
ps. This sentence was quite a jumble!


Z Token

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

Please upload report for BASE (main@6380644). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff            @@
##             main        #1   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         2
  Lines           ?        74
  Branches        ?        15
========================================
  Hits            ?        74
  Misses          ?         0
  Partials        ?         0


Release 2.0.0 PR

PR for the release of the 2.0.0 DSL2 version.

Another small point, I think the convention was to name the major versions 2.0 rather than 2.0.0

Thanks @ggabernet , will have a look at those!
Didn't know about the convention, will change that too, then.

I looked a bit more into the schema:

cluster_options and enable_conda appear as ungrouped parameters.
trim_ecop, trim_linker, trim_5g should be true by default in my opinion.

Weird that it's suddenly failing ...

Weird that it's suddenly failing ...

Did I introduce a formatting error or something with the new version scraping which broke sortmerna?

No idea, but it's sortmerna that's failing. Trying to reproduce locally now.

Closing this for another release PR in its place, with the new template and modules via #55


Update SIOS dataset

Proposes new title and adds summary.
There are some other variables collected by the sensor, such as fDOM and Oxygen concentration and saturation, but I cannot find any appropriate CF standard names for those. They are probably not as important as temperature, etc. so I think we can leave them out, what do you think?
Keywords look good to me as they are.

lgtm, @mar1yS what do you think about the titles?
btw, this is just a metadata draft until we can publish the datasets:)


CCS: Process as map rather than CBOR encoded map

Following up on #273, this hopefully brings the credentials by value in line with the specification.

I've tested this between Lakers devices. No plugtest happend on this solution yet, but I think it's enough of an improvement to merge it once reviewed -- I don't know when I next get around to doing more interop tests with others.

Looks good to me! Leave a :+1: here when you think it is ready to merge.


Fixes for requirePaddingNewLinesBeforeLineComments (consecutive and in blocks)

requirePaddingNewLinesBeforeLineComments: Allowing multiple line comments together. Allowing no padding if first line in block.

Coverage increased (+0.0%) to 98.86% when pulling fa0024197af4ec1f67a5f6f5467515aa5dbc01bb on TheSavior:fix/newLinesBeforeComments into a0c246122431fa2aa8f875d980ad6d5af32b39c2 on jscs-dev:master.

@TheSavior, thanks again for contributing!

Coverage increased (+0.0%) to 98.87% when pulling 751436bb438d3ba55b2b1166a990ccec7b707bdc on TheSavior:fix/newLinesBeforeComments into a0c246122431fa2aa8f875d980ad6d5af32b39c2 on jscs-dev:master.

Alrighty. An additional configuration option has been added. Can you guys review my doc block? I copied it from another rule but I want to make sure that it will be parsed correctly and is in the correct style.

Coverage increased (+0.0%) to 98.93% when pulling d6e8abcfe22efab1e3e53a99fa89d1cff0f6363e on TheSavior:fix/newLinesBeforeComments into c9265a1dfdf006b4aac8fbcfb6b57c640feadf23 on jscs-dev:master.

Bump?

@TheSavior no reason to bump. This will land before we release 1.12 regardless of when it actually makes it in

Ah, gotcha. The flow of when things get merged in on JSCS is just a bit different than what I would expect. Still getting used to it. :)
I was figuring that I would update #1103 after this and #1101 were merged, perhaps I should update it now so it will be ready for you to merge them all at the same time.

Thanks again!


Older git compat

Found that on older git versions, it displayed name-rev like this: "remotes/origin/ci-prepare" grabbing only the branch name with this commit

Coverage increased (+0.1%) to 94.444% when pulling 9cd6c08d7f9c9b75d7de4af30a22d248a7f0d737 on older-git-compat into 75660fed0b2940c1fa7663866e940b3e45344f7f on master.

Coverage increased (+0.1%) to 94.444% when pulling 9cd6c08d7f9c9b75d7de4af30a22d248a7f0d737 on older-git-compat into 75660fed0b2940c1fa7663866e940b3e45344f7f on master.


Unable to use dynamic credentials in ftp_servers options.

I want to use credentials from database table in ftp_servers options. Is there any way to do this ?
1). I have tried using lamda option which paperclip supports but it gives me error:
NoMethodError (undefined method `map' for #Proc:0x007fb0249b3738
has_attached_file :image, {
    storage: :ftp,
    path:  '/Images/:filename',
    url:   '/images/:style/missing.png',
    ftp_servers: -> (attachment) {  attachment.instance.ftp_keys },
    ftp_connect_timeout: 5,
    ftp_ignore_failing_connections: true,
    ftp_keep_empty_directories: true
  }

def ftp_keys
    [
      {
        host: 'host',
        user: 'user_name',
        password: 'password'
      }
    ]
 end

2). I also tried to use function but inside hash i am unable to use any class method.

Hi @hassanakram! I am sorry, but this is not supported at the moment (you are the first one to request that feature).
Could you contribute a PR to add support for it? I would be happy to review.

@boosty No issues.
I have created a pull request, please review it when you have time. I will be more than happy to accommodate required changes.
Thanks

Closing due to lack of activity on #35. Please reopen if necessary.


Clear Planes and Anchor

How I could restart the detection, like clean all, without need to finish and relaunch the activity?
Would expected something like session.clear() or ArFragment.clear(). Maybe is a feature request.

Look at my answer to my own question on stack overflow. You must restart your activity.

@ManuelTS you can read here that exactly saying that I don't wanna relaunch/restart the activity, because this is a solution, but not a good behaviour for the app. So if we don't have something that clean the planes and anchor we should put this as a feature request.
Like pause and resume, didn't had before and came after a feature request.

My mistake and my apologies. But I leave my answer here, maybe it helps someone.

@Canato  - Can you expand on why restarting the activity does not work in your case?  Another alternative is to destroy the ArFragment and Session and recreate them.  Planes are a fundamental feature of ARCore, which uses planes to build a model of the real world and the locations of the planes are refined as more information is gained, so I am not sure what is gained by resetting the session.
Clearing the renderables from the scene can be done by iterating over the scene's children and remove them.
Anchors can be iterated over from the ARCore session using getAllAnchors() and detached from the session.

@claywilkinson I have tried detaching ArFragment and add it again to activity. In this case, I'm getting error which don't crash my app all the time but still crash it some time. Which means I'm unable to use this as a solution.
E/ArSceneView: Error loading libarsceneview_jni.so
    java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.xyz.main-6CoFwVVqYpwyQVwA8mCoXA==/base.apk"],nativeLibraryDirectories=[/data/app/com.xyz.main-6CoFwVVqYpwyQVwA8mCoXA==/lib/arm64, /data/app/com.xyz.main-6CoFwVVqYpwyQVwA8mCoXA==/base.apk!/lib/arm64-v8a, /system/lib64, /system/product/lib64]]] couldn't find "libarsceneview_jni.so"
        at java.lang.Runtime.loadLibrary0(Runtime.java:1067)
        at java.lang.Runtime.loadLibrary0(Runtime.java:1007)
        at java.lang.System.loadLibrary(System.java:1667)
        at com.google.ar.sceneform.ArSceneView.reportEngineType(SourceFile:165)
        at com.google.ar.sceneform.ArSceneView.resumeSession(SourceFile:49)
        at com.google.ar.sceneform.ArSceneView.resume(SourceFile:39)
        at com.google.ar.sceneform.ux.BaseArFragment.start(BaseArFragment.java:507)
        at com.google.ar.sceneform.ux.BaseArFragment.onResume(BaseArFragment.java:315)
        at androidx.fragment.app.Fragment.performResume(Fragment.java:2718)
        at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1364)
        at androidx.fragment.app.FragmentManager.moveFragmentToExpectedState(FragmentManager.java:1686)
        at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1750)
        at androidx.fragment.app.BackStackRecord.executeOps(BackStackRecord.java:443)
        at androidx.fragment.app.FragmentManager.executeOps(FragmentManager.java:2494)
        at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2285)
        at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:2240)
        at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:2143)
        at androidx.fragment.app.FragmentManager$3.run(FragmentManager.java:417)
        at android.os.Handler.handleCallback(Handler.java:883)
        at android.os.Handler.dispatchMessage(Handler.java:100)
        at android.os.Looper.loop(Looper.java:214)
        at android.app.ActivityThread.main(ActivityThread.java:7116)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:925)


Increase item frame chunk limits

This just updates the /chunklimits list. You'll have to change it in the Insights config too.


ads not showing

I just registered an admob account and I tried to display a banner using AdMobBanner component. I can see that didFailToReceiveAdWithErroris not called, also adViewDidReceiveAd is called and shows success but there is still nothing in the banner.
So is there something that I do wrong?

Also AdMobInterstitial will only popup a black screen with a close button but nothing else.

Could you please try the google test id (also used in the example project) to rule out the possibility that the problem has something to do with your admob account?

yes I used google's sample unit id ca-app-pub-3940256099942544/2934735716 and it worked. So I guess that's just my account then.

It appears that AdMobBanner is working and banner is showing up but AdMobInterstitial is not working. my RN is 0.28.0 and React is 15.2.1

No ad to show. means that there is no ad for your admob id to show. Which id are you using?
ca-app-pub-3940256099942544/1033173712 is the test id I'm using in the example project.

 I used an Ad id that only serves banner. Guess that's why...

@ssmlee04 Production mode my banner always return no ad to show. Can u help me ?

I've been struggling with this all day. Banners were rendering correctly on iOS (Simulator) but not Android.
The fix that I was missing is that the AndroidManifest.xml must have an additional intent declaration:
<activity
  android:name="com.google.android.gms.ads.AdActivity"
  android:configChanges="keyboard|keyboardHidden|orientation|screenLayout|uiMode|screenSize|smallestScreenSize" />

Without this declaration, neither banners nor interstitials would load.
As a suggestion, it would be awesome if in the README.md for the repo you could state that the device id is printed in the console upon initialization of the app, and that the simulatorId only applies to emulators.
Many thanks for creating such a great library.


How to show group of checkboxes?

Dear,
How can I create group of checkboxes like the following?
<input type="checkbox" name="animal" value="Cat" />Cats

<input type="checkbox" name="animal" value="Dog" />Dogs

<input type="checkbox" name="animal" value="Bird" />Birds

Regards,
--Nick

Hey Nick,
Are you using the master branch (default npm release) or the ember-1.11 branch?

Hi Duncan,
I am using ember-1.11 branch.
Regards,
--Nick

So I believe you want to do something like:
{{input value='Cat' name='animal' type='checkbox'}}
{{input value='Dog' name='animal' type='checkbox'}}

Does that not work? If not my guess would be that the checkbox is expecting value to be passed a boolean property. If that's the case, you may need to refactor to something like the following and manually build the animal property.
{{input value=isCat type='checkbox'}}
{{input value=isDog type='checkbox'}}

Another option is to use the {{input-group}} component in block form and use HTML for the checkboxes:
{{#input-group value=animal}}
  <input type="checkbox" name="animal" value="Cat" />Cats

  <input type="checkbox" name="animal" value="Dog" />Dogs

  <input type="checkbox" name="animal" value="Bird" />Birds

{{/input-group}}

Finally, you might want to ask if this use case better suits radio buttons.
Does that help?

Yes, that helps. Thank you very much.


azure-monitor-opentelemetry-exporter not compatible with latest opentelemetry-[api/sdk] 1.15.0

Package Name:  azure-monitor-opentelemetry-exporter
Package Version:  1.0.0b10
Operating System: Linux
Python Version:  3.10

Describe the bug
not compatible with opentelemetry-[api/sdk] 1.15.0
To Reproduce
Steps to reproduce the behavior:

pip install azure-monitor-opentelemetry-exporter
python3.10
from azure.monitor.opentelemetry.exporter import AzureMonitorLogExporter, AzureMonitorTraceExporter

see:
.tox/py310-sqlalchemy14-integration/lib/python3.10/site-packages/gmo/core/util/telemetry/__init__.py:8: in <module>
    from azure.monitor.opentelemetry.exporter import AzureMonitorLogExporter, AzureMonitorTraceExporter
.tox/py310-sqlalchemy14-integration/lib/python3.10/site-packages/azure/monitor/opentelemetry/exporter/__init__.py:7: in <module>
    from azure.monitor.opentelemetry.exporter.export.logs._exporter import AzureMonitorLogExporter
.tox/py310-sqlalchemy14-integration/lib/python3.10/site-packages/azure/monitor/opentelemetry/exporter/export/logs/_exporter.py:8: in <module>
    from opentelemetry.sdk._logs.severity import SeverityNumber
E   ModuleNotFoundError: No module named 'opentelemetry.sdk._logs.severity'

Expected behavior
import to work
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Thank you for your feedback @jabbera . We will investigate asap and get back to you

The PR is merged. However, since this is an ongoing issue, let's keep this open until the next release. I am looking into whether we can do a release before January. In the meantime, use OTel 1.14

There's another issue: when using the fixed version of the exporter with OTel 1.15, I consistently get the following warning:
...\site-packages\werkzeug\serving.py:716: ResourceWarning: unclosed <socket.socket fd=1304, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=0> self.socket = socket.fromfd(fd, address_family, socket.SOCK_STREAM) ResourceWarning: Enable tracemalloc to get the object allocation traceback
I've determined that no commit in exporter 1.0.0b11 has caused this. It's too early to know. But it seems to be an issue with how opentelemetry-instrumentation-flask==0.36b0 or opentelemetry-instrumentation-wsgi==0.36b0 use Werkzeug. I've confirmed the fixed version of the exporter does still send telemetry correctly. However, since the new version needs to be pinned to 1.15, we'll be encouraging people to use 1.15 seems to have some issue that needs to be addressed.
I am not able to release the exporter as is because of the memory issue with OTel 1.15. Instead, we can pin the exporter to 1.12<=x<=1.14 before the module path was changed. That way, we can avoid the memory allocation issue as well as the severity import breaking change.

My release PR is approved. But my understanding is others have the exclusive permissions to merge it and trigger the release pipeline.
https://github.com/Azure/azure-sdk-for-python/pull/27958

@jabbera Please use newly released 1.0.0b11. It blocks OTel 1.15. Resolving issue.


docs(ngOpen): add note about browser compatibility

Add note about browser compatibility and recommend ng-show/ng-hide

This should address #7337.

Thanks for the PR. We should also add a note about incomplete two-way binding in browsers that support details / summary, so if that's something you'd like to document: https://github.com/angular/angular.js/issues/3867


Support more command options

Some comments:
1. Please separate the commit for invoking as a script into a separate PR (if you still want to propose it, I'm still not fond of it)

Okay.

2. The new options are not used in the code yet

They are passed as keyword arguments to lpar.scsi_dump.

3. Add an entry in the change history

Sure.
Thanks for the feedback.

Thanks for the update. Sorry for missing how the options are handled - I simply forgot about that. I have approved the PR.


Send Content-Length to eliminate missing Content-Type error in some cases

We are having trouble managing projections on our eventstore cluster. Locally there were no problems but apparently in our production setup things work slightly different. Dont ask me why, but this PR fixes the 'Missing or invalid Content-Type' message.

POST requests require the Content-Length header to be set.

If a request contains a message-body and a Content-Length is not given, the server SHOULD respond with 400 (bad request) if it cannot determine the length of the message, or with 411 (length required) if it wishes to insist on receiving a valid Content-Length

https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html

Looks good. Merged into master. Releasing a new npm version with the fix.


Generators for factories and fixtures

Factory would be
ember g mirage-factory contact

which creates
//app/mirage/factories/contact.js
import Mirage from 'ember-cli-mirage';

export default Mirage.Factory.extend({
});

and fixture would be
ember g mirage-fixture contact

which creates
//app/mirage/fixtures/contact.js
export default [
];

This was implemented.


[fix][broker] fix the local police IsolationGroups cover the defaultIsolationGroups when init managedLedger bookkeeper client

Motivation
Fix the local police IsolationGroups cover the defaultIsolationGroups when init managedLedger bookkeeper client, which will cause even we delete-bookie-affinity-group and always used the last BookieAffinityGroup setting.

If we set bookkeeperClientIsolationGroups=aGroup in broker.conf  and local policy set set-bookie-affinity-group public/default --primary-group bGroup,  the defaultIsolationGroups be set as bGroup when init bookkeeper client ,  and after invoke  delete-bookie-affinity-group, there are choice bookies in bGroup, the correct behavior is aGroup which we set int broker.conf.
If we don't set bookkeeperClientIsolationGroups= in broker.conf , and local policy set set-bookie-affinity-group public/default --primary-group bGroup, the defaultIsolationGroups be set as bGroup when init bookkeeper client , and after invoke  delete-bookie-affinity-group, there are choice bookies in bGroup, the correct behavior is no more isolation policy.

As the suggestion, we should use the broker.conf  setting as the defaultIsolationGroups always, and the local policy as the dynamic config to change the Isolation groups though the customMetadata in newEnsemble or replaceBookie method.
To Reproduce
The unit test can help reproduce the bug.
Modifications

When create bookkeeper client In BookKeeperClientFactoryImpl.class  , add the ServiceConfiguration's IsolatedBookieEnsemblePlacementPolicy to bkConf if not empty.
When create bookkeeper client In BookKeeperClientFactoryImpl.class,  cancel to  set the local policy BookieAffinityGroup properties to bkConf, to avoid cover the ISOLATION_BOOKIE_GROUPS if  the ServiceConfiguration's IsolatedBookieEnsemblePlacementPolicy is empty.
Modify bookieMappingCache init time to adapt the  ServiceConfiguration's IsolatedBookieEnsemblePlacementPolicy is empty case.
add some unit test.

Documentation

[X] doc-not-needed

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@Jason918 Could you help to review this PR, Thx.

@codelipenghui @Jason918 @eolivelli Could you help to review this PR, Thx.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Issue 14348 - typeof(x).ident is not accepted as a symbol

Instead of evaluating as a type and then converting to a symbol, re-use the resolve logic to find the symbol.
https://issues.dlang.org/show_bug.cgi?id=14348

Auto-merge toggled on

Thanks!


Add internal onCreate method implementation when user has not overrid…

Currently, if the user has not overridden the onCreate method for a service, the instance of it would not be initialized. This commit fixes this behavior by emitting an onCreate method even if the user has not overridden it.

https://github.com/NativeScript/android-runtime/issues/1373


upd

Codecov Report

Merging #98 (2b220df) into main (e068f13) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2b220df differs from pull request most recent head 41ecb50. Consider uploading reports for the commit 41ecb50 to get more accurate results

@@            Coverage Diff            @@
##              main       #98   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines           29        29
  Branches         5         5
=========================================
  Hits            29        29

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 52840fe...41ecb50. Read the comment docs.


Typescript: Handler not returning state

Hi there, first issue here.
These days using redux-act, I found that the handler of reducer.on isn't using state type received in createReducer.
Example:
interface State {
  loading: boolean;
  name?: string;
  office?: office;
  claims?: Array<Claim>;
}

const reducer = createReducer<State>({}, initialState);

reducer.on(loading, (state, payload) => ({
  ...state,
  loading: payload,
  foo: 'bar',
}));

foo: 'bar' isn't in my interface, and don't throw an error in my typings.
It only works when I explict use State interface as result of my handler function.
Example:
reducer.on(loading, (state, payload): State => ({
  ...state,
  loading: payload,
  foo: 'bar',
}));

Only this way I got the expected behaviour:

src/modules/user.ts|31 col 3 error| 2322[QF available]: Type '{ loading: boolean; foo: string; name?: string | undefined; escritorio?: Escritorio | undefined; claims?: Claim[] | undefined; }' is not assignable to type 'State'. Object literal may only specify known properties, and 'foo' does not exist in type 'State'.

I can reproduce but I have no idea why it's failing to type properly. I'm not a huge TypeScript user myself unfortunately.
// This is the signature of the `on` function. 2nd argument is a Handler<S, P, M>
on<Arg1, P, M={}>(actionCreator: ActionCreatorOrString1<Arg1, P, M>, handler: Handler<S, P, M>): Reducer<S>

// This is the type for a Handler, a function that must return a `S`,
// where `S` is the `State` during `createReducer`
type Handler<S, P, M={}> = (state: S, payload: P, meta?: M) => S

export function createReducer<S>(handlers: Handlers<S> | OnOff<S>, defaultState?: S): Reducer<S>;


Add group field to Diagnostics

See https://github.com/rust-lang-nursery/rls/pull/681

@Marwes Alright, I've made the necessary changes

I can make a release of this tonight (GMT+1).

Forgot about this yesterday but it is now released as 0.30

@Marwes Thank you


Create project organization

https://help.github.com/en/enterprise/2.18/admin/user-management/creating-organizations

created organization, need to transfer this repo

@neha-deshpande001


Proper URL encoding

Hey,
I found an issue with character encoding in urls.
E. g. https://euw.api.pvp.net/api/lol/euw/v1.4/summoner/by-name/Scârry?api_key=
returns 400 Bad Request, but is valid from the dev portal. This is because request does not encode the url properly. There is an open pull request (https://github.com/request/request/issues/1678) for the request package since May 10.
As a workaround the following code can be inserted before https://github.com/amreuland/ashe/blob/master/lib/worker.js#L150
      var urlLib = require("url");
      var parsedUrl = urlLib.parse(url)
      // Encode path if needed
      let parts = urlLib.parse(parsedUrl)
      if (parts.pathname && parts.host !== 'unix') {
        // Prevent double encoding
        parts.pathname = encodeURI(decodeURI(parts.pathname))
        parsedUrl = urlLib.format(parts)
      }
      url = urlLib.parse(parsedUrl)

Do you want to implement it or should I create a PR?

pull request pls


PR view missing file tree

Testing microsoft/vscode#167397
Not sure exactly how I got into this state but after converting a PR to Draft and reloading the Window, the PR just shows the Description without the file tree underneath. Converting away from Draft and reloading didn't help.

Still repros if I'm on the same branch as the PR and I reload the window

Still repros but I just realized I need to wait for a GHPRI update before verifying


support for TK118

it seems that TK118 is not supported. Is it possible to implement support for it?

TK118 uses v680 protocol. it is 5016 port

You should check documentation before raising any "bugs":
https://www.traccar.org/ports-for-tk-gt-and-other-gps-devices/


Initial package signing

This change adds the initial package signing functionality to the SDK.  It currently supports only one entry point within the binary; the C exported function SignPackage.
The current implementation's features are:

The incoming package can be signed or unsigned; if signed it will simply be re-signed with the new data
Only PFX format certificate files are supported
CodeIntegrity catalog is not generated

In addition, makemsix has been updated to include the sign command.  The package, certificate, and format can be supplied, with the format inferred from the certificate file name if needed.
Signing has only been implemented under the OpenSSL Crypto PAL.  While one can still compile for Windows with pack enabled and not use OpenSSL, attempts to sign a package will fail with NotImplemented.
The design of the digest hash collection is to create an accumulator with it's own intelligent routing for the data.  This will allow the writer to pass along data as it is processing it, as well as to do so after the fact when signing an existing package.  This change implements the after the fact signing, but all that is required to sign while creating the package is the piping of the signing info into the empty writer.
Additional changes of note:

Require that target system be little endian.
Add static ComPtr::From to remove the need for almost all calls to QI.
Removed StringStream and renamed VectorStream to MemoryStream.  All uses are now of MemoryStream
Folded ZipObjectReader into ZipObject
ZipObjectWriter targets a ZipObject, rather than being one

Postponing adding this to master until we complete it.


Update index.rst

Describe the change

PR checklist

[ ] I have added description of the change I'm proposing in the OpenMS Documentation.
[ ] I have read and followed OpenMS Documentation Contributing guidelines.
[ ] I have attached a screenshot of the relevant area after this change.
[ ] CHANGELOG.md is updated.
[ ] I have added my name in CONTRIBUTING.md.

close because change seems to be wrong as it removes a quote.


net: silently ignore trust-ad option in /etc/resolv.conf and not to fallback to the cgo resolver

The /etc/resolv.conf file often includes: options edns0 trust-ad. Golang does not recognize either of those options, so it fallbacks to the cgo reoslver.
I think that trust-ad should be silently ignored. Golang does not use the ad flag anywhere, so it should be safe to silently ignore that option and not cause fallback to the cgo resolver.
Look here for details about edns0 option: #51127

cc @ianlancetaylor @neild

This would make sense, Go doesn't have the ability to do DNSSEC validation, so there isn't any alternative to trusting the AD bit.
There is one difference in behavior for glibc with this option though: If it's set, glibc sets the AD bit in queries, otherwise it doesn't. Some DNS recursors might be configured to only do DNSSEC validation if the AD (or DO) bit is set, and otherwise return everything even if it's bogus.
So in theory this could lead to Go receiving bogus data while it would've been filtered for the CGO resolver.
In practice however I think this recursor behavior is getting rare, most of them are configured to always validate DNSSEC (regardless of AD/DO bit) nowadays.

Thanks for including that link to the PowerDNS docs. It is even the default behavior of PowerDNS since 4.5.0.
It can still affect programs when trust-ad is set and the go resolver is forced (via GODEBUG env).
I don't knot if forcing the go resolver (via GODEBUG) is common, but someone might be doing it.

Also it looks like it might affect when compiled with musl libc. It does not set the AD flag when trust-ad is set.

This are the changes that I will create a PR for: trust-ad
But I don't know if I can create a PR before merging the golang/net#136 and updating the vendor directory (in this repo).


Merge/main

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #763 (e645106) into tagged-release/cms-support (2697241) will decrease coverage by 0.00%.
The diff coverage is 92.92%.

@@                      Coverage Diff                       @@
##           tagged-release/cms-support     #763      +/-   ##
==============================================================
- Coverage                       92.05%   92.04%   -0.01%
==============================================================
  Files                             114      115       +1
  Lines                            4204     4351     +147
  Branches                         1163     1210      +47
==============================================================
+ Hits                             3870     4005     +135
- Misses                            313      325      +12
  Partials                           21       21

Impacted Files
Coverage Δ

...eact/lib/forms/form-renderer-helper/value-props.ts
97.43% <ø> (ø)

...ui-react/lib/react-required-dependency-provider.ts
100.00% <ø> (ø)

...-ui-react/lib/utils/forms/array-field-component.ts
100.00% <ø> (ø)

packages/codegen-ui-react/lib/workflow/mutation.ts
96.93% <ø> (ø)

...lib/generate-form-definition/helpers/form-field.ts
100.00% <ø> (ø)

...erate-view-definition/generate-table-definition.ts
95.23% <50.00%> (-4.77%)
:arrow_down:

.../lib/amplify-ui-renderers/amplify-view-renderer.ts
77.77% <71.42%> (-4.05%)
:arrow_down:

...es/codegen-ui-react/lib/react-expander-renderer.ts
81.48% <81.48%> (ø)

.../codegen-ui-react/lib/views/react-view-renderer.ts
80.62% <92.30%> (+0.94%)
:arrow_up:

.../codegen-ui-react/lib/amplify-ui-renderers/form.ts
97.91% <100.00%> (+0.04%)
:arrow_up:

... and 17 more

Continue to review full report at Codecov.

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


Exporting SQL should have unicode prefix as "N" if DB is MS SQL

This is is related to the issue below.
https://github.com/dbeaver/dbeaver/issues/9664
I'm using MS SQL Server. Sometimes I have to use "import & export table data using SQL". These data has Japanese text, so its SQL should have unicode prefix. Because if it doesn't have "N", these data converted to "????".

current exported SQL
INSERT INTO newtable (Column1,Column2,Column3) VALUES
('1','山田','あああ'),
('2','太郎','いいい'),

Hope
INSERT INTO newtable (Column1,Column2,Column3) VALUES
('1',N'山田',N'あああ'),
('2',N'太郎',N'いいい'),

This condition is for MS SQL Server only.

Operating system (distribution) and version Windows10(2004)
DBeaver version 7.3.3
Database name and version Microsoft SQL Server ( Azure )
Driver name Microsoft SQL Server
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

Thank you for the report.

Thank you for the report.

verified


[lint addon] 'tooltipsEditor' option - disableable tooltips over text.

Tooltips over text can now be disabled without disabling tooltips
shown when mouse cursor is hovered over the gutter.

I've pushed an alternative patch that allows this without introducing an extra option -- set tooltips to "gutter" to show tooltips only on the gutter.

Thanks!


options for several methods

Allow passing a message when:
promoting  / demoting moderators
ban / unban a user (API call only, not for /ban /unban commands)
invite user
accept invite
reject invite

@Bogdan-D @pascaldekloe I did not miss a point, please consider my variant more like a proposal. Let's discuss it. Do we really need to copy methods just to provide "optional" options / because Go has no method overloading? There are some straightforward solutions, from rpc-like to Options pattern https://github.com/tmrts/go-patterns/blob/master/idiom/functional-options.md
What do you think?

@ravlio I think all other API clients accept a message parameter, not an option parameter; I also think we should not create an option type for only 1 option. I rather break the library the day we actually need more than just an optional message.

@tbarbugli I'm confused. What to choose? The optional message (as a pointer) was initially. Rolling back to it?

my point is that having methods witch requires to pass nil all the time is ugly. I haven't idea how to avoid it best when I've started work on stream-chat-go, but now I have clear picture.
@tbarbugli you also told the same in past review, now your point of view is changed?

I like this way the best from easy to use:
https://play.golang.org/p/455sVvyCjVU
the only problem godoc has bad support for this yet, to show which options can be passed to function

now should be better @tbarbugli @Bogdan-D


Is it possible for hint to show in each region?

Thank you so much for the hard work on this project, i am writing to ask if it is possible for hints to show in each region.
For example i am making use of the Nigeria map and i want the name of each state to show before hovering or clicking.

Welcome @folarmi ,
Currently there is no way to implement hints without hover. The main issue with this is the irregular shape of many states which makes it unclear to dynamically show the hints by default..
Would have to research it a bit. Would work on this next after #22


Editor inspector fails to display MRTK IconSet

Describe the bug
When selecting a MRTK IconSet in the Unity Editor, the icons are not displayed. Instead an error is displayed in the inspector "You can't nest FoldoutHeaders, end it with EndHeaderFoldoutGroup.".
To reproduce
Steps to reproduce the behavior:

Install MRTK standard assets.
Navigate to default IconSet
Look at inspector pane

Expected behavior
Should see icons in set.
Screenshots

Your setup (please complete the following information)

Unity 2020.2.2f1
MRTK 2.5.4

Target platform (please complete the following information)

HoloLens 2

Additional context

Having the same issue. Is there a temporary workaround?

Hmm, I'm not able to fully repro. Am I missing anything? I'm able to view the asset, but unable to fold out any of the foldouts due to packages being immutable:

I re-imported the project (close project, delete library folder, reopen, set platform and build settings) and I can't reproduce this now. At a guess it could have been  some out of date asmdef.

Hi, I'm having the same issue.

Your setup (please complete the following information)
Unity 2020.3.11f1
MRTK 2.7.0

@RogPodge Was this an issue fixed by https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9999?

Yep! should be fixed

same problem

Same problem,
Created a new icon set, Quad and Sprite Icons error in inspector panel:
"You cant nest Foldout Headers, end it with EndHeatherFoldoutGroup"

This problem still exists in MRTK 2.7.2

The fix is checked into the main repo but is not yet released in one of our official packages.
We are looking to put out a release which contains this bug fix before end of the year, so stay tuned

Thank you for your reply! I downloaded the fix and applied it to the mrtk core package, this solved the problem.

@MilanPolle
I am facing the same issue with MRTK 2.7.2
May I ask what do you mean by you downloaded the fix?

@kkyuri0419 the file with the fix is here: https://raw.githubusercontent.com/RogPodge/MixedRealityToolkit-Unity/02f3e41f650a14eb744b41507270fea0d0470d6a/Assets/MRTK/SDK/Features/UX/Scripts/Buttons/ButtonIconSet.cs
Open the /packages/MixedReality folder, in your Unity project and then open the toolkit.foundation tgz file there (with 7Zip, for instance).
In the tgz file, navigate to /package/SDK/Features/UX/Scripts/Buttons/ and there you can replace the ButtonIconSet.cs file with the one you've downloaded.
Unity will process the changes by itself and then you should be able to add your icons in the editor without the errors that were there.

@kkyuri0419 the file with the fix is here: https://raw.githubusercontent.com/RogPodge/MixedRealityToolkit-Unity/02f3e41f650a14eb744b41507270fea0d0470d6a/Assets/MRTK/SDK/Features/UX/Scripts/Buttons/ButtonIconSet.cs
Open the /packages/MixedReality/ folder, in your Unity project and then open the .toolkit.foundation tgz file there (with 7Zip, for instance).
In the tgz file, navigate to /package/SDK/Features/UX/Scripts/Buttons/ and there you can replace the ButtonIconSet.cs file with the one you've downloaded.
Unity will process the changes by itself and then you should be able to add your icons in the editor without the errors that were there.

Thank you so much @MilanPolle !! it perfectly works!

@kkyuri0419 Glad to hear that! Thanks for letting me know.

Just curious, since this was fixed in March what caused it to be missed in the 2.7.2 release? Also, would love to see this fixed in the next package release. It's kind of painful to apply the hotfix above and the next MRFT update will likely overwrite it. Thanks!

Just curious, since this was fixed in March what caused it to be missed in the 2.7.2 release?

The PR that fixed this was #9999, which was merged in June and just missed the 2.7.2 stabilization cutoff. It'll definitely be in our next release!

The link is not available now.

You may fix the bug with the link.
The easiest way is to update to 2.7.3 fundamental with MixedRealityFeatureTool.
Additionally.
I tried exchanging ButtonIconSet.cs into tar at the package and tied it again.
but It's not working.


Renames and terminology

I'd like to discuss the rename suggestions in dthree/vorpal#95 and put them into one commit:

this.stdout => this.log and this.stderr => this.error: I agree with these in the context of vorpal, but not for the underlying implementation. On this low level, stdout and stderr reflect more accurately what they do.
onInput => data: I like using a stream analogy, so :+1: for this.
cleanup => end: Same as above, but: We could rename the 'exit' event to 'end', in which case 'cleanup' should stay 'cleanup'.

On a similiar note to cleanup => end vs 'exit' => 'end' there is also init (first lifecycle function) and start (instance method of CommandInstance to start the lifecycle) and 'ready' (event emitted when ready to accept input). Feel free to share any better naming ideas for these three.

Vorpal will most likely end up wrapping this module anyways, so it can inject its own terminology where necessary.

Okay I'm good! :+1:


Installed package versions depend upon argument order in uv pip install

Hi, I noticed when installing a package that the versions installed using uv pip install depends upon the order of the packages given to uv whereas when using pip the versions installed are consistent. For instance running
uv pip install importlib_metadata pysnyk produces the following environment:
Package            Version
------------------ --------
certifi            2024.7.4
charset-normalizer 3.3.2
idna               3.7
importlib-metadata 8.2.0
mashumaro          1.24
msgpack            1.0.8
poetry-version     0.1.5
pysnyk             0.3.1
pyyaml             6.0.1
requests           2.32.3
tomlkit            0.5.11
typing-extensions  4.12.2
urllib3            2.2.2
zipp               3.19.2

Whereas if you run uv pip install pysnyk importlib_metadata you get the following environment:
Package            Version
------------------ --------
certifi            2024.7.4
charset-normalizer 3.3.2
decorator          5.1.1
deprecation        2.1.0
idna               3.7
importlib-metadata 6.11.0
mashumaro          3.13.1
packaging          24.1
py                 1.11.0
pysnyk             0.9.19
requests           2.32.3
retry              0.9.2
typing-extensions  4.12.2
urllib3            2.2.2
zipp               3.19.2

Pip seems to always produce the last result no matter the order of the packages, and from reading the docs I would expect uv pip install to produce the same result as pip here consistently?
The examples above were produced with python3.11 and uv 0.2.29.

Yes, that's possible and would be correct behavior. There's a good chance that pip would do exactly the same thing.

We can't invent constraints that don't exist. Who knows, the inverse order could be equally problematic.

If I understood the resolution strategy correctly, if you have for instance a package Y that depends upon numpy and has pinned it to numpy<2. Then if you do uv pip install numpy Y (or lists numpy before package Y in you requirements file), then uv will install the latest numpy version 2.0.1 and then it will find a version of Y that does not have numpy pinned, which is propably a version released before the announcement of the numpy 2 release, so it will most certainly not work with numpy 2?

As a dependency resolution algorithm, we need to trust the packages that their metadata is correct (with the escape hatches of constraints and overrides), so when a package adds a new constraint, we have to assume that the new release has become incompatible. When a package does not provide upper bounds, we have to assume it's compatible with all versions, we have no means to figure out the actual bound in uv.

The thing that surprises me the most about the examples above is the fact that a package that has no dependencies itself, (e.g. numpy) will if placed properly "determine" the version of the package that depends upon it. I would have thought that the opposite would always be the case in simple cases like this. That you start resolving from the highest version of the package that has the other packages as dependencies.

While this is possible algorithmically, it would be prohibitively expensive in uv's case. The by far slowest part is fetching metadata (even when we just read it from the cache), so we make the decision which package to prioritize without metadata. Another aspect is that the number of dependencies can change between releases, so we'd need to come up with a reliable, robust topological prioritization. Our current rules are using tighter specifiers first (numpy==... before torch>=...), which helps trying fewer wrong versions, and otherwise apply them in the order we first saw a package.

I recently had a quite confusing problem related to this. I don't think I understand why the resolver installs the version it does.
requirements.txt:
sphinx
sphinx_rtd_theme

environment:
Package                       Version
----------------------------- --------
alabaster                     1.0.0
babel                         2.15.0
certifi                       2024.7.4
charset-normalizer            3.3.2
docutils                      0.21.2
idna                          3.7
imagesize                     1.4.1
jinja2                        3.1.4
markupsafe                    2.1.5
packaging                     24.1
pygments                      2.18.0
requests                      2.32.3
snowballstemmer               2.2.0
sphinx                        8.0.2
sphinx-rtd-theme              0.5.1
sphinxcontrib-applehelp       2.0.0
sphinxcontrib-devhelp         2.0.0
sphinxcontrib-htmlhelp        2.1.0
sphinxcontrib-jsmath          1.0.1
sphinxcontrib-qthelp          2.0.0
sphinxcontrib-serializinghtml 2.0.0
tomli                         2.0.1
urllib3                       2.2.2

requirements.txt:
sphinx_rtd_theme
sphinx

environment:
Package                       Version
----------------------------- --------
alabaster                     0.7.16
babel                         2.15.0
certifi                       2024.7.4
charset-normalizer            3.3.2
docutils                      0.20.1
idna                          3.7
imagesize                     1.4.1
jinja2                        3.1.4
markupsafe                    2.1.5
packaging                     24.1
pygments                      2.18.0
requests                      2.32.3
snowballstemmer               2.2.0
sphinx                        7.4.7
sphinx-rtd-theme              2.0.0
sphinxcontrib-applehelp       2.0.0
sphinxcontrib-devhelp         2.0.0
sphinxcontrib-htmlhelp        2.1.0
sphinxcontrib-jquery          4.1
sphinxcontrib-jsmath          1.0.1
sphinxcontrib-qthelp          2.0.0
sphinxcontrib-serializinghtml 2.0.0
tomli                         2.0.1
urllib3                       2.2.2

I guess I can omit listing sphinx here since sphinx_rtd_theme already requires sphinx, but what would be the rule of thumb here for properly listing your deps in order? Is it the package at the top of the dependency tree first, in this case sphinx_rtd_theme?
Does anybody know what pip would do here, since order does not seem to matter with pip?

We have sphinx-rtd-theme==2.0.0: sphinx>=5, <8, so a resolver has to make a decision whether to pick the highest version of the theme, or the latest version of sphinx. From a resolver's perspective, both options are equally valid, while i'd expect you want the newest theme version with sphinx 7. It seems that pip does this choice in a different order than uv, where uv considers the order in your input file, pip doesn't. Your best path moving forward is probably adding lower bounds for the theme and for sphinx, this will make any resolver produce the result you want.
From a pubgrub perspective, this is an interesting problem: Pubgrub works so that we first select a package, and then try all versions for that package. That means that we select either sphinx or sphinx-rtd-theme before we know that there is a sphinx-rtd-theme -> sphinx dep we want to consider. What we'd need to do to generate the desirable solution and this and the original case is: We have picked sphinx, and we have found a compatible version of sphinx. We next select sphinx-rtd-theme, and try the first version of sphinx-rtd-theme. We see a dep onto sphinx that is incompatible. We would now have to switch prioritization, lock in that version of sphinx-rtd-theme, remove the selection of sphinx from the partial solution (something that pubgrub doesn't support atm) and start iterating over versions of sphinx instead.

Discussion welcome to continue here — just closing as I've documented this now.


problem in mac

hello i can't get this to work in both methodes , does this works in macos catalina ?

Neither can i

It works for me in mac catalina. I am using chromium though.

Can confirm neither chrome nor Chromium Work on Mac Os Catalina
/Applications/Google\ Chrome.app/Contents/MacOS/Google\ Chrome --remote-debugging-port=9222

/Applications/Chromium.app/Contents/MacOS/Chromium --remote-debugging-port=9222


chore: make sure eye version updates do not remove the newline charac…

…ter from the package.json

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Convert tooltips and popovers to CSS vars

Summary
This PR starts the process of migrating tooltips and popovers to use CSS variables. In doing so, I've deprecated three Sass variables in an effort to improve customization via CSS variables. See _variables.scss for the changes there.
Beyond that, this replaces the bulk of our Sass for tooltips and popovers with customizable CSS variables. Most are reassigned from the Sass variables and respect the same name, but as mentioned above, some deviation was required for a more sensible future friendly path.
Originally I expected tooltips and popovers to be a clusterfuck to customize, but thanks to the customClass option we recently added, this is basically a breeze. I've added the data attributes, wrote some basic styles, and added some lovely docs to explain and demonstrate the situation for folks.
Preview

Tooltips: https://deploy-preview-34622--twbs-bootstrap.netlify.app/docs/5.1/components/tooltips/#custom-tooltips
Popovers: https://deploy-preview-34622--twbs-bootstrap.netlify.app/docs/5.1/components/popovers/#custom-popovers

Todos

[x] Document deprecations maybe in the Migration guide?
[x] Confirm all CSS vars are properly being used (fusv doesn't do that for us)
[x] Adjust bundlewatch

@ffoodd Would love your eyes on this and https://github.com/twbs/bootstrap/pull/34443. In particular, I'm unsure how we should setup the font-size values while still supporting RFS. Rest of this is feeling good to me though!

Regarding font-sizes and RFS, I don't see any other option if we want to use RFS.
Note: regarding my suggestion PR using fallback values, we might check that value is above $rfs-base-value: if so, define var()'s fallback value, else use rfs() as you currently do. Not sure the extra effort is needed though.

Updated this PR to also split the padding values for tooltips, popover headers, and popover bodies.


ASM does not handle standard customer sessions

ASM doesn’t not handle the case where and ASM agent is authenticated + a regular customer session is in progress.
To repro:

Sign in asm as a customer support agent.
Sign in the storefront as a regular customer using a username and password in the storefront login form.

Then:

Observe what the Asm UI displays the same thing as when an emulation session is in progress.
Observe that when the agent logs out, the standard customer session is also terminated.

ASM can handle the case by:

Instead of the UI that is displayed during customer emulation, display a message that a regular customer session is in progress.
In the case of a regular customer session, when the agent logs out, only log out the agent, not the regular customer session.

PR #5284

To test:
Case 1: Emulation

Sign in asm as a customer support agent.
Start a customer emulation session

Then:

Observe that the Asm UI displays the name and email of the emulated customer.
Observer that when the agent signs out, the emulated customer session is also terminated.

Case 2: Standard customer session

Sign in asm as a customer support agent.
Sign in the storefront as a regular customer using a username and password in the storefront login form.

Then:

Observe that the Asm UI displays a message that the customer has to sign out for the agent to use asm.
Observer that when the agent signs out, only the csagent is signed out.

Tested on dev10.
Verified on all 3 browsers (chrome, firefox, safari).

agent logs in, starts customer session, agent signs out, the customer's session is also ended.
agent logs in, starts customer session, end customer session still works.
agent signs in, customer signs in through login form, agent signs out, the customer is still signed in.
customer signs in through login form, agent signs in, agent signs out, customer is still signed in.
agent signs in, customer signs in through login form, customer signs out through myaccount, agent starts another customer's session, agent signs out, customer's session is also ended.

Also verified on mobile, it's ok. the spacing is just a bit too wide, but it's very unlikely that a user will use asm on a mobile.


Feat/cli_commands

Codecov Report
Base: 30.43% // Head: 32.39% // Increases project coverage by +1.95% :tada:

Coverage data is based on head (235d3c0) compared to base (ea657b0).
Patch coverage: 80.28% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
+ Coverage   30.43%   32.39%   +1.95%
==========================================
  Files         103      104       +1
  Lines        2661     2670       +9
==========================================
+ Hits          810      865      +55
+ Misses       1851     1805      -46

Impacted Files
Coverage Δ

...c/configurations/sidecar_spec/package_options.dart
57.14% <ø> (+46.03%)
:arrow_up:

.../src/configurations/sidecar_spec/rule_options.dart
100.00% <ø> (+66.66%)
:arrow_up:

...figurations/sidecar_spec/sidecar_spec_parsers.dart
67.85% <ø> (ø)

...decar/lib/src/services/active_project_service.dart
71.05% <0.00%> (-2.92%)
:arrow_down:

.../lib/src/services/rule_initialization_service.dart
0.00% <ø> (ø)

...configurations/sidecar_spec/package_options.g.dart
50.00% <50.00%> (+25.00%)
:arrow_up:

...rc/configurations/sidecar_spec/rule_options.g.dart
52.63% <50.00%> (+21.38%)
:arrow_up:

...lib/src/configurations/sidecar_spec/add_rules.dart
90.90% <90.90%> (ø)

...configurations/sidecar_spec/sidecar_spec_base.dart
13.33% <100.00%> (-0.56%)
:arrow_down:

...nfigurations/sidecar_spec/sidecar_spec_base.g.dart
100.00% <100.00%> (+54.54%)
:arrow_up:

... and 4 more

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

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