flag Docker in OEX directory

A simple solution for the few cases that I detected might be
to set the flag Docker-available manually by an OEX_admin,
This might also be useful to signal Docker if the package owner
refuses a related PR  or doesn't update his OEX entry.
Should work to get OEX closer to reality

We have developed an alternative method for searching Docker applications, so the this approach is no longer necessary.

great, Thanks!


Change API

factories.register is redundant and ambiguous about relationship.
It can be
@Association(() => [B, C])
class A {
   @Fixture(type: _ => B)
   b: B
   @Fixture(type: _ => C)
   c: C
}

export const AFactory = new Factory(A)

Fix https://github.com/MOCHI-inc-JAPAN/class-fixtures-factory/issues/14, too. This change.


React Storybook is still v9

The react storybook link inside the components is sending me to v9
Detailed description
The react storybook link found inside each component in https://www.carbondesignsystem.com is sending me to v9 and not v10.
Additional information

you caught us in the middle of updating things, should be good now 😄


What does this get me?

This is my first time trying GitHub Actions, so apologies for the basic question.
I see that the example workflow in this repo's readme contains this example step:
- uses: actions-rs/cargo@v1
  with:
    command: build
    args: --release --all-features

Based on the readme of this repo it wasn't clear to me what benefit this would have over just:
- run: cargo build --release --all-features

which appears to work.
Maybe the readme could emphasize some of the benefits?
Thanks!

Hi, @dtolnay.
This is a totally legit and great question! You are right to assume that simple - run: step is not that different, here is what this Action does differently:

It can call cross instead of cargo binary with use-cross: true flag passed and this allows you to construct such a build matrixes:

  compile:
    name: ${{ matrix.toolchain }} / ${{ matrix.triple.target }}
    runs-on: ${{ matrix.triple.os }}
    strategy:
      fail-fast: false
      matrix:
        triple:
          # Tier 1 platforms
          - { os: 'ubuntu-latest',  target: 'x86_64-unknown-linux-gnu', cross: false }
          - { os: 'ubuntu-latest',  target: 'i686-unknown-linux-gnu',   cross: true }
          - { os: 'macOS-latest',   target: 'x86_64-apple-darwin',      cross: false }
          - { os: 'windows-latest', target: 'x86_64-pc-windows-msvc',   cross: false }

          # Tier 2 platforms

          ## ARM64
          - { os: 'ubuntu-latest', target: 'aarch64-unknown-linux-gnu',  cross: true }
          - { os: 'ubuntu-latest', target: 'aarch64-unknown-linux-musl', cross: true }
          ## ARMv7
          - { os: 'ubuntu-latest', target: 'armv7-unknown-linux-gnueabihf',  cross: true }
          - { os: 'ubuntu-latest', target: 'armv7-unknown-linux-musleabihf', cross: true }
    steps:
      - uses: actions/checkout@v2
      - name: Install toolchain
        uses: actions-rs/toolchain@v1
        with:
          profile: minimal
          toolchain: ${{ matrix.toolchain }}
          override: true
      - uses: actions-rs/cargo@v1
        with:
          command: check
          args: --target=${{ matrix.triple.target }}
          use-cross: ${{ matrix.triple.cross }}

It enables problem matcher for cargo messages. Basically, GitHub automatically runs specific regexp on each line of stdout/stderr and if there is a match, nifty annotation is created for it; in this execution you can see that clippy execution failed and in here a few rustc warnings can be found.

This Action was created just after the GitHub Actions became publicly available, and there was no clear vision on how should all that small actions-rs ecosystem behave; for example, actions-rs/components-nightly was deprecated because rustup can do that functionality better, and there are plans to deprecate some other Actions too (actions-rs/tarpaulin is a first candidate).
So, at this point I'm inclined to agree that plain - run step looks much better in a most cases and does almost the same thing. I'll add it to the readme, thanks for pointing it out!

There should be an example featuring the complex usage you showed off here, @svartalf. Currently there are not even any examples that I can find that use macos or windows but I think there should be.

Is there a simple way to get the fancy error messages with a simple run step or is this truly exclusive to using a special action?


Documented version not available in gradle plugins

It looks like all the documentation on the site is for the new API with RC9, but the only versions available on https://plugins.gradle.org/plugin/io.gitlab.arturbosch.detekt are M8 and below.  It does have a beta-5 but I can't seem to find anything if those two are the same.  Is this pending/planned to be published or intentional?

It does seem like RC9 did not get published to the Gradle Plugin portal correctly. @arturbosch can you take a look?

Ups, now it should be up and usable :).

It looks like it's directing to the wrong URL for download of RC9.  It's directing to https://repo.jfrog.org/artifactory/libs-release-bintray/io/gitlab/arturbosch/detekt/io.gitlab.arturbosch.detekt.gradle.plugin/1.0.0.RC9/io.gitlab.arturbosch.detekt.gradle.plugin-1.0.0.RC9.jar?referrer which 404s, when it looks like it's really supposed to be hosted at https://repo.jfrog.org/artifactory/libs-release-bintray/io/gitlab/arturbosch/detekt/detekt-gradle-plugin/1.0.0.RC9/detekt-gradle-plugin-1.0.0.RC9.jar  Is it possible there was an error in how it posted up to the gradle plugin repo?

Where does the first link come from? Within gradle build scripts RC9 gets resolved

I'm not sure, I've not been able to reproduce it since so it maybe was a bug that was resolved somewhere


nixos/nextcloud: default createLocally to false

Description of changes
Follow up PR to address concerns raised on https://github.com/NixOS/nixpkgs/pull/229159.

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

gentle ping @Misterio77

Oops sorry, forgot about this.

the release notes still need to be updated

In relation to the latest PR, you mean? IMO this still works even after this PR's changes:

services.nextcloud.database.createLocally now uses socket authentication and is no longer compatible with password authentication.

