Feature Request:  Review Mode with Checklist for Aider

Context:
When I ask Aider to review my code, it often identifies issues and suggests corrections. While some of these corrections are accurate, others are based on design choices that do not need to be changed. Currently, Aider goes ahead and implements all suggested changes and commits them, which sometimes results in unnecessary modifications that I need to undo, leading to diminishing returns.
Request:
I propose adding a "Review Mode" switch in the Aider browser interface, with an accompanying checklist feature. This mode would allow users to control how Aider behaves when reviewing code. In Review Mode, Aider would:

Identify and list potential issues and suggestions.
Provide a clear distinction between critical issues and those that might be intentional design choices.
Present a checklist of identified issues, allowing users to selectively check off the items they want to be fixed.
Allow users to selectively apply changes based on the review feedback.
Provide an option for users to add new issues to the checklist during the review process.

Workflow:

Review Identification: Aider scans the code and lists potential issues and suggestions.
User Interaction: Users review the checklist, check off the items they want to be addressed, and have the option to add new issues.
Selective Fixes: Aider only applies fixes to the checked items, respecting the user's design choices.
Iterative Review: Users can choose to re-evaluate the checklist after initial fixes to address more items if necessary.

Benefits:

Enhanced user control over code changes.
Reduced need to undo unnecessary modifications.
Clearer understanding of the rationale behind design choices.
More efficient and collaborative code review process.

Implementing this feature would streamline the code review process, ensuring that changes made by Aider align more closely with the user's intentions and providing a more interactive and user-friendly experience.

Aider v0.40.0
Models: claude-3-5-sonnet-20240620 with diff edit format, weak model claude-3-haiku-20240307
Git repo: \<IP_ADDRESS>\YonaVmDataShare\Projects\posexport.git with 28 files
Repo-map: using 1024 tokens
Restored previous conversation history.

I agree totally with this concept. It seems to already exist as we already get prompted by aider to do things like add files that the LLM suggests it wants access to, and for the LLM to add new files and such things.  I have more applications of this concept to add to the discussion as well.
One very big bottleneck that high level LLM tools like aider address very well is the issue of repetitively copy-pasting code. We have to do it a significant amount because that's what's required to update the LLM's understanding of changes to the code which we as the primary programmer are making. Sometimes these updates may be compressed with descriptions or diffs, but just like with math, an LLM is poor at tracking mutated state.
As we blow past this bottleneck, then, the tradeoff is API token consumption which costs directly scale with. When we do the caveman copy paste workflow, we manually manipulate text and get a deeply intuitive sense of token consumption. Granted, with chatbot usage the pricing isn't per-token, but we need to manage token consumption for the purpose of keeping important information in-context anyhow, as results drop off a cliff when we run afoul of that.
What I'm trying to get at is with an open-source tool like aider we have a unique opportunity to give excellent UX that no AI vendor will ever offer, which helps the user review and control resource consumption.
So I would suggest for Review Mode to expand its scope beyond just being an approval process for changes: I want to apply this approval process also to prompts whose token consumption exceed a certain threshold (heuristically or otherwise) as one method to help us keep token related costs in check.
A lite version of this could be some sort of "more raw" debug log to gain access to where we can view all of the raw tokens being sent in and received from the LLM. Even if no Review Mode approval interaction exists we could use a raw communication log to at least see how the tool's automation works so we can learn how to better control it.
One thing i'm not clear on is whether aider could automatically endeavor to autorespond with increased contextual detail if it becomes obvious somehow that not enough context was provided initially. If this were to happen, it may lead to greatly increased or even runaway token consumption.

You might find these docs helpful:
https://aider.chat/docs/config/options.html#--llm-history-file-llm_history_file
If you run aider with --no-stream it will output costs after each message.

Is there a way to get it to display the cost even when streaming?

Unfortunately, no. The streaming API doesn't return cost info.


Assess Moonbeam integration

https://docs.moonbeam.network/

One of the design goals of Moonbeam is to create an environment that is as close as possible to Ethereum, and to offer a set of Web3 RPC endpoints that are compatible with Ethereum. However, Moonbeam is also a Substrate based chain, which means that it exposes Substrate RPCs, and that it has integral functionality that is powered by Substrate such as Staking, Governance, and other features which are not part of the Ethereum API.

Ethereum JSON RPC methods currently supported by Moonbeam:

https://docs.moonbeam.network/builders/get-started/eth-compare/rpc-support/

Using MetaMask:

Moonriver (testnet):

Chain id: 1285
Symbol: MOVR
Provider URL: https://rpc.api.moonriver.moonbeam.network
Explorer URL: https://moonriver.moonscan.io/

Moonbeam (mainnet):

Chain id: 1284
Symbol: GLMR
Provider URL: https://rpc.api.moonbeam.network
Explorer URL: https://blockscout.moonbeam.network/

Moonbase Alpha faucet:

Connect to Discord channel
Under "Miscellaneous" category, #Moonbase-Faucet channel, type: !faucet send <evm_address>
Additional 5 DEV tokens can be claimed every 24 hours

AFAIK, Moonriver is much more popular than Moonbase Alpha. But at the same time, it's not an apples to apples comparison. As Moonriver runs on Kusama and has its own utility token (MOVR), it is more like a stepping stone into the Moonriver ecosystem than a testnet in the "hey this is a test chain to deploy contracts for free" sense.

Exactly :-) For now, we'll try to deploy on Moonbase Alpha just for free, considering also the official description for Moonbase Alpha: "This testnet is a network hosted by PureStake. It features a parachain/relay chain scheme. The goal is to allow developers to test the Ethereum compatibility features of Moonbeam in a shared parachain environment without needing to run their own nodes or network."

Witnet Solidity Bridge deployed on Moonbase Alpha:

Witnet Solidity Bridge:

WitnetDecoderLib: 0xD9a6d1Ea0d0f4795985725C7Bd40C31a667c033d
WitnetParserLib: 0xeD074DA2A76FD2Ca90C1508930b4FB4420e413B0
WitnetProxy: 0x02Cd4089679EAA9431a88170fd784e7dE78A2425
WitnetRequestBoardTrustableDefault: 0x225BAd150B9D5202DC805B34A0DF64B1a77459dF

Witnet use cases:

WitnetPriceRouter: 0x56834Ff8D4b27db647Da97CA3bd8540f7fA0e89D

Created 'bridgemoonbase' stack on bootstrap machine.

Ethereum JSON RPC methods currently supported by Moonbeam:

https://docs.moonbeam.network/builders/get-started/eth-compare/rpc-support/

eth_getFilterChanges seems not to be implemented, nor by the general public (metamask) rpc provider, nor a moonbase alpha endpoint created on Bware Labs platform (infura alik). While this doesn't prevent the whole request/complete workflow to complete, it could eventually turn into an inconvenience if either the poller and/or bridge node start to lack responsiveness because of this. New configuration parameter will be added to the web3-jsonrpc-gateway/ethers implementation as an optional way to mock responses to eth_getFilterChanges requests.

Same thing happening w/ endpoints created on Bware Labs:

New ETHERS_MOCK_FILTERS will be added to web3-jsonrpc-gateway/ethers implementation.


can not install hezar library what is compatible python version with this library

when install hezar library i got this error with python 3.12
ERROR: Could not find a version that satisfies the requirement torch<2.1.0,>=1.10.0 (from hezar) (from versions: none)
ERROR: No matching distribution found for torch<2.1.0,>=1.10.0

to solve it I tried to downgrade my python version to 3.8 or 3.7 but after installation when run my code
from hezar import Dataset
xlsum_dataset = Dataset.load("hezarai/xlsum-fa")

this is log when run above code by python 3.7

