Filtering through foreign table doesn't respect join hint syntax

I have a query in supabase-js that works, and when I translate it to supabase-py it doesn't work. I am on the latest version of each. From what I could gather while poking around, the python version doesn't seem to respect the join-hint PostgREST syntax for referencing tables with foreign keys.
Javascript
supabase
  .schema('pfr')
  .from('games')
  .select(`id,
    date,
    away_score,
    home_score,
    away_team:team_seasons!fk_away_team_id!inner(city, name),
    home_team:team_seasons!fk_home_team_id!inner(city, name),
  `)
  .ilike('away_team.name', awayTeam)
  .ilike('home_team.name', homeTeam);

Python
supabase
  .schema('pfr')
  .table('games')
  .select('''id,
    date,
    away_score,
    home_score,
    away_team:team_seasons!fk_away_team_id!inner(city, name),
    home_team:team_seasons!fk_home_team_id!inner(city, name),
  ''')
  .ilike('away_team.name', awayTeam)
  .ilike('home_team.name', homeTeam);

Schema
The schema is what you would expect; a games table with two foreign keys each pointing at a team_season.
Error Message
In PostgREST, you are supposed to be able to reference aliased foreign keys like this. It shouldn't assume the column is a column on the games table. Instead it should use the join-hint to compute a path to that table based on the constraints/keys available between the games table and team_seasons table (in this case, there are two paths: fk_home_team_id and fk_away_team_id).
APIError: {'code': '42703', 'details': None, 'hint': None, 'message': 'column games.away_team.name does not exist'}

getting what may be a very similar issue
say I'm trying
query = supabase.table('applicants').select(
            '*, profiles(id, name, email)'
)