If you want the module to manage the database for you, unset services.nextcloud.config.dbpassFile (and services.nextcloud.config.dbhost, if it's set).
If your database is external, simply set services.nextcloud.database.createLocally to false.
If you want to use password authentication and create the database locally, you will have to use services.mysql to set it up.

The release notes currently say: If your database is external, simply set [`services.nextcloud.database.createLocally`](#opt-services.nextcloud.database.createLocally) to `false`.
This is no longer needed AFAIU, so it should go.

This is no longer needed AFAIU, so it should go.

Okay! Removed this section now. @Ma27

This just broke my deployment. Is this expected? Some time ago, I got the message, that I need to remove
dbhost and dbpassFile. It would have been great if there were a similar message in this case, because it wasn't very intuitive to figure it out. Would that have been possible without introducing a new configuration variable, that would be required to be set?

@frederictobiasc I think you want to enable createLocally for your deployment, this is what was introduced for a while and reverted because of the current release restrictions.


Change date color and size of feed

Issue

close #90

Overview (Required)

change opacity and style of date text

Links

https://developer.android.com/jetpack/compose/themes#emphasis
https://developer.android.com/reference/kotlin/androidx/compose/material/package-summary#localcontentalpha

Screenshot

Before
After

👀


Failled to add user and code

Alarmo Version
v1.8.6
HA Version
2022.3.5
Bug description
When i want to add a user and password i get this error
" Something went wrong!
Message format incorrect: extra keys not allowed @ data['is_admin']
Response error: 400
Please report the bug "
Steps to reproduce
just i install the alarmo addon and i test, everything it's working fine i can arm and disarm the alarm without code but i can't add code
Relevant log output
No response

is_admin has been removed from the code in november.
Your alarmo instance is out of date.
Check the version number in the top right corner.

You are right I have version 1.7.2 but how I can update this? I remove from integration and after from hacs. I restart ha and downloaded again in hacs version 1.8.6 , restart ha and add in integration and is still version 1.7.2. What can I do??

If the Alarmo configuration panel version is out of date, you need to clear your browser cache.

Yes, you are right again, I clean the cache and everything it's working fine now. Thank you and sorry for this, can be closed now. Thank you!


No syntax highlighting for fresh install in Jupyter Notebook

Describe the bug
I installed the .NET interactive kernel as described by this page.. The installation was successful and I can run .NET code no problem, but it looks like the syntax highlighting is missing:
Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

OS

[ ] Windows 11
[x] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[x] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[x] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[ ] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

Screenshots

Syntax highlighting in Jupyter is not provided or affected by the .NET Interactive kernel, so I'm marking this as external. This must be related to the Jupyter install itself.

Yes, there's usually basic syntax highlighting for C# in Jupyter Notebook and Jupyter Lab.

No, Jupyter doesn't use .NET.

Are you seeing this with brand new notebooks created in Jupyter?

Now is 2024-03-06, I have exactly the same problem as described above.  Still not solved.

Interestingly, I can have R syntax highlighted correctly but couldn't do that for Python (pylance).


cannot able to go fix model setting error

  File "C:\Users\haide\AppData\Roaming\Blender Foundation\Blender\3.1\scripts\addons\cats-blender-plugin-master\ui\armature.py", line 205, in invoke
    return context.window_manager.invoke_props_dialog(self, width=dpi_value * 3.25)
TypeError: WindowManager.invoke_props_dialog(): error with keyword argument "width" -  Function.width expected an int type, not float

location: <unknown location>:-1

Duplicate of https://github.com/absolute-quantum/cats-blender-plugin/issues/379

when it going to fix u already fix it but didnt update it

and how can i avoid changing bones name when fix click   since i do modding bone name changes it fix the mesh and bone but how can i avoid that


89 use data transfer objects dtos

the tests fail to build because i have removed the class's ICheepService and CheepViewModel. The test have to be changed to test the repository


Editgreeting: Allow commas to be in the greeting

Previously, the variable 'newGreetingText' just took arg[1]. However, if there is a second comma, the description will only be set to what text is before that second comma. This should help fix that.
[19:29:27] #XTheElegantShadowX: ;editgreeting [public], Welcome, Master! May I drink your bathwater?
[19:29:27] *The Scribe: Greeting updated: Welcome

Thank you. <3


Error when extracting wind speed

When I want to extract the wind speed with "wind_speed_10m" as parameter I've an issue: 'Parameter wind_speed_10m not available in model mix'.

Dear @TanguyLevent can supply a minimal example? Judging from the error message, I would say you need to supply a unit with the parameter, i.e. 'wind_speed_10m:ms', 'wind_speed_10m:kn', 'wind_speed_10m:kmh' or 'wind_speed_10m:bft' depending on what you are looking for exactly. The complete alphabetic list of available parameters can be look up at https://www.meteomatics.com/en/api/available-parameters/alphabetic-list/
Let me know if that helps.
Kind regards
Markus

thank you Markus, it is exactly what you mentionned. I forget the supply unit...
Best, Tanguy


connect Oracle ORA-12514

when I connect oracle database,it's not work.
ERROR:
ORA-12514: TNS:listener does not currently know of service requested in connect
Database : 19c
Oracle instantclient 12.2
Golang version : 1.11

Images cannot be displayed.


macOS tags

I love this project and use it quite a lot.
I would be great if organize could also support tagging under MacOS.
Library for setting tags seems to be available
https://pypi.org/project/macos-tags/

Good idea! Do you want to set tags with organize or filter by tags?

First priority to set tags but filter would also be nice.

I am trying a workaround not be using the shell command in the yaml file

Fastest solution would be to install the library with pip3 install macos-tags and then creating a custom python action like this:
rules:
  - folders: ~/Desktop/test
    actions:
      - python: |
          import macos_tags
          macos_tags.add("Rot", file=path)

I have the "Rot" tag on my (German) system and this seems to work fine.
To see available tags, run:
python3 -c "import macos_tags as m;print(m.tags())"

But I'll look into creating a action for this!

This is a great solution!
I am pretty clueless about python. I am already using it in trial error approach e.g.
filters:
  - extension: pdf
  - python: |
      return {"year": path.stem[0:4]}

I am using then {python.year} to create a folder and other things. So since there is only one - python statement in your rule syntax I would have to combine your approach above.
filters:
  - extension: pdf
  - python: |
      import macos_tags
      macos_tags.add("Rot", file=path)
      return {"year": path.stem[0:4]}

I would try accordingly. And I am also not sure whether file=path needs to be customized.

You nearly got it, but it should be a python action.
Your rule may look like this:
filters:
  - extension: pdf
  - python: |
      return {"year": path.stem[0:4]}
actions:
  - python: |
      import macos_tags
      macos_tags.add("Rot", file=path)

file=path is correct and does not need to be modified. path is what comes out of the given filters.

Also note that you can have as many python actions or filters as you want.

Thanks for this excellent support.
So I will change accordingly
filters:

extension: pdf
python: |
return {"year": path.stem[0:4]}
actions:
python: |
import macos_tags
macos_tags.add("Steuer " + path.stem[0:4], file=path)

Hope this calculated Tags will work aswell

Please try it out and let me know! If you want to specify a color, check the macos_tags reference: https://pypi.org/project/macos-tags/
But I guess the next version of organize will have this built in ;)

Working!! I am very happy.

Yay 👍

      - python: |            import macos_tags            from macos_tags import Tag, Color            tag = Tag(name="Steuer " + path.stem[0:4], color=Color.GREEN)            macos_tags.add(tag, file=path)
Now supporting color aswell!

Fixed the formatting. Code looks good! Started working on this feature here: #113 but it still misses color, tests and documentation.

I just pushed v1.10 with this feature included (minus the color functionality)

Great!
You are really fast! I have deep respect for your work.
Klasse Arbeit!


Fix typo in src option docs

As far as I can tell, source is not a valid option, so this is probably a typo. This PR fixes it

D'oh, thanks!


[index] Index fix keyvimerger install2

replaces #45
implements and uses a python version of keyvimerger which is compatible to the cpp version.

Coverage remained the same at 93.223% when pulling 8a4d2daaf4aca5b32898e754f7abc81c0e649c9a on hendrikmuhs:index-fix-keyvimerger_install2 into 7159787bae7e118b8b1150877ffd455872ae63e3 on KeyviDev:index.


Revert #1154 (Fix NumPy DocTests)

Description
Reverts #1153 which had reverted #1144, why the NumPy string formats are not working is strange and unexplained as I don't believe it is directly related to NumPy releases

Hello Darkness, my old friend

ok, so it appears to pass CI for numpy<2 but fail for numpy>=2 (from what i can tell)

Ohh, doctest is disabled for 3.8 which is why it passed.
The reason I closed the PR is that reverted the changes, the PR failed on all the changes.
So I reran the main CI that had just failed on doctest and it passed just two days without any changes.
This really is a mystery to understand what is causing the problem.


[FEATURE] Add S3 support for Fileset Hadoop catalog

Describe the feature
Fileset is a new concept brought in 0.5.0 to manage the non-tabular data, the current implementation uses HCFS to manage the physical data. With HCFS, the Hadoop catalog should support different underlying storage, but currently we only verified local file system and HDFS.
In this issue, we should also support S3, to make the fileset hadoop catalog work with S3 object store.
Motivation
The reason to support S3 is that it is vastly used on the public cloud, we should add this support anyway.
Describe the solution
No response
Additional context
No response

I think we can change this feature to Support Object Store provided by Cloud Service, so we can add subtask to support Azure Blob and Aliyun OSS

@xiaozcy can you please leave a message here, so I can assign the issue to you.

@xiaozcy can you please leave a message here, so I can assign the issue to you.

Sure.


Functions configmaps UI

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/test pre-busola-ui-test


Support having unlisted pages

After some recent discussions with the Documentation team I was asked to remove the link to the "Roadmap" page (https://charmhub.io/nginx-ingress-integrator/docs/roadmap) from the navigation. It's linked in the README directly, and was felt to be unnecessary to list separately in the navigation.
To do so I had to add the following the end of the discourse navigation table in the /docs/ index:
|  | roadmap | [Roadmap](/t/nginx-ingress-integrator-docs-roadmap/7432) |

Apparently it breaks if it's not the last element (hopefully if you need multiple hidden elements that possible too...
Is it possible for this project to support that somehow?

Yes it should be possible, I can see how this works. I assume that if the level field is not filled in, it doesn't show up but can be linked to.
We should combine this with #12 and essentially add a configuration file potentially something like:
files:
  <relative path in docs folder>:
    rank: <number indicating the rank, the lower the higher it is listed in its directory, defaults to ranked alphabetically for those items without a rank>
    listed: <boolean for whether to list it in the navigation, defaults to true>

@gregory-schiano any comment?


Monospace Font fixing + Emoji consistency

The font inside code comment are incorrect.
This commit fix the font inconsolata that was loaded in the Tchap Theme. The fix is better for rebase purpose (but we will never rebase this version again ?)
Should also fix emoji on message (use the embedded emoji font instead of the system font)
Linked to https://github.com/tchapgouv/matrix-react-sdk/pull/408
Close #201

Before -> after

This feature has been tested and can be merged


chore(autoware.repos): fix autoware.universe and autoware_launch versions to 0.38.0

Description

[x] #5440

Based on the discussion https://github.com/orgs/autowarefoundation/discussions/5292, we will fix the versions of autoware.universe and autoware_launch.

https://github.com/autowarefoundation/autoware.universe/pull/9266
https://github.com/autowarefoundation/autoware_launch/pull/1226

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

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

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

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

[ ] The PR follows the pull request guidelines.

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

[ ] There are no open discussions or they are tracked via tickets.

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

See my comments here:

https://github.com/autowarefoundation/autoware/pull/5440#issuecomment-2478162005

I think this PR needs to be revised since

https://github.com/autowarefoundation/autoware/pull/5440 was merged and introduced some of the additions from here.

@xmfcx It's ready to review!

@youtalk -san, Looks good overall but since this revert was required, should we maintain 2 different autoware/docker/Dockerfile's too?
Because the dockerfile is referring to package names. In stable and nightly .repos, they will point to different packages. (prefix vs non-prefix or maybe new packages will be added).

@xmfcx Based on that way of thinking, we would also need to maintain two ansible directories, which we want to avoid.
In that case, I think it would be better to separate the main branch for providing a stable version and the development branch for providing the nightly version.

I made 0.39.0 version PR https://github.com/autowarefoundation/autoware/pull/5479 instead.


Make different types of enemies more visually distinct.

During a recent playtest, a player said they could not tell which enemies were firing which shots. I should somehow make this more clear.

I should try to add basic 3D models for each enemy type.

Melee enemies are finalized. Other enemies still require new or revised models.

All enemies are now distinct. At some point I may want to change the model for the mortal enemies as it was originally a place holder, but it works well enough for now.


Cache Molecule Smiles

Description
Adds smiles caching to the FrozenMolecule class to greatly improve performance when hashing molecules.
For a system of 1000 atoms this resulted in a roughly 2x performance gain.
Status

[x] Ready to review
[ ] Ready to go

Codecov Report

Merging #308 into 0.3.0-branch will increase coverage by 0.03%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           0.3.0-branch     #308      +/-   ##
================================================
+ Coverage         75.62%   75.66%   +0.03%
================================================
  Files                17       17
  Lines              5125     5132       +7
================================================
+ Hits               3876     3883       +7
  Misses             1249     1249

Impacted Files
Coverage Δ

openforcefield/topology/molecule.py
86.57% <100%> (+0.1%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #308 into 0.3.0-branch will increase coverage by 0.03%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           0.3.0-branch     #308      +/-   ##
================================================
+ Coverage         75.62%   75.66%   +0.03%
================================================
  Files                17       17
  Lines              5125     5132       +7
================================================
+ Hits               3876     3883       +7
  Misses             1249     1249

Impacted Files
Coverage Δ

openforcefield/topology/molecule.py
86.57% <100%> (+0.1%)
:arrow_up:

Continue to review full report at Codecov.

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


mhci and mhcii allele list different behavior

When passing a list of alleles to the iedb.query_mhcii_binding, it returned predictions for all alleles.
When doing the same for the mhci function, however, it only returns predictions the last allele in the list.
I imagine this is not the intended behavior. Is there any way to make the mhci prediction behave like the mhcii ?

OK, figured it out. It was two things.
When passing multiple alleles to the MHCI API I had to pass a list of equal length of epitope lengths for the predictions.
Also the list is not a python list(), but rather a string where all the lengths are divided by commas. Same thing for the epitope list in this case.
So for instance:
lengths = '9,9,9' #works
lengths = [9,9,9] #doesn't work


Add icon for amenity=bicycle_parking

Thanks for your contributions!
Would it be possible (no hurry) to also create an icon for amenity=bicycle_parking?

Current icon in SJJB repository: http://www.sjjb.co.uk/mapicons/svg/transport/parking_bicycle.svg
Works reasonable well on 16x16, but could perhaps be crisper.

Bicycle parking is now available:

Thanks!


Pac4JProfileInjectorFactory disable CDI Injection Resteasy

Im using jax-rs-pac4j with resteasy and wildfly 10.1.0.Final Im using this config:
@Provider
public class Pac4JFeature implements Feature {

    @Inject
    @Props("vodafone")
    protected Properties properties;

    public Config getConfig() {

        OidcConfiguration config = new OidcConfiguration();

        config.setClientId(properties.getProperty("oidc.client-id"));
        config.setSecret(properties.getProperty("oidc.client-secret"));
        config.setDiscoveryURI(properties.getProperty("oidc.discoveryURL"));
        //
        OidcClient oidcClient = new OidcClient(config);
        oidcClient.setName("appDirect");
        Config result = new Config(oidcClient);
        result.getClients().setDefaultClient(oidcClient);
        result.getClients().setUrlResolver(new JaxRsUrlResolver());
        result.getClients().setCallbackUrl("/openid/login/check");
        return result;
    }

    @Override
    public boolean configure(FeatureContext context) {

        context
            .register(new JaxRsConfigProvider(getConfig()))
            .register(new Pac4JSecurityFeature())
            /*
             Pac4JProfileInjectorFactory breaks CDI injection
             .register(new Pac4JProfileInjectorFactory()) // only with Resteasy
             */
            .register(new ServletJaxRsContextFactoryProvider());

        return true;
    }
}

And this controller:
@Path("/")
@Stateless
public class OpenIdController {

    private static final Logger log = LoggerFactory.getLogger(OpenIdController.class);

    @Inject
    private TokenLogin tokenLogin;

    @Path("/openid/login")
    @Pac4JSecurity(clients = "appDirect")
    @GET
    public Response login(@Context SecurityContext sec, @Pac4jProfile CommonProfile profile) {
        log.info("SecurityContext: {}", sec);
        log.info("Logged, redirecting to dashboard the user {}", profile);

        return buildLoginToken(profile.getEmail());
    }

    @GET
    @Path("/openid/login/check")
    @Pac4JCallback(skipResponse = true)
    public Response loginCB(@Context SecurityContext sec, @Pac4jProfile CommonProfile profile) {
        if (profile != null) {
            log.info("SecurityContext: {}", sec);
            log.info("Logged, redirecting to dashboard the user {}", profile);
            return buildLoginToken(profile.getEmail());
        } else {
            throw new WebApplicationException(401);
        }
    }

    private Response buildLoginToken(String email) {
        String token = tokenLogin.create(email);
        return Response.temporaryRedirect(URI.create("https://dummyyeah.com?token=" + token)).build();
    }
}

The class TokenLogin is a POJO with any annotations. If I add the Pac4JProfileInjectorFactory to FeatureContext context, the tokenLogin is not injected and is null and without the  Pac4JProfileInjectorFactory the tokenLogin is injected as expected.
My workaround is this:
@Priority(Priorities.AUTHORIZATION + 50)
@Provider
public class Pac4JProfileInterceptor implements ContainerRequestFilter {

    @Context
    private HttpServletRequest req;
    @Context
    private HttpServletResponse res;

    @Override
    public void filter(ContainerRequestContext requestContext) throws IOException {
        Optional<CommonProfile> profile = new ProfileManager<>(new J2EContext(req, res)).get(true);
        if (profile.isPresent()) {
            ResteasyProviderFactory.pushContext(CommonProfile.class, profile.get());
        }
    }
}

And replace all the @Pac4jProfile with @Context.
Another option is to cast the  SecurityContext#getPrincipal
Im doing something wrong or is a bug?
Thanks!

@jarnaiz @victornoel I have made some some changes in #42 that could possibly fix this issue as well. I have added tests for resteasy with CDI as part of that.
@jarnaiz you could possibly take those changes and see if it helps your case.

@jarnaiz yes please do so :)
thanks a lot @ganeshs

@jarnaiz did you get the opportunity to see if things were working as expected? For the record, we are going to release 2.3.0 and 3.0.0 soon (see #44)


ArmorChroma Incompatibility

Though backpacks can be worn as armor, ItemBackpack cannot be cast to ItemArmor, causing a crash with ArmorChroma:
[main/FATAL] [minecraft/Minecraft]: Unreported exception thrown!
java.lang.ClassCastException: net.mcft.copy.backpacks.item.ItemBackpack cannot be cast to net.minecraft.item.ItemArmor
	at nukeduck.armorchroma.GuiArmor.drawPartialRow(GuiArmor.java:145) ~[GuiArmor.class:?]
	at nukeduck.armorchroma.GuiArmor.drawPiece(GuiArmor.java:137) ~[GuiArmor.class:?]
	at nukeduck.armorchroma.GuiArmor.draw(GuiArmor.java:92) ~[GuiArmor.class:?]
	at nukeduck.armorchroma.ArmorChroma.onRenderOverlay(ArmorChroma.java:50) ~[ArmorChroma.class:?]
	at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_35_ArmorChroma_onRenderOverlay_Pre.invoke(.dynamic) ~[?:?]
	at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90) ~[ASMEventHandler.class:?]
	at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:182) ~[EventBus.class:?]
	at net.minecraftforge.client.GuiIngameForge.pre(GuiIngameForge.java:907) ~[GuiIngameForge.class:?]
	at net.minecraftforge.client.GuiIngameForge.renderArmor(GuiIngameForge.java:290) ~[GuiIngameForge.class:?]
	at net.minecraftforge.client.GuiIngameForge.func_175180_a(GuiIngameForge.java:155) ~[GuiIngameForge.class:?]
	at net.minecraft.client.renderer.EntityRenderer.func_181560_a(EntityRenderer.java:1098) ~[buq.class:?]
	at net.minecraft.client.Minecraft.func_71411_J(Minecraft.java:1117) ~[bib.class:?]
	at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:397) [bib.class:?]
	at net.minecraft.client.main.Main.main(SourceFile:123) [Main.class:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_181]
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_181]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_181]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_181]
	at net.minecraft.launchwrapper.Launch.launch(Launch.java:135) [launchwrapper-1.12.jar:?]
	at net.minecraft.launchwrapper.Launch.main(Launch.java:28) [launchwrapper-1.12.jar:?]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_181]
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_181]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_181]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_181]
	at org.multimc.onesix.OneSixLauncher.launchWithMainClass(OneSixLauncher.java:196) [NewLaunch.jar:?]
	at org.multimc.onesix.OneSixLauncher.launch(OneSixLauncher.java:231) [NewLaunch.jar:?]
	at org.multimc.EntryPoint.listen(EntryPoint.java:143) [NewLaunch.jar:?]
	at org.multimc.EntryPoint.main(EntryPoint.java:34) [NewLaunch.jar:?]