PS E:\python\hezar> python main.py
Traceback (most recent call last):
  File "main.py", line 4, in <module>
    from hezar import Dataset
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\__init__.py", line 24, in __getattr__
    from .data import Dataset
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\data\__init__.py", line 1, in <module>
    from .data_collators import *
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\data\data_collators.py", line 4, in <module>
    from ..preprocessors import Tokenizer
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\preprocessors\__init__.py", line 1, in <module>
    from ..registry import register_preprocessor  # noqa
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\registry.py", line 30, in <module>
    from .configs import (
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\configs.py", line 26, in <module>
    from .utils import Logger, get_module_config_class
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\utils\__init__.py", line 1, in <module>
    from .audio_utils import *
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\utils\audio_utils.py", line 9, in <module>
    from .integration_utils import is_backend_available
  File "C:\Users\ASUS\AppData\Roaming\Python\Python37\site-packages\hezar\utils\integration_utils.py", line 2, in <module>
    from importlib.metadata import version
ModuleNotFoundError: No module named 'importlib.metadata'

Hello @muhammadCodeSh and thanks for opening this issue. Hezar is currently only installable with Python 3.8 up to 3.11.


pull all nodes

I don't think I'll like this as-is but I want to try how it feels so here it goes.


Independent Fileset is not getting deleted, on deleting the snapshot pvc first and snapshot later.

##Bug Description
Independent Fileset is not getting deleted, on deleting the snapshot pvc first and snapshot later.
How to Reproduce?

Create SC with version = 2.
Create PVC using the using the new storage class.
Create SnapshotClass with window =15 mins.
Create Snapshots of the above PVCs.
Delete all the Snapshots created, and leave the one snapshot.
Delete the PVC.
Delete the snapshot.

Expected behavior
The independent fileset should get deleted.
Data Collection and Debugging
Environmental output
All the Snippets are attached.

Image Used :
quay.io/ibm-spectrum-scale-dev/ibm-spectrum-scale-csi-operator:dev
name: CSI_DRIVER_IMAGE
value: quay.io/ibm-spectrum-scale-dev/ibm-spectrum-scale-csi-driver:dev

@monimeht Please verify this and let me know if the fix works.

Issue is reproduced on Monica's cluster,she should be updating the defect soon.

Verified the issue. Working fine.


How do you place objects on top of one another?

Hello. I am trying to build an application using Android Studio and Sceneform that will allow users to place 3D models of cubes on top of one another (like building blocks). Usually, when you place an object in the same position as another, it just appears inside the first object. I want the object to appear on top of the first object (or under, depending on where the user taps). I also want to add gravity/physics to the blocks so that they can fall.
Is there any way to do this?
Thanks!

Sceneform does not have a physics engine integrated, so you need to handle that yourself.
For positioning, you can use the Renderer.getColliderShape() to get the box enclosing the renderable and perform intersection testing or positioning based on the size.
From your general description, you might be better off investigating using a game engine such as Unity (C#), Unreal Engine (C++) or libGDX (Java).


Add .travis.yml file

Like the others, this project should use Travis CI.
To do that, we have to add a file called  .travis.yml containing build instructions.
For now, copy the one from eva, but without the jacoco:report coveralls:report, only mvn clean install

@salikjan

please tell me what do you mean by fork? Create new branch by git checkout and after that make any changes in that branch and push changes NOT to master but to newly created branch?

But you missed the fork part - you work on a branch but to your repository... so making a fork means copying the repo under your name, you shouldn't clone this repo directly.

Fork it (see upper-right corner of the page for the button)
Clone that to your machine
Make a branch, work on it and push the branch up to your repo (the forked one)
Make a PR.

Your local repository should have 2 remotes:

the original one, from where you get the latest code (you don't have write access there, so no push  there, just pull)
the forked one (yours) where you also push stuff.

At first if might be confusing, but after a few tasks you'll see the pattern. Just keep in mind: never commit on master

@salikjan care to do this one? Add that .travis.yml file (notice the dot at the beginning too), to see the workflow :D

yep. i almost did it. and after that i've read you comment about forking. so... doing again )
just a few minutes

I forked camel. Cloned it. Created new branch named 1. Added .travis.yml file. And commited to branch 1.
Then pushed.

@salikjan ok nice, now make a pull request. Go on your branch and on the right there should be a button

@salikjan This is basically how it should go. Every task should be solved like this, on a sepparate branch :D

I've already cerated PR. Awaiting ;)

A little bit confusing about branches.
How the process looks like?
I have to clone original https://github.com/amihaiemil/camel.git and pull from it?
Or just pull from forked repo?

And i mistyped in .travis.yml.

@salikjan Check your local repo, run the command:
git remote -v, you should see origin: ... - that is your forked repo.
Run
git remote add upstream https://github.com/amihaiemil/camel.git
This should add this repo as a remote to your local one. From time to time, especially now at the beginning, always pull the latest changes so you are sure you are synced

@salikjan there is always this triangle - you pull from this repo to your machine (after adding it as a remote) and you push to your forked repo. Then you make a PR and from your forked repo it gets merged to the original repo.

$ git remote -v
origin  https://github.com/salikjan/camel.git (fetch)
origin  https://github.com/salikjan/camel.git (push)
upstream        https://github.com/amihaiemil/camel.git (fetch)
upstream        https://github.com/amihaiemil/camel.git (push)

And after closing PR i can remove branch created for solving an issue... Yes?

And after closing PR i can remove branch created for solving an issue... Yes?

Yes, sure

$ git pull upstream
You asked to pull from the remote 'upstream', but did not specify
a branch. Because this is not the default configured remote
for your current branch, you must specify a branch on the command line.

@salikjan
To get the latest changes from upstream, run
git fetch upstream
git merge upstream/master
git push

The first 2 commands fetch the latest to your local repo, and git push puts everything in your forked repo.

@salikjan Yes, there are of course more ways of doing the same thing. git pull should also work, it's the direct version of fetch/merge

Ok. Now i see.
I forked repo.
Created local branch from my master. Commited. Pushed. After closing PR i fetch original master branch and make changes to my forked master

@salikjan yes, and the part with

i fetch original master branch and make changes to my forked master

you should also do before starting work, to make sure you're working on the latest stuff.
You'll catch a really good understanding quickly, it's not complicated :)

@amihaiemil i already catched )) it becomes curiouser and curiouser )

@salikjan glad you like it.
I will add some puzzles, think about the architecture and everything.
I will assign you tasks soon (I'll tag you in a comment).
But don't stress about this. It's a project from which all of us can learn (YAML is new to me as well).
I think 15 days deadline for a task should be fine? I'm not sure, just saying.
Nevertheless, this should be something everyone should enjoy and look forward to - don't come home from work and spend nights on it or stuff like that. Whenever you have time and feel like doing it, is fine/
Hopefully others will join as well :)

Ok )

@salikjan chat soon. Cheers and a Happy New Year!


CentOS OCP migration

blocked by https://pagure.io/centos-infra/issue/728

needed only one follow up: https://github.com/theforeman/jenkins-jobs/pull/212/
WOW ;)


using $log.logmethod does not respect log levels

I would expect that if I set the logLevel to {'*', logEnhancerProvider.LEVEL.INFO}, when I call $log.debug('should not see this in the console'), that it would respect the log level.  It is not.
http://plnkr.co/edit/gZw4HdSgtRj6dgahkgQy?p=preview
only the objects from GetInstance() respect the log levels.

I will take a look somewhere this week.

Should be fixed in v1.0.4.


LagLama for Time Serie Classification

Good Morning, and thanks for the open source code. I am trying to use the code for a time series classification task. Do you have some hint on what do I have to change?
Thanks for the answer

Hi, we have not evaluated it for classification before. You could add a classification layer at the end and try to finetune the model.


Fix markdown header parsing

This fixes Markdown getting confused on the ## Quick Start header

@ericbottard what is this?

A fix for this:

Thanks @ericbottard ! This is now merged to gh-pages.


Add abstraction layer for Database and Filesystem

