Ubuntu latests uses already installed mysql instead the declared service

Description
Currently ubuntu-latest is pointing to ubuntu 20.04, after this change my CI system started to fail because the rails application tried to use the mysql 8.0 version installed by default in the ubuntu instead of the mysql:5.7 declared as a service.

The documentation still show ubuntu-latest as the ubuntu 18.04 image.
I found this error debugging my github action with the tmate action

Area for Triage:  Containers

Question, Bug, or Feature?:  Bug

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[x] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Image version

ubuntu-latest
Expected behavior
The container should take the mysql:5.7 service declared in my workflow instead of the mysql 8.0 that is already installed in the container.
Actual behavior
mysql 8.0 installed inside ubuntu 20.04 (latest) is take as the mysql service, despite I have a mysql:5.7 declared in the workflow
      mysql:
        image: mysql:5.7.33
        env:
          MYSQL_ROOT_PASSWORD: root
        ports:
        - 3306
        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=10

Repro steps
This is the workflow that I'm running (I've removed some unnecessary dependencies and folders from the matrix)
name: Main
on:
  pull_request:
    branches: [ daily, develop ]
  push:
    branches: [ daily, develop ]
jobs:
  test_runner:
    name: CI
    runs-on: ubuntu-latest
    # If you need DB like MySQL then define service below.
    # Example for PostgreSQL and Redis can be found here:
    # https://github.com/actions/example-services/tree/master/.github/workflows
    services:
      # How to use MySQL
      mysql:
        image: mysql:5.7
        env:
          MYSQL_ROOT_PASSWORD: root
        ports:
        - 3306
        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=10
      # How to use Elasticsearch
      elasticsearch:
        image: elasticsearch:6.8.3
        ports:
        - 9200/tcp
        options: -e="discovery.type=single-node" --health-cmd="curl http://localhost:9200/_cluster/health" --health-interval=10s --health-timeout=5s --health-retries=10
# https://help.github.com/en/articles/workflow-syntax-for-github-actions#jobsjob_idstrategymatrix
    strategy:
      fail-fast: false
      matrix:
        test_folder: [test/models]
    steps:
      - name: Checkout
        uses: actions/checkout@v2
        with:
          fetch-depth: 1
      - name: Set up Ruby
        uses: ruby/setup-ruby@v1
        with:
          bundler-cache: true
      - name: Create DB
        env:
          RAILS_ENV: test
          DB_PASSWORD: root
          DB_PORT: ${{ job.services.mysql.ports[3306] }}
        run: |
          sudo /etc/init.d/mysql start
          cp config/database.ci.yml config/database.yml
          bin/rails db:setup
      - name: Setup translations
        run: |
          bin/rails makemo > /dev/null 2>&1
      - name: Run tests
        env:
          DB_PASSWORD: root
          DB_PORT: ${{ job.services.mysql.ports[3306] }} # get randomly assigned published port
          ELASTICSEARCH_PORT: ${{ job.services.elasticsearch.ports[9200] }}
          RAILS_ENV: test
        run: |
          bin/rails test ${{matrix.test_folder}}

you can add
- name: Setup tmate session
        uses: mxschmitt/action-tmate@v3

and ensure if you run RAILS_ENV=test bin/rails db or mysql --version you get a mysql 8.0 versión instead the declared mysql:5.7 service.
I've temporary fixed it updating runs-on: ubuntu-latest to runs-on: ubuntu-18.04 but It would be nice to be able to use ubuntu-latest as usual.

Hello @JuanCrg90. Unfortunately we are unable to reproduce the issue with the repro steps in your workflow due to missing some of the configuration files mentioned there.
We have tested the mysql 5.7 service on the ubuntu-latest image using the workflow below:
name: Service_test
on:
  workflow_dispatch:

jobs:
  test_runner:
    name: CI
    runs-on: ubuntu-latest
    services:
      mysql:
        image: mysql:5.7
        env:
          MYSQL_ROOT_PASSWORD: password
        ports:
        - 3306
        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=10

    steps:
      - name: Checkout
        uses: actions/checkout@v2
        with:
          fetch-depth: 1
      - name: mysql version
        run: mysql --host <IP_ADDRESS> --port ${{ job.services.mysql.ports['3306'] }} -uroot -ppassword -e "SELECT @@VERSION"

