[Hub Generated] Review request for Search to add version preview/1.0

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

With these changes we're just adding an optional parameter (entityType) to 3 of our existing Search API's. The parameter was already available for one of our existing Search API. With this change it is now available for 3 additional API's. No version change is necessary or required for this change. It is also NOT a breaking change.

@sataneja per the SDK guidelines, the change you are describing is explicitly called out as requiring an API version change:

A service’s API is a contract with customers and is represented by using the api-version query parameter. Changes such as adding an optional property to a request/response or introducing a new operation is a change to the service’s contract and therefore requires a new api-version value. This is critically important for documentation, client libraries, and customer support.

You will need to take this up with the architects in the API review board.

@JeffreyRichter this was the issue discussed in today's meeting.

@tjprescott - I was told that @JeffreyRichter agreed to this change in the review meeting. Would it be possible for us to approve and merge this PR now? Let us know. Thanks!

/azp run Swagger ModelValidation

/azp run


Use statement for RuntimeException required in class ContainerTrait

A use statement needs to be added to class ContainerTrait for the RuntimeException like shown below:
<?php

namespace Psr7Middlewares\Utils;

use Interop\Container\ContainerInterface;
use InvalidArgumentException;
use RuntimeException;

otherwise the RunTimeException class will not be found when thrown as it is looked up in local namespace.

Good catch! Fixed in the new version.
thank you


XD-3512: Initial GemFire binder

Initial implementation of GemFire binder.
See https://docs.google.com/document/d/18gjwnh50CADCVyfnEh2h5cbgrIZm6n4UYQlaHGQW_QU/edit
for more details.
Next steps include:

Partitioning support
Configuration support (including cache topologies)
Automated test suite

This resolves #191.
This closes #198.

This resolves #191.

Closing PR pending upcoming work on consumer group support.


Add openai-vision extension

Description
Get image insights with OpenAI Vision.

ask selected image
ask clipboard image

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

btw, the existing ChatGPT extension already supports vision

hey erics, thanks for mentioning. I didn't know about it.
I installed the extension.
ok I have just tried it.. here are few differences i have notice.
the chatgpt extension doesn't allow me to ask a selected image. I have to open the command and search the image from there.
the chatgpt extension doesn't allow me to ask a image copied to the clipboard neither.
If you haven't told me, It is hard for me to find out. When installing it. "vision" wasn't mentioned in the extension description and also there is no command that state it can ask image. I had to open "models" command to find out it can enable "vision capabilities". once enabled i still can't paste or select the image right in front of me. to later i had to figure out I have to  open "Full Text Input" by hitting Enter to be able to "open file"
the ability to ask more than 1 images looks useful, but the steps to reach there and not be able to know if it provides vision before installing it are not good

I would also think it would be better to enhance the current GPT extension where we update the keywords to also include "vision"

I would also think it would be better to enhance the current GPT extension where we update the keywords to also include "vision"

Build it into current ChatGPT extension
https://github.com/raycast/extensions/pull/12929


Switch to Rinkeby?

The genesis file mentioned in the instructions for connecting to Ropsten is no longer available for download. Maybe it's time to switch to Rinkeby instead in the readme instructions? https://github.com/machinomy/machinomy/commit/b47ca9d9594de4c23c10613546a693b0797f4046 suggests it should work?

@michielbdejong You do not have to download genesis block or anything to use Ropsten. If you use any recent client, geth --testnet joins Ropsten.

It's now on Rinkeby if you use latest master from github, thanks!


Hide the raw type with private(?) traits

There are a lot of wrapper types that expose internal types, which is unnecessary for most usage scenarios.
#[repr(transparent)]
pub struct FlagAction(xed_flag_action_t);

impl FlagAction {
    pub fn from_ref(raw: &xed_flag_action_t) -> &Self {
        // SAFETY: SimpleFlag is #[repr(transparent)]
        unsafe { std::mem::transmute(raw) }
    }

    pub fn from_raw(raw: xed_flag_action_t) -> Self {
        Self(raw)
    }

    pub fn into_raw(self) -> xed_flag_action_t {
        self.0
    }

    pub fn as_raw(&self) -> &xed_flag_action_t {
        &self.0
    }

    pub fn as_raw_mut(&mut self) -> &mut xed_flag_action_t {
        &mut self.0
    }
}

Maybe we can extract some traits to hide it like AsPtr?
pub trait AsRaw {
    type CType;

    fn as_raw(&self) -> &Self::CType;
}

impl AsRaw for FlagAction {
    type  CType = xed_flag_action_t;

    fn as_raw(&self) -> &Self::CType {
        &self.0
    }
}

And we could add a produce macros to generate those implementations
#[repr(transparent)]
#[derive(FromRaw, IntoRaw, AsRaw, AsRawMut)]
pub struct FlagAction(xed_flag_action_t);

I think I mentioned all this in the review over in #5 but I'll add it here for posterity :)
Getting rid of the copy-paste here is definitely a good idea! I just think that doing it via traits is the wrong approach. Rather, using a macro to take care of the definitions means that we don't have to copy paste anymore but still keeps these methods declared with the type in the documentation and when people are using autocomplete to find them.


Task.Delay Hangs Page

Describe the bug
Please provide as much information as you can.
Did this error occur while using dotnet try or online?

[ ] dotnet-try
[x] online

What kind of error was it?

[ ] User Interface (UI):  For example the output never displayed
[ ] Service Error: For example "The service is temporarily unavailable. We are working on it"
[x] Other: Page Freezes

Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following:

OS

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

Browser

[x] Chrome
[x] Edge
[ ] Safari

Hello. I tried running some code with a delay and found the page would freeze. In this simpler example, no output is produced, even for the first write line, and the page becomes unresponsive.
using System;
using System.Threading.Tasks;

public class Program
{
    public static async Task Main(string[] args)
    {
        Console.WriteLine("Before Delay");
        await Task.Delay(100).ConfigureAwait(false);
        Console.WriteLine("After Delay");
    }
}

Nevermind, just saw this duplicates #527


[FLINK-30400][docs] Add some notes about changes for flink-connector-base dependency in externalized connectors

