Updated PlasmoPlots to work with v0.6.0+

PlasmoPlots had not been updated to work with the new release  of Plasmo (v0.6). I updated the source code to handle the new version, including upding some of the function calls (e.g., num_all_variables is replaced by num_variables or replaced getnodes with local_nodes). I also updated the label calls (which need to access label.x instead of label since label.x results in BaseRefValue{Symbol} being included in the label) and the call to the clique projection. The tests and Project.toml files have been updated.
I did not update the PlasmoPlots.jl version number in the Project.toml file, which I think should be updated to 0.2.0. @jalving do you normally do that as a separate PR, or should I have included that in this PR?

I did not see that there was already another PR open (#5) for this same thing. I am going to close this one and instead look at that other one. I see the tests here work, but the tests on #5 are failing, so I will take a look and see if I can figure out why that is.  Apologies for duplicating the PR.

@dlcole3 I forgot I had opened this. If your implementation works, we can close the other one in favor of this.
The version number can be updated here or afterwards. It just has to get incremented before we register.

@dlcole3 I forgot I had opened the other PR. If your implementation works, we can close the other one in favor of this. The version number can be updated here or afterwards. It just has to get incremented before we register.

Sounds good. I copied over the examples and tests from your PR (#5) and followed a couple of the formatting things from your source code. On my local machine, the tests all run, but the overlapping matrix_plot example does not work as I expected. I will try to look into the code more tomorrow. layout_plot works as expected, and matrix_plot seems to work okay for the non-overlapping case, but I have to go deeper in the code to address the overlapping problem I think. I believe the constraints are not tracked properly in the overlapping case because some of the node labels go below the xaxis of the plot. Was this a problem on the old version of PlasmoPlots as well? I see there was not a test for the overlapping case before.

overlapping used to work. i suspect issues have to do with erroneous tracking due to the v0.6 updates.

overlapping used to work. i suspect issues have to do with erroneous tracking due to the v0.6 updates.

It looks like it was not strictly an issue with the overlapping case. The num_constriants(node) was not using the include_variable_constraints argument, so it was counting more node constraints than were initially being counted for the graph. In addition, plotting the link constraints for the non overlapping case was calling all_link_constraints instead of local_link_constraints. For the overlapping case, it also offset the column by 1 (it started at 1 instead of 0), which does not appear to be consistent with the non-overlapping case, and this resulted in there being one node that was being placed outside the plot. The plots now look about like I expected for the three examples. I did plot the three examples with the old version of Plasmo/PlasmoPlots, and the matrix plots look the same.

For the overlapping case, it does look like both the old and new (i.e., this PR) versions of PlasmoPlots do not directly overlap the matrix_plot. For instance, the plot from example 3 looks like this

which says it has 33 variables listed on the x axis. The graph technically only has 19 variables on it (the additional 14 come from the overlap). Is that the expected behavior from plotting? It seems like there is an alternative visualization where the x-axis only has 19 variables on it. However, the current approach seems okay because it highlights the variables that are "overlapping"

Is that the expected behavior from plotting? It seems like there is an alternative visualization where the x-axis only has 19 variables on it. However, the current approach seems okay because it highlights the variables that are "overlapping"

Yes, IIRC, I never produced a visual that directly overlaps the matrix plot. I think this version was meant to show the 'size' of the overlapped problems, but a direct representation would also be nice to have.

The num_constriants(node) was not using the include_variable_constraints argument, so it was counting more node constraints than were initially being counted for the graph

I think Plasmo needs to change the default such that include_variable_constraints is always false. I believe JuMP makes that their default as well.


Suppress logging for specific requests (e.g. health checks)

We're using ASP.NET Core with Kestrel dockerized in Google Container Engine. The load balancer performs regular health checks (to a customizable route) which, depending on the configuration, can make the logs somewhat to extremely unreadable. That is, we end up with up to hundreds of log lines per second using the standard console logger (possible machines x replication factor x each line a log entry) - without any useful information.
Currently, each requests generates at least some lines similar to
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://localhost:12345/foo/bar/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 1962.8502ms 200 application/json; charset=utf-8

While I was able to suppress all other messages by writing a custom middleware, I would like to also suppress these log lines if and only if the request comes from a specific IP subnet and/or has known HTTP headers set.
For requests different from health checks, I do want to see log output though (e.g. from Microsoft.AspNetCore.Mvc.Internal, etc.).

This issue was moved to aspnet/Logging#459


Links to Welsh versions of content should be understandable even when out of context

This issue is from May 2023 external accessibility audit report.
WCAG Reference:
2.4.9 Link Purpose (Link Only) (Level AAA)
Understanding Link Purpose (Link Only) | How to Meet Link Purpose (Link Only)
Issue ID:
DAC_Link_Purpose_Out_Of_Context_01
URLs:
https://signin.account.gov.uk/sign-in-or-create
Screen shots

The link provided for users to change the language of the page to Welsh, has been included within a paragraph to provide that context. However, for users navigating the page out of context using the link text alone, the links purpose is not made entirely clear, as the link text is only presented to the user as ‘In Welsh (Cymraeg)’.
Current code ref(s)
#main-content > div > div > div
Screen-reader comments

“When I moved out of context for example by using the JAWS links dialogue menu ‘Insert + f7’ I located a link which began with the phrase ‘In Welsh’. It was not immediately clear to me from the link text what would be displayed in Welsh.

When I moved in context with the down arrow key for example, I found that a piece of text advising me that the service could be displayed in Welsh appeared directly before the link but had not been included within the link text itself.

Including this text within the link text will enable me to make an informed selection both in and out of context.”

Solution
Ensure that links are descriptive enough when navigating out of context, allowing for users of screen reading assistive technologies to discern their purpose and functionality when doing so.
The link should include more text to clearly inform the user that it is used to change the language to Welsh.

We have merged the above PR today that includes the link purpose within the link text. Link text is now the English and Welsh equivalents of "use GOV.UK One Login in Welsh (Cymraeg)"

Hey @claireashworth, since we don't currently have a 'translation toggle link' component or pattern, I'm thinking guidance around how to word translation toggle links could sit in the 'links' portion of our typography style guidance?
It seems like a convention we should standardise, since it is a fairly important piece to get right from a usability and accessibility standpoint.
As an example, perhaps the guidance could say something like:

<h3> Translation toggle links </h3>
When a link is being used to guide people to different translations of a page or service's content, be sure that the link text contains enough context to be understandable on its own.
For example, the link can say "use [name of service] in Welsh (Cymraeg)" instead of just "Welsh (Cymraeg)".

Thoughts on this? It seems there's a few conversations to be had:

What standard translation toggle link text should say
Where the guidance should be placed, and in which format
What the surrounding guidance content should include

Need to share this finding with Head of Content, GOV.UK on Monday 21st August

I seem to recall having this conversation, but obviously did not record it here. We can probably reference the content communities guidance on publishing in Welsh as there's a rigorous process for translation and making sure that translation is findable. For example, one of the steps for adding Welsh content is:
3/ Add the standard call out
Add the following call outs so that users know the content is also available in English or Welsh.
For the Welsh version use:
^Mae'r canllaw hwn hefyd ar gael yn Saesneg (English).^
For the English version use:
^This guide is also available in Welsh (Cymraeg).^
For start pages use:
^Mae’r dudalen hon ar gael hefyd yn Saesneg (English).^
and
^This service is also available in Welsh (Cymraeg).^
Place the callout near the top of the first chapter. Most call outs are placed as the second paragraph so that users have a consistent way of knowing a different language is available. But you can use your judgement to decide exactly where to put it. If the callout gets in the way of the main call to action, you can put it further down the page.
I'll also share a presentation showing research on translations on gov.uk.

This issue can be resolved in the short term with the Links to meet WCAG 2.1 pull request #3044, which as guidance to the links sections of the Typography guidance.
The longer term, more comprehensive resolution would be to have more in-depth guidance, and potentially a new pattern or component for translations.
There seems to be a lot to take into consideration when expanding the link to increase the amount of text needed to be translated. More detail on Translations on GOV.UK and the surrounding complexities have been explored in this slide deck.


Restore tab indentation to LeftAndMain.php

See #6160

Current coverage is 70.30% (diff: 100%)

Merging #6162 into master will not change coverage

@@             master      #6162   diff @@
==========================================
  Files           418        418
  Lines         35960      35960
  Methods        4720       4720
  Messages          0          0
  Branches          0          0
==========================================
  Hits          25283      25283
  Misses        10677      10677
  Partials          0          0

Powered by Codecov. Last update 0d173f2...f130ebf


Dates

Use this issue to discuss the Dates pattern in the ONS Design System

It would be good to be able to set attributes on each input element to add things like autocomplete parameters.

The date input example should have a mode to render it as a question, where the legend is a H1 and the description is 1rem with the margin moved to below the description rather than between the legend and description. The individual field labels should also be bold.

The date input component wraps itself in a fieldset even if it's only configured to ask for one field (e.g. just the year field)

When the date component is configured to only show one field, the fields are incorrectly wrapped in a field-group

Users have fed back frustrations around the ability to add more than two or four characters in the date inputs.  This is especially frustrating for users who expect auto-tabbing, because they end up entering everything into the first input (which is too small to easily edit/delete)


Sanitize tokens in recorded tests for auth

We'll need to sanitize token values in both headers and message bodies in our tests once.
In the .NET test framework, I needed to add:

Sanitization for tokens in headers values
Sanitization for tokens in message bodies

For message bodies, .NET did not support sanitization of application/x-www-form-urlencoded content types.  This needed to be added specially to the ACR-specific sanitizer.
I don't know how the test framework works in other languages, but this issue represents the work to investigate and implement sanitization in recorded tests so we don't check in secrets in our test recordings.

This has been done.


mlc_llm package is ERROR: returned non-zero exit status[Bug]

🐛 Bug
I am running on the Android platform Qwen-7b-chat-q4f161-mlc This model
To Reproduce
Steps to reproduce the behavior:

add rust1.78
conda activate myName
python -m pip install --pre -U -f https://mlc.ai/wheels mlc-llm-nightly mlc-ai-nightly
git clone https://github.com/mlc-ai/mlc-llm.git
cd mlc-llm
git submodule update --init --recursive
cd android
add NDK
ANDROID_NDK: %HOME%/AppData/Local/Android/Sdk/ndk/25.2.9519653
TVM_NDK_CC: %ANDROID_NDK%/toolchains/llvm/prebuilt/windows-x86_64/bin/aarch64-linux-android24
mlc_package

MyErrorLog:
[2024-06-05 18:01:16] INFO package.py:327: MLC LLM HOME: "E:\project\mlc-llm-main"
[2024-06-05 18:01:16] INFO package.py:28: Clean up all directories under "dist\bundle"
[2024-06-05 18:01:16] INFO jit.py:43: MLC_JIT_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-06-05 18:01:16] INFO download_cache.py:227: Downloading model from HuggingFace: HF://mlc-ai/Qwen-7B-Chat-q4f16_1-MLC
[2024-06-05 18:01:16] INFO download_cache.py:29: MLC_DOWNLOAD_CACHE_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-06-05 18:01:16] INFO download_cache.py:166: Weights already downloaded: D:\Users\W9057210.ADC\AppData\Local\mlc_llm\model_weights\hf\mlc-ai\Qwen-7B-Chat-q4f16_1-MLC
[2024-06-05 18:01:16] INFO package.py:81: Model lib is not specified for model "Qwen-7B-Chat-q4f16_1-MLC". Now jit compile the model library.
[2024-06-05 18:01:16] INFO jit.py:160: Using cached model lib: D:\Users\W9057210.ADC\AppData\Local\mlc_llm\model_lib\e28f0896e747de1e51f09c8055e9061c.tar
[2024-06-05 18:01:16] INFO package.py:154: Dump the app config below to "dist\bundle\mlc-app-config.json":
{
"model_list": [
{
"model_id": "Qwen-7B-Chat-q4f16_1-MLC",
"model_lib": "qwen_q4f16_1_eab54620bed8dc427ee39dcd5a262599",
"model_url": "https://huggingface.co/mlc-ai/Qwen-7B-Chat-q4f16_1-MLC",
"estimated_vram_bytes":<PHONE_NUMBER>
}
]
}
[2024-06-05 18:01:16] INFO package.py:211: Creating lib from ['D:\Users\W9057210.ADC\AppData\Local\mlc_llm\model_lib\e28f0896e747de1e51f09c8055e9061c.tar']
[2024-06-05 18:01:16] INFO package.py:212: Validating the library dist\lib\libmodel_android.a
[2024-06-05 18:01:16] INFO package.py:213: List of available model libs packaged: ['qwen_q4f16_1_eab54620bed8dc427ee39dcd5a262599'], if we have '-' in the model_lib string, it will be turned into '_'
[2024-06-05 18:01:16] INFO package.py:256: Validation pass
[2024-06-05 18:01:16] INFO package.py:270: Moving "dist\lib\libmodel_android.a" to "build\lib\libmodel_android.a"
[2024-06-05 18:01:16] INFO package.py:274: Building mlc4j
info: component 'rust-std' for target 'aarch64-linux-android' is up to date
[2024-06-05 18:01:18] INFO prepare_libs.py:90: Entering "E:\project\mlc-llm-main\android\MLCChat\build" for MLC LLM and tvm4j build.
[2024-06-05 18:01:18] INFO prepare_libs.py:94: Set TVM_SOURCE_DIR to "E:\project\mlc-llm-main\3rdparty\tvm"
[2024-06-05 18:01:18] INFO prepare_libs.py:22: Running cmake
[2024-06-05 18:01:18] INFO prepare_libs.py:48: Using ninja in windows, make sure you installed ninja in conda
-- The C compiler identification is Clang 14.0.7
-- The CXX compiler identification is Clang 14.0.7
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: D:/Users/W9057210.ADC/AppData/Local/Android/Sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/windows-x86_64/bin/clang.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: D:/Users/W9057210.ADC/AppData/Local/Android/Sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/windows-x86_64/bin/clang++.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 (set):
Syntax error in cmake code at
E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1