or_conditions = []
or_conditions.append(f"profiles.name.ilike.%{keyword}%")
or_conditions.append(f"profiles.email.ilike.%{keyword}%")
query = query.or_(', '.join(or_conditions)

I'm getting
{'code': 'PGRST100', 'details': 'unexpected "a" expecting "not" or operator (eq, gt, ...)', 'hint': None, 'message': '"failed to parse logic tree ((profiles.name.ilike.%.me%,profiles.email.ilike.%.me%))
which makes sense that .name' is not an operator
then I've tried doing profiles->name and profiles->email, and in that case, I'm getting
{'code': '42703', 'details': None, 'hint': 'Perhaps you meant to reference the column "applications.profile_id".', 'message': 'column applications.profiles does not exist'}

@teodos this is not related and you should also provide your table details when posting. Your error is making reference to applications but your table is called applicants. Without seeing the tables we don't know what issue you could be having. Also open a separate issue with all the details I mentioned in this post.

Can't you just figure out the table structure from the post? (which btw it's not what this issue is about)
It's not about the structure or naming, but the join syntax that's not working (as the issue title suggests).
One thing you are right about is the table name (it's applications, not applicants)
Tables:
applications:
- id
- profile_id (fk profiles.id)

profiles:
- id
- name
- email

So again, same question with corrected table name:
I'm doing:
keyword=".me"
query = supabase.table('applications').select(
            '*, profiles(id, name, email)'
)

or_conditions = []
or_conditions.append(f"profiles.name.ilike.%{keyword}%")
or_conditions.append(f"profiles.email.ilike.%{keyword}%")
query = query.or_(', '.join(or_conditions)

I'm getting
{'code': 'PGRST100', 'details': 'unexpected "a" expecting "not" or operator (eq, gt, ...)', 'hint': None, 'message': '"failed to parse logic tree ((profiles.name.ilike.%.me%,profiles.email.ilike.%.me%))
which makes sense that .name' is not an operator
then I've tried doing profiles->name and profiles->email, and in that case, I'm getting
{'code': '42703', 'details': None, 'hint': 'Perhaps you meant to reference the column "applications.profile_id".', 'message': 'column applications.profiles does not exist'}

@teodos the docs clearly states how the .or_ operator works and you aren't following it's syntax. As I've mentioned before your issue is not related to the OPs issue, you should open your own issue and stop adding to this one.

I’ll see if I can find time to set up an example.
The simplest case I can think of is the messages example from the js docs with a users table and a message table that references users twice, once for sender and once for recipient.
On Nov 6, 2024 at 4:13 PM -0800, Andrew Smith @.***>, wrote:

@teodos the docs clearly states how the .or_ operator works and you aren't following it's syntax. As I've mentioned before your issue is not related to the OPs issue, you should open your own issue and stop adding to this one.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

@notjustinshaw I tested the message example from the docs and didn't get any issue with it.

Here is a screen recording of me testing the sample data you mentioned in the docs with the Python client.
Turn your volume down as I might be a bit loud when talking in the video.
https://github.com/user-attachments/assets/560444ca-a607-474f-a1b8-e2ad95c5caa3

No reply from OP from over a month now. Closing this out.

it’s still broken, I just haven’t had time to contribute the fix.
On Dec 30, 2024 at 4:45 AM -0800, Andrew Smith @.***>, wrote:

No reply from OP from over a month now. Closing this out.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Perhaps now that this is an officially supported package from the Supabase team, someone could take a look to make sure clients are compatible with one another?
On Dec 30, 2024 at 4:45 AM -0800, Andrew Smith @.***>, wrote:

No reply from OP from over a month now. Closing this out.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Depth Rendering issue

Have run in to a weird issue and I'm out of ideas to try now.
Have a set of shaders for point sprites(sort of), when rendering the color pass (even if i do it without a Z prepass and draw everything) things work just fine.
However when rendering the depth pass (even standalone without the color pass) takes an insane amount of time(100-2000ms and sometimes just crashes) and it's doing less work then the color pass.
Rendering about 7000 points * 4 vertices which is not even a large amount.
Any suggestions on what could cause this?
Using Miniengine but i suspect that the problem is not specific to it.

Crashes dissappeared when i removed unused data being passed between GS and PS and slowdown was caused by mistyped index buffer. Closing the case.


Deduplicate headers for overloaded methods

Closes #167
The script was creating a title for every
 tag inside a
 tag. This is almost always correct given that we only have a
 tag for each method. The problem with overload methods is that we have a
 for each @typing.overload definition.
For example, the quantumcircuit.py file overloads the definition of for_loop:
https://github.com/Qiskit/qiskit/blob/244940a10c17f1dc3d74e2665a3ae869dc3d7291/qiskit/circuit/quantumcircuit.py#L4862-L4884
This is an example of the HTML generated

    for_loop

    for_loop

Create a for loop on this circuit.

To solve the problem this PR adds an extra condition to check whether the
 tag has an id or not. Only the first
 tag will have an id, so we can use this fact to create only one
 for each method.

I think it's a good idea to add tests. What do you think about adding them as a follow-up together with more cases for the processMembersAndSetMeta function? I'm going to test the generation with runtime and provider to make sure it doesn't break anything


Increase code coverage

As a developer, I want to have 100% code coverage.
The main obstacle currently is testing of the OAuth 2.0 implicit flow.

very old


[GSSoC'23] Scientific Calculator Project Completed

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.

This program will provide the user an attractive user interface of Scientific Calculator
This calculator can do Basic Arithmetic Operations like *( +, -, , / )
Also performs all the trignometric calculations like (sin, cos, tan)
Also performs all the logarithmic calculations like (log, ln)
Also performs all the inverse calculations like (1/x)
Also performs all the exponential calculations like (e^x, x^n, n^x, etc.)
Also performs all the angular calculations like (deg, rad)
Also adding constants like (pi, e)
Adding all these functions will definately give a better user experience along with the attractive user interface
Also adding instructions to use this calculator in more efficiently

Fixes #271
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I've edited the SCRIPTS.md and link to my code.
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@omazoz @avinashkranjan @vanshitaverma please review this pull request and reqeusted to merge it into the main branch for a successful contribution under GSSoC'23 program

Not Needed, Do Read the Desription of the Project, How does this come under Pentesting and Hacking Scripts??


Symbol ERROR in hip-clang

There is a really big problem when we use symbol(hipMemcpyToSymbol and hipMemcpyToSymbol) in program compiled by hip-clang. hipMemcpyToSymbol and hipMemcpyFromSymbol call read_agent_global_from_process() to get the address of symbol through hipGetSymbolAddress. In HCC the function can return the correct address of symbol.  But In hip-clang the function can't find the address and return NULL.
The implementation of read_agent_global_from_process() is the reason for this problem. There is a std::call_once funtion call inside of read_agent_global_from_process().
The process of read_agent_global_from_process():

lamda function  call get_executables() to get all executable of agent
lamda function  scan all executable of agent
for each executable, lamda function call read_agent_globals() get all symbol(kernel descriptor and global variable name) and store in globals_from_process
read_agent_global_from_process call read_global_description to find the symbol from globals_from_process.

But this process is different in hip-clang, module executable have already created by function __hipRegisterFatBinary on the .init_array stage. When we call get_executables() in read_agent_global_from_process() the problem arises.
First, program compiled by hip-clang can't call  get_executables(). There is a std::call_once function in get_executables() to initialize the executables through get_code_object_blobs().  But this funtion only work for the program has '.kernel' segment.
Secondly, __hipRegisterFatBinary will create module executable for each agent. That means all symbol in executable has already registered to agent on hsa.  If we only fix the segment problem in get_code_object_blobs(), get_executables() will create all symbol(global variable) twice. The symbol get from hipGetSymbolAddress and kernel execution will be different.
In our solution, we create several global variables in hip_clang.cpp to save the modules create by  __hipRegisterFatBinary.  g_modules is the modules return by __hipRegisterFatBinary and g_moduledevices is label to mark the order of the agent. read_agent_global_from_process() use clanginit to decide using g_modules or get_executables() to initialize the globals_from_process.
extern std::vector<hipModule_t>* g_modules;
extern std::vector<hsa_agent_t>* g_moduledevices;
extern bool clanginit;
Currently, __hipRegisterVar in hip_clang.cpp is not implemented. I think the best way is to redesign hipGetSymbolAddress function and implement __hipRegisterVar.

Hi louishp,
I found the same issue as you mentioned above. The hipcc built with clang cannot handle hipGetSymbolAddress() as expected, but hipcc built with hcc makes it! But it seems that in rocm of 2.6 version, hipcc built with clang can also handle hipGetSymbolAddress() well, which version of rocm were you using when you are posting above?


Compiler should create multiple class files

When compiling a proto file with mutltiple message items it in, the compiler should create a php class file for each of the message items. Currently it creates one file with all the classes in it. This breaks PSR-0 guildlines and therefor autoloaders will not load the classes.

i think this very import!!

It's the case now.


build: use -pthreads for tests with autotools

This prevents tests from using non-reentrant implementations of common
APIs (e.g errno) when using POSIX threads and building with autotools.
This is consistent with the way tests are builds when using gyp.
The problem was found when investigating tests failures on SmartOS for
one test added by https://github.com/libuv/libuv/pull/640. These
failures were due to that test using POSIX threads and also using a
non-reentrant errno global variable, instead of the reentrant ___errno
function.

Thanks @misterdjules! (I also used this PR as a guinea pig for GitHub's new review thingy) /cc @bnoordhuis

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/140/

Oops, make that https://ci.nodejs.org/view/libuv/job/libuv-test-commit/141/

CI one more time, the pi buildbots had infrastructure failures during the last run.  Probably harmless but better too rigorous than not rigorous enough.
https://ci.nodejs.org/view/libuv/job/libuv-test-commit/142/

Ben, the project is called libuv-1 for @misterdjules, so I started one more CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/145/

Are the failing tests in the latest CI run above known as being flaky?

Yeah.  Nothing related to this.

Unless anyone (@bnoordhuis ?) has any objections, I'll land this tomorrow.

Chers Julien, landed in ca107b7 ✨

Thank you!


Consider search topics

The newest release of Oxygen XML Editor includes the ability to specify search labels on the page.
Clicking on one of these will find other documents that also carry that label.
If we can identify a suitable set of labels it could provide a useful capability, if they produce sufficiently few matches.
Documentation here:
https://www.oxygenxml.com/doc/versions/26.0/ug-webhelp-responsive/topics/wh-labels.html
@IanMayo to arrange contact time with SMEs.

Nope. We can't do this with our license - it needs the Oxygen Feedback upgrade.


fix #22 set link target

I tend to -1 this pull request. Adding such a target robs users the choice of opening the link target in the same tab/window or in a new one. Every modern browser has the ability to open links without a target in new tabs/windows (context menu, middle click, ctrl+click, shift+click) but if it is explicitly specified, you cannot prevent it from being opened in a new window/tab.
I think, it is already a nuisance at the publication links in the admin table and should be fixed there as well, not made worse like this.

Completely disagree. Publication links take you to a different
service/location and by default I don't want it taking you out of
Opencast. You can always close the tab that Opencast was open on, but I
doubt anyone is using the publication links in the Admin UI as a way of
just navigating to the Publication, the link is just there so an
administrator can check that it is correct.
On 01/07/17 10:30, Lars Kiesow wrote:

I tend to |-1| this pull request. Adding such a target robs users the
choice of opening the link target in the same tab/window or in a new
one. Every modern browser has the ability to open links without a
target in new tabs/windows (context menu, middle click, ctrl+click,
shift+click) but if it is explicitly specified, you cannot prevent it
from being opened in a new window/tab.
I think, it is already a nuisance at the publication links in the
admin table and should be fixed there as well, not made worse like this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opencast/opencast/pull/23#issuecomment-312421684,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC0PE_xqnBgThVeX1AoXc88PhvMnqpGvks5sJhHTgaJpZM4OKsx-.

--
James S. Perrin
Media Technologies Team
J20, Sackville Building
The University of Manchester
Oxford Road, Manchester, M13 9PL
t: +44 (0)<PHONE_NUMBER>
e<EMAIL_ADDRESS>"The test of intellect is the refusal to belabour the obvious"

Alfred Bester

The thing is that an administrator ca always open a link in a new tab/window with one click. That is something which already works right now. If you know that you want to open it, check it and close it: No problem.
But this action prevents the other option and if, for example, I just want to quickly check something from somewhere else, I often still go to the event via admin ui, but in the end, I do not want to stay there but close all parts of Opencast.
It's just that if you want it open in a new tab, with your patch the click count is the same. If you do not want that, you now have more clicks.

@JamesUoM, can you bring this up as a proposal on list and if the outcome is positive raise another pull request at Bitbucket? I'm going to close this now to finish the migration back to Bitbucket.


Constraint Violation as a Penalty - Multi-Objective framework

Hello,
first of all, thanks for the great package.
I am facing some problems; I would like to implement CV as a penalty:
 Id = 1e-6
Vth = 0.7
 Va = 100

    algorithm = NSGA2(
        pop_size=50,
        n_offsprings=20,
        sampling=FloatRandomSampling(),
        crossover=SBX(prob=0.9, eta=15),
        mutation=PM(eta=20),
        eliminate_duplicates=True
    )

    termination = get_termination("n_gen", 100)

    problem = CurrentMirror(
        Id = Id,
        Vth = Vth,
        Va = Va
    )

    res = minimize(
        ConstraintsAsPenalty(problem, penalty=np.array([1,1,100])),
        algorithm,
        termination,
        seed = 42,
        save_history = True,
        verbose = True
    )

but if I include it in my (working) code I get the following error:
Traceback (most recent call last):
  File "C:\Users\andreasala\Desktop\projects\circuit-optimizer\optimizer.py", line 121, in <module>
    res = minimize(
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\optimize.py", line 67, in minimize
    res = algorithm.run()
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\core\algorithm.py", line 141, in run
    self.next()
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\core\algorithm.py", line 161, in next
    self.evaluator.eval(self.problem, infills, algorithm=self)
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\core\evaluator.py", line 69, in eval
    self._eval(problem, pop[I], evaluate_values_of, **kwargs)
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\core\evaluator.py", line 90, in _eval
    out = problem.evaluate(X, return_values_of=evaluate_values_of, return_as_dictionary=True, **kwargs)
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\core\problem.py", line 187, in evaluate
    _out = self.do(X, return_values_of, *args, **kwargs)
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\pymoo\constraints\as_penalty.py", line 36, in do
    out["F"] = F + self.penalty * np.reshape(CV, F.shape)
  File "<__array_function__ internals>", line 200, in reshape
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\numpy\core\fromnumeric.py", line 298, in reshape
    return _wrapfunc(a, 'reshape', newshape, order=order)
  File "C:\Users\andreasala\anaconda3\envs\dev1\lib\site-packages\numpy\core\fromnumeric.py", line 57, in _wrapfunc
    return bound(*args, **kwds)
ValueError: cannot reshape array of size 50 into shape (50,3)

where 50 is my pop size.
I've already tried to specify an array of penalties instead of a single value, since I have 3 objective functions, but it does not work either.
Is CV as a penalty supported for MOO problems?
Thanks

https://github.com/anyoptimization/pymoo/blob/6fac683d5b9cc8f5c4088e8ad62d01da1cd0a696/pymoo/constraints/as_penalty.py#L35-L36
It seems not support multi-objectives well. You may change the code to out["F"] = F + self.penalty * np.mean(CV) to let the code run.

Ok thanks, I'll fork the repo and make my own changes.

If you find a solution that generalizes more can you also create a PR with your fix?

What about a simple
try:
    out["F"] = F + self.penalty * np.reshape(CV, F.shape)
except ValueError:
    out["F"] = F + self.penalty * np.sum(CV)

?


Attribute table not set for classes using single table inheritance

I'm trying to update Flask-Sqlalchemy from version 2.1 to the 2.3.1 on one application I'm maintaining (sqlalchemy version 1.1.14). The issue I'm having is that __table__ attributes are not being set for classes that use single table inheritance. I would assume that the __table__ attribute is always present on those classes. That is also how the pure sqlalchemy solution behaves.
Minimal code showing the issue:
from flask_sqlalchemy import SQLAlchemy

db = SQLAlchemy()

class Item(db.Model):
    __tablename__ = 'item'

    id = db.Column(db.Integer, primary_key=True)
    info = db.Column(db.String(255))
    type = db.Column(db.String())

    __mapper_args__ = {
        'polymorphic_on': type
    }

class ItemA(Item):
    __mapper_args__ = {
        'polymorphic_identity': 'a'
    }

class ItemB(Item):
    __mapper_args__ = {
        'polymorphic_identity': 'b'
    }

assert ItemA.__tablename__ == 'item'
assert ItemB.__tablename__ == 'item'
assert ItemA.__table__ == Item.__table__  # This assertation fails
assert ItemB.__table__ == Item.__table__  # This assertation fails

Here is the pure sqlalchemy solution that passes:
from sqlalchemy import Column, Integer, String
from sqlalchemy.ext.declarative import declarative_base

Base = declarative_base()

class Item(Base):
    __tablename__ = 'item'

    id = Column(Integer, primary_key=True)
    info = Column(String(255))
    type = Column(String())

    __mapper_args__ = {
        'polymorphic_on': type
    }

class ItemA(Item):
    __mapper_args__ = {
        'polymorphic_identity': 'a'
    }

class ItemB(Item):
    __mapper_args__ = {
        'polymorphic_identity': 'b'
    }

assert ItemA.__tablename__ == 'item'
assert ItemB.__tablename__ == 'item'
assert ItemA.__table__ == Item.__table__
assert ItemB.__table__ == Item.__table__

This is not possible given how the hook we're using works. Use SQLAlchemy's inspection API to discover information about models.
m = db.inspect(ItemA)
print(m.mapped_table)
print(m.mapped_table.name)

cc @zzzeek

Thanks for the fast response!
That sounds a bit unexpected behaviour as it's not the same as with pure sqlalchemy, but I do see that the proposed way of accessing the information with inspection API is completely reasonable.
The actual code that tries to access the __table__ is inside sqlalchemy-searchable source, so the fix needs to be implemented there. I'll report this issue to that package.

Actually, I can fix this. But you should still use inspect when possible.

Released 2.3.2. pip install -U flask-sqlalchemy

Thank you for the fix, it fixes the issue in the minimal case I showed. Unfortunately it seems that the minimal case I created was a bit too minimalistic when compared to what actually happens with the sqlalchemy-searchable. The sqlalchemy-searchable tries to read the __table__ attribute in instrument_class event listener, and even with the Flask-Sqlalchemy version 2.3.2 the __table__ attribute is not set at that point of time.
I tried to use the inspect to get the list of table's columns for sqlalchemy-searchable, but for some reason that raises NoInspectionAvailable exception. I'm not sure if there is some limitations whether the inspect can be called at that point of time. The full exception text is NoInspectionAvailable: No inspection system is available for object of type <class 'flask_sqlalchemy.model.DefaultMeta'>
Here is the code that can be used to reproduces both issues (__table__ not set during event listener and inspect raising NoInspectionAvailable exception.):
from flask_sqlalchemy import SQLAlchemy
from sqlalchemy import event
import sqlalchemy as sa

db = SQLAlchemy()

def process_mapper(mapper, cls):
    assert cls.__table__ is not None  # This assertation fails for ItemA
    if cls.__table__ is None:
        db.inspect(cls)  # This raises NoInspectionAvailable exception for ItemA

event.listen(
    sa.orm.mapper, 'instrument_class', process_mapper
)

class Item(db.Model):
    __tablename__ = 'item'

    id = db.Column(db.Integer, primary_key=True)
    info = db.Column(db.String(255))
    type = db.Column(db.String())

    __mapper_args__ = {
        'polymorphic_on': type
    }

class ItemA(Item):
    __mapper_args__ = {
        'polymorphic_identity': 'a'
    }

OK, I see why this is failing, the event is triggered while the class is being mapped, which is before __init__ can reset it.
But you already have the mapper at that point, you don't need to use inspect there. mapper.mapped_table

I think the easiest way of fixing this is to make the change to sqlalchemy-searchable, so I made an PR there to fix this issue: https://github.com/kvesteri/sqlalchemy-searchable/pull/64
Thanks for your help!


add ESM2 and ProGen2

protein LM

Hi, thanks for the contribution!
There are several places needed to be modified:
For ESM2:

It's better to include New York University, Stanford University, MIT in the affiliations.
According to the paper, there is another 3B version of ESM2.

For ProGen2:

Columbia University is also one of the affiliations.
There is another 2.7B version of ProGen2.
There is a trailing space in line 11 (I suggest deleting that empty line).

Please check them :)


Background color revealed in Wallet and Transactions tab when pulling down to refresh is wrong #1912

Closes #1912

Looks good. Can you do the same thing for the Transactions tab too?

@hboon  Done.

Looks good, but can you split out the changes to the Settings tabs to a different PR? It's for a different purpose. In that PR, can you also check if "Support" and other similar screens opened from the Settings tab works the same way?

ok, np.


Cannot login with create-v0.3.1a

while I log in lightfall server it callback me this:

lightfall cmd:

arclight (arclight-forge-1.16-1.0.16):

mod (client):
the server just install create-v0.3.1a

through i have some tests, create v0.3e hasno this problem.
hope fix it

Please test with latest build.


Implement virtual editor for showing workspace settings in multi root workspaces

In Multi-root workspaces, workspace settings come from the workspace config file which has other configurations like id, folders.  Eg:
{
	"id": "1499978571481",
	"folders": [
		"file:///.../vscode",
		"file:///.../vscode-docs"
	],
	"settings":{
	    "editor.insertSpaces": false,
	    "files.eol": "\n",
       }
}

Hence, Settings editor needed a virtual editor for displaying only settings section of the config file for Workspace Settings. This should also support editing just like in other editors.

To get consistent experience in Settings Editor across, while editing settings in virtual editor (Workspace Settings), content is written directly into the real configuration file on disk. This needs #23413 Auto saving of settings for consistent experience.

We will wait for feedback for current approach before taking the next step. Hence closing this.


Create a custom RSpec matcher for the various flash statuses

Summary
I did this at a previous job and found it to be useful. I'll pick this one up if no one else grabs it after a while.
What we want is a custom Rspec matcher that will work like this:
expect(page).to have_notice("some part of the notice")

Additionally, a general one like:
expect(page).to have_flash(notice: "some part of the notice")

Create matchers for the various flash statuses we're using (I think it's :alert, :error, :notice, :success) -- they're defined in app/helpers/application_helper.rb, and the generic flash one that accepts a hash.
A flash[:notice] message like "Distribution created successfully" should pass an expectation like: expect(page).to have_notice("success") and also expect(page).to have_notice("Distribution created successfully")
Criteria for Completion

[ ] A custom matcher, per above

Bonus Round
If you're feeling up for it, go back and implement this matcher in the existing specs that are checking it manually.

I'd be happy to take this one on!


x/tools/gopls: support fill struct behavior

What version of Go are you using (go version)?

$ go version
go version go1.14 linux/amd64

Does this issue reproduce with the latest release?
n/a (feature request)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/frew/.cache/go-build"
GOENV="/home/frew/.config/go/env"
GOEXE=""
GOFLAGS="-mod=readonly"
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/frew/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org"
GOROOT="/home/frew/sdk/go1.14"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/frew/sdk/go1.14/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/run/shm/go-build445574914=/tmp/go-build -gno-record-gcc-switches"

What did you do?
n/a (feature request)
What did you expect to see?
n/a (feature request)
What did you see instead?
n/a (feature request)
Feature request
vim-go had a command called :GoFillStruct and it would transform (for example), this:
x := Person{}

into
x := Person{
	Name: "",
	Employed: false,
	Age: 0,
}

Basically it'd set all the fields to their zero value.  In https://github.com/govim/govim/issues/709 I was told that this needs to be added to gopls first, so here I am :)

@muirdm: Do you think we could fit this behavior into completion?

For my understanding, why is this useful behavior?

I don't know if it's particularly useful for the Person case described above, but I would find it useful if the struct contained map values. (My classic example is go/types.Info, which I always find super annoying to create.) My ideal scenario when completing that struct is that, if I have a value of the correct type in scope, it should be used -- otherwise, we should make a map in the struct declaration.

The zero value of map is "nil", so this proposal as written wouldn't help in the types.Info case.
In my experience having the literal completion that gives you a candidate for make(map[some.Long]annoying.Map) is already a major improvement. Having structs get automatically filled in with in-scope objects sounds hard to avoid false positives.

the main use case here is creating fresh struct with lots of fields. This
is common when working with, for example, the aws sdk where completing
methods is a very small help since you then need to pass in a struct with
sometimes dozens of fields set.
On Mon, Mar 2, 2020, 1:19 PM Muir Manders<EMAIL_ADDRESS>wrote:

The zero value of map is "nil", so this proposal as written wouldn't help
in the types.Info case.
In my experience having the literal completion that gives you a candidate
for make(map[some.Long]annoying.Map) is already a major improvement.
Having structs get automatically filled in with in-scope objects sounds
hard to avoid false positives.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/37576?email_source=notifications&email_token=AAAB7Y5OHXJKRI2XLFM4CA3RFQPGNA5CNFSM4K6CIMAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENRAR5Q#issuecomment-593627382,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB7YYIU35CO53T5KJSNV3RFQPGNANCNFSM4K6CIMAA
.

I can see how that could be part of your workflow, but it also seems to overlap with existing facilities such as textDocument/hover which will show you all the struct fields or just a normal completion request in the struct literal, which also will list all the struct fields and their types.
To answer the original question of fitting this into completion, we could but I'm not sure what would trigger the completion or how it would show up clearly to the user. Maybe better as a code action?

I don't know what a code action is, but in the original setting it was something the user would explicitly request, rather than fitting into the same workflow as method completion.  I personally think that's fine; this is something I'd do a few times a day, not constantly, like method completion.

I don't know what a code action is, but in the original setting it was something the user would explicitly request, rather than fitting into the same workflow as method completion.  I personally think that's fine; this is something I'd do a few times a day, not constantly, like method completion.

I support the codeAction suggestion, seems like it would be a better fit (maybe of kind  refactor.rewrite.fillstruct).
As a feature this is something that, at least I, hear people ask for quite often. With the use case to get the field names quickly added when creating a new struct instance. Hover does show the same info, but it would be really helpful if gopls could add it for you so that you don't have to add them manually.

Is anyone working on this? Mind if I take a look at implementing this feature?

I don't believe so, so please go ahead, @luciolas! As a starting point, I would suggest implementing this behavior as a command.

Ahh, I went ahead and wrote some code in code_action.go. Is there a reason for implementing it as a command?
Secondly, I also tested if completions are viable for a struct fill.
(See GIF below for a small demo on using code action and completions)

Thirdly, a few things I have in mind:

Does Code Action support placeholders?
Is it possible to trigger completion while in a placeholder?

Ahh, I went ahead and wrote some code in code_action.go. Is there a reason for implementing it as a command?

I think either way probably works. I wasn't sure if it necessarily fit under "refactor.rewrite", but I think it works.

Secondly, I also tested if completions are viable for a struct fill.
(See GIF below for a small demo on using code action and completions)

Looks great! Thanks for working on this. I think the code action is a good starting point, and we could possible extend this to be part of completions if we think of a good user experience for it.

Thirdly, a few things I have in mind:

Does Code Action support placeholders?

I don't believe so.

Is it possible to trigger completion while in a placeholder?

I think it should be possible, yes.

This looks great, I'd also like to see a code action to fill a struct from a variable (merge).
type A struct {
  ID string
  Name string
  Phone string
  Email string
}

type B struct {
  Name string
  Phone string
  Email string
  Fax string
}

In this case, the snippet would need to know the source and destination variable names (they should be in scope). Given variables a and b of type A and B respectively, the snippet would produce something like this.
var a A
// Not present on B.
// a.ID = b.ID
a.Name = b.Email
a.Phone = b.Email
a.Email = b.Email
// Not present on A.
// a.Fax = b.Fax

This would prevent a common mistake of forgetting to copy a field from one type to another, missing a field, or overwriting the wrong names (e.g. a.Fax = b.Email). These types of things pop up as the "last line effect", where developers switch off as they keep track of the fields they have or haven't mapped.
Runtime tools exist such as https://github.com/imdario/mergo but these must use reflection, resulting in poorer performance, and the use of a 3rd party library.
Not sure if this should be a completely different thing (merge?), but this feature looks like a good starting point.


Media details do not show depiction

I uploaded a file and set a depiction to it.
The depiction was correctly uploaded, but is not visible in the app even after trying several times on wifi:

https://commons.m.wikimedia.org/wiki/File%3AMouillette.jpg
Strangely it works fine for this other picture uploaded like an hour before, with the same app version: https://commons.m.wikimedia.org/wiki/File%3APink-yellow_strawberries_in_Tokyo.jpg

I just uploaded this and it seems fine?
I can't seem to find mouilette when searching in the app, is there an easy way for me to view this item in app to debug the requests?

You can find it in Explore at the bottom of the list. By the way, there are two "L" characters:

The problem was fetching the depiction labels, the web request asks for a language, in this case "en" and no such entry exists for a few of these so we got an empty list which resulted in the throwing of an exception.
What is confusing me is this parameter to the endpoint

Which sounds like it should help give a result but only works in specific scenarios instead of defaulting to something.
Anyway I am pushing a fix


Requests

As a user I should be able to:

[ ] Apply for vacations
[ ] Apply for compensation days

[ ] form should have:
- [ ] reason for compensation
- [ ] dates of work done that are being compensated

[ ] Request HR letters

[ ]  form should have:

[ ] addressee
[ ] if salary should be mentioned (checkbox)
[ ] if certain dates should be mentioned (checkbox + date picker)

[ ] Request official docs from HR

[ ] form should have:

[ ] comments on what is needed

@0oM4R
see Hr Letters fields

no in case I need an hr letter i can mention I want it on the exact date as I understand, is it?@rkhamis

in an HR letter, sometimes you want specific dates to be mentioned that the company approves you're on vacation during that period. those are the dates meant here

As we discussed with Nayer, We need :

optional fields of start and end date,
mandatory field of date to receive the letter with
can you please confirm that ? @rkhamis


Hard refresh (hard page reload) causes 401 (no serviceWorker token injection)

Issue and Steps to Reproduce

Open your app (everything should work fine, serviceWorker injects headers, all good)
Do a hard refresh of your app (cmd+r)

Versions
Current latest v6.5.9
Screenshots
401 on hard refresh (console):

regular page refresh (network tab):

serviceWorker after hard refresh:

Expected
Hard reload\refresh should work exactly the same way as a regular page load \ page refresh
Actual
Network requests fail on hard refresh
As a temporary workaround I'm doing a page reload in the case if the token is presented and API response is 401

Do note that this is currently how service workers are specified, a hard reload will trigger an unregistration of the service worker. So natively this is by design. I'm unsure whether we can fix this or if this is something third party devs will have to do:
https://stackoverflow.com/questions/51597231/register-service-worker-after-hard-refresh

Ok, thanks!


Support GitHub Enterprise

The app should support using GitHub Enterprise

:+1:

:shipit: ?

Is anyone looking into this feature request? We still need an app for GitHub Enterprise.

+1


Composer update does not emulate module requirements

Cow currently tries to emulate requirements of the recipe being released, but not its modules.
If if the host system has PHP 7.3, but the recipe states >=7.1, then we use composer config.platform to emulate the proper php version and all the extension requirements.
However, if a module has a requirement for an extension that isn't set in the recipe, Cow doesn't know about it and recipe installation may fail.
E.g. kitchen-sink fails currently, because realme requires mcrypt extension, but kitchen-sink itself does not:
- silverstripe/realme 3.1.x-dev requires onelogin/php-saml ~2.10
- onelogin/php-saml v2.15.0 requires ext-mcrypt * -> the requested PHP extension mcrypt is missing from your system.

To fix this we need to add those extension requirements explicitly into the recipe composer.json.
Otherwise, cow currently does not have any means to read this information. This is a chicken-egg problem since to get those files it needs to run composer install, but composer install won't work without those extensions installed.


Add card URL bar

https://github.com/cardstack/boxel/assets/12637010/8f758988-bcc7-408a-b4b3-01c56107f05e

I I took this for a spin and I noticed the following behaviours which I think can be improved:

Looking at the url bar it does not look to me that this is actually a URL:

Only when clicking into it, the URL shows up. I find this a bit unintuitive. I know this is the behaviour in Safari but to me personally it would be more clear and simple if we just displayed the whole URL. It would also be much more convenient for inspection and debugging

When clicking the URL, the left portion will disappear. This feels jarring because the position of thing that you are editing changes. I think it would be better if it didn't disappear:

Unexpected cursor jumps

@FadhlanR can you make this a bit easier to review again? There was a bunch of comments and no replies apart from new code. I suggest adding a summary/description of the notable adjustments, adding screenshots/videos of things that changed, etc. All this helps with making the review process easier. I would also like to get your opinions regarding https://github.com/cardstack/boxel/pull/571#issuecomment-1693432943.

I have made the necessary updates to the code based on all the provided feedback. Here is the list of changes that should be highlighted:

[ ] Added the assertNever call to the updateSubmode function.
[ ] Simplified the implementation of the lastCardInRightMostStack function and removed any unnecessary comments.
[ ] Utilized the cardService.getRealmURLFor function to validate whether the URL corresponds to a recognized realm.
[ ] Revised the error state names in the code URL component. The isInvalidURL state reflects an internal error condition, signifying the validity of the user-input string as a URL. Meanwhile, the notFoundError argument indicates the occurrence of a "not found" error originating from the parent component.
[ ] Updated the codePath within the operatorModeStateService only when the URL points to a valid Card URL.
[ ] Adjusted the behavior of code URLs to ensure the omission of the card type's name display and the consistent visibility of realm information, irrespective of the scenario.

Here is a screenshot for your reference:
https://github.com/cardstack/boxel/assets/12637010/9f4827cb-9e79-44bf-930f-f05fcade8858
(Note: Please disregard the sub-mode switcher button, as it appears to be an error stemming from the main branch. I will address this issue in a separate pull request.)

Good point. We’ll definitely need to load modules too.

@FadhlanR I was wondering. When you input a card url this function is only reserved for card instances but not modules/gts files? The .loadModel is called which essentially will load a card instance

What if we do that on another ticket that really use that modules? Maybe we have a ticket to implement the code editor in code mode.

@FadhlanR the PR i was picking up was just around showing monaco.  it wasnt specifically for modules. @lukemelia should we create a new ticket for loading modules specifically in code mode? or was the intent for this ticket that it encompase both instances and modules?

@FadhlanR the PR i was picking up was just around showing monaco. it wasnt specifically for modules. @lukemelia should we create a new ticket for loading modules specifically in code mode? or was the intent for this ticket that it encompase both instances and modules?

This should be able to open any file in the realm that can be shown in the monaco editor.

Another thing I noticed is any dark colored icon will not be very visible on this search bar. We can already see this issue with the black Cardstack logo for the Drafts realm. We should get some design feedback about this

@FadhlanR the PR i was picking up was just around showing monaco. it wasnt specifically for modules. @lukemelia should we create a new ticket for loading modules specifically in code mode? or was the intent for this ticket that it encompase both instances and modules?

This should be able to open any file in the realm that can be shown in the monaco editor.

i think this means that we also need to incorporate the node-like resolution that the current code route does. I.e. for modules you should be able to load these either with or without their extensions

@FadhlanR I don't see anywhere that we are passing the current file or URL into this component, or binding it to a service property. How is this component going to update its URL properly when a file is opened via a method other than typing into the URL bar?


Can't switch to chapters with percentages in them

The click handler blindly looks for the parent of the clicked element and we wrap the text in a span.

fixed in v2.3.153


Error: Failed to upload:  XML document structures must start and end within the same entity.

Hmm, very strange.
Everything worked before. No new agreements to sign at developer.apple.com and appstoreconnect.apple.com.
Maybe it has something with installing the latest Catalina supplemental update?
Could someone test that please?
Os X: 10.15.6
"electron-builder": "^22.7.0" (also tried on 22.8.0)
"electron": "8.3.1"
"electron-notarize": "^1.0.0"
Error: Failed to upload app to Apple's notarization servers

2020-08-14 15:46:05.395 altool[4345:35320]  Error:
[2020-08-14 15:46:05 MSK] <main>  INFO: ParseError at [row,col]:[1824,2]
Message: XML document structures must start and end within the same entity. Exception's name: javax.xml.stream.XMLStreamException, Exception's message: ParseError at [row,col]:[1824,2]
Message: XML document structures must start and end within the same entity.
java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at com.apple.transporter.launcher.Application.start(Application.java:220)
        at com.apple.transporter.launcher.Application.main(Application.java:658)
Caused by: com.apple.transporter.bootstrap.BundleNotFoundException: bundle=[org.apache.felix.configadmin] version=[1.9.10,2.0.0) not found.
        at com.apple.transporter.bootstrap.BootstrapperPhase1.downloadNeededBundles(BootstrapperPhase1.java:269)
        at com.apple.transporter.bootstrap.BootstrapperPhase1.bootstrap(BootstrapperPhase1.java:97)
        at com.apple.transporter.bootstrap.BootstrapperPhase1.bootstrap(BootstrapperPhase1.java:59)
        at com.apple.transporter.launcher.Launcher.launchBootstrapper(Launcher.java:37)
        ... 6 more
2020-08-14 15:46:05.409 altool[4345:35320]  Out:
2020-08-14 15:46:05.423 altool[4345:35313] *** Error: Unable to notarize app.
2020-08-14 15:46:05.423 altool[4345:35313] *** Error: code -10000 (An error occurred uploading to the App Store.)

    at node_modules/electron-notarize/src/index.ts:87:13
    at Generator.next (<anonymous>)
    at fulfilled (node_modules/electron-notarize/lib/index.js:4:58)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

I just tried to notarize another electronic project that remained untouched for about a month. Same error.
Next I

Okay, most likely Apple had a bug. Now it works fine.


Update vda-subscription-activation.md

Updated scenario 1 as customers’ options for using Windows 10/11 outside their own data centers are expanded to include any Authorized Outsourcer’s shared servers.
fixes#https://github.com/MicrosoftDocs/windows-itpro-docs/issues/11280

@JohanFreelancer9 Please copy review

@frankroj Please merge this PR. Thanks

@frankroj Gentle reminder. Thanks

Confirming with PM if change is ok.

Due to merge conflict, making these changes in PR https://github.com/MicrosoftDocs/windows-docs-pr/pull/8062


Typo in README (flake8 configuration)

There is a typo in README.md, relative to flake8 configuration (Line length section).
As stated in flake8 documentation, underscore instead of hyphens must be used in option name when used in an INI-file.

Can you be more specific?

My apologies, I read too fast.


Make sourcemap order in CF templates deterministic

When multiple lambdas in a stack have sourcemaps, the order of these sourcemaps within the CF template changes between runs. This means that:

sst diff will show these as changes
sst deploy will needlessly redeploy these
sst start also takes longer to fully deploy its stubs

Sorting the list of sourcemaps fixes this problem. This fix can be either applied inside constructs/Function.ts (in useFunctions().sourcemaps.forStack()) or constructs/App.ts (during the creation of the SourcemapUploader CustomResource). It seems cleaner to me to fix it in Function.ts, so this is the approach I took.

Hmm interesting. Let me share it with the team.

Small update:
Using srcKey as the sorting key for this is insufficient. Changes in lambdas will change this hash, which will change the ordering, causing more false changes.
Instead this PR now uses props.handler as a sort key, which should keep the order deterministic in more cases.


Implement LuckPerms for MinecraftForge

GIT - https://github.com/MinecraftForge/MinecraftForge.
Permission API - https://github.com/MinecraftForge/MinecraftForge/blob/1.16.x/src/main/java/net/minecraftforge/server/permission/PermissionAPI.java
Example permissions hook for command (with mixin) -
@Mixin(CommandSource.class)
public class MixinCommandSource {
    @Final
    @Shadow
    private ICommandSource source;

    @Inject(method = "hasPermissionLevel", at = @At(value = "HEAD"), cancellable = true)
    public void onHasPermissionLevel(int level, CallbackInfoReturnable<Boolean> callbackInfo) {
        if (this.source instanceof PlayerEntity) {
        //TODO

Hey, my suggestion for people wanting to use permissions on Forge servers is to install Sponge. :)

Hey, my suggestion for people wanting to use permissions on Forge servers is to install Sponge. :)

Hey, my suggestion for people wanting to use permissions on Forge servers is to install Sponge. :)

At the moment, it does not support versions higher than 1.13, and in addition to advantages it has disadvantages. For example, in real tests, forge will withstand more online than spongeforge

Hey, my suggestion for people wanting to use permissions on Forge servers is to install Sponge. :)

At the moment, it does not support versions higher than 1.13, and in addition to advantages it has disadvantages. For example, in real tests, forge will withstand more online than spongeforge

Hey, my suggestion for people wanting to use permissions on Forge servers is to install Sponge. :)

SpongeForge brings a few issues with it. There are communities out there that are looking forwards a forge only server and simply do not use sponge.


Set correct license classifier in setup.py

As far as I can see, pip-compile-multi was always MIT licensed, but the classifier in setup.py was incorrectly set to "BSD".
This also shows up incorrectly on PyPI:

This PR updates setup.py to use the correct classifier from https://pypi.org/classifiers/
Thanks for this great tool, I like it a lot! 👍

Thank you!


is Possible to set gradient colour to the indicators view

How to apply gradient colour to the NVActivityIndicatorView.

Sorry, it doesn't support that the moment, and I don't have a plan in the near future.


Algolia integration using UUI library

This integration serves as an alternative to the previous dashboard design using the Umbraco 10 backoffice UI and API.

UUI walkthrough: https://drive.google.com/file/d/1qQFROv9oaL5vcKGrMJyrbR2HT0bUOzDO/view?usp=sharing
Umbraco 10 Backoffice UI: https://drive.google.com/file/d/1nZ4t9P_g-bnmcMPsio9zUsd_WzH4Nof4/view?usp=share_link

Looks pretty good so far... think it would be nice to use the UUI library here.  We can get used to it, and by open sourcing the project it'll be a good reference for others.
Just a couple of things that look a bit off to my eye:

The little "eye" icon you need to click on the open up the properties.  Wondering if that's very clear or necessary, could you not just click on the content item name and icon?
The delete property prompt hovers over the "Selected" button.
The icons for the document types are gone, would be nice if they could be retained.

Don't forget to reach out to the front-end team with issues if you can't find how to solve them with the library.  Even if they can't help, will be good feedback on issues being run into in use.

Additional changes to the Algolia integration and an updated UI here.
Updates included with the last commit:

no-dirty-check directive
disable index name field in edit mode - the data is sent to Algolia based on the name of the index. Changing the name of an existing index will result in a new index created in Algolia and schema definition inconsistency.
remove selectable attribute from uui-ref-node component displaying the list of content types. With the selectable attribute, when displaying the list of properties for the content type the focus from the ref node was removed, and there isn't a solution to mark the node as selected in the current UI library. Toggling the properties will be handled by  selecting the content type name.
tag component changed to primary at the suggestion of Niels.


Boolean PropertyAttributes best way to handle propProp="false" as false

Hello community,
I would like to handle this:
<mycomp isactive="false" />
as false value, it is only false if the attribute is not set.
I saw the quote: https://lit-element.polymer-project.org/guide/properties
For Booleans, when the attribute is:

    non-null, set the property to true.
    null or undefined, set the property to false.

So the best way is to write a custom converter for the value? Or is there a better way?
Also, the sample also has a bug, I talk about the prop1, prop2, prop3 sample here:
https://lit-element.polymer-project.org/guide/properties
if I press the changeAttributes button, prop3 always stays true, no matter if the value is prop3="true" or prop3="false"
event if I set
this.setAttribute('prop3', null);
or this.setAttribute('prop3', undefined);
the value stays true.

In HTML boolean attribute are "truthy" based on their presence not their value.

The way to set a boolean attribute to "false" is to remove it:
this.removeAttribute('prop3');

This is what LitElement does if you set the prop3 property to false:
this.prop3 = false;
await this.updateComplete; // Wait for the LitElement to update
this.hasAttribute('prop3'); // false

https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes#Boolean_Attributes

My Problem is just that I use the Blazor Framework on top of my webcomponents and there binding works like this:
<my-link isactive="@_isActive"></my-link>

@code
{
        bool _isActive = true;
        public void MakeInactive()
        {
              _isActive=false;
        }
}

Removing attribute on false requires lots of not nice workarround code.
So even if _isActive would be false the link would still be active, since the attribute is truthy...
Also if I write in my examples
<my-link isactive="true"></my-link>
The first thing everybody tries to do is set isactive="false" and wonder why it is not working...
This would be nasty bugs nobody would find quickly.
So I will need to write for all my booleans custom converter.

We could offer a convert for this case, but the attributes would not be boolean in the HTML sense. They would be more akin to enum attributes.

Thanks for the quick answer :)
like a BoolEnum with Values True and False?
So far this was my solution:
Is there a better way?
    static get properties() {
        return {
            itemid: { type: String },
            label: { type: String },
            href: { type: String },
            isactive: { converter: getBoolConverter() }
        };

export function getBoolConverter() {
    return {
        fromAttribute: (value, type) => {
            if (value && value != 'false') {
                return true;
            }
            else {
                return false;
            }
        },
        toAttribute: (value, type) => {
            if (value) {
                return 'true';
            } else {
                return 'false';
            }
        }
    };
}

What you've done seems pretty reasonable. Needing custom logic around handling attributes is exactly why we exposed the converter in the first place. Good job figuring that out.

@sorvell I've seen this come up a few times, and HTML does have a couple of enum-values attributes, including ones that accept literal "true" and "false" strings. I wonder if we should offer an EnumConverter in a module?

What you've done seems pretty reasonable. Needing custom logic around handling attributes is exactly why we exposed the converter in the first place. Good job figuring that out.

Must say the documentation is quite good if you actually ready it ;) @justinfagnani @sorvell


This Project is exculded from Hacktoberfest2023

Yes, i'm aware of this and that's why this project does not have hacktoberfest or hacktoberfest-accepted labels. There is some information about the situation in the Tasks section of README.


FileNotFoundError: [WinError 2] The system cannot find the file specified

Hi,
I am trying to reproduce "play a web stream" example from here. I am using the stream URL that is suggested in the documentation of python-sounddevice, which is http://icecast.spc.org:8000/longplayer.
However, I'm getting the error FileNotFoundError: [WinError 2] The system cannot find the file specified at this line:
info = ffmpeg.probe(args.url)

Python version: 3.8.3
OS: Windows 10

Do you have any suggestions how to fix this error?
Thanks.
BR,
L.

Thanks for the report!
Please provide the unabbreviated error message and the actual command you were using, see also https://python-sounddevice.readthedocs.io/en/0.4.1/CONTRIBUTING.html#reporting-problems.


add error selection feature

I'd like the ability to select which mypy errors I want to fix, in both the CLI as well as the API. This is primarily driven by the want to iteratively suppress errors, rather than suppressing all of them at once. This could probably be done by adding a multiple flag (--fix-error <name of mypy error>) to the CLI and an Optional[List[str]] input to the API. By default, if not supplied, it could fix all errors.

Thanks for the suggestion! I've drafted a working version over at logging.

this is added in v0.0.1b6


add compress flag to backups

Reordered help file to be alphabetic as well

Looks fine to me

Looks pretty good to me. Though your option list has gotten to the point that you would save yourself some headache by using bash getopts.


Add support for on-demand JREs for JDK16+

Add a new argument to the build scripts: --create-jre-image.
A JRE image is produced when this argument is set. It is optional for
JDK16 and above.
Signed-off-by: Violeta Sebe<EMAIL_ADDRESS>

@AdamBrousseau @gdams please review, thank you!

No changes to the pipelines configuration files in the ci-jenkins-pipelines repo for JDK8 and 11 are required due to https://github.com/adoptium/temurin-build/blob/457f16cda71c3d7c754f6d63bd8513cf2e5cd6ff/build-farm/make-adopt-build-farm.sh#L219

No changes to the pipelines configuration files in the ci-jenkins-pipelines repo for JDK8 and 11 are required due to
https://github.com/adoptium/temurin-build/blob/457f16cda71c3d7c754f6d63bd8513cf2e5cd6ff/build-farm/make-adopt-build-farm.sh#L219

Aah, gotcha. Nevermind, then.

Are we good to merge this then?

Happy for this to go in.


refmt globstar doesn't include base directory

Version: 3.0.4
Expected
Passing refmt a globstar, like in refmt --in-place src/**/*.re, on a project that looks like:
src
  one.re
  subdirectory
    two.re

Should format both one.re and two.re.
Running it on a project without sub-directories should work.
Actual
Running refmt --in-place src/**/*.re on a project with out sub-directories in src throws this error:
$ refmt --in-place src/**/*.re
refmt: FILENAMES... arguments: no `src/**/*.re' file

Running it on a project with some files in sub-directories only formats the .re files in the directories.
Alternatively, refmt --in-place src/*.re only formats the files not in sub-directories, as expected.
It seems like the globstar /**/ is being treated as meaning "at least one directory" instead of "zero or more directories" as expected.
I poked around the refmt code a bit, but I'm not sure where the file glob is actually getting parsed. If someone could point me to it, I could try to make a PR.

yeah the globstar is a feature of your shell, not of refmt.


pred_contrib doesn't add up to the prediction value (for some objective functions)

Description
As the title says, pred_contrib output doesn't add up to the actual prediction.
The problem seems to be related to the choice of the objective function for a regression model, in particular affected are poisson, gamma and tweedie.
This happens with tree-linear=False (as it is the default and I don't change it).
Reproducible example
import lightgbm as lgb
import pandas as pd
from sklearn.datasets import load_boston

data = load_boston()
X = pd.DataFrame(data.data, columns=data.feature_names)
# scale the target to make the difference more pronounced
y = data.target * 1000

# try out different objective functions and for each compare prediction with the sum of feature contributions
for obj in ["l2", "l1", "huber", "fair", "poisson", "mape", "gamma", "tweedie"]:
    model = lgb.LGBMRegressor(objective=obj).fit(X, y)
    preds = model.predict(X)

    shap_values = model.predict(X, pred_contrib=True)
    sum_of_shap = shap_values.sum(axis=1)

    diff = preds - sum_of_shap
    # print out the difference between the sum and the prediction *for the first sample*
    print(
        f"{obj}: Difference= {diff[0] / preds[0]:.1%}.    Prediction = {preds[0]:.1f}, sum of shap values = {sum_of_shap[0]:.1f}"
    )

Output:
l2: Difference= -0.0%.    Prediction = 24274.4, sum of shap values = 24274.4
l1: Difference= -0.0%.    Prediction = 25055.9, sum of shap values = 25055.9
huber: Difference= 0.0%.    Prediction = 22541.8, sum of shap values = 22541.8
fair: Difference= 0.0%.    Prediction = 57641.8, sum of shap values = 57641.8
poisson: Difference= 100.0%.    Prediction = 24978.1, sum of shap values = 10.1
mape: Difference= -0.0%.    Prediction = 25148.8, sum of shap values = 25148.8
gamma: Difference= 100.0%.    Prediction = 24221.7, sum of shap values = 10.1
tweedie: Difference= 100.0%.    Prediction = 24475.2, sum of shap values = 10.1

Environment info
LightGBM version or commit hash:
lightgbm = 3.3.3
Command(s) you used to install LightGBM
conda install lightgbm

python = 3.8
Additional Comments
The problem is related or same as in #3998, but that issue is closed and I have no right to re-open it. Feel free to close this one, if you prefer to re-open and follow-up in the previous.

I'm going to close and lock this as a duplicate of #3998. I've reopened #3998, and let's keep the discussion focused there.
You can trust that when you comment on a closed issue, every maintainer (and in fact many of the 450 watchers on the repo) will be notified.


Julie is creating odd Role Binding for Schema Registry subjects

Describe the bug
Using the following descriptor:
---
context: context
source: test
projects:
  - name: "foo"
    consumers:
      - principal: User:greta
    topics:
      - name: "foo"
    schemas:
      - principal: User:greta
        subjects:
          - "context.test.foo.foo-key"
          - "context.test.foo.foo-value"

results in creation of
  Principal  |      Role       | ResourceType |               Name               | PatternType
-------------+-----------------+--------------+----------------------------------+--------------
User:greta | ResourceOwner   | Subject      | Subject:context.test.foo.foo-key | LITERAL
User:greta | ResourceOwner   | Subject      | Subject:context.test.foo.foo-key | LITERAL

Expected behavior
Imho the "Subject" in the rolebinding role name is superfluous and incorrect
I would rather expect this:
  Principal  |      Role       | ResourceType |               Name               | PatternType
-------------+-----------------+--------------+----------------------------------+--------------
User:greta | ResourceOwner   | Subject      | context.test.foo.foo-key | LITERAL
User:greta | ResourceOwner   | Subject      | context.test.foo.foo-key | LITERAL

UPDATE: This happens also when creating Subject role bindings via custom roles:
roles:
  - name: "subjectowner"
    acls:
      - resourceType: "Subject"
        resourceName: "test"
        patternType: "Literal"
        host: "*"
        role: "ResourceOwner"

Thanks for reporting this issue. I add a fix, including regression tests as well. This should be fixed with https://github.com/kafka-ops/julie/pull/405, this is, for now, available in master and in the latest nightly release build from https://github.com/kafka-ops/julie/actions/workflows/nightly-artifacts-build.yml
It will be included in next release, it would be awesome if you can grab from master and let me know here all is good.


Agents don't see virtual interfaces?

Agent information doesn't show virtual interfaces? When run in a VM (limited testing), only the local interface is shown as having an address. Possible go-ism?

This seems to be working? Not sure what I did. Closing for now, will reassess if it starts happening again.


fix draw-interactions

Fixing issues with user-interactions while drawing->selecting->drawing...
OlDrawHandler:

use _setSelection, setSelection and this._select.getFeatures().clear in correct manner
introduce _setSelected-Method for calls, where feature is not in vectorsource per ID available

DrawToolContent:

add clearDescription to tool.activate-function to reset Description for each new Drawing
symbolSrc is now created on each drawing-type and call of changeColor

IconService:

start loading of icons in constructor-call to enable access in getIconResult-method

draw.action/draw.reducer:

add clearDescription()

add/update tests

very nice. still one Problem..
Wehen click at tow overlapping Geometrys both of them are going to be selected...

ahh...thanks a lot, i missed that and will change it ASAP!


Show IP address on a map

It might be an interesting feature to have a map that shows the locations for the ip addresses users are logging in from.

https://dev.maxmind.com/geoip/geolite2-free-geolocation-data allows 1000 requests per day to their API with a key, that's a pretty reasonable limit, plus I guess it could be an option to pass through your own keys via environment variables?
Their response looks like this: https://dev.maxmind.com/geoip/docs/web-services/responses#geoip2-city-body-example (the free API is less accurate, though)
I'm currently having a play at using this for something similar on a fork (specifically, being able to ad-hoc click the IP of a given session to show geodata for it), but it could easily be extended to build a map with the result coordinates 🤔
What do you think?

Hey @DaftFuzz , thanks for looking into this, i had looked into it once before but couldn't really find an api with a decent quantity of free requests. il look into using this one aswell as what other ones are available

In case it can help, I found a free geoip API: https://ip-api.com/
The limit is 45 requests per minute.


indoor_localization: 0.0.1-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository indoor_localization to 0.0.1-0:

upstream repository: https://github.com/inomuh/indoor_localization.git
release repository: https://github.com/inomuh/indoor_localization-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.7.2
previous version for package: null

indoor_localization
* Update .travis.yml

Should I re-release the package with bloom  after fixing them?

If you want to change the version number you'll have to rerun bloom after making the change upstream. If you want to just change the branch you can just edit it in this PR branch or even through the web ui.
Note that we're holding new releases for a few days. https://discourse.ros.org/t/preparing-for-kinetic-sync-2019-03-01/8078/2

Replaced by #20437


Adding an option for console warnings on throttle retries

Adding an option console_warn_on_request_retry for console warnings every time the client retries a request when throttled.
There was no easy way of seeing when the library is retrying requests without going into the source code. While developing I ended up having a lot of loops in the background with requests not knowing about it.

@amz-tools We can discuss if you have suggestions on a different solution.

Hi @YuriiHerasymchuk,
no, thanks for the PR. Will have a look at it latest on the weekend.

Hi @YuriiHerasymchuk,
merged the PR. Only changed some minor things in the README.


{Refactor} Remove all occurences of assertDictContainsSubset

Description
Python 3.12 removed assertDictContainsSubset, so we are removing all usages by replacing it with subset assertion.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Refactor

Why was this put back into the backlog? It's not a breaking change.

Sorry, we don't have the bandwidth to deal with Python 3.12-related PRs currently.
We'll review and merge them when we decide to move to 3.12.

Just a note: You don't need to deal with all Python 3.12-related PRs at once. If you merge trivial ones (maybe this is not, but #29120 definitely is) first, the work would be much easier in the future and it wouldn't feel huge.

maybe this is not

Indeed. The removal of assertDictContainsSubset is documented at https://docs.python.org/3/whatsnew/3.12.html#id3

But the doc didn't mention the recommendation for assertDictContainsSubset's replacement.
I have verified using assertLessEqual and .items() works as expected.
import unittest

a = {'a': 'b'}
b = {'a': 'b', 'c': 'd'}

class TestA(unittest.TestCase):
    def test_dict(self):
        self.assertDictContainsSubset(a, b)
        self.assertLessEqual(a.items(), b.items())

It'll be better to get Python developers' confirmation: https://github.com/python/cpython/pull/28268#issuecomment-2228070826.

The assertLessEqual and .items() approach is also mentioned at https://github.com/pytest-dev/pytest/issues/2376#issuecomment-338407594


No DbContext was found in assembly 'NFine.Code'. Ensure that you're using the correct assembly and that the type is neither abstract nor generic.

启动项目为NFine.Web,运行Add-Migration报错啊
PM> Add-Migration
位于命令管道位置 1 的 cmdlet Add-Migration
请为以下参数提供值:
Name: NFineDb
No DbContext was found in assembly 'NFine.Code'. Ensure that you're using the correct assembly and that the type is neither abstract nor generic.

字符串没改development那个，导致一直无法连接数据库，改了就可以了！


add_convs in simplify_algebra cause error for opset7/tf_pnasnet_large

When compile the above model, it generates the following error message
src/include/migraphx/op/concat.hpp:63: CONCAT: Non-axis dimensions should match
By removing add_convs, the error message is gone.
Also applied to the same model in other opset versions.

Where can I get a copy of this onnx file?

You can unzip the file 20191107.zip in the folder /home/pfultz/onnx in the test machine on which you have a lot of onnx files.


Escape all single quotes, not just the first

Motivation
If a model contains more than one single quote, only the first is escaped. This will escape others.
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docs have been updated (if applicable)
[ ] Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

Create a content model with a name that has two single quotes, e.g. Template - Category Page - 'Other' Categories
Prior to this fix, running the generator will result in the following error:

SyntaxError: ',' expected. (28476:60)
  28474 |                 id: '...',
  28475 |                 externalId: undefined,
  28476 |                 name: 'Template - Category Page - \'Other' Categories',
>       |                                                            ^
  28477 |                 elements: {

Observe that the first quote was escaped, the second was not.
After this patch, the model generates as expected.

Thanks for the PR @mboynes! :)
the code looks right, the only nitpick is that you could use replaceAll :) I know that this is not supported for node < 15. However, now the LTS is 18 so it's good.
Also in this repository, we are using https://www.conventionalcommits.org/en/v1.0.0/. Could you please rename the commit to follow this convention?
Thank you.

Thank you for letting us know. I've fixed it via https://github.com/kontent-ai/model-generator-js/commit/636d772f178da4f0ab2fd9931e1cf7d0e955fd78 so that we do not have to wait for changes in this PR ;)


Not showing in release mode

I use this in my app, but when I switch to release mode, the views stop showing. What can I do please ? did I go wrong somewhere ?

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

ID: a781e5d5-c0c4-4bc0-2049-97442139bee3
Version Independent ID: 94e2df3f-c624-98d8-2edb-ef98134835b9
Content: Native Views in XAML - Xamarin
Content Source: docs/xamarin-forms/platform/native-views/xaml.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @DamienDoumer
What platform are you having the problem on?

I am using this code in my app. but it says.
The type or namespace name 'App' could not be found (are you missing a using directive or an assembly reference?)

 <ios:UITextView
                                Text="{lEx:CMSTranslate terms.termsDescription}"
                                View.HorizontalOptions="FillAndExpand" />

Working fine in Debug mode, but gives issue in Release mode saying default constructor not found for UITextview.
Please suggest, do I need to linkskip something?


refactor(dev): update postgresql-rds image tag

postgresql-rds "-1" images are being deprecated; moving to postgresql-rds:cyndi-13 instead
Secure Coding Practices Checklist Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[ ] Input Validation
[ ] Output Encoding
[ ] Authentication and Password Management
[ ] Session Management
[ ] Access Control
[ ] Cryptographic Practices
[ ] Error Handling and Logging
[ ] Data Protection
[ ] Communication Security
[ ] System Configuration
[ ] Database Security
[ ] File Management
[ ] Memory Management
[ ] General Coding Practices

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.79%. Comparing base (0d66408) to head (5e0ef9b).
Report is 33 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #494   +/-   ##
=======================================
  Coverage   74.79%   74.79%
=======================================
  Files          25       25
  Lines         849      849
=======================================
  Hits          635      635
  Misses        214      214

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/retest

/retest

/retest

/retest


🐛 [flutter_performance] Requests not collected for gRPC

Does the flutter_performance plugin record requests for gRPC? What about grpc-web?
What if I'm hitting  an ip-based domain over HTTP? 90% of my requests are to a static IP address over gRPC (on device) and grpc-web (on web), however I'm seeing 0 metrics in the firebase console.
Is this a known deficit or is there something I can do to fix this?

Hey @steeling, the Firebase performance plugin was reworked at the tail end of 2021. Not sure if the metrics are now appearing in your console?


[Easy] Switch default colors to MPL defaults?

Since 2.0, the MPL defaults (c0, c1, ...) are very good. Should we not transition there with, for example, our line plot defaults?

Which line plots do you have in mind?

All plots that use mne.viz.utils.COLORS, i.e., the annotation plots.

Sounds good to me. I think we can just change these and put a note in the API section.

See PR .. is this what you meant @jona-sassenhagen? Here are the colors.
However, this change means going from
['b', 'g', 'r', 'c', 'm', 'y', 'k', '#473C8B', '#458B74',
 '#CD7F32', '#FF4040', '#ADFF2F', '#8E2323', '#FF1493']

to
[u'#1f77b4', u'#ff7f0e', u'#2ca02c', u'#d62728',
 u'#9467bd', u'#8c564b', u'#e377c2', u'#7f7f7f', u'#bcbd22', u'#17becf']

i.e., fewer colors. Is that an issue?

We could just stick with 'c0', 'c1', 'c2', ... right?

And we do have conditionals that look for 'r' in the color cycle somewhere in the viz plotting code having to do with annotations. So that will need to be changed.


integrate new metadata schema (4-2-stable)

Hm, I'm stumped as to why https://github.com/d-w-moore/irods_capability_indexing/commit/f2ae92062a5cd3d503b7c9c718b5e839c6965e7f , although on the source branch for this PR, does not show up here.

I see it here... https://github.com/irods/irods_capability_indexing/pull/64/commits

make sure to work through the 'hidden conversations' on this page... some are now outdated.

@trel - I've reviewed the hidden conversations, and everything seems resolved.  The plugin tests pass on all OS'es ... so, more or less ready for merge.  If more #, squash, and issue labelling are needed, let me know.

still see a lot of unresolved comments/conversations here.   may be of no consequence but we should agree to that :)

Most of them have an unresolve button, meaning they've been resolved?  There is the one I see, with a tag for Mike Conway's notice, but that doesn't seem to have attracted any attention.

there are "19 hidden conversations"... flip those open...

OK, this is squashed down to the feasible minimum .

can use the housekeeping issue [44] for the two versioning commits

ok, done with the issue attribution, except for the #'ing

All remaining codacy points are irrelevant / inaccurate.

just a couple things, and then yes... please # when ready.


ovirt: perform snapshot removal sequentially

Perform snapshot removals sequentially, as the first snapshot removal will block the others and will result in only one snapshot being removed
Signed-off-by: Benny Zlotnik<EMAIL_ADDRESS>

and please correlate this with https://bugzilla.redhat.com/2053183

@mansam Do you see issues with this approach or is it a sensible solution to cleaning up oVirt's snapshots sequentially?

@bennyz let's continue this in kubev2v/forklift


security/clamav: reworked menu, added logviewer

Added Logviewer for clamd.log and freshclam.log

Merged, thanks. :D


Unstyled Tailwind class autocomplete in omnibox

Basic level of support for tailwind class autocompletion in the omnibox:

[ ] All tailwind base classes hardcoded into a list
[ ] Use react-select-virtualized or similar to support feeding that entire list into a select control
[ ] Expose the above select control in the omnibox when typing class names

This ticket needs more detail on how we go from "user starts typing" to "this is supposed to be a tailwind class so let's offer autocomplete" - @maltenuhn to advise


Bridge the formal block type in a block-to-func thunk

When building a block-to-func reabstraction thunk, be sure to bridge
the block's formal parameter and result types, because otherwise
the bridging code gets real confused.
Fixes rdar://35402696

@swift-ci Please test and merge.

@swift-ci Please smoke test Linux.


Support for Nx > 20>

Hi,
I wanted to know if you are planning to support NX 20+ versions?
Thank you

@francoisauclair911 I do plan to support nx 20, but we use PHP less than we used to, If you're willing, this project is pretty simple typescript and I'd be happy to accept a PR with the upgraded nx version.


Update Spinner API with UnsetFinalText

What this PR does / why we need it
This PR adds a new Spinner API to unset the spinner's final text if it has already been set.
Which issue(s) this PR fixes

Fixes #
Describe testing done for PR

Release note

Additional information
Special notes for your reviewer

Can you explain why this will be needed for anyone in the PR description?

Can you explain why this will be needed for anyone in the PR description?

Updated.


Degrade to stack string?

The current code is something like this:
var capability = require("capability");

var StackAdapter;

if (capability("Error.captureStackTrace"))
    StackAdapter = require("./v8");
else if (capability("Error.prototype.stack") && capability("arguments.callee.caller"))
    StackAdapter = require("./non-v8");
else
    StackAdapter = require("./unsupported");

module.exports = StackAdapter;

By environments with strict mode only the Error.prototype.stack can be accessible. Wouldn't it be good to degrade to the stackString in these environments?

The problem with these environments, that I cannot omit the frames generated by the e3 lib using the adapter.presentStack(). These are the UserError.constructor, the StackAdapter.presentStack and the FrameStringSource.captureFrameStrings. Probably we can add another one, if we want to calibrate for arguments.callee.caller. So yes, the amount of the shifted frames can be guessed, it will be 4. If we add to the documentation that using the stackAdapter in other classes or adding another UserError.factory is not recommended, then it is possible to do this.

My problem with this that I don't know of any browser that does not support arguments.callee.caller or enforced strict mode.

If I can do this without arguments.callee.caller then only the Error.prototype.stack will be a required dependency of non-v8 apis. This way I won't need to degrade to stack string, since every browser with stack string will be supported, so I can keep the current code and degrade to empty stack. I have to move the arguments.callee.caller check from the adapter selector to the non-v8 adapter definition and support an alternative way by browsers which does not support this feature. Future browsers might enforce strict mode, so this will be the most future proof solution.

I close this in favor of #21


Remove .00 from the format

How can I set the number of decimal number after . ?
For example, I want $100.00 to be $100.

Hi @kiddo13,
Please, update to version 0.8.5 and use new fraction feature:

<input type='text' ng-currency fraction='0'>


Adds events for External Links

Ticket: https://trello.com/c/eo8gJE4C/713-data-tracking-for-new-flow

@galund - updated with your changes.


Ionic App fails to recognize the "Updated"Code in Visual Studio Code and it still executes the same old code it had processed 3 days ago....

Hello There ,
Short description of the problem:
I am facing an issue on Windows 10

Where the app is unable to recognize my updated visual studio code
Live Reloading is not happening

What behavior are you expecting?
When saving the changes should be reflected in the ionic app with the live-reload .
Steps to reproduce:
1.Run ionic serve then open chrome for localhost:8100
2.Edit some file in the project and save

Which @ionic/app-scripts version are you using?
$ ionic info
cli packages: (C:\Users************************\AppData\Roaming\npm\node_modules)
@ionic/cli-utils  : 1.19.2
ionic (Ionic CLI) : 3.20.0

global packages:
cordova (Cordova CLI) : 8.0.0

local packages:
@ionic/app-scripts : 3.1.8
Cordova Platforms  : none
Ionic Framework    : ionic-angular 3.9.2

System:
Node : v10.1.0
npm  : 6.0.1
OS   : Windows 10

Environment Variables:
ANDROID_HOME : not set

Misc:
backend : pro

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
This was the error i got ->

It was saying that "support()" is not a function .
The "Black box "shows the code it refers to . I have removed any trace of the word "support()" in the code and yet it refers to the old code ......
I have added two more functions which the app still did not recognize .
Note :
The above issue persists despite
1. Refreshing the browser a lot of times
2. Saving the code a lot of times
3. Uninstalling and Re-installing the entire ionic, cordova, Visual Studio Code and other
softwares (mysql,mongo db,etc.... )too
4.Checking the live reload option in chrome
Any help would be Appreciated and Thanks in Advance ^-^ 👍

Hello ? Anyone Out There T^T  .......... Hears only Crickets Chirping.........
(Sorry if i`m a bit impatient)

I have the same problem

@fdambrosio  no reponse from the devs yet , looks like this is gonna be a dead thread soon T^T

This is a difficult issue to track down. In our tests, we have not been able to replicate this. So not sure how much more we can track down. If you dont mind, can you create a minimal test case app, and push it to github? This will help us best debug the issues.

Thanks a lot for the reply @mhartington . Ok I will upload the link asap

I have the same problem.


First Contribution by Tanishka Goel

Name: Tanishka Goel
Project Name: signac (Link: https://signac.io/)
Link to Project Governance Model: https://signac.io/governance/
What is signac?
The signac is a framework that helps researchers manage their project data with a well-defined, indexable storage layout for their data and metadata. It streamlines the post-processing and analysis, making data collectively accessible.
Governance Model
The governance model of this project is a derivative of the Meritocratic Governance Model and has the following features: Consensus-based, inclusive, and open to all (community-based).

The project is open source and is managed and sustained by people who have an interest in the project and are willing to be a part of it whether by using the framework, contributing to the development and upgradation of the project, or managing the project community.
It has various specified project roles, each with a different responsibility (discussed below). A community member can identify with a specific role based on their technical abilities, experience with the project, and commitment to the project.
It has consensus-based decision-making, achieved with inclusive majority voting.  The decisions are required for progressing roles in the community and managing the future workflow of the project. These are made by the voters considering the involvement of the member, responsibilities, and the accomplishments of their current role; deemed suitable or unsuitable for future role or involvement.

Project Roles

Users
Community members who use the project with no special requirement to become one. These members help in : sharing the framework and creating awareness about it, delivering their perspective for more improvements and bug fixing, citing it in their academic works, or giving moral support ( e.g. "I liked your product").

Contributors
Support new users, contribute code and/or documentation, identify requirements, assist with project infrastructure, and attend developmental meetings. They submit code via pull request and may be asked to review other PRs by the committer.

Committers
Primary reviewer for pull requests across all repositories, may triage or close issues and is permitted to merge it and attend development meetings via Slack or Gitter channel. This role requires some technical expertise.

Project Maintainers
Maintaining a project roadmap, scheduling and creating releases, managing all the GitHub repositories, triaging or merging PRs, and representing the project to the community. Also, involved in decision-making.

Administrators
They manage the infrastructure required for the continuity of the project. This includes - GitHub, domain name, social media accounts, Slack channel, and website.

Project Managers
Managers ensure the forward progress of the project. The previous managers contact the new occupant for the position.

Triage Manager
Consists of all maintainers and all committers. They are responsible for contacting authors and reviewers of stalled PRs and checking on progress, tagging new issues, and bringing critical issues to notice to the team.

Community Manager
Responsible for scheduling and setting the agenda for biweekly signac meetings, maintaining social media accounts, and preparing product updates for the newsletter.

Decision-Making

Committers: (Technical expertise and a high level of understanding of the project strategy and alignment towards its objectives is required) Members apply for this role by self-nomination. The project maintainers may accept or reject the nomination by majority vote. The project maintainers may also revoke a committer by majority vote if warranted (e.g. due to long inactivity).
Project Maintainers: (Exceptional commitment to the signac project and overall strong understanding of project strategy is required) Project maintainers are chosen by existing maintainers by 2/3rd votes. The maintainer may also be removed by 2/3rd votes if required.
Project Managers: Managers are selected by rotation from a pool of project maintainers or volunteer project committers (all committers are allowed for a triage manager position). They are selected periodically in the alphabetic order of their last name.

How easy/difficult was the governance model to find and understand for me?
The governance model was easy to locate on the website. It was also easy to understand, given that I had a little idea about the open-source terms being used and have considerable fluency in the English language. I think people who are completely new to open-source or do not have fluency in English may find it a bit difficult to get through. But, beginners to open-source who are at least familiar with the terminology would easily grasp the model.
Overall level: Moderate (for understanding), Very Easy (to find)

@arliss-NF Please have a look at it. I will be glad to have your suggestions. :)
Thank you!

Hello Tanishka, under the roles, we were asked to list them out.
So you don't need to explain what they do, all you have to do is list them out.

Hello Chimnwendum, thank you for your suggestion! I'll edit it :)

@tg-100 - great first task.  I appreciate how you responded to feedback from the community. 💯

@arliss-NF Thank you so much!


Question: How to reduce symbol length?

Would appreciate some advice! I read the advice section in the docs and used boxing in each function and used choice for any >=3 or's. My compile time was okay (~10 seconds on an LTO'ed build, excl. dependencies) but I get giant symbol names which feels wasteful, for example this 29 byte function get a 30kb+ name, is the only option here stripping the binary?
Some naive grepping give me this histogram, I attached the full name of the longest symbol.
symbol.txt