Currently (as of #34) our BadgeHubDataPort is implemented by an adapter that goes directly to the DB.
Before implementing file upload and download, we should move this code to an adapter implementing a port specifically for the Project Metadata.
So then the Idea is that

We add a Project Metadata Repository  Port/Interface, which represents what we store in the database
We add a Project File  Repository Port/Interface, which we represents what we store on disk
The BadgeHubDataPort implementation will use A Project Meta Data Port and Combine it A Project File Port usage.

This way we will have Database specific code encapsulated  as well as file management code and then the implementation details of working with the file system and database are not cluttering the Business logic code.
This way we will also more easily to switch to another way of storing metadata or files.

For naming:
Files:

BadgeAppFiles as port name
NodeFSBadgeAppFiles as adapter name
Metadata:
BadgeHubMetadata as port name
PostgreSQLBadgeHubMetadata as adapter name


Us 14 buy now - User is not logged in

The test case of US-14 and some updates for gitignore

Same issues than before


UnsupportedClassVersionError

I'm trying to use your library with java 7. When I compile, I get
Java unsupported major minor version 52.0
so I believe the jar is compiled with Java 8.
My application needs to be compatible with java 7 and 8. Have you got a suggestion? using 2.0.1?

The maven-compiler-plugin needs to be configured to get a consistent target JVM version:
https://maven.apache.org/plugins/maven-compiler-plugin/examples/set-compiler-source-and-target.html
Currently it isn't configured so will use whatever default method maven-compiler-plugin is setup to use to choose these.

Looking further into it, the author has configured it in the parent pom for all of their projects to be 1.8:
https://repo.maven.apache.org/maven2/de/malkusch/parent/oss-parent/1.10/oss-parent-1.10.pom
However, that can be overriden locally for this project to downgrade it to Java7.
Opened a pull request with a fix for the next version:
https://github.com/whois-server-list/public-suffix-list/pull/10


Alert rules are injected with incorrect label filters

Bug Description
Grafana agent charm has default alert rules for prometheus. However, when the charm is deployed and relate to COS, the alert rules are injected with incorrect JujuTopology (juju_*) labels, and that matches to no timeseries.
Take for example, the HostOomKillDetected alert rule. In Prometheus web UI, it has was modified to the following expr

However, in Prometheus, node_vmstat_oom_kill metric does not have any timeseries with those labels.

So there's a label mismatch between actual metric from grafana-agent via node_exporter and from the alert rule.
To Reproduce

Deploy / with existing COS-lite
Deploy ubuntu charm and grafana-agent machine charm
relate ubuntu and grafana-agent; relate grafana-agent to COS-lite
Open Prometheus web UI, check the alert rules and inspect the expr

Environment
Built from latest source @ 73543ef892912848c3babf062cbd17f8f7e20730, or lastest/edge
Relevant log output
None

Additional context
No response

As we discussed today, this is not a bug, but a design decision which we want to improve.
But at the same time, we came to the conclusion that grafana-agent charm should offer the possibility of adding basic alert rules that are linked with principal application (JujuTopology is provided w/ information about principal application and not grafana-agent).
These rules should not be configurable and should have reasonable thresholds so that they fire only in situations when it is really critical. An example is an alert rule for memory usage, which has a threshold set to 95%. Such a threshold is high and therefore generic, so if any principal application provides another alert rule for memory, it will definitely be with lower threshold (more specific for application).

@rgildein, you are mostly correct. 😅
There is a bug here where the alert rules get the juju_charm label, but the metrics and logs does not. We need to drop this label selector as it will mean that it will never match anything.

You are right juju_charm is redundant and should be removed. However you also need to change labels for juju_application e.g. in Ray example juju_application=grafana-agent should be juju_application=ubuntu.
Also I did not realized that there is no juju_unit in topology, so we will need to add by (instance) to all alert rules, or not?

Three different changes across three different repos are required for this.

[ ] cos-lib: Update cos-lib to give alert rules without juju_charm
[ ] promethus: Update prometheus-scrape and remote-write (and loki-push-api?) to use the alerts class from cos-lib
[ ] grafana-agent: Update grafana-agent to manually insert the principal charm topology


Code cleanup. Parameter encoding.

Readme update for current library usage. Made typealias for parameter type.
Added Encoding protocol for parameter encoding.
Motivation and Context
Code cleanup mostly.
Moved parameter encoding to protocol and structs extending that protocol.
How Has This Been Tested?
Tested on linux and mac os
Checklist:

[x] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

@rob-deans is there any reason for parameter type being an Array of Dictionaries?
And wouldn't it be better to move parameter encoding to protocol and make extensions for URL, JSON and Multipart? I can do it this PR.

The reason was dictionaries don't keep their order and as a result the testing was inconsistent with regards to adding parameters to the url, but if there are other ways we're all ears.
That probably be a better way of approaching this.

Wouldn't the use of .contains method solve this problem, since it's only testing issue? There're hardly any HTTP request that depends on parameters order.

Thanks for the changes, I'll merge now, I'll have a look at the array of dictionaries problem


Widevine no longer works

OS/Platform
macOS
Installed
Other (please use 'Additional context' field below to mention how exactly)
Version
Version 130.0.6723.69 (Official Build, ungoogled-chromium) (arm64)
Have you tested that this is not an upstream issue or an issue with your configuration?

[X] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[X] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
Widevine streams no longer work
How to Reproduce?

Install widevine:

wget -c https://dl.google.com/chrome/mac/universal/stable/GGRO/googlechrome.dmg
hdiutil mount googlechrome.dmg
cp -R /Volumes/Google\ Chrome/Google\ Chrome.app/Contents/Frameworks/Google\ Chrome\ Framework.framework/Libraries/WidevineCdm /Applications/Chromium.app/Contents/Frameworks/Chromium\ Framework.framework/Libraries/WidevineCdm
hdiutil unmount /Volumes/Google\ Chrome
rm googlechrome.dmg

Go to https://bitmovin.com/demos/drm

Actual behaviour
"The MediaKey object could not be created/initialized
(DRM_MEDIA_KEY_INITIALIZATION_FAILED)"
Expected behaviour
Stream works
Relevant log output
[25849:259:1103/024844.951550:ERROR:cdm_module.cc(115)] Error: dlopen(/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib, 0x0001): tried: '/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib' (code signature in <4C4C44E5-5555-3144-A15E-9CF13BC16D64> '/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib' not valid for use in process: mapping process and mapped file (non-platform) have different Team IDs), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib' (no such file), '/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib' (code signature in <4C4C44E5-5555-3144-A15E-9CF13BC16D64> '/Applications/Chromium.app/Contents/Frameworks/Chromium Framework.framework/Versions/130.0.6723.69/Libraries/WidevineCdm/_platform_specific/mac_arm64/libwidevinecdm.dylib' not valid for use in process: mapping process and mapped file (non-platform) have different Team IDs)

Additional context
installed via brew - https://formulae.brew.sh/cask/eloston-chromium#default

Meanwhile, a very non-ideal workaround:
wget -c https://dl.google.com/chrome/mac/universal/stable/GGRO/googlechrome.dmg -O /tmp/googlechrome.dmg
hdiutil mount /tmp/googlechrome.dmg
cp -R /Volumes/Google\ Chrome/Google\ Chrome.app/Contents/Frameworks/Google\ Chrome\ Framework.framework/Libraries/WidevineCdm /Applications/Chromium.app/Contents/Frameworks/Chromium\ Framework.framework/Libraries/WidevineCdm
hdiutil unmount /Volumes/Google\ Chrome
rm /tmp/googlechrome.dmg
xattr -cr /Applications/Chromium.app
codesign --force --deep -s - /Applications/Chromium.app

Meanwhile, a very non-ideal workaround:
wget -c https://dl.google.com/chrome/mac/universal/stable/GGRO/googlechrome.dmg -O /tmp/googlechrome.dmg
hdiutil mount /tmp/googlechrome.dmg
cp -R /Volumes/Google\ Chrome/Google\ Chrome.app/Contents/Frameworks/Google\ Chrome\ Framework.framework/Libraries/WidevineCdm /Applications/Chromium.app/Contents/Frameworks/Chromium\ Framework.framework/Libraries/WidevineCdm
hdiutil unmount /Volumes/Google\ Chrome
rm /tmp/googlechrome.dmg
xattr -cr /Applications/Chromium.app
codesign --force --deep -s - /Applications/Chromium.app

This command works on Bitmovin's DRM demo, but it still results in a license error on the Spotify web player.

Spotify isn't expected to work, they have DRM requirements we cannot meet


Would you mind turning this into an article?

e.g. for towardsdatascience.com ?

This is a very good working example with Neo4J and OLAP. I am curious. Based on your working through of this example, would you consider Neo4J for an OLAP project? Just curious.

This is a very good working example with Neo4J and OLAP. I am curious. Based on your working through of this example, would you consider Neo4J for an OLAP project? Just curious.

As I mentioned in the repository presentation, my interest (and past involvement) in OLAP technologies (especially Microsoft SQL Server Analysis Services) lead me to experiment similar concepts with graph databases (where relationships are a key concept).
This must be seen as a proof of concept, and an opportunity to better understand graph databases.
One thing is sure: each time I must address a relation-oriented problem, I always consider it with graph databases (and obviously Neo4j) in mind.
I you are interested in analytics and graph, I heard of AnzoGraph solution (note I never took time investigated it).


Error comes while Adding LazyLoading in routes for Custom Library

I am creating a angular library, having routes as well module specific structure which are having their own routing, as we creates in App/SRC.
Below is the code description:-
Library Code:-
ProjectNameModuleRoutingFile:-
const routes: Routes = [ { path: '', redirectTo: 'role', pathMatch: 'full' }, { path:'new', component:CoreUserManagementComponent }, { path: 'role',loadChildren:'./role-management/role.module#RoleModule' } ]
@NgModule({ imports: [RouterModule.forChild(routes)], exports: [RouterModule] })
Applicarion Src :-
AppRoutingFile :-
const routes: Routes = [ { path:'test', loadChildren: () => import('project-name').then((res) => res.ProjectNameModule) } ]
@NgModule({ imports: [RouterModule.forRoot(routes)], exports: [RouterModule] })
Here while creating build of the library, no error comes, also no error comes while running ng serve for the application after adding the library. But while accessing the route from the browser, for which i have added library, it throws error like:-
"core.js:15723 ERROR Error: Uncaught (in promise): Error: Cannot find module './role-management/role.module'"
Thanks

Lazy loading should happen at an application and not a library level.
See the last 2 post in https://github.com/angular/angular-cli/issues/6373#issuecomment-453006158 for more context.

So Is there no substitute available for such type of scenario?
Like can we mention something library path in projects/project-name/ng-package.json, so that It can we use, something like this. So that this error can be removed "Error: Cannot find module".


Allow UseRecordsResult.fetchData to accept options

Describe the problem feature solves
It's not possible to set query params, e.g., page, per page, when making a request for resources when using the hook in a custom component.
Describe the solution you'd like
Pass options from fetchData to the resourceAction and fall back to current behavior.
Describe alternatives you've considered
The alternative is to use the ApiClient directly and manually specify the params.
Acceptance criteria

[ ] pass options from fetchData to resourceAction with current behavior as a fallback if option is blank

fetchData reads these from search params so you can achieve this by setting perPage, etc in your browser's address bar. We could technically allow to pass custom options though.


Series.map with a dict

We should add https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Series.map.html#pandas.Series.map
With just handling dictionary of simple types for now. It should be a simple wrapper around the pandas implementation for the time being. This is useful for simple value substitution.

Resolved by #389.


Fix invalid phpdoc for casted property without return (#1236)

Summary
Fix invalid phpdoc for casted property without return type
Type of change

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

Checklist

[ ] Existing tests have been adapted and/or new tests have been added
[ ] Add a CHANGELOG.md entry
[ ] Update the README.md
[ ] Code style has been fixed via composer fix-style

Yes, I can try add some test for this case


foundation-cli run errors

Used npm install --global foundation-cli to install foundation cli (received 0 error messages during the install) however with foundation new I get the following error:
/Users/mappelman/.rvm/rubies/ruby-2.2.3/lib/ruby/site_ruby/2.2.0/rubygems/dependency.rb:315:in `to_specs': Could not find 'foundation' (>= 0) among 171 total gem(s) (Gem::LoadError)
Checked in 'GEM_PATH=/Users/mappelman/.rvm/gems/ruby-2.2.3:/Users/mappelman/.rvm/gems/ruby-2.2.3@global', execute `gem env` for more information
	from /Users/mappelman/.rvm/rubies/ruby-2.2.3/lib/ruby/site_ruby/2.2.0/rubygems/dependency.rb:324:in `to_spec'
	from /Users/mappelman/.rvm/rubies/ruby-2.2.3/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_gem.rb:64:in `gem'
	from /Users/mappelman/.rvm/rubies/ruby-2.2.3/bin/foundation:22:in `<main>'
	from /Users/mappelman/.rvm/gems/ruby-2.2.3/bin/ruby_executable_hooks:15:in `eval'
	from /Users/mappelman/.rvm/gems/ruby-2.2.3/bin/ruby_executable_hooks:15:in `<main>'

Anyone that could give me a hand with fixing this please?

Your system's PATH puts Ruby gems before Node modules, and because you have the old CLI installed, the foundation command is still pointing to it.
If you can live without the Foundation 5 CLI (it's only needed to set up new projects, not run existing ones), run gem uninstall foundation to remove it. Then the Foundation 6 CLI will work.
If you anticipate needing both in the near future, there's a way to create a command alias that points to the F6 CLI. I can break that down for you if you'd like.

Running gem uninstall foundation did not do much, still get those same errors as above. If you could point me to some docs on how to fix that, that be great.

I'm getting the same issue too! Ran gem uninstall foundation but still get the error :(

Right when you uninstall something, sometimes you have to open a new command line window to get everything up to date. Also, if you're using rbenv, you have to run rbenv rehash to update global gem commands. I'm not sure if RVM does the same thing.

I still get the same error. I think the gem is cached or something... I can;t get the CLI to work...

@rockstarwebdesign I got the CLI working by doing the following:
gem uninstall foundation
rvm get stable
rvm reset
npm install --global foundation-cli

My rvm is somehow not working or not installed. Seems weird... do you know how can I solve that? Do I need to reinstall OS X again?


golang filter: DecodeData called twice - last time w/ empty buffer

golang filter: DecodeData called twice - last time w/ empty buffer
Disclaimer: I'm new to Envoy filter writing.
I have a Golang filter that modifies the contents the body.  My DecodeData method reads the bytes from the buffer, alters them such that the body is a different size, then calls Set(modifiedBody) on the buffer and returns api.Continue.   I notice two things.

the body is not modified
DecodeData is called twice, the first integer is the length of the buffer

DecodeData(1090, false)
DecodeData(0, true)

I'm not sure if this is a bug or ignorance on my part.  I'm happy to create a self contained example if what I'm reporting seems bizarre.

DecodeData can definitely be called multiple times. The boolean flag (endStream) indicates when you're receiving the last call for a given request. I'm not certain if it always makes a final call of DecodeData(0, true), but it's probably best to handle either case as the behavior may change in the future. You should also assume that you might receive multiple calls with data and endStream false.
BufferInstance.Set appears to be implemented. I tested it quickly by modifying the go http filter example and it worked as expected. Note that modifying data in DecodeData will modify the request being sent to the upstream server (if any). If you want to modify the response to the client you need to implement StreamEncoderFilter and modify data in EncodeData, as in the example.

Thanks!  I'll take a look at my code.

Ok, I double checked and the upstream server is not getting the modified body.  I added a "debug" upstream filter that logs the contents of the body and it's not getting a modified body either.

body data modification should be work, please provide more information.
yep, a self-contained example would be better.


Adding a “last updated” page with the latest changes [on doc]

This issue follows this thread : https://discourse.gohugo.io/t/adding-a-last-updated-page-with-the-latest-changes-on-doc/10412 and request from bep.

It is difficult to know what are the latest improvements on the documentation.
We could add a page : “latest changes” with the latest X changes on documentation.

So

I posted on the above thread a code attempt for a shortcode
@bep added the need for a  similar page with all TODO (via a new shortcode)
He propose to create a new section (name?) with a simple list template?

This belongs in the Hugo Docs repo, not here.

Oops. I change it. Sorry for the noise


URL Hash Change on Page Change

I can't find a solid answer to this, but I think it would be helpful if some gave a go to answer for this.
Basically what I am looking for is when I go to page 2, the url changes to url.com?page=2.
This I accomplished by
$scope.pageChanged = function(pageNumber) {
  $location.search({page: pageNumber});
  updateQueryString({
    page: pageNumber
  });
};

The issue I am having is when I click back, the URL changes, but the content and the number bar at the bottom does not update.
I came up with the code below, but am having issues with setting the $scope.currentPage within the watch. Any ideas:
   //Detect page param and set the currentPage to that on page load.
    if($location.search().page){
        $scope.currentPage = $location.search().page;
    }

    //This doesn't seem to work, but the idea here is on change of page param, it will set the currentScope to that param.
    $scope.$watch(function(){ return $location.search() }, function(params){
        if($location.search().page){
            $timeout(function() {
                $scope.currentPage = params.page;
            });
        }
    });

Hi,
Your solution is conceptually fine. Since we cannot use routing to take care of this, we do need to wire up the $watch ourselves.
Looking at your code, I think the $watch is not working because your watch function returns the $location.search() object. The Angular change detection is performing an equality check based on object reference, and is seeing the same object reference each time, so is not triggering the handler. For more detail see: http://stackoverflow.com/a/19455564/772859
Try either:

returning $location.search().page, which will compare the value rather than the reference.
setting the third argument to true. This solution is theoretically less performant than the first however.

In either case, you shouldn't need the $timeout.

$location broadcasts events whenever the URL changes (also fires when the user presses back).
You can watch these events with $scope.on(...). Here is an example:
$scope.$on('$locationChangeSuccess', function(){
        if($location.search().page){
                $scope.currentPage = params.page;
        }
    });

See https://docs.angularjs.org/api/ng/service/$location for more information on the events.


Vertica hotfix version

The "-*" part of a Vertica version is related to its hotfix number. There is no hotfix number when we internally compile Vertica from source. This PR makes having hotfix number optional in VerticaPy so that it would be easier for us when we work with VerticaPy against an in-development Vertica.
This closes #674.

The tests are failing please look why.


MM-12038 Fix attachment icon styling

Summary

Fix attachment icon background-color.
Fix attachment icon alignment when downloading.

Ticket Link
https://mattermost.atlassian.net/browse/MM-12038
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Has UI changes

Device Information
This PR was tested on: Emulators (IOS and android)
Screenshots

@hmhealey Addressed review comments


Appending a ParamList object with a Parameterized object?

Hello,
I am trying to append a ParamList object with a Parameterized object. I get the following error:
ValueError: Non parameter type cannot be appended to the list.

I found this a bit strange as I am able to initialise a ParamList object with a list of Parameterized objects, but cannot append it further. Is there a reason behind this behaviour or is it just a bug?
Thanks!

Should be solved by #645


Customize default request body when executing function

When clicking "Execute Function Now", the request body is set to { "name": "Azure" } by default.

There is sample.dat in function directory but it looks like it doesn't read it. I checked the src code. It looks like tryGetSampleData have to read it, but it doesn't.
Is it possible to customize this default value?

Currently it's not possible to customize the default value. However, this is a good idea and we'll see if we can include it in the next feature release.

There is sample.dat in function directory but it looks like it doesn't read it. I checked the src code. It looks like tryGetSampleData have to read it, but it doesn't.

Just to clarify, this function gets the sample data from the template provider. The templates themselves have sample files associated to them including a sample.dat. We never intended to let users provide custom sample.dat files.
I think we would need to spend some time to design this feature.  For example, Execute Function Now... can also be used for remote triggers. In those cases, would it make sense to use a sample.dat or should this be exclusively for local projects?
@oleksandr-ivanov-ah Where would you expect the sample.dat to live in your project?

Should it be in the root of the Functions project?
Should it be in the individual folder of each Function trigger?

The issue with option 1 is that we wouldn't know which custom sample data to use for each trigger. I think we could structure the sample.dat to be a JSON file that lives in the root, and the keys would be the trigger names that should be used as the custom default request bodies.
Issue with option 2 is that not all function runtimes separate the triggers into folders. I think that we could still enable this scenario, but fall back to option 1 if either:

There is no sample.dat in the function's folder
Functions aren't structured in such a way that they are in individual folders

This feature request is now a candidate for our backlog. The community has 240 days to upvote the issue. If it receives 5 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Ah, I was similarly confused by sample.dat and assumed it was an error when "Enter request body" did not grab data from it. (This codebase does seem to reference sample.dat but perhaps I'm not following how it's using it).
Is there a reason to include the generated sample.dat file in the function if developers cant really use it themselves? (i.e. could it be excluded for now so that it feels less like a bug.. I edited that file so many times!)

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

@nturinski i would prefer to have sample.data in function app root folder. Like this:

I thin json format is preferable: sample.json

Just stumbled upon this when implementing a Telegram Bot via Azure Functions. The webhook called via Telegram has a specific JSON structure in its body. It's possible to copy/paste a sample into VSCode everytime I wanna test locally, but being able to just define a sample.json (or sample.dat) that includes the sample data would make it a lot easier to work efficiently.
A workaround for this I'm currently looking into is to implement the suggested structure for unit tests and instead of testing the full function locally fall back to running a unit test with a mock HttpRequest.

I have been using Postman to call my locally running function. That way I don't have to set up the input every time. It still would be nice to just use VS Code and change the default value.

Just wanted to offer this for anyone still looking for a good solution, but my preferred way to make quick and reusable HTTP requests inside VS Code is to use the REST Client extension. https://marketplace.visualstudio.com/items?itemName=humao.rest-client


Add Google Drive service/recipe

Would it be possible to add Google Drive as a service/recipe onto Franz?
Thanks!
Franz Version used: 5.0.0.beta.14
Operating System and version: Windows 10 Pro Fall Creators Update

This repository is only for the Franz client. Please use this form for service requests or check out the guide to create your own service integration.


How can I use this?

There are little to no instructions.

I'm pretty sure it works nearly the exact same as https://github.com/haoheliu/versatile_audio_super_resolution/ but it was just changed to work with the applio voice conversion software.

Oh, thanks for the tip! I got the original to work (I only have the problem that the output is slowed down and lower pitched for some reason). I'll give the project a try in the next few days, the longer input definitely sounds useful, so thanks for the response! :)

If its changed pitch and length that means the sampling rates are mismatched.

If its changed pitch and length that means the sampling rates are mismatched.

At the end we found out that the problem is only the Gradio UI demo having this problem weirdly


Fixes table & mapping creation commands

Addresses #26323

@i3arnon : Thanks for your contribution! The author, @orspod, has been notified to review your proposed change.

@i3arnon @orspod Thanks for assisting.
One extra tip - when you are addressing a GitHub issue, there is a keyword fixes that you can use on the commit comments or on the PR title to auto-resolve the related issues.
For this thread it will be Fixes MicrosoftDocs/azure-docs#26323

#sign-off

@JasonWHowell: I'm sorry - only the author of this article, @orspod, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Strikethrough Like this. crashes wysiwyg

Description
wysiwyg crashes if you add the following line in your markdown:
Strikethrough uses two tildes. ~~Scratch this.~~
Steps to reproduce
create a markdown file with the following contents
Strikethrough uses two tildes. ~~Scratch this.~~

open the page and click edit (using inline editing and the wysiwyg)
Expected result
you should be able to edit the page
Actual result
What actually happened? The page crashes giving many errors. including the following.
ErrorEvent {isTrusted: true, message: "Uncaught Error: Token type `s_open` not supported by Markdown parser", filename: "http://localhost:8000/commons.js", lineno: 6853, colno: 71000, …}      in AppContainer (created by HotExportedComponent)     in HotExportedComponent (created by PageRenderer)     in PageRenderer (at query-result-store.js:86)     in PageQueryStore (at root.js:51)     in RouteHandler (at root.js:73)     in EnsureResources (at root.js:61)     in LocationHandler (at root.js:119)     in LocationProvider (created by Context.Consumer)     in Context.Consumer (created by Location)     in Location (at root.js:118)     in Root (at root.js:126)
Environment
Tinacms Package Versions
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>

Adding a gif to prove I'm not lying :)

Gif for clarity

@PaulBunker I'm not able to reproduce this in demo-gatsby or demo-next. Could you check if it has been resolved?

I did a quick test and I get a blank screen when adding Strikethrough uses two tildes. ~~Scratch this.~~ in a post on Gatsby demo in Safari... can't even open the developers tools.

I think this is just the screen of death issue but I happened to get it adding 'tildes'  I reproduced using the letter 'h' a few times just now.

In my case, it was OK, and it went AWOL just when adding the double tildes. Trying to get rid of this screen of death so see if tildes are really responsible for this.

Closing for now could be a different cause


Emit make-compatible dependency file

Adds a --dependency-file FILE option to WDLC to permit emission of a make-compatible dependency file a given Proto input.

@googlebot I signed it!


Added prelease warning

Added a warning to avoid user issues such as the complaints in #107 and similar issues.

No need any more...
Released v3


Document release process

Acceptance Criteria

[x] Release process documented in a markdown file in the repository

resolved by #114


Update CodeQL version in CI config

CI logs have a warning

This version of the CodeQL Action was deprecated on January 18th, 2023, and is no longer updated or supported. For better performance, improved security, and new features, upgrade to v2. For more information, see https://github.blog/changelog/2023-01-18-code-scanning-codeql-action-v1-is-now-deprecated/

We should update CodeQL vesrion in codeql-analysis.yml

Hmm... What're your thoughts on removing codeql-analysis.yml altogether, and enabling it via the repository settings, instead?

https://github.com/hamirmahal/FilmFinder/ doesn't have a codeql-analysis.yml file, but it still has CodeQL checks run occasionally, anyway.

https://github.com/hamirmahal/FilmFinder/tree/main/.github/workflows

I just found out that we can run CodeQL with no config and it will auto detect most suitable config. I will try it on days, thanks!


Fixes #10556: considering non open files dirty count using file dirty event

Fixes #10556. Using the same kind of event listener thats been used to update ActivityBadge.

Hi @ramamurthynagaraj, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Coverage remained the same at 62.554% when pulling 2c53f3b5d42be3777eacc0acb3e2209f1756a5da on ramamurthynagaraj:issue_10556 into ae1d54571607c779fd06933786983afafce71938 on Microsoft:master.

Looks goot to me - thanks :beers:


Export @vitejs/plugin-legacy options interface

Description
As a developer building a wrapper library around @vitejs/plugin-legacy (nuxt-vite-legacy), I need to re-export the type @vitejs/plugin-legacy uses to accept user options, because I need to allow a user to configure @vitejs/plugin-legacy through nuxt-vite-legacy.
Currently, @vitejs/plugin-legacy only exports this:
export { cspHashes, viteLegacyPlugin as default, detectPolyfills };

I need it to also export interface Options from packages/plugin-legacy/src/types.ts:
https://github.com/vitejs/vite/blob/2687dbbd4e19c86f9888ee784c9b51598e8b79ca/packages/plugin-legacy/src/types.ts#L1-L5
Without that, I have to copy/paste the type into my project, which is not only sub-optimal but also plain wrong. My library supports<EMAIL_ADDRESS>(that is 4.0 and 4.1 at the moment), and the actual version may vary depending on user's project. If I copy/paste the options type from 4.1, this gives false expectations (advertise unsupported renderModernChunks) to users using 4.0, and if I copy/paste the type from 4.0, this will not expose renderModernChunks to users that have upgraded to 4.1.
Suggested solution
In packages/plugin-legacy/src/index.ts we could do:
export type { Options } from "./types"

Alternative
No response
Additional context
If it were the application code, I could derive the type with:
import legacy from "@vitejs/plugin-legacy"
type Options = Exclude<Parameters<typeof legacy>[0], undefined>

However, that only works for the library itself, this doesn't allow to re-export the type for library users.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think we can have it. A PR is welcome.


Moto X 2014 Framework Flags Error

Information

Apktool Version (apktool -version) - 2.2.0-SNAPSHOT
Operating System (Mac, Linux, Windows) - Windows
APK From? (Playstore, ROM, Other) - Stock ROM

Stacktrace/Logcat
W: Config flags size > 52. Size = 56. Exceeding bytes: 0x16662214.
W: Invalid config flags detected: string-mcc204-mnc4-ERR0
W: Config flags size > 52. Size = 56. Exceeding bytes: 0x16662214.
W: Invalid config flags detected: array-mcc204-mnc4-ERR1
W: Config flags size > 52. Size = 56. Exceeding bytes: 0x16662214.
W: Invalid config flags detected: bool-mcc204-mnc4-ERR2
I: Framework installed to: C:\Users\heheoops\AppData\Local\apktool\framework\1-2014.apk
Steps to Reproduce

apktool if -t 2014 framework-res.apk

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app)
Rename the .zip to .apk
framework-res.zip

Hmm. Not good at all. It has a 56 byte header for the configurations. This is what I'm expecting for Android Nougat, I just don't know what the 4 extra bytes are yet.
This however is a 2014 framework so either 1) They used new build tools, thus 56 byte header or 2) changed something in resource configurations to cause the difference.
Will investigate after Nougat source drops - #1223

