Add page not found handling with custom 404 error page

Description:
This pull request introduces a custom 404 error page that improves user experience when they navigate to non-existent routes on the site. When users visit an invalid URL, they will now see a clear message indicating that the page was not found. The page also includes a link to redirect them back to the home page.
Changes made:

Implemented a NotFound.jsx component for the custom 404 error page.
Updated the routing logic to display the custom 404 error page for invalid routes.
Styled the error page to make it user-friendly with a simple, clear message and a call-to-action button to return to the homepage.

Special notes:

Route /docs behavior: In previous versions of the code, the /docs route redirected users to a documentation page. With this update, navigating to /docs will now trigger the 404 error page, as the /docs page has not been implemented or is no longer part of the site. If the documentation feature is to be reintroduced, this route should be updated accordingly.
Please note that the /docs route is considered a non-existent page in this update, and users will be directed to the 404 error page when trying to access it.

Hi @nayakOPS, the docs site is now fully set up and functional, though it still lacks some information. Since it's operational, this PR will be closed. Thanks for your concern

Hi @utsavdotdev,
Thanks for your feedback. The goal of my update was to handle invalid URLs by rendering a custom 404 error page. My implementation uses hash-based routing (e.g., /#home works, and /#randomRoute shows the 404 page).
However, routes like /docs in the main repo use path-based routing, causing inconsistencies. I suggest standardizing all routes to path-based (e.g., /home, /about, /docs) to maintain consistency and ensure the 404 page works seamlessly across the site.
Let me know your thoughts!

You're right about your concern. The docs site is built using Docusaurus, so we need to route it through a proxy during local development and In production, It is handled by copying build files in one place. On your local system, is the docs site running at localhost:5173/docs? and Yeah I also missed setting up the 404 page. You can handle that, but make sure that when users are redirected to the /docs route, it displays the docs page correctly.

i got it. The proxy setup is missing on the your version. It is fixed in recent changes you can check it.

Merged 🚀


[Bug]: towhee build docker image error with a local private BERT model

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When executing towhee.build_docker_image, the op contains a local private BERT model, and the following error occurs:
factory.py-factory:118 - ERROR: Loading operator with error: yyyy/chinese-bert-wwm-ext is not a local folder and is not a valid model identifier listed on 'https://huggingface.co/models'
Which level directory in the docker should this model be copied to?

Expected Behavior
No response
Steps To Reproduce
No response
Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS): centos7
- CPU/Memory:
- GPU: T4
- Others: cuda 11.7

Anything else?
No response

Use towhee.build_pipeline_model to create Triton models without using Docker.  wiki: https://towhee.readthedocs.io/en/latest/triton/triton.html
And then build a docker image to run the models,  need to install some deps
Some example dockerfiles: https://github.com/towhee-io/towhee/tree/main/towhee/serve/triton/dockerfiles

nice


Allow mountPropagation configuration by merging the helper Pod template's volumeMount

The helper Pod template's data volumeMount is merged with the defaults in order to allow users to specify the mountPropagation within the template.
Split from PR #166.
Closes #165.
Relates to #127.

hm, the e2e test TestPVCTestSuite/TestPodWithSecurityContext fails.
For some reason the local-path-test Pod has no permissions to write into the volume and logs:
uid=1000(jenkins) gid=1000(jenkins) groups=1000(jenkins)
total 8
drwxr-xr-x 2 root root 4096 Mar  6 22:18 .
drwxr-xr-x 1 root root 4096 Mar  6 22:18 ..
Hello from local-path-test
cp: cannot create regular file '/test/test.txt': Permission denied

Fixed. This is ready for review now.

Can we get this change merged anytime soon?

Can we get this change merged anytime soon?

Thanks for the notification. I will review it.

Hi @derekbit
have you had a chance to review this PR yet?


Invalid FS bundle size

HI everyone,
I'm trying to use electric-sql without the need of using cdn and I got this error while I'm loading the db:
Uncaught (in promise) Error: Invalid FS bundle size: 792 !== 2987805
    at Object.getPreloadedPackage (electric-sql-bundle.js:11787:39)
    at loadPackage (electric-sql-bundle.js:2021:84)
    at electric-sql-bundle.js:2133:7
    at electric-sql-bundle.js:2134:7
    at te3.Pe2 (electric-sql-bundle.js:11821:57)

I'm using a bundled version (electric-sql-bundle.js) created with esbuild
this is the code of the esbuild
const esbuild = require('esbuild');
esbuild.build({
target: 'esnext',
  bundle: true,
  splitting : false,
  format: 'esm',
  entryPoints: [
<EMAIL_ADDRESS>  ],
  outfile: './dist/electric-sql-bundle.js',
  define: {
    'process.env.NODE_ENV': JSON.stringify('development'),
  },
  external: ['@electric-sql/pglite'],
}).catch(() => process.exit(1));

Probably i did something wrong but i dont find any kind of documentation related to this.
If I did something wrong in the bug report im sorry

I'm getting this as well from following the getting started instructions(yarn)

solution(for me at least): https://discord.com/channels/933657521581858818/934234468514922567/1301530280212828221

