Fedora - Use vendor instead of rpm dependencies

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

In order to get this merged, we have to:

Remove all go deps from the spec file
Add bundled provides (see https://src.fedoraproject.org/rpms/podman/blob/8d5f758a1d3b695815945f9daa0907c9c2977aa4/f/podman.spec#_82 )

This must be enforced by upstream CI otherwise we will go mad.

change %goprep to %goprep -k, see https://docs.fedoraproject.org/en-US/packaging-guidelines/Golang/#_prep_goprep_removing_vendoring

(copied from https://issues.redhat.com/browse/COMPOSER-1608, RH-internal only)

@ondrejbudai How do you envision item 2? If I understand correctly, this means that every time dependabot updates a package, the RPM spec file needs to change as well. Do you agree?
Furthermore, Is this a must? I mean, and correct me if I am wrong here, does vendoring really bring in the package to the system? Does it not only compile it into the executable?

@ondrejbudai How do you envision item 2? If I understand correctly, this means that every time dependabot updates a package, the RPM spec file needs to change as well. Do you agree?

 I wonder if we can do this when pushing the spec file downstream. Although, this would break the rule that we ship the exactly same specfile to both up- and downstream.

Furthermore, Is this a must? I mean, and correct me if I am wrong here, does vendoring really bring in the package to the system? Does it not only compile it into the executable?

If I understand it correctly, this is enforced by the packaging guidelines: https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling The important sentence is this one:

All packages whose upstreams have no mechanism to build against system libraries MAY opt to carry bundled libraries, but if they do, they MUST include an indication of what they bundle.

cc @gicmo

Thanks for the links.
Reading the packaging guidelines, I think that their use of the word "Provides" here is misleading because this RPM does not provide any of these libraries to the system on which it is being installed. They are actually being statically compiled into the executable. Instead, they want to get the list of these libraries in order to track possible issues. So, I guess, they wanted to reuse a keyword.
In any case, yes, this means that all dependabot PRs will have to have an additional manual step before being merged. In addition, we will have to make sure the list is up to date in the CI.
So, I think this PR will require the following scripts:

Generate the lines listing the packages from the go.mod file into the RPM
CI - Verify that the list of packages in go.mod are all listed in the RPM spec file

We should also probably think of licensing in the spec file. Since we are building everything from the SRPM, I think we have to change the license field to include licenses of all deps. WDYT?

We should also probably think of licensing in the spec file. Since we are building everything from the SRPM, I think we have to change the license field to include licenses of all deps. WDYT?

Looking at the Podman example, I don't think so. It will turn into quite a rabbit hole

Will defer to @ondrejbudai but it looks good to me!


http_auth parameter not available in Puppet Forge module

Hi,
There seems to be a difference between master branch and Puppet Forge module, the http_auth, user & password parameters are not available in the forge module, but are in GitHub master branch.
Would it be possible for the forge module to be updated please? Authentication options are needed if using Curator with Elasticsearch Shield. I'm using r10k so pulling this module from Puppet forge.
Thanks,
Luke

This can be closed, auth parameters seem to be in forge module.


[v2] injectGlobal break @next

Im not using v2 but just tested and i see if you use minify option it break parsing.
ENV :
node v7.0.0
webpack ^2.2.1
webpack-dev-server 2.3.0
styled-components@next with babel-plugin-styled-components
Steps to reproduce
const Box = styled.div`
  background-color: orange;
  width: 100px;
  height: 100px;
`

Actual Behavior
.app__Box-75m7jb {}.dwklpT {background-color: orange;width: 100px;height: 100px;}
i don't know if this is bug, because i think it has  to work with actual behavior !

This is strange, It's not styled-components , i just removed && installed node_modules folder its working now!

Found what this cause but i don't know why it break, when i import './utils/globalCss' wich is injectGlobal i get that behavior, now i have removed import './utils/globalCss' and its working fine.

It's working fine!

That means that some code inside normalize.css breaks stylis. @thysultan are you aware of this issue?

@mxstbr Will look into it.

The problem is probably how stylis tries to remove comments when a block comment is inlined with a selector.
button,
html [type="button"], /* 1 */
[type="reset"],

will push a patch for this soon.

@mxstbr pushed a patch for this in v1.0.5 changelog... was related to comments and attribute selectors.

@mxstbr  @thysultan i can say its not about comments, i have removed comments manually

@garmjs Then it must be attribute selectors, found the comment issue when i was trying to narrow down the issue using an un-minified normalize.css

when i debug this i found transition mixin fuction cause this bug
import { injectGlobal, css } from 'styled-components'

import normalize from './normalize'
import variables from './variables'

const globalStyles = () => injectGlobal`
  @import url(https://fonts.googleapis.com/css?family=Open+Sans:300,400);
  @import url(https://fonts.googleapis.com/css?family=Lato:300,400,700);

  ${normalize}

  html {
    height: 100%;
    box-sizing: border-box;
    line-height: 1;
  }

  *,
  *::before,
  *::after {
    box-sizing: inherit;
  }

  * {
    margin: 0;
    padding: 0;
  }

  body {
    font-family: 'Lato', "Helvetica Neue", helvetica, Arial, sans-serif;
    letter-spacing: 0;
    position: relative;
    height: 100%;

    background: #f7f8fa;
  }
`

const clearfix = () => css`
  &::after {
    content: "";
    display: table;
    clear: both;
  }
`

const transition = (...properties) => css`
  ${''/* transition: ${properties.join(', ')}; */}
`

const boxShadow = (...properties) => css`
  box-shadow: ${properties.join(', ')};
`

const borderRadius = (...properties) => css`
  border-radius: ${properties.join(', ')};
`

const cubicBezier = () => {
  return 'cubic-bezier(.16, .55, .42, 1)'
}

export {
  globalStyles,
  normalize,
  clearfix,
  transition,
  boxShadow,
  borderRadius,
  cubicBezier,
  variables,
}

intjectGlobal is always appended after component styles.

@astyfx its not that case here, but if you want to append before make sure to import intjectGlobal  before components

@garmjs where is the proper place?

i think injectGlobal must be on the top of other <style> in html

@garmjs @thysultan Do you know if this bug is still present in 2.0.0-15?

This was patched stylis.

Alright, we can reopen the issue if this problem persists :tada:


[R] fix for the 32 bit windows issue

@hetong007 @thirdwing
Fix for #2319 . Win32 R (or the specific mingw version in Rtools) doesn't seem to handle the destruction of thread_local-based storage properly, so I've disabled it only for this specific platform.
The dual 64+32 architecture check and build of a binary package (as an appveyor "artifact") for windows are now enabled in CI.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2994   +/-   ##
=========================================
  Coverage     43.83%   43.83%
  Complexity      226      226
=========================================
  Files           156      156
  Lines         12269    12269
  Branches        466      466
=========================================
  Hits           5378     5378
  Misses         6695     6695
  Partials        196      196

Continue to review full report at Codecov.

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

Great! Thank you for handling this.


Make installable via "raco pkg install stacker"

It would be helpful for this to be installable as a "normal"  racket package. This would allow a single package for  a given course to depend on all of the relevant bits and pieces.
EDIT: With a little experimentation I see that I can specify URLs as dependencies, so perhaps the extra indirection does not add value? Feel free to close the issue in that case.

Thanks for the feedback! Registering this package at the Racket package catalog would certainly make the installation process more accessible. I will leave this issue open.

Now it is there:
https://pkgd.racket-lang.org/pkgn/package/Stacker

I don't see an update of the README?

Resolved with the latest commit f33b4ae


http::cache: Adds serving HEAD requests from cache.

Signed-off-by: Eleonora Kiziv<EMAIL_ADDRESS>Commit Message: Adds serving HEAD requests from cache. Does not add storing HEAD responses in cache.
Additional Description: Relevant issue: https://github.com/envoyproxy/envoy/issues/10020
Risk Level: Low
Testing: Unit, integration

Also you need to fix format before CI runs.
/wait

/assign-from @envoyproxy/senior-maintainers


When calling os.Read() on a named pipe with O_NONBLOCK, that call will sometimes block. I only see the the bug on my M1 Mac. When I run this code in Linux or in the go playground, I do not see the bug. Here is a full program to reproduce:
https://go.dev/play/p/Ryc4jmgag3t
What did you see happen?
jgoel@jgoels-Air bug % go run main.go
2024/03/10 21:37:12 Opening pipe <nil>
2024/03/10 21:37:12 About to read...
2024/03/10 21:37:12 Read from pipe 5 <nil> Hello
2024/03/10 21:37:12 About to read...
2024/03/10 21:37:13 Read from pipe 5 <nil> Hello
2024/03/10 21:37:13 About to read...
2024/03/10 21:37:14 Read from pipe 5 <nil> Hello
2024/03/10 21:37:14 About to read...
2024/03/10 21:37:15 Read from pipe 5 <nil> Hello
2024/03/10 21:37:15 About to read...
2024/03/10 21:37:16 Read from pipe 5 <nil> Hello
2024/03/10 21:37:16 About to read...
2024/03/10 21:37:17 Writer closed pipe <nil>

(...program hangs indefinitely blocking on Read() when it should loop infinitely...)

What did you expect to see?
I expected to see the same output as above, plus the following lines looped infinitely:
2024/03/11 01:35:04 About to read...
2024/03/11 01:35:04 Read from pipe 0 EOF ol
2024/03/11 01:35:04 About to read...

This program does behave the way I'd like if I make any of these changes:

If I replace os.OpenFile() and os.File.Read() with syscall.Open() and syscall.Read(), then the program does work as expected on Mac. It does not block, and it loops infinitely. Here is an example:  https://go.dev/play/p/cflSTMW3RCk
If I remove the call to time.Sleep() in the goroutine, then the program also works as expected.
Finally, the original program works as expected on Linux. I only observe the blocking Read() on Mac.

See also #24164 and #63937.
The combination of CL 118566 and CL 156379 disabled the registration to netpoller for fds of kindOpenFile, which would have prevented something like FIFO with O_NONBLOCK flag in this issue from being added to netpoller. Then CL 495079 came along and enabled the registration for fds of kindNonBlock to netpoller. Therefore, FIFO with O_NONBLOCK flag will be re-added to netpoller.
Besides, the comments:
https://github.com/golang/go/blob/48b10c9af7955bcab179b60a148a633a0a75cde7/src/os/file_unix.go#L180-L185
It seems to be outdated after all those CLs mentioned above and should be removed or updated.
Even so, I still don't see why removing the time.Sleep() call can fix this issue:

If I remove the call to time.Sleep() in the goroutine, then the program also works as expected.

I've drafted a CL for this, working on it.
CC @bcmills @ianlancetaylor

There are two alternatives for this: patch 1 and patch 2, not sure which one is better.


SIL: builtin willThrow does not modify memory or release

This enables access enforcement analysis to classify a dynamic begin_access in access patterns (such as the one below) involving a throwing function as not having nested conflicts.
struct Stack {
  var items : [UInt8]

  mutating func pop() throws -> UInt8 {
    guard let item = items.popLast() else { throw SomeErr.err }
    return item
  }
  ...
}

class Container {
  private var ref : Stack

  @inline(never)
  internal func someMethod() throws {
     try ref.pop()
  }
  ...
}

rdar://141182074

@swift-ci test


Support Swift Language Version 4.2

Also Bump podspec version to 6.1.0

@filipejtdias We are looking at this currently. BTW can you please let us know the problem that you are having? You should be able to use IterableSDK module from your Swift 4.2 projects.


@storybook/addon-knobs not rendering Dropdowns and Text areas as expected in 4.0.0

When I started updating @storybook/addon-knobs version to 4.0.0, observed some issues with Dropdowns and Text areas of addon-knobs.
Issue with Dropdowns:
I am trying to do something like below:
import { select } from '@storybook/addon-knobs';
And in my story, I am adding knobs with dropdowns like:
select('Dropdown 1', ['', 'top', 'bottom'], 'top')
Here I am using options of Array type. On using options of Array type, the rendered dropdowns just have a output of k value like below:

If I use options of Object type then it works fine. It does not work fine with Array type only. I guess that is happening because of the reduce function used in the file @storybook/addon-knobs/src/components/types/Select.js like:
options.reduce((acc, k) => Object.assign(acc, { k }), {})
Issue with TextAreas:
I am trying to do something like below:
import { text } from '@storybook/addon-knobs';
And in my story, I am adding knobs with text areas like:
text('message', '

This is some lorem ipsum foo bar content.

')
Above text is not escaping the html tags. Screenshot for the same is:

Re: dropdowns, you're absolutely right, that was it. #4560 was the duplicate to your issue and I made the fix earlier (#4586), it just hasn't landed in 4.0.0.

This got fixed in 4.0.1, closing!


"/var/lib/postgresql/data/pg_xlog/archive_status": Permission denied

Under docker 1.3.0 and boot2docker 1.3.0 postgres:9.3 quits with the following error:
docker run --rm --name postgres --volume /Users/remmelt/postgres:/var/lib/postgresql/data postgres

The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.

The database cluster will be initialized with locale "en_US.utf8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "english".

fixing permissions on existing directory /var/lib/postgresql/data ... ok
creating subdirectories ... initdb: could not create directory "/var/lib/postgresql/data/pg_xlog/archive_status": Permission denied

Boot2Docker-cli version: v1.3.0
Git commit: deafc19

Docker version 1.3.0, build c78088f

I think this is due to the new boot2docker volume support in 1.3.0. I can see the directory mounted in the boot2docker volume, they are owned by docker(1000)/staff.
$ id docker
uid=1000(docker) gid=50(staff) groups=50(staff),100(docker)

My local OSX user has id 501.
$ id remmelt
uid=501(remmelt) gid=20(staff)

Inside the postgres container, the postgres user has uid 999, and the directory is owned by an unknown user with uid 1000, probably inherited from boot2docker.
I cannot change the ownership of the mounted volume from inside the container, not even when using root.
I thought that a quick fix would be to give the postgres user uid 1000. This works in that the postgres user is now owner of the /var/lib/postgres/data directory and can create files, but it throws the following immediately after:
fixing permissions on existing directory /var/lib/postgresql/data ... ok
creating subdirectories ... ok
selecting default max_connections ... 100
selecting default shared_buffers ... 128MB
creating configuration files ... ok
creating template1 database in /var/lib/postgresql/data/base/1 ... LOG:  could not link file "pg_xlog/xlogtemp.22" to "pg_xlog/000000010000000000000001" (initialization of log file): Operation not permitted
FATAL:  could not open file "pg_xlog/000000010000000000000001": No such file or directory
child process exited with exit code 1
initdb: removing contents of data directory "/var/lib/postgresql/data"

I see a comment on line 4 of the 9.3 Dockerfile that reads "# add our user and group first to make sure their IDs get assigned consistently, regardless of whatever dependencies get added" so I don't know if you're doing any magic with the uid.

I had the same issue.  Turned out that the VM disk was full (the docker share, run "df" to check).  I resolved by cleaning up some old images.  You may also increase the virtual disk size (see VirtualBox docs).


[suggest] support h265

Hopes to support h265

This can't be done by a player, it requires browser support

This can't be done by a player, it requires browser support

https://github.com/goldvideo/h265player
Decode then canvas


feat(java): :sparkles: added method to verify header in apiResponse c…

Closes: #53
Type of changes?

[ ] Bug fixes: Non-breaking change which fixes an issue
[x] New feature: Non-breaking change which adds a new feature
[ ] Performance improvement: Non-breaking change which improves performance
[ ] Documentation: Non-breaking change which improves documentation
[ ] Other: Non-breaking change which does not fit into the above categories
[ ] Breaking change: A breaking change which requires a new version of the library

Checklist for Java project

[ ] Change not related to Java project

Following checklists are optional if above checkbox is selected:

[x] Copyright banner comment added to newly added files, except *.md, *.yml, *.json?
[x] Proper JavaDoc updated in main and test classes for all public classes and methods?
[x] Tests added for changes?
[x] Tests are passing locally?
[x] Check style checks are passing locally?
[ ] Test coverage is at least 80% for newly added changes?
[ ] There are no SonarCloud issues?
[ ] README updated? (if applicable)
[ ] Documentation website updated for the PR raised?

Checklist for Website

[ ] Change not related to Website project

Following is optional if above checkbox is selected:

[ ] Lint check passes locally?
[ ] Prettier check passes locally?
[ ] Build command working fine locally?

Reviewers
@WasiqBhamla/boyka-core

IMPORTANT: Make sure to check the Allow edits from maintainers box below this window

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @MakodeAjay  could you please sign your commits.
Refer the following link to know more about signing commits.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@mfaisalkhatri sign my commits.

@MakodeAjay , please sign the CLA from the link shared by @CLAassistant in above comments.

@MakodeAjay , please raise a new PR with signed commits.


[NOSTR] Each user should be able to open a new chat window with other flash users by searching for their npub, or their lightning address.

The steps to do this would require:

Search with npub: Load user kind0s on npubsearch, currently looks like this:

Onclick: User should be able to open a new chat window (currently looks like this):

User Should be able to view past chats and be able to send & receive messages.
Improve Chat designs
Make sure it works with other flash users.

I'm planning to make small PRs for each of the sub-items

This should be tracked with https://github.com/lnflash/flash-mobile/issues/100 itself for NIP17, already done for nip04


the agent example in the quickstart documentation failed to run

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.

Example Code
import os
from langchain_openai import ChatOpenAI
from langchain_community.tools.tavily_search import TavilySearchResults
from langchain import hub
from langchain.agents import create_openai_functions_agent
from langchain.agents import AgentExecutor

os.environ["OPENAI_API_KEY"] = "*************************************"
os.environ["TAVILY_API_KEY"] = "*************************************"

search = TavilySearchResults()
tools = [search]

# Get the prompt to use - you can modify this!
prompt = hub.pull("hwchase17/openai-functions-agent")
llm = ChatOpenAI(
    model="gpt-3.5-turbo",
    temperature=0,
    verbose=True,
)
agent = create_openai_functions_agent(llm, tools, prompt)
agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True)

agent_executor.invoke({"input": "what is the weather in SF?"})

Description
Running the example code above will cause an error for both gpt-3-turbo and gpt-4-0613. The error message is:
openai.NotFoundError: Error code: 404 - {'error': {'message': 'Unrecognized request argument supplied: functions (request id: XXXXX)', 'type': 'invalid_request_error', 'param': '', 'code': None}}

I searched for this error and found a solution, which involves adding the parameter api-version="2023-07-01-preview". However, I couldn't find a place to input this parameter. After reading through some source code, I finally figured out how:
function_obj = agent.middle[1]
if function_obj.kwargs:
    function_obj.kwargs["extra_query"] = {"api-version": "2023-07-01-preview"}
else:
    function_obj.kwargs = {"extra_query": {"api-version": "2023-07-01-preview"}}

This led to another error:
openai.BadRequestError: Error code: 400 - {'error': {'message': "Invalid value for 'content': expected a string, got null. (request id: XXXXX", 'type': 'invalid_request_error', 'param': 'messages.[2].content', 'code': None}}

After some debugging, I found the reason. Inside langchain_openai, there is a function _convert_message_to_dict:
elif isinstance(message, AIMessage):
    message_dict = {"role": "assistant", "content": message.content}
    if "function_call" in message.additional_kwargs:
        message_dict["function_call"] = message.additional_kwargs["function_call"]
        # If function call only, content is None not empty string
        if message_dict["content"] == "":
            message_dict["content"] = None

This code turns the search result into an AIMessage and, for some reason, does not allow the content to be an empty string, so it makes it None. However, the OpenAI API does not accept this. To make it work, I had to rewrite the code:
def _convert_message_to_dict(message: BaseMessage) -> dict:
    ...
    elif isinstance(message, AIMessage):
        message_dict = {"role": "assistant", "content": message.content}
        if "function_call" in message.additional_kwargs:
            message_dict["function_call"] = message.additional_kwargs["function_call"]
            # If function call only, content is None not empty string
            # ATTENTION: CHANGE HERE
            # if message_dict["content"] == "":
            #     message_dict["content"] = None
        if "tool_calls" in message.additional_kwargs:
            message_dict["tool_calls"] = message.additional_kwargs["tool_calls"]
            # If tool calls only, content is None not empty string
            if message_dict["content"] == "":
                message_dict["content"] = None
    ...
    return message_dict

def new_create_message_dicts(
    self, messages: List[BaseMessage], stop: Optional[List[str]]
) -> Tuple[List[Dict[str, Any]], Dict[str, Any]]:
    params = self._default_params
    if stop is not None:
        if "stop" in params:
            raise ValueError("`stop` found in both the input and default params.")
        params["stop"] = stop
    message_dicts = [_convert_message_to_dict(m) for m in messages]
    return message_dicts, params

llm.Config.extra = Extra.allow
llm._create_message_dicts = partial(new_create_message_dicts, llm)

I mean, really? I'm not sure what I did wrong, but it's certainly not easy to make it work. If it's not a bug, I hope to get a simpler and more elegant solution.
System Info
langchain==0.1.0
langchain-community==0.0.10
langchain-core==0.1.8
langchain-openai==0.0.2
langchainhub==0.1.14
Related Components

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

UPDATE: a better way to add api-version
llm = ChatOpenAI(
    model="gpt-3.5-turbo",
    temperature=0,
    verbose=True,
    model_kwargs={"extra_query": {"api-version": "2023-07-01-preview"}}
)

UPDATE: Code like this works, but it doesn't use OpenAI's function call and is very unstable; sometimes it results in an uncertain number of API calls.
import os
from langchain_openai import ChatOpenAI
from langchain_community.tools.tavily_search import TavilySearchResults
from langchain import hub
from langchain.agents import initialize_agent, AgentType

os.environ["OPENAI_API_KEY"] = "*************************************"
os.environ["TAVILY_API_KEY"] = "*************************************"

search = TavilySearchResults()
tools = [search]

# Get the prompt to use - you can modify this!
prompt = hub.pull("hwchase17/openai-functions-agent")
llm = ChatOpenAI(model="gpt-3.5-turbo", temperature=0, verbose=True)

agent_chain = initialize_agent(tools, llm, agent=AgentType.STRUCTURED_CHAT_ZERO_SHOT_REACT_DESCRIPTION, verbose=True)
# also trigger error
# agent_chain = initialize_agent(tools, llm, agent=AgentType.OPENAI_FUNCTIONS, verbose=True)

print(agent_chain.invoke({"input": "current weather in San Francisco"}))

Running the code in this manner, I think it doesnt ensure the chat history being well parsed, by that I mean the history is not taken into account.


Support HSTS option on zones

I'd like to be able to configure Cloudflare's HSTS offering from Terraform.

In the web dashboard, the following options are shown:

This is configurable under zone_settings override

Thank you for the pointers. Just fyi the Terraform link above is broken, the new one is: https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/zone_settings_override


How to change font

I'd like to try some other common font, e.g., Times New Roman. Is there an easy way to change that?

Allo,
If you want to change the font, I would suggest to use these commandes (they're shown in the awesome-cv.cls and can be tweaked).
\footerfont{Times New Roman}
\bodyfont{Times New Roman}
\bodyfontlight{Times New Roman}

However you have to make sure the new font you want to use is in the right directory or use a
\RequirePackage[default,opentype]{sourcesanspro} like the one in the awesome-cv.cls like you would use \sourcesanspro after, but I don't know exactly how to use it.
Hope it helps.

You can define a custom font family and use it different sections, here is how I use Montserrat font for the cover letter body:
%-------------------------------------------------------------------------------
%                Configuration for fonts
%-------------------------------------------------------------------------------

\newfontfamily\alishirfont[
  Path=\@fontdir,
  UprightFont=*-Regular,
  ItalicFont=*-Italic,
  BoldFont=*-Bold,
  BoldItalicFont=*-BoldItalic,
]{Montserrat}

% For elements of cover letter

\newcommand*{\mylettertextstyle}{\fontsize{8pt}{1.4em}\alishirfont\selectfont\color{graytext}}

p.s.: You should copy font files in the font directory.

Hello everybody, I am still interested in resolving this issue.
I am not a proficient coder in TeX, but I am very interested in understanding how to properly change fonts.
Like, I got that you have to add the .tff files to the font directory and adjust the code in the awesome-cv.cls.
But this alone doesn't fix the problem for me.
Is there anyone who understands this and would like to share an easy explanation?
Thank you so much!

Hello everybody, I am still interested in resolving this issue.
I am not a proficient coder in TeX, but I am very interested in understanding how to properly change fonts.
Like, I got that you have to add the .tff files to the font directory and adjust the code in the awesome-cv.cls.
But this alone doesn't fix the problem for me.
Is there anyone who understands this and would like to share an easy explanation?
Thank you so much!

@sylhare I got this error when adding three lines mentioned by you:
LaTeX Error: Missing \begin{document}. [\footerfont{T]

Any idea how to fix it?

Well, I found the solution on StackOverflow:
Add these just before \begin{document}
\let\headerfont\rmfamily
\let\headerfontlight\rmfamily
\let\footerfont\rmfamily
\let\bodyfont\rmfamily
\let\bodyfontlight\rmfamily

How does one change the font size? Or make the font a bit thicker

I'm trying to change the font style but I still can't do it

I'm trying to change the font style but I still can't do it

I at some point succeeded, but the end result didn't look well. I suggest keeping to the defaults

The font size can be changed inside awesome-cv.cls lines 191-234


use$SetOnSave issues

Completely busted right now.
From the mailing list:
First:
My User schema definition contains { use$SetOnSave: false } so that I am able to use User.update() to, for example, increment a counter with { $inc: { counter: 1 } }. However, if I don't pass on of the options { safe: true } or { upsert: true }, then the record gets blown away -- only the _id is left, all the other fields are gone. It is NOT an upsert operation, so I don't know why the "upsert" option even works. I also don't understand what the "safe" option is doing. The source says that it enables the use of schema defaults -- which is a great option, should always be true by default, IMO --, but again, I am trying to increment a counter. Without one of those options, mongoose seems to think I should be sending the entire document (which I cannot even do in an update()).
2nd:
Having defined my User schema with { use$SetOnSave: false }, I am not unable to use the method described here to update a record.
E.g.
User.findById( myId, function(err, doc) {
  // do some changes to doc
  doc.save(function(err){
    if (!err) {
      res.redirect('home'); // I don't think this is the wrong place. The process fails no matter where I put this.
    }
  });
});

FWIW, the API documentation should perhaps make mention of the "idiosyncrasies" of using Model.update() right now. I know I've personally spent a lot of time trying to figure out why my update's were performing like $set operations, and then was subsequently further confused when changing the use$SetOnSave value had no effect.  It wasn't until tracking down this section of code that I realized no options are consulted before converting an update command to a $set operation.
So, not to make undue noise over an issue that's obviously already getting attention, but it may be nice to offer some forewarning in the API docs, given that this issue has an effect on persisted data and can potentially be quite harmful if gone unnoticed.

Thanks Aaron (for the update, and the rest of your work on Mongoose)!

:D


Logo not displaying correctly on Ory Cloud Invoices

Preflight checklist

[X] I could not find a solution in the existing issues, docs, nor discussions.
[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines.
[X] This issue affects my Ory Cloud project.
[X] I have joined the Ory Community Slack.
[X] I am signed up to the Ory Security Patch Newsletter.

Describe the bug
Logo not displaying correctly on invoices. See screenshot below, red container.
Screenshot on Imgur
Reproducing the bug
In the Ory console, navigate to 'Usage & Billing', select your Start-Up plan and click on 'Invoices & Active Subscriptions'. Download you latest invoice and open the PDF document.
Relevant log output
No response
Relevant configuration
No response
Version
Cloud Console
On which operating system are you observing this issue?
macOS
In which environment are you deploying?
Ory Cloud
Additional Context
No response

Thank you for the report. @zepatrik can you take a look? We're probably using the wrong logo for the PDF (white on transparent vs blue on transparent) for the given background (paper, white).


ci: add codeql action

Github recommended me to add a CodeQL analysis github action

Codecov Report

Merging #296 (a9746a7) into main (deb5554) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #296   +/-   ##
=======================================
  Coverage   88.07%   88.07%
=======================================
  Files         218      218
  Lines        3926     3926
  Branches      452      452
=======================================
  Hits         3458     3458
  Misses        468      468

Continue to review full report at Codecov.

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


[Extractor] Cannot deploy API resource with a revision.

Where did you encounter this bug?

Extractor

Describe the bug
I get some errors when dploying arm template which was generated by extractor.

The name of api resource doesn't contains revision in basefilename-api.template.json

resource name is just "[concat(parameters('apimServiceName'), '/test-api')]", but other resources in same file depends on "[resourceId('Microsoft.ApiManagement/service/apis', parameters('apimServiceName'), 'test-api;rev=2')]"
The workarround is manually adding revision name of api, but it will be overrided when extract next time.

Cannot change current revision

"ValidationError" occurs when deploy api which "isCurrent: true when other revision is CURRENT.
The workarround is set isCurrent=false after extract and before deployment.

What version of the DevOps Resource Kit is being used?
1.0.0-beta.4
What version of dotnet are you running?
6.0.300
What platform (OS and version) are you running on?
Windows, not WSL
The command line you are running?
dotnet run extract --extractorConfig .\extractSettigs.json
The configuration file being used?
{
    "sourceApimName" : "apim-dev",
    "destinationApimName": "apim-prod",
    "resourceGroup": "myRgName",
    "fileFolder" : "extractor-output",

    "linkedTemplatesBaseUrl" : "https://hoge.storage.url/container/",
    "linkedTemplatesSasToken": "?sastoken=goeshere&type=linkedTemplate",
    "policyXMLBaseUrl":  "https://hoge.storage.url/container/policies/",
    "policyXMLSasToken": "?sastoken=goeshere&type=policyXml",

    "apiName" : "test-api",
    "includeAllRevisions" : true,
    "baseFileName" : "apiconfig"
}

How is your API Management Instance set up?
I created my APIM Developer SKU as Owner of subscritpion. VNet is not configured.
What logs (if any) were produced?
{
    "status": "Failed",
    "error": {
        "code": "ValidationError",
        "message": "One or more fields contain incorrect values:",
        "details": [
            {
                "code": "ValidationError",
                "target": "IsCurrent",
                "message": "Can't set new revision as current."
            }
        ]
    }
}

Expected behavior
The extracted file should be deployed without no change, becase I want edit API definitions in the Azure portal(or Azure CLI).
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hello @ayuina.
Thank you for reporting the issue.
Please be informed that the changes (https://github.com/Azure/azure-api-management-devops-resource-kit/pull/782) addressing the bug are merged into the main branch and will be available in the future releases, hence closing the issue.
Regarding the changes:
IsCurrent property is removed from generated templates and was replaced with additional template Api release.
In order to make the Api current, api-release template should be deployed as well
Release template will be generated for each api revision which is current (given extractor parameters: single Api with all revisions)
Api-release template will include release resources for each current Api if all APIs are extracted
If you will face issues with revisions, please feel free to open a new issue or reopen this one.
Best regards


Add more KeyboardKit-specific view modifiers

To work more like other parts of SwiftUI, KeyboardKit should have more view modifiers that can be used to more easily apply KeyboardKit-specific configurations.
For instance, since SwiftUI has an .autocorrectionDisabled() view modifier, which KeyboardKit honors, perhaps KeyboardKit should have an .autocompleteDisabled() as well?

I will add this over time, closing this for now.


Does it work with the last Factorio release?

Also, that's something worth putting up front in web and README.md IMHO

The web states 0.17 compatibility.

Would it be insane to declare it in the repo?

Well, the current version is written in the front page (go to http://www.doomeer.com/factorio, it says "Current Version -- Recipes were updated for version 0.17!"). Maybe you missed that, or maybe you arrived from one of the forks that are maintained by other users who may not have updated yet?
I can definitely update README.md though.

I was browsing directly in GitHub (IIRC). I see you've decided to put it in the README.md, IMO that file serves to describe what it is, what is it for, version, authors...


Are there any plans to allow socket mode when deploying to the app directory?

Description

Currently, socket mode doesn't seem to be allowed when deploying to the app directory.
Are there any plans to allow socket mode when deploying to the app directory?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[x] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @G-yhlee, thanks for writing in!
Unfortunately, we don't have any updates on it as of today, but we'll share your feedback with the product team. Thanks again for your time and interest on this!


flv to mp4 compress

can we compress mp4 ? what path should i follow
https://github.com/yapingcat/gomedia/issues/23#issuecomment-1226672336

can we compress mp4 ?

you means compress the mp4 file into zip file ?


test: remove unused variables

When enabling all warnings the unused variables are reported:
$ cmake .. -DCMAKE_C_FLAGS=-Wall

Any thought on put this flags in CMakeList.txt ?

Any thought on put this flags in CMakeList.txt ?

I think that would be good if it could be added :+1:

At the very least we should add these flags in CI... but that doesn't' need to block this PR either.

are you interested in updating the CI too?

Sure thing, I'll try to take a look at this later today or early next week.
Landed in 1db9b7202885aa0fcae4b88a3699cd29e7370759.


Arguments implementation plus tests

Took some time but at last the requested Arguments pass through code...  :-( )
What do you think ???

Looks good. I'll have to merge it into the flatiron branch, but should be able to do that tonight.


Jupyter Support and Integration

Is your feature request related to a problem? Please explain.
Currently, Javis does not work within a Jupyter environment (be it lab or notebook). One cannot use Javis to view frames of an animation easily like they could using the built-in Javis image viewer. Furthermore, on some platforms, the Javis image viewer is not performant due to it being built using Gtk.jl.
We could perhaps get around this performance issue as well by providing browser-based support built around using Jupyter as a platform.
Describe the solution you'd like
I would love to build functionality into Javis Jupyter notebook support by creating something like this:

Source: https://blog.dominodatalab.com/interactive-dashboards-in-jupyter/

I think this functionality could be achieved by using Interact.jl. Furthermore, this might be better suited to go into its own repository. So, initial development may take place on the main branch but later move to its own dedicated location.
Describe alternatives you've considered
The best alternative would be the Javis image viewer but since not all platforms best support GTK based GUIs, this would provide greater applicability of Javis.

See this to show gif in Jupyter: https://www.reddit.com/r/Julia/comments/iqp5cr/adding_the_power_of_animationsjl_to_javisjl_v02/

The idea for a dedicated packge to view stuff sounds pretty cool(a similar example is https://github.com/JuliaAstro/AstroImageView.jl). which serves like a frontend for the JuliaAstro image processing tools. If we need that sort of complex functionality for Javis in the future, it's a good way to go ahead. In which case I think a more general "LuxorViewer.jl" package makes more sense since that will automatically support Javis.

I also tried to look into Interact.jl side, the code to replicat the gtk functionality is pretty much simple. The only place where I seem to be stuck is rendering the frame matrix in the cell.
I tried to reuse @TheCedarPrince 's code
Interact.@manipulate for f in 1:length(frames)
        fr = get_javis_frame(video, actions, f)
        # colorview(RGB, fr)

        frame_mat = transpose(fr)
        s = CairoImageSurface(frame_mat)
        context = Javis.Luxor.Cairo.creategc(s)

        image(context, s, 0, 0, video.width, video.height)
    end

But it returns Nothing. MAaybe something I am missing here? Or is Cairo not supported for Jupyter.
Also since Javis has a Images.jl dependecy, I tried using the Images.colorview method to render the image but the Javis.get_javis_frame method returns a Array{ARGB, 2}, and I am not sure I we can get Images to deal with that without some conversions which might make the viewer slow?


Add Campaign Service Report

Not going to do this. It would cost additional API calls and no one seems to care about campaign stats.


Added CI report generation for softmax benchmark

Added report generation for softmax benchmark. New report contains additional fields for datetime, benchmark name, GPU used. This report can be directly uploaded to the benchmarking database.

Please fix the pre-commit issues. Also, could you please make a branch in this repository to run the workflow "Triton benchmarks" on it?


Added onMove() callback

Added onMove() callback allowing injection of code before the move completes. Additionally the user can stop the move by returning false.

For users view this results in item being almost dropped, but get reverted in the last moment.
So I've implemented it as confirmChange which is being invoked on every move (not a final drop only). Check example 3.
Thank you for your time! It helped me a lot before I've done it my way.


Pattern for latest tag compare

Hi, I get an error like this
Run fregante/release-with-changelog@v3
  with:
    token: ***
    exclude: ^meta|^document|^lint|^refactor|readme|dependencies|注释|文档|ci|CI|README
    commit-template: - {title} ← {hash}
    template: ### Changelog

  {commits}

  {range}

    date-format: short
    sort: desc
Computed range: v1.1.1..dev-latest
Error: Validation Failed: {"resource":"Release","code":"already_exists","field":"tag_name"}

It seems that already have "dev-latest" in the last commit.
Can you add tag extract pattern like the tag should start with "v" to avoid the bug?

PR welcome 🙂


issue with images

I notice that the images don't load when I click on a product and while in the product page I click on any link from the side nav and then go back to the home page. do you know what could be the problem? I try using safari and chrome and I have the same result in both.

I have fixed the problem. The issue was the src to the image was relative and not based on the root of the application.
Before:

Now:

Note the / at the begging of the src.


Add exception details to response

This pull request is in regards to issue #1317. I reproduced the 400 error that kenjones-cisco was talking about using this code: gist.github.com/keyanp/bc4d1a3a1e6c5cb5170a
So I took a look at making a subclass of BadRequest, but indeed this is part of Werkzeug. So instead I was thinking of adding the ValueError content to the shell console or to the response body. By just adding the exception to the response body it at least improves the 400 message and probably will make it easier to debug. The response body (which is also in the gist above) would now look like:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>400 Bad Request</title>

Bad Request

No JSON object could be decoded

If this isn't really that helpful perhaps I can just output the exception details to the console? Could someone point me in the direction of where in the codebase Flask handles console logging?

Looks pretty good, if you can manage, please also add tests (I can do that too if you don't have time)
Thanks!

Sure I'll add tests, doesn't seem to be a consensus on checking for DEBUG setting though. Do you suggest using _app_ctx_stack.top similar to its use for max_content_length?

Do you suggest using _app_ctx_stack.top similar to its use for max_content_length?

I was thinking about using current_app and catching RuntimeError, but using the stack directly as you say seems more consistent with the rest of the code.

@untitaker Thanks for looking over the previous commits. I added a couple tests and changed the logic to use _app_ctx_stack similar to max_content_length. Let me know if you see anything that still needs work.

@untitaker Thanks for looking over the previous commits. I ended up added a couple tests and changed the logic to use _app_ctx_stack similar to max_content_length.
The issue I was running into with the tests was that Python 3.4 throws a different exception message for trying to load data from an empty json, and so I ended up hardcoding the exception details. I'm not sure how you feel about that, but I think since the function on_json_loading_failed is only being called in this specific instance it should be safe to pass a string manually. Let me know your thoughts.

It's true, it's easier for assertion for the error message to be consistent. This can be achieved with BadRequest('Failed to decode JSON object: {}'.format(e)) too, though.

Changed so that the response content for Python 3.3 and below (in DEBUG mode) is now:
Failed to decode JSON object: No JSON object could be decoded
While for Python 3.4 it is:
No JSON object could be decoded: Expecting value: line 1 column 1 (char 0)

For some miraculous reason we now leak the request context... will look into this later.

Yeah I'm not sure why the request context was being leaked, but the error I was getting was specific to formatting strings in Python 2.6, where you are required to provide an index for arguments. Build looks good now though, and I incorporated the changes you commented on.

Excellent, thanks!


Panic when running on method with missing receiver

Input:
package main

func () Method() {}

go build:
./foo.go:3: method has no receiver

go lint:
panic: runtime error: index out of range

goroutine 1 [running]:
github.com/golang/lint.receiverType(0xc20803ad50, 0x0, 0x0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:1398 +0x344
github.com/golang/lint.func·002(0x7f96f166a2c8, 0xc20803ad50, 0x0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:319 +0x86
github.com/golang/lint.walker.Visit(0xc20800a5e0, 0x7f96f166a2c8, 0xc20803ad50, 0x0, 0x0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:1432 +0x49
go/ast.Walk(0x7f96f166a270, 0xc20800a5e0, 0x7f96f166a2c8, 0xc20803ad50)
        /usr/lib/go/src/go/ast/walk.go:52 +0x59
go/ast.walkDeclList(0x7f96f166a270, 0xc20800a5e0, 0xc208030720, 0x1, 0x1)
        /usr/lib/go/src/go/ast/walk.go:38 +0xd4
go/ast.Walk(0x7f96f166a270, 0xc20800a5e0, 0x7f96f166a0d8, 0xc20806e380)
        /usr/lib/go/src/go/ast/walk.go:353 +0x2df1
github.com/golang/lint.(*file).walk(0xc2080323c0, 0xc20800a5e0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:1408 +0x83
github.com/golang/lint.(*pkg).scanSortable(0xc208037310)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:324 +0x269
github.com/golang/lint.(*pkg).lint(0xc208037310, 0x0, 0x0, 0x0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:149 +0x82
github.com/golang/lint.(*Linter).LintFiles(0xc20802bc18, 0xc20803aba0, 0x0, 0x0, 0x0, 0x0, 0x0)
        /home/dominikh/prj/go/src/github.com/golang/lint/lint.go:110 +0x531
main.lintFiles(0xc20802bf18, 0x1, 0x1)
        /home/dominikh/prj/go/src/github.com/golang/lint/golint/golint.go:83 +0x268
main.main()
        /home/dominikh/prj/go/src/github.com/golang/lint/golint/golint.go:50 +0x367

I'm not sure if the bug is in golint, or in the parser.

I'm a little surprised that the go/parser package is accepting that, so I'm not surprised that golint is misbehaving on it. I'm not sure it's worth the time to harden golint against all kinds of AST malformations, so consider filing this at https://golang.org/issue/new as a bug against the go/parser package.

Will do. lint also isn't the only program failing on this input. It really should be an upstream issue.

@dsymonds According to https://github.com/golang/go/issues/11271 it's actually the responsibility of the client to check for such abnormalities. Whether you think it's worth it or not is up to you, but go/parser doesn't seem to be the right place for the fix.

It should be easy enough to make golint cope a bit more gracefully.


Improve deploy and undeploy tasks.

This refactor let us to use a closure to deploy/undeploy one or multiple files.
For example, for multiples files:
liberty {
    deploy {
        dir = "C:/resources"
        include = " *.war, *.ear "
        exclude = " *.txt "
    }
}

Include and Exclude could be set with wildcards.
If you wan to deploy a single file you should use the following code instead:
liberty {
    deploy {
        file = "C:/resources/test-war.war"
    }
}

This was updated with the latest changes to the master branch. If anything else is needed here please let me know.

Great. Wondering about two things:

can you make it work with multiple deploy or undeploy elements? We want to support deploying or undeploying multiple applications.
is it possible to see if 'war' or 'ear' plugin has been applied to the project and if there is no deploy element specified used the archive location provided by the 'war' / 'ear' project? That way it could seamlessly work with war, ear, or other projects.

Yes, this could deploy multiple elements if there are in the same dir. I mean, this work as a fileset. For example, to deploy multiple files would be something like this:
 liberty {

        wlpDir = 'c:/wlp'
        serverName = 'myServer'

        deploy {
            dir = 'c:/files'
            include = 'app.war, sample.war'
            exclude = 'test-war.war'
        }
    }

Is this what you asked?, if not I was wondering if you could reply with an example of how this should work. Thanks.

This was updated to support deploy using the WAR/EAR plugins. If anything else is needed please let me know.

@oscarlv Can you please update and resubmit the pull request?

Done. If anything else is needed please let me know.

Thanks @oscarlv. I've sent separately some feedback for minor stuff (couple of typos, formatting, etc).
There's two things I think should be handled with higher priority than those though:

Could you take a look at the changes to README.MD on the commit 2ee6a318823d22836f1c3ca6c1645318e94659e8 ?  Is reporting almost every line as changed.

In the DeployTask I see the logic implemented to address previous comments. However I think that, if I understood correctly, it should be checked first if the deploy element is specified, and if not, check the war and/or ear elements. As is, I think the code will do backwards: first check war and ear, and then the deploy element. Additionally, I think similar logic should be implemented on the UndeployTask, so if no deploy element is provided, it can work based on ear/war.

@jgawor Can you please correct me if I'm wrong in this last comment?

Thanks for the feedback @rmcmx, I've just updated my pull request according to your feedback. Now just is pendig what @jgawor has to say about the logic implemented that you mentioned.

Thanks @oscarlv for the changes. I added some additional ones and merged.


Fix relative jumps

This PR is meant to fix #41 .
The PR is more of a prototype for now (not finished). I raised it to be able to:

discuss the implementation of "emit multiple instructions"
get ideas on a behaviour of binutils-esp32ulp that I cannot yet explain.

1. opcode -> Multiple instructions
For now the implementation (see commit fa09eb3) works by the i_jumpr and i_jumpr methods returning a tuple when multiple instructions should be emitted, and the assembler testing for "is this a tuple" and if so, it looping over the elements. Otherwise it does what it did before, i.e. adds 1 instruction to the section. The current structure for this logic feels a bit rough, but for now I intend mostly to get feedback on the approach.
To help this, for pass 1, when we're not yet calling the i_* methods, because we only want to determine section sizes and label offsets, I added a function that can return "how many instructions will this opcode result in" based on the opcode and its arguments. The default return value is 1, but for the JUMPS and JUMPR opcodes and with the specific conditions that will result in 2 instructions, the function returns 2. Simple and effective, but technically a "duplication of logic" (because i_jumps/i_jumpr methods also make this decision). Perhaps there are other approaches for how to make this even simpler or more elegant?

2. Weird binutils-esp32 behaviour with negative immediate offsets
For a reason I cannot yet figure out, when using negative immediate offsets for JUMPS or JUMPR instructions, binutils-esp32 generates instructions differently than when using label references that are before the current PC (back-jumps).
The behaviour is as follows: for SYM references, if the offset is negative, set the sign bit and use the absolute value of the offset. for IMM references however, the sign bit is set the same way, but for the offset the negative (not absolute) value is used.
At the "field" level in the opcode the difference is such that:

For a negative symbol reference, e.g. -9, sign bit is 1 and the offset is 9
For a negative immediate reference, e.g. -2, the sign bit is 1 and the offset is 126 (which is the 7-bit two's complement of 2)

See commit 3ae1e20 for the implementation that mirrors what binutils-esp32ulp does. So "it works", in that py-esp32-ulp will now generate the same binary output, but I still don't know why binutils-esp32ulp treats these offsets differently (even after a fair bit of scanning the binutils-esp32ulp source code). If we need to jump 4 words back, because there is a label 4 instructions earlier, of whether I give an immediate offset of -4, I would expect the resulting jump instruction to be the same. I have not tested this on the actual device for how the ULP interprets these different "representations".

3. One more thing
One last thing, which is not yet in this PR, is another weird behaviour of binutils-esp32ulp. This relates to using constants defined with .set as arguments to the JUMPS and JUMPR opcodes. I would expect them to behave like immediate offsets, but they are always "interpreted" as 0.
So the following two JUMPS statements result in the exact same instruction (namely 2a00 0084):
  .set const, 42
  jumps 0, 1, lt
  jumps const, 1, lt

I started wondering whether this is a bug in binutils-esp32ulp? Or I have wondered if "resolving the constant" is deferred by binutils-esp32ulp to the linker perhaps, and maybe we're calling the linker wrong in the compat test scripts, but so far I cannot see that this is true. The binutils-esp32ulp test scripts call the linker the same way (then again, sadly they don't have this specific example in their tests).

Happy to get feedback/thought/ideas on the above, and the current implementation.

Nevermind about point 3 above. I just got it! It is in fact the linker that fills in the const and therefore the symbol must be exported with .global const. When const is exported, both py-esp32-ulp and binutils produce the same output. I think I will leave this one as-is then, because technically, with binutils-esp32ulp, it's a case of "using it wrong", if one doesn't export the symbol.

I'm now relatively sure, that point 2 is a bug in binutils-esp32ulp. I dug into that code, debugged around a bit and found the place where it is likely wrong.
I compared what binutils-esp32ulp does with the ULP "macro approach" in the ESP-IDF, and there the offset is always converted to absolute and the 8th bit is set based on sign (as we do in py-esp32-ulp). I also read the documentation (ESP32 TRM), which is mentioned in issue #41, and it too states that that bit 0-6 (7bits) are always a positive number and the 8th bit determines how the ULP will interpret the offset (positive or negative). So 2 sources against 1 encode the negative offsets the same way between values from symbols and immediate values.
I have a fix suggestion for binutils-esp32ulp and will send it as PR to that project and see what they say. Then I guess we'll know more.
Until then I think i'll revert the last patch of this PR, and thus simply "leave-it-as-before" (which I currently think is the right way anyway). Then we can hopefully merge this change already before any response from espressif and if we need to change, we can change that later.

I have now made a PR to the binutils-esp32ulp project (https://github.com/espressif/binutils-esp32ulp/pull/18). Let's see what they say.

So I got a positive response to the PR to binutils-esp32ulp. I will wait until it's fully merged, and then I will update these tests to pass against the updated binutils-esp32ulp.

Now that my fix to binutils-esp32ulp has been merged, I finally managed to clean up this PR.
Points 2 and 3 from the original description of this PR are resolved. Now we can focus on the implementation of the "multi-instruction" approach.
In summary: opcodes.i_* functions can now optionally return a tuple, in which case each element is appended as an instruction to the TEXT section. A separate opcodes.no_of_instr function can be consulted during pass 1 of assembly to determine how many instructions a given opcode will result in.
Happy to get feedback on the implementation.

Fixed all the things you spotted. Thanks for that. Pushed latest code (amended commits).


[NCL-7409] Set artifactQuality for uploads and downloads

It is set to NEW for uploaded artifacts. For downloaded ones it is either new for sources ignored from promotion and IMPORTED for everything else.
Follows https://github.com/project-ncl/pnc-api/pull/109

retest this please


Not all config being fixed

AllowPeerVoice=True also needs to be set in ROEngine.ini.

Thanks! It's been added in version 1.0.1.


Multiple GPUS

When I tested llama2-70b on an A800 graphics card, I encountered the problem of insufficient video memory. I want to ask how I should write the command if I want to test on two A800 graphics cards?I tried this command again：

But an error was reported：

Quick fix: https://github.com/hao-ai-lab/LookaheadDecoding/blob/main/applications/eval_mtbench.py#L511  in this line set DIST_WORKERS=0. I will do a refactor later to fix this thoroughly.

Quick fix: https://github.com/hao-ai-lab/LookaheadDecoding/blob/main/applications/eval_mtbench.py#L511 in this line set DIST_WORKERS=0. I will do a refactor later to fix this thoroughly.

It is now running normally. Thanks to the author for his timely reply! ! !

btw,do you have code for testing accuracy like the medusa project?https://github.com/FasterDecoding/Medusa/blob/v1.0-prerelease/medusa/eval/heads_accuracy.py

I did not implement such a function, but it may not be too hard to compute accuracy by dividing the number of per-step accepted tokens by the number of per-step speculation tokens.

It can run normally when USE_LADE is set to 1.When I just used multiple GPUs to test without using LADE, it turned out that it was out of memory. Is there something wrong somewhere?The following is my execution command：

The error is reported as follows：

When USE_LADE is set to 0, please set --use-pp=1 to use huggingface's pipeline parallelism, or you can use the deepspeed for tensor parallelism as the script I provided https://github.com/hao-ai-lab/LookaheadDecoding/blob/main/applications/run_mtbench.sh#L33

Fixed it!Thank you for your patient reply! If I use a smaller model such as llama2-7b-chat and set USE_LADE to 0, what will be the impact if --use-pp is also set to 0?

I guess it will be put on a single gpu. Check this https://github.com/hao-ai-lab/LookaheadDecoding/blob/main/applications/eval_mtbench.py#L214 and https://github.com/hao-ai-lab/LookaheadDecoding/blob/main/applications/eval_mtbench.py#L250 for model placement configuration.

ok,got it!


Method not found: 'Void SharpAstrology.DataModels.PlanetPosition.set_Longitude(Double)

Hello!
Trying to run examples and got this error:
Method not found: 'Void SharpAstrology.DataModels.PlanetPosition.set_Longitude(Double)
on line:
var chart = new AstrologyChart(date, eph);
How can I fix it?
Code is below:
`using SharpAstrology.DataModels;
using SharpAstrology.Enums;
using SharpAstrology.Ephemerides;
using SharpAstrology.Interfaces;
//var date = new DateTime(1981, 10, 27, 9, 45, 0, DateTimeKind.Utc);
var date = new DateTime(1988, 9, 4, 1, 15, 0, DateTimeKind.Utc);
//using var eph = new SwissEphemeridesService(ephType: EphType.Moshier).CreateContext();
var ephemeridesService = new SwissEphemeridesService(ephType: EphType.Moshier);
using IEphemerides eph = ephemeridesService.CreateContext();
// Create chart
var chart = new AstrologyChart(date, eph);
// Houses
Console.WriteLine(chart.HousePositions is null ? "No houses available" : "Houses available");
// Output: No houses available
// Planet positions
foreach (var planet in chart.SupportedObjects)
{
Console.WriteLine($"{planet} - {Math.Round(chart.PositionOf(planet).Longitude, 2)}° in sign {chart.SignOf(planet)} and constellation {chart.ConstellationOf(planet)} is {chart.MotionOf(planet)} moving.");
}
// Output:
// Sun - 161,73° in sign Virgo and constellation Leo is Forward moving.
// Moon - 82,41° in sign Gemini and constellation Taurus is Forward moving.
// Mercury - 185,87° in sign Libra and constellation Virgo is Forward moving.
// Venus - 116,41° in sign Cancer and constellation Cancer is Forward moving.
// Mars - 10,96° in sign Aries and constellation Pisces is Retrograde moving.
// Jupiter - 65,44° in sign Gemini and constellation Taurus is Forward moving.
// Saturn - 265,95° in sign Sagittarius and constellation Sagittarius is Forward moving.
// NorthNode - 344,1° in sign Pisces and constellation Aquarius is Retrograde moving.
// SouthNode - 164,1° in sign Virgo and constellation Leo is Retrograde moving.
// Uranus - 267,04° in sign Sagittarius and constellation Sagittarius is Retrograde moving.
// Neptune - 277,48° in sign Capricorn and constellation Sagittarius is Retrograde moving.
// Pluto - 220,35° in sign Scorpio and constellation Libra is Forward moving.
// Angles between planets (-180°,180°].
Console.WriteLine(chart.AngleOf(Planets.Sun, Planets.Mars));
Console.WriteLine(chart.AngleOf(Planets.Mars, Planets.Sun));
// Output:
// 150,76609634460928
// -150,76609634460928`

Hi, I guess you are using a version older than 0.8.0? The problem is a private setter for the longitude property. I recognized, that I've made a design mistake and the code should work with version 0.9.0 upwards. Please give me feedback after you tried it so I can block this older versions from NuGet.

Hi!
I'm using fresh nuget packets - SharpAstrology.Base version 0.10.0 and SharpAStrology.SwissEph 0.2.1.


Shop chest is lootable with hopper minecart

Any non-op player that is not shop owner can break the block under a shop chest and loot its inventory with a hopper minecart.

This bug has now been fixed in commit ad61c13.


jq:process_json needs a timeout

When experimenting with the jq function inside the rule engine of EMQX I have found out that jq:process_json needs a timeout. The behavior can be very bad otherwise. A port program get stuck which means that subsequent calls will also get stuck which is not acceptable.

This is fixed for the port implementation in https://github.com/emqx/jq/pull/27. I leave this open until we have fixed this in the NIF implementation as well.

This has also been fixed for the NIF implementation now.


Add element.contains method

LGTM but might as well add it to the document prototype too?
> document.contains(document.body)
true

Right, good call! Added a ref to the document prototype as well as a corresponding test.

Awesome, looks identical to a patch I had in my fork but never got around to adding tests for. :+1:

Awesome! Lemme know when you have a chance to publish to npm.

You'll have to bug @Raynos for that :smile:

Can we throw in https://github.com/Raynos/value-event/pull/21 too ;)


Clean up usage of fixtures and optimize unit tests

Right now, the contracts use hardhat-deploy plugin for deployments. However, we have been wrongly using ethereum-waffle package's createFixtureLoader() method. This essentially means that right now, none of the unit/fork tests are using fixtures at all. Changing them to use hardhat-deploy packages deployments.createFixture fastens the tests by quite a significant time

I've worked on this today in PR https://github.com/OriginProtocol/origin-dollar/pull/1745


Handle New / Old SDK Errors

What
Closes LIT-191. See issue for more details.

This PR makes it so that old error codes (storage_error, not_authorized) from the node, as well as errors thrown from within the SDK (unrelated to node API calls) are all unified into the new style of PascalCase codes, as well as the ability to parse new error response fields such as errorKind, status etc.

If nodes return errorCode: 'storage_error', SDK returns errorCode: 'NodeStorageError'
If nodes return errorCode: 'NodeStorageError', SDK returns the same.

Testing
Run yarn build:packages, then yarn tools --dev --apps to spin up http://localhost:4002 which you can use for manual testing.
Manually tested using http://localhost:4002/manual_tests.html as an example, and the Encrypt, set the conditions, then try to update the conditions with an unauthorized account worked as expected:

@saike Thanks for the PR! Merged and published to latest (2.1.111)!


Update http4s-client, http4s-core, ... to 0.21.15

Updates

org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl

from 0.21.14 to 0.21.15.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Codecov Report

Merging #45 (17c38f4) into master (0e88fff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches        1        1
=======================================
  Hits           57       57
  Misses          1        1

Continue to review full report at Codecov.

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

Codecov Report

Merging #45 (17c38f4) into master (0e88fff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches        1        1
=======================================
  Hits           57       57
  Misses          1        1

Continue to review full report at Codecov.

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


Timers module not working, generates notification.

I am running Blish HUD v0.11.7 and recently Blish notified me of the release of version 0.3.18 of the Timers module. I entered into the Module Repo and updated to the newest version. Blish asked for a restart and I did so. The Timers module was disabled, so I enabled it and it generated the message, "New timers available. Go to settings to update!" In the repo menu, the little green light was on stating I am up to date with the plugin, and it listed the version as 0.3.18. I have tried downgrading, reenabling then updating to the new version with restarts of Blish HUD, etc. I've also tried deleting the module and reinstalling it. The timers don't work and it always generates that message.

@Dev-Zhao FYI

@CedarD @dlamkins When that message appears it means there is a new timers pack update, not a module update.

Ah! That's good to know, thanks! It might be nice to have some sort of indicator that the timers tab has an update, perhaps a little yellow up-arrow over the hourglass to draw our attention there. In any case, thanks again!

That's a good idea :)


Allow kv-file-cache to accept arrays of keys that are hashed

Given key = ['foo', 'bar', 'woz'], the hashed key would be <hash>_<hash>_<hash>.
The use-case here is to avoid collisions by add more granularity to the generated hash. Might end up being worthwhile just using sha1, but I suspect that murmur will function.

Fixed in a74c4b0d4edfee7f0ffd8b587d39b8c36aae0464


Sign out issue with anonymous user

If you sign in a user as anonymous it prevents you from creating an account even after calling a signout function, any way to prevent this?

I fixed the issue but I don't really know if its the best solution, if it seems fine then I'll close the ticket but was wondering if i could get a second opinion.
I changed the rpc function to work like this:
func rpc(function_name : String, arguments : Dictionary = {}, supabase_query : SupabaseQuery = null, anon : bool = false) -> DatabaseTask:
	if anon == false:
		_bearer = get_parent().auth._bearer
	if anon == true:
		var string = ["Authorization: Bearer " + str(get_parent().config.supabaseKey)]
		_bearer = string

where putting a bool arg defines whether or not youre making an anon call. For whatever reason this prevents the issue where you can't create an account after logging in as an anon user.


Instagram CORB

Hi, I'm trying to implement this library, but ran into problems with Instagram.
Instagram example also does not work on your Codepen, so I was wondering if there is a way to get around this:

Thanks!

@Pavel910 Thanks for reporting this issue. I've patched it in newest version. Please upgrade and test again.


add causal option for HiFiGAN

Implemented causal option for HiFiGAN

Oh sorry, the CI errors are not related to this PR.
I will fix it. Please ignore about it.

@chomeyama I fixed CI errors #327. Could you merge the current master?

I have implemented a test code for HiFiGAN and added some modifications to HiFiGAN Generator.

Cool, could you fix ci errors about black?

How do you find this change to hifigan output?


Added code to push image to GCR

We talked about this in chat, for posterity:
I'm not sure this is the right place for this. The Cloud Shell image isn't being deployed with Kubernetes, so it doesn't have to be listed with the other artifacts.
We could consider making a separate profile for building/pushing the cloud shell image on its own. But I'm not sure if there's much of an argument for doing that rather than just doing a regular docker build and docker push


Appveyor Builld SL 5

Enabled SL5 build

p:BuildSL5 missing from AV logs?

appveyor.yml is ignored, so this is not needed


Roll Flutter Engine from d1ebc5fde630 to 3600ec613a00 (1 revision)

https://github.com/flutter/engine/compare/d1ebc5fde630...3600ec613a00
2024-07-06<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from unavXf_l4lhu23MGg... to RSMFdOFyLxulsvY4_... (flutter/engine#53743)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from unavXf_l4lhu to RSMFdOFyLxul
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Tweak QPDFJob::handleUnderOverlay

A side question: under what circumstances are multiple QPDFAcroFormDocumentHelpers for a single QPDF object required?

Re programming style: given that some_map.count(some_key) == 0 is a work-around for the missing !some_map.contains(some_key), should the preferred style be !some_map.count(some_key) until qpdf requires C++20?

A side question: under what circumstances are multiple QPDFAcroFormDocumentHelpers for a single QPDF object required?

As far as I know, you would never want to do this, but it's a loaded question -- I need context to know why you're asking and what problem you're trying to solve.

Re programming style: given that some_map.count(some_key) == 0 is a work-around for the missing !some_map.contains(some_key), should the preferred style be !some_map.count(some_key) until qpdf requires C++20?

Personally, I think the idea of non-boolean values that are true or false is a language design bug, which is why I generally explicitly compare with 0.

I'm not going to look at this PR for the foreseeable future. QPDFJob setup, parsing, etc., is not performance-critical. I feel like this is probably premature optimization. If there is something not stylistic or related to performance in code that is performance-critical, let me know. I want to get functionality working before messing around with tweaks. It will be very disruptive to me to have code I am actively working on being changed while I'm still working in it.

Sorry for my last comment. I've got a lot going and may be a bit irritable. I do welcome the input. We should find a rhythm. Maybe I can leave PRs up for your commentary before merging, treating you more as an equal collaborator on API and efficiency concerns.

I'll try to give this a look before I dive into the next chunk. Keep in mind that a lot of this code is being reworked. I'm going to end up moving a lot of QPDFJob into QPDFAssembler by current design. If you'd like to be a more equal contributor in terms of API, efficiency, and general stylistic concerns, I will await your review on PRs. I think that rhythm will feel less disruptive to me than these after-the-fact PRs. I should have waited for your commentary but had been under the impression that you were not going to be available.

I'm not going to take this one. I'm going to significantly rework this logic in QPDFAssembler.

A side question: under what circumstances are multiple QPDFAcroFormDocumentHelpers for a single QPDF object required?

As far as I know, you would never want to do this, but it's a loaded question -- I need context

The immediate context is that I noticed that QPDFAcroFormDocumentHelpers are cached in various places in QPDFJob and I was wondering whether it would  make sense to have a single m->afdh_cache?
The wider context is that I seem to recall that the helper caches significant amounts of state and that my impression at the time was that if multiple helpers existed at the same time that there was a danger actions by one helper might cause the cached state of the other to become out of date. If my recollection/impression is correct, this would raise the question whether the state should be cached by the QPDF object and all helpers should share this single state.
If I recall correctly, I had the same question about the outline helpers.

I am not suggesting any involvement by QPDF other than to provide a member eg m->afdh_cache for the helper to store its (shared) state, with the helper responsible for creating and managing it. This would make the creation of a new helper instance cheap and eliminate the need for application code, including QPDFJob, to cache helpers to avoid the cost of creating them. This should reduce overall code complexity.
If I recall correctly, of the object helpers maintains a reference to its document helper with the obvious risks if it should outlive the document helper. The solution of storing the state with the QPDF object would remove this risk.

@jberkenbilt can you reopen for the discussion of document helpers.
I think this is an area that warrants further consideration for the following reasons:

It is generally good to avoid stale references to QPDF objects.

Some of the helpers are expensive to create and there is no simple way to avoid the unnecessary creation of them. While they can be cached, this is error prone. To take an example - QPDFJob creates a cache of QPDFAcroFormDocumentHelpers for handling Over/Underlays. It appears to me that separate caches are created for each call to doUnderOverlayForPage and therefore caching is a pure overhead rather than an optimization.

It is very difficult to feel certain that document helpers clean up their resources properly. I certainly had concerns about this when I looked at some of them in the past (without investigating further), and #1177 certainly does nothing to reassure me. Personally I would find it reassuring if QPDF objects tracked their document helpers and ensured that their resources get properly cleaned up on the death of the QPDF object.

@m-holger I don't like how document helpers (or object helpers) work. Are you suggesting I should reopen this PR? You can always create a discussion issue. I'm not sure about having QPDF keep track of them. It feels like leaking in the wrong direction of the abstraction. That said, I recognize your concerns about efficiency, etc. My original conception of these is that they would be short-lived objects, usually stack-allocated, that would just help to reduce boilerplate and reduce clutter of QPDFObjectHandle's interfaces. I think revisiting them may be useful in a broader context.


Move to ESM

Closes #103

bump

bump

@Qix- any chance of a review?  This would also support: https://github.com/Qix-/color/issues/256

#102

I will add TypeScript declarations as well.

This needs more changes to support tree shaking. I will handle this later.

@Qix- Do you have some ideas for the tree shaking?
I have two ideas, the first one is:
import {rgb, keyword} from 'color-convert';

rgb.hsl(0, 0, 0);
rgb.hex(0, 0, 0);
keyword.rgb(0, 0, 0);

Another idea is to make all convert functions to single one:
import {rgbToHsl, rgbToHex, keywordToRgb} from 'color-convert';

rgbToHsl(0, 0, 0);
rgbToHex(0, 0, 0);
keywordToRgb(0, 0, 0);

What do you think?

I feel like a refactor for tree-shakability might be appropriate to have as a separate PR?  Converting to ESM with the API "as-is" (this pr), and then updating for tree-shaking, potentially modifying the api in a breaking way, as a separate change.  Thoughts?

Both are OK for me. That's depends on @Qix-.

Either way, appreciate all the work you've put in.  Hopefully @Qix- gives it a look soon.

@Qix- any chance on feedback here?  This has been pending for almost a year

could you release a separate repository for the ESM version?


Don't let contributed keybindings override ours for showing keybinding in Quick Pick & Menu

1.24

Install GitLens

@octref I don't understand what this issue is about from the screenshots. Can you please explain?

GitLens is doing this keybinding contribution. I don't think it should be allowed.
            {
                "command": "workbench.view.scm",
                "key": "ctrl+shift+g g",
                "when": "gitlens:keymap == chorded && gitlens:enabled"
            }

/cc @eamodio

@octref That is the sole purpose of keybindings contributions: To allow for extensions to customize default keybindings. That is how keymap extensions can work. That is how vim can work.
You can remove this keybinding in your keybindings.json which can overrule both core keybindings and extensions keybindings. Alternatively, you can complain to the author of the extension or you can uninstall the extension.

@alexandrudima OK, I think I misunderstood. I originally thought keybinding contributions are only for adding keybindings to newly registered commands, not overriding existing ones.


Does not build with libusb 1.0.22

The build fails with libusb 1.0.22 because libusb_set_debug is deprecated:
/tmp/guix-build-libusb4java-0-1.396d642a5.drv-0/source/src/SsEndpointCompanionDescriptor.c
/tmp/guix-build-libusb4java-0-1.396d642a5.drv-0/source/src/LibUsb.c: In function ‘Java_org_usb4java_LibUsb_setDebug’:
/tmp/guix-build-libusb4java-0-1.396d642a5.drv-0/source/src/LibUsb.c:133:5: error: ‘libusb_set_debug’ is deprecated: Use libusb_set_option instead [-Werror=deprecated-declarations]
     libusb_set_debug(ctx, level);
     ^
In file included from /tmp/guix-build-libusb4java-0-1.396d642a5.drv-0/source/src/usb4java.h:11:0,
                 from /tmp/guix-build-libusb4java-0-1.396d642a5.drv-0/source/src/LibUsb.c:19:
/gnu/store/j8hzc465xy33cjfcq3wv39szxia4xrzv-libusb-1.0.22/include/libusb-1.0/libusb.h:1300:18: note: declared here
 void LIBUSB_CALL libusb_set_debug(libusb_context *ctx, int level);
                  ^

looks like different calls required depending on version of libusb being used:
libusb_set_debug(libusb_context *ctx, int level);
needs to change to
libusb_set_option(libusb_context, LIBUSB_OPTION_LOG_LEVEL, level);
wrapped in checks to
LIBUSB_API_VERSION and LIBUSBX_API_VERSION

This is fixed in usb4java 1.3.0 which now uses (and requires) libusb 1.0.22. The setDebug method now internally calls libusb_set_option so there is no more deprecation error on compile. But setDebug is now also marked as deprecated so you should use setOption() now.


nb warnings: simple-feature-transformation-exp nb tabular explainer

Describe the bug
simple-feature-transformation-exp nb during tabular explainer use
To Reproduce
Steps to reproduce the behavior:

run notebook
warning occurs during cell to explain model using tabularexplainer

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):
Edge
Additional context
Add any other context about the problem here.

fix was merged, closing issue


Rreme/gpu accelerate mssr

Here is a working gpu and cpu acceleration.
By default, the code runs on the first gpu (usually the faster). If no gpu, it will run on cpus.
To change the gpu it runs on, you can set the environment variable CUDA_VISIBLE_DEVICES.
Example with a system with 4 gpus ordered (0, 1, 2, 3)
$ napari  # The code will run on gpu 0
$ CUDA_VISIBLE_DEVICES=3 napari # The code will run on gpu 3
$ CUDA_VISIBLE_DEVICES=2,1 napari  # The code will run on gpu 2 (first visible device)
$ CUDA_VISIBLE_DEVICES=  napari  # The code will run on cpus (no gpu visible)

We don't want to let the user switch gpus at runtime because it is too costly to recompile the gpu code for the new gpu.
Note: The first call to mssr will be longer as the other ones, as we need to compile the functions (for cpus or gpus)

Also from numba documentation, this is the requirements to make it work on gpus

# Supported GPUs

Numba supports CUDA-enabled GPU with compute capability 2.0 or above with an up-to-data Nvidia driver.

# Software

You will need the CUDA toolkit version 8.0 or later installed. If you are using Conda, just type:

$ conda install cudatoolkit

If you are not using Conda or if you want to use a different version of CUDA toolkit, the following describe how Numba searches for a CUDA toolkit installation.


Allready connected vehicle to elli pro wallbox is not recognized on evcc startup with v0.116.5 (works with 0.116.4)

Describe the bug
The vehicle stays in status "not connected" on the loadpoint although it is already connected and ready to charge.
Also the connection state does not change in the web interface if I change the charge mode to minpv or now.
Workaround: Downgrade evcc to 0.116.4.
Steps to reproduce

Stop evcc
Install evcc version 0.116.5
Connect a vehicle to a Elli Pro Wallbox which is connected to energy manager evcc.
Optional: Authenticate on the Elli Wallbox and enable the charging process. Via App or RFID card.
Start evcc

Configuration details
# open evcc at http://evcc.local:7070
network:
  schema: http
  host: openhabian.local # .local suffix announces the hostname on MDNS
  port: 7070

log: debug
levels:
  cache: info
  eebus: trace
  lp-1: debug
    #lp-2: info
  renault: info
  zoe: info

interval: 10s # control cycle interval

chargers:
- type: template
  template: ellipro
  ski: D3A7 1EA0 5AFD 9511 E352 18B7 5D8A 775B 6F8C 9678 # ski of charger from web interface
  ip: <IP_ADDRESS>
  name: wallbox_garage # wallbox-2123A1ENSV
#- type: template
#  template: ellipro
#  ski: 99DF DA82 F7C8 894E 19EF 140B 54FF 6B8E 9B1C D0E1
#  ip: <IP_ADDRESS>
#  name: wallbox_stellplatz # wallbox-2123A1EKJG

vehicles:
- type: template
  template: renault
  title: Zoe
  user<EMAIL_ADDRESS>
  password: xxxxxxxx
  vin: VF1XXXXXXXXXXX
  capacity: 50 # kWh
  phases: 3 # uses a 3 wire charging cable
  cache: 15m
  mode: now # auto charging mode if this vehicle is connected to any charger
  minSoC: 40 # if under minSoC immediately charges to minSoC with mode 'now'. Ignores modes pv/minpv
  targetSoC: 80 # under targetSoc should charge with mode 'pv' up to targetSoC
  minCurrent: 8 # Zoe kann nur minimal 8A!
  maxCurrent: 16
  name: zoe

loadpoints:
- title: Garage
  charger: wallbox_garage
  vehicle: zoe # zoe = assigned default vehicle if any car connects
  #vehicles: # tries automatic identification of the vehicle via mac/rfid
  #  - zoe # does not work with eebus yet
  mode: minpv # off/pv/minpv/now
  phases: 3 # 0 = automatic (switch phases), 1|3
  #minCurrent: 6
  maxCurrent: 16
  resetOnDisconnect: true
  soc:
    poll:
      mode: charging # charging/connected
      interval: 60m
    estimate: true
#- title: Stellplatz
#  charger: wallbox_stellplatz
#  vehicle: # zoe = assigned default vehicle if any car connects
#  mode: pv
#  phases: 3
#  #mincurrent: 6
#  maxcurrent: 16
#  resetOnDisconnect: true
#  soc:
#    poll:
#      mode: charging # charging/connected
#      interval: 60m
#    estimate: true

meters:
#- name: wallbox_garage_meter
#  type: custom
#  power:
#    source: script
#    cmd: /bin/bash -c "evcc -c /home/openhabian/evcc.yaml charger wallbox_garage | grep 'Power' | sed 's/[^0-9]*\([0-9]\+\)W/\1/g'"
#    timeout: 10s
#- name: wallbox_stellplatz_meter
#  type: custom
#  power:
#    source: script
#    cmd: /bin/bash -c "evcc charger wallbox_stellplatz | grep 'Power' | sed 's/[^0-9]*\([0-9]\+\)W/\1/g'"
#    timeout: 10s
- name: dummyPV
  type: custom
  power:
    source: script
    cmd: /bin/bash -c "echo 0"
    timeout: 10s

site:
  title: Mein Zuhause
  meters:
    pvs:
    - dummyPV

eebus:
  interfaces:
    - enxb827ebfcf6fe
  certificate:
      private: |
          -----BEGIN EC PRIVATE KEY-----
          my very secret private key
          -----END EC PRIVATE KEY-----
      public: |
          -----BEGIN CERTIFICATE-----
          MIIBhzCCAS2gAwIBAgIBATAKBggqhkjOPQQDAjAzMQswCQYDVQQGEwJERTENMAsG
          A1UEChMERVZDQzEVMBMGA1UEAwwMRVZDQ19IRU1TXzAxMB4XDTIyMTAxMjE1NTA0
          M1oXDTMyMTAwOTE1NTA0M1owMzELMAkGA1UEBhMCREUxDTALBgNVBAoTBEVWQ0Mx
          FTATBgNVBAMMDEVWQ0NfSEVNU18wMTBZMBMGByqGSM49AgEGCCqGSM49AwEHA0IA
          BL9+vfj+Cnc88oQzcnUR7FwnVtjsKNiFrojnrN5+kvPayUN1FQ4R00QL0PIuprZp
          yx5Mo+HlOD6b9LHTj02Du0+jMjAwMA8GA1UdEwEB/wQFMAMBAf8wHQYDVR0OBBYE
          FDra1O1jilB+nJTItziOM+vHZTkbMAoGCCqGSM49BAMCA0gAMEUCIQDfSa+g1JGN
          ow/THIj8D08djmYuA+SD99VMUeWdbjsGWAIgX0yTa3SsF1QpiTFG/nqrF+PYOzsp
          xIUP4q4cNctTIro=
          -----END CERTIFICATE-----

tariffs:
  currency: EUR
  grid:
    # static grid price
    type: fixed
    price: 0.34 # EUR/kWh # ab 04/2023

  feedin: # Verguetung fuer eingespeisten Strom (nach EEG)
    type: fixed
    price: 0.0796 # EUR/kWh

Log details
See attached logfile:
evcc-2023-04-26_1325.log
What type of operating system are you running?
Linux
Version
evcc version 0.116.5

I think that this is not related to the Elli charger. I had the same situation today with my Easee Charger and the Skoda Enyaq. But to be honset, I did not investigate the full details yet.

Found the solution for Easee wallbox - see #764:
Replace the mobilnummer if you use it for the login with your email.
It is also working now for me again.

Found the solution for Easee wallbox - see #764:
Replace the mobilnummer if you use it for the login with your email.
It is also working now for me again.

Glad to hear that you found a solution for your problem. I think that this is related to the maintenance done by Easee recently.
The original problem described here is definitely not related to the Easee maintenance. I have no errors or warnings from the Easee charger in my log when I faced the issue.

I can confirm this is fixed with the current nightly build version.
Thanks @DerAndereAndi and @andig ! :thumbsup:


',",... may be more preferred then ‘’, “”, …

When I convert this to a pdf file, the quotation marks(‘’, “”) and ellipsis characters(…) become illegible.
',",... may be more preferred.

I solved this problem by adding a line "\usepackage[utf8]{inputenc}" in the tex file.


Adding overall basic doc

Some documentation, with a few clarifications for Aurora.

Will document dev as advised.


chore(docs): Update README.md to mention semantic versioning

@mehalter tagged v1.0.0 with these changes (and the generated doc/smart-splits.txt that came after them in #58). Feel free to update AstroNvim to pin to 1.x.x or however you want to notate it.

Going forward, changelogs will be more consistent now that we have a CI job to validate PR titles.

awesome this is perfect! Thank you so much!


Website stylesheet has larger font size for h3 then h2

Steps to Reproduce
Visit http://<IP_ADDRESS>:4000/debugging/
Notice how debugger() statement is a sub-topic under the Dart Observatory topic, yet it has a larger font size.

@sethladd this makes the new IntelliJ docs I am writing really hard to parse. Who knows the details of the stylesheet for the website?

Any of us. None of us. :) We should fix this. Patches welcome if you beat
us to it.
On Thu, Oct 20, 2016 at 07:33 Michael Thomsen<EMAIL_ADDRESS>wrote:

@sethladd https://github.com/sethladd this makes the new IntelliJ docs
I am writing really hard to parse. Who knows the details of the stylesheet
for the website?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/6416#issuecomment-255123114,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAVZ_X7Emw8B1yVse833Irs0nxy71lfks5q13uigaJpZM4Kb-cj
.

@qchong - I know you updated the style sheets a while ago - is this something you could help out with?

Thanks!


Problems with lines chart with all y-values being 0

The following example gets no y axis labels and not line.
(function basic(container) {
  Flotr.draw(container, [[[0, 0], [1, 0],[2, 0]]]);
})(document.getElementById("flotr-example-graph"));

Apparently, that is the case if all y values are the same and \leq 0. If I set them all to 3 or to 0.001, the graph is drawn as expected.
AFAICT, the culprit is either Flotr.getTickSize or Flotr.getMagnitude, the latter returning NaN or the former not handling NaN as a return value.

Ok this was a deep one, sorry it took me a bit.  I'll have a fix shortly.


Add me!

Name
Pauline Hans
Picture

Role:
Bachelor student
GitHub:
paulineHans
ORCID

Alumni

Phone

Email

Done! Will soon be live!


Datatable Enhancements

Pull request
Proposed changes
Resolves #183 with a separate table-view for the browse page.

update DataTables to to v2.1.8, including a few plugins:

Buttons 3.2.0, Column visibility 3.2.0, HTML5 export 3.2.0, FixedHeader 4.0.1, SearchBuilder 1.8.1

add SearchBuilder for advanced queries
add all metadata fields to the data table

by default, only a few are displayed for better appearance. Column visibility can be toggled using the corresponding button.

turn hyperlinks into clickable links

keep original markdown formatting for xslx/csv exports

separate multiple (longer) entries with horizontal line, e.g. sources or other references

subject column: tags are separated by comma and lead to the browse page

Types of changes

[x] New feature (non-breaking change which adds functionality).
[ ] Enhancement (non-breaking change which enhances functionality)
[ ] Bug Fix (non-breaking change which fixes an issue).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Summary by CodeRabbit

New Features

Enhanced multilingual support with new translations for search functionalities and table views.
Added buttons for switching to table view across various layouts.
Introduced a new markdown page for table display.

Bug Fixes

Corrected HTML syntax for GitHub link in data download modal.

Style

Updated styles for improved layout and visibility of data tables and search functionalities.
Enhanced responsiveness and visual consistency across themes.

Chores

Updated DataTables CSS to the latest version for improved styling and layout.

@maehr @koilebeit Do you have an opinion on which columns to display initially? At the moment, I have set title, date, description, subject, era, source, and rights to be visible initially. However, the description takes up a lot of space and other columns might be more useful.

Not really related but also not nice: The LINK is messed up ;)


Change access control and apply Stepdown Rule

No change in behavior.

Stepdown Rule is a good idea, bearing there are multiple schools for that:

those who sort by public -> internal -> private (or the opposite)
those who sort by overriden properties -> properties -> overriden methods -> methods
those who group by protocol conformance

Anyway, the only side-effect this pull request may have is in case of someone subclassing ToastWindow (because some methods are now private). Let me test it on a project where I did subclass, and I'll approve it if it's still working.

Thank you, I will wait for your review. Maybe it will okay, because I only changed internal -> private.

Well, no special usage on my end, so I'll merge it.