We got the 5.7 version as expected (https://github.com/nikita-bykov/test-repo/actions/runs/657369769).
Most likely your workflow is configured to use mysql from the host instead of docker container. Ubuntu 18.04 has mysql 5.7 installed on host as you configured in the container, and this is probably why you've had successful builds before.

Hello @JuanCrg90, please feel free to contact us if you have any questions. If not, then we will close the issue.

@JuanCrg90, I closed the issue, but if you have any questions feel free to reopen it.


A11y keyboard focus control in the Dialog Component

Bug, feature request, or proposal:
Bug: Dialog missing page structure.
What is the expected behavior?
The Dialog modal should define the main content region for context understanding.
What is the current behavior?
Currently the first focusable element in the Dialog is at the bottom of the page, skipping the screen reader user over all of the Dialog content.
What are the steps to reproduce?
See below.
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
What is the use-case or motivation for changing an existing behavior?
Create a good screen reader user experience.
Which versions of Angular, Material, OS, browsers are affected?
Using screen reader JAWS17 with browsers Firefox, Internet Explorer 11, and Google Chrome, on Windows10.
Is there anything else we should know?
Screen reader user experience follows:

Press Tab key to the "Dailog with a result" demo component in the Tab Panel.
JAWS says "LAUNCH DIALOG button", and the braille display shows "LAUNCH DIALOG btn".

Note, the typing error in "dailog".
The CSS should be used to style text instead of using all capital letters in text labels.

Press Spacebar or Enter key on the "LAUNCH DIALOG" button.
JAWS says "option 1 button", and the braille display shows "OPTION 1 btn".

This is a simple Dialog, and presents no accessibility challenge. However, the first focusable element is at the bottom of the Dialog modal, skipping over all the content.
The md-dialog-title module is defined by the h1 tag, but the md-dialog-content module (Primary scrollable content of the dialog) and the
md-dialog-actions module (Container for action buttons at the bottom of the dialog) have no defined structure markup.
The CSS should be used to style text instead of using all capital letters "OPTION".
Note, the buttons are labelled "OPTION 1" and "OPTION 2", which perform the same function "Close". The md-dialog-close is not clearly labelled.
The Arrow and Tab keys function as expected in the Dialog modal, and the Option buttons close the Dialog and return the focus to the "LAUNCH DIALOG" element as expected. Pressing Down Arrow announces the message "You chose: Option 1" .

The "Overview" Tab of the Dialog demo has a "OPEN DIALOG" button, which does not open a Dialog modal. Pressing Spacebar or Enter key on this button appears to do nothing, as the screen reader does not change focus or announce any change. However, the text "Hi, I'm a dialog!" appears several times at the bottom of the screen. This may be a demo issue.

Hi DaveBeast99,

Please confirm: should pressing "Escape" key close the dialog without taking any action?
See this other implementation which close the dialog on Escape :

http://demos.wijmo.com/5/Angular2/Explorer/Explorer/#/input/popup
http://shlomiassaf.github.io/angular2-modal/#/js-native-demo

Typically, the Escape key is considered to be a Cancel key. So, if a dialog has been modified, the Escape key should cancel the changes. However, there is no fixed rule for using the Escape key, but consistency, in all pages and components, is important to achieve a good user experience. That is, a ComboBox selection should perform the same result. A Dialog box may have an Ok and Cancel button, and the Escape key would mimic the Cancel Button. So, I would say Yes to your question.
David
From: vipdorian<EMAIL_ADDRESS>Sent: April 15, 2017 01:54 PM
To: angular/material2
Cc: David Best; Author
Subject: Re: [angular/material2] A11y keyboard focus control in the Dialog Component (#3769)
Hi DaveBeast99,

Please confirm: should pressing "Escape" key close the dialog without taking any action?
See this other implementation which close the dialog on Escape :

http://demos.wijmo.com/5/Angular2/Explorer/Explorer/#/input/popup
http://shlomiassaf.github.io/angular2-modal/#/js-native-demo

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/angular/material2/issues/3769#issuecomment-294308297 , or mute the thread https://github.com/notifications/unsubscribe-auth/AVPBNgnIjkbSZkyGU39agHYNor3KSMSsks5rwQQtgaJpZM4MoxZK .  https://github.com/notifications/beacon/AVPBNqG5r3h_L6CJIhzCFMsNVViwhJ1_ks5rwQQtgaJpZM4MoxZK.gif

The issues here all seem to be problems with the demos rather than the underlying dialog component.

The overview demo has no focusable elements inside of the dialog, so focus doesn't move into the dialog. This isn't an intended use case for the component and the demo should be fixed.
It seems like the demo under examples should have a paragraph tag inside of the md-dialog-content container.


appears the recursion is only 1 level deep.

Need infinite recursion.  Current code only lists parent's child's dep.

The program should analyze the dependencies of a program and the dependencies of the dependencies through all levels.
What commandline did you use?
What project (if I could find it on github)`?

Closed as stale.


Support for addional input resources

Description
Add support for additional input resources that are referenced as resource-only.
Motivation and Context
This allows processing e.g. individual topics with key references without needing to create a wrapper map that defines the keys.
Example
topic.dita
<topic id="topic">
  <title>Installing <keyword key="product"/></title>
  …
</topic>

keys.ditamap
<map>
  <keydef keys="product">
    <topicmeta>
      <keywords>
        <keyword>Product</keyword>
      </keywords>
    </topicmeta>
  </keydef>
</topic>

When processed with
dita -i topic.dita -r keys.ditamap -f html5

Will result

Installing Product
…

Example of cases where this will not work as expected. If we have root.ditamap
<map keyscope="root">
  <mapref href="keys.ditamap">
  <topicref href="topic.dita">
</topic>

In topic.dita we reference a key in the scope root
<topic id="topic">
  <title>Installing <keyword key="root.product"/></title>
  …
</topic>

If we try to convert the topic alone with keys as the resource
dita -i topic.dita -r keys.dita …

We will get an error message

[DOTJ047I][INFO] Unable to find key definition for key reference "root.key" in root scope…

This is because the root map uses the scope root and that's what the topic expects, but the key map just inherits it. This is not a bug in the code, but rather just a limitation. This could be fixed by adding yet another property, like root-key-keyscope. That would allow you to manually change the key scope all scopes will be added to. So if you set --root-key-keyscope=root for the case above, that would work. But I'm not sure if it has any other real use cases, because I don't know how people really use key scopes in the wild.

So the result is as if the entire context keys map is referenced with processing role resource only from another map which also includes the published topic? That’s clever. Does this work with both preprocess and preprocess2 targets?
Will try to build off the branch and look more into the code.
From the list of possible types of key refs:
Keyword with Keyref,
Image with keyref
Conkeyref
Xref with keyref
Probably the last one does not work, especially in a PDF but I would not expect it to.
Curious about what is shown in place of link though...

It should work fine with any type of keyref usage, since after the key is resolved, other preprocess steps handle the rest. Like conrefs or link texts.

Curious about a couple of things, and I don't know the answer to all of them:

If you have multiple input resources, which key will win? I'm assuming they are evaluated in order (so if the first and second resource maps define keys="example", the first one would win)?
That might get uglier if you have to go into submaps to get keys, due to the evaluation rules about depth-of-definition. For example, if your input resources are first.ditamap and second.ditamap, where first defines after several levels of nested maps, and second defines the same key directly.
If the key resolves to a relative path, what do you evaluate that against? The usual DITA-OT issue with paths, if the resource and input topic are not in the same directory, where does the path go?
Can I use a topic as an input resource, or just maps? (Not sure what the reason for using a topic would be, since we will already pull in conref and link targets as needed.)

The "args.resources" new param probably needs to be added to the base plugin.xml as a new param. In this way Oxygen will be able to show it in the Parameters list for the transformation scenarios.
I built and stared using the DITA OT with the changes on a topic which has image with keyref, plain keyref, link with keyref and conkeyref.
I'm attaching a sample zip below:
testSingleTopicOutput.zip
When publishing with plain HTML5 the conkeyref is the only one not getting resolved for some reason, the target for it seems to be searched for in the wrong folder.
I also published a topic from the Oxygen's user manual DITA project which contained only keyrefs to the "product" key. The "branch-filter, keyref, move-meta-entries" stages are lightweight and performed only on the published topic, all other steps including the "html5.topics.common.inner" and "image-metadata" seem to be performed on all topics.

Adding args.resources to plugin.xml means we also have to add support for parameter type files or path, because args.resources is a list of files delimited by path separators.

In order to only get HTML output for the single file in @raducoravu example, we need to dynamically add processing-role="resource-only" to resource map root.

@raducoravu in your testSingleTopicOutput.zip, are you running topics/install-eclipse.dita as the input file and simple.ditamap as a resource?

@jelovirt I'm actually publishing "test_topic.dita" with the "simple.ditamap" set as additional resource, sorry that I did not state this from the beginning.

Fixed conkeyref resolution.

@raducoravu, are you able to test this again? I've been testing with your testSingleTopicOutput.zip and that seems to work now. Would appreciate a bit more testing before merging this.

I'm running this command line:
      dita -i C:\Users\radu_coravu\Downloads\testSingleTopicOutput\testSingleTopicOutput\topics\test_topic.dita -f pdf -o "d:\outputTest" --args.resources=C:/Users/radu_coravu/Downloads/testSingleTopicOutput/testSingleTopicOutput/simple.ditamap

and I get this error:
     Error: net.sf.saxon.trans.UncheckedXPathException: Supplied element name is a zero-length string

which is similar to what I reported here: https://github.com/dita-ot/dita-ot/issues/3395
and again it's hard to get the stack trace of the error or the template where this occurs.

@raducoravu added catches for unchecked XPath exception. Are you able to test again, because I'm not able to reproduce that issue.

@jelovirt added your latest changes, compiled the project. Now I cannot reproduce the "Supplied element name is a zero-length" when generating the PDF. Generation works, PDF looks good inside, keyrefs, conkeyrefs expanded. So I don't really undestand why this did not work for me before.

I've been having problems obtaining plain PDF output from a simple DITA Map, something like:
      dita -i D:\projects\eXml\samples\dita\flowers\flowers.ditamap -f pdf -o "d:\outputTest3" -t "d:\abcTMP3"  -d -v

no longer works on a DITA Map which has a plain topic referenced in it, I get a "topic.fo" which is invalid, contains nothing in the "xsl-region-body" flow:
      [fop] [ERROR] net.sf.saxon.trans.XPathException: org.apache.fop.fo.ValidationException: "fo:flow" is missing child elements. Required content model: marker* (%block;)+ (See position 1:10681)
  [fop] [ERROR] Anttask - Error rendering fo file: d:\abcTMP3\topic.fo <org.apache.fop.apps.FOPException: org.apache.fop.fo.ValidationException: "fo:flow" is missing child elements. Required content model: marker* (%block;)+ (See position 1:10681)

The HTML5 output on the topic from the original sample project works but it has its quirks, the output folder contains both an "index.html" which is the actual topic with all keyrefs and conkeyrefs expanded (but which has a broken reference to the image file). There is also an HTML equivalent of the original topic in a "topics" subfolder and in this case the image reference is properly resolved.

PDF topic still has issues.

@jelovirt also not sure if you've seen my comment above about regular DITA Map to PDF not working anymore: https://github.com/dita-ot/dita-ot/pull/3412#issuecomment-561512604

@raducoravu PDF works now, either with or without resources.

I  can confirm things now work out on the sample project I initially tested with.
Then I went back to the Oxygen user's manual and tried to publish a topic to PDF, it seems that if you have submaps things no longer work that well.
For example the zip below has a submap which refers to topics and defines keys. The conkeyref inside the "test_topic.dita" no longer resolves on my side. Neither the keyref link from the test_topic.dita to the other topic.
testSingleTopicOutputReusablesSubmap.zip

@raducoravu with the submap example, what did you convert with what arguments?

@jelovirt I run this command line:
  dita -i C:\Users\radu_coravu\Downloads\testSingleTopicOutput\topics\test_topic.dita -f pdf -o "d:\outputTest" --args.resources=C:/Users/radu_coravu/Downloads/testSingleTopicOutput/simple.ditamap -d -v

@raducoravu I created https://github.com/dita-ot/dita-ot/pull/3429 to fix this. The root cause is that when you have topic input file, then we set noMap. It was used to disable mapref processing, but now that we can have maps from additional resources, that guard/optimization needs to be relaxed.
Based on my testing this fixes your problem with additional resources with submaps.


hello 0.1.0-82428530+autotest_by_mosteo@kukumo

Created via alr publish with alr 2.0.2-dev

Closed using alr 2.0.2-dev with reason: 'automated testing'


[W5][W14-3]Ian Wu Ling En

Add 'Edit' command
Add IO tests for 'Edit' command
Add JUnit tests for 'Edit' command
Update UserGuide.adoc for 'Edit' command
Update 'Find' command to case-insensitive search
Update IO tests for 'Find' command
Update JUnit tests for 'Find' command
Update UserGuide.adoc for 'Find' command

This is a great feature! Now I can sort the names and stalk each of them alphabetically. Thanks Ian!

This makes everything so convenient. Well done!

Very well documented PR, keep up the work.

Making find case-insensitive is a good choice and will improve users' convenience surely.

Good job adding and detailing the Edit and Find commands!


HAWQ-1051. Failing in reverse DNS lookup causes resource manager core…

… dump

It would be great if the commit message can be updated to include both stanlyxiang and jiny2 as author.

@huor Could you give a commit example like this?

@xunzhang, here is a tool with that support: https://github.com/git-duet/git-duet. You can also find the example in page home page.

Merged.


Brightness Slider does not work reliably in Home Assistant

I noticed that the Brightness Slider in Home Assistant is often unresponsive, especially if you change the brightness quickly a few times.
Did anyone notice the same thing? MQTT hick-up?

I just tested it, and it seems to work fine. But I could think, that there might be a timing issue on slow networks. When you moved the slider the value is send to PictureFrame. Changed values are send back to home assistant.
In that way home assistant will be in syc with the web-interface.
But if you change the values too fast (you have to release the slider), home assistant might receive an  older value and thinks it's a remotely set value and will update the value to an old value.

But if you change the values too fast (you have to release the slider), home assistant might receive an older value and thinks it's a remotely set value and will update the value to an old value.

Yes, I think that's it. However, my network is super fast, so maybe there is a limit of MQTT commands per second.

By the way. You can change the slider to input field. That might 'slow' you down. Sorry, but for now, I don't have a better solution.


add VECTOR doc

DOC-10634

This looks correct from an Engineering perspective. Perhaps @mgartner would want to weigh in as well.
From the product perspective, I think we should consider adding some more exposition about how to use the feature for a simple embedding search with common data, such as embeddings from OpenAI. But I will let @dikshant weigh in on that.

I think we should consider adding some more exposition about how to use the feature for a simple embedding search with common data, such as embeddings from OpenAI

We will cover some of this in an upcoming blog post (targeted for mid September) and we can update the docs then.


ValueError during Processing Foldseek output

Thanks for making and maintaining pharokka and phold!

pharokka version: 1.7.1
phold version: 0.1.4
Python version: 3.10.14
Operating System: Red Hat Enterprise Linux 8.8 (Ootpa)

Description
I'm running pharokka -> phold on a set of 853 complete (single-contig) phage genomes. phold gives a per_cds_predictions.tsv file for all phage except one: Pseudomonas phage PIP. After rerunning a few times, this does not appear to be a memory issue. Perhaps an interesting edge case?
What I Did
module load miniconda/24.3.0
conda activate /home/acv38/project/conda_envs/pharokka
mkdir -p /home/acv38/palmer_scratch/psa_promoters/annotations/pharokka/OR687155.1
pharokka.py -i /home/acv38/project/databases/pseudomonas_phage_16Apr2024/fna/OR687155.1.fna -o /home/acv38/palmer_scratch/psa_promoters/annotations/pharokka/OR687155.1 -f -p OR687155.1 -d /home/acv38/project/databases/pharokka/pharokka_v1.4.0_databases -t 8 -g phanotate
conda deactivate
conda activate /home/acv38/project/conda_envs/phold
mkdir -p /home/acv38/palmer_scratch/psa_promoters/annotations/phold/OR687155.1
phold run -i /home/acv38/palmer_scratch/psa_promoters/annotations/pharokka/OR687155.1/OR687155.1.gbk -o /home/acv38/palmer_scratch/psa_promoters/annotations/phold/OR687155.1 -f -p OR687155.1 -d /home/acv38/project/databases/phold/phold_structure_foldseek_db -t 8 --cpu
conda deactivate

Please find the original fasta file, the pharokka gbk file, my conda yml files, and all the relevant log files at this Dropbox link:
https://www.dropbox.com/scl/fo/29tp5my2me718pr6wwb5c/AH8ggBgegenkhEhSvlZGdSs?rlkey=ivmz8w6zkdr7i3v89013ocwth&st=fcppkjqi&dl=0
Error traceback
2024-06-17 11:05:32.017 | INFO     | phold.results.topfunction:get_topfunctions:35 - Processing Foldseek output
Traceback (most recent call last):
  File "/home/acv38/project/conda_envs/phold/bin/phold", line 10, in <module>
    sys.exit(main())
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/phold/__init__.py", line 1355, in main
    main_cli()
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/phold/__init__.py", line 298, in run
    subcommand_compare(
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/phold/subcommands/compare.py", line 372, in subcommand_compare
    filtered_topfunctions_df, weighted_bitscore_df = get_topfunctions(
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/phold/results/topfunction.py", line 58, in get_topfunctions
    foldseek_df[["contig_id", "cds_id"]] = foldseek_df["query"].str.split(
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/pandas/core/frame.py", line 4299, in __setitem__
    self._setitem_array(key, value)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/pandas/core/frame.py", line 4341, in _setitem_array
    check_key_length(self.columns, key, value)
  File "/gpfs/gibbs/project/turner/acv38/conda_envs/phold/lib/python3.10/site-packages/pandas/core/indexers/utils.py", line 390, in check_key_length
    raise ValueError("Columns must be same length as key")
ValueError: Columns must be same length as key

Hi @acvill ,
This is a very weird phage! Essentially, the error exists because Phold (fold seek) found 0 hits. My suspicion is that this is due to phanotate providing crappy gene calls. There were 179 CDS in your GenBank, whereas prodigal found only 90 in the paper with prokka (https://journals.asm.org/doi/10.1128/spectrum.03719-23). I will look into this separately for other reasons, because that is bizarre!
In the dev branch, I have added a line of code to warn and exit the users if Foldseek finds 0 hits - an unlikely occurrence but nonetheless possible.
George

Thanks for looking into this @gbouras13 ! A weird phage indeed...

Hi @acvill , I'm closing this issue as there does not seem to be a phold issue - perhaps try prodigal-gv with this phage
George

I am now meeting with the same issue and tried the prodigal-gv of pharokka to produce the .gbk file. but still being remindered with this "...ValueError("Columns must be same length as key")
ValueError: Columns must be same length as key..". Any other good ideas and suggestions?
thank you very much!

@Tsingsjeen  - do you mind sharing the gbk or raw FASTA? We never figured the issue above :)
George

thanks!
this is Ecoli T2 phage reannotated by Pharokka. Its .gbk and .faa files were attached.
T2_pharokka.gbk.txt
T2_prodigal-gv.faa.txt


If Type Array is empty then it is failing.

For Example if my input JSON is below :
{
"metadata": {
"properties": []
}
}
Then schema is not generating properly . we are getting below exception
"main" java.lang.NullPointerException.
Because in code we have taken directly
node = jsonNode.get(key).get(0);
Do you have any solution for this ?

hi please try adding value to 'properties' field.  let me know if it is failing thanks

Thanks for your reply. For non-Empty array it is running properly . But We have huge JSON with lot of empty arrays.  We are planning generated schema for empty arrays also.


Cannot render partials

Not sure if this is a slime issue. Trying to render a partial, following the book "programming phoenix":
= render "_user.html", user: user

Gives me an error:
Could not render '_user.html' for Rumbl.UserView, please define a matching clause for render/2 or define a template at "web/templates/user". The following templates were compiled:

* _user.html
* index.html
* show.html

Funnily it says the template is _user.html is compiled, so what's going on?

Got it working. SUper strange issue, my sublime editor seem to have added a hidden character somewhere. Retyping the exact code solved it:
= render "_user.html", user: user


small small widget changes

addressing https://github.com/CDCgov/cfa-ngm-widget/issues/57

@afmagee42 I'm merging on @paigemiller 's request; if you have further edits, we can do them in a future PR


Question: How to disable parallelism for a single feature file?

Hey, I have a case when the scenarios in one of the feature files depend on the same external resource that I can't stub or mock. In a plain Playwright we can disable parallel mode for a single spec file https://playwright.dev/docs/test-parallel#parallelize-tests-in-a-single-file , but couldn't figure out whether it's possible with playwright-bdd as well. Is it?

False alarm, found it: https://vitalets.github.io/playwright-bdd/#/writing-features/special-tags?id=modexxx
Many thanks for your work, great job!


union operation broken in 9.0.0

I was updating our rails app to 5.2.0 and hence arel from 8.0.0 to 9.0.0
The union operation seems to be broken( even though its only deprecated in 9.0.0 ).
Here is an example of what the issue is:
left and right are two ActiveRecord query objects. Individually they work, but left.union(right) is breaking now in 9.0.0.
left.to_sql
#=> "SELECT *, 1 as priority FROM `notifications` WHERE `notifications`.`receiver_id` = <masked> AND `notifications`.`read_at` IS NULL AND `notifications`.`target_type` = 'photo_type' AND `notifications`.`created_at` > '2018-04-09 10:11:12.732444'"

right.to_sql
#=> "SELECT *, 2 as priority FROM `notifications` WHERE `notifications`.`receiver_id` = <masked> AND `notifications`.`target_type` IN ('type1', 'type2', 'type3', 'type4') AND 1=1"

left.union(right).to_sql
#DEPRECATION WARNING: Delegating union to arel is deprecated and will be removed in Rails 6.0. (called from build_union_query at (pry):20)
#DEPRECATION WARNING: Delegating ast to arel is deprecated and will be removed in Rails 6.0. (called from build_union_query at (pry):20)

#=> "( SELECT *, 1 as priority FROM `notifications` WHERE `notifications`.`receiver_id` = <masked> AND `notifications`.`read_at` IS NULL AND `notifications`.`target_type` = 'photo_type' AND `notifications`.`created_at` > '2018-04-09 10:11:12.732444' UNION SELECT *, 2 as priority FROM `notifications` WHERE `notifications`.`receiver_id` = <masked> AND `notifications`.`target_type` IN (?, ?, ?, ?) AND 1=1 )"

As can be seen the generated SQL is invalid at this point:
 `notifications`.`target_type` IN (?, ?, ?, ?)

Per #523, Arel development is moving to rails/rails.
If this issue is still relevant, please consider reopening it over there. (Note that the Rails repository does not accept feature request issues, and requires reproduction steps that rely on Active Record's documented API.)


Duplicated events for Workflow Node Event

Checklist:

[x] I've included the version.
[x] I've included reproduction steps.
[x] I've included the workflow YAML.
[x] I've included the logs.

What happened:
Duplicated Workflow Node Events are emitted for task/step while running workflow.
The event duplication happens on both Steps and DAG.
https://github.com/argoproj/argo/blob/master/docs/workflow-events.md
What you expected to happen:
Workflow Node Events should be emitted only once for each task/step completion.
How to reproduce it (as minimally and precisely as possible):
Can be reproduced by running examples with multiple tasks/steps.
https://github.com/argoproj/argo/blob/master/examples/dag-diamond.yaml
https://github.com/argoproj/argo/blob/master/examples/steps.yaml
Anything else we need to know?:

Last task/step in DAG/Steps has no event duplication.
The difference between duplicated events seems to be only resourceVersion of the involvedObject (=Workflow).
Argo CLI hasn't been used. (Only kubectl command)

Environment:

Argo version: 2.8.0-rc3

$ argo version

Kubernetes version :  v1.17.4-gke.10

$ kubectl version -o yaml

Other debugging information (if applicable):
None.
Logs
Following logs are event logs from kubectl describe wf.
DAG with Task A only: No event duplication
Events:
  Type    Reason                 Age        From               Message
  ----    ------                 ----       ----               -------
  Normal  WorkflowRunning        <invalid>  dag-diamond-dzkhc  Workflow Running
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-dzkhc  Succeeded node dag-diamond-dzkhc.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-dzkhc  Succeeded node dag-diamond-dzkhc
  Normal  WorkflowSucceeded      <invalid>  dag-diamond-dzkhc  Workflow completed

DAG with Task A,B: Event duplication for Task A.
Events:
  Type    Reason                 Age        From               Message
  ----    ------                 ----       ----               -------
  Normal  WorkflowRunning        <invalid>  dag-diamond-jgxfq  Workflow Running
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jgxfq  Succeeded node dag-diamond-jgxfq.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jgxfq  Succeeded node dag-diamond-jgxfq.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jgxfq  Succeeded node dag-diamond-jgxfq.B
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jgxfq  Succeeded node dag-diamond-jgxfq
  Normal  WorkflowSucceeded      <invalid>  dag-diamond-jgxfq  Workflow completed

DAG with Task A,B,C,D: Event duplication for Task A,B,C
Events:
  Type    Reason                 Age        From               Message
  ----    ------                 ----       ----               -------
  Normal  WorkflowRunning        <invalid>  dag-diamond-jzt9b  Workflow Running
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.B
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.C
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.C
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b.D
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-jzt9b  Succeeded node dag-diamond-jzt9b
  Normal  WorkflowSucceeded      <invalid>  dag-diamond-jzt9b  Workflow completed

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Can you please try with argoproj/workflow-controller:fix-2961?

Thanks for rapid response and fix!
I have checked that the fixed controller has no event duplication in both DAG and Steps examples.
DAG:
Events:
  Type    Reason                 Age        From               Message
  ----    ------                 ----       ----               -------
  Normal  WorkflowRunning        <invalid>  dag-diamond-xm8cg  Workflow Running
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-xm8cg  Succeeded node dag-diamond-xm8cg.A
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-xm8cg  Succeeded node dag-diamond-xm8cg.B
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-xm8cg  Succeeded node dag-diamond-xm8cg.C
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-xm8cg  Succeeded node dag-diamond-xm8cg.D
  Normal  WorkflowNodeSucceeded  <invalid>  dag-diamond-xm8cg  Succeeded node dag-diamond-xm8cg
  Normal  WorkflowSucceeded      <invalid>  dag-diamond-xm8cg  Workflow completed

Steps:
Events:
  Type    Reason                 Age        From         Message
  ----    ------                 ----       ----         -------
  Normal  WorkflowRunning        <invalid>  steps-vtfgb  Workflow Running
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb[0].hello1
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb[0]
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb[1].hello2b
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb[1].hello2a
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb[1]
  Normal  WorkflowNodeSucceeded  <invalid>  steps-vtfgb  Succeeded node steps-vtfgb
  Normal  WorkflowSucceeded      <invalid>  steps-vtfgb  Workflow completed

brilliant - thank you


Application Form - Organization Address

User story
As an applicant
I want to report the location and mailing address for my organization
so that I can complete the form
Description
Wireframe
https://www.figma.com/file/nhOquOVmWD1P3BxEY0KIh5/CCBC-Intake?node-id=1059%3A6329

[x] Add the title for different address

@MarsRomer this is ready for review. Please log into the app then go to https://ccbc-test.apps.silver.devops.gov.bc.ca/form/2 and select 'No' from the bottom radio button to see the additional mailing address be loaded on the bottom of the screen.


Potential problems with this image : No problem

When uploading a picture, I got this screen:

Usually a reason (examples: picture too dark, picture downloaded from Facebook, etc) is displayed, but the problem here is that zero reason is listed.
I pressed "Yes" and the picture got uploaded successfully to https://commons.wikimedia.org/wiki/File:Omotesando_station_exit_2_at_night.jpg

For reference, here is an example of how reasons should be listed:

Hey, I am working on this Issue

Hey @nicolas-raoul I think that you have uploaded an image that was downloaded from internet. And the error dialog poped up due to the merge of #2581. However I think slight change was needed in ImageUtils.java. I have made the changes here. Kindly review :)


SUBMARINE-525. [doc] Model training example on k8s

What is this PR for?
Update doc of model training example on k8s
What type of PR is it?
[Documentation]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-525
How should this be tested?
https://travis-ci.org/github/pingsutw/hadoop-submarine/builds/697721465
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Need some modify after https://github.com/apache/submarine/pull/305 merge.
So change this PR to Draft temporarily.


Added text to the Arc DC deploy tile that user must have done az login before continuing.

Quick fix for the release: add text to the deployment tile as shown below.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.371%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27568

Relevant Lines:
60822

💛  - Coveralls

Are people going to notice this? I wonder if it'd be better to have a new intro page that gives info about Azure Arc including this.
Or what about checking if the user is logged in when fetching the list of items - right now you said it's just returning an empty list right? Have you tried having it throw an error to see if an actual warning popup appears? I think that should be the behavior for the dynamic option stuff...

So right now it doesn't just show an empty list, but rather throws an error in the console and does not load Page 5 of the wizard (if user is not logged in). I put the text here because if I put it in the wizard, by the time they read it it would be too late. I could add it to the wizard anyways in case they miss the text on the tile, if that helps.
I could also take some more time and see if I can prevent the error from causing the page to not load, but this was option 2 in my email and it seemed option 1 of just adding text got more support. What do you think? We can also discuss this offline


Add some properties to NotionDBLoader

fix #13356
Add supports following properties for metadata to NotionDBLoader.

checkbox
email
number
select

There are no relevant tests for this code to be updated.

Link to relevant docs https://developers.notion.com/reference/page-property-values

Thanks Erick!


[CLOSED] Remove depdency on sun.misc.BASE64Encoder

 Issue by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT
Originally opened as https://github.com/twogee/missing-link/issues/7

Remove depdency on sun.misc.BASE64Encoder, without introducing a third party
library dependency.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 15 Mar 2011 at 12:09

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT

Added public domain Base64 encoder obtained from
http://iharder.sourceforge.net/current/java/base64/

Original comment by<EMAIL_ADDRESS>on 15 Mar 2011 at 12:17

Changed state: Verified

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 2:27

Added labels: Project-ant-http

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 15:04 GMT

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 2:28

Added labels: Milestone-1.0


Create Questions.yaml

GitHub is complaining about a YAML parse error

Accidentally committed on main branch rather than forked branch. Sorry.


Use webpack asset modules

Replace deprecated file-loader with Webpack asset modules, as described in https://webpack.js.org/guides/asset-modules/.
Fixes #2914 and fixes #3249 for my setup.

Because people are fairly reliant on file-loader we'll want a way to opt-in to asset modules. I suggest adding a config option for this (so you can enable it via mix.options). You'll be able to pull from the config there using mix.config.
Maybe something like mix.options({ assetModules: true }) though I'm very open to other suggestions for naming.

I do want to require them in the next major version. It should've been caught when we upgraded Mix to v6 but sadly I missed it. 🙃

Thanks for your feedback! I've added a config option now that toggles the new behavior. Also I adjusted tests to use this new toggle by default and copied the two main tests which test these rules to test both old and new version. Finally force-pushed to remove the commit that deleted the file-loader dependency, as it is still used.
Let me know if you have any more suggestions and/or improvements!

Used this as an opportunity to do a tiny refactor of the rules code. I've always felt it was far too messy. I think it's better but it doesn't feel 100% where I want it to be. But better is better in any case. Thanks again for doing this! ❤
I think there's a few other things I'll want to take care of before tagging a patch release but I'm aiming to cut a release on either Friday or Saturday.

Used this as an opportunity to do a tiny refactor of the rules code. I've always felt it was far too messy. I think it's better but it doesn't feel 100% where I want it to be. But better is better in any case. Thanks again for doing this! ❤
I think there's a few other things I'll want to take care of before tagging a patch release but I'm aiming to cut a release on either Friday or Saturday.

Any update on the next patch release? 😇

Heh yeah sorry! Life got in the way. Soon™ — I want to see if I can fix the [object Module] in Vue 2 with these first.

Heh yeah sorry! Life got in the way. Soon™ — I want to see if I can fix the [object Module] in Vue 2 with these first.

Great! Would be nice to have before June 1st, when Node 17 reaches EOL, so I can upgrade my builds to Node 18 by then.

Ah yesss it does fix it! 💯
Just more evidence that I really shouldn't have missed this the first time around when I upgraded Mix to webpack 5. lol


MX server wuldn't be domain name

Hi, i have problem with loading MX records, where server si not domain name, but just "mail".
For example:
                        MX      10 mail
mail                    A       some_ip

It's valid record, but you don't load it because you expect full domain name like mail.example.com.

Can you post the full zone?

$TTL 2m
@               1d      SOA     ns.eurosignal.cz. hostmaster.eurosignal.cz.<PHONE_NUMBER> 1d 1h 4w 10m
                        NS      ns1.eurosignal.cz.
                        NS      ns2.eurosignal.cz.
                        NS      ns3.eurosignal.cz.

                        MX      10 mail

mail                    A       <IP_ADDRESS>
                        AAAA    2a00:1aa8:1::38
www                     A       <IP_ADDRESS>
                        AAAA    2a00:1aa8:1::37

You will need to define a $ORIGIN directive at the top of your zone file.
Now, bind will handle zone without an $ORIGIN directive, the origin is inferred by the zone "<zone-name>" {} clause.
I suggest your zones have this directive to be more portable.
However! I can see this being a pain in the arse, we need a way to load a zone and pass it a origin.
How about this:
zone = DNS::Zone.load(zone_as_string, 'example.origin.com')

Where the second param is optional, and any ORIGIN defined within the zone will ultimately win?
It is possible for a zone to use multiple ORIGIN directives, the lib does not handle things like that, yet, pull request anyone? :)

I'm not the OP, but adding a param that allows you to specify a default ORIGIN that would be overwritten by an ORIGIN within the zone would be useful to me.

You can add the second paramets for specify origin, but it doesn't solved my problem. Because when i added $ORIGIN to zone file, i still dind't get the MX record.
$TTL 2m
$ORIGIN kalle.cz
@		1d	SOA	ns.eurosignal.cz. hostmaster.eurosignal.cz.<PHONE_NUMBER> 1d 1h 4w 10m
			NS	a.ns.eurosignal.cz.
			NS	b.ns.eurosignal.cz.
			NS	c.ns.eurosignal.cz.
			MX	10 mail

mail			A	<IP_ADDRESS>
			AAAA	2a00:1aa8:1::38

www			A	<IP_ADDRESS>

pp zone
#<DNS::Zone:0x007ff53b27b2c0
 @origin="kalle.cz",
 @records=
  [#<DNS::Zone::RR::SOA:0x007ff53b27ac08
    @email="hostmaster.eurosignal.cz.",
    @expiry_ttl="4w",
    @klass="IN",
    @label="@",
    @minimum_ttl="10m",
    @nameserver="ns.eurosignal.cz.",
    @refresh_ttl="1d",
    @retry_ttl="1h",
    @serial=2012102410,
    @ttl="1d">,
   #<DNS::Zone::RR::NS:0x007ff53b385a08
    @klass="IN",
    @label="\t",
    @nameserver="a.ns.eurosignal.cz.",
    @ttl=nil>,
   #<DNS::Zone::RR::NS:0x007ff53b385580
    @klass="IN",
    @label="\t",
    @nameserver="b.ns.eurosignal.cz.",
    @ttl=nil>,
   #<DNS::Zone::RR::NS:0x007ff53b385288
    @klass="IN",
    @label="\t",
    @nameserver="c.ns.eurosignal.cz.",
    @ttl=nil>,
   #<DNS::Zone::RR::A:0x007ff53b384900
    @address="<IP_ADDRESS>",
    @klass="IN",
    @label="mail",
    @ttl=nil>,
   #<DNS::Zone::RR::AAAA:0x007ff53b3841d0
    @address="2a00:1aa8:1::38",
    @klass="IN",
    @label="\t",
    @ttl=nil>,
   #<DNS::Zone::RR::A:0x007ff53b383d20
    @address="<IP_ADDRESS>",
    @klass="IN",
    @label="www",
    @ttl=nil>,
   #<DNS::Zone::RR::A:0x007ff53b383a00
    @address="<IP_ADDRESS>",
    @klass="IN",
    @label="brana",
    @ttl=nil>],
 @ttl="2m">

@mrideout I've opened another issue to address that: #7
@arius Couple of things:

You should add a . (dot) at the end of your ORIGIN domain-name (e.g. $ORIGIN kalle.cz.)
You are correct, even with a ORIGIN the code still doesn't work correctly, your use of a unqualified domain-name is valid.

@arius Can you give it a try using the master branch?

OK it does work for me even without origin directive. Thank you


[e2e]: generic schedule main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11098686928
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Mon Sep 30 03:22:57 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11117942582
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Oct  1 03:18:51 UTC 2024
Tests are passing now. Closing this issue.


23 select filter enhancement

This updates the filter in the digital twin select.
New filter options:

only surfaces shared by you
only surfaces published by you

Did i miss something in the "git etiquette"?
And should we discuss the content of this merge request here or in the corresponding   Issue #750?

Looks good. I would discuss the content here.

There are some tests that fail, please look at them and try to fix.

Ich würde noch gerne einen test hinzufügen

I notices something:
publishing creates a copy of a surface right?
Is it possible that this copy is shared?
Because i thing this would not be possible to model with the current schema.
Since sharing only sets a permission, a permission published surfaces have as well.

I got no clue why "skip_postgeneration_save" broke. Any ideas?

I noticed something: publishing creates a copy of a surface right? Is it possible that this copy is shared? Because i thing this would not be possible to model with the current schema. Since sharing only sets a permission, a permission published surfaces have as well.

Publishing creates a copy, but with frozen in permissions. The whole world has view permission, nobody has any of the other permissions. Resharing a published dataset does not make sense as it is visible to everyone anyway.

These trackstat errors can be fixed by adding the handle_usage_statistics fixture to the test. (Just add handle_usage_statistics to the list of arguments of the test. That argument is never used in the test itself.)


Generate random errors in cartservice

Similar to #694 but demonstrates errors from the cart service (.NET). I'm unsure if I've done things correctly for adding the new feature flag cartServiceFailure. I will submit a corresponding docs PR soon.

I'm all for adding more scenarios, but I need help understanding how this adds value beyond the other 2 existing error scenarios we already have. This will also add 2 more spans for the feature flag service on 10% (random sample) of all traces for EmptyCart requests.
I don't have any issues with the code and will approve it. I just want to ensure we aren't adding new scenarios "just because."

The motivation is to demonstrate an error recorded using OpenTelemetry .NET. Go and Java are covered with the existing scenarios. When speaking with end users, I have often found it useful to hone in on a particular service in the language they're most familiar with to speak to the features the OpenTelemetry SDKs provide.

@puckpuck thoughts?

Hello @alanwest,
We have started some discussion regarding this at one of the SIG meetings, but we didn't go through yet, because of the 1.4 release for KubeCon.
As KubeCon is this week, I don't think we will have time to discuss this through, but please bare with us for a little bit longer.

@cartersocha Fixed conflicts. After this is merged should I submit a PR to update the docs here https://opentelemetry.io/docs/demo/feature-flags?

Yes please do @alanwest. Thanks for your patience and following up


support checking the GCM tag value and measuring the impact of CipherInit/Final in openssl speed

When running openssl speed -evp id-aes256-GCM on version 3.0.7, I noticed that you do the following for each data size:

EVP_CipherInit_ex once
EVP_EncryptUpdate a bunch of times
EVP_EncryptFinal_ex once

And when passing in -decrypt, you do similar steps. But you never check the GCM tag value using something like this:
    unsigned char gcmTag[16];
    OSSL_PARAM params[2];

    get the gcmTag from the input;
    params[0] = OSSL_PARAM_construct_octet_string(
            OSSL_CIPHER_PARAM_AEAD_TAG,
            gcmTag,
            16);
    params[1] = OSSL_PARAM_construct_end();
    EVP_CIPHER_CTX_set_params(ctx, params);
    EVP_DecryptUpdate();
    check the return value of EVP_DecryptFinal_ex();

The performance results need to take EVP_CIPHER_CTX_set_params (and EVP_CIPHER_CTX_get_params) into account.
Also, the performance results should take EVP_CipherInit_ex and EVP_Encrypt/DecryptFinal_ex into account. A typical application may call EncryptInit/update/final/get_params on the sender side and then DecryptInit/set_params/update/final on the receiver side, so the performance test should execute those sequences of steps many times, not just the update many times.

Arena couldb't ve decrypted. Fun Thing Is to secure nothing.Can't stop l a tak.....Zasláno z Outlooku pro Android

Isn't that what the -aead parameter of openssl speed is for?  Without it, you simply test throughput of the encryption/decryption of that cipher.  If it add it (obviously only allowed for AEAD ciphers), you will test the whole AEAD cycle.  So please try openssl speed -evp id-aes256-GCM -aead and check if that solves your problem.

Thank you for pointing that out. But -aead only results in EVP_CIPHER_CTX_ctrl(EVP_CTRL_AEAD_SET_TAG) getting called on the decryption side. On the encryption side, you're still not calling EVP_CIPHER_CTX_ctrl(EVP_CTRL_AEAD_GET_TAG) to get the tag that needs to be sent to the receiver.


Tensor shape error when loading quant Llama-2-70B

Hello! Thanks for the nice work!
I want to quantize Llama-2-70B. I was able to export the quantized model without any error. However, when I test the model:
python test_model.py \
  --model-path=$quant_model_path \
  --tokenizer-path=$model_path \
  --model-class=llama --prompt="something to say"

I encounter this error:
/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/utils/generic.py:441: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  _torch_pytree._register_pytree_node(
/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/utils/generic.py:309: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  _torch_pytree._register_pytree_node(
/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/utils/generic.py:309: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  _torch_pytree._register_pytree_node(
Loading checkpoint shards:   0%|                                                                                | 0/15 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "/home/mmilin/projects/ming_benchmark_vllm/AutoSmoothQuant/autosmoothquant/examples/test_model.py", line 62, in <module>
    main()
  File "/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/mmilin/projects/ming_benchmark_vllm/AutoSmoothQuant/autosmoothquant/examples/test_model.py", line 40, in main
    model = Int8LlamaForCausalLM.from_pretrained(args.model_path, quant_config, device_map="sequential")
  File "/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/modeling_utils.py", line 3706, in from_pretrained
    ) = cls._load_pretrained_model(
  File "/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/modeling_utils.py", line 4116, in _load_pretrained_model
    new_error_msgs, offload_index, state_dict_index = _load_state_dict_into_meta_model(
  File "/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/transformers/modeling_utils.py", line 778, in _load_state_dict_into_meta_model
    set_module_tensor_to_device(model, param_name, param_device, **set_module_kwargs)
  File "/home/mmilin/projects/ming_benchmark_vllm/autosmq-venv/lib/python3.9/site-packages/accelerate/utils/modeling.py", line 345, in set_module_tensor_to_device
    raise ValueError(
ValueError: Trying to set a tensor of shape torch.Size([1024, 8192]) in "weight" (which has shape torch.Size([8192, 8192])), this look incorrect.

BTW, I was able to convert and load Llama-2-7b model without any error. Any idea how to fix it? Looks like group attention related.
Many thanks ahead!

Hi! We haven't encountered this problem before. Could you please post your config.json for both models?

Hi! We haven't encountered this problem before. Could you please post your config.json for both models? And which linear layer does the shape mismatch exist?

Sorry for the late reply. My config.json:

@MingLin-home Hi, have you tackled this issue? I had the same problem.


Fix issue #710: fix suggest delete binding

By generalizing code action: suggestions of delete unused top binding to suggestions of delete unused binding.
This is basically a reimplementation which traverses the AST.

 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.

Merged, thank you Ray!


Blueprint export includes expressions.

I exported the data from a pseudocolor plot using Blueprint and I also got variables from any expressions defined in the file.
To reproduce:

Open rect2d.silo.
Create a pseudocolor plot of "d".
Export to Blueprint using all the defaults.
Open the exported file and you will see the scalars "d", "speed", "vel_magnitude" and the vector "vel".

"speed", "vel_magnitude", and "vel" all are either expressions from the database or auto-generated. It seems strange to include expressions, especially when they are all actually invalid in the exported file.
This happens with 3.2.1.

Similar to #17324

Expressions shouldn't be saved at all unless listed and then it should be the actual variable, not an expression.


Indicate who's talking

Use colors or icons to indicate when someone is speaking

I added this feature in the photon branch - awaiting review

Fixed


Add support for Helidon 1.2 (MP 2.2)

The recent release of Helidon 1.2 supports MP 2.2. (See https://helidon.io/docs/latest/#/microprofile/01_introduction.)
It would be great if the starter could be updated to reflect this.

PR is merged.


feat: Promote imgtfjs projects out of beta

Now that Visual Recognition service has been deprecated, I need
to start down a path of removing it from the site. This commit
lays the groundwork for me to be able to remove 'images' projects
from any new classes, while still leaving it for classes who
already have existing VR API keys.
Contributes to: #374
Signed-off-by: Dale Lane<EMAIL_ADDRESS>

travis run https://travis-ci.org/github/IBM/taxinomitis/builds/750696649


Deprecate/remove getData method

The getData method is never safe to use over the cloneData method, since you should never really change the data of a matrix.
Therefore I propose to remove the method and change all uses to cloneData.

Might as well, if it's never used. Only think is, that cloning takes a little longer than just returning a pointer? Perhaps have some way of caching the value(s) if we are returning a lot.

But it'll need to return a new double[][] object each time anyway. Not
quite sure how to achieve that and still cache.
On Wed, 11 Feb 2015 10:30 projectgoav<EMAIL_ADDRESS>wrote:

Might as well, if it's never used. Only think is, that cloning takes a
little longer than just returning a pointer? Perhaps have some way of
caching the value(s) if we are returning a lot.
—
Reply to this email directly or view it on GitHub
https://github.com/Sheepzez/Matrixonator-Java/issues/31#issuecomment-73860838
.


fix(rbac): fix labels and dropdowns in dark theme by aligning/downgrading components to MUI v4

Fixes: RHIDP-1249
The RBAC plugin uses Material UI v4 and v5 components together. To solve some issues with the dark theme I aligned/downgraded the code to use v4.
Before:
https://github.com/janus-idp/backstage-plugins/assets/139310/c9860b3a-8661-4879-b108-1dc4e0b105cc
With this PR:
https://github.com/janus-idp/backstage-plugins/assets/139310/738e5882-7339-4e1f-87bd-2dee32d36eb0

/assign
/cc @debsmita1
/kind bug


Library not included in  kimageformat-plugins 5.79?

The release page states that as of kimageformats 5.78 the avif library is included. I found the KDE merge, but as you can see below support is not included as of 5.79. Am I missing something?
➜  ~ apt info kimageformat-plugins
Package: kimageformat-plugins
Version: 5.79.0-0xneon+20.04+focal+build12
Priority: optional
Section: kde
Source: kimageformats
Maintainer: Neon CI<EMAIL_ADDRESS>Installed-Size: 527 kB
Depends: libc6 (>= 2.27), libgcc-s1 (>= 3.0), libilmbase24 (>= 2.3.0), libkf5archive5, libopenexr24 (>= 2.3.0), libqt5core5a (>= 5.15.2+dfsg), libqt5gui5 (>= 5.15.2+dfsg), libqt5printsupport5 (>= 5.15.2+dfsg), libstdc++6 (>= 5)
Homepage: https://projects.kde.org/projects/frameworks/kimageformats
Download-Size: 114 kB
APT-Manual-Installed: yes
APT-Sources: http://archive.neon.kde.org/user focal/main amd64 Packages
Description: additional image format plugins for QtGui

N: There are 4 additional records. Please use the '-a' switch to see them.

Maybe I need to build from this repo?
➜  ~ dpkg-query -L kimageformat-plugins
/.
/usr
/usr/lib
/usr/lib/x86_64-linux-gnu
/usr/lib/x86_64-linux-gnu/qt5
/usr/lib/x86_64-linux-gnu/qt5/plugins
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_ani.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_eps.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_exr.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_hdr.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_kra.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_ora.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_pcx.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_pic.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_psd.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_ras.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_rgb.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_tga.so
/usr/lib/x86_64-linux-gnu/qt5/plugins/imageformats/kimg_xcf.so
/usr/share
/usr/share/doc
/usr/share/doc/kimageformat-plugins
/usr/share/doc/kimageformat-plugins/README.md
/usr/share/doc/kimageformat-plugins/changelog.Debian.gz
/usr/share/doc/kimageformat-plugins/copyright
/usr/share/kservices5
/usr/share/kservices5/qimageioplugins
/usr/share/kservices5/qimageioplugins/ani.desktop
/usr/share/kservices5/qimageioplugins/dds.desktop
/usr/share/kservices5/qimageioplugins/eps.desktop
/usr/share/kservices5/qimageioplugins/exr.desktop
/usr/share/kservices5/qimageioplugins/hdr.desktop
/usr/share/kservices5/qimageioplugins/jp2.desktop
/usr/share/kservices5/qimageioplugins/kra.desktop
/usr/share/kservices5/qimageioplugins/ora.desktop
/usr/share/kservices5/qimageioplugins/pcx.desktop
/usr/share/kservices5/qimageioplugins/pic.desktop
/usr/share/kservices5/qimageioplugins/psd.desktop
/usr/share/kservices5/qimageioplugins/ras.desktop
/usr/share/kservices5/qimageioplugins/rgb.desktop
/usr/share/kservices5/qimageioplugins/tga.desktop
/usr/share/kservices5/qimageioplugins/xcf.desktop

The source code is included in kimageformats but the package you installed obviously do not contain kimg_avif.so
The maintainer of the package did not add libavif dependency so AVIF module was not built probably. I suggest to ask the maintainer why. Maintainers have freedom to decide what format to include or exclude.
You can of course build libqavif.so from this repo. You can delete libqavif.so later when you will have kimg_avif.so installed.

Thank you! I will build from your repo and link the maintainer to this issue. I appreciate the assistance and will close out this issue.

Perhaps the package is for old version of Ubuntu where they don't have libavif yet.
The future version of Ubuntu will have libavif, so it will allow maintainer to build the AVIF plugin conveniently.
https://packages.ubuntu.com/search?keywords=libavif-dev


Changing interpolation characters in a controller.

Is it possible to change the interpolation characters in a function (inside a controller),  or is that possible only in the .config() method?
SO question about this
This is Angular 1.3.15

This is only possible in the config phase, since it is global for the whole app. What's your use case for changing this in the controller?

@Narretz I have a directive that accepts a string as a template. That string contains some variables names that don't exist in the scope which is managing the directive, but in the objects that the directive is handling*. Each variable name is delimited by {| and |}.
Real world example: A dropdown-like widget filled with items. Each item's text is a string in the format {| var1 |} {| var2 |} .... The directive receives an array of objects, and each object contains var1, var2, ....
When the directive renders each item in the dropdown it needs to parse the format string ({| var1 |} {| var2 |} ...) and interpolate it with the content of the currently rendered item.
If I could change the interpolation characters at runtime (outside of the .config() method) that wouldn't require doing this

@alexandernst, fwiw, what you are doing now is much more concise and clear from temporarily replacing the interpolation symbols:
_fmt = _fmt.replace(/\{\|/g, "{{").replace(/\|\}/g, "}}");

vs
var _s = $interpolate.startSymbol();
var _e = $interpolate.endSymbol();
$interpolate.startSymbol("{|");
$interpolate.endSymbol("|}");
...
$interpolate.startSymbol(_s);
$interpolate.endSymbol(_e);

I don't see much merit in being able to replace the symbols at runtime (not to mention the risks of failing to properly restore them).

It makes less operations to do the trick, but it's unclear what I'm doing there. There could be more real-world examples for what I'm suggesting.

Changing the interpolation symbols after config has side-effects that I think everybody would like to avoid. The only option I think makes sense (even when I do not really like the idea) is to add an optional parameter to $interpolate to specify the begin/end tokens just for this interpolation. The main issue would be that this parameter would be its 5th...

Hmm, why not add the locals argument to a function generated by $interpolate (and then use-case described here could be handled using a standard interpolation delimiters - if I understand the use-case correctly....)

Javascript is single-threaded, I can't understand how would changing the interpolation symbols affect annything else (if restored correctly, after doing what one needs to do). Care to explain a little bit?

Would it be possible to understand why not just use the standard {{ and }} symbols?
The only change would be to change this line to
// Warning untested code
var _fmt = element.attr(attrs.$attr.itemLabel);

@lgalfaso That won't work as the itemLabel is set inside the directive, like this:

, if I do it with the normal angular brackets, angular will try to interpolate that.

@alexandernst var _fmt = element.attr(attrs.$attr.itemLabel); should be able to retrieve the uninterpolated attribute, at least within link. I did not dig too dip into this directive implementation so I do not know when is this function called, but that aside, this mechanism is provided exactly for this.

@lgalfaso I just tried, but that is returning empty strings. Does that requires Angular 1.4?

Angular 1.4 is not needed. Now, this is somehow off topic as this is not really an issue with Angular.. If you would like to keep this conversation, please put a plunker together and we can continue this on StackOverflow.
Trying to go back to the original issue that caused this issue, I think there are other means to achieve this so I lean towards "wont fix".

Angular provides a way to escape interpolation symbols:

...

I can't test it right now, but you should be able to make it work without needing to swap the intetpolation symbols or replace them in _fmt.

@lgalfaso Fair enough. Please check http://stackoverflow.com/questions/30725108/passing-format-string-to-a-directive-and-interpolating-it (includes a demo showing the problem).
@gkalpak That doesn't work. Please check my SO question and the demo from plunkr.

Answered at SO

@alexandernst, I had misunderstood the usecase. I added an alternative approach on SO (there are other ways of doing it, based on the exact requirements, but swapping interplation symbols shouldn't be one of them).

@gkalpak I saw your answer, but the question here isn't about evaluation math expressions, but about rendering items. I'll update my SO question, please check it again. /cc @lgalfaso

I see that you already changed it with https://github.com/alexandernst/angular-multi-select/commit/dd306338b68dcf7f9f17cba5281af7bc8c15e4eb
BTW, you can reuse the result of $interpolate(...) as that construction is not super cheap

@lgalfaso You mean save the result of $interpolate(...) and just to _savedResult(obj) in my _createLabel function?

yep

@lgalfaso Done :) Thanks for the tip!


Create package for Ubuntu 22.04

Create a package for Ubuntu 22.04 (Jammy Jellyfish) and publish it over launchpad

A Ubuntu 22.04 package is released with V1.17.2


[Question] Use of --delete-after with move command and exclusion?

General question:
I am currently using move with an excluded folder (shown below) since move functions like a copy+delete when there is an excluded folder, can I use the --delete-after flag so that files are not deleted off the source until the complete copy of all files is finished?
rclone --min-age 30d --checksum --bwlimit 4M --exclude excludefolder/** move /home/user/localfiles/ remoteacd:server/remotefolder

move always deletes files after the copy.
If there are any IO errors during the copy then move won't delete the files.
I'll make this a bit clearer in the docs.

When moving a folder, is it possible to delete every file right after the file is copied instead of deleting the folder only after all files have been copied?

@makdisse Not at the moment, no. Do you want to make a new issue?

I've re-worked the move docs and hopefully made them clearer - I'm going to close this now


Stencils no longer work

When trying to use canny, openpose or scribble I get this error:
Traceback (most recent call last):
File "gradio\routes.py", line 442, in run_predict
File "gradio\blocks.py", line 1392, in process_api
File "gradio\blocks.py", line 1111, in call_function
File "gradio\utils.py", line 346, in async_iteration
File "gradio\utils.py", line 339, in anext
File "anyio\to_thread.py", line 33, in run_sync
File "anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 807, in run
File "gradio\utils.py", line 322, in run_sync_iterator_async
File "gradio\utils.py", line 691, in gen_wrapper
File "ui\img2img_ui.py", line 241, in img2img_inf
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_stencil.py", line 267, in generate_images
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_stencil.py", line 138, in produce_stencil_latents
File "shark\shark_inference.py", line 142, in call
return self.shark_runner.run(function_name, inputs, send_to_host)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "shark\shark_runner.py", line 101, in run
return get_results(
^^^^^^^^^^^^
File "shark\iree_utils\compile_utils.py", line 504, in get_results
result = compiled_vmfunction_name
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "iree\runtime\function.py", line 137, in call
File "iree\runtime\function.py", line 162, in _invoke
ValueError: Error invoking function: D:\a\SHARK-Runtime\SHARK-Runtime\c\runtime\src\iree\modules\hal\utils\buffer_diagnostics.c:225: INVALID_ARGUMENT; input 2 shape dimension 1 mismatch; expected 64 but have 512; expected shape 2x64x768, actual shape 2x512x768; while invoking native function hal.buffer_view.assert; while calling import;
[ 1]   native hal.buffer_view.assert:0 -
[ 0] bytecode module@0:9168 -

I was finally able to test it with shark 901, it still does not work. There is a different error now. Here it is:
Traceback (most recent call last):
File "gradio\routes.py", line 488, in run_predict
File "gradio\blocks.py", line 1431, in process_api
File "gradio\blocks.py", line 1123, in call_function
File "gradio\utils.py", line 349, in async_iteration
File "gradio\utils.py", line 342, in anext
File "anyio\to_thread.py", line 33, in run_sync
File "anyio_backends_asyncio.py", line 2101, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 828, in run
File "gradio\utils.py", line 325, in run_sync_iterator_async
File "gradio\utils.py", line 694, in gen_wrapper
File "ui\img2img_ui.py", line 241, in img2img_inf
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_stencil.py", line 295, in generate_images
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_stencil.py", line 152, in produce_stencil_latents
AttributeError: 'Tensor' object has no attribute 'shapes'


soc: ace_v1x: stoping WDT during transiotion to D3

During transition into D3 state, watchdog timer have to be paused. FW can
resume it when core is re-enabled.
Signed-off-by: Tomasz Leman<EMAIL_ADDRESS>

@tmleman  can you rebase please?


[BACKLOG-3826] - Implemented universal directory variable and created fallbacks for legacy variables.

Commit is originally from wseyler fork.


227 display highest and lowest pressure range on the heat map

I'm unable to figure out how to pass the linter, it's not picking up that the two variables are defined in resultsParser.js and continues to flag them. Code is good to merge and works perfectly fine.

now passes linter

There is a bug when the window is resized it adds a second heatmap key below the first one

Every time the window is resized it adds another one. It's essentially appending another child in the html every time the window if resized. Probably a easy fix


MINOR: bump 2.5 versions to 2.5.1

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I tested this locally by running ./gradlew clean systemTestLibs and also:
TC_PATHS="tests/kafkatest/tests/streams/streams_application_upgrade_test.py::StreamsUpgradeTest.test_app_upgrade" _DUCKTAPE_OPTIONS='--parameters '\''{"from_version":"2.5.1","to_version":"2.7.0-SNAPSHOT","bounce_type":"full"}'\' bash tests/docker/run_tests.sh

to be sure that the new artifacts are resolved properly.
It looks like Jenkins isn't running these (probably due to the migration), and I believe the local tests to be sufficient for this change.


build fails on iOS in xCode with Parse issues on LocationManager.h

Your Environment

Plugin version: 2.2.5
Platform: iOS
OS version: iOS 10.10.5
Device manufacturer and model:
Cordova version (cordova -v): 5.0.0
Cordova platform version (cordova platform ls): ios
Plugin configuration options:
Link to your project:

Context
When compiling in xCode, the following Parse errors are thrown (all builds fine for Android). Apologies for the resolution of the image, I had to screenshot a remote desktop session.

Expected Behavior
Build without Parse errors
Actual Behavior
Parse errors thrown on build for iPhone 5 in xCode
Possible Fix

Steps to Reproduce

Install plugin
Try to build for iPhone 5 in xCode

Context
Trying to build for store submission
Debug logs

You probably didn't setup Xcode properly. Is your app building with cordova cmd?

You were right, it appeared to be a mis-configured xcode environment, unfortunately amongst the other things I was fixing I don't know which change solved this one.


Enhancement: Improve FAQ Help Center Page with Additional Q&A and How-To GIFs

Enhancement: Enhance the FAQ Help Center Page with Additional Q&A Content and How-To GIFs, providing users with comprehensive guides on various actions and processes.
Current Help Center:

Hi @jvJUCA, I'm ready to start working on this task. Could you give me more details about the GIFs you want? If possible, could you share a link to another website that has a similar feature to what you're envisioning?

Hi @jvJUCA, I'm ready to start working on this task. Could you give me more details about the GIFs you want? If possible, could you share a link to another website that has a similar feature to what you're envisioning?

Hi @TheMohit2003 ,
Thank you for jumping in to work on this task! Regarding the GIFs, we're looking for simple demonstrations of how to perform the actions mentioned in each FAQ question. For example, if the question is about how to reset a password, the GIF would show a user navigating to the password reset page and following the steps to reset their password.
We don't have a specific website in mind for reference, but we're aiming for clear and concise GIFs that visually guide users through the process. Let me know if you have any further questions or need more clarification!
Best regards,
@jvJUCA

@TheMohit2003 i am willing to help you with this, tell me with what can i ?

@hharshas , thank you for your help, but I can handle this myself . I have already created css classes and loading skeletons for gifs.

The issue has already been resolved in #366.


Remove os.exit in Run

kube-burner-ocp workloads are using workload helpers for Run command. However this Run() has os.exit(), which is blocking kube-burner-ocp to run any PostRun() commands.
As we removed os.exit() here, kube-burner-ocp workloads explicitly need to define PostRun() with os.exit().

This change is needed to add any cleanup in PostRun of kube-burner-workload.
I am adding network policy workload in kube-burner-ocp which creates a new namespace with pod, svc and route. I want to delete the namespace in PostRun of this kube-burner-workload.


Bluetooth: Mesh: Only block mod_sub if status was requested

Adds a cli_reset to mod_sub if no status response was requested. This
adds a non-blocking mode to the mod_sub calls, matching the behavior of
other cfg_cli functions.
Signed-off-by: Trond Einar Snekvik<EMAIL_ADDRESS>

@LingaoM This PR is in the same spirit as some of the fixes you made in #30086, but that doesn't appear to have this change.


Update randomText

I have changed both files and I don't know which is the best answer

I have deleted one of the conflicing files


Flush file writer buffer when unable to reconnect to the API

log.Fatalf() causes the CLI to exit (os.Exit(1)) which skips over the cleanup function and can leave data in the file writer buffer.
Returning from the receive loop should allow the buffer to flush in all cases.
I haven't tested this yet. Maybe it's better to explicitly call ss.CloseFileWriter() instead?

recv loop runs in separate thread, so i suspect returning out of the loop would not terminate the CLI. i guess you can test this by 'unplugging' your wifi ?

Tested with killing the network connection-- no errors and upon restoring connection stream resumed without issues.
Testing with API errors (killing pods) and after reconnect all data was received. Noticed that ctrl+c was properly printing pass report and exiting whereas current master was not printing pass report and often required pressing ctrl+c twice to exit.

But after looking at the logs, it did not hit this codepath (error connecting to API stream was not displayed).

Switched back to Fatalf but making sure to close the file writer first.

This has been tested to work by inducing an error state on the server, although it should be noted that the pass report still will not be printed in this case.


Change uptime to runtime and modify calculation to support functions

Current implementation uses the difference between now and the original deploy/launch time to determine the uptime value for a given workload. This value has the potential to confuse people. By changing to runtime, that value should now have the same meaning for both functions and services/elf, though calculated differently.

For services - the value will continue to be calculated as it is currently: now - deploy time
For functions - the value is an accumulated sum of actual execution time. We will time each execution of the function and add that time to a growing counter that will be used for the total elapsed execution time (runtime).

Having runtime be an accumulator for functions also makes for a nice prometheus metric.
We should also continue to maintain the original deployment time and make that available to control client queries, as it might be really useful to know that a function was deployed 3 weeks ago and has only accrued 12 minutes of total execution time.

Closed by #166


RTSP type h264 can't play

Hello
I create rtsp proxy server on fedora 23 server and test rtsp type mp4 it ok, but when i test by rtsp type by h264 it can't play for me
Please tell me what i wrong config
Thanky everyone

hello
It's not enought information please explain it
give me your the config code to understand what you do

Thank you @victorgrenke   for reply
Ok. first I Create vmware and install fedora server 23 and then i install ws_rtsp_proxy,
I config port 8015 and run command service ws_rtsp start
and i create html5 file for client and use test.bundle.js from http://specforge.com/html5playerstream/index.html and change websocket from my websoceket
ps. url test is ="rtsp://<IP_ADDRESS>/vod/mp4:BigBuckBunny_115k.mov"
and test run oh.it ok can play lok so good
then after i change url it real ipcamera = "rtsp://view:1234567@xx.xx.xx.xx:1554/h264/ch3/sub/av_stream"
it can't play me i dont know how to check what i wrong
Please tell me
Thank you for everyone

Hello
At current version our player is not supporting user:password link
Try the camera without user password.
Some latter we support it.

Also waiting for this feature.
It would help a lot if supports username/password in the url.

@ChampsReprobate I just found that the underlying RTSPConnection of rtsp/connection.js support username/password authentication. however, RTSPPlayer of rtsp_player.js doesn't pass the username/password to RTSPConnection constructor.

you can fix it like below and re-pack your test.bundle.js:

Thank you verymuch for this , I will test it  Thank you

For the current version of player I included your changes, thank you.
Authorization of flows (to proxy side and player) is also supported in new version
It will be posted on GitHub as soon as all current problems with proxy infrastructure will be solved,
because new version works with proxy, backward incompatible that now is in repository.

Hello. I'm experimenting a similar issue. I can't get stream with a URL like "rtsp://192.168.xx.xx:554/cam/realmonitor?channel=CHANNEL&subtype=ENCODING&authbasic=USER:PASS_BASE_64" This works well with VLC media player and VXG Chrome plugin, but I can't get any stream from the proxy.
The proxy works fine with the url "rtsp://<IP_ADDRESS>/vod/mp4:BigBuckBunny_115k.mov" though.
Any ideas on how to make it work? Thanks!

@ferglezt maybe you can try: rtsp://{User}:{PlainPassword}@192.168.xx.xx:554/xxxxxx

@tsingyee I just tried urls like:
rtsp://user:pass@192.168.xx.xx:554/cam/realmonitor?channel=1&subtype=00
rtsp://user:pass@192.168.xx.xx:554/cam/realmonitor
rtsp://user:pass@192.168.xx.xx:554/
But still not getting any results. I don't know the reason why I can't catch any stream here, but VLC and VXG plugin work fine. Thanks!

Fixed in latest version


chirp: 20180707 -> 20180815

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/chirp-daily/versions.
meta.description for chirp is: '"A free, open-source tool for programming your amateur radio"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815/bin/.chirpw-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815/bin/chirpw had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 20180815 with grep in /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815
directory tree listing: https://gist.github.com/db4dbc76dafac23bc988c99b017521b6
du listing: https://gist.github.com/40e1a48873d77c743ae246832dd071a2

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 4f2dbff38c004bb717b6ed874b1395e189f0c514 && nix-shell --pure -I nixpkgs=. -p chirp

cc @the-kenny for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: chirp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: chirp
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815/bin
patching script interpreter paths in /nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815
/nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815/share/chirp/chirpw: interpreter directive changed from "/usr/bin/env python" to "/nix/store/9gfk6klv3zajmibdsvn4yhz20z6vg5gw-python-2.7.15/bin/python"
/nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815/share/chirp/rpttool: interpreter directive changed from "/usr/bin/env python" to "/nix/store/9gfk6klv3zajmibdsvn4yhz20z6vg5gw-python-2.7.15/bin/python"
checking for references to /build in /nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815...
/nix/store/2n0y2k7bmgy03w90xg97ljsa9jy6dcac-chirp-daily-20180815

Success on x86_64-linux (full log)
Attempted: chirp
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815/bin
patching script interpreter paths in /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815
/nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815/share/chirp/chirpw: interpreter directive changed from "/usr/bin/env python" to "/nix/store/9qjc0d5557h3dkpsvfq98b2k96lnb6pw-python-2.7.15/bin/python"
/nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815/share/chirp/rpttool: interpreter directive changed from "/usr/bin/env python" to "/nix/store/9qjc0d5557h3dkpsvfq98b2k96lnb6pw-python-2.7.15/bin/python"
checking for references to /build in /nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815...
/nix/store/9f094sgqbg3pl9azikqzplq7fi5g0n7c-chirp-daily-20180815


Update RestAction.java

allow to override the default success/failure consumers

Would it be better to do this through. Method of some sort?


Docs PLO adjustments for SEO

Hey @susanev! Here's the PR for the adjustments to the docs PLOs. There aren't any drastic keyword changes since our target keywords look good - we were mostly making sure title tags were within recommended character lengths so they don't get cut off by search engines.
Checklist:

[x] I have reviewed the style guide.
[x] If blogging, I have reviewed the blogging guide.
[x] I have manually confirmed that all new links work.
[x] I added aliases (i.e., redirects) for all filename changes.
[x] If making css changes, I rebuilt the bundle.

@susanev sweet! I reverted those title tags back to the originals, so hopefully everything is good now :) Thanks again!


Possible issue with implementation

Hi, thanks for the great work and the corresponding implementation. In the following:
https://github.com/zhangbinchi/certified-deep-unlearning/blob/912a66c141e6a6b332c97a2b7c3c3133e2190a5f/unlearn.py#L204-L209
It seems that you do not use the unlearning set. Shouldn't we pass the gradients wrt the unlearn set to the newton_update function?

Hi, sorry for the confusion caused. In the paper, it is mentioned in the Computational Efficiency section that we can replace the gradient over the retained set with the gradient over the unlearned set for a lower complexity. However, this can cause more error in the sequential unlearning setting where the model to be updated is getting far away from the original minimum point. Hence, in sequential unlearning, we do not use the gradient over the unlearned set to estimate the gradient over the retained set. That is why the unlearned set is not used in the code lines you mentioned. When you are not handling a sequential unlearning task, you can use the comment lines (line 204 and 205) for a single unlearning operation.
Hope this helps. Please let me know if you have further concerns.

Thanks for your detailed response. In the Computational Efficiency Section of the paper you propose replacing the retain gradients in the paper with the negation of the unlearn gradients, but if I understand correctly this is not what happening in the code: In the code you replace the grad on the unlearn with that of the retain, no?

I got you. In the unlearn.py file, I provide both ways, using negative grad wrt unlearn set and using grad wrt retain set. Both works, and the difference is that the former has a smaller complexity but larger approximation error (as the original model is not an exact minimum). You can choose either way to go. The former one corresponds to the combination of lines 204-205 and 211 (which appears as comments in the current version), and the later one corresponds to the combination of lines 206-207 and 212. Hope this helps.


Add parade: a PostgreSQL-based task queue

Instructions for reviewer:

Please read #609 to understand what it's all about.  This PR implements most but not all of that spec.  In particular, ExtendTask will be added later.
Now read the SQL.  It's pretty much self-contained and has almost all of the business logic.  You may want to read about SELECT ... FOR UPDATE SKIP UNLOCKED; good references are the very fine manual and a What is SKIP LOCKED for in PostgreSQL 9.5?.
Now read ddl.go -- mostly a direct access layer to the SQL.  It does use pgx (rather than sql or sqlx) for some operations that are not supported on sql*.  Question for reviewer: should it use only pgx?
Finally read ddl_test.go, which actually uses it.

@guy-har yours for review, thanks!  (And very happy for any other reviewers, this needs any scrutiny it can get...)

Now ready for @tzahij too, I think :-)

LGTM
But I'd prefer that another reviewer with SQL experience (maybe @tzahij ) would also take a look and approve. at least at the SQL part.

@tzahij and/or @nopcoder: could either of you, please?

Thanks!
Pulling...


SCM breeze will always warn if RVM is installed

The following warning will always play, even if the condition is satisfied.
SCM Breeze must be loaded after RVM, otherwise there will be a conflict when RVM wraps the 'cd' command.
Script should check if rvm has been sourced and only report warning if true.

Found the error:
if [ -e "$HOME/.rvm" ] && ! type rvm > /dev/null 2>&1; then
needs to be if [ -e "$HOME/.rvm" ] && type rvm > /dev/null 2>&1; then
Because, in posix shells, return value of 0 is true:

The if compound-list shall be executed; if its exit status is zero, the then compound-list shall be executed and the command shall complete. Otherwise, each elif compound-list shall be executed, in turn, and if its exit status is zero, the then compound-list shall be executed and the command shall complete. Otherwise, the else compound-list shall be executed.

type would return 0 = true, and then the echo statements would run.


Update streams_quick_start.md

Just a little fine tuning of the technical writing

 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.:x: voipengineer sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thank you, @voipengineer!


Add optional env variable to get client IP from a custom header

Some proxy services (like Cloudflare) set a custom header on proxied requests that can be used to extract the original IP address of the connecting client.
This PR adds the ability to use such a header through an optional env variable.
https://developers.cloudflare.com/fundamentals/get-started/http-request-headers

If the env variable is not set, then the server will use request.ip as before.
This means that the change can be integrated without breaking anything.


Doesn't seem to handle module.exports/require correctly for classes

I have a class function in one file that flow reports an issue with. Everything works fine in the code, so not sure why it would be complaining on this. This is (something like) the code:
// doc.js
var Doc = function (params) {
  ...
};

module.exports = Doc;

// elsewhere...
var Doc = require('doc');

var doc = new Doc({ some: 'params });

flow reports:
 57:       this.doc = new Doc(this.find('#doc'), doc_params)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ constructor call. Constructor cannot be called on
 57:       this.doc = new Doc(this.find('#doc'), doc_params)
                          ^^^ exports

What do I need to do to get flow to work for this?

You should annotate that it's a constructor function (flow doesn't infer that it's a constructor for exports I believe):
Have you tried:
const Doc = function(params) {...}
module.exports = (Doc: Class<Doc>)

Is there a way to do that as an external declaration? I'm trying to avoid adding type declarations in the code for the time being and just take advantage of the type inference. I tried to follow the example here but that didn't do the trick for me. Might be I'm declaring something wrong, though.
declare module 'doc_editor/doc' {
  declare class Doc {
    constructor(): void;
  }
  declare module.exports: typeof Doc;
}

Just tried your solution as well, and that didn't seem to work either.

This kind of constructor support is dropped a while ago


Enable prefetch for certain routes

Description
Currently webpack automatically generates prefetch hints for all JavaScript files of our async routes.
Downloading all that chunks at start is not necessary + it leads to additional load on server, so this pr enables prefetching only for certain most common routes.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[x] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Visual context
Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[x] I have performed a self-review of my own code
[ ] I have requested at least 2 reviews (If the PR is significant enough, use best judgement here)
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] If package-lock.json has changes, it was intentional.
[x] The base of this PR is master if hotfix, develop if not

done


Fix DisplayVersion Google.AndroidStudio 2<IP_ADDRESS>

cf. https://github.com/microsoft/winget-pkgs/issues/108921

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

VisualStudio editor in browser seems to use old version of master branch?!

[Policy] reset feedback

@wingetbot run


functions that return promises

Write and Read should return promises and called from async functions
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function
function resolveAfter2Seconds() {
  return new Promise(resolve => {
    setTimeout(() => {
      resolve('resolved');
    }, 2000);
  });
}

async function asyncCall() {
  console.log('calling');
  const result = await resolveAfter2Seconds();
  console.log(result);
  // expected output: "resolved"
}

asyncCall();

And making a promise with a recursion: https://stackoverflow.com/questions/49523900/promise-with-recursion


test: avoid unconditional reverts in stack traces tests

Re-applying the changes from https://github.com/NomicFoundation/hardhat/pull/5467
See discussion in https://github.com/NomicFoundation/hardhat/issues/5443

Thanks for the reviews. I'm going to merge this after the PR in Hardhat is merged.