So I just retried with 2.2.1-81b22d-SNAPSHOT and the issue seems to have been resolved! 👍
apktool if -t 2014 framework-res.apk
I: Framework installed to: C:\Users\JJ\AppData\Local\apktool\framework\1-2014.apk
Fantastic work!

The 4 bytes are still unknown however. Glad it worked, but its more of a band aid then permanent solution.


fixed: ChatAnthropic gives error while invoking it after instantiating with configurable_fields

PR title: "fixed: ChatAnthropic gives error while invoking it after instantiating with configurable_fields"

community: fixed error while invoking ChatAnthropic with configurable_fields

PR message: The get_pydantic_field_names in langchain_core.utils was not able list _client and _async_client.

Description: Variable names changed.
Issue: #20061
Dependencies: NA
Twitter handle: @mynksol

@efriis @baskaryan Please check


Alternatives to engine.flushAndWait()

I'm currently using Filament to render to shared textures and then sending the result off to another library that ultimately displays those textures on a screen.
I'm calling engine.flushAndWait() to make sure the texture render has finished, but I would rather do it through some sort of callback. Is that an option in Filament? I guess this is related to fences, which I never really got my head around.

A fence is what you want except that we don't support GPU fences anymore (in the public API that is). The current fences will only tell you that the render thread has executed all the commands up until the fence.

