Add example for modifying textures on the CPU

The example shows how to modify images using the CPU and then display them.
It binds into the render pipeline by AssetRenderResourcesNode, so I placed it in shaders/.

I figured out how to position the texture within the quad, so the example now does it.

You don't need a custom resource or a shader to do that 🙂
This will do the same (adapted from your example and removed the resource, the shader and the pipeline):
use bevy::{
    prelude::*,
    render::texture::{Extent3d, TextureDimension, TextureFormat},
    utils::Duration,
};
use std::num::Wrapping;

/// This example illustrates how to mutate a texture on the CPU.
fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        // .add_asset::<MyTexture>()
        .add_startup_system(setup.system())
        .add_startup_system(setup_timer.system())
        .add_system(timer_tick.system())
        .run();
}

fn setup(
    mut commands: Commands,
    mut textures: ResMut<Assets<Texture>>,
    mut materials: ResMut<Assets<ColorMaterial>>,
) {
    // Create a texture with varying shades of red.
    let texture = Texture::new_fill(
        Extent3d {
            width: 16,
            height: 16,
            depth: 1,
        },
        TextureDimension::D2,
        &(0..(256))
            .flat_map(|i| vec![255, i as u8, 0, 255])
            .collect::<Vec<u8>>(),
        TextureFormat::Rgba8UnormSrgb,
    );

    let texture_handle = textures.add(texture);

    commands.spawn_bundle(OrthographicCameraBundle::new_2d());

    commands.spawn_bundle(SpriteBundle {
        material: materials.add(texture_handle.clone().into()),
        transform: Transform::from_scale(Vec3::splat(30.0)),
        ..Default::default()
    });
    commands.insert_resource(texture_handle);
}

struct TickTrack {
    count: Wrapping<u8>,
    timer: Timer,
}

fn setup_timer(mut commands: Commands) {
    commands.insert_resource(TickTrack {
        count: Wrapping(0),
        timer: Timer::new(Duration::from_secs(1), true),
    });
}

fn timer_tick(
    time: Res<Time>,
    mut timer: ResMut<TickTrack>,
    mut textures: ResMut<Assets<Texture>>,
    pic: Res<Handle<Texture>>,
) {
    timer.timer.tick(time.delta());
    if timer.timer.finished() {
        timer.count += Wrapping(1);

        let texture = textures.get_mut(&*pic).unwrap();
        let idx = (timer.count.0 as usize) * 4 % texture.size.volume();
        // Each timer interval, one pixel turns black.
        texture.data[idx..(idx + 3)].iter_mut().for_each(|i| *i = 0);
    }
}

In hindsight, this is quite obvious.
So what I did was conflate 2 problems:

Modifying an Asset
Supplying a texture to a shader via AssetRenderResourcesNode

Problem 2. is addressed in array_texture, a similar in mesh_custom_attribute. I would like to see how to pass an array of non-pixel data, but maybe that's not possible, and I can't do it anyway.
Problem 1. is better expressed by what you supplied here. I still think it's useful, because it demonstrates two things that may not be clear:

how to load assets from memory
how to modify things behind a handle

Should that be added as an example?

Yeah I think illustrating how to create custom textures and modify them at runtime is useful (and I also agree that we should scope this to (1) for simplicity).

@dcz-self I started writing this exact example then found your PR 🤦♂ There are a few small tweaks for the latest version of Bevy (below), then hopefully this can be merged! It would have saved me a lot of time at least. Not because Bevy is confusing... but because I was looking around for the unintuitive API I would have expected from Unity/Unreal for at least an hour before I realized how strikingly simple it was 😂
use bevy::{
    prelude::*,
    render::texture::{Extent3d, TextureDimension, TextureFormat},
    utils::Duration,
};
use std::num::Wrapping;

/// This example illustrates how to mutate a texture on the CPU.
fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup.system())
        .add_startup_system(setup_timer.system())
        .add_system(timer_tick.system())
        .run();
}

fn setup(
    mut commands: Commands,
    mut textures: ResMut<Assets<Texture>>,
    mut materials: ResMut<Assets<ColorMaterial>>,
) {
    // Create a texture with varying shades of red.
    let texture = Texture::new_fill(
        Extent3d {
            width: 16,
            height: 16,
            depth_or_array_layers: 1,
        },
        TextureDimension::D2,
        &(0..(256))
            .flat_map(|i| vec![255, i as u8, 0, 255])
            .collect::<Vec<u8>>(),
        TextureFormat::Rgba8UnormSrgb,
    );

    let texture_handle = textures.add(texture);

    commands.spawn_bundle(OrthographicCameraBundle::new_2d());

    commands.spawn_bundle(SpriteBundle {
        material: materials.add(texture_handle.clone().into()),
        transform: Transform::from_scale(Vec3::splat(30.0)),
        ..Default::default()
    });
    commands.insert_resource(texture_handle);
}

struct TickTrack {
    count: Wrapping<u8>,
    timer: Timer,
}

fn setup_timer(mut commands: Commands) {
    commands.insert_resource(TickTrack {
        count: Wrapping(0),
        timer: Timer::new(Duration::from_secs(1), true),
    });
}

fn timer_tick(
    time: Res<Time>,
    mut timer: ResMut<TickTrack>,
    mut textures: ResMut<Assets<Texture>>,
    pic: Res<Handle<Texture>>,
) {
    timer.timer.tick(time.delta());
    if timer.timer.finished() {
        timer.count += Wrapping(1);

        let texture = textures.get_mut(&*pic).unwrap();
        let idx = (timer.count.0 as usize) * 4 % texture.size.volume();
        // Each timer interval, one pixel turns black.
        texture.data[idx..(idx + 3)].iter_mut().for_each(|i| *i = 0);
    }
}

@AThilenius don't hesitate to take this code and submit it as your own - I am currently not playing with Bevy and can't promise I will pull this one to completion.

@superdump this is obsoleted by the 0.6 rendering rework right? Should we close it out or work to port it?
Also @dcz-self could you drop a comment in #2373 so we can merge this in under MIT + Apache? :)

the Texture will have to be replaced by the new Image struct, but other than that it shouldn't be very different. It still has a data fields that's one big array

I agree. I think it’s a useful example to have and so to update to the new renderer.

I commented: https://github.com/bevyengine/bevy/issues/2373#issuecomment-989914183
Feel free to alter this. I'm currently not playing with bevy, so it'd be a bit of a pain to come back to this.

I'll close this out then. Someone else is welcome to pick up the torch and open a new pr if they are so inclined.


In Google Colab StructuredTool raises an error when instantiated

System Info
Environment:
Google COLAB project. When running !cat /etc/os-release it prints:
PRETTY_NAME="Ubuntu 22.04.2 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.2 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Python Environment
!python -V:
Python 3.10.12

!pip show langchain:
Name: langchain
Version: 0.0.267
Summary: Building applications with LLMs through composability
Home-page: https://www.github.com/hwchase17/langchain
Author:
Author-email:
License: MIT
Location: /usr/local/lib/python3.10/dist-packages
Requires: aiohttp, async-timeout, dataclasses-json, langsmith, numexpr, numpy, openapi-schema-pydantic, pydantic, PyYAML, requests, SQLAlchemy, tenacity
Required-by:

Who can help?
@hwchase17
Information