when parsing string
${E:\project\mlc-llm-main\3rdparty\tvm}

Invalid character (':') in a variable name: 'E'
Call Stack (most recent call first):
E:/project/mlc-llm-main/CMakeLists.txt:12 (include)
-- Configuring incomplete, errors occurred!
Traceback (most recent call last):
File "E:\project\mlc-llm-main\android\mlc4j\prepare_libs.py", line 117, in
main(parsed.mlc_llm_source_dir)
File "E:\project\mlc-llm-main\android\mlc4j\prepare_libs.py", line 99, in main
run_cmake(mlc_llm_source_dir / "android" / "mlc4j")
File "E:\project\mlc-llm-main\android\mlc4j\prepare_libs.py", line 50, in run_cmake
subprocess.run(cmd, check=True, env=os.environ)
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\subprocess.py", line 571, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['cmake', 'E:\project\mlc-llm-main\android\mlc4j', '-DCMAKE_BUILD_TYPE=Release', '-DCMAKE_TOOLCHAIN_FILE=D:\Users\W9057210.ADC\AppData\Local\Android\Sdk\ndk\25.2.9519653\build\cmake\android.toolchain.cmake', '-DCMAKE_INSTALL_PREFIX=.', '-DCMAKE_CXX_FLAGS="-O3"', '-DANDROID_ABI=arm64-v8a', '-DANDROID_NATIVE_API_LEVEL=android-24', '-DANDROID_PLATFORM=android-24', '-DCMAKE_FIND_ROOT_PATH_MODE_PACKAGE=ON', '-DANDROID_STL=c++_static', '-DUSE_HEXAGON_SDK=OFF', '-DMLC_LLM_INSTALL_STATIC_LIB=ON', '-DCMAKE_SKIP_INSTALL_ALL_DEPENDENCY=ON', '-DUSE_OPENCL=ON', '-DUSE_OPENCL_ENABLE_HOST_PTR=ON', '-DUSE_CUSTOM_LOGGING=ON', '-G', 'Ninja']' returned non-zero exit status 1.
Traceback (most recent call last):
File "", line 198, in run_module_as_main
File "", line 88, in run_code
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Scripts\mlc_llm.exe_main.py", line 7, in
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\site-packages\mlc_llm_main.py", line 53, in main
cli.main(sys.argv[2:])
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\site-packages\mlc_llm\cli\package.py", line 64, in main
package(
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\site-packages\mlc_llm\interface\package.py", line 361, in package
build_android_binding(mlc_llm_source_dir, output)
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\site-packages\mlc_llm\interface\package.py", line 275, in build_android_binding
subprocess.run([sys.executable, mlc4j_path / "prepare_libs.py"], check=True, env=os.environ)
File "C:\ProgramData\anaconda3\envs\mlc-panghongtao\Lib\subprocess.py", line 571, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['C:\ProgramData\anaconda3\envs\mlc-panghongtao\python.exe', WindowsPath('E:/project/mlc-llm-main/android/mlc4j/prepare_libs.py')]' returned non-zero exit status 1.

can you print out the code in E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 and see what is the content there

can you print out the code in E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 and see what is the content there

just have this context:
set(TVM_SOURCE_DIR ${E:\project\mlc-llm-main\3rdparty\tvm})

Does this mean that my TVM Settings are wrong?

can you print out the code in E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 and see what is the content there

@tqchen just have this context:
set(TVM_SOURCE_DIR ${E:\project\mlc-llm-main\3rdparty\tvm})
Does this mean that my TVM Settings are wrong?
This directory E:\project\mlc-llm-main\3rdparty\tvm no have file

You need to execute a command first to download tvm and other dependencies.
cd mlc-llmgit submodule update --init --recursive

can you print out the code in E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 and see what is the content there

I also encountered a similar error on windows, as follows
set TVM_SOURCE_DIR=..\..\3rdparty\tvm
CMake Error at E:/CCode/mlc-llm/android/MLCChat/build/config.cmake:1 (set):
  Syntax error in cmake code at

    E:/CCode/mlc-llm/android/MLCChat/build/config.cmake:1

  when parsing string

    ${..\..\3rdparty\tvm}

  Invalid character escape '\3'.
Call Stack (most recent call first):
  E:/CCode/mlc-llm/CMakeLists.txt:12 (include)

set TVM_SOURCE_DIR= ../../3rdparty/tvm
-- TVM_SOURCE_DIR: ../../3rdparty/tvm
CMake Error at E:/CCode/mlc-llm/CMakeLists.txt:65 (add_subdirectory):
  add_subdirectory given source "../../3rdparty/tvm" which is not an existing
  directory.

can you print out the code in E:/project/mlc-llm-main/android/MLCChat/build/config.cmake:1 and see what is the content there

I also encountered a similar error on windows, as follows set TVM_SOURCE_DIR=....\3rdparty\tvm
CMake Error at E:/CCode/mlc-llm/android/MLCChat/build/config.cmake:1 (set):
  Syntax error in cmake code at

    E:/CCode/mlc-llm/android/MLCChat/build/config.cmake:1

  when parsing string

    ${..\..\3rdparty\tvm}

  Invalid character escape '\3'.
Call Stack (most recent call first):
  E:/CCode/mlc-llm/CMakeLists.txt:12 (include)

set TVM_SOURCE_DIR= ../../3rdparty/tvm
-- TVM_SOURCE_DIR: ../../3rdparty/tvm
CMake Error at E:/CCode/mlc-llm/CMakeLists.txt:65 (add_subdirectory):
  add_subdirectory given source "../../3rdparty/tvm" which is not an existing
  directory.

Hi PhilCuriosity。I've modified my code the way you did. Now I have the same problem as you. About tvm in Windows 10 path error, have you resolved? Is there a way to do that?

Sorry, I'm not sure how the mlc-llm command works, and I haven't solved the problem yet.

Sorry, I'm not sure how the mlc-llm command works, and I haven't solved the problem yet.

update mlc_llm code.This issue has been fixed. set TVM_SOURCE_DIR E:/project/mlc-llm/3rdparty/tvm  this format

Glad it works out

That worked. Thank you.


Error: Export from "app" must be a function that returns an HTML string

I am trying to use this library in combination with react router.
my webpack plugins has:
  entry: {
    app: ['src/app.js']
  },
  plugins: [
    new StaticSiteGeneratorPlugin('app', ['/']),
  ]

and then
// src/app.js

if (typeof document !== 'undefined') {
  // do client side rendering
}

const template = (htmlRenderedByReact, assets) => `
   <!doctype html>

      <meta charset="utf-8">
      <meta http-equiv="x-ua-compatible" content="ie=edge">
      <meta name="viewport" content="width=device-width, initial-scale=1">

${htmlRenderedByReact}

`;

// Exported static site renderer:
export default (locals, callback) => {
  const history = createMemoryHistory();
  const location = history.createLocation(locals.path);
  match({ routes, location }, (error, redirectLocation, renderProps) => {
    callback(null, template(
      ReactDOMServer.renderToString(<RoutingContext {...renderProps} />),
      locals.assets,
    ));
  });
};

however, I get the error Error: Export from "app" must be a function that returns an HTML string.

Bump, as I am having the same issue

How is your output defined?
  output: {
    filename: "index.js",
    path: '_site',
    libraryTarget: 'umd'
  },

libraryTarget needs to be umd or CommonJS.

Bump

Can we see your entire webpack config?

We've moved on to a more DIY method of static site generation, so my webpack config no longer helpful here. Perhaps @Travmatth's helps

Maybe @Travmatth should open their own issues and you (@richtier) should close this one.

https://github.com/markdalgleish/static-site-generator-webpack-plugin/issues/46


Improve question 33

This little change helps to remove confusion on the meaning of "output" as happened in #122.
The change was applied on every translation as well.

I'll close this PR now due to inactivity. If it's still an issue, feel free to reopen it! 😃

Hi @lydiahallie, what do you mean by inactivity? I was just waiting.


Add post on Pandas dataframe output for sklearn-transformers

Closes: #143
I've taken a screenshot(which depicts setting the configuration) from the video and used it as a feature image.

@SangamSwadiK Thanks for this PR.  Some updates to make:

the name of the project is "scikit-learn" (lower case, not uppercase)
Can you add a space between sentences (after fullstops)?

Thank you.

@SangamSwadiK
For tag, instead of "Sklearn-Transformers", let's use one that is there "performance"

Let's add in some text from this LI post:
https://www.linkedin.com/feed/update/urn:li:activity:6987027021608460289

We should include the thanks to the maintainers part, and link to each of their GitHub handles:

Thanks 🙏🏾 to maintainers: Thomas J. Fan, Guillaume Lemaitre , Christian Lorentzen !!

Thanks @SangamSwadiK !


BROOKLYN-513: fix coercion for jclouds reflective builder calls

Seems the method matching logic is already implemented in org.apache.commons.lang3.reflect.MethodUtils (commons-lang3) which we already have as a dependency, so might be better if we reuse that.

Scratch last comment - doesn't take coercion into account.

I've successfully tested it with a manual build. It covers the use case for specifying ipOptions in jclouds.

Thanks @neykov and @ygy - I'll incorporate some of those comments and then merge.

@neykov I've updated the PR. If you get a chance to review again, that would be great. Otherwise I'll wait for apache jenkins and then squash it into one commit and merge.

LGTM. Jenkins hasn't kicked in yet. Building locally and will merge if no errors.


Different output file names in different scripts

Hi!
The script 'extract_features.py' stores the CNN features and other aspects of the model as 'conv_features.npy' and 'image_counts.npy'. But, the modules 'load_country_lsms' and 'load_country_dhs' in 'fig_utils.py' seem to be looking for the files 'cluster_conv_features.npy' and 'cluster_image_counts.npy' and they are not being generated at any other point in the workflow. The same goes for 'nightlights.npy', 'consumptions.npy' and 'households.npy'. Am I missing something here, or are both the scripts supposed to be referring to the same file?
Thanks

Yes, these should be the same - thanks for catching that!


#86

https://zube.io/projects/3188/cards/174082196

This change is

PR is added to the queue for testing as 1. task. (18032ef)

Hi @vasilevskayaem, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

PR is added to the queue for testing as 2. task. (2d117e4)

Test (Fail) 83481184 build is completed (18032ef)
See https://github.com/ThaliTester/TestResults/tree/8348118418032ef__86_vasilevskayaem/ for the logs

Test (Fail) 83481184 build is completed (2d117e4)
See https://github.com/ThaliTester/TestResults/tree/834811842d117e4__86_vasilevskayaem/ for the logs


Determine XML variant for mimetype 'text/xml'

With mime sniffing, the XML variant is currently only determined for the mimetype application/xml.
RFC3023 also describes the mimetype text/xml, as well as other more exotic types.

If an XML document -- that is, the unprocessed, source XML document -- is readable by casual users, text/xml is preferable to application/xml.  MIME user agents (and web user agents) that do not have explicit support for text/xml will treat it as text/plain, for example, by displaying the XML MIME entity as plain text. Application/xml is preferable when the XML MIME entity is unreadable by casual users.

The determination is handled by python-magic, which is a wrapper fo the libmagic library.
As far as I've tested, the determination seems to be platform dependent (I guess based on the platform's mime type database and package version). For example, on a default Ubuntu 16.04 server, the mime type reported by libmagic for .xml files is application/xml, while on Ubuntu 20.04 text/xml is reported.
The merge request contains simple changes so that the call to get_xml_variant_including_xml_declaration() is performed for both mimetypes mentioned above.