Mods have to learn that it's not just ItemArmor items that could go into the armor slots. It's actually pretty common for server plugins to put blocks on players' heads. In fact, a pumpkin block is not an ItemArmor and it can still be equipped. I wonder if that would also crash ArmorChroma.
This will have to be handled in the other mod. Backpacks won't become an ItemArmor. It is made like this to prevent many mods from treating is as such, breaking it in unexpected ways.

I see. Thanks for the quick reply. I will take it up with ArmorChroma.
Btw pumpkin blocks are not an issue. I guess it's because they don't provide armor, but good point nonetheless.

Thank you for bringing this to my attention. I have fixed the bug.


Progress, receveive only the 99%

Hello,
I'm using PHP-FFmpeg and it work great, but i'm trying to indicate the progression of the convertion to the user and I think it's not working.
When i try to view the progression of the conversion I only get the 99% value and i don't think this is normale.
Here is an exemple :
<?php

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Input\InputArgument;

class FfmpegCommand extends Command
{
    protected function configure()
    {
        $this
            ->setName('app:encode')
            ->setDescription('Encode file.')
            ->addArgument('filename', InputArgument::REQUIRED, 'The filename')
            ->setHelp("This command allows you to encode file...")
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $ffmpeg = FFMpeg\FFMpeg::create();
        $audio = $ffmpeg->open($input->getArgument('filename'));

        $format = new FFMpeg\Format\Audio\Flac();
        $format->on('progress', function ($audio, $format, $percentage) use ($output) {
            $output->writeln("$percentage % transcoded");
        });
        $audio->save($format, 'track.flac');
    }
}

When i execute :
php app.php app:encode fakear.flac
I only get :
99 % transcoded
And the file is working good but got only 99%
Thanks so much for your work !

@David-Crty Hello! Don't worry. php-fmpeg iterates from 0 to 99, not from 1 to 100.

add
echo json_encode($output , true);
$output->writeln("$percentage % transcoded");
after the line

Closing due to inactivity.

Hi, I have the same issue. Did you manage to solve it? 3 year later and I can find no information about the events or how to see when the conversion is complete.
Some information would be much appreciated, thank you.


bad code

This code is buggy and does not compile. Errors in example 1, variable hStdOut is declared as hStdout ..
The line:
if (!WriteConsoleW(hStdOut, sequence, ARRAYSIZE(sequence), &written, NULL))
does not compile (breaks on the macro ARRAYSIZE) .
error C2784: 'char (*RtlpNumberOf(__unaligned T (&)[N]))[N]': could not deduce template argument for '__unaligned T (&)[N]' from 'PCWSTR'
Did not test the other examples. Tested with Visual Studio 2019.
Edit: Here is my fixed version of example 1.
#include <windows.h>
#include <string>

void CLS()
{
	HANDLE hStdOut;

	hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);

	// Fetch existing console mode so we correctly add a flag and not turn off others
	DWORD mode = 0;
	if (!GetConsoleMode(hStdOut, &mode))
	{
		return;
	}

	// Hold original mode to restore on exit to be cooperative with other command-line apps.
	const DWORD originalMode = mode;
	mode |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;

	// Try to set the mode.
	if (!SetConsoleMode(hStdOut, mode))
	{
		return;
	}

	// Write the sequence for clearing the display.
	DWORD written = 0;
	std::wstring sequence = L"\x1b[2J";
	if (!WriteConsoleW(hStdOut, sequence.c_str(), sequence.length(), &written, NULL))
	{
		// If we fail, try to restore the mode on the way out.
		SetConsoleMode(hStdOut, originalMode);
		return;
	}

	// To also clear the scroll back, emit L"\x1b[3J" as well.
	// 2J only clears the visible window and 3J only clears the scroll back.

	// Restore the mode on the way out to be nice to other command-line applications.
	SetConsoleMode(hStdOut, originalMode);

}

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

ID: 98b65d94-0ab7-d7ab-c309-cb883e93381d
Version Independent ID: 09d1fad1-a5f6-22e2-2a78-9ffc970255ef
Content: Clearing the Screen - Windows Console
Content Source: docs/clearing-the-screen.md
Product: console
GitHub Login: @miniksa
Microsoft Alias: miniksa

You brought an end to hours of debugging for me. Thank you for posting this! Hopefully they will fix the example.

You brought an end to hours of debugging for me. Thank you for posting this! Hopefully they will fix the example.

You can potentially end another eon of misery by simply using this code ;-)
void CLS() { 	system("cls.exe"); }

I apologize here. @deleeuwict posted this original bug on the day after Christmas, and by the time I returned from vacation, it was lost in the flood of other holiday email in my inbox.
Now that you've brought it up again, @rickposp, I'll have it fixed this week.

Okay, I avoided using STL because it was supposed to be a C sample, not a C++ one. So I fixed the mismatched case on the hStdOut and I used wcslen for the length of the string as it should be available in the CRT.
I also double checked that the other two samples in the file compile correctly on x86 and amd64. They do.