This is not a supported use case right now. flushAndWait will work, however it is very heavy, it will stall both the current thread and the gl thread until the texture is rendered. Alas, this is the only solution right now.

☹ thanks for the info anyway.

You could also use the async version of readPixels maybe -- this would stall less.


Add a square fill mode to GradientTexture2D

Feature proposal. Draft because I'm not sure about the best implementation yet.
Usage
Square gradient fill could replace the need to make certain common textures in an image editor.
Vignette around the edges of a square (can also be a rectangle, like your screen):

Checkpoint from a retro game:

Current implementation
Currently, I'm going with a rectangle whose halved diagonal goes from FROM to TO.

But maybe keeping it a square or having a rotated square would be better?

Also, the diagonals in cases like the one above are pretty jarring. It's a brain illusion that makes you think they are darker. Maybe something could be done about that, but I think the square gradient is best kept simple.
Feel free to discuss about better implementations, also if you think the feature isn't needed - this is why I've left the PR as a draft.
(Yeah, the PR description took way longer than the PR itself...)

Heh, I was planning on working on this soon :slightly_smiling_face:
I would suggest opening a proposal to discuss use cases. I can think of a few:

Create blob shadow texture for square-shaped objects.
Create grid textures (useful for prototyping).

I don't think this can be used for checkerboards.

Indeed, this can only be used to draw grids. Still, this may come in handy for prototype textures when you don't want to reach out for https://godotengine.org/asset-library/asset/780.

Changed it up, the new implementation forces it to be a square, making it more alike radial textures (it can still be a rectangle if you resize the texture)

Thanks!


pass a view model instead of json data

I'm enjoying using handlebars.  I'm using it along with backbone for mvc and coffeescript.
When I render a template I typically have a (view) model in hand.  I'm converting it to JSON before I render the template.  I would prefer to pass the model to handlebars.  For example...
PersonView: class PersonView extends Backbone.View

  render: ->
    # instead of ...$(@el).html templates.person(@model.toJSON())...would prefer...
    $(@el).html templates.person(@model)

this would allow me to define derived properties in the model class
 fullName: ->
    @firstName + ' ' + @lastName