Example of my code:
pub fn class<'tokens, 'src: 'tokens>() -> impl Parser<
    'tokens,
    ParserInput<'tokens, 'src>,
    Class<'src>,
    extra::Err<Rich<'tokens, Token<'src>, Span>>,
> + Clone {
    let tags = tags();

    let kw = just(Token::Keyword(Keyword::Class));

    let name = select! {
        Token::Ident(name) => name
    };

    let maybe_extends = just(Token::Keyword(Keyword::Extends))
        .ignore_then(path())
        .map(Some)
        .or(empty().map(|_| None));

    let body = just(Token::Ctrl('{'))
        .ignore_then(
            choice((
                var_decl()
                    .then_ignore(just(Token::Ctrl(';')))
                    .map(|vardecl| (Some(vardecl), None, None, None)),
                function().map(|func| (None, Some(func), None, None)),
                typedef().map(|typedef| (None, None, Some(typedef), None)),
                import().map(|import| (None, None, None, Some(import))),
            ))
            .repeated()
            .collect::<Vec<_>>(),
        )
        .then_ignore(just(Token::Ctrl('}')))
        .map(|body| {
            let mut members = Vec::new();
            let mut functions = Vec::new();
            let mut typedefs = Vec::new();
            let mut imports = Vec::new();
            for item in body {
                match item {
                    (Some(vardecl), None, None, None) => members.push(vardecl),
                    (None, Some(func), None, None) => functions.push(func),
                    (None, None, Some(typedef), None) => typedefs.push(typedef),
                    (None, None, None, Some(import)) => imports.push(import),
                    _ => unreachable!(),
                }
            }
            (members, functions, typedefs, imports)
        });

    tags.then_ignore(kw)
        .then(name)
        .then(maybe_extends)
        .then(body)
        .map(
            |(((tags, name), extends), (members, functions, typedefs, imports))| Class {
                tags,
                name,
                extends,
                typedefs,
                imports,
                members,
                functions,
            },
        )
}