user can customize dial timeout

I am using cfhttp in my application and it tries to connect to cloudfoundry cloud controller. Now I got the error below when deploy my application to bosh-lite. How about make user can set the value of dial timeout instead of hardcoded it to 5s?
{"data":{"API":"https://api.bosh-lite.com","error":"Get https://api.bosh-lite.com/v2/info: dial tcp: i/o timeout"},"log_level":2,"log_time":"2018-10-18T03:15:37Z","message":"scalingengine.failed to login cloud foundry","source":"scalingengine","timestamp":"1539832537.023622990"}

Thanks for filing this issue, @qibobo. The Diego team had a chance to review the public interface to the cfhttp package last week, and we plan to make some breaking changes to it that would allow this dial timeout to be configured. We expect to get to this work in a few weeks.
Best,
Eric, CF Diego PM

@qibobo the v2 package support the usage of multiple options in a more generic constructor, for example:
cfhttpv2.New(cfhttpv2.WithDialTimeout(10 * time.Second))

We would like to start moving users over to the v2 package as we think this is more maintainable and a better pattern to stick with
Let us know if you need any more help or clarification in this issue, I will close it for now


Clicking on the effect column in tracker mode seems to freeze GB Studio 4 Beta 1.

Describe the bug
In the current beta build of GB Studio 4, I've encountered a bug where entering tracker mode in the music editor and clicking on the effect column would freeze GB Studio until I close out of the program.
To Reproduce
Steps to reproduce the behavior:

Go to 'Music'
Click on any song
Enter tracker mode
Click on the effect column
See Bug

Expected behavior
Clicking on the effect column in tracker mode shouldn't freeze GB Studio.
Platform (please complete the following information):

OS: Windows 10 Home
App version: 4 Beta 1

Additional context
This issue was absent in previous versions of GB Studio.

Investigating, seems like the first change where this broke is 16d252d1704b62b82f5e033aecda7fbad52fc001

Thanks for reporting @Amixofwordscombinedtomakeausername ! I think I fixed the issue with https://github.com/chrismaltby/gb-studio/pull/1411, will ping you once it merges and can be tested

Thanks Pau!!


Support on Jetson?

hey @cyrusbehr can you guide me with running yolov8 inference with tensorrt cpp on jetson?.
I have a Jetson Orin NX 16 GB with following config:
Jetpack - 5.1.4
CUDA - 11.4.315
cuDNN - <IP_ADDRESS>
TensorRT - <IP_ADDRESS>
OpenCV - 4.6.0 with CUDA

Hi, the problem maybe is the TensorRT version you´re running. I had the same issues running the code since most of the code is written on the latest TensorRT version and specially when creating the .engine file. If I remember correctly, the main error when running it is on the engine.h file of the TensorRT CPP Library. And if I remember correctly is this line in particular:
else if (tensorDataType == nvinfer1::DataType::kFP8) {
                throw std::runtime_error("Error, model has unsupported output type");
            }

Since TensorRT received kFP8 support until 8.6.x, I suggest to you to update your Jetpack to 6.0. If you're having issues, I'm willing to create an edit of this code and maybe be able to make it work and also since I'm currently working in this library on my own for my own project but I really don't promise a lot to have it done in a week or so.


How can it work for Raspberry pi?

Sorry, I'm new using pytorch and tvm so maybe I understand things wrong...
As I know all models in results/tvm_compiled/ are compiled for Jetson TX2.
What do I need to run a compiled model in a Raspberry pi as in tx2_run_tvm.py ?
I guess I have to Compile the model using tvm as explained in this tutorial:
https://docs.tvm.ai/tutorials/frontend/from_pytorch.html
Can anyone confirm?

Hi @Oroelipas
You were correct in that you need to compile the model anew for a different hardware platform. As the models in this repository were compiled specifically for the TX2, they will not be compatible with the Raspberry Pi without re-compilation.

Oroelipas were you able to recompile for raspberry pi?

Hi @ideasxiang, currently I am working in other projects and have not made progress with this. Anyway I am listening if someone publish something about it

@Oroelipas thank you for your reply, I will try recompile using the link you provided. [https://docs.tvm.ai/tutorials/frontend/from_pytorch.html]

@ideasxiang Did you succeed to recompile the model for raspberry pi?


Add events so it's possible to listen to add strategies

Fixes https://github.com/M6Web/AmqpBundle/issues/35

:+1:


Output of cargo metadata may not be machine-readable if the registry needs to be updated

Repro steps:

rm Cargo.lock
cargo metadata

Observe that the JSON output is preceded by:
    Updating registry `https://github.com/rust-lang/crates.io-index`

A workaround is possible by running cargo metadata --quiet, but I can't really imagine that output being wanted in a command whose output is supposed to be machine-readable.
It's a pretty easy edge case to miss since usually your registry doesn't need to be updated. It bit me in SSheldon/rust-test-ios#3.

Oh and I'm running cargo 0.9.0-nightly (8fc3fd8 2016-02-29), apologies if this has since been fixed but I couldn't find anything by searching.

cc @matklad, I wonder if perhaps cargo metadata should imply a quiet shell by default? Either that or perhaps https://github.com/rust-lang/cargo/issues/1473 would fix this

but I can't really imagine that output being wanted in a command whose output is supposed to be machine-readable.

Actually we want that output in IntelliJ-Rust :) cargo metadata may take a while for downloading dependencies, so we use updating registry and downloading foo messages to show a progress bar.
We can make metadata quite by default, and use --verbose to override this.
However I would argue that quite by default is a bad behavior.  metadata will cause dependencies to be resolved and downloaded, and you really want to see the feedback. If cargo metada runs for 10 minutes, you want to know if you Internet is slow, or if you wi-fi died, or if maybe cargo is just stuck picking the versions.
#1473 seems like the correct solution here.
Another option is to better document current behavior and to make sure that something (Registry is up to date?) is always printed before the JSON.

Interesting! Sounds like #1473 is the right solution

closed by https://github.com/rust-lang/cargo/pull/2722

🎊

So this will hit stable Rust with 1.10? I need to remember to fix progress reporting on the IntelliJ side

Yeah that sounds right!


HELP firebase stop and restart

