Revision 120 breaks PwmOut on FRDM-K64F

(Copied as requested from https://developer.mbed.org/users/mbed_official/code/mbed/issues/36 )
With revision 120 of mbed, PwmOut on a FRDM-K64F appears to be broken i.e no output is seen.
A demonstration program is at https://developer.mbed.org/users/infinnovation/code/pwm_regression/
On FRDM-K64F, connect LED+330R between PTD0 (J2.6) and GND (J2.3)
With revision 119 of mbed, LED is lit as expected.
With revision 120 of mbed, the LED is not lit.

cc @mmahadevan108

Thanks, I will take a look and update.

So, I added some code to examine the FTM registers and found several differences:

FTM3_
v119
v120
Comments

SC
8e
8e

CNT
2958
2e2

MOD
493d
493d

C0SC
a8
a0
ELSnB:ELSnA=0 => revert pin to GPIO etc.

C0V
15f8
15f8

C1SC
0
0

C1V
0
0

C2SC
0
0

C2V
0
0

C3SC
0
0

C3V
0
0

C4SC
0
0

C4V
0
0

C5SC
0
0

C5V
0
0

C6SC
0
0

C6V
0
0

C7SC
0
0

C7V
0
0

CNTIN
0
0

STATUS
1
1

MODE
4
5
FTMEN=1: enable FTM-specific registers

SYNC
0
8
SYNCHOM=1: outmask updated by PWM synch

OUTINIT
0
0

OUTMASK
0
0

COMBINE
0
20202020
SYNCENn=1

DEADTIME
0
40
DTPS=1: (bit ignored if higher bit clear)

EXTTRIG
0
0

POL
0
0

FMS
0
0

FILTER
0
0

FLTCTRL
0
0

QDCTRL
0
0

CONF
3
3

FLTPOL
0
0

SYNCONF
0
1fb4
SWSOC,SWINVC,SWWRBUF,SWRSTCNT,SYNCMODE,SWOC,INVC,CNTINC

INVCTRL
0
0

SWOCTRL
0
0

PWMLOAD
0
0

The killer seems to be C0SC where both ELSnA and ELSnB are zero "Pin not used for FTM--revert the channel pin to general purpose I/O or other peripheral control".
I poked 0xa8 into 0x400b900c and, sure enough, my LED came on.

At patch with updated FTM, TPM drivers have been submitted to ARM.

OK, thanks.

I can confirm that this issue is fixed in revision 121.


fix: includes bought domains in the query

After adding the sold status to offers (btw, why not just delete the record?) the query was not including those records for the users that bought the domain.
Adding to the pile, this extends the raw sql query to also include the 'SOLD' offers in the owned domains query.
Disclaimer: Although the query will have to be cleaned up, rather sooner than later, at the moment the priority lays with getting it work. Refactor will come later.

This is a bit question for @leandroyalet but I am also not sure why we need to have the SOLD state? Or actually any state? Shouldn't it be the case for DomainOffer table to have only current offers? Why should we have CANCELED and SOLD as well?


Add docs for StringToDoubleConverter and StringToIntConverter

Fixes

Fixes #22

Docs for Toolkit PR #447
What changes to the docs does this PR provide?
Adds documentation for:

StringToDoubleConverter
StringToIntConverter

PR Checklist
Please check if your PR fulfills the following requirements:

[X] For new pages, used the provided template.
[X] For new features, added an entry in the Table of Contents.
[X] Ran against a spell and grammar checker.
[X] Contains NO breaking changes.

Docs Build status updates of commit 4ce2b8b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/converters/stringtodoubleconverter.md
:white_check_mark:Succeeded
View

docs/converters/stringtointconverter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

These converters were removed as they didn't add that much value. Sorry :(


ERROR:mem0.memory.graph_memory:Error in search tool: 'entity_type'

🐛 Describe the bug
When replacing llm with locally deployed mixtral: 8x22b and executing m.add ("I like pizza", user_id="alice123"), an error is reported: ERROR: mm0. memory. graph. memory: Error in search tool: 'entity type'
TypeError                                 Traceback (most recent call last)
Cell In[22], line 1
----> 1 m.add("I like pizza", user_id="alice123")
File ~/opt/anaconda3/envs/mem0/lib/python3.10/site-packages/mem0/memory/main.py:120, in Memory.add(self, messages, user_id, agent_id, run_id, metadata, filters, prompt)
117     concurrent.futures.wait([future1, future2])
119     vector_store_result = future1.result()
--> 120     graph_result = future2.result()
122 if self.api_version == "v1.1":
123     return {
124         "results": vector_store_result,
125         "relations": graph_result,
126     }
File ~/opt/anaconda3/envs/mem0/lib/python3.10/concurrent/futures/_base.py:451, in Future.result(self, timeout)
449     raise CancelledError()
450 elif self._state == FINISHED:
--> 451     return self.__get_result()
453 self._condition.wait(timeout)
455 if self._state in [CANCELLED, CANCELLED_AND_NOTIFIED]:
File ~/opt/anaconda3/envs/mem0/lib/python3.10/concurrent/futures/_base.py:403, in Future.__get_result(self)
401 if self._exception:
402     try:
--> 403         raise self._exception
404     finally:
405         # Break a reference cycle with the exception in self._exception
406         self = None
File ~/opt/anaconda3/envs/mem0/lib/python3.10/concurrent/futures/thread.py:58, in _WorkItem.run(self)
55     return
57 try:
---> 58     result = self.fn(*self.args, **self.kwargs)
59 except BaseException as exc:
60     self.future.set_exception(exc)
File ~/opt/anaconda3/envs/mem0/lib/python3.10/site-packages/mem0/memory/main.py:252, in Memory._add_to_graph(self, messages, filters)
250         self.graph.user_id = "USER"
251     data = "\n".join([msg["content"] for msg in messages if "content" in msg and msg["role"] != "system"])
--> 252     added_entities = self.graph.add(data, filters)
254 return added_entities
File ~/opt/anaconda3/envs/mem0/lib/python3.10/site-packages/mem0/memory/graph_memory.py:69, in MemoryGraph.add(self, data, filters)
66 extracted_relations = self._extract_relations(data, filters, entity_type_map)
68 search_output_string = format_entities(search_output)
---> 69 extracted_relations_string = format_entities(extracted_relations)
70 update_memory_prompt = get_update_memory_messages(search_output_string, extracted_relations_string)
72 _tools = [UPDATE_MEMORY_TOOL_GRAPH, ADD_MEMORY_TOOL_GRAPH, NOOP_TOOL]
File ~/opt/anaconda3/envs/mem0/lib/python3.10/site-packages/mem0/memory/utils.py:27, in format_entities(entities)
25 formatted_lines = []
26 for entity in entities:
---> 27     simplified = f"{entity['source']} -- {entity['relation'].upper()} -- {entity['destination']}"
28     formatted_lines.append(simplified)
30 return "\n".join(formatted_lines)
TypeError: string indices must be integers

I can see that you're encountering an issue when trying to use Mixtral 8x22B with mem0's graph memory component. The error occurs because the graph memory component expects a specific structured output from the LLM that Mixtral isn't providing correctly.
From the codebase, I can see that the format_entities() function expects each entity to be a dictionary with 'source', 'relation', and 'destination' keys, but it seems Mixtral is returning strings instead.
Here's what's happening:

The graph memory component uses LLMs to extract entities and relationships from text
It expects the LLM to return structured data in a specific format
The Mixtral model appears to be returning unstructured string output instead of the required JSON format

To fix this, you have two options:

Use a supported structured model:

config = {
    "llm": {
        "provider": "openai_structured",  # or "azure_openai_structured"
        "config": {
            "model": "gpt-4o-mini",  # or another supported structured model
            "temperature": 0.1,
            "max_tokens": 2000
        }
    },
    "graph_store": {
        "provider": "neo4j",
        "config": {
            "url": "your_neo4j_url",
            "username": "neo4j",
            "password": "your_password"
        }
    },
    "version": "v1.1"
}

If you must use Mixtral, you'll need to create a custom wrapper that formats its output to match the expected structure:

class MixtralWrapper:
    def generate_response(self, messages, tools):
        # Add formatting logic here to convert Mixtral's output
        # to the expected structure with source/relation/destination
        pass

References:

From the codebase (mem0/memory/graph_memory.py): The graph memory component expects structured output with specific tools and formats
From issue #1822: There was a similar issue with non-structured models, confirming this is a known limitation

Important notes:

The graph memory feature requires structured output from the LLM
Not all models support the required structured output format
Using unsupported models may lead to format compatibility issues

Let me know if you'd like to try either of these solutions or if you need help implementing a custom wrapper for Mixtral.

mixtral:8x22b

Thank you for your response. If possible, we hope to provide support for mixtral: 8x22b and qwen2.5:72b.

我发现您在尝试将 Mixtral 8x22B 与 mem0 的图形内存组件一起使用时遇到了问题。出现此错误的原因是图形内存组件需要来自 LLM 的特定格式输出，而 Mixtral 无法提供正确的输出。
从代码库中，我可以看到format_entities()函数希望每个实体都是一个带有“源”、“关系”和“目标”键的字典，但 Mixtral 似乎返回的是字符串。
假期具体情况：

图形内存组件使用LLM从数据中提取实体和关系
它希望LLM返回特定格式的数据构造
Mixtral模型似乎返回非重构字符串输出，而不是所需的JSON格式

要解决这个问题，您有两个选择：

使用支持的构造模型：

config = {
    "llm": {
        "provider": "openai_structured",  # or "azure_openai_structured"
        "config": {
            "model": "gpt-4o-mini",  # or another supported structured model
            "temperature": 0.1,
            "max_tokens": 2000
        }
    },
    "graph_store": {
        "provider": "neo4j",
        "config": {
            "url": "your_neo4j_url",
            "username": "neo4j",
            "password": "your_password"
        }
    },
    "version": "v1.1"
}

如果必须使用 Mixtral，则需要创建一个自定义包装器，格式化其输出以符合预期的结构：

class MixtralWrapper:
    def generate_response(self, messages, tools):
        # Add formatting logic here to convert Mixtral's output
        # to the expected structure with source/relation/destination
        pass

参考：

来自代码库（mem0/memory/graph_memory.py）：图形内存组件需要使用特定的工具和格式的格式化输出
来自问题[graph_memory]：当前工具不支持非成型模型。 #1822：非成型模型也存在类似问题，确认这是一个已知的限制

重要说明：

图形内存功能需要 LLM 的重构输出
并非所有模型都支持所需的重构输出格式
使用不受支持的型号可能会导致格式兼容性问题

如果您想尝试这些解决方案中的任何一个，或者您需要帮助 Mixtral 实现自定义包装器，请告诉我。
Thank you for your response. If possible, we hope to provide support for mixtral: 8x22b and qwen2.5:72b.

I can see that you're encountering an issue when trying to use Mixtral 8x22B with mem0's graph memory component. The error occurs because the graph memory component expects a specific structured output from the LLM that Mixtral isn't providing correctly.
From the codebase, I can see that the format_entities() function expects each entity to be a dictionary with 'source', 'relation', and 'destination' keys, but it seems Mixtral is returning strings instead.
Here's what's happening:

The graph memory component uses LLMs to extract entities and relationships from text
It expects the LLM to return structured data in a specific format
The Mixtral model appears to be returning unstructured string output instead of the required JSON format

To fix this, you have two options:

Use a supported structured model:

config = {
    "llm": {
        "provider": "openai_structured",  # or "azure_openai_structured"
        "config": {
            "model": "gpt-4o-mini",  # or another supported structured model
            "temperature": 0.1,
            "max_tokens": 2000
        }
    },
    "graph_store": {
        "provider": "neo4j",
        "config": {
            "url": "your_neo4j_url",
            "username": "neo4j",
            "password": "your_password"
        }
    },
    "version": "v1.1"
}

If you must use Mixtral, you'll need to create a custom wrapper that formats its output to match the expected structure:

class MixtralWrapper:
    def generate_response(self, messages, tools):
        # Add formatting logic here to convert Mixtral's output
        # to the expected structure with source/relation/destination
        pass

References:

From the codebase (mem0/memory/graph_memory.py): The graph memory component expects structured output with specific tools and formats
From issue [graph_memory]: The current tools does not support non-structured models. #1822: There was a similar issue with non-structured models, confirming this is a known limitation

Important notes:

The graph memory feature requires structured output from the LLM
Not all models support the required structured output format
Using unsupported models may lead to format compatibility issues

Let me know if you'd like to try either of these solutions or if you need help implementing a custom wrapper for Mixtral.

Thank you for your response. If possible, we hope to provide support for mixtral: 8x22b and qwen2.5:72b.


error when run demo

Testing gpu  [0, 1, 2, 3, 4, 5, 6]
Network [RotateSPADEGenerator] was created. Total number of parameters: 225.1 million. To see the architecture, do print(network).
start prefetching data...
Process Process-1:
Traceback (most recent call last):
File "/home/yangchuqiao/.conda/envs/render/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/home/yangchuqiao/.conda/envs/render/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/home/yangchuqiao/code/github/Rotate-and-Render/data/data_utils.py", line 146, in prefetch_data
prefetcher = data_prefetcher(dataloader, opt, render_layer)
File "/home/yangchuqiao/code/github/Rotate-and-Render/data/data_utils.py", line 99, in init
self.preload()
File "/home/yangchuqiao/code/github/Rotate-and-Render/data/data_utils.py", line 124, in preload
self.next_input = get_multipose_test_input(data, self.render_layer, self.opt.yaw_poses, self.opt.pitch_poses)
File "/home/yangchuqiao/code/github/Rotate-and-Render/data/data_utils.py", line 65, in get_multipose_test_input
= render.rotate_render(data['param_path'], real_image, data['M'], yaw_pose=pose)
File "/home/yangchuqiao/code/github/Rotate-and-Render/models/networks/rotate_render.py", line 100, in rotate_render
tex_a_crop = self.get_render_from_vertices(rendered_images_erode[n], vertices_in_ori_img[n])
File "/home/yangchuqiao/code/github/Rotate-and-Render/models/networks/render.py", line 340, in get_render_from_vertices
textures = img_ori[vertices_in_ori_img[1, :].round().clamp(0, h - 1).long(),
RuntimeError: indices should be either on cpu or on the same device as the indexed tensor (cpu)

I ran into the same problem, did you solve it?

I solved it by adding this before line 340:
vertices_in_ori_img = vertices_in_ori_img.to(device="cpu")


[MDETR] Add tutorial notebook

Stack from ghstack (oldest at bottom):

-> #246
#232
#231
#230

Summary:
Add tutorial notebook for MDETR in TorchMultimodal. The notebook
describes MDETR, shows a couple ways to build the model in TorchMultimodal,
then shows how to build task-specific MDETR models to perform inference on
phrase grounding and VQA tasks on a couple samples from Flickr30k and GQA,
respectively.
Test plan:
N/A

Codecov Report

Merging #246 (e587676) into gh/ebsmothers/22/base (2332785) will not change coverage.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##           gh/ebsmothers/22/base     #246   +/-   ##
======================================================
  Coverage                  91.66%   91.66%
======================================================
  Files                         48       48
  Lines                       2868     2868
======================================================
  Hits                        2629     2629
  Misses                       239      239

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@ebsmothers has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


Add Colegio Nuestra Señora del Rosario Las Condes

@matias-damian First stage of verification passed, official domain for COLEGIO NUESTRA SEÑORA DEL ROSARIO LAS CONDES is colegionsr.cl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Thank you very much.

@Matias-Damian Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


LeNet performance request

Firstly, thank you for a comprehensive evaluation of all these models on CIFAR using PyTorch.
I noticed you have a LeNet network defined in your models, which is more-or-less identical to the one used in the PyTorch CIFAR example. Do you have the performance of this model on CIFAR-10?
It is a simple baseline, and I want to make sure my numbers are matching (getting only 65% test accuracy, which seems a bit suspicious).

I didn't train LeNet-5 on CIFAR10, I think 65% is reasonable. LeNet-5 is too simple, what you got is perfectly fine to me.
And VGG is a good model to start with.


About the model implementation with Minkowski Engine

Hi Sir,
torchsparse does not support GPU-based hash table, but I need to speed up the network I'm working on. May I ask is there any implementation to evaluate the pre-trained SPVNAS with the Minkowski Engine v0.5? Or maybe anyway to turn the pre-trained SPVNAS in Minkowski Engine v0.5?
Thanks for your novel work a lot.

TorchSparse is actually using the GPU-based hash table, and it provides a faster inference speed than MinkowskiEngine. I'm not sure what the motivation is behind switching the backend.

@dante0shy If you need to use the Minkowski backend, you will have to build it out. But as Zhijian said, torchsparse backend already supports GPU-based hash table (it was one of the novel contributions in the paper), and I think you won't get any meaningful performance gain changing backends (minkowski and torchsparse are already very optimized).


Moved and edited pluginsInfo

I moved the pluginsInfo.py to one directory above and resolved all conflicts with it. I attempted to avoid the workspace.xml but... issues occurred to say the least. All is fixed and the pluginsInfo will work no matter how many plugins you shove in that folder Issue #34

@Macr0nmNerd tell me you use pycharm?

@Abdur-rahmaanJ I do indeed


Fix --scope argument in cibuild.cmd

The variable value wasn't being passed to build.cmd
Also add possible value of "build" and make string comparison case insensitive

[belated 👍]

Sorry the longer I waited the more I realized I probably didn't need to get approval for it...


Add contributing file

Adds a very rough invitation to contribute (to be expanded later)
Adds a DCO requirement with instructions

I've also submitted a request to add the DCO app, which @greg-szabo approved. Thanks to @romac for pointing it out.

I initially opened this PR without a signed commit, and the DCO app failed it, the I reworded the commit to include it and we're passing :+1:

Fixed (after some git foundering :). Thanks for the corrections!


Synchronous module blocks

First this sounds like a great proposal, hope it makes it through the approval process!
In the readme you mention

Importing a module block needs to be async, as module blocks may import other modules, which are fetched from the network.

Module blocks are only imported through dynamic import(), and not through import statements, as there is no way to address them as a specifier string.

I think it's a shame to only support dynamic imports because it's forcing everything to be async even in cases where it didn't need to be.
For example a feature that would be nice for bundlers to provide is the ability to get a reference to a module as a module block.
E.g with something like
import somethingModuleBlock from 'module-block:./something.js';

you could get back somethingModuleBlock as a module block for the ./something.js module.
If module blocks could also be run synchronously then internally, the bundler could wrap ./something.js in a module block and it could make it work for normal static imports, and module-block: imports.
If module blocks can only be async, the above would not be possible if anything else in the project wanted to import ./something.js statically.
I can see how a static import of something needed from the network in a module block makes this complicated. Maybe in order for a module block to be imported statically all of its imports must be immediately available?
Am I missing something? Maybe this is already possible.
refs https://github.com/rollup/rollup/issues/3915

Actually I think there would be more problems then the synchronous thing. Module blocks not being able to import other module blocks would be an issue. Looks like the current proposal doesn't really help prevent code duplication for code that could run in or out of a worker? Maybe this proposal wouldn't help solve something like https://github.com/rollup/rollup/issues/3915

Actually I think there would be more problems then the synchronous thing. Module blocks not being able to import other module blocks would be an issue. Looks like the current proposal doesn't really help prevent code duplication for code that could run in or out of a worker? Maybe this proposal wouldn't help solve something like https://github.com/rollup/rollup/issues/3915

As stated in the README, this proposal is explicitly not trying to solve the bundling issue. @littledan has an orthogonal proposal (link in the README) targeting that problem.

As stated in the README, this proposal is explicitly not trying to solve the bundling issue. @littledan has an orthogonal proposal (link in the README) targeting that problem.

:+1: will have a look at that thanks

:+1: will have a look at that thanks


Break GitHub PoC into API for our view w/ Impl

API, Impl, REST service exposing them and receiving callbacks

1dee0f8abbc4ab1cc64f645cec7a67688bff8062


[4.2] [stdlib] Add last(where) and lastIndex(of/where)

This is the implementation of SE-0204, cherry-picked from #16089 and #13337.

@swift-ci Please test


Do not ignore git push errors

Do not ignore git push errors
Also return readible error in case we cannot push to the manifest repo.
2 error cases are handled in this PR:

the ssh key does only have read only access. This returns a normal error (at least on github)
there is a branch protection rule stopping access. This is not a normal error, but signaled via the "PushUpdateReferenceCallback"

In both cases we return the error to the caller instead of "internal error".
In order to test these 2 error cases, I had to introduce 2 Callbacks for ProcessQueueOnce

I think you could construct a very realistic test-case by adding running the cd-service against a bare repo with an update hook.

I think you could construct a very realistic test-case by adding running the cd-service against a bare repo with an update hook.

I tested this manually with a real github repo.


use a custom PAT for deploy & PR creation now

The latest changes of this PR are not available as canary, since there are no linked changesets for this PR.


I need help in deploying the bot

I tried deploying using render but for some reason its failing. (The build is successfull)

@Hemanth-2002 Show me more information, maybe I can help you.

For help, please use Discord API Discussions or Discord Project Post


umap demo notebook

Comparison between spark-rapids-ml umap and local cpu umap,
Currently, runtimes leave a lot to be desired; working on mitigating overheads to improve this. Additionally, the demo test dataset is pretty small (mnist, ~0.4 gb - good for visualization purposes). With a larger dataset, the efficiency of the spark implementation will be better demonstrated.

build

build

build

build


Latent-Refusion v.s. Refusion

Hi,
I've only seen the code for refusion but no code for latent-refusion. Are there the codes for latent-refusion and the latent-refusion pretrained model in this package?
Also, I have no idea about the model architecture of latent-refusion. Is latent-refusion just pretrained autoencoder + NAFNet?
On the other hand, I mainly want to focus on deblurring application, but there is only ir-sde pretrained model and no refusion pretrained model for deblurring in this package.
Thanks.

Hi, the latent-refusion code is the latent-unet. If you want to perform latent-refusion on the deblurring task, you need to pretrain the latent-unet (U-Net autoencoder) on your dataset. Please refer to the latent-dehazing for more details.

Are both the training flow and inference flow of the latent-unet for deblurring application the same as latent-unet for dehazing?
That is to say for deblurring, do we need to compute latent_lq, latent_gt, hidden_lq, and hidden_gt?
But here is a question, when performing inference without ground truth (since actual deblur has no ground truth), how should I perform latent-unet testing?
Thanks.

In inference, we don't need the latent_gt and hidden_gt, you can just set them to None.


[Snyk] Security upgrade aws-amplify from 5.3.19 to 6.0.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

services/ui-src/package.json
services/ui-src/yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-COOKIE-8163060
  44

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

addressed by https://github.com/Enterprise-CMCS/macpro-mdct-mfp/pull/791


Feature: Add a new container Logs API

The need of this feature is to obtain docker logs from the container, collect and ship it using this new API  to the Rancher UI so that the user can always monitor it through the console.

Requirements
Add a new action (API) for container for fetching its logs
Functionality
The result of this new action(API) should return a url to a websocket on the host and also a jwt token. The websocket should connect to the host-api go process which runs on the host.
Tentative Design
The implementation of this new API would probably be based on how the stats action and the exec action on instance have been implemented in Cattle.
Currently, the host-api go process creates a secure websocket that proxies information.  Cattle currently uses it to grab information from "cadvisor".  Here in this new action API, I plan to call docker with appropriate input parameters required for fetching the logs of a container.

You should probably create a /rancherio/rancher feature request as subtasks for this will most likely result in development in both the cattle and ui projects. A couple of questions:

What can the user actually do?
Can user tail the logs?
Can user search for the logs?
Can users download the logs directly from the web interface?
Can users search through the logs?

Done: https://github.com/rancherio/cattle/pull/210


feat: track time to stable mesh for aggregator duties

Motivation

Since v1.10.0, we only subscribe 2 slots in advance of aggregator duty, we want to track more metrics for it

Description

Track time to stable mesh

We need to monitor on 2 slots window, so I check the metric every half slot

Track short lived (committee) mesh peers on scrape

part of #5864

for goerli test, it shows committee subscribed subnets that are 100% unhealthy. For mainnet we have more mesh peers there so should be better

Time to stable mesh in seconds is <=6s most of the time

:tada: This PR is included in v1.11.0 :tada:


ARROW-10449 [Rust] Make Dictionary::keys be an array

This PR:

Makes DictionaryArray::keys be an PrimitiveArray.
Removes NullIter and many of the unsafe code that it contained
Simplifies the parquet writer implementation around dictionaries
Indirectly removes a bug on NullIter's DoubleEndedIterator on which the implementation was not following the spec with respect to end == current (compare with implementation of DoubleEndedIterator for std's Vec's IntoIter).
Fixes error in computing the size of a dictionary, which was double-counting data and values (values is a reference to data)
Adds check that the dictionary's ArrayData's datatype matches T::DATA_TYPE.

Since NullIter was not being directly tested, no tests were removed.
This is backward incompatible and the migration requires replacing dict.keys()  by dict.keys().into_iter() whenever the user's intention is to use keys() as an iterator.

@vertexclique , I remember that you were using DoubleEndedIterator implementation of the NullIter. This PR removes that possibility, but I fielded #8562 to re-add it to the PrimitiveArrayIter, so that that functionality is recovered and supported more generally.

LGTM, this should also fix ARROW-10298. I believe at Signavio we also use the keys array instead of the DoubleEndedIterator now, so removing that should not be a blocker for merging.


-g, I do not know which format the url is for

When I use the -g to get the temp link to a source file. I need the format as well. If you could add an option that outputs the format that that the link is, especially after specifying a preference format list. So I know which format it grabs else it is very hard for me to use the URL.

It sounds like you are calling youtube-dl from another program. If that's the case, use the -j option to get information about the video as JSON, which you can easily parse in any programming language.


Gappless separators, no padding

I tried to remove separators but I still get two space characters inserted between tokens, and also there are padding spaces on left and right token sides. Lot of precious status line real estate gets wasted.
Would be great to be able to decide myself when to use space separators and / or padding:
let g:lightline.active.left = { [  'bufnum' , ' ', 'filename'  ] }

I don't understand what you are saying... you want to insert? or remove? spaces... The right hand side of the screenshots are not preferable for you?

I want to remove spaces, the right side of the screenshot is the desired outlook.

You can specify the component_type to raw
  'component_type': { 'bufnum': 'raw', 'filename': 'raw' }

But I found a bug with the logic, so pull the repository.

It works somehow strange. If I set empty separators, the gap disappears:

Then I define my own separator and set it to raw:
let g:lightline.component = { 'sep': ' '}
let g:lightline.component_type = { 'bufnum': 'raw', 'filename' : 'raw', 'sep' : 'raw' }
let g:lightline.active = { 'left': [ ['bufnum' , 'sep', 'filename'] ] }

but I get 3 spaces instead:

How about adding a space in the bufnum component? (g:lightline.component.bufnum = '%n ')

How about adding a space in the bufnum component?

That works.
Also, I managed to get my separators working:
let g:lightline.component = { 'sep': '%{" "}'}


Coedit patch info schemas

Adds schemas for the patchInfo property in document-state.schema.yaml. Although patchInfo is effectively hidden from ordinary users, only appearing in Cloud Documents behind-the-scenes I felt it makes sense to fully document the data structure in the schemas - this should make the schemas maximally useful in future, in case we are able to integrate them more tightly with Sketch.
Referenced issues
Fixes #94

Looks good to me, but maybe worth /CCing @mkeiser as well.

Looks good to me, but maybe worth /CCing @mkeiser as well.

Looks good. I hope we do not show this to users/plugin devs anywhere though.

@mkeiser Can you clarify your concerns? The file format is a publicly visible specification, and used both internally at Sketch and by 3rd parties seeking to parse and generate their own Sketch documents. As I understand it the patchInfo stuff never appears in Sketch documents opened by users though, so we're specifying it here just for the sake of completeness with one eye on the future for when/if the spec is able to play a more important role in the Sketch code base itself. We've noted in the schema descriptions that patchInfo is more an implementation detail, and not relevant to 3rd parties working with the rest of the document model. Is that enough do you think?

From before:

It's worth just re-iterating that this information should only be found in cloud documents which should be hidden from the user and that we certainly don't want anyone trying to edit any of this information.

FWIW - I agree with @mkeiser and I'm still not 100% sure why we want to document this. I'm worried that doing so could encourage 3rd parties to "have a go at something" which could cause all sorts of problems for us.

Is that enough do you think

Hopefully. Maybe we should tell them they should set the whole patchInfo object (schema/objects/patch-info.schema.yaml) to nil when they modify/create a document?

Converted to draft while we wait for @luisbosque to join the conversation.


Check for packager dependencies in packager by trying to load them and catching the error

npm 3.0.0 changes the file structure for dependencies to try to avoid nesting them. This leads to false positives when checking for a node_modules directory within react-native because it's dependencies may be stored in the parent project's node_modules directory.

I am excited about npm 3 and the potential performance improvements from deduping.. @evansolomon  thanks for taking the initiative!

Can you link to something that describes the changes in npm 3?
We might need to update the resolver too.

The "flat, flat, flat" section here is probably the best description.

Cool. My only concern with this PR is for development, if there is an error in one of the modules (be it an initialization runtime error or a syntax error) then it'll be hard to debug. I left an inline comment on how to avoid this.

thanks, one last nit

Also, please rebase.

Rebased and squashed the feedback commits

Added one more commit to include the remaining local module in the try/catch. It uses NPM modules, so I think it would use the same error message. It's also not actually used at the moment, but that seems to be temporary.

Rebased again to fix a merge conflict from 1461e4a

Any remaining issues with this change? I'd love to upgrade to the latest react native, but managing a fork with this patch gets tedious. Let me know if there's anything else you'd like done here.

This helped fixed my "Unable to resolve module..." error in React Native 0.8.0. Thanks for this!

@evansolomon @amasad Any reasons this isn't merged yet? Packager doesn't work on npm 3 now (

Any updates on this?

Per #5391 this seems to not be a problem anymore


Ensure Dynamodb point in time recovery (backup) is enabled

Possible check to target the following resource

[ ] aws_dynamodb_table

closing, tracking in spreadsheet

closing, tracking in spreadsheet


type 'Null' is not a subtype of type 'int' in type cast

environment:
sdk: '>=2.16.0'
dependencies:
odoo_rpc: ^0.4.5
code:
main() async {   final client = OdooClient('http://<IP_ADDRESS>:8069');   try {     await client.authenticate('odoo17uat', 'admin', 'admin');     print(client);     // final res = await client.callRPC('/web/session/modules', 'call', {});     // print('Installed modules: \n' + res.toString());   } on OdooException catch (e) {     print(e);     client.close();     exit(-1);   }   client.close(); }
err:
Unhandled exception:
type 'Null' is not a subtype of type 'int' in type cast
#0      OdooSession.fromSessionInfo (package:odoo_rpc/src/odoo_session.dart:65:37)
#1      OdooClient.authenticate (package:odoo_rpc/src/odoo_client.dart:262:32)

I traced the code, it successfully login, but this sentense throw err:
_sessionId = OdooSession.fromSessionInfo(result['result']);

I had a similar error after successful authentication, in my case it has to do with the company_id column of the res.partner model that is not returned after authentication. For now I simply removed the field from this file that is specific to the package waiting for the problem to be resolved.

Thanks for reporting!
Please upgrade to v 0.5.4


recalculating strand orientaiton position based on region zoom

Recalculate gene-viz orientation when the region is updated with zoom.  For orientation center calculation, filter out all regions not contained within the zoom region.  If no regions are visible, orientation is in the middle of the region.  If only 1 region is visible, orientation is in the center of the largest empty region.

quarantest build successful. Demo link at
http://a4ca4001.quarantest.iobio.io

quarantest build successful. Demo link at
http://78461ca3.quarantest.iobio.io

quarantest build successful. Demo link at
http://bf0cc3dd.quarantest.iobio.io

quarantest build successful. Demo link at
http://915e0ed0.quarantest.iobio.io

quarantest build successful. Demo link at
http://ff0c32e4.quarantest.iobio.io


fix: --org-name at least 2 parts. Why ?

Description
Using the --org-name flag needs 2 parts.
very_good create super --org-name=app is a invalid command.
very_good create super --org-name=app.super is a valid command and the package name is app.super.super
The very_good_cli concatenate org and project name.
Why only 1 part on --org-name flag is not allowed ?
https://github.com/VeryGoodOpenSource/very_good_cli/blob/main/lib/src/commands/create/create.dart#L32

You can use --application-idfor the single part option. See #280 for more context


Added Ignore to Negative_digit test in AllYourBaseTest

One test (in addition to the first test) in AllYourBaseTest didn't have an Ignore annotation

Good eye @FridaTveit! I'll merge this PR once the Travis CI finishes.

Merging, thanks for the PR @FridaTveit!

Thanks @FridaTveit!


Error in documentation

The documentation states that the settings of an edge must be passed in an object as the 3rd parameter of the addEdge function. See the example below extracted from the website:
/* a directed connection, using an arrow */
g.addEdge("id34", "cherry", { directed : true } );

/* customize the colors of that edge */
g.addEdge("id35", "apple", { stroke : "#bfa" , fill : "#56f", label : "Label" });

I couldn't get this to work. Looking at the source code, I found out that every edge setting is retrieved from the parameter "style", so I changed my approach to the following:
g.addEdge("id34", "cherry", { style: {directed : true} } );
g.addEdge("id35", "apple", { style: {stroke : "#bfa" , fill : "#56f", label : "Label" }});

Now it works fine. It would be great for new users if the docs could be updated.

Thanks for reporting!
I've corrected the docs here https://www.graphdracula.net/documentation/ does it seem right?

Thanks for answering. The documentation for "customize the colors of that edge" is fine now. The first part, "a directed connection, using an arrow", still needs correction though, "directed" also needs to be inside "style".

The fix mentioned here should make both versions work in the latest version (with and without style) https://github.com/strathausen/dracula/issues/48 let me know if that fixed it!


Fix instance getters to support extensions

As discussed in #109, the various getter functions for physical device don't work, because the user isn't able to pass in a chain of mutable structures for the Vulkan driver to fill in.

All getters will probably have to be marked unsafe, since we cannot validate the user passed in a structure with a valid p_next pointer.

I have not touched the getters which don't have any extensions yet. For example, I haven't modified get_physical_device_external_fence_properties since there are no published extensions which affect it (yet).

I don't know how to solve the issue with functions which need to work with vectors, specifically get_physical_device_queue_family_properties2, which does have an extension structure for it (QueueFamilyCheckpointPropertiesNV).

We would have to provide a function to get the length of the returned array.
The user would then allocate an array of structures, create an array of extension structures, then properly set the p_next pointers, and call the function to fill in the array of structure chains.

This change is

@Ralith ok, I've separated those functions into two, one with the _len prefix which retrieves the length of the array, and the other performs the actual call.
Alternatively, we could have a single function, which takes in the return parameter as an Option.

I was imagining the latter option, but on review this seems nicer anyway.


No CLI not working

I have not installed any cli and no cli is not working.

I can or can not confirm this !

I'm pretty sure this is all according to the specs. See line   of paragraph   clearly states:

As you can see, there's clearly no mention of a CLI working.


[bazel] Add BUILD file generator and tests.

This is the generator used to populate current googleapis directory with BUILD.bazel files. It is designed to not have any external dependencies (pure JDK is enough) for future easier integration to more appropriate places (sanitization logic).
This is one of the reasons, why it heavily relies on regexps for parsing all types of files and does not use any of the actual parsers (proto, yaml, json).
The goal of this generator is to remain simple and portable (no dependencies). It is highly unlikely that it will have to be heavily extended in the future, so just relying on regex should be enough for now. We need only tiny portion of information from each file (like a couple of properties), so using actual parsers is an overkill and it will complicate the implementation drastically.

meta question: while this bazel file generator will certainly be very useful, it seems very loosely related to gapic-generator. Would it be possible to put the code in //third_party/googleapis? With bazel you don't have any non-jvm dependencies and the generator itself is bazel-aware, so it shouldn't be hard to move the code around. It'll potentially make our tool easier to use - most API producers will be able to run the generator to generate the bazel files and use the generated bazel files to generate libraries from third_party/googleapis, without needing to talk to us or go to github. WDYT?

@hzyi-google I think your proposal is very reasonable. My reasons to put it here is:

This allows to keep rules and build files generator as a self-contained instance (similarly like gapic-generator is capable of generating gapic yaml)
I would like to avoid adding java-specific code (or any programming code actually) to googleapis. Lets keep googleapis a repo for configs and API definitions, but not for program source code.

This is still in "testing" mode, and I'm not sure what is the proper lace for it (i guess the best place is the sanitization logic). Putting it here as one of the easiest (but still reasonable) places for it.
If you don't have a strong opinion abou it, I would like to keep this code here. WDYT?

Okay that's fair.
I'll be away for a while so please feel free to merge after @noahdietz signs off.

This PR was accidentally closed. Please see the reopened PR #3055.


Automate deployments

[x] Automate deployment pipeline
[ ] Include automated testing (CI) in the pipeline

Added Netlify integration -> https://bordeaux.netlify.app/

Travis integrated in 3cea36c


Added new European signs and updated triangular signs

Added:

danger_ascent (versions for 10 and 12% are generated)
danger_crossroad
danger_danger_tmp
danger_descent (versions for 10 and 12% are generated)
danger_level_crossing
danger_railway_crossing_unsecured
information_bus_stop
information_first_aid
information_limited_access_road
priority_priority_road_end
prohibitory_speed_limit_zone (versions for 20, 30 and 40 are generated)

Updated (now using the vertically centered 512px triangle):

danger_construction
danger_cycle_crossing
danger_danger
danger_pedestrian_crossing
danger_quayside_or_ferry_berth (icon also updated)
priority_next_intersection_left
priority_next_intersection_right

Renamed:

danger_intersection → priority_next_intersection

:+1: Thank you!


Make example queries auto-change species in multispecies selector

We've discussed this previously, please point out if you notice this being a duplicate ticket.
Since Multispecies functionality, our example queries have become far less intuitive. They're a great resource for onboarding new users and should function simply as "click and Search".

Update the XML to include which TAXID an example is relevant for: https://github.com/Rothamsted/knetminer/blob/master/datasets/poaceae-test/config/sample-queries.xml
Lawal implement into frontend. Only display unspecified TaxIDs (blanks, which are relevant to all species) and taxids matching the currently selected species.

This is not the best solution. Instead, we simply only display queries relevant to the currently selected species.
The ideal solution (I think) would be something like this:

Where we wrap the part of the getQueryExamples function in multispecies.js which parses the values to begin with. This way we don't populate the array at all unless it matches with our currently selected species/taxid.
For some reason though, the above does not work and I spent far too long trying to fix something which is probably just a tiny mistake. I welcome anyone to reimplement the above and pick up where I left off.
In the meantime (or indefinitely, I guess), I have implemented a solution which always adds everything into the array, and then only displays it if it matches:

One additional problem this raises is:

The example number is held in the xml
This is fixed by just making the name itself clickable and removing the example counters. I gave it a bullet point instead:

Finally, this breaks the release notes popup. Took me a while to see why, but changing the order we're calling the functions in seems to fix this problem (hopefully it introduces no new issues):

What is still to be done is to add a test which checks if a user is on a Pro plan or not, and add another field in the xml, probably called .
I welcome anyone to make adjustments.

@marco-brandizi thanks for your feedback! True about the screenshots, I was just writing the notes as I went along and didn't really expect the pit to be so deep.
Good tips, thank you :)

Changes are now live on ci-test. I accidentally closed this with a fixes tag in my commit. Reopening this as it needs to be (1) tested and (2) I still want to add an additional check for whether a user is Pro or not, to show fancier, larger example queries.


Option to disable transforms

Hi! I have this code in templates:

Dear {{var customer_name}},

...
{{var quote.getGrandTotal()|formatPrice}}

<EMAIL_ADDRESS>and 1.1.1 output:

Dear ,

...

But I need this code untouched because it's using in futher tools. I using<EMAIL_ADDRESS>and everything fine.
Is there any way to prevent code removing for this case?

I am confident that we will find a solution for your issue. One quick hotfix could be to set options.transformGutter to a sequence that is definitely not in your code.
Can you please supply what output you expect, hence how did<EMAIL_ADDRESS>bake your code.

<EMAIL_ADDRESS>output:

Dear {{var customer_name}},

...
{{var quote.getGrandTotal()|formatPrice}}

Okay, so you do not want to process the variables at all? Then try setting process-option to false:
bake: {
  options: {
    process: false
  }
}

But at the same time I need to process this template.html:

_footer.html:

Unsubscribe

Expected output:
template.html
Unsubscribe

Will this be processed if I set process: false?

No it won't, because setting options.process to false disables the processing (parsing of template variables) completely. Have you tried the hotfix I suggested? Setting transformGutter to unused sequence like:
bake: {
  options: {
    transformGutter: '@!@'
  }
}

Tried on the last version — doesn't work. Input/output as in first message. But with this option grunt doesn't show this error in console:
>> Unknown transform: formatPrice

I think in 0.3.12 if plugin haven't found, for example, {{var customer_name}} variable defined it leave the code as is. But in next versions it's just delete code if variable is not found..

The problems here are not the transforms. Bake removes unknown placeholder since version 0.3.18. I have added an option options.removeUndefined that controls this behavior. You can use the current master (git install git+https://github.com/MathiasPaumgarten/grunt-bake.git) for your project till @MathiasPaumgarten checked the changes and bumped a new version.
@MathiasPaumgarten, make the final decision on the default value for removeUndefined. false restores the old behavior of grunt-bake, true the recent one which I personally prefer, but you are the creator so final decision is up to you :cop:

@david-zacharias Thank you! This was very fast :)

Hey guys, just caught up to this. You guys have detected the "issue" that grunt used to leave unrecognized variables as is. We change that behavior int 1.0.0. Since we had a major bump, I thought it was cool to alter behavior, following semver rules.
For future reference, another hotfix would have been to change the parsing pattern for the variables you want parsed, versus the ones you want left alone with:
bake: {
    options: {
        parsePattern: /<%\s?([\.\-\w]*)\s?%>/g;
    }
}

This would change the grunt parse pattern to something like <% foo %> instead of {{ foo }}.
However, thanks to @david-zacharias there is an even nicer option place now. I like that it removes variables by default. Because more often than not, if I would forget a variable, and I release it I rather want it to be gone instead of cryptic. But that's up to anyone to decide for themselves.
Additionally, in order to not change major version, and not break the behavior of  1.0.0 I leave the default to removing it.
I'll push the new version now.

I pushed the new version, that @hudochenkov has been using off the master branch to npm under 1.2.0. That way you can reference to npm instead of Github. (which Github probably prefers :smile:)
:ship:

Thank you very much! :)


TextPainter.getLineBoundary returns the wrong value when run on web tests

Info
When tests are run on the web TextPainter.getLineBoundary includes the line terminator when the documentation states it does not. When tests are run on the native platform TextPainter.getLineBoundary return value does not include the line terminator.
In a non-test environment, when a flutter app is running on the native platform TextPainter.getLineBoundary return value does not include the line terminator.
Steps to Reproduce
The following test variation from #83392 passes with flutter test test/widgets/text_test.dart but fails with flutter test test/widgets/text_test.dart --platform chrome
  test('getLineBoundary does not include newline characters', () {
    const text = 'aaa\nbbb';

    final paragraphStyle = ui.ParagraphStyle(
      textDirection: ui.TextDirection.ltr,
    );
    final paragraphBuilder = ui.ParagraphBuilder(paragraphStyle)
      ..addText(text);
    final constraints = ui.ParagraphConstraints(width: 1000);
    final paragraph = paragraphBuilder.build();
    paragraph.layout(constraints);

    var range = paragraph.getLineBoundary(const TextPosition(offset: 0));
    expect(
      range.textInside(text),
      equals('aaa'),
    );
    expect(range.start, 0);
    expect(range.end, 3);

    range = paragraph.getLineBoundary(const TextPosition(offset: 5));
    expect(
      range.textInside(text),
      equals('bbb'),
    );
    expect(range.start, 4);
    expect(range.end, 7);
  });

There is also conflicting documentation in text_painter.dart and text.dart
text_painter (last updated #83626):
  /// Returns the text range of the line at the given offset.
  ///
  /// The newline (if any) is not returned as part of the range.

text:
  /// Returns the [TextRange] of the line at the given [TextPosition].
  ///
  /// The newline (if any) is returned as part of the range.
  ///
  /// Not valid until after layout.
  ///
  /// This can potentially be expensive, since it needs to compute the line
  /// metrics, so use it sparingly.

cc @LongCatIsLooong

I replicated the reported behavior using latest stable and master versions.
dhs@Dhss-MacBook-Pro test_app % flutter test test/widget_test.dart
00:03 +1: All tests passed!

dhs@Dhss-MacBook-Pro test_app % flutter test test/widget_test.dart --platform chrome
00:02 +0 -1: getLineBoundary does not include newline characters [E]
  Expected: 'aaa'
    Actual: 'aaa\n'
              ''
     Which: is different. Both strings start the same, but the actual value also has the following trailing characters: \n

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (4 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.8.0-12.0.pre.49, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.8.0-12.0.pre.49 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b08cc8be79 (74 minutes ago), 2023-02-13 22:36:26 -0500
    • Engine revision c4f51bc786
    • Dart version 3.0.0 (build 3.0.0-233.0.dev)
    • DevTools version 2.21.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


Whitelist notification windows (e.g. growl)

Right now, they get tiled like the rest of the windows. I should whitelist them and treat them like floating transients, even if they are top level windows...

FWIW, libnotify dialogs (similar to growl afaik) render just fine. i've actually hooked nwm into notify-send so when i switch workspaces/focus it tells me what the main window's name is.


Interesting, would it be possible to encode as well?

Very interesting project @dawidpieper . Would you see it as more challenging implementing the encoder part of opus?
Or maybe you already tried ?
Kind regards
Jens

Typically encoding does consume more processor cycles than decoding .. IIRC about 1.5x in the case of Opus. Note that  the entire library is already present in this repo, so all the encode functionality is present.
Shorter frame sizes are easier to encode, but you may need to increase the bit rate to get acceptable quality. I'm thinking about using Opus for real-time audio networking, and in this case one does need short frames in order to keep latencies low.
Other settings may affect the processing rate quite a lot. The Opus packet loss concealment modes can chew up a lot of processing, and note that this is erratic, as the increase occurs only when concealment is required. Hence it is usually disabled for real time operations.
The good news is that as of the end of August, you can use a Pico2, which has hardware float support. I managed to convert the float benchmark to use the pico2, and got about a 6x speedup compared with the results reported in the README.

OK, I modified the code so that the decoded buffer was then re-encoded, running on a Pico2, using opus_decode_float and opus_encode_float
This gives the following results:
CPU khz set to 48000.
Bitrate 8, framesize 10, time 3919 ms, ratio 1.306333.
Bitrate 8, framesize 20, time 6791 ms, ratio 1.131833.
Bitrate 8, framesize 40, time 11354 ms, ratio 0.946167.
Bitrate 8, framesize 60, time 16656 ms, ratio 0.925333.
Bitrate 24, framesize 10, time 3263 ms, ratio 1.087667.
Bitrate 24, framesize 20, time 7134 ms, ratio 1.189000.
Bitrate 24, framesize 40, time 13689 ms, ratio 1.140750.
Bitrate 24, framesize 60, time 17472 ms, ratio 0.970667.
Bitrate 48, framesize 10, time 3424 ms, ratio 1.141333.
Bitrate 48, framesize 20, time 7500 ms, ratio 1.250000.
Bitrate 48, framesize 40, time 14880 ms, ratio 1.240000.
Bitrate 48, framesize 60, time 18080 ms, ratio 1.004444.
Bitrate 64, framesize 10, time 3487 ms, ratio 1.162333.
Bitrate 64, framesize 20, time 7779 ms, ratio 1.296500.
Bitrate 64, framesize 40, time 14953 ms, ratio 1.246083.
Bitrate 64, framesize 60, time 17649 ms, ratio 0.980500.
Bitrate 128, framesize 10, time 3596 ms, ratio 1.198667.
Bitrate 128, framesize 20, time 8045 ms, ratio 1.340833.
Bitrate 128, framesize 40, time 14818 ms, ratio 1.234833.
Bitrate 128, framesize 60, time 18141 ms, ratio 1.007833.

CPU khz set to 133000.
Bitrate 8, framesize 10, time 1414 ms, ratio 0.471333.
Bitrate 8, framesize 20, time 2450 ms, ratio 0.408333.
Bitrate 8, framesize 40, time 4099 ms, ratio 0.341583.
Bitrate 8, framesize 60, time 6011 ms, ratio 0.333944.
Bitrate 24, framesize 10, time 1178 ms, ratio 0.392667.
Bitrate 24, framesize 20, time 2574 ms, ratio 0.429000.
Bitrate 24, framesize 40, time 4940 ms, ratio 0.411667.
Bitrate 24, framesize 60, time 6306 ms, ratio 0.350333.
Bitrate 48, framesize 10, time 1236 ms, ratio 0.412000.
Bitrate 48, framesize 20, time 2706 ms, ratio 0.451000.
Bitrate 48, framesize 40, time 5372 ms, ratio 0.447667.
Bitrate 48, framesize 60, time 6526 ms, ratio 0.362556.
Bitrate 64, framesize 10, time 1258 ms, ratio 0.419333.
Bitrate 64, framesize 20, time 2809 ms, ratio 0.468167.
Bitrate 64, framesize 40, time 5398 ms, ratio 0.449833.
Bitrate 64, framesize 60, time 6369 ms, ratio 0.353833.
Bitrate 128, framesize 10, time 1298 ms, ratio 0.432667.
Bitrate 128, framesize 20, time 2904 ms, ratio 0.484000.
Bitrate 128, framesize 40, time 5349 ms, ratio 0.445750.
Bitrate 128, framesize 60, time 6547 ms, ratio 0.363722.

CPU khz set to 270000.
Bitrate 8, framesize 10, time 696 ms, ratio 0.232000.
Bitrate 8, framesize 20, time 1207 ms, ratio 0.201167.
Bitrate 8, framesize 40, time 2018 ms, ratio 0.168167.
Bitrate 8, framesize 60, time 2961 ms, ratio 0.164500.
Bitrate 24, framesize 10, time 580 ms, ratio 0.193333.
Bitrate 24, framesize 20, time 1268 ms, ratio 0.211333.
Bitrate 24, framesize 40, time 2433 ms, ratio 0.202750.
Bitrate 24, framesize 60, time 3107 ms, ratio 0.172611.
Bitrate 48, framesize 10, time 609 ms, ratio 0.203000.
Bitrate 48, framesize 20, time 1333 ms, ratio 0.222167.
Bitrate 48, framesize 40, time 2645 ms, ratio 0.220417.
Bitrate 48, framesize 60, time 3215 ms, ratio 0.178611.
Bitrate 64, framesize 10, time 620 ms, ratio 0.206667.
Bitrate 64, framesize 20, time 1383 ms, ratio 0.230500.
Bitrate 64, framesize 40, time 2658 ms, ratio 0.221500.
Bitrate 64, framesize 60, time 3137 ms, ratio 0.174278.
Bitrate 128, framesize 10, time 640 ms, ratio 0.213333.
Bitrate 128, framesize 20, time 1430 ms, ratio 0.238333.
Bitrate 128, framesize 40, time 2634 ms, ratio 0.219500.
Bitrate 128, framesize 60, time 3225 ms, ratio 0.179167.

Overall, times were about 4x slower for the decode-encode process than for decode alone. This makes sense .. encoding is more difficult than decoding. But note that with the Pico2, the times at 133 MHz are are faster than the decode only process for the Pico 1, and below 0.5. So real-time encode decode is straightforward on this platform.


Add setuptools and wheel as pip dependencies

🗣 Description
This pull request adds setuptools and wheel as pip dependencies.
💭 Motivation and Context
setuptools usually comes along with pip, but wheel does not.  Using wheel where possible to build python extensions is more modern and more security conscious than using the old-style setup.py that pip reverts to if wheel is not installed.
While we're adding wheel, we may as well add setuptools for good measure.
🧪 Testing
All the pre-commit hooks pass.
🚥 Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (causes existing functionality to change)

✅ Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

RELEASE THE LINEAGE KRAKEN!

I'll wait for @mcdonnnj's review and full buy-in before letting slip the dogs of lineage.


Fixes defect when using isValid on textArea

 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.Eric Sewalson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix(cli): fix path rendering on indexController for swagger page

Closes: https://github.com/tsedio/tsed/issues/2375

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

v4.6.2
GitHub release

Your semantic-release bot :package::rocket:


css.global not working in latest Canary release

Bug report
Describe the bug
I just installed v0.0.3-canary.4 to take advantage of the latest StitchesProps typings but now css.global doesn't seem to work anymore. Not sure though if it's an issue with gatsby-theme-stitches and @stitches/react not being in sync.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Create a new gatsby app and install/configure gatsby-theme-stitches plugin
Install @stitches/react
Use css.global to set body's background color
The CSS rule is not applied

Expected behavior
css.global should work properly.
System information

OS: macOS
Version of Stitches: [e.g. 0.0.3-canary.4]
Version of gatsby-theme-stitches: [e.g. 0.0.2]

Hello @Razinsky
As far as I understand, the API was changed in canary, from this discussion: https://github.com/modulz/stitches/discussions/285#discussioncomment-104956

Alright thanks @LexSwed I confirm that this new API works well 👍
Thanks for the swift response!


Document the misleading compilation errors with generic derivation

Here is a simple example:
import java.time.ZonedDateTime

case class Person(name: String, age: Int, date: ZonedDateTime)

import io.circe.syntax._
import io.circe.generic.auto._

Person("john", 12, ZonedDateTime.now()).asJson

Will produce Error: could not find implicit value for parameter encoder: io.circe.Encoder[Person]
Instead of the expected could not find [...] io.circe.Encoder[ZonedDateTime]
IMO the warning section of the documentation should start with this, as it is I guess the most common pitfall of the derivation mechanism.

It might be worth recommending https://github.com/tek/splain
Although the output could use explanation... something like this can be scary if you don't know what you're looking at:


Auto Jupyter Hub for PRs?

Just had an idea. What do you guys think about making a tool for PRs where once a PR passes all checks (CI, ASV, Coverage, etc), a Jupyter Hub is automatically hosted using the branch from the PR. This way multiple people can bang on it and get a feel for functionality/find bugs. Might be useful for code reviews. This could run on powertrip.
As a user story, consider the following. I just submitted #1201. It would be nice if you guys could quickly drop into a notebook and see what the reprs look like yourselves, rather than reading through the code or pulling my branch and running it.

That would be awesome. Preload cells from the cookbook or something?
On Fri, Nov 27, 2015 at 2:55 PM, Anders Pitman<EMAIL_ADDRESS>wrote:

Just had an idea. What do you guys think about making a tool for PRs where
once a PR passes all checks (CI, ASV, Coverage, etc), a Jupyter Hub is
automatically hosted using the branch from the PR. This way multiple people
can bang on it and get a feel for functionality/find bugs. Might be useful
for code reviews. This could run on powertrip.
As a user story, I just submitted #1201
https://github.com/biocore/scikit-bio/pull/1201. It would be nice if
you guys could quickly drop into a notebook and see what the reprs look
like yourselves, rather than reading through the code or pulling my branch
and running it.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/scikit-bio/issues/1202.

Yeah I like that idea. The purpose would probably mostly for UX and visual testing. Things we can't really confirm as easily with automated tests. So loading from the cookbook would give a great starting point and also provide a set of smoke tests. Also has the added benefit of battle testing the cookbook regularly.

:+1: binder may be useful for this.


eq-861 Answer Label Incorrect Semantics

What is the context of this PR?
Fixes #872 by replacing the semantically incorrect label element with a legend.

Changed logic in question template so that, if a fieldset/legend is used the legend is hidden rather than replacing the h2 on the question title.
Added field__legend styling for new legend element.
Legend falls back to repeating question title if no "answer label" is supplied. Also visually hidden in that case.
Improved semantics of UKIS survey by moving "Select all that apply" out of answer label (and therefore legend) and into question description wherever possible.

How to review

start UKIS survey
Check that semantics are correct for as many questions as possible. IE. a legend is used where there are fields with multiple input (ie radios/checkboxes)

The main issue with this is that it now creates an unwanted split between question title and the legend by having the guidance inbetween in some places.

@hamishtaplin what affect does this have an all other questionnaires?

@collisdigital no visible affect as far as I can tell but it does warrant some testing. Existing surveys don't use the answer label so the (hidden) legend just inherits the question title.

What testing and who would need to do it?

@collisdigital Testing the other surveys to make sure they are not negatively affected. I'm pretty sure they aren't having been through Census but it warrants a fresh pair of eyes. Anyone can do it.

Perfect, when can you start?

@collisdigital I've already taken a look, someone else needs to double-check things. Especially the UKIS survey as I'm not as familiar with it.

@hamishtaplin @collisdigital Tested all the surveys (MCI, RSI, Census & UKIS) by going through all the survey pages nothing seems to be broken visually or content wise. Also did test using voice over with different question times and work fine. However I guess we are going to have another set of DAC testing to prove the accessibility.

As agreed with Will, I moved any guidance that is splitting the question in two (as above) to the hidden guidance underneath answers.


Support for SQL Server 2017

I'm trying to setup a new development environment on my mac

Development OS: MacOS 10.13.2
Python: 3.6
pyodbc: 4.0.21
driver manager: unixODBC 2.3.4
-driver: FreeTDS 1.00.79
DB: MS SQL Server 2017 Developer [Microsoft SQL Server 2017 14.0.3008.27 (X64) Developer Edition (64-bit) on Linux (Ubuntu 16.04.3 LTS)] downloaded from https://docs.microsoft.com/en-us/sql/linux/quickstart-install-connect-docker#pull-and-run-the-container-image

Issue
telnet to mssql-linux2017 docker image on 1433 is fine
sqlcmd fails with below error
ideally I want to do
pyodbc.connect('DRIVER=ODBC Driver 13 for SQL Server;SERVER=localhost;PORT:1433;DATABASE=<SOMEDB>;UID=SA;PWD=<myPass>')
[ODBC][22520][1512841146.847871][SQLGetDiagFieldW.c][815]
Entry:
Connection = 0x7fcbe2010c00
Rec Number = 2
Diag Ident = 5
Diag Info Ptr = 0x7ffeec672b3c
Buffer Length = -6
String Len Ptr = 0x0
[ODBC][22520][1512841146.848022][SQLGetDiagFieldW.c][835]
Exit:[SQL_SUCCESS]
[ODBC][22520][1512841146.848195][SQLGetDiagFieldW.c][815]
Entry:
Connection = 0x7fcbe2010c00
Rec Number = 2
Diag Ident = 6
Diag Info Ptr = 0x7fcbe200d200
Buffer Length = 2050
String Len Ptr = 0x7ffeec672b40
[ODBC][22520][1512841146.848367][SQLGetDiagFieldW.c][835]
Exit:[SQL_SUCCESS]
[ODBC][22520][1512841146.848539][SQLGetDiagFieldW.c][815]
Entry:
Connection = 0x7fcbe2010c00
Rec Number = 2
Diag Ident = 4
Diag Info Ptr = 0x7ffeec672b44
Buffer Length = 12
String Len Ptr = 0x7ffeec672b40
[ODBC][22520][1512841146.848711][SQLGetDiagFieldW.c][835]
Exit:[SQL_SUCCESS]
[ODBC][22520][1512841146.848939][SQLGetConnectAttrW.c][211]Error: SQL_INVALID_HANDLE

Check this.
https://github.com/mkleehammer/pyodbc/wiki/Connecting-to-SQL-Server-from-Mac-OSX

Thank you! Those the steps I followed and am still somehow stuck. I wonder if its the MS SQL 2017 version thats causing, so the issue is posted in the same repo.

ok, my bad the docker image wasn't saved after I changed password policy and password on SQL Server 2017 Dev Edn. FreeTDS logging was enabled and this was obvious.
Still strugglign with pyodbc setup for FreeTDS, however pymssql works like charm. I don't want to use pymssql because its some code change to make.

Answered here:
https://stackoverflow.com/a/47738072/2144390

ok, my bad the docker image wasn't saved after I changed password policy and password on SQL Server 2017 Dev Edn. FreeTDS logging was enabled and this was obvious.
Still strugglign with pyodbc setup for FreeTDS, however pymssql works like charm. I don't want to use pymssql because its some code change to make.


Missing Create from YAML button in resource lists

Setup

Rancher version: 2.6.3
Browser type & version: Chrome

Describe the bug
For resources that do not have a edit form, the "Create from YAML" button is missing on the resource list page.
To Reproduce
As a cluster owner, go to the NetworkPolicies resource list.
Result
There is no "Create from YAML" button.
Expected Result
There should be a "Create from YAML" button.
Screenshots
Broken in 2.6.3:

Works in 2.6.2:

Additional context
The resource list items have a edit button in the context menu. The user I tested with has permissions to create and edit these rersources.
The create button is still there in 2.6.2. The commit that breaks it is https://github.com/rancher/dashboard/commit/c4c40a580a93bea30d48f06c273e6bf0e032311c.

My checks PASSED
Reproduction Environment

Component
Version / Type

Rancher version
v2.6.3

Rancher cluster type
Docker

Docker version
20.10.7, build 20.10.7-0ubuntu5~20.04.2

Helm version
v2.16.8-rancher1

Downstream cluster type
not applicable

Downstream K8s version
not applicable

Browser type
Google Chrome

Browser version
97.0.4692.99 (Official Build) (x86_64)

Reproduction steps
Prerequisites

Logged in as an admin user

Starting from the default Rancher homepage /dashboard/home
Click the hamburger menu -> Cluster Management -> Explore next to the local cluster
Click Service Discovery from the left side menu -> Network Policies

RESULT
There is no Create from YAML button shown in the UI
EXPECTED RESULT
For there to be a button Create from YAML

Validation Environment

Component
Version / Type

Rancher version
v2.6-7efbeab2cc2fa1a9dce2063970df6b48fdcaa2cf-head

Rancher cluster type
Docker

Docker version
20.10.7, build 20.10.7-0ubuntu5~20.04.2

Helm version
v2.16.8-rancher1

Downstream cluster type
not applicable

Downstream K8s version
not applicable

Browser type
Google Chrome

Browser version
97.0.4692.99 (Official Build) (x86_64)

Validation steps
Prerequisites

Logged in as an admin user

Starting from the default Rancher homepage /dashboard/home
Click the hamburger menu -> Cluster Management -> Explore next to the local cluster
Click Service Discovery from the left side menu -> Network Policies

RESULT
Checks passed! The Create from YAML button is present and functional


[ENH] Improve deep learning networks test coverage

Reference Issues/PRs
Improves #1233
What does this implement/fix? Explain your changes.
Add network specific tests to improve coverage of DL networks :

inception network coverage 67% -> 100%
encoder network coverage 13% -> 100%
cnn network coverage 98% -> 99% (last percent is from a deprecated class)

Does your contribution introduce a new dependency? If yes, which one?
No
Any other comments?
AE networks have been ignored as they already have someone working on it with specific tests ref.
There are also 2 networks that can have a better coverage (lite and tapnet) but that have been ignored as they are already skipped in test_all_networks, see #1867.
PR checklist
For all contributions

[x] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.

Thanks! More testing coverage is always welcome.
Rerunning CI so it generates a coverage report, but this all looks good in principle.

good idea this way we can update the issue with the coverage reports, thanks @Cyril-Meyer for the changes


New aka.ms link

Aka.ms link name
DevLatest
Category (Optional. Select None to auto-select)
None

Thank you for submitting aka.ms/DevLatest. Your link already exists at akaSearch.net. 🙏✅


odd parentage for "ubiquitin-dependent catabolic process"

It seems odd that
GO:0051603
proteolysis involved in cellular protein catabolic process
is a parent of
ubiquitin-dependent catabolic process
because the proteolysis part is a MF?
it's only one step in a "ubiquitin-dependent catabolic process" pathway.
I suggest we get rid of this term (also makes it difficult to find correct terms if drilling down, so not help annotation specificity )

Has no direct EXP annotation, so no loss...

Sorry there are 63 (you can't use the GO ID in the filter  in AMiGO, only the term name)

some are proteases, would be better
peptidase activity part_of   "other child of this term"
GO:0007039    protein catabolic process in the vacuole
(for instance)
many are clearly one of the ubiquitin pathways:
GO:0006511    ubiquitin-dependent protein catabolic process
or
GO:0043161    proteasome-mediated ubiquitin-dependent protein catabolic process


Test IS 7.0.0 on Azure virtual machines with Hazelcast Azure membership scheme

Describe the issue:
Azure public cloud is getting popular among the public. Hazelcast Azure VM membership scheme needs to be tested with IS 7.0.0.
https://github.com/pabasara-mahindapala/azure-membership-scheme
\

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add packaging for Meteor.js

Hi!
As you maybe know, it is now possible to use AngularJS and Meteor together with the help of the angular-meteor package.
The package does a few things to make that possible and is dependent on an Angular package - a separate package that just holds all the AngularJS releases.
But right now we do it manually, which is hard for each of the releases (angular-animate, angular-sanitize etc...)
There is an effort right now to pull request popular libraries to support the Meteor release as part of their release process.
Top examples of popular libraries that already doing that are:

d3
Bootstrap
moment

and even AngularUI's google maps directive...
I would create this issue as a pull request but it looks like your release process is much more complex than the others.
The only thing that needs to be done is to add one file named package.js and I would love to help you do that anytime that is suitable for you.
Thank you so much for building such a great framework,
Urigo

To clarify, this is a additional packaging per angular "module" - core, animate, route etc. - that includes the whole release history, right?
If so, couldn't this package use bower / npm / github releases?

@Narretz yes it can and until recently we did just that.
The reason for adding Meteor Packaging is that Npm and Bower doesn't really have a good version solver like Rails bundler or Scala, and Meteor does have that support.
That's a big difference for Angular developers working on Meteor, it makes their development process much more streamlined and simpler than falling back to Npm or Bower (what we did until recently).
Maybe in the future Npm will use something similar (if they want to support FrontEnd dependencies in a good way) or even use Meteor's solver, but unfortunately right now they not.
For more information I would really recommend @dgreensp 's talk about the constraint solver, especially the Npm part - https://youtu.be/IWSaZfSVnFM?t=5m58s

@Narretz also, there is no need to publish all history, we already published that (including versions 2.0.0-alpha.x)
Also we moved to the 'angular' namespace instead of 'angularjs' so it will better fit the official angular releases

+1.
I'd really love to develop against latest angular-material (which is right now 1.0.0-RC7).
However, the latest angular-material package available on atmosphere is currently 0.11.4

This never really got traction, and at this point in the AngularJS lifecycle it's probably not worth the effort to include this in the build.

@Narretz thanks, I guess the main reason it didn't got traction is that Meteor supports npm completely for the past year or so, so if anyone end up into this PR, know that the only thing you need to do in order to install angular is just to get it from npm. :)


Adding CHF as a currency

Hey
Like the tool, but really badly want my own currency as an option.
Seemed like a pretty straight forward change, but I am not sure if I caught everything.
How exactly is the symbol (which doesn't exist for CHF, it is either CHF or Fr.) provided?

I am just going to merge this. Please review the updated CONTRIBUTING.MD for more contributions.


view all front pages for title

Descriptive Summary
Our requirements gathering process produced a feature request that users be able to run a search that returns only front pages for a given newspaper title, in chronological order.
This search should be linked from the show page of a NewspaperTitle object.
The search control should have the text Show all front pages, and should be rendered as a button.
Rationale
Participants in the Newspapers Interest Group submitted this feature request, which reportedly gets significant usage.
This feature is required for parity with open-oni. See the "all front pages" link in Chronicling America.
Expected behavior
As a non-logged-in user
When I am on the show page for a NewspaperTitle
I should see a button that says Show all front pages
And when I click the button
I should see a gallery-style search results list
And each result in the list should represent the first page of the issue
And when I click on a result
I should be able to view the first page of the issue
Implementation notes
Two possible approaches come to mind:

The search simply returns all NewspaperIssue objects that are children of the NewspaperTitle
This approach would be fairly straightforward to implement, except that we are planning on excluding Issue objects from the search results, focusing on page-level results instead.
If Issue objects are being excluded by default, this implementation pattern would need to somehow override the default (perhaps by using a different SearchBuilder class).

The search returns NewspaperPage results, somehow limiting the result to only the first page
This approach would require using some kind of search parameter related to the page index. (Could be where 'is_following_page_of_ssi' is nil?) Or a field would need to be added NewspaperPage indexing to explicitly indicate "first page" status.

This ticket will be complete when:
User story described above is functional.

Hold off on this until the persistent-urls branch is merged.
There will be a specific field to indicate first page status: first_page_bsi

Closed via #147.


WF TESTING: DON'T MERGE!!!

What I did

Testing Instructions

Notes to Reviewers

These phishing tests are really getting intricate now.

can we close this now?

can we close this yet?


running droppy start

im having trouble getting this to work. Ive tried both the install script and following the instructions from the Documentation. I run into
Missing devDependencies to compile resource cache, please reinstall or run `npm install --only=dev` inside the project directory
    at compile (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:261:15)
    at ReadFileContext.callback (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:181:14)
    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:281:13)

Hey @dtuando are you using yarn for this?

Yes. Using yarn

I've still had installation problems with the latest release too, looks like that needs more work. I can help debugging this if need to be, although I cannot currently remember how it is broken or what my workaround to get it running/upgraded was.

@dtuando can you drop your version of node and npm please?

Got the same error after installing with npm:
tobias@hpserver:~$ npm install @droppyjs/cli
npm WARN deprecated<EMAIL_ADDRESS>"Please update to latest v2.3 or v2.2"

><EMAIL_ADDRESS>postinstall /home/tobias/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js:
> https://opencollective.com/core-js
> https://patreon.com/zloirock
> https://paypal.me/zloirock
> bitcoin: bc1qlea7544qtsmj2rayg0lthvza9fau63ux0fstcz

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

npm WARN saveError ENOENT: no such file or directory, open '/home/tobias/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN enoent ENOENT: no such file or directory, open '/home/tobias/package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN tobias No description
npm WARN tobias No repository field.
npm WARN tobias No README data
npm WARN tobias No license field.

+<EMAIL_ADDRESS>added 118 packages from 1059 contributors and audited 119 packages in 14.102s

16 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

npm --version: 6.14.16
node --version: v12.22.10

@DerTobi75 you should install the cli globally
npm install-g  @droppyjs/cli

@markhughes
That results in the same Errors. I tried to install without -g because someone mentions that fixed the problem for him.

@markhughes
This is what I got, when I try to install the cli globally:
sudo npm install -g @droppyjs/cli
[sudo] password for tobias:
npm WARN deprecated<EMAIL_ADDRESS>"Please update to latest v2.3 or v2.2"
/usr/bin/droppy -> /usr/lib/node_modules/@droppyjs/cli/bin/droppy

><EMAIL_ADDRESS>postinstall /usr/lib/node_modules/@droppyjs/cli/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js:
> https://opencollective.com/core-js
> https://patreon.com/zloirock
> https://paypal.me/zloirock
> bitcoin: bc1qlea7544qtsmj2rayg0lthvza9fau63ux0fstcz

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/@droppyjs/cli/node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

+<EMAIL_ADDRESS>added 118 packages from 1059 contributors in 11.11s

tobias@hpserver:~$ droppy start -c /home/tobias/.droppy/config -f /home/tobias/.droppy/files

           .:.
    :::  .:::::.   @droppyjs/server 0.2.2 running on node 12.22.10
  ..:::..  :::     config at /home/tobias/.droppy/config
   ':::'   :::     files at /home/tobias/.droppy/files
     '

2022-02-25 00:17:50 [INFO] Configuration: { listeners: [ { host: [ '<IP_ADDRESS>', '::' ], port: 8989, protocol: 'http' } ], public: false, timestamps: true, linkLength: 5, linkExtensions: false, logLevel: 2, maxFileSize: 0, updateInterval: 1000, pollingInterval: 0, keepAlive: 20000, uploadTimeout: 604800000, allowFrame: false, readOnly: false, ignorePatterns: [], watch: true, headers: {}}
2022-02-25 00:17:50 [INFO] Loading resources ...
2022-02-25 00:17:50 [INFO] ENOENT /home/tobias/.droppy/cache/cache.json, building cache ...
2022-02-25 00:17:50 [INFO] Loading resources done
2022-02-25 00:17:50 [ERROR] Missing devDependencies to compile resource cache, please reinstall or run `npm install --only=dev` inside the project directory
    at compile (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:261:15)
    at ReadFileContext.callback (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:181:14)
    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:261:13)

Sorry all. I see the issue, my mistake - fix is incoming.

Resolved in 0.2.3


Forward doesn't work for inlined fields

Hi there,
I was using dal  with the forward option and I noticed (as pointed out in #639) that it doesn't work as well as the live demo of the feature at [http://dal-yourlabs.rhcloud.com/admin/linked_data/testmodel/add/]:
if a value is selected for the owner (for inlined elements) the values proposed by the widget of the "test" field don't change.
I discovered that in select2.js, the script tries to match the DOM element with name="$('[name=' + forward[key] + ']')" but when the field is inlined it doesn't have the name of the model field. Instead it has a name like inlinedModelName-x-x where x is a digit from 0 to 9.
I have made a little fix for select2.js  as follows :
(from line 26 of select2.js)
var forward = element.attr('data-autocomplete-light-forward');
if (forward !== undefined) {
   var data_forward = {};
   forward = forward.split(',');
   prefix = $(this).attr('name').match(/\w+-(\d+-)+/);
   if( prefix != null)
   {
      prefix = prefix[0];
   }
   else
   {
      prefix = '';
   }
   for (var key in forward) {
      data_forward[forward[key]] = $('[name=' + prefix + forward[key] + ']').val();
   }

   data.forward = JSON.stringify(data_forward);
}

Hope it helps!

EDIT
Sorry, I just noticed that the bug was fixed 4 days ago with the commit a8f89ab.

Upgrade version.

Sorry, something between travis and openshift broke and this affected continuous deployment, the demo website doesn't have this commit from 5 days ago. Thanks for reporting !

Fixed by 00a110f
I guess I had done a password reset or something on openshift itself xD


Push Settings to different displays

I believe it's more common use case to configure one display, and do this externally like the navigator, than to have different displays with different configurations.

Revisiting this idea as I'm working on settings: how about a toggle that allows the settings to be configured by other clients?
In our settings, there'll be local, local-private, and global settings in terms of permissions/categories under the hood

For now a toggle is fine. Most people don't even know they can affect the Shabad OS monitors. This will probably need to be re-visited once we have a mobile app and more users.
What is local vs local-private?
What will the user read for the setting of this toggle?

Local = settings that only affect the current device's settings (but can be modified by others, if enabled)
Local-private = settings that can only ever be modified by the current device (e.g. the toggle to allow others)
Global/Server = settings like analytics that can only exist on the server

Another question -- How do you control which display of the host is being changed?
Host can currently have multiple display setups (So one projector can be multiple lines of gurbani, the other can be english translations, TV can be single frame, etc.)
If we can, it might be nice to set up "Scenes" and allow users to apply those to specific displays so they don't have to constantly set them all up separately each time. Then if you have a "scene" you can modify that directly.
This can also apply to OBS in case people want to set up multiple OBS scenes (live captions subtitles and separate full screen presentation during broadcasts).

The way I was envisioning it is that you can click on each device name that's connected to the server (and is allowing their settings to be modified), and then see the same settings as you would for your own machine. These would appear under a "Other devices" option in settings, or similar.
In the case of multi-display-single-device setups, I think more work has to kinda be done to figure out how it'd actually work. Regarding scenes & multi-display-single-device setups I reckon should be a new/different feature.
On second thoughts, maybe we could extend this feature to have each connected instance register themselves separately, and then abstract everything out as screens, as opposed to devices? Then the option would be "Other screens", and would show any connected window to the server.

If it's possible to register connected instances, then you could place each one under the connected devices name (as a tree perhaps). That would be pretty intuitive and powerful

That's a good idea. Could we file this as a separate issue and link back?

Done in dev 🤘


[FEATURE] ArgillaTrainer - allow passing device parameter

Is your feature request related to a problem? Please describe.
Related: https://github.com/argilla-io/argilla/pull/3467#discussion_r1305230854
Describe the solution you'd like
I want users to be able to provide a device so the model can always be put on the right device, no matter whether that's cpu, gpu, mps, etc.:
trainer = ArgillaTrainer(
    dataset=...,
    device="mps",
    ...
)

Currently we support gpu_id exclusively for spaCy. I propose that we replace gpu_id with device, which takes either an int, a string or a torch.device. Initially, we'll have:
if not device and gpu_id:
    warnings.warn(...)
    device = gpu_id
self.device = device

and spaCy will use the self.device as the gpu_id. That way we'll have backwards compatibility there.
Describe alternatives you've considered

Use update_config to change the device and/or move the device with trainer._trainer.model.to("...") or something.
Make sure torch is compiled correctly so that cuda or mps is available.

Additional context
Should be implemented after #3467

Tom Aarsen

Hi @davidberenstein1957 can I pick this up ?

Files can be found in two places:

src/argilla/training (old)
src/argilla/client/feedback/training (new)

The ArgillaTrainer.__init__ should allow for a device param and be handled as self.device as @tomaarsen suggested above.

src/argilla/training/base.py
src/argilla/client/feedback/training/base.py

The framework classes defined by Argilla*Trainer.__init__() should use this device and set for further usage.

src/argilla/client/feedback/training/frameworks/spacy.py
src/argilla/client/feedback/training/frameworks/span_marker.py
src/argilla/client/feedback/training/frameworks/transformers.py
src/argilla/client/feedback/training/frameworks/trl.py
src/argilla/training/peft.py
src/argilla/training/spacy.py
src/argilla/training/span_marker.py
src/argilla/training/transformers.py


Fix 64 bit integer unpacking in map.

I was using Theo and setting a relationship property to an Int64 and it resulted in an UnpackError.unexpectedByteMarker from Int32.unpack. Looks like Map had wrong unpack call for Int64 byte marker.

You are of course absolutely right. Thanks a bunch :-)


Issue with example

using matrix_type = max7219<LCD_WIDTH/8,LCD_HEIGHT/8,PIN_NUM_CS,bus_type,LCD_ROTATION> ;
should be
using matrix_type = max7219<LCD_WIDTH/8,LCD_HEIGHT/8,bus_type,LCD_ROTATION>;

Oooh thanks for the catch. I'm rolling out a new revision of GFX soon and I'll have the demo updated when I update this driver. Thanks again! 1.1.0 will have the sample fixed.


docs: Add info about graph deploy

@Jannis could you review updated code? Thanks!

@Jannis updated as per feedback – could you review and merge? Thanks!


output metric on api calls per policy

event hook on boto  before call to collections.counter, likely this will need to be done at session factory level given current call patterns.

merged in #2844


[Component] - Chat UI

It would be great to have a built in chat UI, to handle openAI like streaming responses.
This could help folks build their own support bots, without relying on third part scripts and control the end to end experience.

Should be quite easy to do it with Tailwind and the components from the library :)


Kubespary and Vagrant bridged network

How can I use Kubespray and Vagrant with bridged network (not nat)?

this does no go to issues, if you have such questions you should join slack and discuss those questions there.