You can use .boxed() to switch to dynamic dispatch, which avoids large types. See here. Note that this doesn't always come with a performance hit: LLVM is often able to devirtualise the parsers and still statically knit them together. Sometimes, performance can even improve!

I only boxed expressions, statements and primitives like ifs, I will try boxing them all tomorrow and report back. Thanks

I boxed everything that is longer than 3 levels that doesn't require Send (lexer part) and got down to about 25 kb and the next largest is 14 kb. My build time got noticeably slower actually (I suppose LTO is trying to see through the box?) but it's still acceptable. I attached the new longest symbol here if there is anything to see but thanks for the help! If there is no more thing to do feel free to close this, thanks!
symbol.txt

I think that's about all there is, sadly (at least: until rustc gets a bit smarter about symbol generation for nested types). The same thing happens with complex Iterator chains, although of course chumsky's pervasive use of the combinator pattern makes it much more common here.

Thanks for the help, and the project overall of course!


Add accessibilityIdentifier to EKButtonView init

Is your feature request related to a problem? Please describe.
I use accessibilityIdentifier for UI tests and enhanced usability for users with disabilities. However, I cannot assign the accessibilityIdentifier to the UIButton that is hidden inside EKButtonView.
Describe the solution you'd like
Add accessibilityIdentifier as an optional init parameter to ButtonContent.init(), that then gets assigned to the underlying UIButton.
Describe alternatives you've considered
Additional context