Fixed in recent merge request #74


Update @opentelemetry/api to latest version

/azp run js - tables - ci

/azp run js - aggregate -reports

@markwolff FYI

Nice! I think the rush exception can be removed now too
https://github.com/Azure/azure-sdk-for-js/blob/4ac9823aab55a75f7408e91094a20fb057284451/common/config/rush/common-versions.json#L64

@markwolff There was a build error with openTelemetry-exporter, pushed https://github.com/Azure/azure-sdk-for-js/pull/10393/commits/6fc48a520c4e2aff77390c4413c5767f7879c8cb

Removed the rush exception in https://github.com/Azure/azure-sdk-for-js/pull/10393/commits/c56b4ad19206a9fdab683ed07e44d2e1e5cc87db


fix: Icon-dropdown overlap in button properties

Description
The border radius property of the Button widget overlaps the Icon dropdown. A small change in the padding in CSS of the input bar in the icon dropdown can be used to fix it.
Fixes #7575
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
NA
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Final result.

Do let me know if any other changes are to be done. The suggestions were of great help and documentation in the project is really good which helped me in setting up the project smoothly.

Hi @nishihere19, thanks for contributing :star_struck: . The Semantic Pull Request check is failing, this check ensures that PR titles are semantic and consistent. You have to add one of the prefixes listed here  (without the square brackets).
Kindly rename this PR, thank you.

Sure, will do.

Thanks @nishihere19! I've tested the DP and it looks good to me. @ohansFavour Please review the code and let's merge this if things look good.

@nishihere19 The padding bottom I suggested is creating issues in other controls. Can you find a way to fix this?

Sure, I didn't notice this. Will find an alternative and fix this properly.

Closing this as there is no activity.

Hi @jsartisan, I tried a lot to find a way to detect if the menu is for icon controls and apply CSS accordingly but couldn't find one. Could you guide me a little and tell me if there is any way to pass on the props in the IconSelectContainerStyles such that it checks for icon-controls and applies the padding?

Looks great. For some reason, the icon alignment is distorted. Can you check?