What is the purpose of the change
This pull request adds some notes about changes for flink-connector-base dependency in externalized connectors.
Brief change log
Add some notes about changes for flink-connector-base dependency in externalized connectors docs.
Verifying this change
This change is a docs work without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: ( no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: ( no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@rmetzger @leonardBang Would you like to help to review this PR?


Unable to do alias path in the server folder

Please provide the environment you discovered this bug in.

Window 11
PNPM
Angular 17

Which area/package is the issue in?
vite-plugin-nitro
Description
when I create an alias path for the server folder it gives me error:
[worker reload] Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@server/auth' imported from D:\Work\ddapp\dd\dist\.nitro\dev\index.mjs
    at new NodeError (node:internal/errors:405:5)
    at packageResolve (node:internal/modules/esm/resolve:887:9)
    at moduleResolve (node:internal/modules/esm/resolve:936:20)
    at defaultResolve (node:internal/modules/esm/resolve:1129:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36)

then I checked the dist\.nitro\dev\index.mjs
and I found the imports are like this:
import { oidcConfig } from<EMAIL_ADDRESS>import { createOidcClient } from<EMAIL_ADDRESS>import { initSession, qlikClientOptions, SESSIONS } from<EMAIL_ADDRESS>import { QlikSession } from<EMAIL_ADDRESS>
the alias paths work fine with the browser files but it throws this error when I use with server files,
I configurated the allies paths as following:
in the tsconfig.json I put:
...
"baseUrl": "./",
    "paths": {
      "@core/*": ["./src/app/core/*"],
      "@components/*": ["./src/app/components/*"],
      "@pages/*": ["./src/app/pages/*"],
      "@environments/*": ["./src/environments/*"],
      "@server/*": ["./src/server/*"],
    },
  ...

in the vite.config.ts I put:
...
resolve: {
    mainFields: ['module'],
    alias: {
      '@core': path.resolve(__dirname, 'src/app/core'), // adjust the path according to your project structure
      '@components': path.resolve(__dirname, 'src/app/components'),
      '@pages': path.resolve(__dirname, 'src/app/pages'),
      '@environments': path.resolve(__dirname, 'src/environments'),
      '@server': path.resolve(__dirname, 'src/server'),
    },
  },
...

Please provide the exception or error you saw
[worker reload] Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@server/auth' imported from D:\Work\ddapp\dd\dist\.nitro\dev\index.mjs
    at new NodeError (node:internal/errors:405:5)
    at packageResolve (node:internal/modules/esm/resolve:887:9)
    at moduleResolve (node:internal/modules/esm/resolve:936:20)
    at defaultResolve (node:internal/modules/esm/resolve:1129:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36)

Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

when I added
 plugins: [
    analog({
      nitro: {
        alias: {
          '@core': 'src/app/core',
          '@components': 'src/app/components',
          '@pages': 'src/app/pages',
          '@environments': 'src/environments',
          '@server': 'src/server',
        },
      },
    }),
  ],

I got this error:
[nitro 11:54:22 AM]  ERROR  Error: Could not load src/server/auth/odic.client (imported by src/server/routes/auth/login.ts): ENOENT: no such file or directory, open 'D:\Work\ddapp\dd\src\server\auth\odic.client'
almost because of the missing .ts extension in the ts imports
import { createOidcClient } from<EMAIL_ADDRESS>

You can use a rollup plugin for this
https://www.npmjs.com/package/rollup-plugin-typescript-paths
installing this as a dev dependency, importing the typescriptPaths plugin, and configuring as:
analog({
nitro: {
rollupConfig: {
plugins: [typescriptPaths({ tsConfigPath: 'tsconfig.base.json', preserveExtensions: true })]
}
}
})

@brandonroberts worked but I start find this warning
"file:///D:/Work/ddapp/dd/node_modules/.pnpm/@analogjs+vite-plugin-nitro<EMAIL_ADDRESS>is imported by "virtual:#internal/nitro/virtual/server-handlers", but could not be resolved – treating it as an external dependency.

@AbdallahR99 try with the latest release. We resolved some issues with Windows specifically

Unforuntately getting the same error

Using version 1.3.0.
In fact just supplying "rollupConfig" causes this, (and removing it works again);
        nitro: {
          rollupConfig: {
          },

Then any call to a .server route fails.

Ran into this also. Got a fix coming soon

@brandonroberts, your fix worked! I've integrated it into my project, and now I'm able to import from @server/.... Here’s the configuration that’s working for me:
import {defineConfig} from 'vite';
import analog from '@analogjs/platform';
import { resolve } from 'path';
import {typescriptPaths} from "rollup-plugin-typescript-paths";

// https://vitejs.dev/config/
export default defineConfig(({mode}) => ({
  build: {
    target: ['es2020'],
  },
  resolve: {
    mainFields: ['module'],
    alias: {
      '@components': resolve(__dirname, 'src/app/components'),
      '@layout': resolve(__dirname, 'src/app/layout'),
      '@lib': resolve(__dirname, 'src/app/lib'),
      '@models': resolve(__dirname, 'src/models'),
      '@services': resolve(__dirname, 'src/app/services'),
      '@server': resolve(__dirname, 'src/server')
    }
  },
  plugins: [analog({
    vite: {
      inlineStylesExtension: 'scss',
    },
    nitro: {
      preset: 'node-server',
      rollupConfig: {
        plugins: [typescriptPaths({ tsConfigPath: 'tsconfig.json', preserveExtensions: true })]
      }
    },
  })],
  test: {
    globals: true,
    environment: 'jsdom',
    setupFiles: ['src/test-setup.ts'],
    include: ['**/*.spec.ts'],
    reporters: ['default'],
  },
  define: {
    'import.meta.vitest': mode !== 'production',
  },
}));

I also made sure to include the aliases directly in my tsconfig.json file rather than in tsconfig.base.json.


OS-native integrations (Windows/macOS) fail with Error: Pattern must be a string

Found a bug during manual QA from https://github.com/ipfs/ipfs-desktop/blob/main/DEVELOPER-NOTES.md#manual-testing
This is blocking 0.19 release.
Bug
Build 3bc3a8a29e16473a11786fa0b9f1d42365f436af fail to import a single file via context menu in Windows' File Explorer with error  Error: Pattern must be a string in error.log
Error comes from add-to-ipfs.js (probably similar problem as one fixed in https://github.com/ipfs/ipfs-desktop/commit/13de17403fcd1892a0830b3da0199d116a14d899)
Notes

fix is most likely fixing type somewhere – needs analysis
we need unit tests for add-to-ipfs.js, it breaks too often, every time there are changes in JS API interfaces

@hacdias would you have time to look at this Friday?


fix(storage): update CustomTime and NonCurrentTime

This updates the sample to account for a breaking change in the
apiary client. This PR also bumps the apiary library to the latest
released version.

LGTM.
But maybe change the library version on this PR instead of in #1247. The change will work either way, but it makes more sense once we are using the latest library version.

Done!

LGTM.
But maybe change the library version on this PR instead of in #1247. The change will work either way, but it makes more sense once we are using the latest library version.

Done!

Thanks, let's wait for tests and @frankyn to confirm.

Thanks, let's wait for tests and @frankyn to confirm.


web joystick: split axes into separate joysticks

Should make it easier to control the body, requested by @briskspirit

No need for an issue here. It's easy enough to just try it out and see if it's better.

Forgot about this, maybe it would be nice to keep these types of issues open so prospective contributors can see it might be wanted. We can also add a low-priority tag for us so we know what is an actual issue, and what is a feature request


[oslog][stdlib-private] Refactor and generalize interpolation of Int type in the new os_log APIs so as to extend to other integer types.

Moved OSLogInterpolation.appendInterpolation method and related helper functions for Int to a new file OSLogIntegerTypes.swift.

Generalized the helper functions so that they can accept any integer type.

Used this to add support for Int32. Support for other integer types will be implemented similar to Int32.

@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 8dbff10f47dfafdfd353acd23c8ae3212166eb1f

Build failed
Swift Test OS X Platform
Git Sha - 8dbff10f47dfafdfd353acd23c8ae3212166eb1f

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 424d443c05fe1b21275968c8e5e257d2aa8ce5c8

Build failed
Swift Test Linux Platform
Git Sha - 424d443c05fe1b21275968c8e5e257d2aa8ce5c8


Improvements for next version

list of all included packages in documentation
change margin comment to use \marginpar and add float comment using \marginfloat
fix issue with \P and \d note displaying correctly at start of a displayed line
add \defas and \defs commands for := and =: respectively

Closes #59 #58 #57 #54

@jhuggins awesome! I will look this over ASAP.
(I did edit your original PR post to include the "closes" tags -- I think github only pays attention to those in the original post)


Seal of the Heretic

This item is missing it's green border.

Additionally, this issue applies to the Debit Card

Additionally, this issue applies to Photodiode


[FEATURE] Add New ML Algorithms

Description:
Given that the salary variable is a continuous variable, it would be right to apply regression models such as linear regression, random forest regressor, decision tree regressor, etc., but here we did apply classification models and reported classification accuracy, which seems a bit odd .

[ ] Need to perform Linear Regression.
[ ] Apply Random Forest Algorithm.
[ ] Add Decision tree Regressor.

I want to work on this issue , please assign this to me

I want to work on this issue , please assign this to me

hello @sanjay-kv sir i would like to work on this issue. i have experience of machine learning algorithms i am confident i would be able to do the implementation correctly please assign it to me.

Hello @sanjay-kv , I'd like to work on this issue please assign it to me under GSSoC'24

I want to work on this issue for GSSoc'24.Kndly assign it to me @sanjay-kv .

Please assign the task to me for GSSOC contribution!

Hi, please assign this to me
GSSOC 2024 Contributor

hi @sanjay-kv  i want to contrtibute .Assign me this issue

Thanks a lot 😊
On Fri, 10 May, 2024, 9:48 pm Shazia Khan, @.***> wrote:

Hi, please assign this to me
GSSOC 2024 Contributor
—
Reply to this email directly, view it on GitHub
https://github.com/Recode-Hive/Stackoverflow-Analysis/issues/8#issuecomment-2104880560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGWBJH2YPRZHTCLIZGRS35LZBTXNFAVCNFSM5WEAWHVKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJQGQ4DQMBVGYYA
.
You are receiving this because you commented.Message ID:
@.***>

Thanks a lot 😊
On Sat, 11 May, 2024, 7:52 am SwarnenduJUIT, @.***>
wrote:

hi @sanjay-kv https://github.com/sanjay-kv i want to contrtibute
.Assign me this issue
—
Reply to this email directly, view it on GitHub
https://github.com/Recode-Hive/Stackoverflow-Analysis/issues/8#issuecomment-2105459514,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGWBJH4QC6G54LKVUMM7Q3DZBV6HNAVCNFSM5WEAWHVKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJQGU2DKOJVGE2A
.
You are receiving this because you commented.Message ID:
@.***>

Thanks a lot 😊
On Fri, 10 May, 2024, 10:47 pm Amarta Waghani, @.***>
wrote:

Hi @sanjay-kv https://github.com/sanjay-kv I want to work on this
issue. Assign this issue to me
—
Reply to this email directly, view it on GitHub
https://github.com/Recode-Hive/Stackoverflow-Analysis/issues/8#issuecomment-2104984377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGWBJH2OCGZ5BEHXLBXEDHDZBT6LLAVCNFSM5WEAWHVKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJQGQ4TQNBTG43Q
.
You are receiving this because you commented.Message ID:
@.***>

hello @sanjay-kv sir i would like to work on this issue sir. Given my background and expertise in machine learning, I would be enthusiastic about leading this effort. I believe I can implement it properly. Please assign this issue to me

Hi I assigned 10 people, try the PR, i will select the best one and merge.
Also if you would like to work on more issue create a new issue and I will create label and assign you.

hi @sanjay-kv  I have basic experience in machine learning . I know how to train and test model on various ml algorithms. I am highly interested in the project kindly assign me the issue

hello @sanjay-kv ! I have experience in model training as well as predictive modelling.I want to contribute to this issue.Kindly assign me this issue .

hello @sanjay-kv sir i would like to work on this issue sir. Given my background and expertise in machine learning, I would be enthusiastic about leading this effort. I believe I can implement it properly. Please assign this issue to me

hello @sanjay.kv , may i  know why i have been unassigned from the issue?

hello @sanjay-kv sir i would like to work on this issue sir. Given my background and expertise in machine learning, I would be enthusiastic about leading this effort. I believe I can implement it properly. Please assign this issue to me

hello @sanjay-kv , may i know why i have been unassigned from the issue?

He mentioned that :
Issue will be assigned to Only 1 person ...FCFS basis. Others get a chance if the issue gets stale & the assignee is inactive for 5+ days
Also if you would like to work on more issue create a new issue and I will create label and assign you.

hi @sanjay-kv this issue is assign to anyone??

yes @aryaVishal1706 you can see it over here

@sanjay-kv  can I work on this project under GSSoC'24 ?


Implement simple bevy-markdown renderer

Support markdown syntax in rectangle. Render markdown once a user stops editing rectangle. As a baseline of features the following online markdown can be used: https://dillinger.io/

consider using https://github.com/trishume/syntect for syntax highlighting

Useful links:

https://github.com/wooorm/markdown-rs
https://github.com/trishume/syntect for syntax highlighting

Bold/italic/links implemented here: https://github.com/StaffEngineer/velo/pull/96

Basic syntax highlighting: https://github.com/StaffEngineer/velo/pull/99

Headers & inline code: https://github.com/StaffEngineer/velo/pull/109

Lists: https://github.com/StaffEngineer/velo/pull/122

Headings support was temporary removed due to migration to cosmic-text.


Add Code of conduct, contribution guidelines and license in readme file.

Describe the bug/feature
Hey @rockingrohit9639
I would like to add code of conduct, license and contribution guidelines in readme file.
As they are present in branch but not linked in readme file.
Please assign it to me.
Thanks
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Logs
If applicable, add logs to help explain your problem (e.g. the error message and/or exception traceback).
Environment (please complete the following information):

OS:
Browser:

Additional context
Add any other context about the problem here.

Go ahead  :)


Problem with launching the game

I'm trying to launch the game with the "launch_mc_vanilla()" function and I always get this error

Why does this error occur and how could it be fixed?

What version of the game are you trying to launch?

1.20
to 13. heinäk. 2023 klo 15.05 Shasank Prasad @.***>
kirjoitti:

What version of the game are you trying to launch?
—
Reply to this email directly, view it on GitHub
https://github.com/shasankp000/mc_launcher_core_rs/issues/1#issuecomment-1634124939,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3OS2K266KPKKVOCSGELE23XP7P67ANCNFSM6AAAAAA2IREM2E
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, I will check if I can reproduce the issue on my end or not and update you

Okay so, sorry for being late by two months, there are some issues in my life which have been preventing me from working these days.
As it turns out there was indeed an issue with the backend repository of this repository, https://github.com/shasankp000/mcdl_core_rs
Since 1.20 the "assetIndex" file structure has been changed so the downloader was unable to download the game assets for version 1.20. I have patched that and I will upload the patch to the github and crates.io repos.
And I will update that patch into the cargo.toml of this repo, please check again a few hours later after this comment has been posted to access the patched version. You should be able to launch the game then.

Actually there are a few more bugs in the dependency of this repo that I will patch, I am keeping this issue opened until I have uploaded the patch, then I will close it.


The input form is generally too complicated

Generally speaking, it is very easy to get lost through the sections. Information with different levels of importance are presented at the same level in the interface and many times I don't know in which section and subsection I am
Reducing the fields or eliminating full sections should be considered.
Before eliminating fields  we could try playing with the colors of the levels of the db. Another option could be that the third level of information should be shown only when it is dropped down. For instance: “Published test protocol” which is inside “Performance test” which is inside of “Instrument” should not be fully displayed and its background should be in yellow (this is just an example).

There will be many changes ...


[FLINK-18347][connector] kinesis connector throw Error java.lang.NoSuchFieldError: NO_INTS

What is the purpose of the change
java.lang.NoSuchFieldError: NO_INTS is caused by conflicting versions of the jackson libraries. flink-connector-kinesis uses the 1.11.754 version of aws-java-sdk-kinesis and aws-java-sdk-core. The version of aws-java-sdk-core uses the 2.6.7 version of jackson-dataformat-cbor, which repository refers to AWS SDK For Java Core 1.11.754. Flink generally uses the 2.10.1 version of jackson and the version of jackson-dataformat-cbor couldn't correctly parse the CBOR data format.
Brief change log

Modify the version of jackson-dataformat-cbor dependency in flink-connector-kinesis to use 2.6.7.

Verifying this change
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@zentol Please help to review the change of harden jackson dependency constraints. The version of aws-java-sdk-core uses the 2.6.7 version of jackson-dataformat-cbor. This version of jackson-dataformat-cbor has no CVE.

Let's discuss this first in the JIRA.


AttachFiles component

PR for #103

Branch rebased, ready for review.

Took another look at this and LGTM. I'm still not super clear on the usage but it'll become more clear when it gets into auto.

Hey Team, this LGTM as well. I think we should merge it so we can go ahead with task x-team/auto#829


Why round()?

https://github.com/NucciTheBoss/charming-exercises/blob/20e2c30e57d8fb3d41dac342b6542f2f6d3b7b55/hot-potato/src/charm.py#L168

Fixed in the latest update.


chore: Add powertools specific user-agent-suffix to the AWS SDK v2 clients

Issue #:
https://github.com/aws-powertools/powertools-lambda-java/issues/1274
Description of changes:

Checklist

[x] Meet tenets criteria](https://docs.powertools.aws.dev/lambda-java/#tenets)
[x] Update tests
[ ] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks great! I only have two questions:

Can you also instrument clients that customer pass to the utility? Atm it only happens in if (client == null) blocks.
Is it feasible to extract overrideConfiguration in a dedicated place? Atm every client runs exactly the same line. Might be easier to have it in one place and apply changes in the future in one file, instead of going through every constructor.

Overall looks good but I agree with Alex, we should add the user agent even on clients they pass to our modules. And +1 on the second point: can we have a method in the core module where we can pass any client, we override the configuration and return the client ?

Codecov Report
Patch coverage: 61.36% and project coverage change: -0.04 :warning:

Comparison is base (4c42359) 79.19% compared to head (b1809cc) 79.15%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1306      +/-   ##
============================================
- Coverage     79.19%   79.15%   -0.04%
- Complexity      639      649      +10
============================================
  Files            73       74       +1
  Lines          2360     2394      +34
  Branches        258      262       +4
============================================
+ Hits           1869     1895      +26
- Misses          410      418       +8
  Partials         81       81

Impacted Files
Coverage Δ

...ambda/powertools/parameters/AppConfigProvider.java
82.53% <0.00%> (-2.71%)
:arrow_down:

...zon/lambda/powertools/parameters/BaseProvider.java
97.82% <ø> (ø)

...lambda/powertools/parameters/DynamoDbProvider.java
79.03% <0.00%> (-1.30%)
:arrow_down:

.../lambda/powertools/parameters/SecretsProvider.java
64.28% <0.00%> (-1.57%)
:arrow_down:

...oftware/amazon/lambda/powertools/sqs/SqsUtils.java
77.63% <0.00%> (-2.10%)
:arrow_down:

...mpotency/persistence/DynamoDBPersistenceStore.java
87.67% <33.33%> (-0.53%)
:arrow_down:

.../powertools/core/internal/UserAgentConfigurer.java
96.00% <96.00%> (ø)

...azon/lambda/powertools/parameters/SSMProvider.java
87.32% <100.00%> (+0.18%)
:arrow_up:

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

Looks great! I only have two questions:

Can you also instrument clients that customer pass to the utility? Atm it only happens in if (client == null) blocks.

This can happen if we implement the ExecutionInterceptor and modify the request headers, since we can't modify the sdk client as soon as it is built. If we go this way, maybe it makes sense to do it in every case (whether the client already exists or not) so that we have a common implementation. What do you think?

Is it feasible to extract overrideConfiguration in a dedicated place? Atm every client runs exactly the same line. Might be easier to have it in one place and apply changes in the future in one file, instead of going through every constructor.

I thought about doing that initially. But then, I thought that it might be better if we do a refactoring(in a seperate PR) and have a common way of building clients. This is because, duplicate code is also the issue when setting the region and the HttpClient. They are set is several places, the same way. So why extract only the overrideConfguration part? And if we modify something else in the future (either in overrideConfiguration or somewhere else), where should it be placed?
Thinking out loud here :)

Overall looks good but I agree with Alex, we should add the user agent even on clients they pass to our modules. And +1 on the second point: can we have a method in the core module where we can pass any client, we override the configuration and return the client ?
And the build is failing, not sure about the logger you use... (sun ? see Scott's comment about that)

Yes, fixed that already, had accidentaly a wrong import and I didn't notice at all that was the case
@am29d @jeromevdl See my comments inline

@eldimi I agree we should refactor this and have the common sdk client stuff in the core module ("common" in v2). Would you like to do this? (in another PR)

@eldimi I agree we should refactor this and have the common sdk client stuff in the core module ("common" in v2). Would you like to do this? (in another PR)

yes, I can do that

almost OK

I saw some code smells reported by sonarqube - related to the current PR changes.  Any comments on that related to my reply here?

instrument clients that customer

almost OK

What abou instrumenting the clients that customers provide? Any comments on that related to my reply here?

I agree, should be part of the refactor I mentioned. We should first merge this PR and then you move all the common stuff, plus the interceptor somewhere.

Is there anything missing from my side here?

@eldimi, I'll merge that one. Can you write an RFC for the instrumentation and generic client stuff you proposed? So we can elaborate on it...

Looking a bit more into that..The ExecutionInterceptor is only configured in the builder, therefore no way that it can be used to intercept the request for an already provided client instance.
Regarding the refactoring for moving the common sdk client initialization in the core module, do we still need an RFC for that? Also, if we are implementing https://github.com/aws-powertools/powertools-lambda-java/issues/1092, maybe it could be part of that?


expose panel state

I could not find a way to let a panel content know if it is currently being displayed or not.
Because of this, I cannot apply ng-if to heavy panel content.
I tried to decorate with a directive of my own, but it does not work when the panel is opened via keyboard navigation.
Do you plan on exposing this state ?
Thank you
angular
        .module('app.common')
        .directive('exposeExpansionPanelState', exposeExpansionPanelState);

    function exposeExpansionPanelState() {
        return {
            restrict: 'A',
            require: '^^mdExpansionPanel',
            link: ExposeExpansionPanelState
        };
    }

    /* @ngInject */
    function ExposeExpansionPanelState(scope, element, attrs, expansionPanelCtrl) {
        var expandFunction = expansionPanelCtrl.expand;
        var collapseFunction = expansionPanelCtrl.collapse;

        expansionPanelCtrl.expand = function() {
            scope.$isOpen = true;
            expandFunction();
        };

        expansionPanelCtrl.collapse = function() {
            scope.$isOpen = false;
            collapseFunction();
        };
    }

I will add an isExpanded function to the exapnsionPanelService and scope $panel object.
I will post an example with the update

That is great news, thank you!

added an isOpen function to the panel service and directive
expansionPanelCtrl.isOpen()

<md-expansion-panel>
  <md-expansion-expanded>

  </md-expansion-expanded>
</md-expansion-panel>

if added through the $mdExpansionPanelGroup service and controller is assigned
angular.module('app').controller('panelCtrl', function ($panel) {
  $panel.then(function (instance) {
    instance.isOpen();
  });
});


Edit DG

Fix typos and minor details in DG

Codecov Report

Merging #272 (8e32313) into master (d813577) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #272   +/-   ##
=========================================
  Coverage     75.14%   75.14%
  Complexity      701      701
=========================================
  Files           110      110
  Lines          2112     2112
  Branches        255      255
=========================================
  Hits           1587     1587
  Misses          435      435
  Partials         90       90

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 d813577...8e32313. Read the comment docs.


Compilation Failed

Hello,
attempting to use ivy.load results in the Compilation Failed and the traceback says:
cmd1.sc:1: not found: value load val res1 = load.ivy("com.chuusai" %% "shapeless" % "2.20-RC6")
Not sure what the problem is and any help would be appreciated. Thanks!

Try as described in README of this repo.
import $ivy.org.apache.spark::spark-sql:2.1.0

Yeah, the syntax would be more
import $ivy.`com.chuusai::shapeless:2.3.3`


Spanning Table: How to add attribute colSpan

I would like to add colSpan for each column with different number. Is there a possibility in the current version ? Or I am missing something ?
Thank you very much.

Where exactly in the DOM are you looking to add this? It's possible that customBodyRender will let you do what you want, as this feature allows you to send an entirely custom component in to be displayed for each cell in a given column.

I would like to do something like this: Spanning table where you can see they are using colSpan for TableCell.

Have you tried the custom render features?


LCFS - BUG - Marine missing from End use

Describe the Bug:
The option "Marine" is missing under End Use for the Fuel Type/Category: Electricity/Diesel.
Expected Behaviour:
The End Use dropdown should include "Marine" as a selectable option when the Fuel Type/Category is set to Electricity/Diesel.

Add Marine back in the lookup table with EER 2.5
Actual Behaviour:

"Marine" is not available as an option in the End Use dropdown for Electricity/Diesel.
This limits the correct reporting and categorization of fuel usage.

Implications:

Users are unable to correctly categorize "Marine" as an End Use for Electricity/Diesel.
Potential data inaccuracies or incomplete submissions due to the missing option.
User frustration and workflow disruptions.

Steps To Reproduce:
User/Role: BCeID user

Navigate to the Fuel supply schedule.
Select Electricity/Diesel under Fuel Type/Category.
Open the End Use dropdown.
Observe that "Marine" is not listed as an option.

Additional Context:

Verify that "Marine" is correctly configured as a valid End Use for Electricity/Diesel in the system.

Blocked pending completion of #1503 as this requires Seed Data to be inserted


InvalidOperationException thrown in tag helpers using cache in 2.2.0 preview 1

Is this a Bug or Feature request?:
Bug (or breaking behaviour change)
Steps to reproduce (preferably a link to a GitHub repo with a repro project):
See martincostello/website#228, run Build.ps1 or build.sh.
Description of the problem:
I have some custom tag helpers in my code which derive from some built-in tag helper classes. Due to obsolete warnings on the constructor, as part of trying out 2.2.0 preview 1, I changed the constructor used to the new non-obsolete one (commit).
Now when the tests are run, various 500 errors occur due to the below exception.
 [2018-08-26 10:52:31Z] fail: Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware[1]
       An unhandled exception has occurred while executing the request.
 System.InvalidOperationException: Cache entry must specify a value for Size when SizeLimit is set.
    at Microsoft.Extensions.Caching.Memory.MemoryCache.SetEntry(CacheEntry entry)
    at Microsoft.Extensions.Caching.Memory.CacheEntry.Dispose()
    at Microsoft.Extensions.Caching.Memory.CacheExtensions.Set[TItem](IMemoryCache cache, Object key, TItem value, TimeSpan absoluteExpirationRelativeToNow)
    at MartinCostello.Website.TagHelpers.InlineStyleTagHelper.ProcessAsync(TagHelperContext context, TagHelperOutput output) in C:\Coding\website\src\Website\TagHelpers\InlineStyleTagHelper.cs:line 118

It appears that due to TagHelperMemoryCacheProvider using a fixed size, it is no longer possible to use the Cache.Set(string, object) method.
This appears to be a breaking behavioural change due to the internal implementation detail of using a private cache for tag helpers, rather than the overall cache.
Curiously, from my CI builds, this only appears to affect Windows and OS X, and not Linux.

AppVeyor CI
Travis CI

Version of Microsoft.AspNetCore.Mvc or Microsoft.AspNetCore.App or Microsoft.AspNetCore.All:
2.2.0-preview1-35029

Thanks for contacting us, @martincostello.
@NTaylorMullen, can you please look into this? Thanks!

It appears that due to TagHelperMemoryCacheProvider using a fixed size, it is no longer possible to use the Cache.Set(string, object) method.

This is intentional. The recommendation is to use a custom instance of IMemoryCache if you need to continue using the MemoryCache.Set. Alternatively specifying the size is also viable. Yet another alternative is to continue using the deprecated ctor for ImageTagHelper - this should leave you unbroken until the 3.0


Primes and significant markdown changes

Oooh resolving merge conflicts, my favorite

Validated everything is still working, merging into main


Support "defer" as well as "async"?

It might be nice if this could handle defer scripts as well as async, as an option.
e.g. having a consolidated interface to allow notification either type of scripts being parsed. Would allow async scripts to wait on defer ones, etc.

Let me think about this some more... in the mean time you can use the before callback to set the defer attribute:
loadjs(['/path/to/foo.js', '/path/to/bar.js'], {
  success: function() {},
  error: function(pathsNotFound) {},
  before: function(path, scriptEl) {
    scriptEl.setAttribute('defer', 'defer');
  }
});

Yep - I guess this is a similar request to https://github.com/muicss/loadjs/issues/84 - if there's an easy way to set those flags innately in the library, would prevent a lot of extra boilerplate needed in consumers.  And should hopefully be a tiny change?

@asos-tomp @amorey
I think something like:
    { success:function(){}, async: true, defer: false, crossorigin: "anonymous",
      importance: "high" }

Adding native way to set defer, async, crossorigin and importance...
My real example:
<script defer importance="low">
  loadjs(
    ["https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.11.2/css/all.min.css",
<EMAIL_ADDRESS>    { success:function(){}, async: true, defer: false, crossorigin: "anonymous",
      importance: "high" }
  );
  loadjs(
    ["https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.11.2/css/all.min.css",
<EMAIL_ADDRESS><EMAIL_ADDRESS>     "https://cdn.jsdelivr.net/combine/npm/short-and-sweet@latest/dist/short-and-sweet.min.js,npm/jquery-vjs-rails@latest/src/rails.min.js,npm/bootstrap@latest/dist/js/bootstrap.bundle.min.js,npm/plyr@latest/dist/plyr.min.js,npm/clipboard<EMAIL_ADDRESS>     "https://cdnjs.cloudflare.com/ajax/libs/vanilla-masker/1.2.0/vanilla-masker.min.js",
     "/javascripts/footer.js"],
    { success:function(){}, async: false, defer: true, crossorigin: "anonymous",
      importance: "low" }
  );
</script>

Thanks!

I did some browser testing and found that when you add a <script> tag dynamically (e.g. document.head.appendChild(scriptEl);) the file gets loaded after the DOMContentLoaded event fires even if async=false. This means that the defer attribute won't have an impact on files loaded with LoadJS.

Thanks!@amorey
How about crossorigin and importance?

Currently, you can add crossOrigin manually using the before() method. I'm not familiar with the importance attribute. Can you refer me to some documentation about it?


Release v0.7.1

Placeholder issue for v0.7.1.

Release notes:
## Changes by Kind

### Bug or Regression

- Fixed a bug that prevents helm install to work when installing on a cluster where the namespace already exists. (#1568, @tuxerrante)

## Dependencies

### Added
_Nothing has changed._

### Changed
- github.com/containers/common: [v0.51.0 → v0.51.1](https://github.com/containers/common/compare/v0.51.0...v0.51.1)
- google.golang.org/grpc: v1.53.0 → v1.54.0

### Removed
_Nothing has changed._

Done https://github.com/kubernetes-sigs/security-profiles-operator/releases/tag/v0.7.1


chore(hits): update banner link type

Summary
This minor PR updates makes the url property of the new banner type (part of SearchResults.renderingContent.widgets.banners) required. link is still optional, but if link is present then we want to ensure that the url is also present.
Part of EMERCH-1370
Result
Green CI (all tests still pass)

looks good, no change in the code?

Thanks for taking a look @Haroenv - that prompted me to double check and I did find one other change.
Should the Banner type be added to the SearchResults namespace within algoliasearchHelper (like Facet) and then used in SearchResults class definition or is that excessive? I assume it could be use in instantsearch-ui-components/src/components/Hits.tsx instead of manually creating a banner type there.


Hidden workspace entry for imported pipeline

Importing a single pipeline doesn't show an entry in the workspace dialog, making it impossible to select it.
Bisected to d0dc3525, cc @rexissimus

Now importing a workflow doesn't close the first empty untitled vistrail, like the "open" function does.
Should this logic really be duplicated everywhere this way? It is obviously error-prone.

Ideally add_vt_window should go into create_view. But for some reason (that I don't remember) this caused problems. This code is brittle and I did not want do do too many changes.
Fixed the first vistrail in b29fb763f234f4a9eb58ef18da6a7f23f3f6daaf.


Fix/housekeeping

This is a purely housekeeping PR, focused mainly on the specs.  For easiest review, I suggest looking at the commits individually.

Approved!

Loved the system's API!
On May 11, 2015 1:43 PM, "seandc"<EMAIL_ADDRESS>wrote:

Approved!
—
Reply to this email directly or view it on GitHub
https://github.com/crashlytics/crashlytics-services/pull/76#issuecomment-100974780
.


Needs to support large files (>1MB)

My first attempt to open a file failed and the error shown reports that the app does not support a file greater than 1MB.
IMO This is an 'essential' feature for a notepad alternative.
Keep up the good work though.  Thanks.

I do agree it is better to support large files but considering the work that I need to at this moment, I have to put it into my backlog for now. I would say, give me sometime and I will get back to this. I do have a plan tho and I know exactly what needs to be done to solve this missing piece.

Maybe this is naive, but is it possible to (relatively) simply implement pagination by moving forward and backward through a FileStream in correlation to the scroll position?

@x3haloed The blocking issue is around ScrollViewer instead of RichEditBox. Actually RichEditBox can work with fairly large file but the ScrollViewer just cannot handle the size of the rendering. Believe it or not, if you set Font size to be 1-2, you can actually paste whole more content into Notepads.

You might be interested in this issue in WinUI https://github.com/microsoft/microsoft-ui-xaml/issues/1842.

Uhm... estimated time? 3 years have passed and this should have top priority IMHO

Uhm... estimated time? 3 years have passed and this should have top priority IMHO

Microsoft still doesn't support this scenario.

Great. Finally a really beautiful, dark mode supporting (basically perfect) notepad for Win 10. But with this 1 MB file size limitation, it's simply not usable. Unfortunately had to remove Notepads again because of this, too bad.

F ... I love this app but ... srsly this nneed to be solved, 3 yrs passed!

I upgraded to Win 11 and use the native editor (Notepad) again (my primary use-case for Notepads was the dark mode which the native one provides too on Win 11). Upgrade took me less then 3 years. Goodbye.

Wow ... thought this was just bothering me ... nope. Back to Notepad3. Keep it up, though!


[Android] autocomplete: Fix text doesn't autocompletes on selecting item

This got introduced in 41ce55, where background color is set to the
parent of absolute autocomplete view.
This seems to be a RN issue that, absolute positioned components
doesn't receive touchable events if inside component which has
background color set. (will try to verify it on new RN project)
So set background color to compose input view, sibling of autocomplete
view instead of parent of autocomplete view.
Fixes: #3378

I tried to reproduce it on new RN project, but can't. May be I am not able to get exact scenario where it will be reproducible or may be there is not RN issue, just we are doing something wrong.
But on the current master if we just remove
backgroundColor: 'rgba(127, 127, 127, 0.1)',

from style then it is not reproducible.

Thanks @jainkuniya ! Merged as 3d6968523. (Neglected to update this PR on push, so it'll show as closed rather than merged, sorry.)
I think it'd be quite useful to spend some more time pursuing the underlying RN issue, because this looks to be the second time we've run into it. Filed #3383 for that.


[Documentation] Add screenshot of GLSE to styles page

Select the subject of this issue:

[x] GL documentation (readthedocs)
[ ] Docstrings and/or comments within the code

Not necessary now that GLSE has its own doc website


Converting CSS Framework to Material 2

Updates

Upgraded @angular/cli to 1.7.1 due to error (see https://github.com/angular/angular-cli/issues/9307).

Added @angular/material with npm install @angular/material --save --save-exact.

Added link to externally-hosted Roboto fonts in index.html. (Roboto is Material Design’s default font.) We can host the font files locally rather than requesting fonts over the wire, if that is preferred.

Added mat-typography CSS class to submission.component.html, which applies default Material Design styles to all descendant tags. We can move this class to an ancestor element once we start to skin additional components. See https://material.angular.io/guide/typography#usage for details.

Updated Bootstrap col-x classes to be col-xs (rather than col-lg and col-sm) as Bootstrap col-x classes are mobile first.

Added Material Design classes to h1 and h3 tags. See https://material.angular.io/guide/typography#what-is-typography- for details.

Converted submission state display from Bootstrap badge to Material Design chip.

Installed @angular/flex-layout with npm install --save --save-exact @angular/flex-layout@latest.

Updated submission metadata tab-related components to use flex-layout.

Updated submission metadata tab-related components to use Material tabs. There is a defect with nested tabs where the selected tab UI is not displayed correctly (see https://github.com/angular/material2/issues/687). I have not implemented the suggested workaround. I have also left the top-level tabs with their out-of-the-box styling rather than updating them to match the pill-like styles in existing ingest-UI.

Updated metadata component to use Material cards.

To Discuss/Questions

Versioning in package.json.

TS imports in submission.component.html.

Multiple nested container-fluid without corresponding row or col-x dependents.

Use of <a href="javascript:void(0)" (click)="foo()"...>.

encapsulation: ViewEncapsulation.None on metadata-list.component.ts.

To Do

[x] Convert grid from Bootstrap to Material.

[x] Convert tabs from Bootstrap to Material.

[x] Convert card from Bootstrap to Material.

[ ] Convert metadata table from ngx-datatable to Material.

Hello @MillenniumFalconMechanic , sorry it took a while to merge this PR. Changes look good! Many thanks!


Lambda python 3.11 does not start with current boto 1.33.2

Describe the bug
Python Lambda with
boto3==1.33.2
botocore==1.33.2
Does not start
Expected Behavior
Starts without problem
Current Behavior
[2023-11-29T11:28:51+01:00] (2023/11/29/[$LATEST]9664a7f7e9e248f886b4d98e973b9c5a) INIT_START Runtime Version: python:3.11.v18	Runtime Version ARN: arn:aws:lambda:eu-central-1::runtime:6ebff6b58cf714d30879a40cc31554cd1bbc242da7bf75a000bc0c3052c6ebbc
[2023-11-29T11:28:52+01:00] (2023/11/29/[$LATEST]9664a7f7e9e248f886b4d98e973b9c5a) START RequestId: fb119c11-0401-456b-a9df-3b692ec3e9de Version: $LATEST
[2023-11-29T11:28:52+01:00] (2023/11/29/[$LATEST]9664a7f7e9e248f886b4d98e973b9c5a) [ERROR] Runtime.ImportModuleError: Unable to import module 'app': cannot import name 'is_s3express_bucket' from 'botocore.utils' (/var/lang/lib/python3.11/site-packages/botocore/utils.py)
Reproduction Steps
Create a lambda with requirements txt. It does not have to have any functions
Possible Solution
Publish current boto, the function
def is_s3express_bucket(bucket):
if bucket is None:
return False
return bucket.endswith('--x-s3')
existis.

Additional Information/Context
This could be a problem with new re-invent feature
SDK version used
1.33.2
Environment details (OS name and version, etc.)
Lambda Python 3.11

Same on Python 3.9

Same with Python 3.8
{
  "errorMessage": "Unable to import module 'my_module': cannot import name 'is_s3express_bucket' from 'botocore.utils' (/var/runtime/botocore/utils.py)",
  "errorType": "Runtime.ImportModuleError",
  "stackTrace": []
}

We have to pin the boto3 version because otherwise it could crush our production services

[WORKARROUND]
I needed to force the downgrade of boto3 and botocore in the Lambda, using pip install boto3==XXXXX botocore==XXXXX --upgrade inside the code. XXXXX was the last version that worked.
That's working for me now

Hi everyone, thanks for reaching out. We're currently taking a look at this but it appears something is going wrong with how these packages are being updated on Lambda in your cases.
is_s3express_bucket was released yesterday with boto3 1.33.2 which requires botocore 1.33.2. You can see the function defined here in the release. If Lambda is unable to find this, it implies it's using an older version of Botocore despite other modules (boto3 and/or s3transfer) being updated.
Can anyone provide the exact versions of these three packages they're importing into their function?

boto3
botocore
s3transfer

For immediate mitigation while we investigate, you'll need boto3<1.33.2, botocore<1.33.2, s3transfer<0.8.1. Note that you cannot update any of these independently of each other.

I just re-launch my script and still the same import error raised. Following packages were used.
boto3-1.33.0
botocore-1.33.0
s3transfer-0.8.1
sagemaker-2.183.0

For @MostafaHosseinian, the culprit is almost certainly s3transfer. You'll need to downgrade that to 0.8.0 unless you have the latest versions of boto3/botocore. It's only compatible 1.33.2+.
Those dependencies look correct @FernandoAndrade83 and run fine off Lambda. I'm moving to testing on Lambda now.

In case it helps - We have the same versions as @FernandoAndrade83 has, and it works for us off Lambda as well

I've packaged a lambda deployment zip that looks like:
$ tree -L 1
.
├── lambda_function.py
├── boto3
├── boto3-1.33.2.dist-info
├── botocore
├── botocore-1.33.2.dist-info
├── s3transfer
└── s3transfer-0.8.1.dist-info

This was generated from downloading the wheels off PyPI with python -m pip download boto3, unzipping them into the directory with my lambda function, and zipping them all into a .zip archive for Lambda.
The function I'm using is just to sanity check versions being used:
import json
import boto3
import botocore
import s3transfer

def lambda_handler(event, context):
    response = f"Boto3: {boto3.__version__}, Botocore: {botocore.__version__}, S3Transfer: {s3transfer.__version__}"
    return {
        'statusCode': 200,
        'body': response
    }

Uploading this to Lambda on all available Python versions is producing the same response:
{
  "statusCode": 200,
  "body": "Boto3: 1.33.2, Botocore: 1.33.2, S3Transfer: 0.8.1"
}

So, I think there's something common with how users in this thread are packaging their deployments that's leading to version mismatches. If there's anything I've used above you think may not be relevant to your case, please let me know.
@Saarett @FernandoAndrade83, can you provide any more info on how you created your lambda packages and how you confirmed the versions you listed are in the zip/being picked up by Lambda?

@nateprewitt So, I didn't run the lambda as you did (which is a very good idea!).
We're creating a layer that contains all the relevant packages. After downloading and unpacking the zip, I found the list of all packaged packages. Neither boto3 nor botocore were there, but s3transfer was present under the name s3transfer-0.8.1.dist-info. When I looked at the METADATA file, I found under the Requires-Dist section botocore (<2.0a.0,>=1.33.2).
Feels like your way is a bit easier though 😄

The default Lambda runtimes for Python 3.11 are boto3-1.27.1 botocore-1.30.1.  I also tested using a Lambda layer including boto3/botocore 1.33.2 with the code referenced above and it ran successfully.
Steps for creating the layer (paraphrased from this article):

mkdir -p boto3-layer/python
pip3 install boto3 -t boto3-layer/python
cd boto3-layer
zip -r /tmp/boto3-layer.zip .
aws lambda publish-layer-version --layer-name boto3-layer --zip-file fileb:///tmp/boto3-layer.zip --compatible-runtimes python 3.11

And then attach the layer:

Using the code referenced above results in:

Thanks for confirming, @Saarett! It appears we now have 2 of the three cases confirmed to be s3transfer being newer than boto3/botocore. You'll either want to downgrade s3transfer back to 0.8.0 or ensure you pull boto3/botocore 1.33.2 into your layer.
I think we can state the general guidance is if you've upgraded to s3transfer==0.8.1 you MUST also update to boto3>=1.33.2 and botocore>=1.33.2 in your layer or zip.

@nateprewitt But according to the METADATA file of the s3transfer package, it appears that I am using botocore>=1.33.2, aren't I? Otherwise, the package resolution wouldn't work,  please correct me if I'm wrong.
I do understand Lambda runtimes for Python 3.11 use lower version of botocore and boto3, so I'm not sure what I'm missing.

@Saarett you're correct that s3transfer requires Botocore>=1.33.2. But if you're not including that within your layer, it won't work. It's likely you have newer versions of boto3/botocore installed on your local system that are satisfying the requirements for s3transfer when creating the layer.
If those boto3/botocore copies don't end up in the final zip though, your function will fallback to the old copies supplied by Lambda, causing a failure.

After downloading and unpacking the zip, I found the list of all packaged packages. Neither boto3 nor botocore were there, but s3transfer was present under the name s3transfer-0.8.1.dist-info.

Please let me know if I'm misunderstanding this statement. I'm reading it to be your final layer has s3transfer==0.8.1, but not boto3 or botocore.

I'm experiencing the same issue using default boto3/botocore provided in 3.9 runtime:
INIT_START Runtime Version: python:3.9.v36	Runtime Version ARN: arn:aws:lambda:us-east-1::runtime:386a1053a16b9122f1622b361a8a78f39595e5a53df0c82932eacae85b33cd62
[ERROR] Runtime.ImportModuleError: Unable to import module 'runner_s3.app': cannot import name 'is_s3express_bucket' from 'botocore.utils' (/var/runtime/botocore/utils.py)
Traceback (most recent call last):

@nateprewitt Correct. It does make sense. In that case, sounds like I have 2 possible solutions:

Add the relevant boto3 and botocore versions to the layer, and they will override the Lambda runtime versions
Pin the local boto3 and botocore versions to be equal to those which are being used in the Lambda runtime (currently boto3-1.26.90, botocore-1.29.90)

Is that correct?

@nateprewitt Correct. It does make sense. In that case, sounds like I have 2 possible solutions:

Add the relevant boto3 and botocore versions to the layer, and they will override the Lambda runtime versions
Pin the local boto3 and botocore versions to be equal to those which are being used in the Lambda runtime (currently boto3-1.26.90, botocore-1.29.90)

Is that correct?

The second option you noted in my side is as follows, and it worked.

Lambda: boto3~=1.20.50  -  botocore~=1.29.109
local: boto3==1.29.5  -  botocore==1.32.5

@Saarett that's correct. s3transfer should only be used by boto3 and awscli. If you're updating either of those, you should always have a matching version of botocore in those layers, otherwise it's going to be susceptible to these kinds of failures.
The two options you listed are sufficient:

Pin to a known working set of dependencies for boto3, botocore, and s3transfer.
If you choose to upgrade, all three must be included in the layer/zip.

@nateprewitt Thank you for the assistance, it makes sense. We'll ensure that we pin the dependencies and also remove s3transfer from the layer.
As a small suggestion, it would be helpful to state in the documentation that we (the users) should also remove s3transfer from the layer. This information could have prevented our issue. Now I'm asking myself if there are similar dependencies I'm not aware of in other frameworks we use (i.e NodeJS) 🙂

As a small suggestion, it would be helpful to state in the documentation that we (the users) should also remove s3transfer from the layer.

Ideally this shouldn't be necessary. Do you know what other dependency was updating s3transfer in your layer? If it's not packaged with the awscli or boto3, I'm surprised it ended up on the layer to begin with.

@MostafaHosseinian, have you found any solution that helps you?

@nateprewitt Even if I have no dependencies using s3transfer, and I'm the only one to include boto3, botocore3 in my Pipfile, when I zip them into a layer excluding boto3 and botocore, all their dependencies, including dateutil, six, urllib3, jmespath, and s3transfer itself, are still getting zipped. I didn't realize I also needed to remove s3transfer.

@MostafaHosseinian, have you found any solution that helps you?

I kept the lambda as it was:
boto3~=1.20.50
botocore~=1.29.109
requests~=2.28.2
Also, pin the local boto3 and botocore versions to be the same as the last successful job (currently boto3-1.29.5, botocore-1.32.5)
My job run successfully, then

I ran into the issue and this was the cause.

However (at least in my case - looking at this dockerfile) s3transfer wasn't in the list of packages to remove. So if you updated the boto3/botocore dependencies and then build using a dockerfile like this that removes boto3 you'll get the lambda runtime boto3 but the latest s3transfer, which are incompatible. The solution is that if you're relying on the lambda runtime boto3 package then delete s3transfer too.

In an ideal world, we would be able to reference a boto3 package that always matches what is provided by lambda in our poetry/pip files, so that running things locally behaves the same as it does on lambda.

Is this resolved ? I tried to update to the latest versions but still I get the error :
After downloading and unpacking the zip, the below are the versions:
boto3 - 1.33.2
botocore - 1.33.2
s3transfer - 0.8.2

Is this ticket resolved? I was able to roll back and ship with Lambda by adding the following to my requirements.txt file.
python3.10
boto3<1.33.2 botocore<1.33.2 s3transfer<0.8.1

The ticket is still open for visibility but there isn't an issue with boto3/botocore to be resolved. The case this occurs is when s3transfer is updated without updating boto3/botocore violating the requirements specified by each package. The current cases where this may happen is if you're removing boto3/botocore for space savings but not removing s3transfer.
You can see the repro above demonstrating everything works as intended when all files are updated. If you need to remove boto3/botocore from your deployment to save space, you'll also need to ensure s3transfer is omitted. One example of how you might do this was provided @jasongi above, but this will vary depending on how your lambda is packaged.

@nateprewitt This is perfect. Thanks so much Nate for the quick response!

sorry if someone already said this.
I started getting this error after installing awsebcli.
Steps to fix:

pip3 uninstall awsebcli
pip3 uninstall awscli
pip3 uninstall botocore s3transfer
pip3 install awscli

saw same error on mac when using aws eks update-kubeconfig ... cli, and below steps fixed it for me on mac:
$ aws eks update-kubeconfig --name <AWS EKS cluster name> --region <AWS region>
Traceback (most recent call last):
  File "/usr/local/bin/aws", line 27, in <module>
    sys.exit(main())
  File "/usr/local/bin/aws", line 23, in main
    return awscli.clidriver.main()
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/clidriver.py", line 69, in main
    driver = create_clidriver()
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/clidriver.py", line 78, in create_clidriver
    load_plugins(session.full_config.get('plugins', {}),
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/plugin.py", line 44, in load_plugins
    modules = _import_plugins(plugin_mapping)
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/plugin.py", line 61, in _import_plugins
    module = __import__(path, fromlist=[module])
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/handlers.py", line 78, in <module>
    from awscli.customizations.s3.s3 import s3_plugin_initialize
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/customizations/s3/s3.py", line 15, in <module>
    from awscli.customizations.s3.subcommands import ListCommand, WebsiteCommand, \
  File "/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/awscli/customizations/s3/subcommands.py", line 18, in <module>
    from botocore.utils import is_s3express_bucket, ensure_boolean
ImportError: cannot import name 'is_s3express_bucket' from 'botocore.utils' (/Users/vhosakot/Library/Python/3.9/lib/python/site-packages/botocore/utils.py)

# saw above error with below versions
$ pip3 list | grep 'aws\|boto\|s3'
awscli                     1.32.80
boto3                      1.28.38
botocore                   1.31.85
s3transfer                 0.6.2

below steps fixed it for me:
pip3 uninstall -y awscli boto3 botocore s3transfer
pip3 install awscli

# below command worked for me
aws eks update-kubeconfig --name <AWS EKS cluster name> --region <AWS region>


Prisma 5 compatibility

I forked this repo for my own personal use case and stumbled upon some issues when attempting to upgrade to prisma 5.1.1
Just wanted to leave some notes here of findings thus far.
Feel free to take it over.

[...]CountOutputTypeArgs types have been renamed e.g.
AddressCountOutputTypeArgs becomes AddressCountOutputTypeDefaultArgs
Added this after the isAggregateInputType check in Transformer.generateExportObjectSchemaStatement
    if (name.endsWith('CountOutputTypeArgs')) {
      name = name.replace(/Args$/, 'DefaultArgs')
    }

The one I got stuck on after that was the [...]WhereUniqueInput types.
So where previously a type might look like
  export type AssetWhereUniqueInput = {
    id?: number
    uuid?: string
    checksum?: string
  }

Is now wrapped with a AtLeast<> type utility that requires at least one of the values to be present instead of them all being optional. This conflicts with the the ZodType argument in these type of files:
const Schema: z.ZodType<Prisma.AssetWhereUniqueInput> =
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

I narrowed it down to the fact that extendedWhereUnique is enabled by default in prisma 5.
Here's a related issue to watch: https://github.com/prisma/prisma/issues/20619
I don't know if there's an answer to this with zod?

I don't fully understand the code and how it's all laid out so I got a bit overwhelmed and tapped out, but hopefully this helps if you decide to keep supporting this package (which you should! it's great!)

Found a potential solution to Prisma's AtLeast type helper in zod:
https://stackoverflow.com/a/73294947
Will try and implement it when I get time, leaving here for later.


performance improvement for large drop downs

Our site has some drop downs with a ridiculous number of options in them (over 1,000, don't ask).
The current implementation requires a complete iteration of all options more than once to return a response.
This implementation will return a true response as soon as it finds it, and doesn't add much overhead for iterating through everything to give a false response if it isn't found.

Seems fine to me, though I'm a little confused with matched vs matches variable names, though I didn't come up with better naming.

Yeah I'm going to change matches to found_match I think. Thanks.


Opt refactor and opt time round floor function

Proposed changes
Before：
 [testdb]>select hour_floor(`@timestamp`, 4) as t, count() as cnt from httplogs_date group by t order by t limit 10;
+---------------------+--------+
| t                   | cnt    |
+---------------------+--------+
| 1998-05-01 00:00:00 |    324 |
| 1998-05-01 04:00:00 | 174137 |
| 1998-05-01 08:00:00 | 140461 |
| 1998-05-01 12:00:00 | 145945 |
| 1998-05-01 16:00:00 | 213248 |
| 1998-05-01 20:00:00 | 293426 |
| 1998-05-02 00:00:00 | 226038 |
| 1998-05-02 04:00:00 | 119502 |
| 1998-05-02 08:00:00 | 101933 |
| 1998-05-02 12:00:00 |  95275 |
+---------------------+--------+
10 rows in set (2.97 sec)

after:
 [testdb]>select hour_floor(`@timestamp`, 4) as t, count() as cnt from httplogs_date group by t order by t limit 10;
+---------------------+--------+
| t                   | cnt    |
+---------------------+--------+
| 1998-05-01 00:00:00 |    324 |
| 1998-05-01 04:00:00 | 174137 |
| 1998-05-01 08:00:00 | 140461 |
| 1998-05-01 12:00:00 | 145945 |
| 1998-05-01 16:00:00 | 213248 |
| 1998-05-01 20:00:00 | 293426 |
| 1998-05-02 00:00:00 | 226038 |
| 1998-05-02 04:00:00 | 119502 |
| 1998-05-02 08:00:00 | 101933 |
| 1998-05-02 12:00:00 |  95275 |
+---------------------+--------+
10 rows in set (0.67 sec)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Use xi-unicode for language agnostic line breaking

xi-unicode is a crate that implements Unicode line breaking standards. Line breaking by space has limits (mostly on Asian languages), so we should use a well standardized implementation instead.

Good idea, I'll look into it.

For version 0.2 I've replaced my previous trait responsible for this with LineBreaker and added a default implementation StandardLineBreaker which is backed by the xi-unicode iterator.
So now the default line break logic is UAX#14, whilst it can be custom implemented if desired.
Please re-open if the implementation seems no good!


Cannot read property 'projectName' of undefined

Actual Behavior
Hover + alt does nothing.
Error in GraphQL Language server:
[Error - 12:01:36] Request textDocument/hover failed.
  Message: Request textDocument/hover failed with message: Cannot read property 'projectName' of undefined
  Code: -32603

Expected Behavior
Peeks…?
Steps to Reproduce the Problem Or Description
Installed as per instructions. I have this config:
{
  "schemaPath": "src/schema/*.graphql",
  "includes": ["database/*.graphql"],
  "extensions": {
    "endpoints": {
      "dev": "http://localhost:4000"
    }
  }
}

These are my schemas:
database/enums.graphql
database/types.graphql
src/generated/prisma/prisma.graphql
src/schema/directives.graphql
src/schema/enums.graphql
src/schema/index.graphql
src/schema/inputs.graphql
src/schema/types.graphql

Note that './database/*` is the source that generates prisma.graphql.
In src/schema/index.graphql I also import schemas as per:
# import * from '../generated/prisma/prisma.graphql'
# import * from './inputs.graphql'
# import * from './directives.graphql'
# import * from './types.graphql'
# import * from './enums.graphql'

In addition, symbol lookup doesn't work (when I hit cmd+r) it's just empty.
Specifications

GraphQL for VSCode Extension Version: 0.1.7
VSCode Version: 1.33.1
OS Name: Mac
OS Version: 10.13.6

Logs Of TS Server || GraphQL Language Service
[Error - 12:01:36] Request textDocument/hover failed.
  Message: Request textDocument/hover failed with message: Cannot read property 'projectName' of undefined
  Code: -32603

@remy You've probably moved on by now, but I found the issue.
includes should be include (not plural).
That's it.

Now in graphql-config@3, documents is what we use for this, though i think there is still legacy support for include. please see the graphql-config docs or readme for updated graphql-config@3 config examples or other issues with this!


Routes, resources & urls -> Centralised Services

This is going to be a catch-all issue for a whole bunch of refactoring I'm working on.
Below are 3 specific problems I want to solve, and 3 specific use cases I hope will be achievable at the end.
Note: The goal is to move towards these things being possible internally & in internal apps to see what sort of structures/APIs we end up with. Once the refactoring work is done, we'll address exposing functionality in other ways.
Problems:

Resource <-> URL 2 way generation E.g. what is the url for a post? which post does a url represent?
Easy way to generate sitemaps, or other lists of urls, with meta data
Having routing be enumerable, and then configurable, extensible, etc

3 main use cases we currently cannot support

custom home pages
rendering a resource at a non-standard URL, e.g. /tag/recipes/ -> /recipes/
custom routes and their meta data, e.g. whether they should be in the sitemap, E.g. /subscribe/, /featured/, /recipes/, /private/, /*/amp/.

The expected outcome of this refactoring work is that we will have new services which control the ability to register routes, describe what happens at that route, and provide meta data about the route. Also that can then manage individual URLs, when they are added or removed from the system, and track some meta data about them.
Rough task list:

[x] Apps, register routes
[x] Centralised concept of what routes are available
[ ] Centralised concept of resources
[ ] Centralised concept of controllers
[ ] Centralised concept of renderers
[ ] Improve url generation to be driven from shared config
[ ] Separate out the "url generation" from sitemaps -> url service
[ ] Ability to manage urls: add/edit/remove + events
[ ] 2 way generation resource <-> url
[ ] Decorate routes, urls, controllers etc with meta data they need
[ ] url utils moved into service
[ ] App routes: when are they enabled? -> feature service?!

Then:

[ ] Rewrite sitemap code
[ ] Revisit controllers, how can we generalise them?
[ ] Revisit full configuration and API, what else can be improved?
[ ] Tags & Authors get urls in JSON API responses

Notes:

ensure we also refactor post.published events to never trigger on import as part of this.
get rid of the weird secure/setRequestIsSecure everywhere (was #6593)
figure out how route keywords fit into the route system (was #8492)
improve what is included in sitemaps (revisit ideas in #4588)

Closing in favour of #9601


I don't know how to uninstall

I used this gem in the beginning.
However, it is no longer used as a function.
Currently there are only tables and records in the DB (it's okay to delete them).
The installation method is written, but I don't know the procedure to uninstall it.
I would like to know the procedure for uninstalling.
Thank you.

Try rake db:rollback STEP=n

Same issue


Embed built-in pipelines

Melange's built-in pipelines (e.g., fetch, patch, etc.) are defined here, and copied into /usr/share/melange/pipelines when you run make install, or available in that location when you use the Melange container image.
To make it easier to use Melange without considering shared state on a filesystem, and to avoid hacks where one user updates /usr/share/melange/pipelines/fetch.yaml to mine crypto or whatever, we should embed these pipeline definitions at build-time using go:embed.
We might want to have a release or two where both the shared dir and compiled-in embedded definitions are considered to avoid breaking, then remove support for /usr/share/melange/pipelines in a subsequent release.

Since after this work the contents of the pipeline aren't as easy to inspect as before, maybe we should add a followup item to add a command like melange describe pipeline autoconf/make, that resolves the pipeline's location in --pipeline-dir or embedded, and prints it. Maybe with some indication of where it came from printed to stderr.


Lesson Hacks and Notes

Hacks
Notes

I like your notes, vocab, and blog in general.    I like the recap table of the things you did each week, just missing the last couple.  There were a couple of things that could use a little clean up on blog.

"search" and "about" are usually to the right of a menu
there is one item in submenu that 404's
vocab on menu seems redundant and less interesting that vocab you just did.

It seems you have the ability to organize data, that should be important in weeks to come.


The "Contributions" link from Statistics page is not displayed in AMO-prod

Steps to reproduce:

Load Statistics page for an add-on in AMO-prod https://addons.mozilla.org/En-US/firefox/addon/firebug/statistics/?last=30
See the "Contributions" link

Expected results:
The link is displayed (and usable) under the "by Add-ons Status" link.
Actual results:
The link is not displayed.
Notes/Issues:
Verified on FF43(Win 7). Issue is reproducing on AMO-prod.
Screenshot for this issue:

Access to Contributions is controlled by a different permission you don't have for firebug on prod.


Relative to cursor position

Add a way to only grab classes relative to the current cursor position? maybe as a second key binding
Example code:

 ||

            Home

            About Us

            Contact Us

and the cursor is at the || and then run eCSStractor the expected result would be
.main_nav {
	&__item {
	}
	&__link {
	}
}

As oppose to returning all classes.
Thanks.

You can execute standard command “Expand Selection to Tag” (Cmd+Shift-A) or “Match Tag Pair Outward” (Ctrl+D) if you have Emmet installed. Then run eCSStractor.

Yes I just noticed you can run it on selected code when I was reading through code, that's great, thanks.


[feature] Rename origin

Describe your idea
Currently there is no way to tell who initiated a rename for a field/class/method node, and in particular there is no way to know if it is automatic or manual (and of course, in case it is automatic no way to tell by which plugin).
I would love to see an "origin" or "reason" field added (the rename reason now is optional and also not persistent - next reload deletes it).

@ykaridi sure, I plan to force rename reason apply for a long time, I hope will manage to implement it soon.
Also, this related to #1862: allow proposing and store/show several possible renames with provided sourceId/origin.


How to remove "schemas-upnp-org:device" from Notify Device Type Messages

I want to use RSSDP to get my car's charging Station discovered by a smart Meter. The problem is, the smart meter only listens to device Type NT: urn:schemas-simple-energy-management-protocol:device:Gateway:1 but RSSDP always adds schemas-upnp-org:device:

How can I get rid of it?

OMG I am so stupid... commented out the Line of DeviceTypeNamespace...

 Glad you got it figured out tho :)


Constant spamming of Error detected while processing function 144_TabEnterHandler[16]..SNR>144_RestoreNERDTreeViewIfPossible: E16: Invalid range: 2wincmd w

Expected behavior
Be able to change tabs with NERDTree open, with the keep NERDTree position across tabs option on, without this error.
Seems to be a re-surfacing of https://github.com/neomake/neomake/issues/350.
Steps to reproduce

set let g:nerdtree_tabs_synchronize_view=1 in neovim config.
Try and tab between tabs in different directories. Doesn't seem to always happen, but once it starts, it happens without fail switching between tabs of files in different directories in the same project.

Output from (verbose) NeomakeInfo
This command doesn't exist in my neovim? Will investigate and provide if needed.

The command :verbose NeomakeInfo does not work?
It would be helpful to have the logfile from when this starts to happen, and any custom settings you have.

I assume that 582ff9c3 fixed this.


Prune sname

This fixes a bug in prune whereby SNAMES of pruned variants were not retained and occasionally variants that should have been pruned were emitted.

Changes Unknown when pulling 3efa42586b72cff645cc46128898cc687c70b85c on ernfrid:prune_sname into ** on hall-lab:develop**.

:+1:  tests are for cool people!


Can't download to external storage

Hello,
rcloneExplorer seems to not be able to download to external storage (microSD card).
I have a 32 Go internal storage (named "sdcard" on rcloneExplorer, but it's not a microSD card) and my real external 64 Go microSD card is not detected.

Plus, two entries are flagged as "storages" but they are not. See screenshot below:

Phone: Xiaomi Redmi Note 3 Pro Prime (ARM64)
System: LineageOS 13 Official Stable build (Android 6.0.1_r81 with security patch level 2018-06-05).

Starting with Android KitKat, apps are required to use Storage Access Framework in order to freely write to external storage. On rcloneExplorer, rclone is in charge of saving files, so there's no way of using official Android API to save files. I had to use a hacky way to support saving to external storage. Apps can still save to external storage without using Storage Access Framework, but only to the app's folder. So this is exactly what rcloneExplorer does. It saves files to app's media folder on the external storage. This means that if you want to save your files to a specific folder on your external SD card, you'll have to use a file explorer to move them afterwards.
So looking at your last screenshot:

/sdcard is your internal storage
/storage/emulated/0/Android/media/ca.pkay.rcexplorer is also the internal storage. This entry should not be displayed here, and is a side effect of adding support for external storage. I'm trying to figure out a way not to display it here.
/storage/86E0-9033/Android/media/ca.pkay.rcexplorer this should be your external SD card

Have you tried selecting the third storage? Does it not work?

Selecting the third storage may work but it won't allow to sync / backup the DCIM folder (for instance) with a rclone-compatible remote. I'm ok to download to /storage/86E0-9033/Android/media/ca.pkay.rcexplorer and move everything afterwards, but not being able to sync my external SD card pictures / videos / files is a huge limitation. :smiley:

I really can't figure out a way around this. Since rclone doesn't use Storage Access Framework, it can only save to the media folder on any external SD card. :disappointed:
I looked at termux, if it handles external SD cards differently, but it also saves to the apps' data folder on external card.

Ok, thanks for your answer. I have opened an issue: https://github.com/ncw/rclone/issues/2468

I do not believe this issue is something that can be fixed in rclone either.
Here's a link that goes in detail what's required in order to have full write access to external SD card.
Basically it requires to launch an intent asking user for write access, checking the result, and then using the supplied Uri (Android doesn't return paths in a format that rclone uses) to write to external SD card using Storage Access Framework.
This means that rclone would need to use Android APIs, which in turn would require it to be written in Java (or Kotlin) and compiled for Android. Therefore I don't think it's something that can be easily addressed :disappointed:

So basically, if rcloneExplorer (written in Java) has external SD access, it means that rclone (written in Go) won't inherit? :disappointed: Ok, i understand now.


Rewrite tab system with ttk.Notebook

this can be done by adding a button when a new tab gets created instead of rebuilding the whole list

@Moosems what do you think about that?

I like it. When removing a tab, what if we just slide over the tabs rather than deleting them?

Im afraid that would be too complicated

@sumeshir26 @Moosems I have already started work on this. Will commit when its somewhat stable

@not-nef Good to know


Provide a way for debug adapters to register UI-hidden debug types

In order for a debug adapter to receive events for a debug type, that debug type needs to be registered as a contribution in its package.json in addition to the usual adapter descriptor factory.
In js-debug we have a rather esoteric use case where we want to have a debug type that's used internally (to inject a debug configuration from command code into the adapter) and not exposed to the user.
We could technically shim overload this into one of our existing debug types, this is a rather ugly solution which I'd rather avoid.

We could get the same effect by using an existing type + some hidden property flag, right?

It is. It'd work, but it somewhat reduces the soundness of our internal types. Instead of saying "this type of object is a request to launch a Node.js process", we'd say "this type of an object is a request to launch a Node.js process, unless __delegateId is set in which case it's not related to Node and instead injects an existing session".
Maybe this use case is weird enough that that would be better than adding more vscode API surface, though.

@connor4312 I've read the esoteric use case and I got the impression that we have already something very similar (that does not rely on a hidden debug type).
For the "Auto attach" feature VS Code has a built-in "companion" extension "debug-auto-launch" that implements the UI and detection logic. With this extension VS Code can be put into an "auto attach" mode where it watches for processes in debug mode running in the integrated terminal and starts new node-debug sessions if it detects any.
Since this functionality is implemented as a separate extension, we can have a UI (in the status bar) that is always on, but does not activate node-debug too early (e.g. on startup).
Could we try to generalise this functionality so that it covers the js-debug cases as well?

The mechanism of action for auto-attach and JS debug's debugger terminal is different. Auto-attach polls for child processes and connects Node processes that run with the --inspect flag. In debugger terminal, any Node.js process launched by the user or other terminal processes will cause a debug session to start without additional flags. This is powerful, for instance a user can simply run npm start without touching their existing files or configuration and get a full debugging experience.
The way this works is by setting NODE_OPTIONS to --require a bootloader script that forces the process into debug mode. When this bootloader script is required, it creates a companion process that proxies the CDP connection to a pipe server set up by the debug adapter. This is also how we can automatically connect to all child processes; by default they will inherit the current environment, get the bootloader, and through that call back to the debug adapter.
The existing auto-attach polling lets us easily create a 'normal' launch configuration containing a request to attach to a debug process on a specific port. However, js-debug does not know that a debuggable process exists until it's knocking on the door with a CDP connection. Because of this, it needs some way to inject the CDP connection it just received into a debug session--something the basic launch and attach configurations don't do, and something that will not be able to done except through this entrypoint. That's why I added the new 'delegate' launch config type to js-debug, and why I want to hide it from appearing to users.
I hope that makes a little more sense 🙂

No, I did not mean to directly use anything in "debug-auto-launch" for the auto-debug feature of js-debug.
But couldn't the "debug-auto-launch" extension (or another new extension created only for this purpose) not be used to "set the NODE_OPTIONS to --require a bootloader script" and "start the pipe server", and listen for the "knocking on the door with a CDP connection"?
I try to understand why we have to shoehorn this functionality into a fake debug session with a fake debug type?

I think that's a question from what I'm looking for here. Whether the debug terminal is launched from js-debug or a helper extension, someone needs to call vscode.debug.startDebugging to create a debug session.
For auto attach, can create an attach configuration very similar to what a user could normally write in their launch.json. There's an __autoAttach flag, but it's generally the same. For the delegate debug type, it does not fit into any of our existing launch or attach configurations; we don't need a port to connect to, program to run, or browser to boot, we just need a delegate ID. The code paths for providing configuration and handling the launch within the adapter are different than the existing Node.js  and Chrome handling.
I mentioned above, while it would be possible to overload an existing config and have some special-case logic to reroute the launch request, I would rather be able to register a distinct launch configuration type for delegate sessions.


Filling a filter with torrent titles currently in qbittorrent

Might be nice to be able to fill a filter with the strings of current torrents in qbittorrent, to help with setting up cross-seed.
Would only work when title based torrent matching is used, but that's probably fine for most users.
Happy to contribute back, just wanted to start a discussion about it first before pushing a PR

You can do this with https://github.com/kylesanderson/upgraderr

With filterUpdate?

Yeah - docs are on .com. use a post script in your client with curl. I really wanted this feature in omegabrr and couldn't wait any longer.

Awesome thx! I'll give it a go


skill checks

Skill checks should provide both standard and opposed results at the same time (that is, adding and subtracting the character's skill). Presently, they're only doing one kind.
This will require setting up our first roll card, which ordinarily wouldn't be part of an mvp build.

This is going to be more complicated than I initially thought, and it not actually required for an mvp build in the first place.


golangci-lint config is broken

$ golangci-lint run
ERRO Can't read config: can't read viper config: While parsing config: yaml: unmarshal errors:
  line 63: mapping key "maligned" already defined at line 57

my guess is that this is the result of an error while resolving merge conflicts. We don't want maligned configured twice.

Also confirmed the maligned config that was preserved matches tendermint/tendermint:
https://github.com/tendermint/tendermint/blob/main/.golangci.yml#L43-L44


resume management for solo headhunter

Dear sir, Is this software suitable for personal headhunter to management, quick filtering, notes and keywording job seekers' resumes?

No. This is just a basic implementation where people can upload their
resumes and recruiters can view all submitted resumes.
On Wed, Jul 19, 2023, 9:45 PM Golamoka @.***> wrote:

Dear sir, Is this software suitable for personal headhunter to management,
quick filtering, notes and keywording job seekers' resumes?
—
Reply to this email directly, view it on GitHub
https://github.com/jeffrypaul37/Online-Resume-System/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTSZCC4Z77EUKKVN6J2NXTXRAB23ANCNFSM6AAAAAA2QFHV3U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


support windows vnext

Windows support is out of date. This gets things mostly working with 0.63.5 but is waiting on at least
https://github.com/microsoft/react-native-windows/pull/6285 so that events get passed to JS correctly.
There's a little work to do to re-enable merging track metdata but I thought I'd get the PR started so someone that knows what they're doing can start to look at it.

thanks for the contribution @ryfow ! Adding  @dannyvv to take a look at the C# module stuff

closing this now as we've merged in #1159 #1180 and #1204 to go into the v2 release.


Mobile display of editor

Please change display of preprint and review editor vertically in mobile mode

Closing as complete!

@rudietuesdays Thank you for doing this! I did notice the preprint doesn't scroll past the first page on the mobile phone.

Hi @murkatr ! I wasn't able to replicate this issue, but I did notice the PDF / fallback window was being cut off by the bottom reviews bar on mobile. I added some extra margin to the bottom and that should be fixed, but can you let me know if the issue persists and perhaps send a screenshot or two? Thanks!

@rudietuesdays I'm looking at this now. It seems that on the mobile only the abstract is visible as you scroll down from the preprint editor header. I'm cool with that. One thing I found though is that when I click on read more for the longform/full PREreview, it opens it on the side instead of below.

@dasaderi Thanks for catching this styling issue! I hadn't caught the bug on the longform  (now full) review reader, and updated the mobile styles for the rapid review reader as well.
As far as seeing the abstract, it sounds like that's your mobile device choosing how to handle the PDF. My phone does the same -- since the PDF is so large, it reverts to the fallback we've created, which is the base information including the abstract, and a link to the full PDF directly if the user chooses to visit it. This is better than forcing the PDF or load or error out, which would mean the user could be facing extremely slow load speeds (especially over data) or not receiving information from the fallback and instead being left with an error.
Let me know if you have questions about this, otherwise the fix for the "read more" is up on the dev site!

@rudietuesdays, thanks for fixing that bug and for the explanation. It makes total sense. I'll close this issue as that is fixed.