I use quasar so I had to apply this in my quasar.conf.js:
 extendViteConf(viteConf) {
        // Extend the Vite configuration to exclude dependencies from optimization
        viteConf.optimizeDeps = {
          ...viteConf.optimizeDeps,
          exclude: ['@electric-sql/pglite'], // replace 'some-library' with the module you want to exclude
        }

        // Optional: Exclude from Rollup build as well
        viteConf.build = {
          ...viteConf.build,
          rollupOptions: {
            ...viteConf.build?.rollupOptions,
            external: ['@electric-sql/pglite'], // ensure that Rollup does not bundle this module
          },
        }
      },```

related docs: https://pglite.dev/docs/bundler-support

Hi, using pglite via
import { PGlite } from<EMAIL_ADDRESS>It has been working perfectly fine for about a week or two, until suddenly it stopped working with following error message:
Uncaught (in promise) Error: Invalid FS bundle size: 2987805 !== 4827589
getPreloadedPackage postgres.js:9
loadPackage postgres.js:9
Module postgres.js:9
Module postgres.js:9
Ee postgres.js:9
te postgres.js:9
 table_edit.js:9
Any help appreciated

I cleared the web browser cache (FF in my case) and the error is gone.


Failed to connect VTTablet

Vitess 0.12, and keep getting errors:
Nov 26, 2021 3:51:48 PM io.vitess.shaded.io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference cleanQueue
SEVERE: ~~~ Channel ManagedChannelImpl{logId=23354, target=<IP_ADDRESS>:15999} was not shutdown properly!!! ~~~
Make sure to call shutdown()/shutdownNow() and wait until awaitTermination() returns true.
java.lang.RuntimeException: ManagedChannel allocation site
at io.vitess.shaded.io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference.(ManagedChannelOrphanWrapper.java:93)
at io.vitess.shaded.io.grpc.internal.ManagedChannelOrphanWrapper.(ManagedChannelOrphanWrapper.java:53)
at io.vitess.shaded.io.grpc.internal.ManagedChannelOrphanWrapper.(ManagedChannelOrphanWrapper.java:44)
at io.vitess.shaded.io.grpc.internal.ManagedChannelImplBuilder.build(ManagedChannelImplBuilder.java:596)
at io.vitess.shaded.io.grpc.ForwardingChannelBuilder.build(ForwardingChannelBuilder.java:255)
at com.jd.jdbc.queryservice.TabletDialer.dial(TabletDialer.java:49)
at com.jd.jdbc.discovery.TabletHealthCheck.getTabletQueryServiceLocked(TabletHealthCheck.java:132)
at com.jd.jdbc.discovery.TabletHealthCheck.getHealthCheckQueryService(TabletHealthCheck.java:121)
at com.jd.jdbc.discovery.TabletHealthCheck.startHealthCheckStream(TabletHealthCheck.java:217)
at com.jd.jdbc.discovery.HealthCheck$1.lambda$run$0(HealthCheck.java:410)
at java.base/java.util.concurrent.ConcurrentHashMap.forEach(ConcurrentHashMap.java:1603)
at com.jd.jdbc.discovery.HealthCheck$1.run(HealthCheck.java:383)
at java.base/java.util.TimerThread.mainLoop(Timer.java:556)
at java.base/java.util.TimerThread.run(Timer.java:506)

ManagedChannel allocation site

Hi DongBin,
Thanks for your report. Could you please provide more detail, such as complete Exception stack, database configuration?
According to our experience, this error is caused by an unauthorized database account.
If you use a vitess account to connect, you need to configure MySQL to allow this account to access the data node directly. There is a sample script and quick-start. Hope it can be useful for you.

@dongbin Hi, dongbin!
Now we provid a docker image for vtdriver, you can build it easily with only two command as following:
git clone git@github.com:vtdriverio/vtdriver.git
cd vtdriver/src/test/resource/vitess_env
./setup.sh

# wait few minutes...
docker run -e HOST_IP=<IP_ADDRESS> -it --name=vtdriver-env -p16100:16100 -p16101:16101 -p16102:16102 -p16300:16300 -p16301:16301 -p16302:16302 -p16400:16400 -p16401:16401 -p16402:16402 -p15000:15000 -p15001:15001 -p15306:15306 -p15100:15100 -p15101:15101 -p15102:15102 -p15300:15300 -p15301:15301 -p15302:15302 -p15400:15400 -p15401:15401 -p15402:15402 -p2379:2379 -p17100:17100 -p17101:17101 -p17102:17102 -p17300:17300 -p17301:17301 -p17302:17302 -p17400:17400 -p17401:17401 -p17402:17402 vitess/vtdriver-env:latest

# or in a no-macos computer
docker run -e HOST_IP=<服务器IP> -itd --name=vtdriver-env --network=host vitess/vtdriver-env:latest

More details are in https://github.com/vtdriverio/vtdriver/blob/main/docs/QuickStart.md
Thank you for your report!


Move from nox to tox

To be added on top of #47.

add a coverage report environment
move to src layout to fix packaging (build name is reserved at root
and will be filtered out by https://github.com/pypa/setuptools/blob/master/setuptools/command/egg_info.py#L592)
add documentation build environment

Moving the module to a src folder should be in a separate PR, and btw, I do not agree.

Can you expand on this? The package the way it is does not build, and the reason is in this PR is that once I moved to tox the issue was exposed. So had to move to src to fix the issue. In theory, could be a before PR, but not a fan of trying to make it work first with nox just so that then we move it to tox.

The package the way it is does not build

What do you mean? It builds perfectly fine here, both with python -m build and python setup.py build.

python -m build

That only works because Python adds the CWD to the sys.path and you're picking up the local files from the root, not because the build files are installed into the repository.

python setup.py build

Not sure about that, but python setup.py sdist passes, but then does not include the build package because https://github.com/pypa/setuptools/blob/master/setuptools/command/egg_info.py#L592 removes the build folder as it's what's also used to populate the binary metafiles. Ship the sdist and users can't use this package at all.

That only works because Python adds the CWD to the sys.path and you're picking up the local files from the root, not because the build files are installed into the site-packages of your host python.

That is not the point.

Not sure about that, but python setup.py sdist passes, but then does not include the build package because pypa/setuptools:setuptools/command/egg_info.py@master#L592 removes the build folder as it's what's also used to populate the binary metafiles. Ship the sdist and users can't use this package at all.

You're right. The wheel works though.

You're right. The wheel works though.

Both needs though.

That is not the point.

What is then?

FWIW I'm also +1 on a src/ layout, which is also something we've got a rough consensus to be promoting as the default in the PyPA, and seems especially germane for this project, since there's a name conflict with the location that setuptools currently uses to do builds in.
Good background for other reasons to structure it like this:

Hynek Schlawack's Testing & Packaging blog post.
Ionel Mărieș' Packaging a Python Library (which is increasingly out of date for actual recommendations, but still makes a reasonable case for the src/ layout).

If we don't use a src/ layout, we should probably do some variation on the setup I describe in Testing your python package as installed.
I don't care if it's a separate PR or this PR, but given that it's required if we want to continue building with setuptools, we should almost certainly do it.
I'm inclined to say do it as a separate PR and rebase this one on that PR, considering @gaborbernat is all about the squash merge.

It's not a big deal, but should be in a separate PR.


Application not signed

Bug description
I am using the game-ci/unity-builder@v4 on github workflows with a personal unity license to build my unity project.
The resulting .exe file is not signed and I'm getting warnings that the publisher is unknown.
How to reproduce
This is the part of the workflow
uses: game-ci/unity-builder@v4
id: build
env:
  UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}
  UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
  UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
with:
  customImage: unityci/editor:${{ matrix.targetImage }}
  targetPlatform: ${{ matrix.targetPlatform }}
  customParameters: -target ${{ matrix.target }}
  buildMethod: BuildScript.Build

I copied my own personal license as described in the docs in secrets.
These are the relevant logs
User *** logged in successfully
[LicensingClient] Channel doesn't exist: "LicenseClient-root"
[Licensing::Module] Successfully launched the LicensingClient (PId: 38)
[SignatureVerifier] Application signature verification not supported on this platform.
[LicensingClient] Handshaking with LicensingClient:
	version: 1.10.0-rc.1+0c717dc
	Session Id: de5b4c034b5d443ea0456b8218b49027
	Machine Id: j6bT3DmylWNQWQ6f3fzOYk3+Xlw=
[Licensing::Module] Successfully connected to LicensingClient on channel: "LicenseClient-root" (connect: 1.30s, validation: 0.07s, handshake: 0.00s)
[Licensing::Module] Connected to LicensingClient (PId: 38, launch time: 0.00, total connection time: 1.37s)
Entitlement-based licensing initiated
[LicensingClient] Successfully updated the access token
[Licensing::Module] Successfully updated the access token ObKfcfbLBuQwK6ya8ncHx_jHV21tuQu1qG36R5eqBo8008f
[LicensingClient] Successfully processed license management request
[Licensing::Module] Successfully activated the entitlement license
[LicensingClient] Successfully queried for the EntitlementGroupsDetails
[LicensingClient] Successfully activated ULF license
[Licensing::Module] Serial number assigned to (masked): "***"
[LicensingClient] Successfully resolved entitlements
Pro License: NO

When trying to open the application windows warns me that their is no publisher
Get-AuthenticodeSignature -FilePath .\StandaloneWindows64.exe returns Status:NotSigned as well
Expected behavior
The application should be signed

Generally, Unity doesn't sign the .exe, so you would have to sign it yourself.
You could try using this Microsoft SignTool.
But instead of that, it may be better to distribute through a platform like Steam.

Thanks for your help, I thought the license was needed for signature.

Is there no way to integrate the code signing with certificate using GameCI?

Code signing is arguably not part of building ("ci"), and arguably more part of distribution ("cd").
The unity-builder action therefore isn't really where the code signing should be done.
To help with distribution, GameCI does provide a Steam deploy action, and Steam will handle code signing for you: https://game.ci/docs/github/deployment/steam/
If you want to use other platforms for distribution, you would need to look into that platforms recommended process.
Unfortunately, it doesn't seem like itch provides a good solution: https://itch.io/jam/7drl-challenge-2021/topic/1245681/code-signing


Support pass-through for variable report_build_status

What
Exposes report_build_status as an external variable, defaulting to false.
PR includes updated README documentation.
Why
This flag is useful for building out Github PR-verify workflows.

@chriswynnyk  thanks for the PR.
Looks good, just run terraform fmt (CI/CD is failing https://travis-ci.org/cloudposse/terraform-aws-codebuild/builds/500050794?utm_source=github_status&utm_medium=notification)


Current cell after tables

After any table is built, the current cell state should point to just below the bottom left of the table.

Need regression tests.

Fixed in 0.0.33.


Code Error 1

Hi,
I get this error Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-n9luq3fy/scapy/ after this command sudo pip install -r requirements.txt
I've already installed pip, with -get-pip.py
What's the problem and how to resolve it ?
Thank You

Hey there. Please have a look at Issue #37. Let me know if this helped solve your issue. Thanks!

It didn't


[BUG] Can't evoke the floating window to edit the table

Describe the bug
Can't evoke the floating window to edit the table
Suspended plugin installed
and MDtable edite
updated to the latest version
0.2.1
Steps to reproduce the behavior:
1.make a  new MD file
2.make a test table
3. Use the command to call the floating window to edit the table
4. Can't evoke the floating window to edit the table
Expected behavior
I don not know
Screenshots
yes
Desktop (please complete the following information):
windows 10

This plugin is not validated for compatibility in Obsidian Release v0.14.2 (Insider build), so maybe that's the problem

Please manually turn off the plugin in plugin list and then turn it on.

@cumany, did @AndyLiu9's suggestion help you?

@MugenLee, also have you installed the obsidian-hover-editor plugin?


open link in new window

Hi !
This is a suggestion to open links in new window (popup) rather than in the current form.
Thanks !
Olivier

Did you want it to open in a popup, or just a new window?

A popup, so it has the same behaviour than grappelli's popups for foreign keys

I'm conflicted. I like to be able to click Edit to go to the referenced object's edit page full screen. I use it as a navigation mechanism. So the popup mechanism gets in the way of that use case. I'm trying to think of a way to allow either method.

Maybe the best would be to allow both behaviours, either with a setting, or with two distinct mixins.
Here, my users always loose some changes because of those links (they do not have the reflex to use ctrl+click to open in a new tab).


[Feature Request] Support deploy multiple shuffle servers in a single node

We have 1.2T memory per host, so we need deploy multiple shuffle servers in a single node.
But current partition assignment policy is not suitable to do that.
Because partition maybe assign to a same host and client maybe fail if one host down

As i know, partition assignment policy could be implemented by yourself.

As i know, partition assignment policy could be implemented by yourself.

Yes, but not only assignment policy need to be change, such as start script, i will create a pr for this

I'm also curious why we need to modify start script?

I'm also curious why we need to modify start script?

start script will process existence

If our plan is deploy on k8s, this issue should close?

K8S Deployment can't solve this issue.

I'm also curious why we need to modify start script?

start script will process existence

Yes, this is the limit with current implementation

K8S Deployment can't solve this issue.

@xianjingfeng I think you can continue this.

K8S Deployment can't solve this issue.

@xianjingfeng I think you can continue this.

ok


Bundle MutationListener Plugin with our JetBrains Plugin

User story

As a user
I want / need to our mutation listener plugin to be bundled with our jetbrains plugin
So that I don't have further complexities using the pitest plugin

Acceptance criteria

Mutation Listener Plugin is bundled with our JetBrains plugin
Edit the existing error message for missing plugins appropriately

Definition of done (DoD)

All acceptance criteria are met.
Work products are uploaded to the GitHub repository.
A pull request is created for each related branch.
The work products in the pull requests are reviewed.
GitHub CI Workflow passes for the branches
The corresponding branches are merged and closed.
The bill of materials section of the planning documents is updated.
Tests are written for the added features, if suitable

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

@lheimbs @nikomall34
Would you say this ticket is understandable in that we want to bundle the mutation listener stuff as discussed in the industry partner meeting?

Its not the jetbrains plugin but the gradle plugin we - if at all - should do the bundling with I think.
But @nikomall34 and I discussed this on Friday and we felt it would be best to publish the coverage reporter properly and then have the gradle plugin automatically add it as a pitest dependency at runtime.
@nikomall34 has already created a PR in that direction: #168.
Obviously if POs prefer the coverage reporter bundled as a jar with either the jetbrains or gradle plugin that is still doable but the IP had the very valid concern that direct bundling of a .jar with either of the plugins will make versioning a nightmare, whereas having a published version merely requires changing the version string.


Add caveat that setting yAxis max doesn't actually mean that's the max.

Customer complaint. They set max for chart to 1.5M, but the actual graph showed 2M. I believe this is due to the nicing, but we should document that and suggest how they can try different number of y-axis-ticks to make it look closer to what they want.
(assuming you can't turn off nicing. if you can we should document that instead)

Interestingly, there's a broken link to nicing in the docs for smartAxis

This no longer appears to be an issue. Additionally, the nicing probl was reported and fixed as part of issue 278.


[RFC] Rules loader

I've been wanting to have greater control over how rules are loaded. Specifically, being able to load them from places other than a local directory; for example, a mongo database.
This PR creates a means to do just that. It uses a similar methodology as Enhancements and Alerters in that there is a base class RulesLoader which is specified in the config.yaml under the rules_loader property. You can use the format module.file.ClassName to use your own loader. The default is called FileRulesLoader and this behaves exactly like the current way. If you don't make a change you will see no difference in the way that EA works.
RulesLoaders are subclasses of RulesLoader, found in elastalert/loaders.py. They are used to
gather rules for a particular source. Your RulesLoader needs to implement three member functions, and will look something like this:
class AwesomeNewRulesLoader(RulesLoader):
    def get_names(self, conf, use_rule=None):
        ...
    def get_hashes(self, conf, use_rule=None):
        ...
    def get_yaml(self, rule):
        ...

You can import loaders by specifying the type as module.file.RulesLoaderName, where module is the name of a python module, and file is the name of the python file containing a RulesLoader subclass named RulesLoaderName.
As an example loader, let's retrieve rules from a database rather than from the local file system. First, create a
modules folder for the loader in the ElastAlert directory.
    $ mkdir elastalert_modules
    $ cd elastalert_modules
    $ touch __init__.py

Now, in a file named mongo_loader.py, add
from pymongo import MongoClient
from elastalert.loaders import RulesLoader

class MongoRulesLoader(RulesLoader):
    def __init__(self, conf):
        super(MongoRulesLoader, self).__init__(conf)
        self.client = MongoClient(conf['mongo_url'])
        self.db     = client[conf['mongo_db']]

    def get_names(self, conf, use_rule=None):
        if use_rule:
            return [use_rule]

        rules = []
        self.cache = {}
        for rule in self.db.rules.find():
            self.cache[rule.name] = rule.yaml
            rules.append(rule.yaml)

        return rules

    def get_hashes(self, conf, use_rule=None):
        if use_rule:
            return [use_rule]

        hashes = {}
        self.cache = {}
        for rule in self.db.rules.find():
            self.cache[rule.name] = rule.yaml
            hashes[rule.name] = rule.hash

        return hashes

    def get_yaml(self, rule):
        if rule in self.cache:
            return self.cache[rule]

        self.cache[rule] = self.db.rules.find_one({'name': rule}).yaml
        return self.cache[rule]

Finally, you need to specify in your ElastAlert configuration file that MongoRulesLoader should be used instead of the
default FileRulesLoader, so in your elastalert.conf file:
    rules_loader: "elastalert_modules.mongo_loader.MongoRulesLoader"

Awesome! I've been looking for something like this, but loading the rules from Elasticsearch itself. I'll see if I can write a RuleLoader for my purposes

I've just updated this to resolve the conflicts that were introduced. What is the likelihood of this being merged?

Closing this PR as a copy of this has been merged into the beta branch. Thanks @Qmando.


Updated index.md - Repeat a String - removed advanced solution does not meet problem criteria

The advanced solution does not meet problem criteria.
"The built-in repeat()-method should not be used"

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Congrats @asims2404 on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Reverted dispatcher change for bookmarks.

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3558

PR #3563   👈
PR #3561
PR #3560

This comment was auto-generated by Graphite.


Comments in .vimrc aren't highlighted properly

Description of the problem or steps to reproduce
Comments in .vimrc have no closing quotes, but Micro's syntax highlighter is treating the beginning of one comment to the beginning of the next as an entire string:

Specifications
You can use micro -version to get the commit hash.
Commit hash: 1856891
OS:
Terminal:

ko hieu lam


Improve FAQ Section

Hey, I want to work on this issue. Please assign this to me. The FAQ section doesn't look good.
Check the gif given:

Ok sure. @Abhijeet03s you are assigned


Fix CVE dependency issue

Fix issue #563   by update dependency jackson-dataformat-csv.version to 2.14.2 @authorjapps

Waiting for this to be addresses:
https://github.com/authorjapps/zerocode/issues/563#issuecomment-1913135923 (🟡 )

@authorjapps This PR might be closed. jackson-dataformat-csv has been upgraded to version 2.15.4.


[HW] Devansh

Name: Devansh Lodhi
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/devansh-l-6911bbaa/

Hey Devansh Lodhi, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/bIq9m8K1TqWFFaneJ49xjQ


fix: fix broken avatar directive

After a recent styling refactor the avatar directive was broken because
the file it was referencing for the default profile image was removed.
Here we simplify things and just make it the avatar directive fully self contained,
as their is no real reason to pass window.host or window.origin as an argument since
it is a simple globally accessible value.

Current dependencies on/for this PR:

main

PR #262

PR #263

PR #264

PR #265   👈

This comment was auto-generated by Graphite.


Don't redeclare generic private method in subclass

If a superclass contains a method that returns a parameterized type, a subclass that narrows that type shouldn't include a declaration of that method in the .m file if it is a private method since the subclass is restricted from calling that method. In practice, this was causing compilation errors if the method argument list contained types that are not already imported in the subclass.
This patch fixes the issue and includes a test case.

Looks good with one comment to fix.

@danieldickison do you think you could provide a test case that reproduces the compile error mentioned? That seems like it might be a bug that's not specific to private methods but I can't reproduce it. In my test I have a "@class" declaration for the parameter type as expected.

@kstanger I failed to come up with a test case that reproduces that error. I see it happening in a particular class in our app's actual code base using j2objc 1.2, but there must be other conditions that cause the bug to appear.
What I see is that with our app's code, using the 1.2 distribution of j2objc, the subclass gains the method declaration in the .m file's "private" @interface block, but it doesn't add a @class declaration for the argument type JavaIoIOException, causing an objc compilation error. However, when I run the testGenericPrivateMethodNotAdded test case against the 1.2 tag, I find that the subclass .m file does not contain the method declaration, or an @interface block at all. So it seems I haven't reproduced the spurious private method declaration bug against 1.2 after all.
On the other hand, against the current master branch, testGenericPrivateMethodNotAdded does add the spurious method declaration in the subclass .m file. However, when I add an argument to that method, it also adds the @class for the argument type, thereby avoiding the error (as you saw).
So I'm not sure what's causing the bug with our app code but I don't think I'll have the time to investigate this much further. I do think the change in this pull request is still valid, though let me know if you disagree. (I've made the ElementUtil.isPrivate() change as requested.)

Thanks for taking the extra time with this.

It's a good change and a good test addition. Thanks for the contribution! It doesn't seem worth extra effort to track down the potential type declaration issue.


straight-server is not working

after using 'straight-server -p 9696'command  I'm getting following error:
Pending migrations for the database detected. Migrating.../usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:931:in `type_literal_generic': Unsupported ruby class used as database type: Bignum (Sequel::Error)
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:916:in `type_literal'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:544:in `column_definition_sql'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:452:in `alter_table_add_column_sql'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:445:in `alter_table_op_sql'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:496:in `alter_table_sql'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/adapters/shared/sqlite.rb:265:in `alter_table_sql'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:526:in `block in alter_table_sql_list'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:526:in `map'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:526:in `alter_table_sql_list'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/adapters/shared/sqlite.rb:246:in `block (2 levels) in apply_alter_table'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/adapters/shared/sqlite.rb:246:in `each'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/adapters/shared/sqlite.rb:246:in `block in apply_alter_table'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/transactions.rb:215:in `_transaction'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/transactions.rb:190:in `block in transaction'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/connecting.rb:280:in `block in synchronize'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/connection_pool/threaded.rb:107:in `hold'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/connecting.rb:280:in `synchronize'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/transactions.rb:156:in `transaction'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/adapters/shared/sqlite.rb:238:in `apply_alter_table'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:433:in `apply_alter_table_generator'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:81:in `alter_table'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/database/schema_methods.rb:39:in `add_column'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/straight-server-1.0.0/db/migrations/015_add_amount_paid_to_orders.rb:4:in `block (2 levels) in <top (required)>'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:106:in `instance_eval'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:106:in `apply'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:560:in `block (2 levels) in run'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:476:in `checked_transaction'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:559:in `block in run'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:556:in `each'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:556:in `run'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/sequel-4.41.0/lib/sequel/extensions/migration.rb:401:in `run'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/straight-server-1.0.0/lib/straight-server/initializer.rb:110:in `run_migrations'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/straight-server-1.0.0/lib/straight-server/initializer.rb:43:in `prepare'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/straight-server-1.0.0/lib/straight-server/server.rb:10:in `initialize'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/goliath-1.0.4/lib/goliath/application.rb:105:in `new'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/goliath-1.0.4/lib/goliath/application.rb:105:in `run!'
        from /usr/local/rvm/gems/ruby-2.1.8/gems/goliath-1.0.4/lib/goliath/application.rb:133:in `block in <module:Goliath>'

What to do ?

Changing Bignum Into Float Solved this issue.


Proto

There were some side effects behind this pull request so I had to change the code by putting it inside a conditional as a way to avoid problems with existing user.
It's been added to the release 1.228. You can see the change in this commit.
I hope that will not break your use case. Please add the xForwardedProto=true parameter to your services before updating to the new release.


Appsec action 6.2: Target URL Confusion / Host Header Injection (low)

The service appears to implicitly trust the user-supplied Host header. If this input is not properly validated, an attacker could inject harmful payloads through the Host header, manipulating server-side behaviour.
During this assessment, it was possible to inject an arbitrary domain name into the Host header, resulting in the server responding with a 308 redirection response. This vulnerability allows an attacker to potentially poison the web cache. By exploiting this vector, an adversary could cause future requests to be redirected to a malicious domain, leading to various harmful outcomes.
Additionally, if the target URL does not match the Host header, the application appears to redirect users to a different website. This mismatch can create confusion for the server and intermediary systems such as proxies, load balancers, or caching layers.
If an attacker repeatedly sends requests with a malicious domain in the Host header, and the website uses a web cache, this could lead to cache poisoning. Consequently, all future requests to the site could be redirected to an attacker-controlled domain. This redirection could facilitate further malicious activities, such as phishing, malware distribution, or data theft.
Proof of Concept:
curl --path-as-is -i -s -k -X $'GET' \
    -H $'Host: dev.find-moj-data.service.justice.gov.uk' -H $'Connection: close' \
    $'https://www.google.co.uk/'

HTTP/2 404
content-type: text/html; charset=UTF-8
referrer-policy: no-referrer
content-length: 1561
date: Mon, 02 Sep 2024 10:53:14 GMT
alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000

<!DOCTYPE html>
[...]

Similarly if the host header is changed, it redirects to exploit-db.com
curl --path-as-is -i -s -k -X $'GET' \
    -H $'Host: www.exploit-db.com' -H $'Connection: close' \
    $'http://dev.find-moj-data.service.justice.gov.uk/'

HTTP/1.1 308 Permanent Redirect
Date: Mon, 02 Sep 2024 10:50:38 GMT
Content-Type: text/html
Content-Length: 164
Connection: close
Location:[ https://www.exploit-db.com/](https://www.exploit-db.com/)

<title>308 Permanent Redirect</title>

308 Permanent Redirect

nginx

Likelihood is as follows:
Social Engineering Attack: The likelihood of tricking a user to click a malicious link through social engineering is mitigated by browsers that ensure a valid host header is always supplied. This reduces the risk of such an attack being successful through mere social engineering tactics.
Scripted Attack: The likelihood of a scripted attack that exploits the host header to poison the web cache is higher, provided that the attacker has the capability to manipulate the host headers and the targeted website has not implemented adequate protections against cache poisoning. This type of attack requires specific conditions (e.g., a vulnerable caching mechanism) to be successful but is plausible if these conditions are met.
Remediation
To enhance security in NGINX configurations, it's crucial to validate host headers effectively. This involves setting up checks that only allow requests from recognized hostnames, thereby preventing unauthorised access and potential security vulnerabilities like web cache poisoning. By implementing a validation process that includes a whitelist of approved hostnames and rejecting requests that do not match these names, organisations can safeguard their environments from various host header attacks. This approach is fundamental in maintaining the integrity and security of web applications served by NGINX.
https://blog.shalvah.me/posts/fixing-the-host-header-vulnerability-with-nginx

The permanent redirect from HTTP -> HTTPS is this kubernetes default behaviour https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#server-side-https-enforcement-through-redirect
This kicks in before the request reaches django, which does validate the host header.
The remediation above assumes we are directly configuring nginx, but we're configuring it via kubernetes annotations so this needs adapting.
Modsecurity documentation is here
https://user-guide.cloud-platform.service.justice.gov.uk/documentation/networking/modsecurity.html

Were unable to fix on a call so raising a ticket for cloud platform to pick up in their next sprint


feat(event): first iteration of gift card mailing to participants

Proposed changes

Sends one pdf-file from the request to each participant who has attended
The user who sent the request will receive an email with an overview of which user received which pdf and if the email was sent successfully or not.

If we and NoK see it fit to make use of this solution some time into the future due to covid, we should iterate further on this and perhaps provide an overview in event-admin.
Issue number: closes #398
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] CHANGELOG.md has been updated. Guide
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] The fixtures have been updated if needed (for migrations)
[x] Pull request title follows conventional commits (type(scope): description)
[x] Format and lint (make format and make flake8) was run locally without failures

Further comments
Dette er første iterasjon for å kunne tilby et enkelt grensesnitt vi enkelt kan bygge videre på ved senere behov.

Eventuelle problemer siden informasjonen ikke lagres:

Hvis et unntak heves midt under utsendingen, feks at epostklienten frakobles, har vi ingen måte å vite hva som ble sendt til hvem
Hvis ikke eposten til avsenderen av requesten blir sendt har vi ikke oversikten heller

Ser videre på en løsning som håndterer dette


css fix for inline code

Description
fix PR https://github.com/lsegal/yard/pull/1142/
see comment https://github.com/lsegal/yard/issues/1141#issuecomment-354515623
Completed Tasks

[x] I have read the Contributing Guide.
[x] The pull request is complete (implemented / written).
[x] Git commits have been cleaned up (squash WIP / revert commits).

Coverage decreased (-0.01%) to 93.505% when pulling 898e7b6a8d2c088b501c5a3a062a795a76f01705 on noraj1337:patch-1 into 87080b5773aea49a81fbc6793c98ca2424390785 on lsegal:master.


JZ - Metal Defects Detection creation

When you added a new program / programs , put a cross on the  OS that you have test it on
(Example: [ X ] Windows  )
[ x ] Windows
[  ] Linux Ubuntu
[  ] Mac
[  ] Others  (State here -> xxx )
Assign a reviewer to review your code.

@jingzhi93 can we prioritize in fixing and merging this?
cc @choowilson

@jingzhi93 Code is working, would be great if you can include a little note on:

how the program will not work in GPU settings
training time is long (approx. 40 minutes for 1 epoch), so it's advisable to use pre-trained weights given by trainer
update README

@jingzhi93 Code is working, would be great if you can include a little note on:

how the program will not work in GPU settings
training time is long (approx. 40 minutes for 1 epoch), so it's advisable to use pre-trained weights given by trainer
update README

Currently using pretrained metal surface defects detection weight hosted on archieve.org. The program will download the pretrained weight automatically from archieve.org and train. This will helps to reduce the training time.
Furthermore, due to the memory problem, the training can only be done using CPU.

Updated readme as well

@jingzhi93 MetalSurfaceDefectDetector_YOLOv2 CPU hit NaN.

@peirkern-skymind I tested the code on 2 pcs without Nan value. Did you change any parameters / settings?

@peirkern-skymind I fine tuned the hyperparameters per latest commit. Can you rerun the code and see if it hits the Nan problem again?

@peirkern-skymind I tested the code on 2 pcs without Nan value. Did you change any parameters / settings?
i just switched to CPU mode, will try it again now see if the issue persist.

@peirkern-skymind @jingzhi93
essentially this is a problem where are we able to reproduce the error and fix it subsequently
do practice using gist to track the output for a better debugging process
common things you guys already know are Invalidate Caches/Restart in Intellij, reimport project, clear javacpp cache...
alternatively use the shared station at Bobby place to reproduce it.

@jingzhi93 kindly share your training gist here https://gist.github.com/peirkern-skymind/c15dabb96495e38e48e08ab369ea7b08 to cross check.
just a note here, we are in the midst of sorting out why the training outcomes are different on individual machine.

All gist and image inference for beta 6 shows consistent results after 3 rounds of experiments. Currently proceed to fine-tuning.

We got consistent result using beta6. Currently pull codes from beta 6 branch and merge together with the current branch. Performing code test to make sure all codes can run before commit.


Minor typo in docs for --output-directory

dotnet test always run tests from the output directory.

That should probably be

dotnet test always runs tests from the output directory.

instead

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

ID: 61ca4064-7481-a263-60fc-c70127f2ad16
Version Independent ID: 2878af44-8208-7bd8-46d4-3acda3e82ce0
Content: dotnet test command - .NET Core CLI
Content Source: docs/core/tools/dotnet-test.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Thank you for opening this issue. I'll add it to the backlog. If you would like to contribute, you can submit a fix by editing the original article. Click on the Content Source link at the bottom of your original comment. To learn how to edit, see the Editing files in a repository article from GitHub.
Thanks again!

I considered doing that but assumed there would be procedures in place etc, which is why I created a ticket instead.
PR entered now. I didn't touch the ms.date line; I assume that's maintained by tooling. Incidentally, what tooling is used to process things like the s?

ms.date updates are manual but we don't always update it for small changes -- it's mainly for significant updates or when we've done a review to verify that the doc is up-to-date.

what tooling is used to process things like the s?

If you mean how do we correct typos like that, it's by creating a PR, having someone review it, and having someone merge it -- it's all a manual process.

Ah github removed the tag - I was referring to the xrefs etc that get resolved to links on docs.ms.com

That's done automatically when the Markdown files are converted to HTML by OPS (the Open Publishing System).


Update index.md

Fix zh_TW ChangeLog to link to master, not 1.x branch.

Nice catch, thanks.

Documents are still fetched from v1.x instead of master in all languages.

Huh? I thought the got switched... They're all supposed to be from master now..

Sorry, forgot to change that earlier, just did a quick search on the repo. I'll fix it tomorrow unless you find a few minutes for it today.

Fixed the links.


[FLINK-34130][core] Mark setBytes and getBytes of Configuration as @Internal

What is the purpose of the change
[FLINK-34130][core] Mark setBytes and getBytes of Configuration as @Internal
Brief change log
[FLINK-34130][core] Mark setBytes and getBytes of Configuration as @Internal
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.

@flinkbot run azure

Thanks all of you for the review and suggestion! Merging~


Assign site configuration to global context

The /about endpoint provides several configuration values necessary for ensuring all help/profile links are current without needing to commit and build a new release of the website. Set up a single component that makes the request for all the global content and assigns it to a global context once on first page load.

A useGlobal hook is now available to access the global variables returned from the /about endpoint.
import { useGlobal } from '@/shared/services/global';

const MyComponent = () => {
  const data = useGlobal();

return <p>{data.version}</p>;
};


Better large file error, fixes #2486

First stab.

It looks like the linter is still a little bit unhappy. There's a single trailing space on the HTTPBadRequest line. It might be better to format it like this to get around the indentation problem?
                    raise _exc_with_message(
                        HTTPBadRequest,
                        "File too large. Limit for project {} is {}.".format(
                            project.name, file_size_limit))

Hi @Miserlou -- how's this going?

(I pinged @Miserlou elsewhere and he said he'll be able to wrap this up)

In a Warehouse developers' meeting today we decided a fix to #2486 belongs in the first milestone, the MVP for package maintainers. So we're looking forward to getting an updated PR and reviewing it! Thanks.

Closing this since it's in our first milestone, so someone else can have a chance at wrapping it up sooner than later. Thanks for giving it a shot @Miserlou!


fix(check): Do not list local variables twice when renaming

Also fixes some fallout which was masked by fixing the double listing

@bjz Though this will fix the immediate error in #37 I think I will take a look at the normalizing issue as well before merging this. If you like I can merge this right away though.

Take your time - I'm in no hurry! :)

That works :)

Yep, I was just taking a look at the normalizing problem (which isn't actually caused by normalizing but by the unification traversal function changing the order of the arguments).


Support qwen1.5 in turbomind engine

Note: window attention is not supported.

Hi @lvhan028 Does this pr support Qwen1.5-0.5B and Qwen1.5-1.8B? Thanks.

0.5B no, since its head_dim is 64 while the turbomind engine hardcode the head_dim 128
1.8B Yes.

0.5B no, since its head_dim is 64 while the turbomind engine hardcode the head_dim 128 1.8B Yes.

Thanks for your reply. Maybe we could also update the supported table. Thanks.

0.5B no, since its head_dim is 64 while the turbomind engine hardcode the head_dim 128 1.8B Yes.

Thanks for your reply. Maybe we could also update the supported table. Thanks.

sure. updated.

opecompass results look OK

模型
mmlu
gsm8k

qwen1.5-7b-chat-HF
61.48
55.65

qwen1.5-7b-chat-TB
61.47
54.74

Have you tried lmdeploy convert command to convert the model?

Have you tried lmdeploy convert command to convert the model?

not yet. I'll test it asap

lmdeploy convert qwen <qwen1.5-model-path>

Traceback (most recent call last):
  File "/home/chenxin/miniconda3/envs/38/bin/lmdeploy", line 33, in <module>
    sys.exit(load_entry_point('lmdeploy', 'console_scripts', 'lmdeploy')())
  File "/home/chenxin/ws3/vl/lmdeploy/cli/entrypoint.py", line 26, in run
    args.run(args)
  File "/home/chenxin/ws3/vl/lmdeploy/cli/cli.py", line 151, in convert
    main(**kwargs)
  File "/home/chenxin/ws3/vl/lmdeploy/turbomind/deploy/converter.py", line 265, in main
    tokenizer_path = get_tokenizer_path(model_path, tokenizer_path)
  File "/home/chenxin/ws3/vl/lmdeploy/turbomind/deploy/converter.py", line 45, in get_tokenizer_path
    assert tokenizer_path, 'please supply tokenizer path by --tokenizer-path'
AssertionError: please supply tokenizer path by --tokenizer-path

Traceback (most recent call last):
  File "/home/chenxin/miniconda3/envs/38/bin/lmdeploy", line 33, in <module>
    sys.exit(load_entry_point('lmdeploy', 'console_scripts', 'lmdeploy')())
  File "/home/chenxin/ws3/vl/lmdeploy/cli/entrypoint.py", line 26, in run
    args.run(args)
  File "/home/chenxin/ws3/vl/lmdeploy/cli/cli.py", line 151, in convert
    main(**kwargs)
  File "/home/chenxin/ws3/vl/lmdeploy/turbomind/deploy/converter.py", line 265, in main
    tokenizer_path = get_tokenizer_path(model_path, tokenizer_path)
  File "/home/chenxin/ws3/vl/lmdeploy/turbomind/deploy/converter.py", line 45, in get_tokenizer_path
    assert tokenizer_path, 'please supply tokenizer path by --tokenizer-path'
AssertionError: please supply tokenizer path by --tokenizer-path

What's the command?

dose this support qwen1.5-14b-chat int8 kv cache ?
I run the calibrate cause error

RuntimeError: Currently, quantification and calibration of Qwen2ForCausalLM are not supported.

hi, @xiaoxiaoyuwen
we are going to remove offline kv int8 and adopt online kv int8 in PR #1377
PR #1412 provides the guide.
This feature will be released in the next version. Stay tuned.


When identifying a macOS or Windows platform, do not include the version

Addresses #160

Why do we want to drop this? Some browsers might still send the information, I see no reason to discard it when available.

This comment goes into detail, but, in short, specific versions can't be trusted anymore. As an example, Chrome on Windows 11 will report itself as running on Windows 10, so if you see that a user is using Windows 10, you don't know if it's the real Windows 10 or if it's actually Windows 11 claiming to be Windows 10.

Do Windows XP, 7 and 8 also spoof the OS version in user agents?

It is not the OSes that provide the useragent, it is the browser. So if you would run a recent chrome on Win8 etc, you will also not see Win8 but this static version of Windows 10.
The reason why I think we should no longer display the version of the OS is because of this: if you are logged and check your open sessions and you see Windows 10 but you do not have a Windows 10 device but only a Windows 11 device or Win8 or what you can get concerned for no reason. If we can’t know the version of the os (and with 99% of traffic this is the case) we should not show data that is mostly wrong.


This code seems to be right but it's not working

Challenge Waypoint: Size your Images has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/601.2.7 (KHTML, like Gecko) Version/9.0.1 Safari/601.2.7.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Closing as not an issue with the tests. In the future please seek help and code review in the Gitter Help Channel before filing an issue with the challenge.
Thanks and happy coding!

There’s nothing in the instructions that indicate how to do this.
It leads us to believe it needs to be done in the style section, which doesn’t work

On Nov 3, 2015, at 3:00 PM, Logan Tegman<EMAIL_ADDRESS>wrote:
You need to add the class smaller-image to your img tag for it to take effect.
Closing as not an issue with the tests. In the future please seek help and code review in the Gitter Help Channel https://gitter.im/FreeCodeCamp/Help before filing an issue with the challenge.
Thanks and happy coding!
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4150#issuecomment-153470554.

Earlier lessons like this one cover how css classes are used to style an element.

Thanks, but that’s what I’m doing and it’s still not working

On Nov 3, 2015, at 3:08 PM, Logan Tegman<EMAIL_ADDRESS>wrote:
Earlier lessons like this one http://www.freecodecamp.com/challenges/waypoint-use-a-css-class-to-style-an-element cover how css classes are used to style an element.
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4150#issuecomment-153472473.

You've added class="smaller-image" to the img tag so that it looks like ?

I really appreciate your help! But I’m still confused - here’s the instructions
CSS has a property called width that controls an element's width. Just like with fonts, we'll use px (pixels) to specify the image's width.
For example, if we wanted to create a CSS class called larger-image that gave HTML elements a width of 500 pixels, we'd use:

Create a class called smaller-image and use it to resize the image so that it's only 100 pixels wide.
This seems like the code should go in the style section, but I understand now from you it should go in the image tag?

On Nov 3, 2015, at 3:11 PM, Logan Tegman<EMAIL_ADDRESS>wrote:
You've added class="smaller-image" to the img tag so that it looks like ?
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/4150#issuecomment-153473244.

It needs to go in both. When styling using CSS class selectors you need to declare the rule (this is what you're doing in the)

-----
------ section

this is making the picture smaller but when i did this
----
----(what i did to pass)

-------
-----section

("side note" i'm more lost then ever ???????????????????)

@XavierHappyTime, you could go to the Help chat room for getting challenge related help.
Happy Coding!

Thanks for the help.

Agreed, i struggled with them one as well. Wasn't clear till I read the above.
Thanks

hi I created a class https://bit.ly/fcc-relaxing-cat it didn't work when I typed smaller image in style section

@menachemgold, Please use the Help chat room for getting challenge related help.
Happy Coding.

This is not very good, it expects you to do something from several lessons back, but that example was with an H2 element. You have to basically guess at how to apply the tag.   class="smaller-image" Has to be inserted BETWEEN  .....plain as mud and was not covered earlier. So the only way to get the answer is take up people's time in the chat. In books and tutorials I have used you are never left at having to guess at things, this should be fixed, wasting time due to lack of information is not good.

Thank you soooo much Brian123456789. You resolved my problem.

Thanks for help @Itegman

thank you baby jesus and all contributors for this thread, helped me out immensely

I'm having troubles with the same challenge. The smaller-image class works if I insert it into the tag before the other classes, but then the other classes do not work. Am I doing something wrong?
here it is:
.smaller-image {
width: 100px;
}
.thick-green-border {
border-color: green;
border-width: 10px;
border-style: solid;}

CatPhotoApp

@Riverino
Please use the Help chat room for getting challenge related help.
Happy Coding.


missing image after minting

recently I've seen a few times that the image is missing after minting
refreshing usually (but not always) solves it
yet is a bad minting experience
Reproduce:
Mint anything
seen on AHK

adding wait time on minting page would solve this? is picture right away available on image worker?

adding wait time on minting page would solve this? is picture right away available on image worker?

I think not
seems to me that as long as you don't look at the card from another page (explore/ collection/ profile)
it stays empty
once it load on another page then all works as it should

any error response from network/console?

any error response from network/console?

unrelated console error:
something to do with auto teleport?

there it is. it appeared eventually. just took verrrrryyyyy looooooong time

wow 6 min, so long
I added preheat the file, but it seems like it does not work https://github.com/kodadot/nft-gallery/blob/70a1fccd6af6b9778d02cee81bdb3f74db57d370/utils/ipfs.ts#L165

adding wait time on minting page would solve this?

how about adding this method? ensure the image return 200 and then redirect to gallery detail

how about adding this method? ensure the image return 200 and then redirect to gallery detail

wouldn't that just move the waiting to the minting page?

wouldn't that just move the waiting to the minting page?

yes actually
or
directly upload the image to our cf-images (after preheat) https://imagedelivery.net/jk5b6spi_m_-9qC4VTnjpg/CID/public
then, redirect to gallery detail page while adding the cf-images to the query params kodadot.xyz/ahk/gallery/268-2?image=https://imagedelivery.net/jk5b6spi_m_-9qC4VTnjpg/CID/public
but currently we don't have some endpoint to upload to our cf-images
with this, we can skip IPFS stuff I guess

I wonder why it began happening in the first place
didn't use to be a problem

I guess that because ipfs gateway needs time to resolve newly created assets
Scenario:

Let's say we are pinning ipfs assets to gateway-a

on image workers:

check on gateway-b, ❌
check on gateway-c, ❌
check on gateway-d, ✅. then return
this is why the worker will take longer to return, because need to check through gateway-b and gateway-c first

On image worker, it will check from ipfs gateway one by one within three gateways until resolved or return null. ref: https://github.com/kodadot/workers/blob/main/image/src/utils/ipfs.ts


Mainland & al. 2015 Incorrect molecules.csv

Hi,
Great project ! It is an awesome work done here. I just maybe noted a small issue on Mainland & al. 2015 that i wanted to report : The final molecules.csv has incorrect Odor Code and Molecule association.
It should come from the merging done on the main. Merging is done on index, and should be done on CID. Moreover, CIDs are re-computed even though they are already present in raw molecule file (Odor.tsv), and sometimes lead to wrongful new CID.
Finally, the merge implicitly delete mixtures. Maybe it's intentional.
Also, just for information, in the file 'Odor.tsv', there are several times the same molecules for different odor codes.
I suggest we can fix the merging problem by replacing in main:
odorants = pd.read_csv('Odors.tsv', sep='\t')

molecules = pd.DataFrame(from_cids(odorants[~odorants.CID.isna()]['CID'].astype(int)))

molecules = molecules.merge(odorants, on='CID') #Outer merge if mixtures are wanted

molecules = molecules.drop(['SMILES'], axis=1).drop_duplicates(subset=['Odor'])
molecules = molecules.rename(columns={'Odor': 'Odorant'})
molecules.to_csv('molecules.csv')

Cheers,
Maxence Lalis

@Maxence-Lalis
Thanks for the correction!  I am planning an audit before a release announcement later this year to catch more issues like this.  Would you like to submit a PR to make this change or would you like me to make it?

Thanks for your response! I would more than happy to submit a PR and make this change.
Moreover, I wanted to add that while testing this get_cids function (https://github.com/pyrfume/pyrfume/blob/main/pyrfume/odorants.py#L411), I noticed that sometimes instead of the CID of the isomer we get the CID of the racemic mixture. It could be seen by name (or another identifier). I understand it's a problem in data, but we should be aware of it. Maybe we can add an optional identifier in order to cross-check CID ?

I look forward to the PR.
get_cids is fundamentally limited by the accuracy of the PubChem REST API in its response to the query, and how it orders the hits, so that is not something we have much control over.  If you did want to check the quality of the result, I could imagine the logic might be very complex; for example you could check if the input and output SMILES matched, but there are a number of reasons they might not match even if the CID was correct (e.g. one used rdkit and one used some other program).  The problem would even worse for names.  But if there is a specific, consistent error that PubChem REST API makes then that is something we could temporarily patch in Pyrfume (or in pubchempy) and then ask PubChem to fix long term.
I'm actually building a new exchange identifier service to be more robust to this.

Once again, thanks for your reply. Indeed to check, the logic can be very complex and may not even be correct at the end.
Looking forward your robust new exchange identifier, I'm sure it would save a lot of people from spending hours of manually checking the molecules!

Fixed in #77.


IGNITE-12934 Introduce JUnit/Surefire integration for runtime TC test…

… reporting.
Also make sure that test start/stop messages are visible in IGNITE_QUIET mode.

This PR is associated with a resolved ticket. Closing the PR, please reopen it if you still need it.


fix bug when opening monthly and weekly notes.

problem with quarterly notes I did not fix. after looking for 3 hours. I cannot seem to find the error.

Could you specify better how to reproduce the issue?

The remaining issue with quarterly notes is the following:
When clicking on a quarter: Q3 in 2024. It does create a note.
But when I click again on it, instead of opening the file up, I get the following message:

Which makes sense, because there is already a file there. So the check does not work as expected.
When going down the rabbit hole it looked like there is something wrong with the 'daily-notes-interface' dependency.
But I am just a guy with AI, so maybe I am wrong.
The original errors where the same for weekly and monthly notes. I could create them, but not open them back up, when clicking the same month / week.

I cannot reproduce the error from my end. Could you give me your data.json of the plugin or if possible the whole .obsidian folder

You mentioned that the issue might be related to the 'daily-notes-interface' dependency. Does the error also occur with the release version available in the repository, or is it specific to your development environment? If it's the latter, keep in mind that to run the development environment, you need to apply the patches provided in the repository's patch folder.

I cannot reproduce the error from my end. Could you give me your data.json of the plugin or if possible the whole .obsidian folder

https://file.io/YwYnlOAkCIi6

You mentioned that the issue might be related to the 'daily-notes-interface' dependency. Does the error also occur with the release version available in the repository, or is it specific to your development environment? If it's the latter, keep in mind that to run the development environment, you need to apply the patches provided in the repository's patch folder.

it does also happen with the release version.
What do you mean with I need to apply the patches provided in the patch folder? How?
Thanks in advance for your patience.

I just downloaded the .obsidian folder, and you're right—I can see the bug. It appears to be related to how your template name is handled. I simplified the name to test if the issue was caused by German special characters, but it seems to be a deeper problem. I'll investigate further, but it’s likely a bug in the daily notes interface, as you mentioned.


running problem

hello!when I follow your instruction :::::
python3 main.py --algo ppo --num-frames 5000000 --num-processes 16 --num-steps 80 --lr 0.00005 --env-name MiniWorld-Hallway-v0
it went wrong as follow::::::::
Failed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionFailed to establish dbus connectionCreating frame stacking wrapper
Saving model
/home/lab1207/anaconda3/lib/python3.9/site-packages/numpy/core/_asarray.py:171: FutureWarning: The input object of type 'Tensor' is an array-like implementing one of the corresponding protocols (__array__, __array_interface__ or __array_struct__); but not a sequence (or 0-D). In the future, this object will be coerced as if it was first converted using np.array(obj). To retain the old behaviour, you have to either modify the type 'Tensor', or assign to an empty array created with np.empty(correct_shape, dtype=object).
return array(a, dtype, copy=False, order=order, subok=True)
/home/lab1207/anaconda3/lib/python3.9/site-packages/numpy/core/_asarray.py:171: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray.
return array(a, dtype, copy=False, order=order, subok=True)
Traceback (most recent call last):
File "/home/lab1207/liujiong/gym-miniworld/pytorch-a2c-ppo-acktr/main.py", line 227, in
main()
File "/home/lab1207/liujiong/gym-miniworld/pytorch-a2c-ppo-acktr/main.py", line 163, in main
np.mean(episode_rewards),
File "<array_function internals>", line 5, in mean
File "/home/lab1207/anaconda3/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 3419, in mean
return _methods._mean(a, axis=axis, dtype=dtype,
File "/home/lab1207/anaconda3/lib/python3.9/site-packages/numpy/core/_methods.py", line 188, in _mean
ret = ret.dtype.type(ret / rcount)
AttributeError: 'torch.dtype' object has no attribute 'type'
I dont know what does it mean,could you please help me solve this problem?
hoping for your reply,thanks again for your notice!!!!

No longer maintaining this reinforcement learning code, would recommend using another RL algorithm implementation


Is there a recommended way to avoid "Claims Collision" / Applying Claims to users

This is more a question about using MR with Identity Server. (Or just claims management in general)
With the need for different applications to have their own specific claims and other claims to be more generic eg. cross application claims like Name, Email etc.
Is there a recommended way of handling this scenario?
My current idea is to use the following:
Have two classifications of claims:

Profile/Descriptive claims e.g. FirstName, LastName, Phone etc.
Security Claims e.g. EditAllowed, ReadAllowed, CreateNewItem etc.

In order to avoid one claim being "mistakenly" applied from one application to the next I am thinking of applying the following naming convention:
ApplicationName.ContextualElement.ClaimName
So for example if we had 100 different applications they would each start off the claim name with their specific application name. e.g. Application1.xx.yy , Application2.xx.yy, Service1.xx.yy etc.
Then the second element would be a contextual identifier such as Profile, Security, General etc.
The final element would be the actual claim e.g. ProfileEdit, UpdateRecords etc.
Now in order to manage Access Rights more easily I have two ideas:

Add any new security rights (security claims) to all users and have disabled by default, then explicitly apply Allowed access.
Only Add security rights as and when allowed access is granted.

In my current non-ids solution I am using option 2 but I wonder if option 1 is the right way to go as this then ensures all my users have all the security claims added and then it is a simple case of switching them on and off within my User Management system rather than having to remember which users have access to these rights.
My only concern with Option 2 is that the cookie being generated could get very large. I guess the way round this would be to have a "basic" cookie and then do a check on the server everytime for the appropriate access (although would this introduce more issues with network overhead/ performance issues) using this "remote" checking solution for every security right would allow for security changes to be registered straight away without the user needing to log out and back in again.
So I guess this is more a open question to the community and just wondering what others are doing in this situation.

In IdSvr 2.0.0 we've added the client ID making the token request to the user service get get profile API. This allows you to decide what claims to return to which client.
As for the storage of the claims and how they map to clients -- yea, MR doesn't have that concept, so your naming convention might just be as good as anything.

It's fine Brock, I am sure you are a very busy boy with all the various conferences that seem to be occurring this year.
Is this same feature of getting the profile data/claims for the calling app in idSvr 3.0 as well?
Well I have implemented my "solution" so we will have to see how well it works. I think the concept of it is sound but only time will tell once we have implemented it into a number of solutions. and see how flexible it is. As for forcing a refresh of the Token from the client is it possible to point me in the right direction within idSvr 3.0 to do this as it would be a nice feature to implement "under the hood" to get people the access rights straight away rather than having to log out and back in again as we are currently having to do during testing.

Sorry -- I meant IdSvr3 build 2.0.0 in my prior statement.

Is Build 2.0.0 out yet as it seems to be 1.6.2 as the latest build.
edit: sorry I can see 2 is currently in rc1 mode. I guess we will hang on to using 1.6.2 for now as we are still getting to grips with it and changing to an rc1 version is a bit too risky at the minute.
I will have a look into the new features and respond back via the IdSvr project if I have any issues.


Enhancement: Improve JSON serialization

:scroll: Description
All non-encodable classes are now serializable due to calling toString() on them.
:bulb: Motivation and Context
Reduce  errors while serializiation
Closes #523
:green_heart: How did you test it?
I added tests.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[ ] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #528 (b3aaaf3) into main (c3dbab0) will increase coverage by 0.07%.
The diff coverage is 93.33%.

:exclamation: Current head b3aaaf3 differs from pull request most recent head 27f58a2. Consider uploading reports for the commit 27f58a2 to get more accurate results

@@            Coverage Diff             @@
##             main     #528      +/-   ##
==========================================
+ Coverage   91.42%   91.49%   +0.07%
==========================================
  Files          74       67       -7
  Lines        2425     2176     -249
==========================================
- Hits         2217     1991     -226
+ Misses        208      185      -23

Impacted Files
Coverage Δ

dart/lib/src/sentry_envelope_item.dart
97.14% <90.90%> (+7.95%)
:arrow_up:

dart/lib/src/sentry_envelope.dart
100.00% <100.00%> (ø)

dart/lib/src/utils.dart
100.00% <100.00%> (ø)

.../lib/src/navigation/sentry_navigator_observer.dart

...tter/lib/src/flutter_enricher_event_processor.dart

flutter/lib/src/file_system_transport.dart

flutter/lib/src/sentry_flutter_options.dart

flutter/lib/src/widgets_binding_observer.dart

flutter/lib/src/sentry_flutter.dart

flutter/lib/src/flutter_sentry_attachment.dart

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 c3dbab0...27f58a2. Read the comment docs.

looks like CI isnt happy today


Strange behave in tdlib

Hello everyone!
I couldn't find a similar problem.
For some reason tdlib starts sending chats and channels that my contacts are subscribed to. How to disable this behavior?

This is expected behavior. You can receive updates about users and chats any time.

This is expected behavior. You can receive updates about users and chats any time.

But how do you stop doing that? I only want my chat, but not my friends' channels and chats

You can't. App needs all of the updates.

You can't. App needs all of the updates.

It's a strong omission, I guess ...
Okay, how am I supposed to filter this case? Any extra fields for those chats that don't belong to me?

You receive all information about the chats and can filter them in any way using the fields in the object.


Search filter for all fields

In the order filter, one can specify ~ to indicate that ordering for any field should be supported.
The sort filter doesn't seem to support that feature, would it be possible to have this added, or should I rather implement an own filter?

It's supported by all existing filters: https://github.com/dunglas/DunglasApiBundle/blob/master/Resources/doc/filters.md#enabling-a-filter-for-all-properties-of-a-resource

Gotcha. However, specifying exact and partial isn't supported in that scenario, correct? Also, I guess that specifying "all fields must be exact matches, but fields X and Y can be partial matches" isn't possible?

Exactly.

I think I need to implement an own filter class for that. Thank you!


EKS cluster setup

Prepare terraform scripts to setup EKS cluster. Not integrate with epiphany code yet, might be some new directory.

Ticket owner:

Please, put into description, which components are mandatory (from our side) to deploy complete EKS cluster. For example: IAM, VPC, storage etc.
Which parameters are expected in configuration, to smoothly integrate it with our platform.

Summary:
From my side, I can own this ticket as its super interesting! As a starting point, I suggest to go thru official TF docs and AWS TF modules:

https://learn.hashicorp.com/terraform/kubernetes/provision-eks-cluster
https://github.com/terraform-aws-modules/terraform-aws-eks
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/docs/autoscaling.md

@mum0l I don't think we need IAM in that phase. VPC, Security Groups, AutoScaling Groups, EKS Cluster, Storage, etc, would be ok. As of parameters that would be considered in #1445 and #1448.

This task is similar to #1440 in it's aim. It should be discussed between persons preparing those scripts.

There is comment from @rpudlowski93 regarding networking drivers.

@rpudlowski93 can you test this?

Tests done:

12.2.0 version of aks module require at least 0.12.9 version of terraform, in our case terraform in version 0.12.6 works fine with 7.0.0 module of aks - tested.
cluster creation works fine on Linux OS but there is a issue when You want to run the cluster creation process on Windows OS - it looks like a bug reported here: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/607


[FEAT] access expandable rows grid api from parent row

While creating a global filter for my grid, I ran into some small obstacle.
My filter needed to filter all of the data. For instance, I have customers list with purchases for each customer. So I have the main grid to be a grid of customers and every expandable shows their purchases.
Now, I want to have a global search field (e.g. input) that searches either by customer name (simple) or by product name (e.g. filter the expandable grids, and show only customers with purchases of products that fulfill the criteria).
When getting to filter the expanded grids I ran into a problem - I had no access to the API of the expanded grid...
My workaround for this was to add onRegisterApi for each subgrid and add it to the data itself:
onRegisterApi: function(gridApi){this.gridApi = gridApi;}

This way I had access to the grid api from the gridOptions of each expanded grid (this was actually not working well in version rc.19. For some reason, the wrong gridApi would be sent to my row. It seems to have been fixed in rc.20).
I thought it might be nice to have such access from parent row to child grid built in the system (so one could traverse all the grid from any element currently processed).

This would also be useful when trying to save the state of the subGrid - at present we have no easy way to save it.  I would suggest that we put it on the gridRow rather than the data though, putting it on the data seems against the general principle of the grid, and I guess might cause garbage collection problems when the grid is destroyed (the data would still be holding a reference to the grid).

Checking if there could be an update on this enchantment.
If there is not, would you recommend to update the parent GridRow  with the subgrid API. I can do this and commit and raise a pull request.


Fr/requests

Outside:
auth = vManageAuth(base_url, username, password, verify=False)
vManage_session = vManageSession(url, username, password, auth=auth)
r = vManage_session.server()

UI:
s = create_vManageSession(url, username, password)
print(s.session_type)

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/33


Update sbt-mima-plugin to 0.7.0

Updates com.typesafe:sbt-mima-plugin from 0.6.4 to 0.7.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe", artifactId = "sbt-mima-plugin" } ]

labels: sbt-plugin-update, semver-minor

@larsrh looks like the native build can't find 0.3.1. It looks like 0.3.1 isn't there:
https://search.maven.org/search?q=a:paiges-core_native0.3_2.11
Did the publish of native fail?

It's there, but for Scala Native 0.4.0-M2: https://oss.sonatype.org/content/repositories/public/org/typelevel/paiges-core_native0.4.0-M2_2.11/0.3.1/

any idea why the build is failing? I restarted it once, got the same error.

No clue, sorry.

I can backport a release for older native and sjs versions (doing that in a moment).

I ran out of time because apparently it is very hard to perform a manual release when sbt-ci-release is involved.

I'm willing to disable the native build to get passed this. Does anyone even use the artifact we make on native?

No idea, but @olafurpg maybe for scalafmt? Or is he using native-image?

We’re using native image. Feel free to disable the scala native build.
sbt-ci-release doesn’t change the manual publish step, you can always publishSigned locally like normal.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #245   +/-   ##
=======================================
  Coverage   99.51%   99.51%
=======================================
  Files           7        7
  Lines         415      415
  Branches       42       42
=======================================
  Hits          413      413
  Misses          2        2

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 c5e0062...d1d1a27. Read the comment docs.


HISTORY PAGE - calendar

right side calendar cant go back to previous month

The right calendar cannot go before the left calendar. Report back if this is not the case.


Replace dic-iit with ami-iit after change of name of organization

The CI failure is unrelated and will be fixed in the next PR. Merging


Index.html/Homepage JavaScript Random Button

Starting this issue to say that I have added this.

Button added, js added as well


Please add an Upstart job.

I have written one, however I do not know where to place it so that the Deian package ships with it. In addition, the "upgrade" option is not provided for.
Here is the job: https://gist.github.com/CameronNemo/c337e99c091589e7a496

@CameronNemo under debian, what is system location to which the script must be installed?

/etc/init/

Does this interfere with systemd or can it always be installed?

Does not interfere.

I think we can definitely close the upstart chapter, and lean on remaining users to use the system V init scripts. https://en.wikipedia.org/wiki/Upstart


Leaflet.PM sources

Where can I find the source code of Leaflet.PM version 2.2.0?

Here: https://github.com/geoman-io/leaflet-geoman/tree/2.2.0

Thanks ;)


PT-BR - C Book - Graph Algorithms

What does this PR do?
Add resource(s)
For resources
Description
This is a book that has graph algorithms developed in the C language and are taught in data structure and algorithm design courses
Why is this valuable (or not)?
It optimizes and fixes the knowledge of how this data structure works on a basic level, without the use of structures that facilitate this knowledge, so that the students know how to identify and construct methods and graphs that are functional.
How do we know it's really free?
This content is from a professor at a public university, which is free to study, building a public material and posting it on the internet to help different students including his students
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[X] Read our contributing guidelines.
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction).
[X] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Could you share the code to generate the low-light images based on raw RGB images?

Thank you for your interest. In this work, the low-light images are adjusted using Adobe Lightroom. In our previous work (LLCaps), we used gamma correction and brightness reduction, which can be easily implemented using OpenCV. Thanks.


Pre-release changeset edits

This PR edits the Changeset notes for our upcoming release.
At the moment, I'm of the opinion that we should aim to be more concise in the actual changelog notes, and improve some of the PR writeups they link to, with additional context, usage notes, implementation details, etc. For example, I've cut way back on the detail about the Pagination component, but did some polish on the PR itself. The changeset itself isn't the place to document tradeoffs and edge cases, I think.
Checklist

[x] I've read the Contributing Guidelines

Great updates! I'm a bit confused about the skeleton changesets, though.
I thought we were going to use these to provide upgrade steps to the users with diffs, etc. to use them automatically in the h2 upgrade command (eventually). However, the updated changesets don't really show steps or anything like that.
Should we put the upgrade steps somewhere else?
cc @juanpprieto


Consider removing gc_collect_cycles() from DataObject::destroy()

I have a GridField with around 800 items in it, removing gc_collect_cycles() halves the CPU time for print or CSV exports (both GF components call $item->destroy() on each DataObject) and has no impact whatsoever on memory usage, or peak memory usage.
Is PHP clever enough nowadays that we can just leave it alone to handle this itself?

How does that affect mem usage? Removing from milestone, since it's marked as low impact

XHProf runs suggest there’s absolutely no impact on memory at all. I’ve run two types of test:

benchmarking the request for a GridField print view (~800 items)
creating, and then subsequently deleting, DataObjects and triggering destroy():

for ($i = 0; $i < 1000; $i++) {
    TestObject::create()->update(['Foo' => 'Bar'])->write();
}

foreach (TestObject::get() as $item) {
    $item->delete();
    $item->destroy();
}

die('done');

gc_collect_cycles() doesn’t cause such a (relatively) large slowdown in the second example, because most of the time is taken up with database queries - it adds around 10% extra CPU time for 1000 objects created/deleted.
I noticed this when benchmarking why the GridField print and CSV export buttons were so slow for my app. There’s only one database query to fetch all the items for rendering, but then ->destroy() is called on each of them so the relative time that gc_collect_cycles() is taking is much higher, around 50%.
So we could either:

Remove gc_collect_cycles() from DataObject::destroy()
Remove calls to DataObject::destroy() from GridFieldPrintButton::generatePrintData() and GridFieldExportButton::generateExportFileData()

what version of PHP are you using? I have no doubt PHP 7 is much better at this now.
I'd be happy to get it out of SS 4, maybe SS 3 depending on impact on older versions of PHP...

Tested on 5.6, I don't have XHProf set up for 7 (I don't know if there's
even a PHP 7 compatible version)
On Fri, 1 Sep 2017 at 18:05, Daniel Hensby<EMAIL_ADDRESS>wrote:

what version of PHP are you using? I have no doubt PHP 7 is much better at
this now.
I'd be happy to get it out of SS 4, maybe SS 3 depending on impact on
older versions of PHP...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/silverstripe/silverstripe-framework/issues/7326#issuecomment-326633863,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABlC_D4aq42vheW32_DWJrRag8Nskt4sks5seDlQgaJpZM4PHbpQ
.

Sorry I just realised you've addressed memory usage in your original report - too much skimming during bug triage :) In that case, happy to review a PR removing gc_collect_cycles() from SS4, it's always been hacky.

PR https://github.com/silverstripe/silverstripe-framework/pull/7334


refactor: split languages/clients into packages

Saw @zackradisic's comment about splitting up the packages from this project and figured I'd drop a proof-of-concept.
Intro
I think it's a great idea to split up the packages, especially since it looks like each language has a different set of dependencies, and the separation would be easier to maintain.
This splits into two parts:

Create a clean interface for each language/client package to implement
Monorepo infrastructure to split up packages (part 1 of this pr)

A great example of a project that does this is Expo. Check out the packages folder specifically.
Lerna is a great tool to accomplish this, and I've used it in the past for previous large mono-repo package styles projects. For those new to the tool, here's a great example.
Demo

cd run-wasm
yarn
yarn bootstrap This will install/build all packages
cd example-nextjs
yarn You may need to npm link each package if you're having trouble
yarn dev

See the project works with split packages! 🙌 🎉
What's next
It looks like each client uses the following, but not sure if loadPackages or a bootstrap should be included.
type RunArgs = {
  code: string
};

interface IWasmClient {
  run(args: RunArgs): Promise<string>;
}

Also curious on folk's opinion for exporting since the example used is
import ts from `@slip/run-run-wasm-ts`

not
import { createTSClient } from '@slip/run-run-wasm-ts'

In order to get deploy previews working, you'll have to prepare the other packages, THEN install the demo app.
here's the vercel override build command I used
pushd ../ && yarn && yarn bootstrap --no-ci && popd && yarn
Example: https://run-wasm-wheat.vercel.app/

@kennethcassel and I chatted -
Since the "run-wasm" is (for the most part) empty, it may be a great place to add the react interface and any UIs in a follow up Pr.

Ahh okay, then maybe the core run-wasm could be that then? The nice react interface and ui? You just import which clients you need and plug it in, or use the modules by themselves if you’re DIY

React-data-grid structures it that ways. It’s the core, then you install any addons like toolbars and UI

Instead of using this naming convention - run-wasm-python, wouldn't it be a better idea to use something like - @run-wasm/python?

Instead of using this naming convention - run-wasm-python, wouldn't it be a better idea to use something like - @run-wasm/python?

Totally, I think there's a couple ways. I heard @slip/run-wasm-python as well. I'd love @slipHQ and maintainer friends to chime in on what (slightly permanent) name's yall would like.

Instead of using this naming convention - run-wasm-python, wouldn't it be a better idea to use something like - @run-wasm/python?

Yeah I like this naming convention. Using @run-wasm instead of @slip is preferred. It's not unlikely that we will end up rebranding Slip to another name in the future.

Awesome work @cameronmoreau! Got the vercel build preview working, but looks like the  jest tests are failing now

Instead of using this naming convention - run-wasm-python, wouldn't it be a better idea to use something like - @run-wasm/python?

I agree! I like this naming convention too. Plus, it's unlikely you'd be able to get run-wasm-[language] for all packages on npm so having it under an org namespace increases our ability to keep package names consistent.

Hmm...the tests can't run because the tsc check is failing:
$ tsc
src/index.tsx(2,19): error TS2307: Cannot find module 'react' or its corresponding type declarations.
src/index.tsx(20,5): error TS2503: Cannot find namespace 'JSX'.
src/index.tsx(20,5): error TS4060: Return type of exported function has or is using private name 'JSX'.
src/index.tsx(22,5): error TS7026: JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists.
src/index.tsx(25,7): error TS7026: JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists.
src/index.tsx(25,19): error TS7026: JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists.
src/index.tsx(26,5): error TS7026: JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists.

Maybe there's an error in the tsconfig? There's an example here

Love it, what about the main package though? @kennethcassel lmk and I'll update all the package names

@run-wasm/core ?
@run-wasm/run-wasm ??
@run-wasm/react???
Naming is hard

@kennethcassel oops, nice catch. Think I fixed tests. Although I can't figure out how to get them to run in this pr 🤔 Only Vercel deploy is triggered. yarn test runs fine. Think it was a gh action setup issue.

@jsjoeio are you running tsc to build or check types? Since the packages are split, they each have their own tsconfig/build (so they can be published independently). This means:

The top level directory is now just a workspace and has base config files each package imports
You can't just run tsc or jest top (workspace) level, but rather check-types and test (their alias)

Running yarn check-types and yarn test - Lerna will go into each package and run those commands individually, like the independent packages they are.
Hope this helps. Checkout lerna commands and this example for more info.

@jsjoeio are you running tsc to build or check types? Since the packages are split, they each have their own tsconfig/build (so they can be published independently). This means:

Thanks for the detailed response! So neither (when I set it up) 🤔 (I think). We were just using jest to run tests. I believe when it runs, it does type checks as well (i.e. so if you have any TS issues, Jest won't run your tests). Hope that helps. Looks like you got it working just fine 🙌

Getting this issue on the vercel build

12:48:28.039
Failed to compile.

12:48:28.040
./pages/index.tsx:2:36

12:48:28.040
Type error: Cannot find module '@run-wasm/python' or its corresponding type declarations.

12:48:28.040
1 | import React, { useEffect, useState } from 'react'

12:48:28.040
> 2 | import { createPythonClient } from '@run-wasm/python'

12:48:28.041
|                                    ^

12:48:28.041
3 | import Editor from '@monaco-editor/react'

12:48:28.041
4 | import Script from 'next/script'

12:48:28.041
5 | import GithubButton from '../components/GithubButton'

12:48:28.063
error Command failed with exit code 1.

12:48:28.063
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

12:48:28.087
Error: Command "yarn run build" exited with 1

@kennethcassel did some digging and for some reason once adding the @run-wasm scope, the packages no longer get automatically built when running yarn bootstrap. This is odd because each package has npm run build in their pacakge.json's prepare.
Even more odd because yarn bootstrap properly builds on my machine, but not vercel's deployment. Soooo, as a fix you can add yarn build to the top level repo. I can do some digging on what happened here (because now I'm curious), and publish any findings in a subsequent PR.
TLDR; update the build command (one last time) to -
pushd ../ && pwd && yarn && yarn bootstrap --no-ci && yarn build && popd && yarn

This is odd because each package has npm run build in their pacakge.json's prepare.

@cameronmoreau is this helpful? Wish I had more experience with Lerna to help. Or could it be that you're mixing yarn and npm commands?

TLDR; update the build command (one last time) to -

Trying to test after updating the command but vercel builds are degraded right now!  https://www.vercel-status.com/incidents/dtmzmybbbzyh
Will test soon :) hoping to get this one merged in soon! Thanks for all the work @cameronmoreau 🎉


Run bug of 0.3.10

What is the issue?
I'm sorry for raising this problem. This problem mainly comes from when I installed ollama0.3.10, I was prompted that I could not find VCRUNTIME140.dll, MSVCP140.dll, VCRUNTIME140_1.dll, and Error was displayed: llama runner process has terminated: exit status 0xc0000135, so I came to seek answers from netizens
OS
Windows
GPU
No response
CPU
Intel
Ollama version
0.3.10

I tried 0.3.9 with the same result

https://aka.ms/vs/17/release/vc_redist.x64.exe
https://aka.ms/vs/17/release/vc_redist.x86.exe

I tried 0.3.11 on windows，with the same result：
“ollama_llama_server.exe - 系统错误：由于找不到VCRUNTIME140_1.dll，无法继续执行代码。重新安装程序可能会解决。”
But I succeeded with version 0.3.2.

@nikelius the fix is in 0.3.12

Hello, I am having the same problem. I have ollama version 0.3.14 installed on windows 11 version 23H2, MSVCP140.dll is not found.

@LeonelRFF can you try the latest 0.4.0 RC and see if that clears it up?  If not, please share the app.log and server.log if present.

Perfect in version 0.4.0 RC it works.


Update LAB_AK_05_Lab1_Ex1_Deploy_Sentinel.md

Need a Header
Changes proposed in this pull request:

The CSV file need a Header to be update.

The instructions already include adding a header. See Task 2, step 2.


Exposing SlackRubyBot::CommandsHelper.find_command_help_attrs

@dblock - per #172

Merged.
I wonder whether we should rename this method and refactor this a bit now that we're trying to expose this publicly.

CommandsHelper is a confusing utility name, because it's really about Help, so maybe it should just be CommandsHelp?
CommandsHelper#find_command_help_attrs(name) could be just Help#get(name).
#capture_help should be private and executed lazily when someone tries to get anything from Commands the first time.
commands_help_attrs becomes an internal attrs

Thoughts?

✅ ✅ ✅  & ✅
spot on, would really simplify the interface.

Looking forward to a PR :)

haha ok, will put on my list!