this looks good to me.
@somangshu can we merge this in as well?

Hi, I wanted to know if this PR will be considered for hacktoberfest.

/ok-to-test sha=3d3bd4c

Test result: https://github.com/appsmithorg/appsmith/runs/4152661366

Hi, I wanted to know if this PR will be considered for hacktoberfest.

Yes. This will be considered as part of Hacktoberfest as well.

@jsartisan just a reminder that you need to take this PR ahead!

Closing as it will get fixed in #9714


net: l2: ieee802154: Fix LL address handling on frame reception

6lowpan module can swap the original buffer with a newly allocated one
during decompression in case the decompressed header would not fit into
the original buffer. Therefore, storing the LL address offset and
restoring the pointer after decompression as it is done today is not
correct, as the new packet with decompressed IPv6 header will not
contain the LL header.
As the 6lowpan module doesn't deallocate the original buffer and
doesn't overwrite the LL header, its fine to use the original
pointers as they are.
Fixes #36172
Signed-off-by: Robert Lubos<EMAIL_ADDRESS>

CC @niflostancu

Yep, that's the workaround I've been working with. Thanks!


pick 3 bugs export csv

pick 3 bugs for Export CSV from the Find Your Bugs doc
detail which 3 bugs you are fixing in the PR comments
closes #170

Bug #36: Export CSV: Boxes are not sorted chronologically**
Bug #16: Export CSV: All box details should be checked on page load
Bug #45: Export CSVView Launched Organically Checkmark isn’t Centered


cloverAggregateDatabases is not working

HI, I have project, which contains multiple sub-projects, each sub-project has source set and test set, clover is able to instrument classes with individual sub-project and generate code coverage within the same sub-project, problem here is my 90% test cases i have written in one particular sub-project, gradle is having reference to other sub-projects. While generating code coverage for whole project, i'm observing that my code coverage is completely wrong(comparing against intelliji clover plugin).  I was thinking that gradle-clover-plugin is not working as expected, how do i fix this issue?

how do i pass each sub-projects clover.db files to each sub-projects?
Any other alternatives suggestions would be welcomed.

here is log: i do have multitple .db file, but its not picking up to merge. what parameters i need to mention, so that i can pick up the other .db files.
[ant:clover-merge] Merging database 1 of 1: /Users/user/workspace/projectb/projectb-test/build/.clover/clover.db-test

example:
project {
project A {
source set
test set(having minimal tests)
}
project B {
source set
test set(having minimal tests)
}
project C {
source set
test set(maxinum test cases resides here)
}
}

The cloverAggregateReports task is used to correctly merge and report on multi-project configurations. However, you are looking for indirect code coverage which may not work because of the way the plugin instruments the code just in time to run unit tests. One option that may give you better results is to try passing the property -PcloverInstrumentedJar=true on the command line. This will allow the classes from projectA and projectB to be left in instrumented state. That should generate the coverage recordings you are looking.
I tried the above but I am not getting the extra coverage in a test project so this may not work either. I tend to avoid indirect code coverage in my projects, I always make the unit tests closest to the unit under test provide the needed coverage.

For what it is worth, in my experiments the cloverAggregateReports task correctly shows multiple databases getting merged and the aggregate report shows all relevant coverage from each project in one report.

I do not consider testing indirectly or in a different module from the one the code is building from a good practice. This works as intended when direct testing is applied which always gives the best kind of coverage. I am closing as "won't fix" since I find this is not a desirable feature.


Add new experiment status records instead of overwriting previous records

Supun, can you review this and see if this can be safely merged onto 0.15 branch?

@scnakandala can we incorporate this feature in new registry design

Yes. We can do that @smarru


AM2315 and more I2C devices - problems

Hi Rob,
The AM2315-library works good, also when I use other I2C-pins, in my case SCL (19) and SDA (5).
Problems come up, with a BME280 and connected, in my case, to SCL 18 and SDA 23.
The AM2315 is still ok, but the BME280 is giving weird values as you see below:

For the BME280 I'm using these Adafruit-libraries.
  #include <Wire.h>
  #include <Adafruit_Sensor.h>
  #include <Adafruit_BME280.h>

Attached an extract of my sketch showing these problems.
#define enableAM2315 false     // if true then weird values from BME280
Set this compiler directive to true and see the output in Serial monitor
Thanks,
Jop
Multiple_-_AM2315_en_BME280.zip

If the BME is used stand alone does it give faulty errors?

#include "Switch.h"
^~~~~~~~~~
compilation terminated.
Which library is this?

created a minimal version with these 2 sensors - can you give it a try?
weather_station.zip

Done some reading about BME280 and causes found so far is (1) wiring and (2) incorrect calibration data.
I am working a lot with the MS5611 last couple of days - https://www.amazon.de/gp/product/B07Y1QZKF4/
https://github.com/RobTillaart/MS5611 - library is pretty stable, last release will be tested tomorrow.

#include "Switch.h" ^~~~~~~~~~ compilation terminated.
Which library is this?

library I discovered here:
TTGO ESP32 MINI
this is also the ESP32 I use.
Because of possibility external Wifi-antenna. (this is really a problem - to find such ESP32 boards)

If the BME is used stand alone does it give faulty errors?

No - if I leave out by
#define enableAM2315 false
no weird values from BME280
I disabled also the power to the AM2315

mmm could indeed be a power problem.
I would test the following.