that I could reference in my handlebars template
 Full Name: {{#fullName}}

Something similar has been done for SproutCore: https://github.com/sproutcore/sproutcore20/tree/master/packages/sproutcore-handlebars. It should be done as an add-on to handlebar, not part of handlebars itself.


Event attributes trigger event twice in HTML encoding

Using eventattributes such as
  <mesh src="teapot.json" onclick="console.log('Teapot');"></mesh>

trigger twice. This affects only HTML encoding and event attributes. Using addEventListener or the same scene in XHTML are not concerned.

I've added checks to the mouse event attributes polyfill that will prevent it from adding a second event listener to the element in HTML documents.


Trigger click link when drag the carousel

when drag the carousel, it always trigger click link if the slide items is anchor tag

Thanks @panacholn for reporting.
Could you provide more data?

OS with version
browser with version
react-carousel version
code using react-carousel
a video or at least some screenshots

Also seeing this behavior. Can I hook the drag event of this component so that I can check to see if it's dragging and prevent my click?
Codesandbox:
https://codesandbox.io/s/dreamy-framework-4it5h

@johnatrevcontent
in the codesandbox, after dragging, I see www.google.com refused to connect. - is there the same behaviour on your side?

@piotr-s-brainhub
Yes, I see the same behavior from it. I'm looking for a way to prevent that click when drag finishes if possible. Thanks for looking into this.

I just validated it on https://brainhubeu.github.io/react-carousel and it's fixed.

@panacholn @johnatrevcontent
Can I ask you to fill in this form https://docs.google.com/forms/d/1tfRs1Io49tJBwz-06aDnz8hrJ3WySJTi45y4k6hHGaQ ?

@CHaNGeTe
I have a question: are you able to withdraw any funds from the issuehunt?
I ask because after writing to<EMAIL_ADDRESS>asking them for an invoice and waiting more than one week, I still have no reply.
And in order to withdraw any funds, it's required to send an email to<EMAIL_ADDRESS>as well.


Where to find the Objective function in the code

In 4.3 of the paper, objective function is used to calculate the weight of each expert, I guess it is in test_all.py, but I can't find it.
Please tell me the spefic position in the code, thank you!

Hi test_all.py is a file to test models. Please refer to test_training files.

I find it, thanks!


Unable to install genomes

Greetings,
I've been attempting to install bcbio quite a few times but it always end up failing when trying to install the genome files.
The below is the error I keep getting.
I'm wondering if bcbio will work if I take an existing genomes directory containing the genomes, and modify the .loc files in the galaxy sub-directory? Thank you.
The basic installation works:
python ./bcbio_nextgen_install.py /mnt/projects/XXX/wgs/tools/bcbio/1.2.3 --tooldir=/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/tools --aligners bwa
But the error appears when I try to install the genomes:
bcbio_nextgen.py upgrade -u skip --genomes GRCh37 --genomes hg38 --genomes mm10
2020-08-18 08:25:19 (246 KB/s) - ‘variation/dbsnp-153-orig.vcf.gz.tbi’ saved [2998587/2998587]
Writing to .
Could not read: https://gist.githubusercontent.com/matthdsm/f833aedd2d67e28013ff1d171c70f4ee/raw/442a45ed3ddc6e85c66c5e58e0fa78e16a0821c8/refseq2ucsc.tsv
[E::bcf_hdr_read] Input is not detected as bcf or vcf format
Could not read VCF/BCF headers from -
Cleaning
tbx_index_build failed: variation/dbsnp-153.vcf.gz
Traceback (most recent call last):
File "/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/anaconda/bin/bcbio_nextgen.py", line 228, in
install.upgrade_bcbio(kwargs["args"])
File "/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/anaconda/lib/python3.6/site-packages/bcbio/install.py", line 107, in upgrade_bcbio
upgrade_bcbio_data(args, REMOTES)
File "/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/anaconda/lib/python3.6/site-packages/bcbio/install.py", line 377, in upgrade_bcbio_data
args.cores, ["ggd", "s3", "raw"])
File "/home/XXX/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 354, in install_data_local
_prep_genomes(env, genomes, genome_indexes, ready_approaches, data_filedir)
File "/home/XXX/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 480, in _prep_genomes
retrieve_fn(env, manager, gid, idx)
File "/home/XXX/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 875, in _install_with_ggd
ggd.install_recipe(os.getcwd(), env.system_install, recipe_file, gid)
File "/home/XXX/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 30, in install_recipe
recipe["recipe"]["full"]["recipe_type"], system_install)
File "/home/XXX/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 62, in _run_recipe
subprocess.check_output(["bash", run_file])
File "/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/anaconda/lib/python3.6/subprocess.py", line 356, in check_output
**kwargs).stdout
File "/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/anaconda/lib/python3.6/subprocess.py", line 438, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bash', '/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/genomes/Hsapiens/hg38/txtmp/ggd-run.sh']' returned non-zero exit status 1.
[XXX@n111 ~]$ bcbio_nextgen.py upgrade -u skip --genomes GRCh37 --genomes hg38 --genomes mm10

Hi @gis-nlsim !
Yes, you may just symlink
/oldbcbio/genomes -> /newbcbio/genomes
/odbcbio/galaxy/tool-data /newbcbio/galaxy/tool-data
instead of reinstalling data.
Is it https://github.com/chapmanb/cloudbiolinux/blob/master/ggd-recipes/hg38/dbsnp.yaml recipe that fails?
We need to fix it then.
Can you show the script in /mnt/projects/XXX/wgs/tools/bcbio/1.2.3/genomes/Hsapiens/hg38/txtmp/ggd-run.sh
Sergey