I use Esp32, I use bluetooth and wifi separately. That's why I stop wifi, connecting to bluetooth, I'm stopping bluetooth.
wifi connecting but firebase not working despite wifi connection
stop
Firebase.getInt(firebaseData,"s1");
if (firebaseData.intData() == 1) {
Firebase.setInt(firebaseData,"s1",2);
Firebase.end(firebaseData);
Firebase.reconnectWiFi(false);
WiFi.mode( WIFI_MODE_NULL );
restart
void onDisconnect(BLEServer* pServer) {
 BLEAdvertising *pAdvertising = pServer->getAdvertising();
    pAdvertising->stop();
    BLEDevice::getAdvertising()->stop();
  WiFi.mode( WIFI_MODE_NULL );

  delay(500);
  WiFi.begin(WIFI_SSID, WIFI_PASSWORD);

Serial.println("Bağlanır");
while (WiFi.status() != WL_CONNECTED)
{
Serial.print(".");
delay(300);
}
Serial.println(WIFI_SSID);
Serial.println(WiFi.localIP());
Firebase.begin(FIREBASE_HOST, FIREBASE_AUTH);
Firebase.reconnectWiFi(true);
delay(500);
}

Your problem is your WiFi configuration and not related to library.
You should set WiFi mode to WiFi_STA.

Sizin sorununuz, WiFi yapılandırmanızdır ve kitaplıkla ilgili değildir.
WiFi modunu WiFi_STA olarak ayarlamalısınız.

WiFi.mode (WIFI_STA);
I tried but it didn't work I guess I can't stop firebase because wifi connecting is getting ip

Firebase is not a problem.
It has the connection management inside which works normally.
Try Arduino examples about https request. If you can run example and sever connected, the library would work too.

At first, any third party integration conflict should be solved yourself.
If you cannot solve yourself, you should create the test example which minimum, clean (remove all third party libs) and clear to produce the error which I can test with no  further hardware requirement and post the error and the test code here.
If the problem is not related to the library after I've checked your example code, you should post your problem at the esp8266 forum.

All examples of library provide the code that always check the error in every function called which you never used it and never didn't provide any debug at any check point of your code.


Update disks-pools.md with additional regions

Updated regional availability of Disk pools

@adarshv98 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@adarshv98: I'm sorry - only the author of this article, @roygara, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Replaced CHECK_ by TORCH_CHECK_

Stack from ghstack:

-> #2582

CHECK_ were deprecated in upstream so we should replace them here as
well
Similar to https://github.com/pytorch/vision/pull/6322, relates to https://github.com/pytorch/pytorch/pull/82032
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

ROCm builds are failing since we are behind on ROCm builds on core pytorch nightlies

@seemethere has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


[skyr-url] Changed version number to 1.12.0

Describe the pull request

What does your PR fix? Fixes #

Updates version number of skyr-url

Which triplets are supported/not supported? Have you updated the CI baseline?

x64-osx, x64-windows, x64-linux

Does your PR follow the maintainer guide?

Yes

LGTM, thanks for the update!


Allow namespace files in MailSend activity

Feature description
I have a MailSend activity with an HTML content which includes images.
Currently, the MailSend activity does not have a namespaceFiles attribute. It expects an internal storage URI that I cannot provide unless I download the image into the Kestra's internal storage every time the flow is executed, creating copies of the same image.
I am currently using the img src as a base64 inline, and while this works this causes the webserver UI to crash due to the huge string which makes the flow difficult to work with.
As a side note it would be great if there was some more detailed documentation on using an html content with the embedded image option.

you can use all namespace files in all plugins already as long as you read the file's content rather than accessing it via path i.e. {{ read('your_file.png') }} should already work. Additionally, you can wrap your MailSend task in a WorkingDirectory task which has namespaceFiles.enabled option allowing you to then access Namespace Files via path too
if this info doesn't help, can you share your flow? (redact sensitive info)

Trying to get a workaround about this case. This flow almost works:
id: email
namespace: kestra.sandbox
tasks:
  - id: hello
    type: io.kestra.plugin.core.log.Log
    message: Hello World! 🚀

  - id: download
    type: io.kestra.plugin.core.namespace.DownloadFiles
    namespace: kestra.sandbox
    files:
      - "test.jpg"

  - id: send_email
    type: io.kestra.plugin.notifications.mail.MailSend
    from<EMAIL_ADDRESS>    to<EMAIL_ADDRESS>    username: resend
    password: "{{ secret('RESEND_API') }}"
    host: smtp.resend.com
    port: 465
    subject: "test"
    embeddedImages:
      - name: test.jpg
        uri:  '{{ outputs.download.files["/test.jpg"] }}'
        contentType: image/jpeg
    htmlTextContent: |
      "Failure alert for flow "

I've uploaded test.jpg into the NamespaceFiles.
Email is sent, but the image is corrupted

The issue is on namespace file, I tried to upload a PNG into namespace files and download it and the file is corrupted.
Moving the issue inside the Kestra namespace.

This is a UI issue as if the file is send via cURL it works.
Example:
curl -XPOST http://localhost:8080/api/v1/namespaces/kestra.sandbox/files?path=/logo-light-version.png -F<EMAIL_ADDRESS>


General project context/history questions

Hi there -- sorry if the issue queue is the wrong place for this:

Any synchronous chat space?
Is this repo in any way related to the prior Themis project that was exploring digital corporate constitutions, related to the defunct One-Click Org project?

https://circus-foundation.org/current-projects/themis-designing-emergent-democracy/
http://web.archive.org/web/20110413163732/http://www.oneclickorgs.com/2010/08/19/joi-ito-joins-one-click-orgs-advisory-board/

Thanks!

Thanks for the invite! Was making my way through the blog posts in advance of the event tomorrow, anyhow :)
(I'm a member of @hyphacoop who should be jumping on a call with y'all in January fyi)
Would it be helpful if I created a PR to add the Zulip reference to a section of the README? Also, if there's a canonical repo somewhere in the org, it could go there. no pressure if you'd just like it to stay here though
(A few orgs I've participated in keep an foo/overview repo for general stuff -- licensing, comms, 10,000ft context, etc: https://github.com/edgi-govdata-archiving or https://github.com/civictechto/)

Oh nice! I am hoping to attend your virtual holiday internet thingy on Sunday.
Thanks for the suggestion for an overview repo. I'll report that back internally.
PRs are welcome, tho I'll be sure to add the Zulip link to the readme once I sync up internally.

Close in light of https://github.com/informalsystems/themis-contract/pull/110


Mlops added

Added MLops

@ranjan2829 we do not have an MLOps roadmap at this time. You can add this community under the DevOps Roadmap


fix some valgrind memcheck warnings when N_proc < NN

PR description:
valgrind memcheck complained about uninitialised values in a few new places where we loop over NN.  This PR quiets them.  My working theory is that a loop over constant NN with an if on N_proc is more vectorizable than a loop over an unbounded N_proc.
PR validation:
Compiles, quiets valgrind, trivial technical fix.

after the prop to plane was merged I did a resync in our/trackreco fork.
So, this PR now needs an update to resolve the conflict in PropagationMPlex.cc


Why there is no support to Google AdMob RewardedInterstitial for iOS?

Why there is no support still for RewardedInterstitial for Xamarin.iOS?

Has it ever worked ?
I am trying to implement it now in a Xamarin.Forms app, and you are absolutely right, it does not work!


Xin: UX/UI Design: 2 weeks

Week 1
I Need Help With:
Nothing for this module yet.
What went well?
Learned about how to brainstorming in a group
What went less well?
Hard to visualize the ideas i had.
Lessons Learned
Process of brainstorming.
Sunday Prep Work
How to use figma
---- END CHECK-IN TEMPLATE ---->

@XinGITLou checked-1 is for the coaches when they check your issue.


IndexOutOfRangeException on parse from multiple threads

It's strange and happens only once. Rerun was not reproduced: maybe data race?
I launched parsing from multiple threads and got a lot of exceptions:
System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at System.Collections.Generic.Dictionary`2.set_Item(TKey key, TValue value)
   at CommandLine.Infrastructure.ReflectionCache.set_Item(Pair`2 key, Object value)
   at CommandLine.Infrastructure.ReflectionHelper.RetrievePropertyList[TAttribute](Object target)
   at CommandLine.Parsing.ValueMapper.InitializeValueOption()
   at CommandLine.Parsing.ValueMapper..ctor(Object target, CultureInfo parsingCulture)
   at CommandLine.Parser.DoParseArgumentsCore(String[] args, Object options)
   at CommandLine.Parser.DoParseArgumentsVerbs(String[] args, Object options, Object& verbInstance)
   at CommandLine.Parser.ParseArguments(String[] args, Object options, Action`2 onVerbCommand)
   at OracleScheme.Program.Main(String[] args) in d:\Projects\OracleScheme\OracleScheme\OracleScheme\Program.cs:line 20

version - <IP_ADDRESS>

@turbanoff, so @nemec helped right? I close the issue, but if not feel free to comment...


Windows and paths with n in them

Windows 10 2004.
Somewhere along the way, paths are getting stored in the history.txt file wrong. For instance, I work on a project called NuShell. Here's an entry from my history.txt file.
cd source\\repos\\forks\\\nushell\\
Notice that there are one too many back slashes in the entry. The ramification of this is that when I try to recall it, this happens.

Any ideas what is going on here? I'm assuming it has something to do with either the history loading or saving or both.

This is expected. Both backslash and newline are escaped when writing the history file. They are unescaped when loading it again. The history file has a single history entry per line, so newlines must be escaped as \n. This in turn requires \ to be escaped as \\ to prevent ambiguity when you write \n and not a newline.
https://github.com/kkawakam/rustyline/blob/380995919c0dc0565aa71b352400ed0e5f4a15d0/src/history.rs#L143-L144
https://github.com/kkawakam/rustyline/blob/380995919c0dc0565aa71b352400ed0e5f4a15d0/src/history.rs#L173

This is expected.

So, it's expected to not work properly with any path that has a folder that begins with n?
What about checking for \n at the end of a string instead of in the middle. Would that work?

Notice that there are one too many back slashes in the entry.

I didn't notice this part when I responded. This is not expected. I think the problem here is that \n got turned into \\n when written to the history file. So far so good, but when it got read again, the \n part first got turned into a newline to form \<LF>, this is not correct. When this is written again it forms \\\n. The solution would be to replace the two replace calls with a single pass that replaces "\\n" with "\n" and "\\\\" with "\\" at the same time.

Thanks you for both report and analysis.

See #412
There may be a more idiomatic way to lazily / conditionally make a partial copy (https://github.com/kkawakam/rustyline/pull/412/files#diff-fe6f8eb00378a6fd24f0b8c7f0cb9cebR194) ?

6.3.0 released


Trip Planner page UI

Title and Issue number
Title : Trip Planner page UI
Issue No. : #1054
Code Stack :
Close #1054
Description
Improved the UI of trip plan page to match website theme.
Made the page responsive too.
Added pop-up on submission of form.
Video/Screenshots (mandatory)
https://github.com/user-attachments/assets/22179d26-6e32-4c7c-a275-c5297838ac7f
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[X]  I have performed a self-review of my code.
[X]  I have commented my code, particularly in hard-to-understand areas.
[X]  I have tested the changes thoroughly before submitting this pull request.
[X]  I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X]  I have gone through the  contributing.md file before contributing

##Are you contributing under any Open-source programme?

[X]  I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

@apu52 Sir, please review this PR.
Thank You!

@TBorundia though I have merged the PR just want to mention u a bug that from next time

when u add any image please remove the alamy or copyright label


Managed Disks are not yet supported by k8s persistent volumes

hi folks,
the docs don't reference the fact that managed disks for persistent volumes are not yet supported by k8s, and in fact it looks like 1.8 will be the earliest it could be added (see https://github.com/kubernetes/kubernetes/pull/46360).
Please update the documents to note that fact.  I'm happy to attach a PR with warnings on every page that references managed disks that it is not yet supported for persistent volumes.
Thank you.

Thanks @skippy, we are working to get the disk PR into 1.8.  If you could submit a PR with the warnings that would be great.  We did add managed disks for master / agents because we have some customers who want encryption at rest, but not to use PVs.

@anhowe will do!  I hadn't realized #844 was so recent!  I am still trying to figure this out, but so far I haven't been able to create encrypted OS disks under k8s for masters or agents.  Do you have instructions that you can point me to?  I'm glad to update the docs to include this info if needed.

@anhowe just to be clear, I'm using the https://github.com/Azure/acs-engine/blob/master/examples/disks-managed/kubernetes-vmas.json examples; the disks are showing up in the UI as well as az disks list, but the encryption settings are null on the CLI and showing up as unencrypted on the UI (select agent or master vm -> select disks -> Encryption is 'Not enabled'
so perhaps I'm all set and managed disks is working as expected?

Correct, your managed disks have encryption at rest automatically.
The Disk team confirmed that Portal / CLI shows the status of ADE (https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption).  Portal and CLI do not show encryption at reset which is enabled automatically (https://azure.microsoft.com/en-us/blog/azure-managed-disks-sse/?v=17.23h).

@anhowe gotcha, and I really appreciate the input!!  I've been hitting a wall trying to figure that one out.
Ok... documentation.  I'm on it!

Managed disk support is now in Kubernetes v1.7.2 which should allow us to fix this.

This is solved in three parts:

[x] We've implemented managed disks on masters in https://github.com/Azure/acs-engine/issues/1027.
[x] Merge managed disks into upstream Kubernetes. This is available starting in Kubernetes v1.7.2.
[ ] We also need to implement managed disks on the nodes/agents themselves (for their own filesystems)

This has been release #1072


Mouse input accumulation

Objective

Add the AccumulatedMouseMotion and AccumulatedMouseScroll resources to make it simpler to track mouse motion/scroll changes
Closes #13915

Solution

Created two resources, AccumulatedMouseMotion and AccumulatedMouseScroll, and a method that tracks the MouseMotion and MouseWheel events and accumulates their deltas every frame.
Also modified the mouse input example to show how to use the resources.

Testing

Tested the changes by modifying an existing example to use the newly added resources, and moving/scrolling my trackpad around a ton.

It would be great if the mouse_input or mouse_input_events examples were updated to show these off

I already updated the mouse_input example, but I don't know if this change would really fit in the mouse_input_events example as we only really implemented a Resource, not an Event

It would be great if the mouse_input or mouse_input_events examples were updated to show these off

I already updated the mouse_input example
Whoops, don't know how I missed that 😅 . LGTM


Modify bat file to include required varaibles [Do not merge]

TSL's windows job currently fails, this appears to be due to a slight difference in internal CI setup.  Begin adding missing files.

Closing in favor of copybara making fix


model extends record

Cadl Ranch Contribution Checklist:

[x] I have written a scenario spec
[x] I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
[x] I have written a mock API
[x] I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

note also extends vs is is different, you probably still want is. Extends will say i am inheriting a Record, is will just say I am a Record<string>
See the differently generated openapi
https://cadlplayground.z22.web.core.windows.net/?c=bW9kZWwgTcQGUmVjb3JkVW5rbm93biBleHRlbmRzIMYWPHXGFz4gewogIG5hbWU6IHN0cmluZzsKfQoK2EYyIGnfQsZC

On swagger output of
model ModelRecordUnknown2 is Record<unknown> {
  name: string;
}

Does not have
    ModelRecordUnknown2:
      type: object
      additionalProperties: {}

It does not have anything about "name"?

note also extends vs is is different, you probably still want is. Extends will say i am inheriting a Record, is will just say I am a Record<string>
See the differently generated openapi https://cadlplayground.z22.web.core.windows.net/?c=bW9kZWwgTcQGUmVjb3JkVW5rbm93biBleHRlbmRzIMYWPHXGFz4gewogIG5hbWU6IHN0cmluZzsKfQoK2EYyIGnfQsZC
I don't know how that affect generated client however but I could imagine something like that

class WithExtends extends Dictionary<string, string> {

  public Name {get; set;}
}

class WithIs implements IDictionary<string, string> {
  ... need to imploement dictionary interface
  public Name {get; set;}
}

@timotheeguerin Like weidong said, we shall encourage users extends instead of is otherwise users can't get expected output about

I would not put this in inheritance. TSP uses "extends", but it's the concept of "additional properties". Inheritance is misleading here.
I need a GET as well

I add new folder record under type/model

I prefer to add it under to type/property named with additional-properties. @timotheeguerin I have a question for it. From TSP type relations, it seems custom model with properties are derived from Record. It that mean all custom models are already support additional properties? What's the difference for an explicit Model extends Record<unknown>?

Hhm yeah seems like we have a bug in autorest and openapi3 emitter, those properties should not be dropped. I think the idea was there though that with is you have additionalProperties on the schema itself. Vs with extends it use an allOf.
I think the takeaway is mostly both can happen and I think we should recommend people to use the is variant. If you generate the same thing then maybe its ok

.

there is a discussion on Teams about that
short answer, no models do not have additional properties in the TypeSpec system(I made a wrong statement before), we might need to update the type graph a little bit
model Foo { } !== model Foo is Record<unknown> {}

@msyyc Yuchao Yan FTE I prefer to add it under to type/property named with additional-properties. @timotheeguerin Timothee Guerin FTE I have a question for Record<unknown>. From TSP type relations, it seems custom model with properties are derived from Record. It that mean all custom models are already support additional properties? What's the difference for an explicit Model extends Record<unknown>?

@tadelesh Make sense. I will move it to type/property/additional-properties

@timotheeguerin @lmazuel @tadelesh @weidongxu-microsoft
Updated:
I understand that language emitter shall have same behavior for extends Record<> or is Record<>, I add test for both scenario.


Incorrect parse error for async methods with keywords as names

When parsing an async method in an object that has a keyword as a name, acorn incorrectly reports  a parsing error. This only seems to apply when parsing object methods -- class methods are parsed correctly.
Using acorn 5.1.1:
require('acorn').parse('({ async delete() {} })', { ecmaVersion: 8 });

Expected AST
{
  "type": "Program",
  "start": 0,
  "end": 20,
  "body": [
    {
      "type": "ExpressionStatement",
      "start": 0,
      "end": 20,
      "expression": {
        "type": "ObjectExpression",
        "start": 1,
        "end": 19,
        "properties": [
          {
            "type": "Property",
            "start": 2,
            "end": 18,
            "method": true,
            "shorthand": false,
            "computed": false,
            "key": {
              "type": "Identifier",
              "start": 8,
              "end": 14,
              "name": "delete"
            },
            "kind": "init",
            "value": {
              "type": "FunctionExpression",
              "start": 14,
              "end": 18,
              "id": null,
              "generator": false,
              "expression": false,
              "async": true,
              "params": [],
              "body": {
                "type": "BlockStatement",
                "start": 16,
                "end": 18,
                "body": []
              }
            }
          }
        ]
      }
    }
  ],
  "sourceType": "script"
}

Actual result
SyntaxError: Unexpected token (1:9)

This was originally reported in https://github.com/eslint/eslint/issues/9153.

Thanks for fixing!

@marijnh Would it be possible to get a new release including this fix? We have issues that we want to resolve with dependencies, but this fix is blocking us.

I've tagged 5.1.2

Thank you!


Congratulations!

I just want to thank you for this job.
I work with CSS and I know how hard it can be to create something so beautiful.
Congratulations!

Thanks for the kind words. It means a lot.


no crc error when volume data is corrupted intentionally for fuse mount

Describe the bug
We want to verify for the e2e data integrity checking for file read, here is the steps:

Create a new seaweed with fuse mount (or create a fuse mount with new collection)
Write a file to the fuse mount (collection user1)

mkdir /opt/udm/fs1/test01

dd if=/dev/mapper/rl-root of=/opt/udm/fs1/test01/1.dat bs=1MB count=4096 skip=1003
4096+0 records in
4096+0 records out
4096000000 bytes (4.1 GB, 3.8 GiB) copied, 7.40632 s, 553 MB/s

Edit the volume dat file, which is to simulate a data corruption

[root@~]# ls -lh /mnt/ssd_test/user1*dat
-rw-r--r-- 1 root root 315M Sep 12 22:00 /mnt/ssd_test/user1_10.dat
-rw-r--r-- 1 root root 312M Sep 12 22:00 /mnt/ssd_test/user1_11.dat
-rw-r--r-- 1 root root 251M Sep 12 22:00 /mnt/ssd_test/user1_9.dat
[root@~]# ls -lh /mnt/weed_ssd/user1*dat
-rw-r--r-- 1 root root 332M Sep 12 22:00 /mnt/weed_ssd/user1_12.dat
-rw-r--r-- 1 root root 284M Sep 12 22:00 /mnt/weed_ssd/user1_13.dat
-rw-r--r-- 1 root root 299M Sep 12 22:00 /mnt/weed_ssd/user1_14.dat
-rw-r--r-- 1 root root 338M Sep 12 22:00 /mnt/weed_ssd/user1_8.dat

Just pick one dat file and make some change to it.

Drop the cache

echo 3 > /proc/sys/vm/drop_caches

Copy the file to another place

cp /opt/udm/fs1/test01/1.dat /mnt/weed1/

I would expect the copy should fail here, but it will succeed.

System Setup

List the command line to start "weed master", "weed volume", "weed filer", "weed s3", "weed mount".

 /usr/bin/weed -logtostderr=true master -mdir=/data -ip=xxx -volumePreallocate=false -volumeSizeLimitMB=4096 -defaultReplication=000
/usr/bin/weed -logtostderr=true volume -dir=/data -dir=/mnt/weed1 -mserver=localhost:9333 -max=0 -port=8080 -metricsPort=9325 -preStopSeconds=1
/usr/bin/weed -logtostderr=true volume -dir=/data -dir=/mnt/weed2 -mserver=localhost:9333 -max=0 -port=8081 -metricsPort=9326 -preStopSeconds=1
/usr/bin/weed -logdir=/var/log/udm filer -defaultReplicaPlacement=000 -master=localhost:9333 -metricsPort=9328 -maxMB=128
weed -v=4 mount -filer=localhost:8888 -dir=/opt/udm/fs1 -cacheCapacityMB=0 -concurrentWriters=8 -filer.path=/opt/udm/fs1 -collection=user1 -chunkSizeLimitMB=8

OS version
Rocky Linux release 9.4
output of weed version
version 30GB 3.72 linux amd64
if using filer, show the content of filer.toml

Expected behavior
The file read should fail with crc check error
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Please create some scripts and/or docker compose files to make a case reproducible locally.

hi @half-life666, which version are you using? There were several parches about week ago which should trigger error (at least for consequent reads). See #5980 and #5981

3.72

Try just released 3.73. The relevant changes modify volume server behavior.

Try just released 3.73. The relevant changes modify volume server behavior.

OK, let me try 3.73 later

I tried 3.73 today, and I can still recreate this issue (with a fix in pr 6021).
It's very simple to recreate this issue. In the newly created fuse mount, you just write the first file, and then edit the volume dat file to corrupt the data, for example, below is the volume dat files after I create the first 4GB file:
[root@udm deploy]# ls -lh /mnt/weed_ssd/user3*dat
-rw-r--r-- 1 root root 337M Sep 14 19:50 /mnt/weed_ssd/user3_75.dat
-rw-r--r-- 1 root root 255M Sep 14 19:50 /mnt/weed_ssd/user3_76.dat
-rw-r--r-- 1 root root 421M Sep 14 19:50 /mnt/weed_ssd/user3_78.dat
-rw-r--r-- 1 root root 423M Sep 14 19:50 /mnt/weed_ssd/user3_79.dat
-rw-r--r-- 1 root root 359M Sep 14 19:50 /mnt/weed_ssd/user3_80.dat
[root@udm deploy]# ls -lh /mnt/ssd_test/user3*dat
-rw-r--r-- 1 root root 385M Sep 14 19:50 /mnt/ssd_test/user3_77.dat
-rw-r--r-- 1 root root 276M Sep 14 19:50 /mnt/ssd_test/user3_81.dat

I just do:
echo "1234" >> /mnt/weed_ssd/user3_78.dat
echo "1234" >> /mnt/weed_ssd/user3_79.dat

then drop the caches by echo 3 > /proc/sys/vm/drop_caches, and copy the file from fuse mount to another place:
cp /opt/udm/fs3/test01/1.dat /mnt/weed1/file2.dat

this should fail with CRC errors but it can succeed with no problems.
I think this is very serious issues, even though the probability of data corruption like this is not likely to happen, but when that happens, it should detect the data corruption and fail with very clear errors.
(note that I create the setup with replica 000, so no volume level fix is applied)

You probably checked it, but could you verify that the content of the file is incorrect after copying? The reason I ask: the dat file have some fillers if you edit them you are not corrupting the data itself (I was bitten by this once).

One more thing: make sure that you restart volume server and mounting process. They both can have some caches.
Then if I attempt to copy corrupted file from the mount, I see (after about 23 seconds or retrieval attempts)
> cp mnt/t.txt t.txt
cp: error reading 'mnt/t.txt': Input/output error
> echo $?
1

I.e. as expected the copy failed, and properly reported it.
At the same"weed mount" complains in the log
I0914 14:34:05.077353 http_global_client_util.go:471 retry reading in 5.0625s
E0914 14:34:10.140913 reader_at.go:159 fetching chunk &{FileId:138,091b463941 OffsetInChunk:0 ViewSize:531 ViewOffset:0 ChunkSize:531 CipherKey:[] IsGzipped:false ModifiedTsNs:1726338548682844412}: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error
E0914 14:34:10.140979 filehandle_read.go:61 file handle read /t.txt: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error
W0914 14:34:10.141001 weedfs_file_read.go:51 file handle read /t.txt 0: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error

and eventually quitting.

One more thing: make sure that you restart volume server and mounting process. They both can have some caches. Tested with v3.73, no encryption
Then if I attempt to copy corrupted file from the mount, I see (after about 23 seconds or retrieval attempts)
> cp mnt/t.txt t.txt
cp: error reading 'mnt/t.txt': Input/output error
> echo $?
1

I.e. as expected the copy failed, and properly reported it.
At the same"weed mount" complains in the log
I0914 14:34:05.077353 http_global_client_util.go:471 retry reading in 5.0625s
E0914 14:34:10.140913 reader_at.go:159 fetching chunk &{FileId:138,091b463941 OffsetInChunk:0 ViewSize:531 ViewOffset:0 ChunkSize:531 CipherKey:[] IsGzipped:false ModifiedTsNs:1726338548682844412}: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error
E0914 14:34:10.140979 filehandle_read.go:61 file handle read /t.txt: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error
W0914 14:34:10.141001 weedfs_file_read.go:51 file handle read /t.txt 0: http://redacted:18081/138,091b463941?readDeleted=true: 500 Internal Server Error

and eventually quitting.

Thanks for the reply. I tried to write a very small binary file, and then I can easily corrupt some bits in the volume file, and then I restarted volume server, filer, mount, the copy will fail, and in volume server I can see the data corrupt logs:
I0915 14:30:22.113448 volume_server_handlers_read.go:165 read /123,05ff8664c39048 isNormalVolume true error: CRC error! Data On Disk Corrupted

One question here is, is there a simple way to identify where the needle data is located in the volume file from the chunk info? We want to make this data corruption case as a error injection case to our test process, but it's very uneasy for a tester to perform.

Another thing is, I think the fuse mount log should contain detail error message, instead of just see 500 Internal Server Error

One question here is, is there a simple way to identify where the needle data is located in the volume file from the chunk info? We want to make this data corruption case as a error injection case to our test process, but it's very uneasy for a tester to perform.

It is an excellent idea to add a test like this. So far I personally went a manual road:
I generated long enough random file. This makes it is not compressible and thus you can see its content byte to byte in the volume .dat file. Then I modify some symbols. This works if encryption is disabled.
The other way may be by modifying  approach of
https://github.com/seaweedfs/seaweedfs/blob/17b9ce89ee9368d88ec1c64dd6d9014fd49e17b6/weed/storage/needle/needle_write_test.go


Resize view when keyboard appears to prevent hiding text

The view now updates its insets when the keyboard appears/disappears so that you can see the whole file.

Hello! Thank you for your contribution. You're putting several changes into one PR, which makes it hard to review. I don't think the text view ought to resize when the keyboard appears – Apple's own UITextView doesn't. As for the extra lexer, that is most welcome!

Sorry for the multiple changes in a single PR.
Apple UITextView doesn’t resize itself when you place it in a full screen view in your app, but if a user try to write more than ~20 lines in that textview, the text will be hidden by the keyboard and the last lines will not be visible until the user dismiss the keyboard. The same problem was happening with the SyntaxTextView.

Right, and I expect folks to solve this the same way it's solved in UITextView: they adjust the bounds by hand.

Okay, that’s a good idea. I'll close this pull request and probably make one only for the Shell Lexer.
I also created a sample project on a new branch, that showcase Sourceful in a very simple SwiftUI project ( https://github.com/alessionossa/Sourceful/tree/sample ). Do you think it can be useful to add to this repo?

I think a sample project would be most welcome. Is it possible to add the same thing in UIKit?


NIFI-13587 remove RestrictedSSLContextService interface in favor of t…

…he identical SSLContextService
       rename StandardRestrictedSSLContextService to more concise name RestrictedSSLContextService

Summary
This PR may be somewhat controversial since it renames a heavily used controller service which is clearly a breaking change. However, as we near the release of 2.0.0, now is an appropriate time to make such a change.
It has always seemed odd that there is a service named StandardRestrictedSSLContextService with both "standard" and "restricted" in its name. Historically, there is logic to this naming. There is an interface, RestrictedSSLContextService, which at one time was distinctly different from the SSLContextService interface. However, the two are now identical.
This PR proposes to consolidate naming conventions. The RestrictedSSLContextService interface has been removed. All references to it have been updated to SSLContextService. Removing the interface also allows the implementation name to become RestrictedSSLContextService rather than StandardRestrictedSSLContextService.
There are no functional changes here. Rather this only updates the naming of interfaces and corresponding implementations.
NIFI-13587
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

This is not workable.  The disruption this causes is not met by gain.
I am not fond of the current naming either but the breakage of basically every flow isn't great.  We've had some cases of breakage that are specific to certain components and that creates enough challenge.
This one is just too much.
Assuming there is agreement on what the names should be the next question is how best to transition people. This needs to be automated to be worthwhile.

I understand (and expected) the argument against this. Would a more appropriate approach? Rename the interface (internal, non-user facing change) and create a new controller service "RestrictedSSLContextService" while marking the current StandardRestrictedSSLContextService as deprecated - yet keep it fully functional.
This approach does beg the question of when then would it be appropriate to remove the deprecated service - only when we get to a future 3.0?

I haven't looked into the codebase fully myself.
If we have something now on the 2.x line which is deprecated then yeah we gotta get rid of that.
If you're saying we want to deprecate something that isn't now then that means deprecate on 1.x and remove/change on 2.x.  We've done that pattern with a number of things.  But for something like this we really want to make migration as automatic as possible.
I have a couple ideas on how to do that but dont want to blurt them out as I've not thought deeply on this.  But if this change is happening - we need options explored.

I'll cancel this PR and give it some more thought. There's the possibility of introducing a migration script to be run on flows when upgrading to 2.x which would translate the implementation name. That's at least half the problem solved, but still leaves an issue for saved flows in Registry or flow definition files. Perhaps there's a way to internally handle this translation that happens - as you say - automatically. Open to thoughts, if you have some already. If so, feel free to update the Jira issue.


Add test and some config files.

I just moved this test https://github.com/stealjs/steal/blob/master/test/test.js#L209-L211 from steal; test is running in local; I'll all the Gruntfile to run it from the command line.

Cool, let's get CI set up as well, i can help you with that.

@matthewp wohooo, we have builds now! :) https://travis-ci.org/stealjs/steal-css/builds/64957854

sweet, merge it in.


[CHANGE ME] Re-generated to pick up changes from self.

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/53fd5751-94d8-4b88-87bd-66c5c5c0b675/targets

[ ] To automatically regenerate this PR, check this box.

this is incorrect


Friendlier direct node iteration

Perhaps I'm using it wrong or couldn't find the idiomatic way to iterate nodes directly (I see there's a mutable version, node_weights_mut but perhaps an oversight there's no node_weights iterator?), but I really dislike this pattern:
for id in cfg.node_indices() {
   if let Some(&thing) = cfg.node_weight(id) {

In panopticon's graph algos I explicitly added direct mutable/immutale label/weight iterators because that pattern is so common and so unpleasant (imho).
E.g.:
pub type VertexLabelIterator<'a, K, V> = std::collections::hash_map::Values<'a, K, V>;
pub type VertexLabelMutIterator<'a, K, V> = std::collections::hash_map::ValuesMut<'a, K, V>;

impl<'a, V: 'a, E> VertexListGraph<'a, V, E> for AdjacencyList<V, E> {
    type Vertices = std::iter::Map<std::collections::hash_map::Keys<'a, Self::Vertex, V>, fn(&Self::Vertex) -> Self::Vertex>;
    type VertexLabels = std::collections::hash_map::Values<'a, Self::Vertex, V>;

    fn num_vertices(&self) -> usize {
        return self.vertex_labels.len();
    }

    fn vertices(&'a self) -> Self::Vertices {
        return self.vertex_labels.keys().map(std::clone::Clone::clone);
    }

    fn vertex_labels(&'a self) -> Self::VertexLabels {
        self.vertex_labels.values()
    }
}

This was easier since it just used hashmaps as backings and I could typedef their iterators, and then reexport them (since they themselves were then used in further iterator definitions).
I don't know how it would work for your library (haven't looked at internals), but I'm hoping can get both node_weight and edge_weight non-mut iterators.

Yes I have missed them too. It would be good to add.

Oh, friendlier node iteration already exists, it's hidden away in the IntoNodeReferences graph trait.
Import the trait, and graph has the node_references method which yields the iterator element (NodeIndex, &N)

I see. I wonder if it will be good to have a trait/graph native method chart to illustrate the api surface/differences?

A table sounds great, or any other illustration we can think of (anything that renders in rustdoc is easier to publish, but we use illustrations already, so that's fine too.)
Maybe to be really fun we'd make a dot illustration of the methods, but that's probably a lot more work for the same reward :smile:
I think we can close this, since .node_references() -> Iterator<Item=(NodeIndex, &NodeWeight) exists?

Yes definitely can close. :+1:

Super rust noob here. I also had the same problem.
Maybe I'm wrong, but isn't this a very basic operation/use case. I had to research this for half an hour.
Is there a reason why graphs don't simply have a nodes_iter() method or something like that?

@fiws Please use https://docs.rs/petgraph/0.4.12/petgraph/visit/trait.IntoNodeReferences.html and report new issues if there are bugs

I got that, I was just pointing out my ergonomic issue here.
Is there a reason why a feature like this is hidden in some trait without documentation?
Is the usecase uncommon? Or do most people know to search for a trait like that?

@fiws It would be cool if you could contribute the doc improvements that you imagine. There's no better person for this than you, that knows what was hard to find.  The answer in an open source project is always that nobody got around to making it better.


Security fix #41 : Stored XSS

Security fix: Cookie hijacking through Create User function

Fixed #41


[Feature] PhpStorm integration

It would be nice, if IDE can recognize contracts and help developers to work with them natively.

Implemented in https://github.com/goaop/idea-plugin/issues/5


org-contacts has disappeared

Did an update today with emacs 28.1 and org-mode 9.5.4 using this recipe and the develop branch of straight.el
(straight-use-package
 '(org-plus-contrib
   :type git
   :repo "https://code.orgmode.org/bzg/org-mode.git"
   :local-repo "org"
   ;; `org-version' depends on repository tags
   :depth full
   :pre-build ("make" "autoloads")
   ;; Org's make autoloads generates org-version.el
   :build (:not autoloads)
   :files ( :defaults
            "lisp/*.el"
            ("etc/styles/" "etc/styles/*")
            "contrib/lisp/*.el")
   :includes org))