Can you connect the power of the sensors to the VIN pin of the ESP?
Then it will be directly powered from the power supply.
(dont know if the BME280 is a 5V device, if not stick to 3V3

Can you add a delay of 10 seconds between reading teh AM2315 and the BME280?

Can you change the order in which the sensors are read?

created a minimal version with these 2 sensors - can you give it a try?
weather_station.zip

ok, but what pins you planned me to use as well voor the BME280 as for the AM2315?
(no info about that in the sketch you've sent me)

mmm could indeed be a power problem.
I would test the following.

Can you connect the power of the sensors to the VIN pin of the ESP?
Then it will be directly powered from the power supply.
(dont know if the BME280 is a 5V device, if not stick to 3V3
Can you add a delay of 10 seconds between reading teh AM2315 and the BME280?
Can you change the order in which the sensors are read?

Order of reading the sensors and increasing delay between reads to f.e. 10 secs. doesn't help.
Still weird data read from BME280.
But here a new hint. I discovered that the orde of initializing AM2315 and BME280 in setup() makes a big difference.
When initializing the BME280 after the AM2315 the latter one is giving problems with weird data.
Here the output:

Here relevant code in setup() as it is now and giving results a above:
  #if enableAM2315
  initAM2315();
  #endif

  #if enableBME280
  initBME280();
  #endif

So when changing this order back to initBME before initAM2315 we read weird data from BME280.
I checked this twice.
Apparently the initializing code of the one is disturbing the code of the other and the other way around.
Do you have yourself a BME280 to try?

I assume both the BME and AM initialize the Wire object. If they do this with different pins one of the two devices becomes unreachable.
Solution is to have both devices on the same i2c pins.
Other possible point of attention is to keep the i2c clock to 100 KHz. Think that is not the problem.
I don't know if I have a BME280 around, do not recall seeing it recently.

Hi Rob,
Here is the solution and we solved it together. For the other pins a second Wire-object should be initialized. And this object (Wire1) is available. I forgot this a bit, but my previous code was implementing this already for an BMP280 aside of the BME280. See also the links in the comments coming from Random Nerd Tuturials.
The initBME280 in the code I've sent you should be extended.
Now part in this code is:
`
   Wire.begin(BME280_SDA_PIN, BME280_SCL_PIN);

                                                // NB: niet Wire1()  !!
                                                // default settings
                                                // (you can also pass in a Wire library object like &Wire2)
                                                //unsigned statusBME = bme.begin(&Wire);
    unsigned statusBME = bme.begin(0x76);
            // see: https://randomnerdtutorials.com/esp32-i2c-communication-arduino-ide/#7
            // and: https://randomnerdtutorials.com/esp32-pinout-reference-gpios/

change it in this code:
#if enableAM2315
Wire1.begin(BME280_SDA_PIN, BME280_SCL_PIN);
unsigned statusBME = bme.begin(0x76, &Wire1);
#else
Wire.begin(BME280_SDA_PIN, BME280_SCL_PIN);  // NB: niet Wire1()  !!
                                          // default settings
                                          // (you can also pass in a Wire library object like &Wire2)
                                          //unsigned statusBME = bme.begin(&Wire);
unsigned statusBME = bme.begin(0x76);
#endif
        // see: https://randomnerdtutorials.com/esp32-i2c-communication-arduino-ide/#7
        // and: https://randomnerdtutorials.com/esp32-pinout-reference-gpios/

This runs ok. You should know I'm a novice in the area of I2C and Wire.
Thnx for all your support. I think we can close this issue.
Have a nice day,
 Jop

Good to hear it is solved.
Do you have a reason why not to use the same I2C pins for the 2 devices?
It can handle up to ~100 devices on 2 pins.

https://randomnerdtutorials.com/esp32-i2c-communication-arduino-ide

Good explanation.

It's a good idea to use the same I2C pins for these 2 devices: AM2315 and BME280.
After some struggle (finding the I2C-address of this AM2315 with an i2c scanner  sketch working bad for unknown reasons) I managed to get this done.
But I read now that the AM2315 needs two pull-up resistors 2K-10K on SCL and SDA.
https://learn.adafruit.com/am2315-encased-i2c-temperature-humidity-sensor/arduino-code
That's not a problem when connecting a BME280 on same pins?

I am very pragmatic with pull ups, standard one set of 4K7 on the whole I2C bus. If the line is longer than a few meter it drops e,g 1K at 5-8meter. If you need longer wires other tech is needed to be robust.
You need to check is if there are already pull ups on e.g. a breadboard.
Tip: https://www.amazon.nl/Practical-Electronics-Inventors-Paul-Scherz/dp/1259587541  (cheaper in germany)

Thnx for the hint.
About connecting these 2 I2C-devices on one set of pins is working good on SDA 23 and SCL 18 and I can use either Wire or Wire1. But I can NOT get this don - the same trick - with the other set of pins SDA 5 and SCL 19, I originally planned to use for the AM2315. (so connecting both devices  on these pins).

Think it can be done if you initialize thing in right order (been there struggling)
From my head:

associate Wire with the pins you want.
associate devices with Wire without assigning pins again

#include "Wire.h"
#include "AM2315.h"
#include "Adafruit_Sensor.h"
#include "Adafruit_BME280.h"

Adafruit_BME280 bme;
AM2315 sensor(&Wire);     // connect to Wire

void setup()
{
  Serial.begin(115200);

  Wire.begin(5, 19);     // assign pins of Wire

  sensor.begin();
  BME.begin(&Wire);     // connect to Wire

....
}

Should also work for Wire1

mmm,
BME.begin(&Wire) calls init which calls Wire.begin() again.
seems a bit more tricky ...

The constructor for bme is bme.begin( 0x76. [, &] ).  // address: 0x76 or 0x77
But what you gave above is working, as well for Wire as for Wire1
Still don't know what was wrong with my implementation. But it's ok.

Still don't know what was wrong with my implementation. But it's ok.

It looks a bit confusing with all the conditional ifdefs.
Write for maintainability, just imagine you have to understand your code 6 months from now. That is one important lesson I learned in Computer Science :)

I understand quite well what you wanna say.
It has pros and cons. Leaving out the ifdefs could mean I have to split up in different sketches for creating different configurations. In that case the total overview is is suffering more and I have to maintain more different sketches.
And I'm lucky with having a rather good memory, so even after a long time I easily remember why I did it like I did.
Anyway, the code is meant for myself... ;)


Change default log configuration

This PR changes the install page and the docker default
logging configuration to match the suggested configuration
that I repeatedly end up suggesting on issues.
It further improves the logging configuration docs to
recommend specific instructions for how to configure logs
for posting to issues.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #13088 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #13088      +/-   ##
==========================================
- Coverage   42.61%   42.61%   -0.01%
==========================================
  Files         673      673
  Lines       73858    73862       +4
==========================================
- Hits        31477    31475       -2
- Misses      37271    37277       +6
  Partials     5110     5110

Impacted Files
Coverage Δ

routers/install.go
0.00% <0.00%> (ø)

modules/log/event.go
59.90% <0.00%> (-0.95%)
:arrow_down:

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 1c523e2...83a5495. Read the comment docs.


Remove sklearn dependency and run training tests on CI

Replaces training on polynomial features with training on random linear data and spiral classification data. Also makes pytype linting work with deleted files, and excludes failing targets from tap in iree_e2e_cartesian_product_test_suite.

I'm a bit worried about having entire training runs running on a regular CI like TAP or our Kokoro setup. How repeatable are these? How fast are they? I guess size defaults to medium, so they can't be that slow?

The models are tiny, so unless IREE does something horrifically inefficient during the backward pass they'll all complete in a few seconds. On tf they passin in about 7 seconds, which is on par with the rest of the e2e tests.

For reference this is what the spiral dataset looks like:


[e2e]: go schedule main adversarial-build-provenance slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3081312414
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.adversarial-build-provenance.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.adversarial-build-provenance.slsa3.yml
Trigger: schedule
Branch: main
Date: Mon Sep 19 09:31:59 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3088979045
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.adversarial-build-provenance.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.adversarial-build-provenance.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Sep 20 09:31:14 UTC 2022
Tests are passing now. Closing this issue.


Sentry fix for TypeError: t.entities.canvasWidgets[e.widgetId] is undefined

Description
Added checks for typeError
Fixes #6177
Type of change
Sentry Issue
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=93c60ed

/ok-to-test sha=dd5a4d8


Fix & redesign the new property add form.

Description
A clear and concise description of what the PR does.

This PR does the following:

Change the description from input field to textarea field.
And also arrange this Price, Location & Country field in one row.
Add a heading for this form & shift the button in right possition.

Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #124 "

Changes
List the detailed changes made in this PR.

Added a new feature to ... That changes input field to textarea field for description, Add a heading, And also make Price, Location & Country field in one row.

Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Soujanya2004   Please gave me label-2 for this if possible. It would be grate for this.
@Soujanya2004   Please marge This along with all the labels thats are in the ISSUE #124
Please add individuals points label for individual PR and marge previous PR first, it helps to track changes in better way @Soujanya2004

@Suvadip-sana It has conflicts please resolve

@Soujanya2004   Marge This. I resolved the conflict.
Please marge it after that I resolve the next conflict otherwise conflict arise again

@Soujanya2004  Please marge after that I am able to proceed further.


share revoke should not fail when a role is deleted

We've ran into multiple situations where our users delete a role which is actively used in data.all. We can detect this (with custom scripts) and data.all will let you revoke the S3 share via UI but it will fail trying to revoke a LakeFormation share with an error like:
Failed to get role ROLE_NAME due to: An error occurred (NoSuchEntity) when calling the GetRole operation: The role with name ROLE_NAME cannot be found.
This can only be fixed by manually updating the share to FAILED in RDS and then the share can be deleted via UI. This is very inconvenient because only data.all engineering team can fix such issues rather than making it a responsibility of the share owner.
I suspect that this issue also affects the "share validator" released in 2.3 and that it fails to fix a share when a role is deleted. I propose the following enhancement for this problem:
When attempting to revoke a share detect that a role has been deleted/missing and simply put the share into FAILED mode so that the share can be deleted via UI. This should also happen when share fixer is trying to fix the share either via manual request or when it runs in the background.
This fix is related to this issue https://github.com/data-dot-all/dataall/issues/1029. The above error was observed on 2.2 release.

Confirming re-apply shares wont work if role was deleted for a successful share:
I had a successful share, then I deleted the consumption role.
Verify shares, marked share as unhealthy with following errors:

When I hit re-apply shares, it does work and fails with error:

I think, the most logical way to handle this is:

Check the existence of the role during the healthcheck. Make it the first important statement in the result.
If during the healthcheck the role is missing, dataall must revoke all resource permissions and delete dataall managed policies, the share must become 'failed'
Revoke/Reapply share must lead to status 'Share failed', if there is no consumption role. Additionaly dataall must revoke all resource permissions and delete dataall managed policies
@zsaltys @dlpzx @anushka-singh what do you think?

Completed in #1161


Update ractive-decorators-select2.js

Current version doesn't work w/ up-to-date ractive

Thanks!

No problem! :)


Possible issue with Array.prototype.map or disjoint unions of array types

Array.prototype.map seems to fail if given a disjoint union of different array types
TypeScript Version:  2.2.2 (also tried 2.3.4. Reproducible on the Playground as well)
Code
function map<T>(items: T[] | string[]) {
    items.map(x => x)
}

Also fails:
function map<T>(items: Array<T> | Array<string>) {
    items.map(x => x)
}

Expected behavior:
Maybe this should work?
Actual behavior:
Type Error:
Cannot invoke an expression whose type lacks a call signature. Type '{ <U>(this: [string, string, string, string, string], callbackfn: (value: string, index: number, ...' has no compatible call signatures.

Workaround
The following does work:
function map<T extends object | string>(items: T[]) {
  items.map(x => x)
}

Perhaps not a bug since a workaround exists - I don't know if the workaround would be a preferred method?

Duplicate of https://github.com/Microsoft/TypeScript/issues/7294

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


I just facing the exception in my first running : Unhandled exception. System.BadlmageFormatException: An attempt was made to load a program with an incorrect format.

*My mac info :

M1 macbook pro
Sonoma 14.0
.Net 8 installed

How I fix that issue ?

Do you have .NET 6 installed also?

Do you have .NET 6 installed also?

Yes .NET6 Installed.

Next check that you have the ARM version of the .NET SDK installed. The error you reported typically means an assembly is not compatible with your processor architecture.

I installed new version 0.7.0, and it worked.


Chore: Command palette zhcn title

Ref: #186
其中一个效果图：

reviewed

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/0.2.0@72ec564). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/0.2.0     #199   +/-   ##
================================================
  Coverage                 ?   78.54%
================================================
  Files                    ?       25
  Lines                    ?      965
  Branches                 ?      187
================================================
  Hits                     ?      758
  Misses                   ?      207
  Partials                 ?        0

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 72ec564...e61fe5b. Read the comment docs.


Unknown option --output

I installed the latest from npm and followed the examples, without success (it looks like this page is for the upcoming 2.0 release already?).
$ npx @manifoldco/swagger-to-ts spec.yml --output schema.ts
error: unknown option '--output'

$ npx @manifoldco/swagger-to-ts spec.yml
[INFO]: 下载 Swagger..., apiUrl:
[ERROR]: 下载 Swagger 失败
Error: Expected a file path, URL, or object. Got undefined

@henhal thanks for opening an issue. Yes—there is a 2.0 release coming soon that’s almost ready but not quite. Maybe it works better for you?
Just add @next to the end of the library name like so:
$ npx @manifoldco/swagger-to-ts@next …

This is now in the latest version! We’ve rewritten the entire library for<EMAIL_ADDRESS>which includes OpenAPI 3.0 support and fixes a number of issues. Hopefully this should fix your problem! Please file another issue if it doesn’t.

Cool! I missed the previous comment, will try this out! 👍

I now get this to work, however with some limitations that kind of break it for me, so I will place another issue for you to consider.


Chapter 12, Typo Found in Learning Course

In the code block found in Chapter 12.4, there seems to be a typo error regarding the variable name. I believe the intended variable name should be FormSchema instead of InvoiceSchema.

Thank you @krsntn 🙏🏼 updated, should be landing soon.


Auto-detect create_app() in find_best_app() #2265

#2265

I'd check for both create_app() and make_app()

Will resubmit due to some git issues.

FTR, superseded by #2297


Android Basics: Fragments Navigation component

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-fragments-navigation-component?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-fragments-navigation-component#3
In which task and step of the codelab can this issue be found?
Fragments and the Navigation Component, Step 4 Create Fragment and Layout files
Describe the problem
In the content for fragment_letter_list.xml at the bottom of the step, tool:context is falsely set to ".WordListFragment" instead of ".LetterListFragment"
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

@MiGei333
Thank you so much for reporting this issue. This is fixed and will be published soon.


Improve regex filtering in surfacers

From: https://github.com/google/cloudprober/pull/583 / https://github.com/google/cloudprober/pull/583#discussion_r615179757
 I think we'll need to have something more generic, something like
map<string, string> allowed_metrics
map<string, string> ignore_metrics
where key is the label key, and value is a regex.

Moving this comment to an issue so that its not lost in the PR.

Thanks for capturing it here, @robpickerill.
I'll soon submit a change for this.

Thanks Manu, I started on this PR to update the docs to help out: https://github.com/google/cloudprober/pull/598


Any idea why my selected indicator is wider than the other circles?

This is my layout:
<me.relex.circleindicator.CircleIndicator
           android:id="@+id/banner_indicator"
           android:layout_width="wrap_content"
           android:layout_height="6dp"
           app:ci_width="6dp"
           app:ci_height="6dp"
           app:ci_orientation="horizontal"
           android:layout_marginTop="4dp"
           app:layout_constraintLeft_toLeftOf="@+id/banner_pager"
           app:layout_constraintRight_toRightOf="@+id/banner_pager"
           app:layout_constraintTop_toBottomOf="@+id/banner_pager"/>

And this is what it ends up looking like:

The trouble is with default animator included in library scale_with_alpha.xml. Create another one manually and reference it from layout with app:ci_animator=

Thanks, that fixed it. Is there a recommended way to do the animator? I just changed the 1.8 to 1.0.

Does not work
“Create another one manually and reference it from layout with app:ci_animator=”

It worked for me but I changed the animator from 1.8 to 1.0.

#129

https://github.com/ongakuer/CircleIndicator/issues/129#issuecomment-1204974400


add reusable helpers recipe and implement missing expression features

Add a new recipe about creating reusable helpers.
This PR also implements some obvious things I discovered while implementing a bunch of different kinds of helper functions.
This is just a first draft and any input is welcome.

I don't understand why CI is failing. It's suddenly not finding mocha or playwright. Probably all executables are suddenly not found. @igalklebanov Any ideas? I tried disabling the npm cache but it didn't help.

@koskimas first CrowdStrike killing many services, now this. 😨


function get_3d_sobolev_filter is not being used

You have two types of sobolev filters. one is a 3D data in get_3d_sobolev_filter, the other one is a 1D data in decompose_sobolev_filter. It looks like you decided to hardcode some data in the end without using "get_3d_sobolev_filter". Can I ask what's the reason?

i couldn't find a working c++ library for higher-order svd of tensors so i did the decomposition in matlab for a number of different parameters and hardcoded the values


Unable to make file uploads go to correct pod in Kubernetes

Hello!
We currently make our pods stick to the same session using the X-Chainlit-Session-Id header. However, when uploading files, this header does not exist, so our file is being processed to a random pod. Would it be possible to add this header to that type of request or would you suggest another identifier which can be used to track where requests come from?
We are currently using chainlit 1.0.100

This is a good point. I guess you have a load balancer. Does it support sticky sessions (with a cookie for instance). That would make things easier

In that case all of this user sessions will be rooted to the same instance, which seems acceptable? I replied on your PR. I tried this yesterday but this won't fix your issue.


how to import the adblock list from a file

In fact, the nogo is not tend to be worked well in china or other region。
my be the service import the adblock list from  adblock plus of chrome extension is well or compatible with the adblock list。
I try to do this as below，the affect is not good。
./nogo --dns-proxyto "xxx.x.xx.x:53" --import easylist.txt
head easylist.txt -n 20
==> easylist.txt <==
[Adblock Plus 2.0]
! Version:<PHONE_NUMBER>40
! Title: EasyList
! Last modified: 26 Feb 2017 18:40 UTC
! Expires: 4 days (update frequency)
! Homepage: https://easylist.to/
! Licence: https://easylist.to/pages/licence.html
!
! Please report any unblocked adverts or problems
! in the forums (https://forums.lanik.us/)
&ad.vid=
&ad_box_
&ad_channel=
&ad_classid=

Hello,
As the -import switch's help output says, the import file must be in the format of a "traditional" hosts file (or one domain per line).
Browser plugins like adblock plus use a completely different type of file format since they are meant for the browser and therefore target sources to block in a much different way.
Check out one of these lists instead for the appropriate format: https://github.com/StevenBlack/hosts


New PR to test SignCLA at 2017-11-08 12:02:28.583092611 -0800 PST m=+56.286797718

Body of PR

corp-pass-fork-staging: test succeeded


Errors in Admin Page - Plugins

I'm using local server "Xampp".
`Twig_Error_Runtime thrown with message "An exception has been thrown during the rendering of a template ("GPM not reachable. Please check your internet connection or check the Grav site is reachable") in "partials/plugins-list.html.twig" at line 19."
Stacktrace:
#31 Twig_Error_Runtime in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:402
#30 RuntimeException in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\GPM\GPM.php:473
#29 Grav\Common\GPM\GPM:findPackage in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:604
#28 call_user_func_array in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:604
#27 Twig_Template:getAttribute in C:\xampp\htdocs\grav-admin\cache\twig\b6\b6a16b9e99993d5305205028e010e1f471b3dfa16d9791de3725566abe70287b.php:72
#26 __TwigTemplate_812dd152f02b4c4e995fb2a2e134fb73f18fa1fc2bdaa0c164751063205c25e5:doDisplay in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:387
#25 Twig_Template:displayWithErrorHandling in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:355
#24 Twig_Template:display in C:\xampp\htdocs\grav-admin\cache\twig\4d\4d52b4af49aed4b34218dfba0d4aacb0e49c5ec1836447acc79aafdc1f78bf2f.php:218
#23 __TwigTemplate_d45efbb2ecfd1b6d175e48517c93ba1d3d4dbe975bc59085cd8b1e31d7a923f9:block_content in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:167
#22 Twig_Template:displayBlock in C:\xampp\htdocs\grav-admin\cache\twig\bc\bcedad5a9d9c096c0555639fe81c8eae8ad587d56962c88e545ddb97fbb1e4cb.php:205
#21 __TwigTemplate_a7caedeb0e8f9004b2ff859f14111da968f804e03a9687e4b90ccc5a5f872af1:block_page in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:167
#20 Twig_Template:displayBlock in C:\xampp\htdocs\grav-admin\cache\twig\bc\bcedad5a9d9c096c0555639fe81c8eae8ad587d56962c88e545ddb97fbb1e4cb.php:158
#19 __TwigTemplate_a7caedeb0e8f9004b2ff859f14111da968f804e03a9687e4b90ccc5a5f872af1:block_body in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:167
#18 Twig_Template:displayBlock in C:\xampp\htdocs\grav-admin\cache\twig\bc\bcedad5a9d9c096c0555639fe81c8eae8ad587d56962c88e545ddb97fbb1e4cb.php:45
#17 __TwigTemplate_a7caedeb0e8f9004b2ff859f14111da968f804e03a9687e4b90ccc5a5f872af1:doDisplay in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:387
#16 Twig_Template:displayWithErrorHandling in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:355
#15 Twig_Template:display in C:\xampp\htdocs\grav-admin\cache\twig\4d\4d52b4af49aed4b34218dfba0d4aacb0e49c5ec1836447acc79aafdc1f78bf2f.php:63
#14 __TwigTemplate_d45efbb2ecfd1b6d175e48517c93ba1d3d4dbe975bc59085cd8b1e31d7a923f9:doDisplay in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:387
#13 Twig_Template:displayWithErrorHandling in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:355
#12 Twig_Template:display in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Template.php:366
#11 Twig_Template:render in C:\xampp\htdocs\grav-admin\vendor\twig\twig\lib\Twig\Environment.php:347
#10 Twig_Environment:render in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Twig\Twig.php:340
#9 Grav\Common\Twig\Twig:processSite in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Service\OutputServiceProvider.php:19
#8 Grav\Common\Service\OutputServiceProvider:Grav\Common\Service{closure} in C:\xampp\htdocs\grav-admin\vendor\pimple\pimple\src\Pimple\Container.php:113
#7 Pimple\Container:offsetGet in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Processors\RenderProcessor.php:17
#6 Grav\Common\Processors\RenderProcessor:process in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:121
#5 Grav\Common\Grav:Grav\Common{closure} in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:396
#4 Grav\Common\Grav:Grav\Common{closure} in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:372
#3 call_user_func_array in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:372
#2 Grav\Common\Grav:__call in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:122
#1 Grav\Common\Grav:measureTime in C:\xampp\htdocs\grav-admin\system\src\Grav\Common\Grav.php:122
#0 Grav\Common\Grav:process in C:\xampp\htdocs\grav-admin\index.php:45
`

Make sure you satisfy these requirements https://learn.getgrav.org/troubleshooting/common-problems#cannot-connect-to-the-gpm

I checked everything, but it's still not working.

This is an admin issue, in future open here: https://github.com/getgrav/grav-plugin-admin/issues

"GPM not reachable" - means what it says. Grav can't reach the GPM info on https://getgrav.org.  Do you have a proxy server? Do you have a restrictive firewall?  If not then, the cannot-connect-to-gpm documented flavio linked contains all the common issue we know of.  Are you sure OpenSSL is installed and you have restarted your webserver since enabling it?

As I know, I don't have a proxy server. I've tried to disable firewall. I've check that OpenSSL is enable. I've tried everything. Also I get red alert every time I load admin page say: this.data.feed_data is undefined at Grav</d</<.value@http://localhost/grav-admin/user/plugins/admin/themes/grav/js/admin.min.js?58888ea7a7:4:25071 Grav</d</<.value/<@http://localhost/grav-admin/user/plugins/admin/themes/grav/js/admin.min.js?58888ea7a7:4:24275 Grav</</o/<@http://localhost/grav-admin/user/plugins/admin/themes/grav/js/admin.min.js?58888ea7a7:4:23322 and Error while trying to download 'https://getgrav.org/blog.atom'

Please make sure it is actually enabled because I know XAMPP doesn't enable openssl by default.
Edit the proper php.ini file (or all of them if you have multiple PHP versions) and make sure you have this line uncommented:
[PHP_OPENSSL]
extension=php_openssl.dll

I've checked in php.ini, it's enable.

same for me

Can you both try MAMP for Windows https://www.mamp.info/en/downloads/ and check if the issue persists? In case it's a matter of server configuration, if not it's something else.

I've tried MAMP, still not working.

Have a look at this issue starting from the point i link down https://github.com/getgrav/grav/issues/1008#issuecomment-243041651 . Maybe that'll help you.
Specifically I'm providing some commands that helped determine the issue seemed to be related to the hosting provider. I know you are working locally but it might give us some hints.
$ curl -I https://getgrav.org/downloads/plugins.json

Ok I am confused. Question: is if you go in the Plugins section, are you able to check for updates or is it failing? Do you get the error only in the Dashboard?

Dashboard:

Plugins and Themes:

Thanks. And if you do bin/gpm index -f you get the same?

Ok. Can you do me a favor and look in your php.ini if you have allow_url_fopen.  Then try again and see if it makes any difference.
Sorry for all the directions I'm giving, I'm quite baffled and trying to understand what is causing the issue because from my point of view, it should work.

allow_url_fopen was enable, I turned it off, but nothing is changed.
Thanks,

btw, I've been using my apartment's WiFi, it's quite restrict. Could that be the problem? I have not tried using Grav with other any other connection though.

You should get at least a different message in the error, as it should be going through cURL. I wonder if it didn't trigger it.
Also, if you can reach https://getgrav.org/downloads/plugins.json directly from your browser, you shouldn't have any issue with Grav.
Alright I have one other thing to try. Try editing the Response.php class (https://github.com/getgrav/grav/blob/develop/system/src/Grav/Common/GPM/Response.php#L176-L185) and invert the order of the checks, so instead of fopen first and curl later, make it curl first.
    private static function getAuto()
    {
        if (self::isCurlAvailable()) {
            return self::getCurl(func_get_args());
        }
        if (!ini_get('open_basedir') && self::isFopenAvailable()) {
            return self::getFopen(func_get_args());
        }
    }

Disclaimer: it is ALWAYS a bad idea to directly editing source files, this is for testing purposes only so for posterity, don't do this :)

I can reach https://getgrav.org/downloads/plugins.json.
After I've tried editing Response.php. I got errors like this:

And, all the same in Plugins section.

In that same Response.php file, after line 43 (https://github.com/getgrav/grav/blob/develop/system/src/Grav/Common/GPM/Response.php#L43), can you add CURLOPT_SSL_VERIFYPEER => false?

Btw we are getting somewhere! It looks like the issue is how your machine interprets the SSL certificate we have for getgrav.org.
I'm not quite sure why it fails to verify that, it's a totally valid certificate. 🤔

CURLOPT_HEADER => false this line is already there in the file.

You should keep that one and add the CURLOPT_SSL_VERIFYPEER => false after.

Oh sorry my bad. lol.

Good news! It's working now.

Very interesting. I'm glad this works for you but at the same time I'm confused as to why it didn't in the first place 😄
If you look at our certificate analysis it is marked as grade A which makes it all weird as of why your machine was complaining about verifying the peer.
I think we will add a new system option to disable the peer (so you don't have to touch the source files manually) and a note in the docs at the troubleshooting section.
Thank you so much for bearing with me on this and helping out figuring out the problem. None of the team could reproduce this issue!

You're welcome, it's been my pleasure helping out.
Thank you too, I can now work on my class project. Thank you everyone 😀

Works for me too with response.php edits, thanks!

Thanks we'll get this sorted for the next Grav release!

@maximelebreton out of curiosity could your describe your setup? Windows/Linux, php version, apache/Nginx and if windows/mac what bundle if any.
Thanks.

Windows 10
Apache/2.4.18 (Win32) PHP/7.0.4
Local server with EasyPhp DevServer

I integrated a new set of options for GPM that will allow to nicely disable the verify peer. #1053
This will be available in the next release.
Thanks.

response.php edit worked for me too. Thanks!

I am sad to report that I have similar case with Grav installed on VPS with CentOS 5.11.
As previously discussed in issue #1008, setting verify_peer = false did help showing the normal pages and some Admin pages. But when I clicked on Plugin or Themes, I got this error message:

Server Error
Sorry, something went terribly wrong!
0 - An exception has been thrown during the rendering of a template ("GPM not reachable. Please check your internet connection or check the Grav site is reachable") in "partials/plugins-list.html.twig" at line 19.
For further details please review your logs/ folder, or enable displaying of errors in your system configuration.

Note that I already edit Response.php and added CURLOPT_SSL_VERIFYPEER => false** below CURLOPT_HEADER => false (line 43).
and also on line 348 (347 on the original file), added:
    curl_setopt($rch, CURLOPT_HEADER, false);
    curl_setopt($rch, CURLOPT_SSL_VERIFYPEER, false);

I feel sad because I really like Grav but tired of all of this SSL incompatibility issues :)

Update:
Out of curiosity I just commented out line 19 of user/plugins/admin/themes/grav/templates/partials/plugins-list.html.twig and it works!
    {#% set releaseDate = plugin.date ?: admin.gpm.findPackage(slug).date %#}

I've got similar error when clicking on Theme tab of Admin page

Server Error
Sorry, something went terribly wrong!
0 - An exception has been thrown during the rendering of a template ("GPM not reachable. Please check your internet connection or check the Grav site is reachable") in "partials/themes-list.html.twig" at line 21.
For further details please review your logs/ folder, or enable displaying of errors in your system configuration.

So, again I just commenting out the mentioned line in user/plugins/admin/themes/grav/templates/partials/themes-list.html.twig:
    {#% set releaseDate = theme.date ?: admin.gpm.findPackage(slug).date %#}

and it works!
Still not sure why and what I am missing here.

My two cents, on MAMP/Mac, switching to PHP 7 solved the issue

I've been having the same issues on Azure with PHP 7 in IIS, I've tried all the edits to response.php, enabled and disabled settings everywhere but the solution @judhi posted worked for me! Thanks @judhi !

Removing that line works because eliminates the connection to the GPM, but I guess installing or updating any plugin/theme won't work.

@flaviocopes  you're right, installing and updating does not work once that line is removed.
I ended up putting those lines back in, deleted the cache, restarted the server then ran the following from the grav troubleshooting page through the built in CMD interface on Azure. Using anything else apparently doesn't actually change the permissions on the folders.
find . -type f | xargs chmod 664
find ./bin -type f | xargs chmod 775
find . -type d | xargs chmod 775
find . -type d | xargs chmod +s
After that everything was working. I also went back into the configuration and turned on Remote Verify Peer and Remote Fetch Method to auto.

I was experiencing the same problem today. @AVDW is correct. The grav documentation is correct. Run the commands he mentions in the Azure app service console or the KConsole. Restart the app service and no more errors. FYI...the commands only work at the wwwroot folder. Any higher up and you will get errors.


Make MonadControlT the transitive closure of MonadControl

Summary
Changes the types of MonadControl.restoreM and MonadControlT.restoreM.
See https://github.com/leanprover/lean4/issues/2894 for more information, I'm not sure whether the difference between the existing types is or is not intentional

The current PR title makes it sounds like an instance is missing. Perhaps change to "feat: align definition of MonadControlT with MonadControl"?

Please rebase onto origin/nightly so we can get a Mathlib CI run.

Update: a PR touching MonadControl and MonadControlT is likely to take a long time to review, because this is a complex system that affects many internals. If this is blocking progress on something critical, please let us know, but otherwise this will probably be marked as low-priority.


💡 Log egress throughput improvements

The current architecture has throughput limits we'd like to move past. We see a maximum of about 2,000 logs/second/node.

Closing this for now


[Bug]: Exception thrown when clearing search field.

What happened?
When clearing the select field with the cross button after a previous search an Exception is thrown.
Tapp\FilamentGoogleAutocomplete\Forms\Components\GoogleAutocomplete::Tapp\FilamentGoogleAutocomplete\Forms\Components\{closure}(): Argument #1 ($state) must be of type string, null given, called in /var/www/html/vendor/filament/support/src/Concerns/EvaluatesClosures.php on line 35

How to reproduce the bug
Here is my setup:
GoogleAutocomplete::make('google_search')
        ->label('Adresse')
        ->language('fr-FR')
        ->countries(['MQ', 'GP'])
        ->placeTypes(['address'])
        ->withFields([
            Components\TextInput::make('postal_address')
                ->label('Adresse postale')
                ->extraInputAttributes([
                    'data-google-field' => 'formatted_address',
                ])->columnSpanFull()
        ])->columnSpan(2),

Package Version
1.0.0
PHP Version
8.3.9
Laravel Version
11.9
Which operating systems does with happen with?
Linux
Notes
No response

@qle-bevi  Thank you for reporting :)
Fixed on https://github.com/TappNetwork/filament-google-autocomplete-field/pull/3


[Bug]: Show full model description

What happened?
I don't think model description should be automatically hidden. No one is reading the descriptions of my models, including example prompts for the very questions they routinely ask.
Description, representation, usage restrictions, etc, in any case, should be right there on the main page of a model, not hidden away at any point without the user manually doing it -- especially when we can't actually select licenses to store and distribute with models (which is technically required to share 1.4/1.5/2.0/2.1 models and derivatives, and isn't being honored)
Steps to reproduce the problem
Visit any model page
What should have happened?
All model authors information presented to the user
What platforms do you use to access the site?
Windows
What browsers do you use to access the site?
Mozilla Firefox
Additional information, context and logs
No response

Model descriptions are too long to have them show all of the time, and most likely even if we did show them, people wouldn't read them anyway.
You mention a few uses for the description area, and we've done things to address each of them that should eliminate the need for too much in the description:

Description: Yes, this should go in the description
Prompts (Representation?): No, this should be in the example images. Any prompts/guides in the description should be considered as extra.
Usage Restrictions: Yes (kind of), if you have something other than the Open-Rail license you should include that in the description. However, it's worth noting that the Permission indication system should be used so that users can more quickly digest what's acceptable.

In regards to licenses, we do include the Open-rail license on all checkpoint pages (as required by the license). We haven't added support for custom licenses at this time.

You should work around that then by having description be tabbed with versions (since version description is hardly used and can be condensed like it is. Often there is just one version, so the version block is a huge redundant area wasting space over far more focused content regarding the model
A model should be a users page, to represent their model. Not summarized for consumption. That's very counter intuitive of content sharing, and not really supportive of the authors driving the site (especially paying).
Regarding Open-Rail, it's not about displaying it with the model, it has to distribute with the model. That's the stipulation with any of the models use or derivatives, and that also naturally entails any license a user would put it under subsequently, as usually always required of a license.

Regarding Open-Rail, it's not about displaying it with the model, it has to distribute with the model. That's the stipulation with any of the models use or derivatives (TIs, LoRas, etc), and that also naturally entails any license a user would put it under subsequently, as usually always required of a license (I've never seen a license that doesn't require this). This is legally required with licensed products (free or otherwise) because once downloaded, and referred to later: where does it come from? What's it's usage? etc. This is echoed in the fact model names don't seem to be searchable and often don't relate to the model's title on the site.

How are other sites handling this? As far as I can tell everywhere you can download the SD models just also offer you the License file for download.

Regarding Open-Rail, it's not about displaying it with the model, it has to distribute with the model. That's the stipulation with any of the models use or derivatives (TIs, LoRas, etc), and that also naturally entails any license a user would put it under subsequently, as usually always required of a license (I've never seen a license that doesn't require this). This is legally required with licensed products (free or otherwise) because once downloaded, and referred to later: where does it come from? What's it's usage? etc. This is echoed in the fact model names don't seem to be searchable and often don't relate to the model's title on the site.

How are other sites handling this? As far as I can tell everywhere you can download the SD models just also offer you the License file for download.

I don't know that what other sites do matters. This is just what the license requires. The only "official" form of download packages up the whole HF repo, including license, unless you go through and download each file of the model one by one (for Diffusers models, or ckpt by itself). So maybe a mini repo type deal could be in order, download all the files in a zip, or just manually download the model by itself, but that would have to be explicitly a users choice to not be breaching the license as far as distribution clause goes.
Also with HF Hub, you explicitly agree to the license in code, downloading the models, and it's stored with the code as well.

Should be closed, I don't think there's any intention to stop truncating descriptions. License customization should be brought up in discussion. I myself would use apache2 more regularly if it was an easy option.


update thunderbird desktop package

Checklist

[X] I have checked the moderation queue that no new version has been submitted (only visible when logged in), or the submitted package version has failed the automated checks.

Please navigate to the following link to view the moderation queue.

[X] I have verified that the new version is a stable release.
[X] I have looked to see if there are any Bug reports that prevents a new version being submitted and that no existing Outdated Reports have been created.
[X] I have verified the new version has a Windows binary (EXE or MSI installer, Zip or other archive, etc).
[X] I have verified that this is the correct repository, and the package is maintained by the chocolatey-community user.

New Software Version
128.3.1
Download location
https://download-installer.cdn.mozilla.net/pub/thunderbird/releases/128.3.1esr/win64/en-US/Thunderbird Setup 128.3.1esr.exe
Package Page
https://community.chocolatey.org/packages/thunderbird

Duplicate of #2515


DataBinding update values onClick of button demo

Note: Please don't commit auto generated files/directories like build folder.

I have deleted all autogenerated files, infact compared folder structure with chintan's demo.

Yes @rathodchintan also did a mistake in committing those folder, let's not repeat that mistake again!

Thank you @ravirupareliya for the data binding button click demo. Look forward for more such examples/demos and awesome contribution!

Thank you , and sorry for those silly mistakes, will take care from next time.

No problem! eventually you will learn and will not repeat such mistakes! Anyway, contribution matters a lot :)


Add new page listing all applications currently on DelphiDabbler.com

Add a new top level page that lists all applications currently on delphidabbler.com.
Add a top level "Programs" or "Applications" menu item for the page.
See http://delphidabbler.com/programs for the for list
Include external links to page on DelphiDabbler.com for now

Implemented in development branch at commit a9216e2ae38ae939708d8bd3711678b37b3dad9d.
Will be included in next release.


affected/package: net/http Request.AddCookie does not set all properties of the cookie correctly

net/http.Request.AddCookie does not set all properties of the cookie correctly.
What version of Go are you using (go version)?

$ go version
go version go1.17.8 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Not relevant, seeing the source code of AddCookie().
What did you do?
package main

import (
	"fmt"
	"net/http"
	"time"
	"bytes"
)

func main() {
	name := "name"
	cookie := &http.Cookie {Name: name, Value: "value", Secure: true, HttpOnly: true, Expires: time.Now().Add(60*time.Minute)}
	buf := &bytes.Buffer{}
	req, err := http.NewRequest("GET", "example.com", buf)
	if err != nil {
		panic(err)
	}
	req.AddCookie(cookie)
	var found *http.Cookie
	for _, search := range req.Cookies() {
		if search.Name == name {
			found = search
			break
		}
	}

	if found == nil {
		fmt.Println("Cookie not set?")
	} else {
		fmt.Printf("Cookie set: %+v\n", found)
		// But the extras are NOT set
		if !found.Secure || !found.HttpOnly {
			fmt.Println("Extra properties not added by request.AddCookie().")
		}
	}
}

https://play.golang.com/p/wmBsBuYkRmo
What did you expect to see?
All extra properties of the cookie set by http.Request.AddCookie().
What did you see instead?
Only the name and value of the cookie set by http.Request.AddCookie().

After reading the cookie RFC I realized that this is a difference between the Cookie and the Set-Cookie header. Sorry for the noise, closing as works as intended.