Thank you for your quick reply, as requested, the ggd-run.sh script is as follows:
[XXX@n111 txtmp]$ cat ggd-run.sh
#!/bin/bash
set -eu -o pipefail
export PATH=/mnt/projects/XXX/wgs/tools/bcbio/1.2.3/tools/bin:$PATH
build=153
version=GCF_000001405.38
url=http://ftp.ncbi.nih.gov/snp/archive/b$build/VCF/$version.gz
remap_url=https://gist.githubusercontent.com/matthdsm/f833aedd2d67e28013ff1d171c70f4ee/raw/442a45ed3ddc6e85c66c5e58e0fa78e16a0821c8/refseq2ucsc.tsv
ref=../seq/hg38.fa
mkdir -p variation
wget -c -O variation/dbsnp-$build-orig.vcf.gz $url
wget -c -O variation/dbsnp-$build-orig.vcf.gz.tbi $url.tbi
[[ -f variation/dbsnp-$build.vcf.gz ]] || bcftools annotate -Ou --rename-chrs $remap_url variation/dbsnp-$build-orig.vcf.gz |
bcftools sort -m 1G -Oz -T . -o variation/dbsnp-$build.vcf.gz &&
tabix -f -p vcf -C variation/dbsnp-$build.vcf.gz
tabix -f -p vcf variation/dbsnp-$build.vcf.gz
Regards,
Ngak Leng
From: Sergey Naumenko<EMAIL_ADDRESS>Reply to: bcbio/bcbio-nextgen<EMAIL_ADDRESS>Date: Wednesday, 19 August 2020 at 4:10 AM
To: bcbio/bcbio-nextgen<EMAIL_ADDRESS>Cc: SIM Ngak Leng<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [bcbio/bcbio-nextgen] Unable to install genomes (#3326)
Hi @gis-nlsimhttps://github.com/gis-nlsim !
Yes, you may just symlink
/oldbcbio/genomes -> /newbcbio/genomes
/odbcbio/galaxy/tool-data /newbcbio/galaxy/tool-data
instead of reinstalling data.
Is it https://github.com/chapmanb/cloudbiolinux/blob/master/ggd-recipes/hg38/dbsnp.yaml recipe that fails?
We need to fix it then.
Can you show the script in /mnt/projects/XXX/wgs/tools/bcbio/1.2.3/genomes/Hsapiens/hg38/txtmp/ggd-run.sh
Sergey
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bcbio/bcbio-nextgen/issues/3326#issuecomment-675688205, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMRXQUNW776S46VC5WJ4P5TSBLN4ZANCNFSM4QDBRLZA.
This e-mail and any attachments are only for the use of the intended recipient and may contain material that is confidential, privileged and/or protected by the Official Secrets Act. If you are not the intended recipient, please delete it or notify the sender immediately. Please do not copy or use it for any purpose or disclose the contents to any other person.

Hi @gis-nlsim !
Sorry about the delay.
The files look available:

https://ftp.ncbi.nih.gov/snp/archive/b153/VCF/GCF_000001405.38.gz
https://gist.githubusercontent.com/matthdsm/f833aedd2d67e28013ff1d171c70f4ee/raw/442a45ed3ddc6e85c66c5e58e0fa78e16a0821c8/refseq2ucsc.tsv

Maybe it was a temporary Github glitch?
Were you able to run it the recipe? What happens if you run ggd-run.sh?
Sergey

install.error.txt
Sorry, but I have yet to successfully install bcbio. Attached is the entire output from the bcbio installation process. Seek your assistance in this.
I have tried installing on 2 different servers (from 2 different institutions), both without success. So I don't think it's an issue with the servers on my side. Thank you.

Hi @gis-nlsim!
Sorry about the continuing issues!
I just did two fresh installations of bcbio development instances successfully.
From your log:
Traceback (most recent call last):
  File "bcbio_nextgen_install.py", line 290, in <module>
    main(parser.parse_args(), sys.argv[1:])
  File "bcbio_nextgen_install.py", line 51, in main
    subprocess.check_call([bcbio, "upgrade"] + _clean_args(sys_argv, args))
  File "/home/projects/13001264/tools/**bcbio/v1.1.9**/anaconda/lib/python3.6/subprocess.py", line 291, in check_call
    raise CalledProcessError(retcode, cmd)

Is it possible that you are mixing two bcbio installations? The old bcbio should not be in the PATH when installing the new one!
Sergey

Yes I have an older version that we’re using for production. I’ll remove that from the path and try installing again. Thanks!
From: Sergey Naumenko<EMAIL_ADDRESS>Reply to: bcbio/bcbio-nextgen<EMAIL_ADDRESS>Date: Thursday, 5 November 2020 at 11:51 AM
To: bcbio/bcbio-nextgen<EMAIL_ADDRESS>Cc: SIM Ngak Leng<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [bcbio/bcbio-nextgen] Unable to install genomes (#3326)
Hi @gis-nlsimhttps://github.com/gis-nlsim!
Sorry about the continuing issues!
I just did two fresh installations of bcbio development instances successfully.
From your log:
Traceback (most recent call last):
File "bcbio_nextgen_install.py", line 290, in
main(parser.parse_args(), sys.argv[1:])

File "bcbio_nextgen_install.py", line 51, in main
subprocess.check_call([bcbio, "upgrade"] + _clean_args(sys_argv, args))

File "/home/projects/13001264/tools/bcbio/v1.1.9/anaconda/lib/python3.6/subprocess.py", line 291, in check_call
raise CalledProcessError(retcode, cmd)

Is it possible that you are mixing two bcbio installations? The old bcbio should not be in the PATH when installing the new one!
Sergey
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bcbio/bcbio-nextgen/issues/3326#issuecomment-722124638, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMRXQULXCBL5OF4DW3IFM63SOIOLVANCNFSM4QDBRLZA.
This e-mail and any attachments are only for the use of the intended recipient and may contain material that is confidential, privileged and/or protected by the Official Secrets Act. If you are not the intended recipient, please delete it or notify the sender immediately. Please do not copy or use it for any purpose or disclose the contents to any other person.

some users use modules to maintain several bcbio installations.
Also to the issue of reproducibility - having modules bcbio1.2.0, bcbio1.2.1, bcbio 1.2.2 etc and just linking data installation to each of them helps.
https://www.admin-magazine.com/HPC/Articles/Environment-Modules

bcbio.error.report.txt
Sorry, I've removed the old bcbio paths, now encountering this issue (please see attached file) Thanks for your help.

I think it have hit the network timeout when accessing anaconda.org:
    Traceback (most recent call last):
      File "/home/users/astar/gis/simngl/tools/anaconda3/lib/python3.7/site-packages/conda/exceptions.py", line 1079, in __call__
        return func(*args, **kwargs)
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 900, in exception_converter
        raise e
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 894, in exception_converter
        exit_code = _wrapped_main(*args, **kwargs)
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 853, in _wrapped_main
        result = do_call(args, p)
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 741, in do_call
        exit_code = create(args, parser)
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 620, in create
        return install(args, parser, "create")
      File "/home/projects/13001702/tools/bcbio/v1.2.4/anaconda/lib/python3.7/site-packages/mamba/mamba.py", line 570, in install
        downloaded = transaction.prompt(PackageCacheData.first_writable().pkgs_dir, repos)
    RuntimeError: Download error (28) Timeout was reached [https://repo.anaconda.com/pkgs/main/linux-64/python-3.6.12-hcff3b4d_2.conda]

try from a less busy server or at not a peak usage time? contact network admins for advice?

bcbio.error.install.nopath.txt
Hi, I've removed any references to the old bcbio installation, but I'm still getting the same issue (please see attached file). Thanks.
My installation command:
python bcbio_nextgen_install.py /home/projects/13001702/tools/bcbio/v1.2.4 --tooldir=/home/projects/13001702/tools/bcbio/v1.2.4 --nodata

Hi!
It is another network timeout this time, so it seems you have sporadic connection issues
RuntimeError: Download error (28) Timeout was reached [https://conda.anaconda.org/bioconda/linux-64/bioconductor-bubbletree-2.6.0-0.tar.bz2]

Try talking you your network's sysadmins.
SN

closing for now! Let us know if you are still having installation issues!


Type.IsInstanceOfType implemented

Type.IsInstanceOfType implemented.

It's nice when these fixes are like 3 lines :-)


[Baselines] Spectra datasets benchmark

Spectra datasets benchmark
Add benchmarking for properties inference and spectra generation.
Contacts:
Participants: Maya
Goals and deliverable
Resources needed
Some basic spectral analysis expertise
Detailed description

Opened #hackathon-spectra

There's also the ARIEL data challenge. this is about exoplanet atmosphere retrieval

I have opened the #hackathon-stellar-metrics channel on Slack to discuss what baselines we could implement.


chore(github): add code owners

Before creating my pull-request, I ensure:

[x] Documentation is written
[x] Unit tests are written and green
[x] I've cleaned my WIP or meaningless commits
[x] Examples are up-to-date

Codecov Report

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

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   91.66%   91.66%
=======================================
  Files           4        4
  Lines         144      144
=======================================
  Hits          132      132
  Misses          9        9
  Partials        3        3

Continue to review full report at Codecov.

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


Custom Color for Table Border not working

Color of table border stay black after change it to brighter color. Only encounter at navigation table, not sure if it happen too in novel content.

can you give me the novel to check?

CtG Volume 1 Chapter 1, all of table border are black.
Make sure to enable custom color to encounter this trouble.

Sorry, there are only two table which are noticeable, first one and navigation table.

Border color changed on illustrations page.

The Table of Content and prologue are not using table, but div.

but the quest one, maybe I can force it to follow the color defined in the settings.

Ah, that's why the color is not changed. (Already figure out it's div, not table actually)
Actually, what I mean is this.

Before I enabled custom color settings, the color is white. What I suspect is (become black in custom color) it caused by Color Invert feature.

The original border color is black, and it's div not table. But, can you do something about that? If can't, it can't be helped.

I can replace it, but it will affect all table border because the navigation menu doesn't have specific class/id.. Is it OK?
Refer to: https://github.com/calvinaquino/LNReader-Android/blob/master/app/src/main/res/raw/style_custom_color.css#L184

Actually, I'm not familiar with css html stuffs. I'm leave this to you.
Thank You.


mempool: add Tor hidden service

What
add-on: create and expose a tor hidden service for mempool app
Why
user requested add-on: #996
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix
[x] simple addition

Fixes #996

port 8999 referred to mempool's backend -> port 80
HiddenServiceVersion 3 explicitly set

not sure if you guys have solved this. but here is my snippet which I did yesterday. it runs into an «unable to connect» error:
$ sudo nano /etc/tor/torrc
HiddenServiceDir /var/lib/tor/mempool/
HiddenServiceVersion 3
HiddenServicePort 4081 <IP_ADDRESS>:4081
$ sudo systemctl reload tor
$ sudo cat /var/lib/tor/mempool/hostname

not sure if you guys have solved this. but here is my snippet which I did yesterday. it runs into an «unable to connect» error:
$ sudo nano /etc/tor/torrc
HiddenServiceDir /var/lib/tor/mempool/
HiddenServiceVersion 3
HiddenServicePort 80 <IP_ADDRESS>:4081
$ sudo systemctl reload tor
$ sudo cat /var/lib/tor/mempool/hostname

tried also this, with the same result:
HiddenServiceDir /var/lib/tor/mempool/
HiddenServiceVersion 3
HiddenServicePort 80 <IP_ADDRESS>:8999


[MINOR] Fix some typos

What changes were proposed in this pull request?
Two mistakes in the code comments.
Why are the changes needed?
Modify these two comment mistakes and make code comments more standardized.
Does this PR introduce any user-facing change?
No
How was this patch tested?
No test required

Can one of the admins verify this patch?

Just make the two changes - this can reopen

How to reopen this PR?
I don't have permission to reopen.

Push a new conmit


Kafka instrumentation

Hi @marcingrzejszczak, as per our Zoom call we're raising a PR for Kafka instrumentation (including reactor kafka). Please have a look and let us know what you think.

@flavium31 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@flavium31 Thank you for signing the Contributor License Agreement!

Closing this PR, will raise a new one against 3.1.x


refactor: assets & icons scripts for iOS

This PR aims to avoid recreating PDF files for assets & icons automation even if the SVGs don't change at all.

From my understanding of this PR, we are only comparing the data of two files instead of their rendering. This approach will always fail because certain IDs and dates may change or am I missing something ?
   fs.readFile(`.temp-svg-assets/all/${file}`, (err, data1) => {
     if (err) throw err;
     fs.readFile(`build/assets/svg/all/${file}`, (err, data2) => {
       if (err) throw err;
       if (!data1.equals(data2)) {


pip install textwrap

ERROR: Could not find a version that satisfies the requirement textwrap (from versions: none)
ERROR: No matching distribution found for textwrap

pls use pip install textwrap3, it works.


git branch name disappears in sorin's theme

@sorin-ionescu hi thanks for maintaing this repo. prezto is great.
The sorin theme is my favorite theme. Sometimes, however, after some extended usage the branch name disappears from the theme. I've attached an image to illustrate:

I have no clue where to start with this. I really want to be able to see my branch name. The repo i'm in has submodules, although i'm not sure that has anything to do with this.
Do you (or anyone) have any quick ideas on what i'm doing wrong / how to fix this?
Thanks,
Martin

How to make the git status a part of the PS1 prompt like in the first message of this issue? Currently when using git module with the sorin theme, the git info goes to the RPROMPT.

You'll have to revert the theme to commit https://github.com/sorin-ionescu/prezto/commit/087fce85486505443bfce24db5a9ab00ac27d52c. You'll lose asynchronicity, and the prompt will be slower.

if you activate git module on .zprestorc and after that git-config on :+1:

How to make the git status a part of the PS1 prompt like in the first message of this issue? Currently when using git module with the sorin theme, the git info goes to the RPROMPT.

I would like to know this also! How can I get the information to the left side?


[Feature request] Compare JSON

Hi, maybe you can use the feature diff text, that will be added in a near future #15

@MaxiTalenti
You can use the Text Diff feature of v0.0.10 now! https://github.com/ObuchiYuki/DevToysMac/releases/tag/0.0.10


Genesis help does not know about "add" or "use"

Genesis "help" is missing some of the cammands needed for normal operations like how to add a release
$ genesis help
USAGE: genesis <command> [arguments]

Common commands:

    genesis new deployment      Create a new deployment directory, with all the
                                necessary files for deploying with Genesis.

    genesis new site            Set up the files and directories for a new site.

    genesis new environment     Set up the files and directories for a new environment.

    genesis build               When run from inside of an environment directory,
                                paste together all of the template files, in order
                                and pop out a manifest file.

    genesis diff                When run from inside of an environment directory,
                                show a semantic diff of the generated manifest for that
                                environment, against some other manifest file.

    genesis refresh             Refresh the cached copies of site and/or global
                                definitions inside the current environment directory.
                                Can also be used to recreate the Makefile after a
                                Genesis upgrade.

    genesis bosh                Generates a deployable manifest (which may have secrets
                                in it) and then runs the `bosh` CLI with remaining
                                arguments.  Useful for recreating workers.

    genesis help                Peruse the help!  Give it a topic argument, or the name
                                of another command to get more detailed information.
                                Documentation!

$ genesis help add
Unrecognized help topic 'add'.
Try one of these:

    genesis help usage
    genesis help <command>

$ genesis add release rabbitmq
Using the latest version of rabbitmq available via the Genesis Index
$

We should make sure known commands show up in at least the initial help page

Partially this was undocumented because we weren't sure we liked the add / use / set verbs, esp. since their usage is distinct, but their distinctions are not so much (particularly, use vs. set)

... and once you document something, people start to use it.  😄

Fixed in #74


Update LOCALIZATION_CONTRIBUTORS

added my name and git ID at Korea section
Changes

Implementation details

How to read this PR

Screenshots

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

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

Joined GitLocalize


Client UnAuthorized Handler - Javascript

Maybe add Interception idea to WebRequests Callbacks.
I mean it would be great to be able to handle all 401 responses in one place.
PS: If its already implemented im sorry couldnt find it.

You should be able to do this with the filter option off the client:
https://github.com/Azure/azure-mobile-services/blob/master/sdk/Javascript/src/MobileServiceClient.js#L85
Also, here's it in use in our test code to mock the server response: https://github.com/Azure/azure-mobile-services/blob/master/sdk/Javascript/test/winJS/tests/unit/mobileServicesClient.js#L59
If you make a filter on your top level client, and use that filtered client everywhere you should be able to handle all 401's in one place.  Let me know if it doesn't work for you.

Closing for now, I'll reopen if you are unable to accomplish your goal with the filters.

Worked like a charm. :)


rootPath deprecation

From the debug console:

[Deprecation Warning] 'workspace.rootPath' is deprecated and should no longer be used. Please use 'workspace.workspaceFolders' instead. More details: https://aka.ms/vscode-eliminating-rootpath

It looks like this may require us to upgrade to a newer version of the language server.

Fixed in #173.


IoTHubDeviceClient connect()/disconnect(): Exception when used in function. (Synch method)

@HoLue commented on Tue Apr 07 2020

Package Name:  azure.iot.device
Package Version:  ?
Operating System: Windows 10
Python Version:  3.7.4

Describe the bug
Unsing the synch method, a simple connect() and disconnect() of a device client (IoTHubDeviceClient) gives an error if it is done in a function. (Without the function ist works.) The exception message after about 10 seconds is:
Exception caught in background thread.  Unable to handle.
["AttributeError: 'NoneType' object has no attribute '_on_connected'\n"]
To Reproduce
Just run the following script with a valid connection string, the error occurs after about 10 seconds:
import os
from azure.iot.device import IoTHubDeviceClient
import time

def f():
    conn_str = 'YourConnectionString'
    device_client = IoTHubDeviceClient.create_from_connection_string(conn_str)
    # connect the client.
    device_client.connect()
    # usually do something but this dies not matter
    # Finally, disconnect
    device_client.disconnect()

# using the function
f()

# wait - after about 10 seconds the error occurs
for i in range(20):
    print(i)
    time.sleep(1)

Expected behavior
No exception (and therefore no exception message)

Is there a specific reason that you wish to do everything inside a function ?
I have figured out the root cause of the issue. While there is no immediate solution to the problem I can provide you a different way of doing if that is suitable in your scenario.
If possible please construct the client outside the function and let the function handle only connect and disconnect and other functionality like below :-
conn_str = "YourConnectionString"
device_client = IoTHubDeviceClient.create_from_connection_string(conn_str)

def f():
    device_client.connect()
    # usually do something but this does not matter
    # Finally, disconnect
     device_client.disconnect()

# using the function
f()

# wait - after about 10 seconds the error occurs
for i in range(20):
    print(i)
    time.sleep(1)

Well, this works. But may I ask what is done in the background? There seems to be some connection? Data transfer?

If you are referring to the error that you received, that is simply a handler that lets you know that the you are connected.
Please close the issue if this has helped you.

I cannot close, can I? (lmazuel opened this issue 6 days ago)

ok we will close

@lmazuel, @HoLue, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


Txn Manager Crashes on UPDATE without WHERE

The system crashes when you try to execute an UPDATE query that does not have a WHERE clause in it. This fails regardless whether the codegen feature is enabled. I only tried running this with the new optimizer enabled.
postgres=# CREATE TABLE foo (id INT PRIMARY KEY);
CREATE TABLE foo (id INT PRIMARY KEY) 0
postgres=# INSERT INTO foo VALUES (123);
INSERT INTO foo VALUES (123) 1
postgres=# INSERT INTO foo VALUES (456);
INSERT INTO foo VALUES (456) 1
postgres=# UPDATE foo SET id = id + 1;
server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.
The connection to the server was lost. Attempting reset: Failed.

Here is the server's assert message:
peloton: /home/pavlo/Documents/Peloton/Github/peloton-master/src/concurrency/timestamp_ordering_transaction_manager.cpp:656: \
virtual void peloton::concurrency::TimestampOrderingTransactionManager::PerformDelete(peloton::concurrency::Transaction*, const peloton::ItemPointer&, const peloton::ItemPointer&): \
Assertion `(GetLastReaderCommitId(tile_group_header, old_location.offset) == current_txn->GetCommitId())' failed.

The backtrace is attached.
backtrace.txt

Wasn't this fixed in #558 ?

@PauloAmora I think that this problem is different. It definitely works correctly if you include the WHERE clause:
postgres=# CREATE TABLE foo (id INT PRIMARY KEY);
CREATE TABLE foo (id INT PRIMARY KEY) 0
postgres=# INSERT INTO foo VALUES (123);
INSERT INTO foo VALUES (123) 1
postgres=# INSERT INTO foo VALUES (456);
INSERT INTO foo VALUES (456) 1
postgres=# UPDATE foo SET id = id + 1 WHERE id > 0;
UPDATE foo SET id = id + 1 WHERE id > 0 2
postgres=# SELECT * FROM foo;
 id
-----
 124
 457
(2 rows)

The system crashes because each tuple can be updated two or more times when performing update without where.
When processing a query like UPDATE foo SET id = id + 1, the DBMS connects the update executor (parent executor) to a sequential scan executor (child executor). It scans the table and then updates each accessed tuple version. Every time the DBMS updates a tuple version in the table, a new tuple version will be inserted into the table. This newly inserted version can also be visited by the sequential scan executor, and subsequentally updated by the update executor. Hence, we may observe that a tuple is updated multiple times. The DBMS can keep running when performing update on a table without primary index. However, the DBMS crashes when updating a table with primary index.

This problem is related to #106

There are two possible solutions to address this problem.

Record the read/write set for each operation. Every time the DBMS finds that a single operation is trying to modify (update/insert/delete) a version that it just created, then the DBMS should directly ignore this version. However, maintaining operation-level read/write set can bring certain performance overhead, especially when a single operation reads/updates lots of tuples (e.g., update table).

Switch from append-only storage to time-travel storage. The disadvantage is that it can bring additional pointer-chasing overhead when visiting older versions.


Reduce peak memory usage

Measure and try to reduce the peak memory overhead over the input file. Basically find out how much memory is used by roxmltree metadata itself.

[ ] Make token ranges optional
[ ] Try removing ExpandedNameOwned::prefix
[ ] Remove 4GB input limit
[ ] Try using custom ShortRange for attributes and namespaces indexes. Like 20.12 packed in u32
[ ] Test peak memory usage

:eyes:

👀 I'll spend a few hours on this.

@tomjw64 Thanks. Removing ExpandedNameOwned::prefix is the most important one. It's just garbage. But would require some changes to the parser.
A custom ShortRange a bit more tricky. I will probably look into it myself.

Token ranges are surprisingly not that memory-heavy. On the 1GB XML from #18 they produce about 5% memory overhead. They still should be optimized though.


chore: build protocol-kit with cjs and esm targets

Over at Zodiac Pilot, we've had some trouble with the protocol-kit package, as it ships only as CJS. We're upgrading our codebase to the ESM future.
This PR splits up the build for the protocol-kit package to create ESM and CJS sources. I've also extended the package.json to conform to everything from bundlers to Node JS native module resolution.
You'll notice that I've removed a call to tsc-alias. AFAICT, the sources are bundled, which means any aliases have already been resolved. Please correct me if this assumption is wrong. I've tested this locally with our extension, and it works like a charm. We're now picking up the ESM build, and everything works as expected.
Let me know if anything is missing, if you need further clarification, or if anything else comes up!

I have read the CLA Document and I hereby sign the CLA

We will be testing this solution, also the other kits should be compatible with esm and cjs


Question about fetching headers only

Is HTTP/1.1 405 Method Not Allowed an expected behavior when we execute curl -I http://playin.coz.is/index.html ?

Hey! At the moment the response function rejects anything that's not a GET request with 405, so it is expected but not ideal. The HTTP semantics still need some work.
void respond(Request request, ResponseBuilder *b)
{
	if (request.major != 1 || request.minor > 1) {
		status_line(b, 505); // HTTP Version Not Supported
		return;
	}

	if (request.method != M_GET) { // <-------------------------- HERE
		status_line(b, 405); // Method Not Allowed
		return;
	}

	if (string_match_case_insensitive(request.path, LIT("/hello"))) {
		status_line(b, 200);
		append_content_s(b, LIT("Hello, world!"));
		return;
	}

	if (serve_file_or_dir(b, LIT("/"), LIT("docroot/"), request.path, NULLSTR, false))
		return;

	status_line(b, 404);
	append_content_s(b, LIT("Nothing here :|"));
}

Ah I see; good to know.
Keep up the good work :+1:


Struct case-insensitivity: parser error when field name matches existing Struct name, regardless of casing

Steps to reproduce

Created IDL, test.idl, with contents below:

struct A
{
    int32 int_value;
};

struct B
{
    A a;
};

fastrtpsgen -example CMake -replace test.idl

Current behavior:
Loading templates...
Processing the file test.idl...
test.idl:8:7: error: Illegal identifier: a is already defined (Definition: com.eprosima.idl.parser.tree.TypeDeclaration@78e67e0a)
Exception in thread "main" java.lang.NullPointerException
        at com.eprosima.fastrtps.fastrtpsgen.execute(fastrtpsgen.java:336)
        at com.eprosima.fastrtps.fastrtpsgen.main(fastrtpsgen.java:1199)

Expected behavior:
IDL parser should generate valid C++ definitions.

Hi @pwolf15
As the IDL 4.2 standard defined (section <IP_ADDRESS>)

Identifiers that differ only in case collide, and will yield a compilation error under certain circumstances. An identifier for a given definition must be spelled identically (e.g., with respect to case) throughout a specification.

So, the IDL-Parser is working as expected.
The standard way to work around this feature, is to prefix an underscore to the member identifier:
struct B
{
    A _a;
};

If you need to support these kind of IDLs and cannot be changed, fastrtpsgen allows you to apply case sensitive parsing with the -cs option:

-cs: IDL grammar apply case sensitive matching.

fastrtpsgen -example CMake -replace -cs test.idl

Great, thanks! Worked as expected. I was unaware of the rule. rtiddsgen, RTI's generator, succeeded without error on the same kind of input. Additionally, I also just now noticed the -cs option in the usage message.