My start-up gave me the error that 'org-contacts could not be found. It is not in the org-plus-contrib directory, but as far as I can see from the change log at orgmode.org they have not dropped it. Does anyone have any ideas?
And can I have the current recommended way to hand the split of org, org-contrib, and org-plus-contrib? Trying to piece it together from old issues leaves me a little uncertain if what I am doing is the most future proof way, or just something that luckily works for now.
One solution for the 'org-contacts issue is to grab an old version of the file and manually move it into my plus-contrib directory, but that is obviously just a band aid
Thank you,

Looks like orgmode has removed several files to their own repositories: https://git.savannah.gnu.org/cgit/emacs/nongnu.git/commit/?h=elpa/org-contrib&id=3dd98415225823f21fb7e12f272c0f199ac7407f
This will require rewriting recipes for these with their unique url found in each file.
I am getting errors using org-contacts, but this does get the file installed to my computer.
(use-package org-contacts
  :straight (org-contacts
	     :repo "https://repo.or.cz/org-contacts.git"
	     :local-repo "org-contacts")
  :config (setq org-contacts-files '("~/org/apptTodos/contacts.org")))

Is this the correct way to be doing things?

Yes. That looks correct.
The goal of org-contrib is for all of those packages to eventually be independently maintained and hosted by interested parties. I will update our org-contrib recipe to reflect the packages removed (though it doesn't look like you're relying on straight's provided recipe anyhow, so this should not have any effect on your set-up).

Sounds like the expected way to use org-contacts now is to update MELPA and run from there: https://melpa.org/#/org-contacts


Fix 17469 - View source code for object and dmd is broken

Due to 17131 a DUB dependency upgrade was necessary. I discovered that the assert-rewrite pipeline doesn't like empty files like the ones generated from the publictests target. A proper solution is to make the assert_writeln pipeline more selective, but the first commit should is a simple solution for now.

Oh I didn't realize that I already submitted a fix for the "ddmd" ones: https://github.com/dlang/dlang.org/pull/1601

(rebased to master)


Feature Question "extend Fzflua usefulness to the feature for vim command"

Info

Operating System: ubuntu 2204
Shell: zsh
Terminal: wsl2 on windows10
nvim --version: 0.8.1
fzf --version: 0.35.1

[ ] The issue is reproducible with mini.sh

fzf-lua configuration

require('fzf-lua').setup({
})

Description
Will you let me ask you the question like theme?
If not accepted, I'm ok for question to be closed immediately.
I'm in love fzf-lua.
Everyone here may be enjoying using fzf-lua for vim command.
I mean vim command is ":vimgrep...", ":Fzflua xxx..."
Especially I enjoy using " :Fzflua commands "  https://github.com/ibhagwan/fzf-lua#commands
Anyone can call command's argument in fzf-lua ui?
For example,

vim command " : Fzflua commands "
select command in fzf-lua ui, like " Fzflua " again
But, we can call only command but not its argument, right?
If we can also add argument into ":Fzflua commands" ui,  it's so convenient for us, I think.

Thanks for this place.
update1
add image

You can call arguments in two different ways:
:FzfLua grep search=foo
:FzfLua files cwd=~ previewer=false

Or lua
:lua require'fzf-lua'.grep( { search = 'foo' })
:lua require'fzf-lua'.files( { cwd = '~', previewer = 'false` })

@ibhagwan
Thanks.
Do you mean that we can add not command itself but argument into ":Fzflua commands" option?
like this images...

I’m not really sure what command you want to these?
Iyou’d like the query to be input into the interface useFzfLua commands query=Plug

Reading again your original post I think query will be what you want, it works by pre-filling the fzf query field, this can also filter arguments as it works on the entire line, this works in every provider commands, keymaps etc:

:FzfLua commands query=FzfLua
:FzfLua keymaps query=space

Or Lua
:lua require'fzf-lua'.commands( { query='Fzf'})
:lua require'fzf-lua'.keymaps( { query='space'})

If you’re referring to filtering based on the arguments in the preview window of the command, there is no such option yet.

""" If you’re referring to filtering based on the arguments in the preview window of the command, there is no such option yet.
Mayby this is what I wanted.
But query you told me is also good for me.
And I'm impressed that you reply soon.
Thanks for this issue, I'll ask again some day if you accept.

Thanks for this issue, I'll ask again some day if you accept.

If the solution was more straight forward it would be worth it, right now I’m not sure, the command detail table doesn’t necessarily correspond with arguments so I’m not sure what would be the benefit here and the keymaps can already be filtered using query.


Right to Left Languages (like Arabic).

Words in (RTL Languages) are improperly displayed on the Canvas).

Duplicate from:

https://github.com/niklasvh/html2canvas/issues/615
https://github.com/niklasvh/html2canvas/issues/422
https://github.com/niklasvh/html2canvas/issues/405
https://github.com/niklasvh/html2canvas/issues/289
https://github.com/niklasvh/html2canvas/issues/226

In NodeParser.prototype.paintText = function(container) {
I commented the !hasUnicode, so the problem was solved and the arabic characters appear as continous.
the main problem in arabic was that, textList was set as splitted characters instead of splitted words.
var textList = (!this.options.letterRendering || noLetterSpacing(container))
//&& !hasUnicode(container.node.data)
? getWords(characters)
: characters.map(function (character) {
return punycode.ucs2.encode([character]);
});

I had the same problem on 0.5.0-alpha1 which was solved by replacing !hasUnicode by hasUnicode .
Thanks.

@oorabi thanks, fixed the problem by removing "!hasUnicode" on line 2321, html2canvas 0.5.0-alpha1

Just to share some of my findings.
As oorabi already mentioned the problem with Arabic (and some other languages) is that they have logical (char-by-char) presentation and visual presentation (where following chars may effect to the presentation of previous chars). This issue can be solved by removing the check !hasUnicode, since that forced anything that contains non ASCII chars to be handled char by char.
But that is not the whole problem, since they do mix ASCII with the Arabic, for example colons, brackets etc. But those should be presented at the left side of the word that they are following. In order to do this we need to split the ASCII chars to own words. I did this by checking if the script category changes (i didn't care about others than Arabic for now).
	var previousScriptCategory = null;

    while(characters.length) {
		if(previousScriptCategory == null) {
			previousScriptCategory = getUtf8ScriptCategory(characters[i]);
		}
        if (isWordBoundary(characters[i]) === onWordBoundary || getUtf8ScriptCategory(characters[i]) !== previousScriptCategory) {
            word = characters.splice(0, i);
            if (word.length) {
                words.push(window.html2canvas.punycode.ucs2.encode(word));
            }
            onWordBoundary =! onWordBoundary;
            i = 0;
			previousScriptCategory = null;
        } else {
            i++;
        }
        ...

function getUtf8ScriptCategory(char) {
	if(char >= 0x0600 && char <= 0x06ff) {
		return "arabic";
	} else {
		return "default"
	}
}

But it doesn't end there. We still have problem of brackets, sure the previous splits them to separate words, but we are still rendering from right to left so they get rendered the wrong way. So my take was to make another hack for them.
	//If right-to-left, we should swap brackets
	for(var j = 0; j < words.length; j++) {
		for(var k = 0; k < words[j].length; k++) {
			if(words[j][k] == "(" && isRtl(words[j+1])) words[j] = words[j].replaceAt(k,")")
			else if(words[j][k] == ")" && isRtl(words[j-1])) words[j] = words[j].replaceAt(k,"(")
			else if(words[j][k] == "[" && isRtl(words[j+1])) words[j] = words[j].replaceAt(k,"]")
			else if(words[j][k] == "]" && isRtl(words[j-1])) words[j] = words[j].replaceAt(k,"[")
			else if(words[j][k] == "{" && isRtl(words[j+1])) words[j] = words[j].replaceAt(k,"}")
			else if(words[j][k] == "}" && isRtl(words[j-1])) words[j] = words[j].replaceAt(k,"{")
		}
	}
    return words;

function isRtl(string) {
	if((/[\u0600-\u06ff]/).test(string)) { //Arabic
		return true;
	}
	return false;
}

This is really a hacky way around the issue, but I hope someone can propose more proper way to handle these issues.

+1 Same issue here. Please fix this

n-a-g-r-o-m's solutions is very good, but I had to make some modifications to work on my project.
// If right-to-left, we should swap brackets for(var j = 0; j < words.length; j++) {         for(var k = 0; k < words[j].length; k++) {             if	   (words[j][k] == "(" /*&& isRtl(words[j+1])*/) words[j] = replaceAtIndex(words[j],k,")")             else if(words[j][k] == ")" /*&& isRtl(words[j-1])*/) words[j] = replaceAtIndex(words[j],k,"(")             else if(words[j][k] == "[" /*&& isRtl(words[j+1])*/) words[j] = replaceAtIndex(words[j],k,"]")             else if(words[j][k] == "]" /*&& isRtl(words[j-1])*/) words[j] = replaceAtIndex(words[j],k,"[")             else if(words[j][k] == "{" /*&& isRtl(words[j+1])*/) words[j] = replaceAtIndex(words[j],k,"}")             else if(words[j][k] == "}" /*&& isRtl(words[j-1])*/) words[j] = replaceAtIndex(words[j],k,"{")         }     }
function replaceAtIndex(s, index, char) { 	s = s.substr(0, index) + char + s.substr(index + 1); 	return s; }

@hadihashem Sorry I drop using this long time ago, so unfortunately I can't help.

same problem with me Arabic language :( any help please

@hadihashem Do you have a solution ?

@oorabi did you find solution ?

I have a text overlapping problem when Arabic content is set to RTL. I notice this issue has been opened for 4 years. Any update on this?

@EdiWang did you find any solution?
all the solutions I found are for previous versions which have problems in other ways :confused:

@abdelaziz321 unfortunately, no. Currently, I have to let my users for Arabic content use screenshot extension like FireShot to work around the issue, this is very awkward.

I really don't know why this worked, but It did when using these dependencies together:
"dependencies": {
    "html2canvas": "^1.0.0-alpha.12",
    "js-html2pdf": "^1.1.4",
    "jspdf": "^1.5.3"
}

and in one of my components:
<template>
  <v-card>
    <v-toolbar dark color="accent" dense flat>
      <v-toolbar-title class="white--text">طباعة الكارنيه</v-toolbar-title>
    </v-toolbar>

    <v-card-text class="pa-4">

          <img<EMAIL_ADDRESS>alt="front card" style="width: 100%">

{{ student.name }}

          <barcode
            class="student-barcode"
            :value="student.barcode"
            format="CODE128"
          >خطأ في الباركود</barcode>

        <img<EMAIL_ADDRESS>alt="back card" style="width: 100%">

    </v-card-text>

    <v-card-actions style="background-color: #f7f7f7">
      <v-spacer></v-spacer>
      <v-btn color="accent" class="white--text" elevation="1" @click.native="download(student)">تحميل pdf</v-btn>
    </v-card-actions>
  </v-card>
</template>

and here is the result
hope it works for you @EdiWang

I fixed this issue simply by changing the css root direction * {direction: rtl}

@MohammedYoussefSoliman can you please give more details!


Use dark keyboard when a dark theme is active

The app does this in all toolbar views and the sub header currently, where else should this be implemented?

I audited our views and changed it to check theme lightness, not if the theme == .LIGHT