[x] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Create a blank Google Colab Project (https://colab.research.google.com/#create=true) then add a code block and paste in this code:
!pip install langchain==0.0.267
# or try just '!pip install langchain' without the explicit version

from pydantic import BaseModel, Field

class InputArgsSchema(BaseModel):
  strarg: str = Field(description="The string argument for this tool")

# THIS WORKS:

from typing import Type

class Foo(BaseModel):
  my_base_model_subclass: Type[BaseModel] = Field(..., description="Equivalent to the args_schema field in langchain/StructuredTool")

my_foo = Foo(
    my_base_model_subclass=InputArgsSchema
)

print(f"My foo {my_foo} is successfully instantiated")

# BUT THIS DOES NOT:

from langchain.tools import StructuredTool

def my_tool_impl(strarg: str):
  print(f"Called myTool with strarg={strarg}")

my_tool = StructuredTool(
    name="my_tool",
    description="A demo tool for testing purposes",
    args_schema=InputArgsSchema,
    func=my_tool_impl
)

Now run the code block.
Expected behavior
The StructuredTool instance should be instantiated without an exception and the InputArgsSchema should be accepted as argument for args_schema.

Some updates on the topic:
The exact output and error message i get in the Colab environment (as suggested by @dosu-bot):
My foo my_base_model_subclass=<class '__main__.InputArgsSchema'> is successfully instantiated
---------------------------------------------------------------------------
ValidationError                           Traceback (most recent call last)
[<ipython-input-9-8e060e10267e>](https://localhost:8080/#) in <cell line: 28>()
     26   print(f"Called myTool with strarg={strarg}")
     27
---> 28 my_tool = StructuredTool(
     29     name="my_tool",
     30     description="A demo tool for testing purposes",

[/usr/local/lib/python3.10/dist-packages/pydantic/v1/main.py](https://localhost:8080/#) in __init__(__pydantic_self__, **data)
    339         values, fields_set, validation_error = validate_model(__pydantic_self__.__class__, data)
    340         if validation_error:
--> 341             raise validation_error
    342         try:
    343             object_setattr(__pydantic_self__, '__dict__', values)

ValidationError: 1 validation error for StructuredTool
args_schema
  subclass of BaseModel expected (type=type_error.subclass; expected_class=BaseModel)

I can also reproduce this bug locally like so:

Setup a new blank venv and activate it
pip install langchain 0.0.267
remove the colab pip install statement from the code snippet (the "!pip install ...")
Run it

I debugged it locally and found that it seems like it is checking that args_schema is a subclass of the BaseModel in .../site-packages/pydantic/v1/main.py however my class inherits from the BaseModel in .../site-packages/pydantic/main.py (note the missing 'v1' sub-directory).
So to address the issue/bug again it should work with the BaseModel without the 'v1' part as this is the one thats imported when running from pydantic import BaseModel.

@dosu-bot I did provide the error message in my comment

@Markus-sevend thanks for reporting the issue see:

https://github.com/langchain-ai/langchain/discussions/9337
https://github.com/langchain-ai/langchain/blob/master/docs/extras/guides/pydantic_compatibility.md

For me, installing pydantic 1.10.10 worked: pip install pydantic==1.10.10


First-class API to replace tokens while copying files

Substituting values for tokens in certain resource templates is a very common use case. Currently the recommended way to do it is like so:
processIntegrationTestResources {
    inputs.files(configurations.corePersistenceEntitiesJar)
            .withPropertyName('configurations.corePersistenceEntitiesJar')
            .withPathSensitivity(PathSensitivity.RELATIVE)
    inputs.files(configurations.xxxPersistenceEntitiesJar)
            .withPropertyName('configurations.xxxPersistenceEntitiesJar')
            .withPathSensitivity(PathSensitivity.RELATIVE)

    eachFile {
        if (it.sourcePath == 'META-INF/persistence.xml') {
            it.filter(ReplaceTokens, tokens: [
                    corePersistenceEntitiesJar: configurations.corePersistenceEntitiesJar.asPath,
                    xxxPersistenceEntitiesJar: configurations.xxxPersistenceEntitiesJar.asPath,
                    classesDir: sourceSets.main.java.outputDir.path
            ])
        }
    }
}

There are a number of problems with this:

the code is very verbose and somewhat counter-intuitive (the user's intention is not to process each file for example)
it is easy to forget to manually register inputs, or to register them incorrectly (note that the names of the tokens are not captured here as inputs at all)
you need to reference a class from Ant (ReplaceTokens)

Clearly, we can do better here. Something like this might be an option:
processIntegrationTestResources {
    replaceTokens("META-INF/persistence.xml",
        corePersistenceEntitiesJar: configurations.corePersistenceEntitiesJar.map { it.asPath },
        xxxPersistenceEntitiesJar: configurations.xxxPersistenceEntitiesJar.map { it.asPath },
        classesDir: sourceSets.main.java.classesDirectory.map { it.path }
    )
}

This way we could properly capture all the inputs without the user having to specify anything, and by using mapped Providers we'd keep track of dependencies on the inputs as well.

cc: @gradle/execution

+1
would be great if we had some better API for binary filtering too.

Would it also be possible to specify the timestamp on the resulting file?
Binary filtering sounds great to solve a lot of caching issues.

Could you clarify what you mean by binary filtering? It might make sense to open a separate issue, too, if it's a sufficiently different problem. We can deliver them together even so.

I was just made aware of AbstractCopyTask.expand(), which has somewhat different characteristics:

it involves executing Groovy code as part of the substitution, which might be problematic because of Groovy itself, or because it makes the replacements Turing-complete,
it applies to all files within a CopySpec (can work around by declaring files to process as a different CopySpec, which might actually be good practice),
it also does not track dependencies.

It would probably make sense to extend expand()'s logic to handle Providers as inputs. I think scoping to a given set of files could actually work better via a separate CopySpec.

There's an existing issue for binary filtering, which has been marked as solved: https://github.com/gradle/gradle/issues/4010
I have personally faced this problem several times, that the only available filtering is on text.

It seems to me that the best way to handle binary filters would be to reopen #4010.

One other major issue with expand() is that its configuration is currently not tracked as an input. :/

unstale


[Adaptive] ListProperty on ForEachPage step is inconsistent with ForEach step

ListProperty on ForEach step expects a string while the exact same property in ForEachPage step expects an expression. We should be consistent and accept a string in both cases.
var rootDialog = new AdaptiveDialog(nameof(AdaptiveDialog))
{
    Generator = new TemplateEngineLanguageGenerator(),
    Rules = new List<IRule> ()
    {
        new UnknownIntentRule()
        {
            Steps = new List<IDialog>()
            {
                new SetProperty() {
                    Property = "turn.colors",
                    Value = "createArray('red', 'blue', 'green', 'yellow', 'orange', 'indigo')"
                },
                new ForeachPage() {
                    ListProperty = "turn.colors",
                    PageSize = 2,
                    Steps = new List<IDialog>() {
                        new SendActivity("Paging through these items in the collection '{join(dialog.value, ',')}' in the collection!")
                    }
                }
            }
        }
    }
};

Error:
Cannot implicitly convert type ‘string’ to ‘Microsoft.Bot.Builder.Expressions.Expression’ for ListProperty

Why wouldn't we just use expressions?  String is a special case of expression.

Tom explored this and there were issues with the construction of the expressions.

Feel free to discuss with Tom on his adventures on string vs expressions.


HTML/Markdown not supported in Survey plugin

Documentation suggests that markdown/HTML is supported in SurveyJS:

https://surveyjs.io/form-library/examples/edit-survey-questions-markdown/reactjs
https://github.com/surveyjs/survey-library/issues/3673

However, the formatting of radiogroup options does not seem to work

                            {
                                title: "Example",
                                type: "radiogroup",
                                choices: ["My **markown** text", "Some <b>HTML</b> text"],
                                colCount: 0,
                            },

Is there an option I missed to enable inline formatting? Thanks!

Based on this example here it seems like they call an event: https://surveyjs.io/form-library/examples/edit-survey-questions-markdown/reactjs#content-code But I'm not sure if that's doable from JsPsych.
See also using markdown for values in Likert scale

Thanks @DominiqueMakowski - you're right, we should support markdown. It is supported in SurveyJS but doesn't work 'out of the box' - instead we need to install a markdown parser package (Showdown or Marked) and use the Survey's onTextMarkdown event to convert any text elements that allow markdown to HTML. I will definitely add this to the next plugin version.
In the meantime you might be able to get it to work by adding the onTextMarkdown event handler to the survey via the survey_function parameter, and loading Marked or Showdown using their browser/client-side CDN options. My other tip is to make sure you're using the knockout version of the markdown example code. HOWEVER: I haven't actually tried this approach and don't know how difficult it would be to get working, or if it would work at all!


fix: prevent menu event propagation to treeNode

简介

修复事件冒泡导致 contextMenu 会触发父级的 onClick 事件

Related Issues
Closed #253

+1


Drag and Drop is happening from depending branch

Describe the Bug
Expected: The compose action should not be drag/drop from the dependent for each card.
Actual: Drag and Drop is happening from dependent branch.
Steps to Reproduce the Bug or Issue
N/A
Workflow JSON
No response
Expected behavior
Drag and drop should move the action properly
Screenshots or Videos

Platform And Browser

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response
AB#17036984

I wasn't able to reproduce this issue, but if no repro can be produced, should verify with vendor team as they saw a discrepancy between ga designer and new designer

The issue is that in GA we don't allow dragging nodes to positions that make their tokens invalid, like in the screenshot when we move the compose before where the input token is being output.  We either need to just block the move like we do in GA, or remove the invalid tokens after you move.  Just blocking the move without any visible reason as to why might make people confused.


Update upickle to 4.0.2

Supports Scala LTS

Submitting as per https://github.com/lumidion/sonatype-central-client/pull/8#issuecomment-2332080055

@nafg, I'm evaluating this; let me get back to you. I don't like that we're forcing users to upgrade to a certain Scala 3 version, but it may not be that big of a problem in the end.

You mean 3.3.3 to 3.3.4? Wouldn't whatever needs to happen happen automatically?
Do you think users would have to update scalaVersion manually?

I asked on Discord and was told they would not have to upgrade.
https://discord.com/channels/632150470000902164/632150470000902166/1298384722161832009

Hi what's the update?


M-CR510 support

I get these in my home assistant logs
[DenonAVR] Input function NET is not known
[DenonAVR] Input function AUXD is not known
[DenonAVR] No mapping for input source 'AUX:Digital In'
I wish I could tell you which commit of denonavr but I only know I am on Home Assistant 2021.3.4
XML dumps:
M-CR510 xml.zip

My Marantz M-CR611 is also having similar issues.
I'm currently on Home Assistant v2021.4.6
I'm also not sure from which version these messages started appearing.

2021-05-01 14:03:58 ERROR (SyncWorker_0) [DenonAVR] Input function NET is not known
2021-05-01 14:04:08 ERROR (SyncWorker_4) [DenonAVR] Input function NET is not known
2021-05-01 14:04:18 ERROR (SyncWorker_0) [DenonAVR] Input function NET is not known
2021-05-01 14:04:38 WARNING (MainThread) [homeassistant.components.media_player] Updating denonavr media_player took longer than the scheduled update interval 0:00:10
2021-05-01 14:04:38 WARNING (MainThread) [homeassistant.helpers.entity] Update of media_player.marantz_m_cr611 is taking over 10 seconds
2021-05-01 14:04:38 ERROR (MainThread) [homeassistant.helpers.entity] Update for media_player.marantz_m_cr611 fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 292, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 498, in async_device_update
await task
File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/src/homeassistant/homeassistant/components/denonavr/media_player.py", line 145, in update
self._receiver.update()
File "/usr/local/lib/python3.8/site-packages/denonavr/denonavr.py", line 571, in update
if self._update_avr() is not True:
File "/usr/local/lib/python3.8/site-packages/denonavr/denonavr.py", line 642, in _update_avr
if self._update_input_func_list():
File "/usr/local/lib/python3.8/site-packages/denonavr/denonavr.py", line 773, in _update_input_func_list
receiver_sources = self._get_receiver_sources()
File "/usr/local/lib/python3.8/site-packages/denonavr/denonavr.py", line 1217, in _get_receiver_sources
favorites = root.find(".//FavoriteStation")
UnboundLocalError: local variable 'root' referenced before assignment
2021-05-01 14:04:48 ERROR (SyncWorker_0) [DenonAVR] Input function NET is not known
2021-05-01 14:04:58 ERROR (SyncWorker_4) [DenonAVR] Input function NET is not known
2021-05-01 14:05:08 ERROR (SyncWorker_1) [DenonAVR] Input function NET is not known
2021-05-01 14:05:18 ERROR (SyncWorker_4) [DenonAVR] Input function NET is not known
2021-05-01 14:05:28 ERROR (SyncWorker_1) [DenonAVR] Input function NET is not known

Please wait for Home Assistant 2021.5 and check again. This library is completely redesigned. The new version will be included from that version on. Please reopen this issue, if you are facing similar issues then.


Move podNIC -> podnic.go, bindings -> bindings.go

What this PR does / why we need it:
Currectly podinterface.go has a mix stuff related to podnic struct stuff
related to each binding. This change split the two into podnic.go and
binding.go and also moves the pure unit test a podnic_test.go or binding_test.go
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Possible flow ups:

Test getPhase2Binding without runing bindings.
Move DhcpConfig test to pkg/network/dhcp package.

Release note:

NONE

/uncc @ashleyschuett @jean-edouard

/cc @maiqueb

/hold WIP

You'll need to run make generate (possibly deps-update? ...) since you've renamed a file; bazel must be updated.
I'm frown upon renaming the pod interface file, but since all it is left with are indeed the bindings, it might be a good fit.
Please rename the file in a different commit though.

Fixed rebase

/hold cancel

@maiqueb ready for re-review.

/retest

@maiqueb thanks
/cc @AlonaKaplan your turn


repeatMode(ValueAnimator.INFINITE) doesn't work

Hello,
Here is my code:
ViewAnimator viewAnimator = ViewAnimator.animate(textView)
                .tada()
                .duration(4000)
                .repeatMode(ValueAnimator.INFINITE)
                .start();

The animation works but the 'ValueAnimator.INFINITE' value (which is an enum that is equal to "-1") doesn't work to make the animation loop indefinitely.
Could someone help me to find the issue?

use .repeatCount(ValueAnimator.INFINITE)
and .repeatMode(ValueAnimator.RESTART) (or REVERSE)
Le jeu. 24 mai 2018 à 10:30, NayMak<EMAIL_ADDRESS>a écrit :

Hello,
Here is my code:
ViewAnimator viewAnimator = ViewAnimator.animate(textView)
.tada()
.duration(4000)
.repeatMode(ValueAnimator.INFINITE)
.start();
The animation works but the 'ValueAnimator.INFINITE' value (which is an
enum that is equal to "-1") doesn't work to make the animation loop
indefinitely.
Could someone help me to find the issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/florent37/ViewAnimator/issues/42, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfQXEFSxhlHJgFXFbi_698pcyWVVistks5t1m-3gaJpZM4ULygZ
.

How to add delay between loops?

@TurKurT656 I think you need to open a new issue if you want to ask news questions 😊
This topic is closed anyway


⚠ ESPN has degraded performance

In 8462e59, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1704 ms

Resolved: ESPN performance has improved in 437a664.


Pass the allowed IPs as the lock command option

We are placing IPs that are allowed in the config file. This works for the pre determined IPs but what if we need to allow certain IP which is not mentioned in the config.yml ? Is there any possible way to pass the IPs to the lock command ?

Hi, it only works with IP listed in the config.yml.


Unable to see suggestions if I us backspace till .

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
when I use backspace, for example, if I type tf.xyz' and I use backspaces till .symbol, I no longer get suggestion unless I go back one more backspace and retype.and rest i.e..xyxthen only I will get suggestions. This gets very annoying when a spelling mistake happens and I have to go back all the way till.` symbol.
What steps reproduce the problem?

type anything like eg right_side = tf.strib
now hit a back space since we wanted to type strided_slice and not strided_slice
Correct it  and go on, right_side = tf.strid and you will notice no longer suggestion pops up.

What is the expected output? What do you see instead?
It should pop up even if I do backspace on wrong names.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 3.3.4
Python version: 3.7
Qt version: 4.4.4
PyQt version:
Operating System name/version: win 10 build 17763.rs5_release.180914-1434 in test mode

Dependencies

PASTE DEPENDENCIES HERE

Thanks for reporting. We're working to solve this in Spyder 4.


Template links broken

https://get.foundation/templates.html

The image's links are broken here -- "See Demo" is not, just the preview-like image.  Looks like the correct links have "-xy-grid/" added in the URL, where the images do not have that bit in the link leading to a 404.

I posted this in the Slack.  I don't see a way to properly submit a PR at first glance here, but I may be missing something.

Wrong repo. The correct one is https://github.com/foundation/foundation-marketing/tree/master/src/pages
@joeworkman I need rights for this as I can not move the issues to it.

You can create a PR at https://github.com/foundation/foundation-marketing/pulls
Thanks for reporting this.

Hi @abcarroll,
thanks for reporting this.
The correct repository is at https://github.com/foundation/foundation-marketing/tree/master/src/pages.
You can create a PR at https://github.com/foundation/foundation-marketing/pulls.
@joeworkman I need rights for this as I can not move the issues to it.


Readonly account doesn't work

With a fresh db install, I have the following :
openldap        | Add read only user...
openldap        | Processing file /container/service/slapd/assets/config/bootstrap/ldif/readonly-user/readonly-user.ldif
openldap        | adding new entry "cn=readonly,dc=domain,dc=com"
openldap        | Processing file /container/service/slapd/assets/config/bootstrap/ldif/readonly-user/readonly-user-acl.ldif
openldap        | ldap_modify: No such object (32)
openldap        | *** /container/run/startup/slapd failed with status 32
openldap        |
openldap        | *** Killing all processes...

The creation fails with the files bootstrap/ldif/readonly-user/readonly-user-acl.ldif,
 dn: olcDatabase={1}{{ LDAP_BACKEND }},cn=config

With LDAP_BACKEND=hdb
But, the file 02-security.ldif which also has the same syntax work well.
I have tried to swap the content of files in every order, I wasn't able to make it work.

It appear that the problem comes from the fact that {{ LDAP_BASE_DN } is not available for create when the scripts try to execute it.
By moving it's content in a file bootstrap/ldif/99-perms.ldif, the user successfully create.
be careful to change the interpolated string as the startup script does target not interpolate strings in your file
This could be fixed by being able to delay it's importation.

Hello,
this will be fixed in the next release thanks to @danielpanteleit
see pull request #62
Thanks :)


4 add blocklist project functionality

Completed notification system for letting user know if a site on blocklistproject was attempted to be accessed.
Future goals/next steps:
Add the other lists from blocklist project and allow user to choose which lists to block. Update notifs to show in extension popup instead of chrome notifications.

Code was copied and merged in earlier


Microsoft 365/Office 365 Enterprise requirements?

On it says "A Microsoft 365 for enterprise or Office 365 for enterprise product license is required." but this is not mentioned on the doc.
Is 365 Enterprise really required? Or is 365 Business enough? It should be clarified on the docs as it causes confusion.
On https://www.microsoft.com/en-us/microsoft-viva/pricing it says on the prerequisites section "Microsoft 365 Business Basic, Business Standard, and Business Premium."

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

ID: 4031ac91-8224-ebea-a7cb-c9e5a61cd717
Version Independent ID: 4031ac91-8224-ebea-a7cb-c9e5a61cd717
Content: Install Viva Sales
Content Source: Viva/sales/install-viva-sales.md
Service: viva
Sub-service: viva-sales
GitHub Login: @sbmjais
Microsoft Alias: shjais

Hi @JMCA2, thank you for your feedback.
My apologies but I'm not sure if I understand. The following link:
https://www.microsoft.com/microsoft-viva/pricing
Says:
Viva requires a license for:
Microsoft 365 F1, F3, E3, A3, E5, and A5.
Office 365 F3, E1, A1, E3, A3, E5, and A5.
Microsoft 365 Business Basic, Business Standard, and Business Premium.
or SharePoint K, Plan 1, or Plan 2.
Viva Sales is not included, it's available as an add-on. The following link:
https://www.microsoft.com/microsoft-viva/sales
Contains:
"A Microsoft 365 for enterprise or Office 365 for enterprise product license is required to use the Viva Sales app in Outlook, Excel, and Microsoft Teams."
I'll include this information on:
https://learn.microsoft.com/viva/sales/install-viva-sales

@dariomws I see the requirements are different for Viva Sales vs Viva Suites.
Does "A Microsoft 365 for enterprise or Office 365 for enterprise product license" as you added on the PR means that, for instance "Microsoft 365 Busines Premium" + Dynamics 365 Sales Professional would not be enough to be qualified for buying Viva Sales?
I ask because the line "A Microsoft 365 for enterprise or Office 365 for enterprise product license" doesn't include a list of specific SKUs
Also, on the Spanish/Spain page page at https://www.microsoft.com/es-es/microsoft-viva/sales it says on the FAQ "Es obligatorio contar con una licencia de producto de Microsoft 365 para empresas u Office 365 para empresas".
And Microsoft 365 Business Basic/Standard/Premium are called "para empresas" in Spanish/Spain https://www.microsoft.com/es-es/microsoft-365/business/compare-all-microsoft-365-business-products
(the E3/E5 SKUs are called "para grandes empresas" https://www.microsoft.com/es-es/microsoft-365/business/compare-all-microsoft-365-business-products)

@JMCA2, I closed the PR because I found some information in the introduction article:
https://learn.microsoft.com/viva/sales/introduction
As we discussed, Viva Sales is not included with Viva Suite. You can buy it as an add-on or with Dynamics:
https://dynamics.microsoft.com/pricing/
In addition, to use the Viva Sales app in Outlook, Excel, and Microsoft Teams, I understand you need Microsoft 365 Apps licensed with 365 for Enterprise (E3 and E5).
But let me double-check.
@sbmjais, could you help here? Thanks!

@dariomws  @sbmjais Just add (additionally from the confusion I previously described) that on the Dynamics 365 Licensing Guide https://go.microsoft.com/fwlink/?LinkId=866544&clcid=0x409 it says:
"Sales Enterprise licenses also include the Microsoft Viva Sales application."
With no mention of E3/E5 being required

@dariomws  @sbmjais any updates on this issue? Specially important now that AI features are launching. Thanks

@sbmjais, could you help here? Thanks!


irresponsive

Issue Type: Bug
Extension Name: tabnine-vscode
Extension Version: 3.4.17
OS Version: Windows_NT x64 10.0.14393
VS Code version: 1.58.2

:warning: We have written the needed data into your clipboard. Please paste! :warning:
gz#3988

Hi can you provide more details?
Was the installation successful? Is the plugin irresponsive or your VSCode is irresponsive?
Can you reach Tabnine Hub by typing "tabnine::config"?


linked brushing?

Is linked brushing planned or already there?

Linked brushing could easily be implemented as for hover(), i.e. selecting a subset of the aggregate cells, but it's not well defined what to do after that.  You should already be able to implement whatever you like, as long as you are clear what you want.
E.g. if you have two different datashader plots in the same app that are showing different data plotted on the same axes and with the same binning, you could make linked brushing select the corresponding aggregate bins in each one.  Or if you had a non-datashader Bokeh plot alongside a datashader plot with the same axes, then you could select a set of aggregate bins in the datashader plot, then find all the matching datapoints in the Bokeh plot.
Going the other way is very undefined -- if you select a set of specific Bokeh plot points, it's unclear how that should be reflected in the bin-based datashader plot; perhaps it would then select all the aggregate bins that the selected points would fall into were they to be put into the aggregation?  That could be computed, but would be very tricky to do.
Other cases are also problematic; e.g. it's not clear what a selection of a certain set of aggregate bins on one datashader plot would mean for a different gridding in another datashader plot, and in general there is nothing to make different datashader plots have the same gridding at present.
Given all these complications, it's unlikely that we will support linked brushing for datashader plots any time soon, but it's definitely possible in principle, and wouldn't be that hard to do for any one particular case that applied to your application.

Discovered this closed issue through a Google search, and wanted to make a comment.  I think a way in which linked brushing could be well defined is if the brushing were used to update the category of individual data points within a plot, so that subsequent datashader plots could reflect the update.  For example, brushing an x vs. y datashader plot would assign a category 'selected' to the individual points in the dataset bounded by the brushing.  The selection could trigger recomputation though the datashader pipeline of the x vs. y plot as well as other plots, e.g. w vs z, with updates to reflect the new categories.

That's an interesting suggestion.  Just to be clear, there are no individual data points within a datashader plot. Datashader aggregates the data incrementally, adding each new batch of data points to the existing incremental aggregates in a way that never involves storing all the individual points.  But assuming you mean changing the data points in the original source by doing a query based on the brushed shape of the aggregate, yes, backwards tagging them in some way would be a way to keep track of linked brushing.  We don't currently modify the original data sources, which lets us use a wide variety of data formats easily, so it could get tricky to implement...

Thanks for the response.  I'm sure it's not trivial, and understand that datashader doesn't store individual points.  But as I understand it in this code from the datashader pipeline notebook, doesn't the aggregate get recomputed from the dataframe when an interaction happens? i.e. in the line
agg = cvs.points(df, 'x', 'y', ds.count_cat('cat'))
inside the callback function.  Is it not possible to modify the values of a category in the dataframe based on interactive input?

Sure, that's possible, and this approach comes under the response to the original post -- you as a user can already implement this for any particular plot.  I.e. you can register a callback with bokeh that grabs the shape of the brushed area in a datashader plot, does a query on the original points, marks the category on the original data source, and then tells another plot that's using that category information to update.  At least for a box select, doing this wouldn't need any features of datashader at all -- the box select will choose an area of the axes, and you can do a query on the dataset based on this area regardless of how datashader has aggregated it.  So you can do this with Bokeh and pandas now, already, though it would be useful if we provided a good example to follow. :-)
A lasso select is more complicated, and does depend on datashader, because arguably it should draw out a fixed set of aggregate bins, aligned to bin boundaries.  Still, something approximately correct, at least for small bin sizes, can be done independently of datashader -- the user can select a region in the real plane, which user-level code can use as a query both for the datashader plot and for any other linked plot.  So you can already do all of this yourself, it's just that I would not expect datashader to offer functionality to make it easier to do so if it needs to modify the original data source from within code in datashader/.

Thanks again for the response.  I will give it a go.  A good example would make it easier! :)


Fixing missing libs for debug build

Change e748e713afe6556b988f3fcf2797a34e6a151759 introduced Debug build brake.
Debug build fails with:
/usr/bin/ld: /usr/bin/ld: DWARF error: invalid or unhandled FORM value: 0x23
lib/libMLIRTTIRPipelines.a(TTIRPipelines.cpp.o): in function `mlir::tt::ttir::createLinalgToLLVMPipeline(mlir::OpPassManager&, mlir::tt::ttir::LinalgToLLVMPipelineOptions const&)':
TTIRPipelines.cpp:(.text+0x50): undefined reference to `mlir::createConvertElementwiseToLinalgPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x7d): undefined reference to `mlir::createConvertTensorToLinalgPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x118): undefined reference to `mlir::bufferization::buildBufferDeallocationPipeline(mlir::OpPassManager&, mlir::bufferization::BufferDeallocationPipelineOptions const&)'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x12f): undefined reference to `mlir::createBufferizationToMemRefPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x165): undefined reference to `mlir::createConvertLinalgToLoopsPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x19b): undefined reference to `mlir::memref::createExpandStridedMetadataPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x1d1): undefined reference to `mlir::createConvertSCFToCFPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x207): undefined reference to `mlir::createConvertControlFlowToLLVMPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x23d): undefined reference to `mlir::createArithToLLVMConversionPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x273): undefined reference to `mlir::createConvertFuncToLLVMPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x2a9): undefined reference to `mlir::createFinalizeMemRefToLLVMConversionPass()'
/usr/bin/ld: TTIRPipelines.cpp:(.text+0x2df): undefined reference to `mlir::createReconcileUnrealizedCastsPass()'
clang++-17: error: linker command failed with exit code 1 (use -v to see invocation)

This PR adds missing libs. I've checked build times and executable sizes with added libs. There is couple of seconds added to build time. Size of binary increase is negligible. Regardless there is no other way we have to include these libs.
With this change
Debug build time: 72 sec
Debug executable size:

opt - 961MB
translate - 679MB

Release build time: 81 sec
Release executable size:

opt - 881MB
translate - 634MB

With e748e713afe6556b988f3fcf2797a34e6a151759 reverted
Debug build time: 75sec
Debug executable size:

opt - 953MB
translate - 679MB

Release build time: 73 sec
Release executable size:

opt 873MB
translate 634MB

Since this is build break I will check in right away. If someone has followup comments about PR we can continue on this PR.

@mtopalovicTT Thanks for fixing this + sorry for breaking things!  Out of curious, was this building lsp-server or some other target?

Not sure what the build target was

thank you for the fix!


Add py.typed file

Add py.typed file according to this https://www.python.org/dev/peps/pep-0561/#packaging-type-information

LGTM, Thanks

My microscopic contribution to version 1.0 😉


Security

A PR to contribute to the Zep Configs

:rocket: This description was created by Ellipsis for commit 541de0cf8dd3a450927ed009f67d126f5f987131.

Summary:
This PR optimizes resource usage in the zep-k8-deployment.yaml file and enables configuration through environment variables.
Key points:

Reduced requested resources for zep-postgres, zep-nlp, and zep containers in /zep-k8-deployment.yaml.
Adjusted CPU limits for these containers.
Uncommented environment variables section to enable pulling config from the configmap.

Generated with :heart: by ellipsis.dev

seen

Made changes to K8s Configs


Inkscape module

This adds support for rendering vector files with Inkscape. This module has support for a bunch of file formats, width and height parameters, and automatic detection of linked files.

@scribblemaniac Thank you for the contribution!
Regarding to "width" and "height" parameters I am still thinking about the right behavior.
When we deal with resizing, we have two possible situations:

We have files that must inherit the project resolution.
Those are: Blender and Synfig files. We can define any  can define image size with arbitrary aspect and they will render the data without stretching (distortion).  So, those applications do flexible resizing.

Also, we can have asset files (like textures) - they have arbitrary resolution and shouldn't inherit project settings.
In other words, we have to get image dimensions from rendered file. The Krita and Inkscape files fall into this category. Pencil2D files also go here, since Pencil cannot do any resizing at all. ^__^

This is just thinking aloud, I am still not sure what to do here. Probably I am going to introduce a parameter that indicate which case the filetype belongs to. This parameter can be overriden for individual files using a .conf file.
Not sure how to call this option... Something like "use_own_dimensions". ^__^

The new parameter thing sounds like a good idea. Another way of doing it would be to have default behaviors for each type (i.e. Synfig, and Blender use project width/height, and the rest use the resolution defined in the file). Then you define special keywords that can be used in the individual .conf files on a per parameter basis.
For example, Krita files would automatically use whatever size the image was created at. If it had an individual configuration file like this though:
WIDTH=inherit
HEIGHT=inherit

Then it would render at the size specified by the global project configuration file. The opposite could be done for a Blender file with a keyword like source.
Perhaps I'm overthinking this, but you could take it one step further by having the option to specify a path, relative to the main project directory, to inherit from. You may have for example, a shot that you want to render at a higher frame rate, but it contains animation assets. The assets may have a different resolution, but they all should share the same frame rate. It would be nice if you could have a configuration like this for an asset:
FRAMERATE=inherit 01/01Main.blend
WIDTH=800
HEIGHT=600

and then this asset would automatically match the frame rate that's found in 01/01Main.blend. This way, if you wanted to change the frame rate later on, you would only have to change one parameter to change all of the assets within that shot.
Of course this has some additional complexities with it. Depending on how .conf files are currently handled, this could be very easy or very difficult to implement. Also, you would need to set a max recursion depth, or something similar to handle scenarios such as the one where A inherits B and B inherits A. You would also have to add the configurations of inherited files as dependencies, so that when they change, the things that inherit its settings are updated as well.
Anyway, that's just my thinking aloud about you thinking aloud. ^_^

@scribblemaniac I have implemented some advanced handling for width/height in this commit - https://github.com/morevnaproject/RenderChan/commit/c3e4141664a76425d6d681c795c6c09c37cd966e
It is assumed that module can retrieve the width/height of the file through analyze() function. At the moment only Krita module can do that. ^__^

It should be fairly trivial to get that working for gimp and inkscape. When I get a chance, I'll make another PR for them.


from build.fastllama import Model, ModelKind ModuleNotFoundError: No module named 'build.fastllama'

from build.fastllama import Model, ModelKind

ModuleNotFoundError: No module named 'build.fastllama'

There are two reasons this can be caused:

not compiling with python setup.py -l python
not running your examples from root dir.

If it's the latter, It is because we are using relative paths right now. We will fix it in the near future.

@PotatoSpudowski hello, I have runed setup,op -l python
and I am in root dir. macOS got this issue

If you are still having this issue, I can suggest you use sys.path.append to append the path to the fastllama.py inside the build directory. And change the import according to that.

Did this get resolved? @lucasjinreal

If you are still having this issue, I can suggest you use sys.path.append to append the path to the fastllama.py inside the build directory. And change the import according to that.

The problem seems to be that fastllama.py is located in interfaces/python/, not build/interfaces/python. Not sure that setup.py is setting up correctly.

Follow up, manually copying fastllama.py to build/interfaces/python AND copying pyfastllama.so from build/interfaces/python to build makes the app functional on Macs. Hopefully setup.py can be tweaked so the simple install works.

Yup, in this branch we are trying to make it more simplified.

Absolute paths have been removed in the new commit. Can close this for now.


[SDL2-net] build failure

Host Environment

OS: Windows
Compiler: x64, MSVC

To Reproduce
Steps to reproduce the behavior:
./vcpkg install SDL-net
Failure logs
CMake Warning at build/vcpkg/scripts/buildsystems/vcpkg.cmake:834 (_find_package):
  By not providing "FindSDL2-net.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "SDL2-net",
  but CMake did not find one.

  Could not find a package configuration file provided by "SDL2-net" with any
  of the following names:

    SDL2-netConfig.cmake
    sdl2-net-config.cmake

Additional context
Up to yesterday (or the day before) we were able to install SDL2-net and find_package(SDL2-net) without issue. For some reason that's now failing to find the dependency.

An sdl2-net update was merged yesterday, replacing vcpkg's proprietary cmake config with upstream's official cmake config. vcpkg now shows this usage:
https://github.com/microsoft/vcpkg/blob/253704407ae68efa37bf8f5b59b3e06dd40d3d3f/ports/sdl2-net/usage#L1-L4

@dcvz Run the command git pull to use the latest vcpkg.

@dcvz Does the issue still occur?


object 'tmpnames1' not found

Hello,
Seems a recent commit (2438b68) is resulting in an error if normalize != "quantile" (ramclustR.R@461).
Thanks,
Rick

Thanks Rick,
I believe I have addressed this issue, adding a normalize=="quantile" statement to ramclustR.R@461.
Corey

Hi Corey,
Thanks! This seems to fix it.


Request: Make this work with books that are in PDF format already

Hi, this seems great, but I believe that it only works with books that are in ePub format on Perlego. Is there any chance it could be expanded to include books that are in .pdf format already?

Hi, thank you for the interest.
I submitted the commit and PDF downloads are now supported. I apologise, the whole script is basically uncommented spaghetti code. But I added the first working implementation and it does the job. Maybe I'll refactor it later.

Thank you so much for your fast response, it works perfectly! Great work.
Might be worth mentioning in the readme that Debian/Ubuntu users will need the static version of wkhtmltopdf  rather than the lastest release!


'React/RCTBridgeModule.h' file not found

🐛 Bug Report
Not a breaking news : on iOS, my build failed because of React linking conflicts
Due to bugs "I don't know where" between React, React-Native and Cocoapods, I followed this article and link most of my dependencies via Podfile
It works well in Debug, but trying to build in Release, it fails with this tb :
.../node_modules/react-native/Libraries/Settings/RCTSettingsManager.h:10:9: fatal error: 'React/RCTBridgeModule.h' file not found
#import <React/RCTBridgeModule.h>

Notes:

I had to add manually and link all the "RCT*" React subspecs to make the build work on debug -- it seems they are not automatically linked as the normal pods are
I don't want a solution like "stop use cocoapod and link everything manual" : it should be a solution...
I've unchecked parallel builds
It fails with RN 0.58 and 0.59

Environment
  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (4) x64 Intel(R) Core(TM) i7-5557U CPU @ 3.10GHz
      Memory: 664.52 MB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 8.4.0 - ~/.nvm/versions/node/v8.4.0/bin/node
      Yarn: 1.13.0 - /usr/local/bin/yarn
      npm: 6.9.0 - ~/.nvm/versions/node/v8.4.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: ^16.8.3 => 16.8.4
      react-native: ^0.59.1 => 0.59.1
    npmGlobalPackages:
      react-native-cli: 2.0.1

Podfile:
target 'XXX' do
  platform :ios, '9.0'
  # Uncomment the next line if you're using Swift or would like to use dynamic frameworks
  # use_frameworks!

  # Pods for XXX
  ## React
  pod 'React', :path => '../node_modules/react-native', :subspecs => [
    'Core',
    'CxxBridge',
    'DevSupport',
    # React subspecs should be linked manually, as cocoapod seems to not autolink its
    # How to ? See https://stackoverflow.com/a/46096893
    'RCTText',
    'RCTImage',
    'RCTNetwork',
    'RCTWebSocket',
    'RCTSettings',
    'RCTAnimation',
    'RCTLinkingIOS',
  ]
  pod "yoga", :path => "../node_modules/react-native/ReactCommon/yoga"
  pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
  pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'
  pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'

  ## Other dependencies
  pod 'AppCenterReactNativeShared', '~> 1.12.2'
  pod 'AppCenter/Crashes', '~> 1.13.2'
  pod 'AppCenter/Analytics', '~> 1.13.2'
  pod 'react-native-webview', :path => '../node_modules/react-native-webview'
  pod 'RNScreens', :path => '../node_modules/react-native-screens'
  pod 'RNGestureHandler', :path => '../node_modules/react-native-gesture-handler'

  target 'XXXTests' do
    inherit! :search_paths
    # Pods for testing
  end

end

The exact error log :
Found Xcode workspace XXX.xcworkspace
Building using "xcodebuild -workspace XXX.xcworkspace -configuration Release -scheme XXX -destination id=ZZZ -derivedDataPath build"
User defaults from command line:

    IDEDerivedDataPathOverride = .../YYY/XXX/app/ios/build

note: Using new build system

note: Planning build

note: Using build description from disk

Build system information

warning: The iOS Simulator deployment target is set to 7.0, but the range of supported deployment target versions for this platform is 8.0 to 12.1. (in target 'RNGestureHandler')

Build system information

warning: The iOS Simulator deployment target is set to 7.0, but the range of supported deployment target versions for this platform is 8.0 to 12.1. (in target 'RNScreens')

CompileC .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/i386/RCTSettingsManager.o .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler (in target: RCTSettings)
    cd .../YYY/XXX/app/node_modules/react-native/Libraries/Settings
    export LANG=en_US.US-ASCII
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -x objective-c -arch i386 -fmessage-length=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -std=gnu99 -fobjc-arc -fmodules -fmodules-cache-path=.../YYY/XXX/app/ios/build/ModuleCache.noindex -fmodules-prune-interval=86400 -fmodules-prune-after=345600 -fbuild-session-file=.../YYY/XXX/app/ios/build/ModuleCache.noindex/Session.modulevalidation -fmodules-validate-once-per-build-session -Wnon-modular-include-in-framework-module -Werror=non-modular-include-in-framework-module -Wno-trigraphs -fpascal-strings -Os -fno-common -Wno-missing-field-initializers -Wno-missing-prototypes -Werror=return-type -Wunreachable-code -Wno-implicit-atomic-properties -Werror=deprecated-objc-isa-usage -Wno-objc-interface-ivars -Werror=objc-root-class -Wno-arc-repeated-use-of-weak -Wimplicit-retain-self -Wduplicate-method-match -Wmissing-braces -Wparentheses -Wswitch -Wunused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wempty-body -Wuninitialized -Wconditional-uninitialized -Wno-unknown-pragmas -Wshadow -Wno-four-char-constants -Wno-conversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wno-float-conversion -Wnon-literal-null-conversion -Wobjc-literal-conversion -Wshorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wundeclared-selector -Wdeprecated-implementations -DNS_BLOCK_ASSERTIONS=1 -DOBJC_OLD_DISPATCH_PROTOTYPES=0 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator12.1.sdk -fasm-blocks -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -mios-simulator-version-min=9.0 -g -Wno-sign-conversion -Winfinite-recursion -Wcomma -Wblock-capture-autoreleasing -Wstrict-prototypes -Wno-semicolon-before-method-body -fobjc-abi-version=2 -fobjc-legacy-dispatch -iquote .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-generated-files.hmap -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-own-target-headers.hmap -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-all-target-headers.hmap -iquote .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-project-headers.hmap -I.../YYY/XXX/app/ios/build/Build/Products/Release-iphonesimulator/include -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/DerivedSources/i386 -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/DerivedSources -Werror -Wall -F.../YYY/XXX/app/ios/build/Build/Products/Release-iphonesimulator -MMD -MT dependencies -MF .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/i386/RCTSettingsManager.d --serialize-diagnostics .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/i386/RCTSettingsManager.dia -c .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m -o .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/i386/RCTSettingsManager.o

In file included from

.../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m:8:
.../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.h:10:9: fatal error: 'React/RCTBridgeModule.h' file not found
#import <React/RCTBridgeModule.h>

        ^~~~~~~~~~~~~~~~~~~~~~~~~

1 error generated

.

CompileC .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.o .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler (in target: RCTSettings)
    cd .../YYY/XXX/app/node_modules/react-native/Libraries/Settings
    export LANG=en_US.US-ASCII
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -x objective-c -arch x86_64 -fmessage-length=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -std=gnu99 -fobjc-arc -fmodules -fmodules-cache-path=.../YYY/XXX/app/ios/build/ModuleCache.noindex -fmodules-prune-interval=86400 -fmodules-prune-after=345600 -fbuild-session-file=.../YYY/XXX/app/ios/build/ModuleCache.noindex/Session.modulevalidation -fmodules-validate-once-per-build-session -Wnon-modular-include-in-framework-module -Werror=non-modular-include-in-framework-module -Wno-trigraphs -fpascal-strings -Os -fno-common -Wno-missing-field-initializers -Wno-missing-prototypes -Werror=return-type -Wunreachable-code -Wno-implicit-atomic-properties -Werror=deprecated-objc-isa-usage -Wno-objc-interface-ivars -Werror=objc-root-class -Wno-arc-repeated-use-of-weak -Wimplicit-retain-self -Wduplicate-method-match -Wmissing-braces -Wparentheses -Wswitch -Wunused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wempty-body -Wuninitialized -Wconditional-uninitialized -Wno-unknown-pragmas -Wshadow -Wno-four-char-constants -Wno-conversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wno-float-conversion -Wnon-literal-null-conversion -Wobjc-literal-conversion -Wshorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wundeclared-selector -Wdeprecated-implementations -DNS_BLOCK_ASSERTIONS=1 -DOBJC_OLD_DISPATCH_PROTOTYPES=0 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator12.1.sdk -fasm-blocks -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -mios-simulator-version-min=9.0 -g -Wno-sign-conversion -Winfinite-recursion -Wcomma -Wblock-capture-autoreleasing -Wstrict-prototypes -Wno-semicolon-before-method-body -fobjc-abi-version=2 -fobjc-legacy-dispatch -iquote .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-generated-files.hmap -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-own-target-headers.hmap -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-all-target-headers.hmap -iquote .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/RCTSettings-project-headers.hmap -I.../YYY/XXX/app/ios/build/Build/Products/Release-iphonesimulator/include -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/DerivedSources/x86_64 -I.../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/DerivedSources -Werror -Wall -F.../YYY/XXX/app/ios/build/Build/Products/Release-iphonesimulator -MMD -MT dependencies -MF .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.d --serialize-diagnostics .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.dia -c .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m -o .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.o

In file included from .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m:8:
.../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.h:10:9: fatal error: 'React/RCTBridgeModule.h' file not found
#import <React/RCTBridgeModule.h>
        ^~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

** BUILD FAILED **

The following build commands failed:
	CompileC .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/i386/RCTSettingsManager.o .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC .../YYY/XXX/app/ios/build/Build/Intermediates.noindex/RCTSettings.build/Release-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.o .../YYY/XXX/app/node_modules/react-native/Libraries/Settings/RCTSettingsManager.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(2 failures)

Installing build/Build/Products/Release-iphonesimulator/XXX.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22):
Failed to install the requested application
The bundle identifier of the application could not be determined.
Ensure that the application's Info.plist contains a value for CFBundleIdentifier.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Release-iphonesimulator/XXX.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Error: Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Release-iphonesimulator/XXX.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

    at checkExecSyncError (child_process.js:591:13)
    at Object.execFileSync (child_process.js:611:13)
    at Promise.then (.../YYY/XXX/app/node_modules/react-native/local-cli/runIOS/runIOS.js:208:5)
    at <anonymous>

Hello there 👋 this issue doesn't have a repro (which means, a react-native init-ed project with the minimal changes that leads to creating the same issue you are reporting). Unfortunately, I have no way of helping you in a meaningful way – there is no easy way for me to recreate the situation and check that the issue reported is still there when changing the code; also, it looks like it's related to something in your XCode config.
Because of this, we are going to close this issue - but if a repro is shared, we are happy to reopen it 🤗

@Nabellaleen did you try this solution: https://github.com/facebook/react-native/issues/24363#issuecomment-488547280 ?
I had just updated my mac to Mojave and I started getting this. Give it a try. Make sure you close your xcode first before running the command.


Add TypeScript definitions.

Following on #33, here's the PR.
There're a couple of things I'm not 100% sure, though.
I don't think that the types Square, Piece, Position and CustomPieces have to be exported, so I didn't export them. Also the roughSquare receives a squareElement which I believe the appropriate type is SVGElement.
Thank you for your time on this!

Codecov Report

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

@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   62.86%   62.86%
=======================================
  Files          10       10
  Lines         272      272
  Branches       62       62
=======================================
  Hits          171      171
  Misses         80       80
  Partials       21       21

I don't think that the types Square, Piece, Position and CustomPieces have to be exported, so I didn't export them.

That sounds fine if somebody needs them exported I can look into it.
Could you make a new pull request running npm run commit and then just follow the prompts and add it as a 'doc' change?  This helps to integrate the new commits into versioning.
I'm afraid if I amend the commit myself then it won't recognize you as a contributor.

on second thought, this won't affect versioning since it is not a feature change, I'll merge this in.  Thanks!

Thank you very much!

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ Bofrost.at has degraded performance

In bd6ce49, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1446 ms

Resolved: Bofrost.at performance has improved in f5bb79f after 29 minutes.


[C++] Float16.{ToBytes,FromBytes} fail on big-endian machines

Describe the bug, including details regarding any error messages, version, and platform.
The above named tests fail in the following way:
[ RUN      ] Float16Test.ToBytes
/builddir/build/BUILD/libarrow-16.1.0-build/apache-arrow-16.1.0/cpp/src/arrow/util/float16_test.cc:332: Failure
Expected equality of these values:
  load()
    Which is: 7376
  0xd01c
    Which is: 53276

[  FAILED  ] Float16Test.ToBytes (0 ms)
[ RUN      ] Float16Test.FromBytes
/builddir/build/BUILD/libarrow-16.1.0-build/apache-arrow-16.1.0/cpp/src/arrow/util/float16_test.cc:361: Failure
Expected equality of these values:
  Float16::FromLittleEndian(data)
    Which is: 0.00469971
  Float16(0x1cd0)
    Which is: 7376

[  FAILED  ] Float16Test.FromBytes (0 ms)

The tests themselves look suspicious to me, as they start with a "native" check, but use a fixed byte order.
Component(s)
C++

Issue resolved by pull request 44768
https://github.com/apache/arrow/pull/44768


update minimum numpy to 1.10.1

I'd like to upgrade the minimum supported numpy version to 1.10.1. This numpy version was released in Oct 2015.
Numpy 1.10 added two features that are useful to us:

nan support in assert_allclose
scalar support in np.digitize

We've hacked around 1. in the test suite for awhile now, but it would be great to remove the hacked code.
#429 proposes using np.digitize to replace manual binning in the perez function. I could hack around this too, but I'd rather bump the numpy spec.
Any objections to bumping the numpy spec?

closed by #429.


build(client): Upgrade syncpack and simplify config

Unfortunately, version 12 of syncpack has a dependency with a Node20 requirement, so this won't pass CI right now.

Closing since I have other stuff to do. Someone else can do this if they want in the future.

@tylerbutler I'll take a look at downgrading some of syncpack's dependencies which have higher node engines – they were only really bumped as a bit of housekeeping, there's no concrete need to be on the latest versions. I'll see if I can do that to broaden Node support.

@tylerbutler I'll take a look at downgrading some of syncpack's dependencies which have higher node engines – they were only really bumped as a bit of housekeeping, there's no concrete need to be on the latest versions. I'll see if I can do that to broaden Node support.

@JamieMason Certainly appreciated, but don't go to any great lengths. I noticed the node req dropped to 18.18 in syncpack 13, and our CI is only a little below that. Truly we should be running the latest 18.x - once that upgrade is done we should be unblocked. That said, I would welcome reduced requirements :) and you no doubt have other users that would benefit.

@JamieMason Separately, thanks for the awesome tool and great documentation. You've steadily improved it over the months and the comprehensive docs really help with onboarding. It's always exciting to pick up a new release and see features that greatly simplify things for you.

Hey @tylerbutler, FYI there's an alpha preview of a Rust rewrite available at 14.0.0-alpha.1 now. I've been developing it against the FluidFramework repository so you should hopefully be in a good starting point 🤞

Hey @tylerbutler, FYI there's an alpha preview of a Rust rewrite available at 14.0.0-alpha.1 now. I've been developing it against the FluidFramework repository so you should hopefully be in a good starting point 🤞

@JamieMason I had a chance to try this today, and it seems to hang, at least with the config I have. You should be able to reproduce it on the syncpack-config branch on my fork. I'm happy to file an issue if it's helpful. Not sure what other info to collect.

Thanks @tylerbutler, which OS are you using?

@JamieMason I've tested on Ubuntu 20.04 and mac OS Sequoia ARM. Same behavior on both. I can try Windows 11 if it's helpful. I also tried node 18.20 and node 20.18 when I briefly forgot it was now written in rust.


Playlist: relative Path not working when going up two levels

What version of Music Assistant has the issue?
2.0.3
What version of the Home Assistant Integration have you got installed?
No response
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
relative Path in local Playlists not working

How to reproduce
Use relative Path in Playlists
Music Providers
local disc
Player Providers
any
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.5.3
What type of installation are you running?
Home Assistant Core
On what type of hardware are you running?
Raspberry Pi

Relative paths are supported but not in an higher level than the folder you added to the file provider.

A bit of context is needed:
The container mounts your music on /media.  Naturally, any playlist that tries to traverse above that folder (using relative paths) will look invalid from the perspective of MASS.

All music Files are located here (no Folders for Arist or whatever):
/volume1/media01/musicLibrary/musicLibrary/
All Playlists are located here:
/volume1/media01/musicLibrary/musicLibrary/_PLAYLISTS
thats my mounting:
/volume1/media01/musicLibrary/musicLibrary:/media
Sample1 is working: located in /media/_PLAYLISTS/Happy Favorits 2.m3u
../FILE003372.mp3
../FILE004255.mp3
Sample2 is working (wired because the path is not correct): located in /media/_PLAYLISTS/Happy Favorits 3.m3u
FILE003372.mp3
FILE004255.mp3
Sample3 is NOT working: located in /media/_PLAYLISTS/Allgemein/Happy Favorits.m3u
../../FILE000996.mp3
../../FILE004803.mp3

Based on the above it seems the title of this report is incorrect. For paths that only go up one level it seems to work but when going up two levels it does not. Is that the case?

Sample3 is NOT working: located in /media/_PLAYLISTS/Allgemein/Happy Favorits.m3u
../../FILE000996.mp3
../../FILE004803.mp3

Yes, that's not gonna work because it's trying to go above /media.

@Rudd-O i think it goes to media doesn’t it?

Oh!  I stand corrected.  Thanks.

@sebmuc99 Please confirm if this is still and issue in beta 14

We will close this soon due to no follow up

Sorry needed some time to setup a beta environment. issue still exists with version 2.1.0rc3


Update fixes

Pull Request Test Coverage Report for Build 3048

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 3030:
0.0%

Covered Lines:
687

Relevant Lines:
687

💛  - Coveralls


Desktop version portfolio complete

Add following things.

Use of flexbox and grid.
Use media queries.
Use Html and CSS.

Hello Dear @.****
I have changed some code and attached the video so kindly see to confirm
myself about it.
Best regards
On Wed, Jun 7, 2023 at 3:43 PM Jos Kalenda @.***> wrote:

@.**** requested changes on this pull request.
STATUS: REQUIRED CHANGES ♻
Hi @sulaimanahmadi https://github.com/sulaimanahmadi ,
Good job so far! There are some issues that you still need to work on to
go to the next project but you are almost there!
https://camo.githubusercontent.com/2c34e7e812a3dac1247298f8cfde7dbdee5d167283940174f8ad2feee1139ce0/68747470733a2f2f6d65646961332e67697068792e636f6d2f6d656469612f336f36777265626e4b576d7678345a42696f2f67697068792e676966
Highlights

[ ].. ✅ Green linters
[ ].. ✅ Clean code

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you take them into
account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag @.*** https://github.com/joskalenda)
in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.

As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In desktop.css
https://github.com/sulaimanahmadi/Portfolio/pull/4#discussion_r1221401299
:

@@ -159,12 +159,31 @@ body {
flex-grow: 0;
}

+.content-data > h2 {

Your application is not responsive, @sulaimanahmadi
https://github.com/sulaimanahmadi, I think you should better use the
min-width property and make sure you style your app accordingly as for
now, it is not respecting any of your CSS in big screens

[image: 6]
https://user-images.githubusercontent.com/77843050/244030408-b193363a-9aee-4328-bace-d0136ca33218.gif
—
Reply to this email directly, view it on GitHub
https://github.com/sulaimanahmadi/Portfolio/pull/4#pullrequestreview-1467371965,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4QVLCTINMQPBHFVHVRXCELXKBO5DANCNFSM6AAAAAAY5UMZGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello Dear  @.****
On behalf of my pull request, I requested the previous code reviewer also
to check my attached video therefore I made the desktop version portfolio
on a desktop screen and it is responsive so kindly watch my video.
best regards,
On Thu, Jun 8, 2023 at 9:17 PM Tresor Sawasawa @.***>
wrote:

@.**** requested changes on this pull request.
STATUS: REQUEST CHANGES ♻♻
Hi @sulaimanahmadi https://github.com/sulaimanahmadi
Great work on making the changes requested by the previous reviewer 👏👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
[image: almost-there]
https://user-images.githubusercontent.com/57562869/143511358-9cde7daf-4ad0-4db2-8f15-18b54288ff91.gif
To highlight 🍾

All linters are passing ✔
Good commit history ✔
Your design looks good 💯 ✔
Good PR description ✔

Required changes ♻ ♻

Check the comments under the review.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if
something is not 100% clear. Please ping me @tresorsawasawa
https://github.com/tresorsawasawa when you comment so I can receive the
notification or use slack with the same name.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In style.css
https://github.com/sulaimanahmadi/Portfolio/pull/4#discussion_r1223306174
:

background-color: white;
margin: 0;
padding: 0;

width: 100%;
}

.container {

As mentioned by the previous reviewer and for the project
requirements, kindly make sure your application is responsive and the
design sticks as much as possible to the one from your chosen design. Below
you can see hoew your application behaviors when changing the screen size.
[image: git]
https://user-images.githubusercontent.com/77843050/244030408-b193363a-9aee-4328-bace-d0136ca33218.gif

—
Reply to this email directly, view it on GitHub
https://github.com/sulaimanahmadi/Portfolio/pull/4#pullrequestreview-1470385893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4QVLCTE22XEPCFZN772ZKDXKH62RANCNFSM6AAAAAAY5UMZGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


refactor: migrate to workspace

Start of a bigger refactor to move different parts of the code into their own crates to hopefully decrease compilation times.
This PR:

Turns the repository into a cargo workspace.
Converts all package dependencies to workspace dependencies.
Moves the barrier_cell functionality into its own crate.

I've restarted the windows job. It looks like a flacky test but I'm not a 100% sure. So hopefully that helps!


fix: markdown list indentation is not working on selected range (close #219)

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] It does not introduce a breaking change or has a description of the breaking change

Description

catch up to the latest continuelist markdown plugin.
it also fixes increment numbers when item is added to the middle
change codemirror command name newlineAndIndentContinue to newlineAndIndentContinueMarkdownList
change codemirror command name subListIndentTab to indentOrderedList

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

[6/20] 리뷰 완료하였습니다. 코드미러 컨벤션이 그닥 아름답지는 않네요 ;ㅅ; 고생하셨어요~

한줄짜리 if문과 , one var 로 한줄로 변수가 선언되는 부분이 익숙치 않아서 코드 리딩이 헷갈리네요

리뷰 완료했습니다. 수고하셨습니다.


Support pure meta model lm_head tp

Add lm_head tp support when checkpoint not provided to deepspeed.init_inference().

Hi @Yejing-Lai can you describe the high level scenarios that this PR enables?  Thanks!


Fix issue with certain modded items

I had reported an issue a while ago about a modded character that would make Vault crash because it used a skill that started with "Temp"; a keyword in programmation, I believe.
I've found something similar : loading a character that has (in its inventory or storage area) a modded item that starts with "For"; another keyword in programmation (which makes me believe that's the issue.)
The resulting error message goes like this.

When ignored, the container will be unusable or very buggy. It displays wide boxes with a red X inside that cover the affected area.

The previous issue was related to the skill name starting with the same text as one of the item tags.
This looks a bit different in that it's complaining about parenthesis in the expression parser during the calculation of the dynamic requirements.
It would be helpful for you to specify which particular mod you are using and to upload the character file which is causing the issue.

The mod used is Shadow Champions on its latest version. https://steamcommunity.com/sharedfiles/filedetails/?id=1949340853
The character : https://mega.nz/file/0Is1HThZ#lSnH85w5InuaM42PczvYiogxLXz8D_aTJEPLjOFs0Cc
The item in cause is the leg armor called "Forgotten Boots". It's in the first additional bag of its inventory.

This mod have "warning and notices" very well done.
They know already that it may have problems with TQVault.

The math expression is coming from de mod database so this is not a TQVault issue but a mod issue.

Oh, I would've never figured it out! Thank you for your help :)


[Bug]:  Fail to access model(llama3).ERROR: [Errno 111] Connection refused

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
c3b21a
Other environment information
No response
Actual behavior
集成ollama出现的错误
Expected behavior
No response
Steps to reproduce
集成ollama出现的错误

Additional information
No response

Make sure the IP address you input the blank is accessible from the ragflow docker container.

Fail to connect to es: Connection error caused by: ConnectionError(Connection error caused by: NameResolutionError(<urllib3.connection.HTTPConnection object at 0x717580c45ad0>: Failed to resolve 'es01' ([Errno -3] Temporary failure in name resolution))).新的错误

Use http://host.docker.internal:11434/

I was presenting the same problem in this thread and with your comment I was able to solve the problem, thank you.

提示 : 102
Fail to access model(llama3:instruct).ERROR: [Errno 111] Connection refused.还是同样的错误

提示：102 无法访问 model（llama3：instruct）。ERROR： [Errno 111] Connection refused.还是同样的错误

Make sure that your ollama service is bound to an IP and that the ip:port can be accessed by ragflow-server

提示 : 102 Fail to access model(llama3:instruct).ERROR: [Errno 111] Connection refused.还是同样的错误

解决了吗

http://host.docker.internal:11434/
still error
Fail to access model(llama3).ERROR: [Errno -2] Name or service not known
how to solve it

需要修改ollama配置
由于默认ollama绑定在<IP_ADDRESS>的11434端口，其他IP都拒绝
使用docker只能访问到自己不能访问到宿主机上，所以需要将ollama服务暴露给网络
方法修改如下文件：

vim /etc/systemd/system/ollama.service

在[Service]下继续添加如下内容，使ollama绑定到<IP_ADDRESS>的11434端口

Environment="OLLAMA_HOST=<IP_ADDRESS>"

填入的基础URL为

http://host.docker.internal:11434/

设置了，Environment="OLLAMA_HOST=<IP_ADDRESS>" 也没有用

Use http://host.docker.internal:11434/

That also help me solve my problem, thank you!


Refactored data stream class, fixed CORS including PUT and DELETE met…

…hods,

Pull Request Test Coverage Report for Build 455

21 of 22 (95.45%)  changed or added relevant lines in 12 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 97.857%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

source/Cosmos-Metadata-Tests/MetadataSystemUserStoryTests.class.st
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

source/Cosmos-Triggers-Tests/DataStreamHasNotBeenUpdatedTriggeringConditionTest.class.st
4
92.0%

source/Cosmos-Triggers-Tests/DataStreamCurrentValueTriggeringConditionTest.class.st
4
82.05%

Totals

Change from base Build 453:
-0.2%

Covered Lines:
4887

Relevant Lines:
4994

💛  - Coveralls


fix(server): cors on browsers

This pull request proposes a solution to the CORS policy error that has been blocking cross-origin requests from http://localhost:4200 to https://relay.pkarr.org. While this is just an hypothesis, I believe the issue might stem from the server not correctly handling CORS preflight OPTIONS requests and missing Access-Control-Allow-Origin headers in the response. Consequently, the browser is preventing the requests due to an incomplete CORS configuration.
To address this potential root cause (we'll see if that works), this PR adds Method::OPTIONS to the list of allowed methods using .allow_methods([Method::GET, Method::PUT, Method::OPTIONS]) and changed .allow_headers(Any) to allow any headers. While I'm not entirely confident this will fully resolve the issue, it will help rule the server out as cause.

#83 tries to be as permissive as we can, let's see if it fixes this issue.


Add: user-info in iframe Issue #98

Issue #98
Hi @igoradamenko Pls check my PR. I'm only moved user-info component to iframe with minimal UI changes.

Had to revert this PR due to styling issues:

Also I think we can avoid manual creating of DOM node for user iframe.
Just create it dynamically in embed.js


Simple API equivalent to JmsTemplate

I'm looking for a simple way to send a message to a Topic and subscribe to that Topic.
All the examples I've read are inherently complex. Is there no simple API similar to JmsTemplate?
My goal is to replace ActiveMQ w/ Kafka in a Spring Boot project, but keep it as simple as possible. Right now I'm creating ZookeeperConfiguration and ConnectionFactory beans from application.yml, but don't really know where to go from here.
Ideally, I'd like to stay away from any XML based configuration. Any ideas or pointers would be appreciated.

Yes, there's a KafkaTemplate.

Can a FetchRequest be used for both a send and receive?

It currently only supports receive operations; the intent is to eventually flush it out to support other operations.
You can easily wire in a Kafka outbound channel adapter into your boot app - see the sample here (which is a boot app).

That example is much better. Thank you. I only found the example from the readme in this project and on the spring.io blog.

Thanks; I added a link to that sample to the README,


[SPARK-37237][K8S][BUILD] Upgrade kubernetes-client to 5.9.0

What changes were proposed in this pull request?
This PR aims to upgrade kubenetes-client from 5.8.0 to 5.9.0.
Why are the changes needed?
This will bring the following bug fixes and improvement.

https://github.com/fabric8io/kubernetes-client/releases/tag/v5.9.0

Does this PR introduce any user-facing change?
No. (The supported K8s versions are identical with 5.8.0).
To make Scala 2.13 compilation pass, the new Action type, Bookmark, is handled in the test case.

https://github.com/fabric8io/kubernetes-client/pull/3488

How was this patch tested?
Pass the CIs.

Test build #144976 has started for PR 34511 at commit 97e06c0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49447/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49447/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49447/

Test build #144983 has started for PR 34511 at commit f09c2e3.

Thank you, @sarutak !

Test build #144976 has finished for PR 34511 at commit 97e06c0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49454/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144976/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49454/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49454/

Although the UT passed, K8s integration seems to hit Jackson incompatibility issue. I'll take a look
KubernetesSuite:
*** RUN ABORTED ***
  java.lang.NoSuchMethodError: com.fasterxml.jackson.dataformat.yaml.YAMLFactory._createContentReference(Ljava/lang/Object;)Lcom/fasterxml/jackson/core/io/ContentReference;
  at com.fasterxml.jackson.dataformat.yaml.YAMLFactory.createParser(YAMLFactory.java:374)
  at com.fasterxml.jackson.dataformat.yaml.YAMLFactory.createParser(YAMLFactory.java:348)
  at com.fasterxml.jackson.dataformat.yaml.YAMLFactory.createParser(YAMLFactory.java:15)
  at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3548)
  at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3516)
  at io.fabric8.kubernetes.client.internal.KubeConfigUtils.parseConfigFromString(KubeConfigUtils.java:47)
  at io.fabric8.kubernetes.client.Config.loadFromKubeconfig(Config.java:591)
  at io.fabric8.kubernetes.client.Config.tryKubeConfig(Config.java:557)
  at io.fabric8.kubernetes.client.Config.autoConfigure(Config.java:279)
  at io.fabric8.kubernetes.client.Config.<init>(Config.java:245)

This seems to be the root cause at 2.13.0

https://github.com/FasterXML/jackson-dataformats-text/commit/e5dc40f55321161c94b4d1088a030cf9de936497

Yeah, I also confirmed it depends on Jackson 2.13.0.
https://mvnrepository.com/artifact/io.fabric8/kubernetes-client/5.9.0

Let me update it and run together to see the Jackson upgrade feasibility.

Test build #144990 has started for PR 34511 at commit 634fe17.

Test build #144991 has started for PR 34511 at commit 213b752.

I spun off the Jackson part to SPARK-37241 .

https://github.com/apache/spark/pull/34516

Test build #144983 has finished for PR 34511 at commit f09c2e3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49462/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144983/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49463/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49462/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49463/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49463/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49462/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144990/

Test build #144991 has finished for PR 34511 at commit 213b752.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #145006 has started for PR 34511 at commit 8e6a637.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49479/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49479/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49479/

All test passed except 2 K8s integration test cases.
KubernetesSuite:
- Run SparkPi with no resources
- Run SparkPi with no resources & statefulset allocation
- Run SparkPi with a very long application name.
- Use SparkLauncher.NO_RESOURCE
- Run SparkPi with a master URL without a scheme.
- Run SparkPi with an argument.
- Run SparkPi with custom labels, annotations, and environment variables.
- All pods have the same service account by default
- Run extraJVMOptions check on driver
- Run SparkRemoteFileTest using a remote data file
- Verify logging configuration is picked from the provided SPARK_CONF_DIR/log4j.properties
- Run SparkPi with env and mount secrets.
- Run PySpark on simple pi.py example
- Run PySpark to test a pyfiles example
- Run PySpark with memory customization
- Run in client mode.
- Start pod creation from template
- PVs with local hostpath storage on statefulsets
- PVs with local hostpath and storageClass on statefulsets
- PVs with local storage
- Launcher client dependencies
- SPARK-33615: Launcher client archives
- SPARK-33748: Launcher python client respecting PYSPARK_PYTHON
- SPARK-33748: Launcher python client respecting spark.pyspark.python and spark.pyspark.driver.python
- Launcher python client dependencies using a zip file
- Test basic decommissioning
- Test basic decommissioning with shuffle cleanup
- Test decommissioning with dynamic allocation & shuffle cleanups *** FAILED ***
- Test decommissioning timeouts
- Run SparkR on simple dataframe.R example *** FAILED ***
Run completed in 33 minutes, 17 seconds.
Total number of tests run: 30
Suites: completed 2, aborted 0
Tests: succeeded 28, failed 2, canceled 0, ignored 0, pending 0
*** 2 TESTS FAILED ***

This is not a regression because the master branch has it already (https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49466/)

The last commit is the same with the approved commit because Jackson upgrade is handled via another PR. I'll merge this, @sarutak .

Test build #145006 has finished for PR 34511 at commit 8e6a637.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145006/


Feature/profile/CLK32-900-Change Password Views

Prerequisite

[x] Self-review :monocle_face:
[x] No Crashes ☠

Root cause

UI Screen Changepassword

Solution

// Explain how you fix it

Impact
-UI Screen Changepassword
Screenshots (Optional)

Light

Dark

          1 Warning

:warning:
Big PR

[ ] I understand that "Home.GroupChat" is now existed in Localized service

[ ] I understand that "Home.Message" is now existed in Localized service

[ ] I understand that "Home.Title" is now existed in Localized service

[ ] I understand that "Login.SignInWith" is now existed in Localized service

[ ] I understand that "ChangePassword.CurrentPassword" is now existed in Localized service

[ ] I understand that "ChangePassword.NewPassword" is now existed in Localized service

[ ] I understand that "ChangePassword.Save" is now existed in Localized service

[ ] I understand that "ChangePassword.Title" is now existed in Localized service

[ ] I understand that "ChangePassword.TitleButton" is now existed in Localized service

[ ] I understand that "General.ConfirmPassword" is now existed in Localized service

[ ] I understand that "General.Busy" is now existed in Localized service

[ ] I understand that "General.Online" is now existed in Localized service

[ ] I understand that "General.Online".localized) : Text("General.Busy" is now existed in Localized service

[ ] I understand that "Home.Profile" is now existed in Localized service

[ ] I understand that "Home.Signout" is now existed in Localized service

[ ] I understand that "Message.Input.PlaceHolder" is now existed in Localized service

[ ] I understand that "Message.Banner.Content" is now existed in Localized service

[ ] I understand that "Message.Mute" is now existed in Localized service

[ ] I understand that "Message.Reply" is now existed in Localized service

  Generated by :no_entry_sign: Danger


[Technical Question] Create IoT Device in IoT Hub using ARM template

Hi,
(I'm not sure this is the right place for this question. Please redirect me if it is not.)
I'm currently trying to use an ARM template to create an IoT Device in IoT Hub. I played around with the Device Provisioning Service Python SDK, but I wasn't able to actually create the device. I was only able to enroll an existing device. This doc on creating an IoTHub using an ARM template was the closest thing I could find, but this also doesn't mention creating a new IoT Device. Do you know whether this is possible? And, if so, how can I accomplish this?
Thank you!

@swathipil you should be able to use az cli to create the device
az cli is also python based as it can be installed as pip module though it is not a component of this github repository

@AndreV84 I was actually able to set it up using the portal. I wanted to see if it was possible using an ARM template because our test pipeline sets up Azure resources using the ARM template. If not, no problem. I don't think I need this any longer. You can close this issue. Thanks!

noted
https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/template-syntax

@swathipil I've reached out to someone who may be able to help you with this, will return with information when we have it.

Hey @swathipil
We do not support the ARM templates on the Python SDK. However, please check out this doc (ref link: https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/template-tutorial-create-first-template?tabs=azure-powershell) that guides through the process of setting up an ARM template. Hope this helps.

Closing due to inactivity