@smileypop thanks for noting this subject. SwiftEntryKit's labels, images and buttons are now supporting accessibility (1.0.4) 🎉


update of master_root_volume_size

I noticed that while you can set the new value of compute_root_volume_size and new nodes will have the desired volume, setting master_root_volume_size and updating the cluster does not make any effect. I also tried to restart the master node but without luck. Is there any way to update the size of the master root volume?

I don't think that is currently possible. Because the volume is created when the chef recipes are built which is done only once on cluster creation for the master volume. For the compute nodes, it is done each time a compute node comes online. I will keep investigating to see if there is another way this could be achieved and also file an enhancement for this feature.

There is not a way through CfnCluster.  CfnCluster's mechanism for modifying instance parameters is the AWS CloudFormation stack that it creates when you create the cluster.  CloudFormation's solution to the change would be to recreate the instance, which isn't what you want.
However, the master root volume is just an ordinary EBS volume.  If you follow https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ebs-modify-volume.html, you should be able to resize the root volume.  You are likely to have more luck if the base os is Amazon Linux or Ubuntu 16.04, but it should work with any Linux distro that the instructions say is supported.  Note that this process isn't entirely risk free; changing the size of a live filesystem is as tricky as it sounds.

Just to correct what I formerly said, chef recipes do not control the root_volume_size. It is controlled in the cloud formation template and any change to an block device mappings of an instance needs to launch a replacement instance.
See: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-instance.html#cfn-ec2-instance-blockdevicemappings

I see. Shouldn't then the cfncluster warn me that this setting has changed but won't be updated? It just silently did the update.

Release 2.8.0 is out (and 2.8.1 too). One of the enhancements of the 2.8.0 is: "Renewed user experience and robustness of cluster update functionality" and this failure during the update will be prevented.
I'm going to close this issue, feel free to open a new one if you encounter similar problems.


Add compute=False option to to_snowflake

Users sometimes want finer control over exactly when this compute is triggered. This matches what we do in other to_* methods like to_csv, to_parquet, etc.

Will plan to merge in a bit if no further responses. This should be a pretty uncontroversial change.

@jrbourbeau maybe go ahead and merge now?

Thanks for the reminder. Forgot to circle back to this one