Update to use SObject. getPopulatedFieldsAsMap

You should be able to remove the JSON hack with this method instead, SObject.getPopulatedFieldsAsMap.

@afawcett I like your idea, but I do not think that would simplify the number of lines by much. Maybe it could increase the performance of the code, which will be important because this code could be called multiple times within the same transaction.
One of the things I considered when I first built this code was that I needed to check a query like this:
[SELECT Id, Name, Account.Name, Account.BillingCountry, (SELECT ID, CaseNumber, ParentId, Parent.CaseNumber FROM Cases) FROM Contact]
I need to work with all the records because I can't guarantee the first record will have all the fields I need to check.. Let me give you an example:
If the first Contact does not have an account, or the first contact's account does not have a billing country, or the first case does not have a parent... the first record will not have all the fields that need to be tested... This is the reason I decided to check all the contact records, all the parent account records, and all children cases records for any type of combination
I could use the SObject.getPopulatedFieldsAsMap, to replace something around lines 115 but I need to check all the records so I need to call this multiple times (once per contact). The JSON format has another benefit over a Map<String, Object> is that it's generic and "easy" to do reflection.
Did I understand your suggestion, or did you see something I didn't?
Thanks

No i think you got it, its just that i see the performance is something as being an inhibitor for using this and wanted to suggest something that might help improve it some what. I'm confident even using  SObject.getPopulatedFieldsAsMap multiple times will be faster than doing the JSON trick multiple trick. Worth benchmarking the two?

I finally got some time to update this code. I have made several performance improvements. I did the change you suggested, but the tests I have done are not conclusive. More testing is required with lots of data ;-)
I also created couple public methods that you can use to validate the data without having to incur in these expensive operations. Basically you can execute the test once and re-use the same object/fields over and over again without having to search which was making it slow. Since I am using a String as a parameter you could even "hard coded" the strings.
Enjoy.

Excellent, thanks for the update. Look forward to the results of your testing.


ADC gain changes causing discontinuities

There was another signal (not shown) that was pushing the upper limit
the output from the LED circuits opamp was ~3.2V. I expect this discon. to be less impactful if we used a higher LED output (which will be the default)

02:48:10 | pioreactor2 | adc reader | ADC gain updated to 8.
...
22:44:00 | pioreactor2 | adc reader | ADC gain updated to 4.

One thing that is strange is that the jump is beyond just the new resolution:
2021-07-10T01:27:10.342742|pioreactor2|0.032282235175634|45 vs 90 vs 135 test 7|45|0
2021-07-10T01:27:15.343661|pioreactor2|0.0324384899441511|45 vs 90 vs 135 test 7|45|0
2021-07-10T01:27:20.343174|pioreactor2|0.0322041077913755|45 vs 90 vs 135 test 7|45|0  # PGA update here
2021-07-10T01:27:25.343306|pioreactor2|0.0341260414441359|45 vs 90 vs 135 test 7|45|0
2021-07-10T01:27:30.343253|pioreactor2|0.0343447981200598|45 vs 90 vs 135 test 7|45|0

Notice the deltas between the signals jumps beyond the new resolution.

I was confused about this for the entire experiment - until I realized it might be the ADC gain changing. Regular users aren't going to know about this.


Required package 'dmidecode' not found even after installing it

Hello, my chromebook is currently running arch linux (this is my first arch install and was also just installed). when I tried to run the firmware script I got the error Required package 'dmidecode' not found; cannot continue.  Please install and try again. so obviously I installed dmidecode with the command sudo pacman -S dmidecode it installed version 3.4-1. Dmidecode runs fine but only with sudo which I suspect is normal. How can I fix this error as I can see that my bios version is 2+ years old?
Some info about the chromebook as gathered from dmidecode is:
BIOS Information
	Vendor: coreboot
	Version: MrChromebox-4.12
	Release Date: 06/04/2020
	ROM Size: 8 MB
	Characteristics:
		PCI is supported
		PC Card (PCMCIA) is supported
		BIOS is upgradeable
		Selectable boot is supported
		ACPI is supported
		Targeted content distribution is supported
	BIOS Revision: 4.12

Handle 0x0001, DMI type 1, 27 bytes
System Information
	Manufacturer: GOOGLE
	Product Name: Gnawty
	Version: 1.0
	Serial Number: 123456789
	UUID: Not Settable
	Wake-up Type: Reserved
	SKU Number: Not Specified
	Family: Not Specified

I'd boot a Debian live USB and just update from there. Too many variables to try and debug in Arch

Hello again I have found out what was going on! I spent the last three hours trying different stuff in this order:
on Arch Linux

downgrading dmidecode to 3.2-1 from 3.4-1

Unable to read current firmware; cannot continue.
Either add 'iomem=relaxed' to your kernel parameters,
or trying running from a Live USB with a more permissive kernel (eg, Ubuntu).
See https://www.flashrom.org/FAQ for more info.

added iomem=relaxed to my rEFInd config, rebooted and tried again started to work
upgrading from 3.2-1 to 3.3-1 still worked

switch to a GalliumOs live usb

dmidecode 3.1-1 worked
update dmidecode to 3.4-1 still worked despite not working on arch

Back on Arch Linux

updated dmidecode from 3.3-1 to 3.4-1 worked?


[Bug]: 1.4.2 New Tab Wont Trigger

Bug Description
The plugin appears to have stopped working as of 1.4.2. Tested on multiple vaults with a combination of 0 to many other plugins and the default new tab widow just opens by default now.
Relevant Screenshot
No response
To Reproduce
No response
Obsidian Version
1.4.2
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

Checklist

[X] I updated to the latest version of the plugin.

@joriswvanrijn My guess is that it's not the obsidian version, but rather the plugin version, and potentially related to your latest PR(s)?

Hmm, could definitely be.
@ObsidianTTRPGProject  Which platform are you using? Is it possible to confirm that the previous version is working?
@chrisgrieser did i by any chance make a mistake in the default setting which prevents the plug-in from loading?
It’s unfortunate, but I have a 2 week holiday without access to my dev laptop. Maybe we should revert the PR and I’ll test it better when I get back.

not sure, I am on the latest version, and it works fine for me?

oh wait, I haven't made a new release after your second PR, @joriswvanrijn, which could be the issue.
@ObsidianTTRPGProject I just released 0.11.2, which potentially fixes the issue. Could you update and give it a try?

Check! Hope that fixes it.
I just checked for obsidian version 1.4.12 and can confirm it works as well.
@chrisgrieser did this plugin work on mobile before? I haven't considered that in my PR.

Using the Desktop version with Windows 11. I'm not near the pc right now but will report back.

Ok updated and everything is working again :) thanks guys.


Connecting to the emulator w/o project ID

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like

If you don't specify the ProjectId in the FirestoreDbBuilder, the library will (currently) try to determine it by contacting the GCP metadata server - that's probably not what you want in this case. We could potentially have a different defaulting approach when connecting to the emulator, but that's not something that's going to happen imminently.

When working with GCP locally, the team is using a Dockerized emulator (https://firebase.google.com/docs/emulator-suite). I'm forced to specify the project ID. I want to be able to use the emulator w/o tying to a specific Project ID because I have different solutions that require the use of the emulator (different collections, the database could be the same).
Describe alternatives you've considered
Building emulator image per solution and passing the ProjectId. Unfortunately, that means I cannot run more than one .NET solution simultaneously, as ports required by containers will conflict. And it means we can't have an image that any project can use. Instead, each project needs to build its own image.
Additional context
It shouldn't matter what ProjectId is when working with an emulator. Ideally, omitting ProjectId from FirestoreDbBuilder (null, empty string, or a special value such as emulator) should imply using a local emulator.
The thread started in StackOverflow with @jskeet.

I'll try to look at this in January, but I can see two potential issues:

We'll need some way of getting "the emulator project" from the emulator. (I don't know if that's feasible.)
We'll need to handle the case where the other emulator (via gcloud) is being used; that may well not have the concept of a project ID at all

One workaround you could consider is using an environment variable in the Docker container (which can then be passed in when starting it). I assume the Docker image uses (or could use) the --project flag. If you take that from an environment variable, you could pass that in when starting the container, so you only need a single image which can work with any project. You'd still need to specify the same project ID in the .NET code, but at least you wouldn't need multiple images.

Looks like the workaround we've discussed is going to be materialized as an official Docker image, released with every update to the NPM package (https://github.com/firebase/firebase-tools/issues/1644). Which is a welcomed change to simplify working with the emulator.

@SeanFeldman: So where are we in terms of .NET library support? If there's an environment variable that we should use to automatically detect the project ID when connecting to the emulator, I'd be happy to support that (even if it's only used in one of the two emulators).

The entry point in the image must run the following command.
firebase emulators:start --project PROJECT_ID

To avoid hardcoding PROJECT_ID it would need to be passed to the container via an environment variable. What that environment variable name will be, I can't answer. I suppose this could be answered by @joehan who is working on the official firebase-tools image (PR in progress).

It would be great to know what's the status of https://github.com/firebase/firebase-tools/issues/1644 and the associated PR https://github.com/firebase/firebase-tools/pull/6255 before this issue can be closed or anything is done.

Given the lack of movement here, I'd like to close this issue for the moment. @SeanFeldman are you happy for me to do that, and then if other aspects are unblocked, you can comment here and I'll reopen it?

Yes, @jskeet.


libcontainer: Make the workloads injectable

It allows flexibility by allowing libcontainer users to create and inject workloads.
For example, users of libcontainer can add their favorite wasm runtime and not use the normal workloads.

Codecov Report

Merging #1403 (baf6b96) into main (a48d000) will increase coverage by 0.13%.
The diff coverage is 45.61%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1403      +/-   ##
==========================================
+ Coverage   68.65%   68.79%   +0.13%
==========================================
  Files         119      120       +1
  Lines       12692    12829     +137
==========================================
+ Hits         8714     8826     +112
- Misses       3978     4003      +25

@Furisto May I ask you to review this one?

@Furisto May I ask you to review this PR?

@Furisto PTAL


Feature request - manual check warranty

For those of us that don't have access to GSX, it would be useful to have a "Check Warranty" button linking to https://checkcoverage.apple.com/gb/en/?sn=SERIALNUM instead of "Can't lookup warranty"

Hi @ofirgalcon tried to reach out to you in the #munkireport slack channel, but does this work:

I will be adding some localization for this to make it a bit clearer

Hi - yes this would work, but this is what I see for all computers (no gsx) - so unless I can edit the date manually, the 'Re-check warranty status' button should just say 'Check warranty status' and the Warranty Coverage field should be hidden


Adds a new macro to allow getting filter values easily

We've added a new macro filter_values to make it easy to grab the values of a filter box when writing dynamic filter dependent queries.
Usage:
SELECT action,
count(*) as times
FROM logs
WHERE action in ( {{ "'" + "','".join(filter_values('action_type')) + "'" }} )
GROUP BY 1

Returns:
SELECT action,
count(*) as times
FROM logs
WHERE action in ( "dashboard", "log" )
GROUP BY 1

When filter box is like this:

Fixes #1996

Codecov Report

Merging #5547 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5547      +/-   ##
==========================================
+ Coverage   63.12%   63.14%   +0.02%
==========================================
  Files         349      349
  Lines       22167    22180      +13
  Branches     2462     2462
==========================================
+ Hits        13992    14005      +13
  Misses       8161     8161
  Partials       14       14

Impacted Files
Coverage Δ

superset/jinja_context.py
77.27% <100%> (+3.93%)
:arrow_up:

Continue to review full report at Codecov.

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

Ready for review. Thanks!

This is amazing and exactly what we wanted to fix our reports for a 700 people company - thank you so much @mjsilva! It also addresses this issue

Thanks!
FYI, this is also an effective workaround for issue #5755 .
If I'm not mistaken that's the best way atm to configure SQL Lab queries via the URL parameters of the dashboard.
Just add the following snippet to your superset_config.py to use Manuels code via the jinja context addons until this is merged.
You can then configure the charts via the preselect_filters.
SQL:
...
WHERE foo = {{ filter_values('bar', 'default_bar')[0] }}
...

URL:
.../superset/dashboard/baz/?standalone=true&preselect_filters={"CHART-ID":{"foo":["30"]}}

superset_config.py:
# -----------------------------------------------------------------------------------
# Read filter values in jinja templates
# -----------------------------------------------------------------------------------
import json
from flask import request

def filter_values(column, default=None):
    """ Gets a values for a particular filter as a list
     This is useful if:
        - you want to use a filter box to filter a query where the name of filter box
          column doesn't match the one in the select statement
        - you want to have the ability for filter inside the main query for speed purposes
     This searches for "filters" and "extra_filters" in form_data for a match
     Usage example:
        * SELECT action, count(*) as times
        FROM logs
        WHERE action in ( {{ "'" + "','".join(filter_values('action_type')) + "'" )
        GROUP BY 1
     :param column: column/filter name to lookup
    :type column: str
    :param default: default value to return if there's no matching columns
    :type default: str
    :return: returns a list of filter values
    :rtype: list
    """
    form_data = json.loads(request.form.get('form_data', '{}'))
    return_val = []
    for filter_type in ['filters', 'extra_filters']:
        if filter_type not in form_data:
            continue
        for f in form_data[filter_type]:
            if f['col'] == column:
                for v in f['val']:
                    return_val.append(v)
    if return_val:
        return return_val
    if default:
        return [default]
    else:
        return []

JINJA_CONTEXT_ADDONS = {
    "filter_values": filter_values
}

SELECT action,
count(*) as times
FROM logs
WHERE action in ( {{ "'" + "','".join(filter_values('action_type')) + "'" }} )
GROUP BY 1
It is not giving any result set in sql lab. can you explain where to write

@mjsilva  how can i pass the since and until dates to the query from dashboard.

Could you please provide a detailed example? i have tried it with superset 0.28.1 and it does not throw any error simply not working. I have tried to create a view both in tables and sql lab view.
the above is one datasource both for filter and chart? when i select filter_box i should see action_type in the above example?
Please suggest, thanks

filter_values is broken in the newest superset version 0.30.
'filter_values' is undefined
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/superset/viz.py", line 329, in get_df_payload
df = self.get_df(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/viz.py", line 142, in get_df
self.results = self.datasource.query(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 683, in query
sql = self.get_query_str(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 410, in get_query_str
qry = self.get_sqla_query(**query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 538, in get_sqla_query
tbl = self.get_from_clause(template_processor, db_engine_spec)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 434, in get_from_clause
from_sql = template_processor.process_template(from_sql)
File "/usr/local/lib/python3.5/dist-packages/superset/jinja_context.py", line 109, in process_template
return template.render(kwargs)
File "/usr/local/lib/python3.5/dist-packages/jinja2/environment.py", line 1008, in render
return self.environment.handle_exception(exc_info, True)
File "/usr/local/lib/python3.5/dist-packages/jinja2/environment.py", line 780, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python3.5/dist-packages/jinja2/_compat.py", line 37, in reraise
raise value.with_traceback(tb)
File "

filter_values is broken in the newest superset version 0.30.
'filter_values' is undefined
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/superset/viz.py", line 329, in get_df_payload
df = self.get_df(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/viz.py", line 142, in get_df
self.results = self.datasource.query(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 683, in query
sql = self.get_query_str(query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 410, in get_query_str
qry = self.get_sqla_query(**query_obj)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 538, in get_sqla_query
tbl = self.get_from_clause(template_processor, db_engine_spec)
File "/usr/local/lib/python3.5/dist-packages/superset/connectors/sqla/models.py", line 434, in get_from_clause
from_sql = template_processor.process_template(from_sql)
File "/usr/local/lib/python3.5/dist-packages/superset/jinja_context.py", line 109, in process_template
return template.render(kwargs)
File "/usr/local/lib/python3.5/dist-packages/jinja2/environment.py", line 1008, in render
return self.environment.handle_exception(exc_info, True)
File "/usr/local/lib/python3.5/dist-packages/jinja2/environment.py", line 780, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python3.5/dist-packages/jinja2/_compat.py", line 37, in reraise
raise value.with_traceback(tb)
File "

How do I access the date filter values using this macro? I'm looking to access the "since" and "until" in my date filter.
I tried "__from" and "__to" but to no luck.
{{ "'" + "','".join(filter_values('__from')) + "'" }}

Is it possible to get the current username as the parameter?

@fishfree - Yes you can do that like this, thanks! Just an example in both predicate and select clauses.
SELECT '{{current_username()}}' as user
where '{{current_username()}}' = 'someone'

This is useful if:
- you want to use a filter box to filter a query where the name of filter box
column doesn't match the one in the select statement
- you want to have the ability for filter inside the main query for speed purposes
This does not work as mentioned in the PR, if the column in filter box does not match the one in the other visualization select statement, it throws error because it tries to wrap the query and apply the filter with the column name of the filter box.
All I want is a query like this
SELECT col_1 FROM xyz where x >= {{ something }} and x < {{ something else }}
After that I name the column in Filter Box "something" and "something else" but I get error "something" not present in expr_query

@saurabh1-singh did you ever figure this one out? Having the same problem at our company

How do I access the date filter values using this macro? I'm looking to access the "since" and "until" in my date filter.
I tried "__from" and "__to" but to no luck.
{{ "'" + "','".join(filter_values('__from')) + "'" }}

Did you find any solution for this ?
I'm also looking for same.

SELECT action,
count(*) as times
FROM logs
WHERE action in ( {{ "'" + "','".join(filter_values('action_type')) + "'" }} )
GROUP BY 1
It is not giving any result set in sql lab. can you explain where to write

I guess this was a dummy example, in a real life, wrap it between condition:
{% if filter_values('Product type') | length %}
WHERE shop_order_detail.product_type in ( {{ "'" + "','".join(filter_values('Product type')) + "'" }} )
{% endif %}

@avinasht1 were you ever able to resolve the date filter values asked about here?
https://github.com/apache/superset/pull/5547#issuecomment-473015087


Possibly searching too often, take a rest.

Hello
After today morning git pull i got all the time:
Possibly searching too often, take a rest and bot don`t see anything....
Other Information
OS: Debian Jessie
Git Commit: f7838b8b431ff36b12f6c0bd643dbf673db34be5
Python Version: Python 2.7.9

Same issue as of a couple of minutes ago. After restarting the server it just says 'Walking from ...' and doesn't do anything else but walk.

Seems like an IP ban. Turned off wifi and use 3G data pokemons/stops shows up again.
Maybe Niantic is testing some IP-related bans.

Same was last time api was changed.

#3717

Was indeed an IP ban, works using VPN.

I change IP 5 times... 10 minutes and again same...

Looks like Nantic find a way to detect this bot or newlatest git version got a bug and that`s why nantic detect.. i install completly new server with new IP and after 10 minutes i get ban softban... so this is strange.

I dont get the last version from 2 day, same issue for me :(

#3722

@z4ppy So you don`t git pull for last 2 days and You get softban too?

I haven't pulled for 2 days, got (soft?)banned.

@takenek I didn't update my bot since Mon Aug 8 00:10:22

I got only 1 bot on server i change IP/address/OS and account for login to PGO and same effect - bot detected.
Sounds like they find a way to detect a bot :-(

IP ban sounds a bit harsh. Especially due to the nature of the game. People using free wifi at their local  bar/kids playground/whatever. One of them has a Macbook running PokemonGo Bot.. the ip-ban kicks in and everyone on that wifi network can no longer play Pokemon ?

Yes everyone who use this IP address get a ban :-)

Do not use public Wifi or a well known VPN. Niantic bans these often. The solution is to use a different wifi network or VPN.

Just reviewed my log, but it's not as serious as you guys think it is. At least. Not for me
https://gist.github.com/dogeatdog/8a144185ab0f46d5af57b7b28c02b1de
As you can see, I too got the error. So I quit, waited a few mins and restarted it. It took a little while (walking, walking), but then suddenly it found a pokestop and spun it, and everything was hunky dory from then on. Maybe just trouble retrieving the pokestops for some reason ?

For softban/ban/unable to connect related issues go to #3700 (bans) #3682 (server issues)

@dogeatdog how do you add in [INFO] on remaining bag space?

I did nothing special for this, just got the version for OSX and used the default config.json.
here's my config if that helps, but there's nothing on the loglevels there : https://gist.github.com/dogeatdog/f1738f1be4d805f9936a1ea9a706f96c
My version says it's uptodate : On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean

Got the same issue on Arch running the latest dev branch.

Can we make a setting that to let bot logout after this message ?

I can this message today !

Pokemon got reCaptcha :) http://pokemongohub.net/captcha-game-pokemon-go-bots-trackers/


Trying a fix of not working "migrations has not run" warning

Pending local test…

Tested, doesn't work :(. @mib32

@shkm Actually working for me. I'll add this in separate PR


Apply setters empty values checks

This PR is to handle empty values cases for apply-setters function.

This needs rebase.

Done.


Add support for interval multiplication by integers

Resolves #14641.
I'd like to highlight two decisions I've made while implementing this feature and would appreciate any feedback from you on them:

PostgreSQL supports interval multiplication by doubles. In this PR, I added support only for integers because #14641 specifically addresses multiplication by integers. Also, it was the easiest option, as it didn't require changing internal representation of intervals (org.joda.time.Period).

To perform multiplications the Period::multipliedBy method is used. This method multiplies each period’s element
by a given integer without any normalization. For example:
SELECT INTERVAL '1 second' * 900;

returns:
0 years 0 mons 0 days 0 hours 0 mins 900.0 secs

instead of:
0 years 0 mons 0 days 0 hours 15 mins 0.0 secs

This could be less user-friendly and more prone to overflow errors.  For example:
SELECT INTERVAL '2 second' *<PHONE_NUMBER>;

results in
[XX000] ERROR: Multiplication overflows an int: 2 *<PHONE_NUMBER>

instead of:
0 years 0 mons 0 days 1193046 hours 28 mins 14.0 secs

The currently implemented arithmetic operators (+/-) work in a similar manner; that's why I haven't optimized it yet.

I'm open to any suggestions and am happy to modify the current implementation if necessary.
Checklist

[X] Added an entry in CHANGES.txt for user facing changes
[X] Updated documentation & sql_features table for user facing changes
[X] Touched code is covered by tests
[X] CLA is signed
[X] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Can one of the admins verify this patch?

I've applied @BaurzhanSakhariev's suggestion and created an issue regarding the normalization: https://github.com/crate/crate/issues/14798.
Thank you for the swift review! I'd be happy to contribute in the future.

Thx again @piotrrzysko!
Can I ask you to squash your commits to one and please Closes: https://github.com/crate/crate/issues/14641 to the final commit message (in the "body").


Settings page should match theme?

Reported on LMS forum about white text on light background when skin is using light theme?

Can't recreate. Original report: https://forums.slimdevices.com/showthread.php?p=1066917#post1066917

Changed issue to make settings page match dark/light theme switch.


simp only fails on example that works in Lean 3

In Lean 3, this example proof works:
theorem not_mem_nil (a : nat) : ¬ (a ∈ ([] : list nat)) := id

theorem forall_prop_of_false {p : Prop} {q : p → Prop} (hn : ¬ p) :
  (∀ h' : p, q h') ↔ true := sorry

example (R : nat → Prop) : (∀ (a' : nat), a' ∈ ([]: list nat) → R a') :=
begin
  simp only [forall_prop_of_false (not_mem_nil _)],
  exact λ n , true.intro
end

However, after translation into Lean 4, the example proof fails at the simp only step:
theorem not_mem_nil (a : Nat) : ¬ a ∈ [] := fun.

theorem forall_prop_of_false {p : Prop} {q : p → Prop} (hn : ¬ p) :
  (∀ h' : p, q h') ↔ True := sorry

example (R : Nat → Prop) : (∀ (a' : Nat), a' ∈ [] → R a') := by
  simp only [forall_prop_of_false (not_mem_nil _)]
  -- (fails to simplify)

  sorry

Lean Version
leanprover/lean4:nightly-2022-08-27

@gebner discovered today that the Lean4 version can be made to work by adding some type annotations like this:
example (R : Nat → Prop) : (∀ (a' : Nat), a' ∈ [] → R a') := by
  simp only [(forall_prop_of_false (not_mem_nil _): ((_ → _) ↔ _ ))]
  exact λ _ => True.intro


⚠ PrivateHD has degraded performance

In 0312f86, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1125 ms

Resolved: PrivateHD performance has improved in 2357d1d.


Fix macros

This PR fixes the issues with the macros, introduced in #79.
Using sin.(AT(rand(1000, 1000))) in the test suite:
main
julia> using Revise; include("test/hygiene.jl")
----------- @time
  0.000047 seconds

this branch
julia> using Revise; include("test/hygiene.jl")
----------- @time
  0.046593 seconds (97 CPU allocations: 22.891 MiB, 12.87% gc time) (6 GPU allocations: 45.776 MiB, 0.44% memmgmt time)

So, this appears to fix the issue. I'm not sure what the best way to test this.
Part of the reason is that this PR is technically behavior changing in that the macro now makes an argument-less closure, and times (in the case of @time) the closure.
I think that this is probably fine and, frankly, we could even eliminate the macro and just make the methods, which are far easier to extend/understand (as there is no hygiene to worry about), user-facing. The purely method-based approaches may be easier to test, too, since we could even check @which to ensure that we're calling the extension when we expect.

Closes #82.

I kind of a agree. Methos are so much easier to use and understand. They are not as flexible, but I think we can use more powerful tools if/when we need more power.


3GPP updates

This PR contains:

embedding Listserv 17 besides Listserv 16.5
change preferences to record Comments: To: header field. This will enable the recreation of communication networks

Codecov Report

Merging #481 (f577ac9) into main (1c19fd3) will decrease coverage by 0.01%.
The diff coverage is 76.36%.

@@            Coverage Diff             @@
##             main     #481      +/-   ##
==========================================
- Coverage   75.08%   75.07%   -0.02%
==========================================
  Files          16       16
  Lines        2288     2323      +35
==========================================
+ Hits         1718     1744      +26
- Misses        570      579       +9

Flag
Coverage Δ

unittests
75.07% <76.36%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

tests/webscraping/test_listserv.py
90.62% <44.44%> (-0.72%)
:arrow_down:

bigbang/listserv.py
81.62% <82.60%> (-0.34%)
:arrow_down:

Continue to review full report at Codecov.

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


iox-#2224 Fix const value assignment in optional

Fixes the forwarding reference type in iox::optional::operator= when performing a direct assignment of the underlying value such that assigning const values is supported.
Pre-Review Checklist for the PR Author

[ ] ~Add a second reviewer for complex new features or larger refactorings~
[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #2224

Hmm, it seems like the ASAN build may have crashed or something?
2024-03-13T19:15:11.4614668Z [----------] Global test environment tear-down
2024-03-13T19:15:11.4942698Z [==========] 21 tests from 2 test suites ran. (41 ms total)
2024-03-13T19:15:11.4943362Z [  PASSED  ] 21 tests.
2024-03-13T19:15:11.5740157Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5740803Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5741310Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5741823Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5742317Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5742807Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5743292Z AddressSanitizer:DEADLYSIGNAL
2024-03-13T19:15:11.5743782Z AddressSanitizer:DEADLYSIGNAL

Is there something I can do about that? The DEADLYSIGNAL spew is so much I have to stop the raw log page from loading before it's responsive :sweat_smile:

@tdouglas-latai maybe it was a CI hiccup, I will rerun it again. As far as I can see, the STDLIB_tests did run successfully and then the address sanitizer failed. Without any reason ...

Hmm, seems to have failed in a similar way the second time.  I ran ./tools/ci/build-test-ubuntu-with-sanitizers.sh gcc asan on my local machine and it seemed to pass without issue.  Maybe rerun a third time?

@elfenpiff, I tried running those commands and did get some failures during build with the compiler being killed, even got those errors on master, but I didn't get the same error as you or CI.  I changed the build jobs to 4 and it passed on my local machine using this branch and master.  Do you have any idea from your output if this could be caused by the proposed changes here?

@tdouglas-latai I can reproduce it with your branch but not on master. So it is definitely caused by your changes, but I have no idea what the issue here is. Especially, since it is crashing at the end of a successful test run and the test run varies as well, sometimes after the STDLIB tests, sometimes after the hoofs_moduletests.
I tend to say this is some bug within the address sanitizer, but we cannot ignore this, otherwise we have a failing CI on master. We either need to confirm that it is in fact an address sanitizer bug and disable the CI target, or we have to find the issue on our side

Yes, definitely can't ignore the CI failure, I'll play with it more to try and replicate it locally.

Oh, sweet, thanks!


build: various improvements

ci(gha/submodule): ensure submodules point to the right branch

Will help avoid mistakes if anyone uses update --remote...

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

build(images): add tag argument

Allows one to explicitly build the binaries with a given tag, as opposed to the alias-tag
which simply adds a docker image tag alias.

Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

Tested & LGTM
Steps:

./scripts/release.sh --image upgrade.job --tag develop-tagging-feature
./scripts/release.sh --image "" --tag develop-tagging-feature --build-bins
/nix/store/d54wn1jcmx4475yxz87grg78pi7ahn45-kubectl-plugin-develop-tagging-feature/bin/kubectl-mayastor upgrade

ashish@vm:~$ kubectl get pods mayastor-upgrade-develop-tagging-feature-p6zbq  -n mayastor -o yaml | grep image
    image: docker.io/openebs/mayastor-upgrade-job:develop-tagging-feature
    imagePullPolicy: IfNotPresent
  - image: docker.io/openebs/mayastor-upgrade-job:develop-tagging-feature
    imageID: ""

bors merge


Explore css/js animations for website

[ ] build custom carousel [Update Nov 10: no longer doing this, will have tabs instead]
[ ] animate keyline
[ ] animate svg elements (see floating animation here https://codepen.io/SusanneLundblad/pen/DKRKbN)

Draft PR here: https://github.com/jpmorganchase/uitk/pull/647
@shey-v it would be good to catchup to get your thoughts on our initial implementation

11 Nov - Joana/Felicia have draft PR to be reviewed with Shey

This has been reviewed with Shey and we will start adding these animations as we build the website. I'll close the draft PR as we don't need to merge in these changes.


acrossElements:true option not working as expected.

When I search for the string "Lorem Ipsum" where "Lorem" is Bold and "Ipsum" is plain text, Mark.js is adding two mark tags one for each word and treating them as two results. But ideally, I would expect this to be a single match and a single mark tag should wrap "Lorem Ipsum".
Using the following mark Options :
separateWordSearch: false,
acrossElements: true
Steps to reproduce
Please find the below example to reproduce the issue.
https://jsfiddle.net/rajanikanth05/8ax0n74r/1/
Go to the example page where I have logic to highlight current search with "Orange" background color.
Search for "Lorem Ipsum".
It highlights only "Lorem"  with "Orange" but not "Lorem Ipsum".
On clicking next button, it highlights "Ipsum".
Environment

Browser name and version: Chrome  53.0.2785.143 (64-bit)
mark.js version: 8.4.0

Hi @rajanikanth05,
Thanks for submitting. Actually, this behavior is as expected. If a match is located across several DOM elements mark.js doesn't wrap them all into one  tag. Instead, it wraps all matches that occur in different elements separately.
What problem do you have with this behavior?

@julmot ,
Thank you for the quick response.
May be this is just my use case, but I want to highlight the entire search string match at a time. In this case "Lorem Ipsum".
To achieve this, I had to group all the consecutive  tags that from the search string and highlight the entire group.
But I think it would be nice to have such option inbuilt in the library.

@rajanikanth05 Thank you for your response. I can understand your situation but it would bring up issues. Take the following HTML as an example:

    How

        are you

    doing?

Now, in your case you'd be searching for "How are you doing?" and wrapping it up in one single  would work:

        How

            are you

        doing?

But, mark.js normally just replaces a text node with a single  element. In that case it would need to replace actual DOM elements which would trigger regeneration of the DOM and destroy events.
There may also be edge-cases like:

    How

        are you doing today?

which would e.g. produce the following:

        How

            are you doing today?

and that would be invalid due to the fact that a  needs to appear before a closing .
I don't see a way I could offer this with a consistent behavior and a quality-focused implementation. So sorry, but I can't help you out with a built-in solution here.

While this is true, there is no need to highlight the entire search string as one, the only thing that is needed is to treat the different highlights as one result.
So for this example where we search "Lorem ipsum" one a page that looks like this:

     Lorem ipsum. Lorem ipsum.

Instead of a result array like this:
[
    {domElem: Lorem},
    {domElem: Ipsum},
    {domElem: Lorem ipsum}
]

There should be an option to group:
[
    [
        {domElem: Lorem},
        {domElem: Ipsum}
    ],
    [
        {domElem: Lorem ipsum},
    ]
]

@W3D3 Yes, exactly. This grouping would be very useful, because right now people need to implement the grouping themselves. Note that each function only iterates over individual marks, done function only gives you the total number of individual marks (not the number of occurrences)

@W3D3 @Dalimil Hi, did you find any solution for this issue?
I have an array of sentences that are being 'marked' across elements.
I want to identify each of those sentences uniquely by assigning them an 'id'.
But each function gives me individual marks. When a sentence is spread across elements, this is not helpful.

https://github.com/W3D3/DiffVisualizer/blob/master/src/SearchController.js
@mahendhar9
I have linked my solution to this issue, but it is more of a workaround because it would not work with wildcards. I simply iterate over all results and count characters to group all DOM elements to their actual results.


Add new .lang files

Open up .en-gb,lang and do it on the fly. Ideally your translation should be reviewed by someone else.

I can do spanish!

#10

I'll try to add portuguese.

Pt-Br Done #11

French translations done :) #15

Created a PR for a Dutch translation

Closing for now. See: https://osiewicz.github.io/2018/10/15/Language-barriers.html


RTL support

when will the rtl support be available

still nothing?

is there any hope?

Easy. Let me tell you something.... for converting any layout to RTL form just use android:rotationY="180".
Add it to your reside layout and it's childs. This has worked for me in a lot of cases.


make types initialize a little better

Change Summary
Before this change, when you had something like this:
node = createObject("RoSGNode", "Node")
node.addField("myArray", "assocarray")
node.myArray = { abcd: 1234 }

then brs would not allow the assignment of node.myArray = { abcd: 1234 }. This was because it was reading node.myArray's value kind as Invalid, which didn't match the value kind that was being assigned (associative array is an Object), so brs didn't let the new value get set. This was only happening for arrays and associative arrays because of how brs created their initial types.
There were a lot of tests that had to be updated, sorry, I realize this changes a lot of test files!

@alimnios72 Just a heads up, I made a couple more logic changes here around handling invalid or uninitialized as a special case.


⚠ Jenkins Plugins Artifactory has degraded performance

In 8ffb204, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1260 ms

Resolved: Jenkins Plugins Artifactory performance has improved in cab8754 after 12 minutes.


Recursive binding of interfaces, custom method naming rules

We've been using your library for a while for our Space Engineers plugin:
https://github.com/iv4xr-project/iv4xr-se-plugin
We are trying to keep very clean and tree-structured interface, most of it can be seen here:
https://github.com/iv4xr-project/iv4xr-se-plugin/blob/main/Source/Ivxr.SpaceEngineers/ISpaceEngineers.cs
We want to keep method naming in this dotted fashion. Ex call Blocks.Build and that is our method naming convention.
We've been using your JsonRpcMethod annotation and had mapping class for it:
https://github.com/iv4xr-project/iv4xr-se-plugin/blob/main/Source/Ivxr.SePlugin/Communication/AustinJsonRpcSpaceEngineers.cs
This was fine for a while, but this class is basically only boilercode to do the mapping. We are adding a lot more methods and this is getting annoying and feels unnecessary. We have very clear and simple interface, that already says everything and mapping could be generated straight ouf of it, no need to mark anything (all methods are supposed to be exposed).
Would be cool if there was a mechanism to do that - just bind interface, pass implementation and let the library bind everything, which is part of the interface (that way it also doesn't mess with GetType, Equals, ToString and other generated methods).
I created our own way, based on your ServiceBinder as a working proof. In case you are interested in putting some parts of it into your library too, I am happy to assist.
https://github.com/iv4xr-project/iv4xr-se-plugin/pull/127

Very cool! I'm sure others would enjoy this flexibility as well.


Cant use raw table names with Insert/Update/Delete

I have an odd scenario with an older DB were slowly migrating to a new system. but in the meantime we need to support it with our new API. the problem is, this table name has a . in the name. so we need to wrap it in " in order to call it.
for examples, lets call this tables something_2.0
when selecting from this table, its a simple:
const table = sql<string>`"something_2.0"`;
Fastify.db.selectFrom(table).selectAll().where("id","=", id).execute()

this works great, but we can not do the same thing for inserts and updates, which are arguably the more important place to be using the builder.
this is the error using the same strategy with insertInto and updateTable, which makes sense because it only supports a raw string.
from.includes is not a function
	at parseTable (/home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/node_modules/kysely/dist/cjs/parser/table-parser.js:40:14)
	at Transaction.insertInto (/home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/node_modules/kysely/dist/cjs/query-creator.js:72:103)
	at DAOAgency.create (/home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/src/DAO/Pharaoh/List/DAOAgency.ts:81:25)
	at /home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/src/route/v5/auth/index.ts:186:34
	at /home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/node_modules/kysely/dist/cjs/kysely.js:376:38
	at processTicksAndRejections (node:internal/process/task_queues:95:5)
	at DefaultConnectionProvider.provideConnection (/home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/node_modules/kysely/dist/cjs/driver/default-connection-provider.js:12:20)
	at Object.SimpleBenefitsHandler (/home/zanson/projects/myh/fusion/ts-usurper-v5-edge-api/src/route/v5/auth/index.ts:180:2)

if we use the table name straight up it creates a query like:
Fastify.db.inserInto("something_2.0").values({id:2, something:""}).execute()

insert into "something_2"."0" ("id", "something") values ($1, $2)

The select query definitely doesn't work and shouldn't work. https://kyse.link/1HIUA


Hang on win7 x64

I'm trying to install karma on a win7 x64 machine like so "npm install -g karma" but it hangs at the following message:

<EMAIL_ADDRESS>install C:\Users\developer\AppData\Roaming\npm\node_modules\karma\node_modules\socket.io\node_modules\socket.io-client\node_modules\ws
(node-gyp rebuild 2> builderror.log) || (exit 0)

I tried "npm install -g node-gyp" which works, however running e.g. "node-gyp configure" hangs as well:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
I have Python 2.7.3 installed and in PATH. Any idea what I'm missing?

If you still get this error after following the instructions at https://github.com/nodejs/node-gyp/#on-windows then let me know and I'll reopen.


Access local api

Closes: https://github.com/metarhia/impress/issues/1339

Landed in https://github.com/metarhia/impress/commit/12907618f8910716a376b0a8884eccc66ed1fc67


Hyper-V Component

could you please confirm if we need to install the Hyper-V components in Windows 10 (1909) to enable to application and credential guard?
Thanks

@merwindz
Please ask Windows 10 Support for clarification as the issues section of this repository is intended for product documentation issues.
If there is something needs to be updated or clarified within the documentation, please follow-up with a link to the article URL for further investigation.
Thank you for providing feedback.


Sparklines, csv's, and pop up data tables are missing data

Moving this to milestone 0.5 since it's not technically a bug.

The streaming service now adds the data points directly to InfluxDB. This eliminates the necessity of an updater running every 30 minutes, and keeps the sparklines and tables updated with the most recent measurements as soon as they are received.

Um.  Is there a separate url for the sandbox?  Should I use http://envirodiysandbox.usu.edu/api/data-stream/?

Yes there should be a separate URL. What you have looks right but @jcaraballo17 can confirm if you have errors.

@SRGDamia1 Yes, http://envirodiysandbox.usu.edu/api/data-stream/ is the right url for the sandbox.

@SRGDamia1, have you been able to test this?

@SRGDamia1  @aufdenkampe you can send a post request with an application like Postman without setting up a Mayfly with real sensors. It's basically the same since the web service doesn't care where the request is coming from. Let me know and I can help you build a request if you decide to use that app instead.

I'm sorry this took me so long.  A few quick tests with Postman look good.

@jcaraballo17, thanks for suggesting Postman. In the future, if you are doing your own testing of similar  "live" features on Sandbox, it would be great if you could point us to the URL of that testing site, to avoid these kinds of bottle-necks in testing. Thanks.
@SRGDamia1 , thanks for testing!


reload-only-visible-feeds conflicts with reload-threads

First of all, I really enjoy using newsboat!
It seems like the option reload-only-visible-feeds yes has a conflict with reload-threads 30.
When using reload-all, newsboat loads all visible feeds sequentially and NOT in parallel.
Obviously, this slows down operation considerably.
After commenting out reload-only-visible-feeds yes, all feeds are reloaded again in parallel.
Is this known and/or desired behaviour?
I could not find any information on that in the documentation.
The tag feature is a really useful one, especially when combining it with reload-only-visible-feeds and reload-threads.
Although read feeds are still excluded from being reloaded with reload-all when show-read-feeds no is used.
The absolutely best version in my opinion would be the following:

tags are used
reload-only-visible-feeds with reload-all reloads ALL feeds under the current tag, not just feeds with unread articles
reload-threads plays nicely with reload-only-visible-feeds and reloads in parallel

And again: thank you very much for your work on this useful piece of software.
Originally posted by @danieldoof in https://github.com/newsboat/newsboat/issues/207#issuecomment-1129699495

Here the output of newsboat -v
Newsboat r2.27-28-g746eb
System: Linux 5.15.0-30-generic (x86_64)
Compiler: g++ 11.2.0
ncurses: ncurses 6.3.20211021 (compiled with 6.2)
libcurl: libcurl/7.81.0 OpenSSL/3.0.2 zlib/1.2.11 brotli/1.0.9 zstd/1.4.8 libidn2/2.3.2 libpsl/0.21.0 (+libidn2/2.3.2) libssh/0.9.6/openssl/zlib nghttp2/1.43.0 librtmp/2.3 OpenLDAP/2.5.11 (compiled with 7.74.0)
SQLite: 3.37.2 (compiled with 3.35.5)
libxml2: compiled with 2.9.12

Sorry, I lost a browser tab with this issue among the others!

When using reload-all, newsboat loads all visible feeds sequentially and NOT in parallel.

Reproduced. For whatever reason, Reload::reload_indexes() doesn't use parallelism, but it can certainly be fixed :)

reload-only-visible-feeds with reload-all reloads ALL feeds under the current tag, not just feeds with unread articles

I don't think this is a good idea. There are a number of ways to hide feeds; you mentioned two of them, the third one is filtering. Making reload-all behave differently for tags would be pretty confusing, I think. I suggest you write a macro instead, which shows unread feeds, reloads everything, then hides them back again.


Fixed gloved weapons being able to attack when not equipped.

About the PR

Gloves now won't be able to attack unless they are equipped.
Resolves #16990
Technical details

I thought making a component would be the cleanest way of doing this. I could imagine having "slapping gloves" or something similar which could both attack when equipped or when held. Doing the fix with a component allows for this.
The other option would be hard code checking if the held item is a glove, and if it is don't allow them to attack! This could also be fine but my gut said doing it that way might be a little sketchy. If a code boss wants to weigh in please let me know if I should change to this approach or some other way that I didn't even think of :)
Media

Before:
https://github.com/space-wizards/space-station-14/assets/107373427/454bc84a-f19f-425d-8b4d-440e3560f723
After:
https://github.com/space-wizards/space-station-14/assets/107373427/d32b3a09-f8aa-4d77-8474-b5ea7d00635b
(You can't tell but I'm trying to attack in the second video 😆)
Changelog

:cl: Beck Thompson

fix: You now must equip gloved weapons (E.g boxing gloves) to use them.

(Not sure why the test failed but let me know if its my fault!)

Could the failed test have something to do with what operating system I'm using?? I have no idea how it would know / tell but my computer did break and I'm using a windows laptop for the moment... I don't think that could be it but maybe 😆

OK well I just realize I missread "flag" for "tag"... Let me know if I should the meleweaponcomponent thing instead 😆


coalescent Bayesian skyline plot

i need R script for coalescent Bayesian skyline

Use Tracer to output the values of the skyline plot as a .csv file which you can then plot in R.


Custom meson arguments for editable installs

In Meson-python you can already define arguments to be passed to meson setup/compile/install/dist (tool.meson-python.args.*).
For editable installs it would be nice to add additional arguments. In my case, I like to create debug builds as editable installs, so I can hook up a debugger if something breaks in the C code.
Having the ability to configure extra meson arguments for editable installs will:

Make the the pip install command simpler. pip install --no-build-isolation --config-settings=setup-args="-Dbuildtype=debug" -e . can be written as pip install --no-build-isolation -e ., which is quite a bit simpler.
Tools like PDM do not have an option for config settings, so there's currently no other way than to edit pyproject.toml in order to create debug builds.

Edit: while adding editable arguments, it would also be nice to have an equivalent of the MESONPY_EDITABLE_VERBOSE env var in pyproject.toml.

2. Tools like PDM do not have an option for config settings, so there's currently no other way than to edit pyproject.toml in order to create debug builds.

This seems like an unexpected gap in functionality. I've found that PDM generally follows packaging standards and issues often get addressed quickly. Have you looked for an issue on the PDM issue tracker? If there isn't one for this topic, maybe you can open one?

1. Make the the pip install command simpler. pip install --no-build-isolation --config-settings=setup-args="-Dbuildtype=debug" -e . can be written as pip install --no-build-isolation -e ., which is quite a bit simpler.

There's a downside to this (beyond more complexity in meson-python): if you hard-code non-default behavior like this, it will be surprising to users/contributors that are not you - meson-python always behaving the same will be easier to learn about.
If this is only for yourself, the typical answer is to define a command alias. Something like:
alias mypkg-editable="pip install --no-build-isolation -e . -Csetup-args ....[etc]"

Thanks for the feedback. This makes sense. I agree that meson-python is not the right place.
I'll check with the PDM folks if it's possible to support config-settings for the install target.


Remove screenshots of the tool parameters

There is some tutorials in which the parameters for the tools are shown with screenshots.
For accessibility reason and also to simplify the updates (usually coming with new versions of the tools), we would like to replace them by a text description of the parameters, something like:

Tutorials for which there is screenshots of tool parameters:

Chip-seq

[ ] tal1-binding-site-identification

Transcriptomics

[ ] de-novo
[ ] srna
[ ] rna-seq-viz-with-cummerbund
[ ] rb-rnaseq

Variant-analysis

[ ] non-dip
[ ] dip
[ ] dunovo

As discussed during the Contribution fest in Norwich, here are 3 possible ways we could standardize the description of tool parameters:

"Current" description

> 2. **RNA STAR** {% icon tool %}: Run **RNA STAR** with
>    - "Single-end or paired-end reads" to `Paired-end (as individual datasets)`
>    - "RNA-Seq FASTQ/FASTA file, forward reads" to the generated `trimmed reads pair 1` files (multiple datasets)
>    - "RNA-Seq FASTQ/FASTA file, reverse reads" to the generated `trimmed reads pair 2` files (multiple datasets)
>    - "Custom or built-in reference genome" to `Use a built-in index`
>    - "Reference genome with or without an annotation" to `use genome reference without builtin gene-model`
>    - "Select reference genome" to `Drosophila Melanogaster (dm6)`
>    - "Gene model (gff3,gtf) file for splice junctions" to the imported `Drosophila_melanogaster.BDGP6.87.gtf`
>    - "Length of the genomic sequence around annotated junctions" to `36`

Replacing "to" by ":"

> 2. **RNA STAR** {% icon tool %}: Run **RNA STAR** with
>    - "Single-end or paired-end reads": `Paired-end (as individual datasets)`
>    - "RNA-Seq FASTQ/FASTA file, forward reads": the generated `trimmed reads pair 1` files (multiple datasets)
>    - "RNA-Seq FASTQ/FASTA file, reverse reads": the generated `trimmed reads pair 2` files (multiple datasets)
>    - "Custom or built-in reference genome": `Use a built-in index`
>    - "Reference genome with or without an annotation": `use genome reference without builtin gene-model`
>    - "Select reference genome": `Drosophila Melanogaster (dm6)`
>    - "Gene model (gff3,gtf) file for splice junctions": the imported `Drosophila_melanogaster.BDGP6.87.gtf`
>    - "Length of the genomic sequence around annotated junctions": `36`

With parameter name in italic and with ":"

> 2. **RNA STAR** {% icon tool %}: Run **RNA STAR** with
>    - "*Single-end or paired-end reads*": `Paired-end (as individual datasets)`
>    - "*RNA-Seq FASTQ/FASTA file, forward reads*": the generated `trimmed reads pair 1` files (multiple datasets)
>    - "*RNA-Seq FASTQ/FASTA file, reverse reads*": the generated `trimmed reads pair 2` files (multiple datasets)
>    - "*Custom or built-in reference genome*": `Use a built-in index`
>    - "*Reference genome with or without an annotation*": `use genome reference without builtin gene-model`
>    - "*Select reference genome*": `Drosophila Melanogaster (dm6)`
>    - "*Gene model (gff3,gtf) file for splice junctions*": the imported `Drosophila_melanogaster.BDGP6.87.gtf`
>    - "*Length of the genomic sequence around annotated junctions*": `36`

With parameter name in italic outside quote and with ":"

> 2. **RNA STAR** {% icon tool %}: Run **RNA STAR** with
>    - *Single-end or paired-end reads*: `Paired-end (as individual datasets)`
>    - *RNA-Seq FASTQ/FASTA file, forward reads*: the generated `trimmed reads pair 1` files (multiple datasets)
>    - *RNA-Seq FASTQ/FASTA file, reverse reads*: the generated `trimmed reads pair 2` files (multiple datasets)
>    - *Custom or built-in reference genome*: `Use a built-in index`
>    - *Reference genome with or without an annotation*: `use genome reference without builtin gene-model`
>    - *Select reference genome*: `Drosophila Melanogaster (dm6)`
>    - *Gene model (gff3,gtf) file for splice junctions*: the imported `Drosophila_melanogaster.BDGP6.87.gtf`
>    - *Length of the genomic sequence around annotated junctions*: `36`

With parameter name in bold

> 2. **RNA STAR** {% icon tool %}: Run **RNA STAR** with
>    - **Single-end or paired-end reads** to `Paired-end (as individual datasets)`
>    - **RNA-Seq FASTQ/FASTA file, forward reads** to the generated `trimmed reads pair 1` files (multiple datasets)
>    - **RNA-Seq FASTQ/FASTA file, reverse reads** to the generated `trimmed reads pair 2` files (multiple datasets)
>    - **Custom or built-in reference genome** to `Use a built-in index`
>    - **Reference genome with or without an annotation** to `use genome reference without builtin gene-model`
>    - **Select reference genome** to `Drosophila Melanogaster (dm6)`
>    - **Gene model (gff3,gtf) file for splice junctions** to the imported `Drosophila_melanogaster.BDGP6.87.gtf`
>    - **Length of the genomic sequence around annotated junctions** to `36`

If we decide on a standard, we could remove the screenshots and document it as a good practice.
Any thought? @nekrut, @nsoranzo, @mblue9, @tnabtaf, @shiltemann, @bgruening

My preferred option is 2. Replacing "to" by ":".
In 5. there's too much bold for my liking.

I definitely prefer the colon and agree the bold is too much (same for italics I think). The quotes I don't have a strong opinion on but lean towards removing them.

I've been using option 5 for ~10 years - bold for interface elements they need to update, italics for verbs (set, click, change), and fixed-width for typed or selected options.  I actually want this stuff to stand out and I think bold, italics, and fixed-width achieve that nicely.
That said, whatever the majority wants is fine by me.

With icons?

RNA STAR {% icon tool %}: Run RNA STAR with
:pencil2:  Single-end or paired-end reads to Paired-end (as individual datasets)
:pencil2: RNA-Seq FASTQ/FASTA file, forward reads to the generated trimmed reads pair 1 files (multiple datasets)
:pencil2: RNA-Seq FASTQ/FASTA file, reverse reads to the generated trimmed reads pair 2 files (multiple datasets)
:white_check_mark: Check-me checkbox set to true
🖱 My dropdown list choose the best possible option

As a follow-up to @njall's idea, using fontawsome icons it could looks somthing like this:

but maybe makes it overly complicated to write the markdown?:
 2. **RNA STAR** {% icon tool %}: Map your reads on the reference genome with
>    - {% icon param-text %} "Some text field": `my value`
>    - {% icon param-file %} "Some input file": `my file`
>    - {% icon param-files %} "Multiple files or collection input": `my files`
>    - {% icon param-check %} "Some Checkbox": `yes`
>    - {% icon param-select %} "Some select parameter": `my choice`

I prefer 3 or 4 of the other options, definitely a fan of the colon, rest do not feel super strongly about (except I agree, emphasizing everything in bold is really emphasizing nothing)

I like the icons, especially the  one-file, multiple-file distinction, this will help a lot in the future.
:+1: for @shiltemann :)

solved


GM Settings to filter creatures

Add some settings to allow the GM to filter out creatures from what is shown to the players to be selected. For example filtering by source to exclude PFS, APs, etc. Also a setting to only show either the Bestiary or Monster Core version of a creature if both exist.
In the screenshot, the Deadtide skeleton guard is an Abomination Vaults creature, the exploding skeleton guard is from PFS, and the top creature is a named creature from an AP.

Only partitial implementation, duplicates can't delete for now, maybe in future find good solution for it


x/tools/gopls/internal/lsp: slice bounds out of range error during typerefs.Decode

gopls version: v0.13.2 (go1.19.4)
gopls flags:
update flags: proxy
extension version: 2023.8.1617
go version: 1.19.4
environment: Visual Studio Code darwin
initialization error: undefined
issue timestamp: Wed, 30 Aug 2023 09:56:11 GMT
restart history:
Wed, 30 Aug 2023 09:55:46 GMT: activation (enabled: true)
ATTENTION: PLEASE PROVIDE THE DETAILS REQUESTED BELOW.
Describe what you observed.

panic: runtime error: slice bounds out of range [:1728708865] with capacity 504

goroutine 3780 [running]:
golang.org/x/tools/gopls/internal/lsp/frob.(*reader).bytes(...)
	  frob.go:404
golang.org/x/tools/gopls/internal/lsp/frob.(*frob).decode(0x1400007fad0, 0x140040c5840, {0x1038d2300%3F, 0x14004306000%3F, 0x14000a441c8%3F})
	  frob.go:362  0xfc8
golang.org/x/tools/gopls/internal/lsp/frob.(*frob).decode(0x1400007faa0, 0x140e4b9f840, {0x1038c5fc0%3F, 0x14002605d10%3F, 0x104208108%3F})
	  frob.go:324  0x101c
golang.org/x/tools/gopls/internal/lsp/frob.(*frob).decode(0x1400027b470, 0x140e4b9f840, {0x10395fba0%3F, 0x14002605d10%3F, 0xe380%3F})
	  frob.go:367  0xf80
golang.org/x/tools/gopls/internal/lsp/frob.(*frob).Decode(0x1400027b470, {0x140021c3e00, 0xb3, 0x200}, {0x1038b06e0%3F, 0x14002605d10%3F})
	  frob.go:258  0x1a8
golang.org/x/tools/gopls/internal/lsp/source/typerefs.decode(0x140002b86e0%3F, {0x103a246d8%3F, 0x140042632c0%3F}, {0x140021c3e00, 0xb3, 0x200})
	  refs.go:806  0x6c
golang.org/x/tools/gopls/internal/lsp/source/typerefs.Decode(...)
	  refs.go:42
golang.org/x/tools/gopls/internal/lsp/cache.(*snapshot).typerefs(0x140002b86e0, {0x103a246d8, 0x140042632c0}, 0x14000341200, {0x103fc4098, 0x0, 0x0})
	  check.go:1224  0x188
golang.org/x/tools/gopls/internal/lsp/cache.(*packageHandleBuilder).buildPackageHandle(0x1400290d380, {0x103a246d8, 0x140042632c0}, 0x140003fba40)
	  check.go:1082  0x1c8
golang.org/x/tools/gopls/internal/lsp/cache.(*snapshot).getPackageHandles.func2.1()
	  check.go:891  0xb0
golang.org/x/sync/errgroup.(*Group).Go.func1()
	  errgroup.go:75  0x5c
created by golang.org/x/sync/errgroup.(*Group).Go
	  errgroup.go:72  0xa4
[Error - 4:55:53 PM]
OPTIONAL: If you would like to share more information, you can attach your complete gopls logs.
NOTE: THESE MAY CONTAIN SENSITIVE INFORMATION ABOUT YOUR CODEBASE.
DO NOT SHARE LOGS IF YOU ARE WORKING IN A PRIVATE REPOSITORY.
<OPTIONAL: ATTACH LOGS HERE>

Thanks for reporting this; it's very mysterious? How often have you observed it? I added an assertion that may help diagnose the cause. If you are able to reproduce this crash readily, please try cloning and building gopls from master and let us know what you observe.

Has this reoccurred? If not, we don't have much to go on, and are unlikely to be able to root cause the issue here.
Bumping to v0.15 to keep on our radar.


orange elif

its the least i can do ;)
https://github.com/dimaryaz/jdmtool/issues/11

Basic tests work:
jdmtool read-metadata
Found device: Bus 003 Device 010: ID 0e39:1250
Detected data card: 16MB WAAS (orange)
Database metadata: {2408~42323550}


As a designer for EDS I would like the button component to be made out of a base component, so that it would be easier to maintain

After Auto Layout and Variants came to Figma, it has been a lot clearer that our system is difficult to maintain. A solution could be to add another component set/variant set of buttons, and rename the old one "Old button" or something in that matter. That would make it easier to communicate that this shouldn't be used in new solutions and screens. Anyway, it would be important to maintain both the old and the new component set for a while.
I would have made the new component set with the help of a base component. That would be a component that includes everything; icon left, label, icon right, hover hand, disabled hand, pressed overlay, focus border, and loading icon. Name this ".base-button", and create an instance of the base component. Make that instance into a component, and start creating variants by hiding and showing layers, as well as changing the colors. Create the base component with the help of auto layout, and the result will be a component that is really easy to use, as well as maintain.

This may not be possible for the time being.

Is this still relevant? @equinor/eds-ux

This issue is blocked @mimarz We want to set auto-layout to all our component but ourr focus state won't let us do it. Waiting for Figma update or delete the focus state for our component.

Need to decide if we need the focus state.

The main goal is to get auto-layout.

Already done by EDS 2.0 - issue close.


Add go-further section about the frame budget

Rebased this hone after merging the other PR.

Rebased this hone after merging the other PR.


Crashes on com.Metal.CompletionQueueDispatch thread

I'm in a weird situation where a Metal thread keeps crashing on me and i'm not sure where i should be looking next so i'm hoping someone here could provide some ideas.
The crash is only occurring when i'm using a separate queue family to execute transfers.
I've set MVKConfiguration::specializedQueueFamilies to true, this causes my family index detection to use index 0 for graphics and index 3 for transfers. Then while uploading data i'm following the path as-if it's a dedicated gpu. I'm creating a staging buffer and an exclusive owned image. The staging buffer is filled with data and a copy is recorded into a command buffer. Then a release barrier is recorded and the command buffer is submitted to the transfer queue. Then another command buffer is recorded for the graphics queue containing just an acquire barrier. These two buffers are synchronised using a semaphore. The command buffers originate from pools created for their respective queues and the queues are locked while submitting to them.
Naturally i've been trying to disable stuff to try and find what's causing it and the leanest i could get it was an empty transfer command buffer that signals a semaphore and then the graphics command buffer waiting on that semaphore while initialising the image. This setup also crashes, so the issue is probably in the semaphore synchronisation.
The buffers aren't being destroyed, the semaphore and command buffers are but only after a fence got signalled by the graphics command buffer finishing execution. The weird thing is that it only crashes sometimes. Sometimes it crashes right away and sometimes i can run it for a few seconds, uploading data each frame. It doesn't seem to crash during initialisation where i'm not rendering frames yet though, so perhaps something swapchain related?
On windows i have no issue on either NVidia or AMD. No validation layer errors are reported on either windows or moltenvk. I can work around this by setting the transfer queue family index the same as the graphics index, but i'd prefer solving this.
The crash i'm getting most of the time shows this:
EXC_BAD_ACCESS
0x10021583b <+107>: callq  0x10023a1f0               ; MVKGPUCaptureScope::endScope()
0x100215840 <+112>: movq   0x40(%r13), %r14
0x100215844 <+116>: testq  %r14, %r14
0x100215847 <+119>: je     0x100215868               ; <+152>
0x100215849 <+121>: xorl   %ebx, %ebx
0x10021584b <+123>: nopl   (%rax,%rax)
0x100215850 <+128>: movq   0x38(%r13), %rax
0x100215854 <+132>: movq   (%rax,%rbx,8), %rdi
0x100215858 <+136>: movq   (%rdi), %rax
0x10021585b <+139>: xorl   %esi, %esi
->  0x10021585d <+141>: callq  *0x58(%rax)
0x100215860 <+144>: incq   %rbx
0x100215863 <+147>: cmpq   %rbx, %r14
0x100215866 <+150>: jne    0x100215850               ; <+128>
0x100215868 <+152>: movq   0x50(%r13), %rdi
0x10021586c <+156>: testq  %rdi, %rdi
0x10021586f <+159>: je     0x100215876               ; <+166>
Other times i'm getting an empty callstack on some thread that's not mine, and then the assembly window sais it cant find symbols for some type (sometimes even types of mine, which is weird that it needs that on an external thread)
Any ideas where i could be looking next?

@MennoVink Are you able to provide a better stack trace, showing the call stack?

Unfortunately not much is available:
Thread 6 Queue : com.Metal.CompletionQueueDispatch (serial)
#0	0x00000001002054dd in ___ZN31MVKQueueCommandBufferSubmission28commitActiveMTLCommandBufferEb_block_invoke ()
#18	0x000000010256049b in start_wqthread ()
Would it help if i build MoltenVK from source and use that instead of the sdk library?

I too found intermittant problems when having separate queues (and queue families, as moltenvk only presents one queue per queue family) for transfer, graphics and presentation.  Your stacktrace looks familiar.  My workaround was to use only one queue (of one queue family) for everything.  Seems stable after that.

Would it help if i build MoltenVK from source and use that instead of the sdk library?

That would provide more info. As would tuning your app from Xcode.
But the stack trace I'm looking for should show up already when your app crashes. I'm interested in the stack trace, in the calling thread, of the calls that lead up to the EXC_BAD_ACCESS
in MVKGPUCaptureScope::endScope().

I'm assuming that was a typo and you ment to say 'running' it from xcode should show more info? This is already as it is running in xcode. I'm not sure what you mean when you say 'stack trace', my previous comment contains the stack trace, is that not what you are looking for? It's not an application thread calling into MoltenVK and it crashing synchronously if that is what you are thinking. It's something that seems to get queued for asynchronous execution on a dispatch queue.

Yeah...fat finger and autocorrect strike again!
I was hoping to get something along the lines of the following call stacks. The first is a debug call stack captured in Xcode. The second is a crash report when the app is run outside of debugging.

For example purposes, I triggered these by forcing a null access inside the method you reported. It would be helpful to see something similar on the crash you're seeing.
Or in general, some way to force the same issue in a trackable manner.

Right so the callstack was from a release build so probably inlining caused that information to be lost. Here's the callstack with debug MVK built from tag v1.1.1.
I have no idea about MVK's internal workings, but perhaps that refcount is an indication of a released object?

I'm compiling the application with ARC enabled but i'm no expert on obj-c reference counting so not sure if that would affect anything?

Right so the callstack was from a release build so probably inlining caused that information to be lost. Here's the callstack with debug MVK built from tag v1.1.1.
I have no idea about MVK's internal workings, but perhaps that refcount is an indication of a released object?

I'm compiling the application with ARC enabled but i'm no expert on obj-c reference counting so not sure if that would affect anything?

I have no idea about MVK's internal workings, but perhaps that refcount is an indication of a released object?

Yes. That refcount definitely looks odd.

I'm compiling the application with ARC enabled but i'm no expert on obj-c reference counting so not sure if that would affect anything?

Are you compiling a build of MoltenVK with ARC enabled? Or just your app? It shouldn't matter about your app, but it would definitely matter if you are building MoltenVK itself with ARC enabled.

I have no idea about MVK's internal workings, but perhaps that refcount is an indication of a released object?

Yes. That refcount definitely looks odd.

I'm compiling the application with ARC enabled but i'm no expert on obj-c reference counting so not sure if that would affect anything?

Are you compiling a build of MoltenVK with ARC enabled? Or just your app? It shouldn't matter about your app, but it would definitely matter if you are building MoltenVK itself with ARC enabled.

ARC is just enabled for the app not for MoltenVK.
I've done some digging an i believe i know what may be happening. What i'm submitting is cmdA -> sema -> cmdB -> fence. Then my code is waiting on the fence and once it's triggered it'll immediately destroy the command buffers and the semaphore.
Now i see in MoltenVK that the MVKQueueCommandBufferSubmission::finish call is dispatched to a different thread using the addCompletedHandler function. I don't know exactly how this works but i see two threads that are capable of handling these finish calls. My hypothesis is that these two command buffers may finish at the same time causing the finish callbacks to run in parallel. Now if the cmdB finish callback is executed first, that will trigger the fence. Then if my thread is scheduled in i'll destroy the semaphore in response to the fence being triggered. Then the finish for cmdA gets scheduled in, trying to trigger the semaphore (which i just destroyed).
I made some changes to test:

In the MVKQueueCommandBufferSubmission constructor i've added ss.first->retain().
In the finish() callback i've added ss.first->release()
This solves the crashes for me.

Would you say this lack of semaphore retention would be the issue here?

ARC is just enabled for the app not for MoltenVK.
I've done some digging an i believe i know what may be happening. What i'm submitting is cmdA -> sema -> cmdB -> fence. Then my code is waiting on the fence and once it's triggered it'll immediately destroy the command buffers and the semaphore.
Now i see in MoltenVK that the MVKQueueCommandBufferSubmission::finish call is dispatched to a different thread using the addCompletedHandler function. I don't know exactly how this works but i see two threads that are capable of handling these finish calls. My hypothesis is that these two command buffers may finish at the same time causing the finish callbacks to run in parallel. Now if the cmdB finish callback is executed first, that will trigger the fence. Then if my thread is scheduled in i'll destroy the semaphore in response to the fence being triggered. Then the finish for cmdA gets scheduled in, trying to trigger the semaphore (which i just destroyed).
I made some changes to test:

In the MVKQueueCommandBufferSubmission constructor i've added ss.first->retain().
In the finish() callback i've added ss.first->release()
This solves the crashes for me.

Would you say this lack of semaphore retention would be the issue here?

Any idea on how to make progress on this issue? I don't have enough metal knowledge to validate how the second command buffer could be completed before the cross-queue semaphore was triggered but i do believe this is an issue in MVK and not my implementation. Should i just provide a pull request adding the retains?

Any idea on how to make progress on this issue? I don't have enough metal knowledge to validate how the second command buffer could be completed before the cross-queue semaphore was triggered but i do believe this is an issue in MVK and not my implementation. Should i just provide a pull request adding the retains?

In the MVKQueueCommandBufferSubmission constructor i've added ss.first->retain().
In the finish() callback i've added ss.first->release()
This solves the crashes for me.

Based on that, it looks like this was fixed in #1367, triggered by a different bug.
Can you pull the latest version of MoltenVK, and verify that it has been fixed, please? This will also be in next week's Vulkan SDK release anyway, so you can test it there if you prefer. But testing it before Monday 28th will ensure the fix works before being baked into the pending Vulkan SDK release.

jep that fixes it, thanks 👍


what is rev2

Hi,
I'm interested in implementing this design, but I'm curious what is rev2 mentioned in latest firmware change? Is PCB different from available in this repo?

Hi
Yeah, I’ve already built one and figured this out. Thanks for sharing all the docs and code, btw, I like using this keyboard very much!

Sorry for replying so late. Rev.2 in QMK is Rev.1 in this repo. Rev.1 in QMK is not available.

This needs to be in the repo's README @omkbd, I will be happy to raise a patch for the same if you want me to.


Bug: Backslash in code display as ￥ symbol

Environment
Host

item
version

OS
Amazon Linux 2

GROWI
3.2.10-RC

node.js
v8.12.0

npm
6.4.1

Using Docker
yes

Using growi-docker-compose
yes

(Accessing https://{GROWI_HOST}/admin helps you to fill in above versions)
Client

item
version

OS
Window 10

browser
Firefox Quantum 64.0 (64-bit)

How to reproduce? (再現手順)

Create new wiki
Add content with code block has backslash inside.

What happens? (症状)

Markdown side

View side

What is the expected result? (期待される動作)

Backslash inside code will display as backslash, not ￥ symbol.

#887 will fix this issue.


Add support for the new random.org API

Although the new API doesn't support anonymous requests and therefore can't completely replace the old one, it should still be available to users who supply a key for it. Documentation of the new API is at https://api.random.org/json-rpc/1/basic.

Done in 1.2, releasing tonight.


Use labels in native container code

Use labels instead of environment variables to detect rancher uuid and rancher networking.
Dont merge. This requires some coordination because it requires that we upgrade to 1.6.
I'll coordinate with @ibuildthecloud and @cloudnautique on when it can be merged.

LGTM.  Just curious.  Is the fallback to checking environment variables temporary?

@sonchang yes. temporary backwards compatibility

updated to io.rancher.container.*


WheelBuilder: use python version as cache key

When building wheels with a source distribution, we also need to take the the PythonVersion into account when hitting the cache. Currently, we only use the contents of the sdist.
When creating the WheelKey we should probably add this to the hash.
Pixi will also need to supply this information later.
@baszalmstra when using this in the content-hash it will not actually be the content, but I don't think we can utilize the cacache metadata in this case. We could also associate it with a specific key, WDYT?

Yeah you'd need to add the python version as part of the key.

Yeah in that case the Python-version needs to be part of the key. So if you would use the shahash as the key previously now it would be something along the lines of:
<sha>-<python-version>
You can of course encode anything in the key.

Closed by: #159


Allow for dynamic credentials file

Currently, the proxy can establish a connection using static credentials, but there doesn't seem to be a clean way to re-establish a connection if the credentials file changes. This can be problematic, especially in a Kubernetes sidecar serving the proxy to other pods.
Possibly a flag to check the credentials periodically and restart if needed?
Example:
.\cloud_sql_proxy.exe -instances=myproject-12345:us-central1:test=tcp:5433 -watch-credentials

This is what I'm doing for the sidecar at the moment to get this working via an inline bash script.
# Cloud SQL Auth Proxy Side Car
- name: {{ .Values.cloudSql.containerName }}
  image: {{ .Values.cloudSql.image }}:{{ .Values.cloudSql.tag }}
  command:  ["/bin/sh", "-c"]
  args:
    - |-
        echo $(date "+%Y/%m/%d %H:%M:%S") 'Starting proxy';
        LTIME=`stat -c %Z /etc/secrets/service_account.json`;
        /cloud_sql_proxy \
          -instances={{ .Values.cloudSql.project }}:{{ .Values.cloudSql.region }}:{{ .Values.cloudSql.instance }}=tcp:{{ .Values.cloudSql.port }} \
          -credential_file=/etc/secrets/service_account.json & PPID=$!;
        echo $(date "+%Y/%m/%d %H:%M:%S") 'Proxy started';
        while true;
        do ATIME=`stat -c %Z /etc/secrets/service_account.json`;
        if [ "$ATIME" != "$LTIME" ];
        then echo $(date "+%Y/%m/%d %H:%M:%S") 'Credentials changed, restarting proxy';
        kill -s SIGTERM "$PPID";
        /cloud_sql_proxy \
          -instances={{ .Values.cloudSql.project }}:{{ .Values.cloudSql.region }}:{{ .Values.cloudSql.instance }}=tcp:{{ .Values.cloudSql.port }} \
          -credential_file=/etc/secrets/service_account.json & PPID=$!;
        echo $(date "+%Y/%m/%d %H:%M:%S") 'Proxy restarted, process pid:' $PPID;
        echo $PPID;
        LTIME=$ATIME;
        fi;
        sleep 15;
        if ! kill -0 $PPID 2>/dev/null;
        then echo $(date "+%Y/%m/%d %H:%M:%S") 'Proxy not alive, exiting';
        exit 1;
        fi;
        done;
  securityContext:
    {{- toYaml .Values.securityContext | nindent 12 }}
  volumeMounts:
  - name: secrets
    mountPath: /etc/secrets
    readOnly: true

in a Kubernetes sidecar serving the proxy to other pods

Would you mind elaborating what you mean here?

I have a sidecar container initializing the proxy so other containers can use that proxy to establish connections to the database. That sidecar is using a shared mount volume with the secrets file. Another sidecar container loads that file with the SA Info and then updates the credentials when the lease on the secret has expired.
The issue I run into is when the credentials file is updated, the proxy doesn't re-establish a connection with the new credentials, and when the proxy is used, we get the following error:
2021/04/30 23:10:54 failed to refresh the ephemeral certificate for database-name:db-region:db-instance before expir
ing: Post https://sqladmin.googleapis.com/sql/v1beta4/projects/database-name/instances/db-instance/createEphemeral?alt
=json&prettyPrint=false: oauth2: cannot fetch token: 400 Bad Request
Response: {"error":"invalid_grant","error_description":"Invalid JWT Signature."}

The proxy container doesn't fail on this though and the container just keeps running.

OK. Thanks for the clarification.

Closing this as a duplicate of #205. Let's continue the discussion as needed over there.


Fix allowing encrypted volumes

An issue discovered from a user in the slack community. The encrypted flag was not being passed to libstorage correctly. This is a API as seen from dvdcli and rexray cli below. This fixes the issue.
REX-Ray CLI
{
   "name": "rexray",
   "availabilityZone": "",
   "encrypted": true,
   "iops": 0,
   "size": 1,
   "type": ""
}

DVDCLI
{
    "name": "dvdcli",
    "availabilityZone": "",
    "iops": 0,
    "size": 5,
    "type": "gp2",
    "opts": {
        "encrypted": "true",
        "size": "5",
        "volumetype": "gp2"
    }
}

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


fix(hydration): don't error if data-allow-mismatch provided for fragment

Fixes hydrateFragment method throws hydration mismatch error despite of data-allow-mismatch attribute exists in parent node.
fragment not enough children test was wrong since Hydration completed but contains mismatches. error was ignored in test environment and not checked at all
__TEST__ flag prevented to catch error for testing
added toHaveBeenErrored method that looks same as toHaveBeenWarned, but error level

Thanks for the PR. I've made some minor tweaks https://github.com/vuejs/core/pull/12505/commits/9e909466d103658fac2604f72e9287e8559edc90
LGTM

@edison1105 thanks you so much for instant help!
I'm very hope the PR will be merged next release


TypeError: Function.prototype.bind called on incompatible target

I applied a PS object to an application in different parts of my application. And, in the side menu section (which is generated on-the-fly using AJAX) I have an error with the statement "TypeError: Function.prototype.bind called on incompatible target" when I drag (only dragging, not mouse scrolling) on the scrollbar object.
As an illustration, I will use the following DOM sample:

...

 ....

I really believe this is an error triggered by PS, because when I did not initialize PS, then this error will not appear.
Can you help me out of this problem?
NOTE:

This error occurred only from the side bar, not from all part of my DOM.
You can see my application (download it first) from here
I am sorry for my bad English

UPDATE:
Hey, it turns out this error appears also in some other places.
Do you have any solution for my case?

Hii @noraesae,
After upgrading my PS object into version 0.7.0 this problem was not coming again. Thanks for your new version.


Typescript support

This adds typescript support in the simplest way possible.

I would suggest just adding these as suggestions to the readme for typescript users to add to their application. I don't see any reason for having to automatically add this.
Add this to your tsconfig.json:
"compilerOptions" {
   ...
   "baseUrl": "./src",
   "paths": {
     "@assets/*": ["./assets/*"],
     "@components/*": ["./components/*"],
     "@pages/*": ["./pages/*"],
     "@store/*": ["./store/*"],
     "@utils/*": ["./utils/*"]
   }
}

Already discussed in previous PR, currently when i find the time i ve been doing a rewrite in the DEV Branch, it will include jsconfig and tsconfig support! :)


Crossguard Saber

Hello,
I have a Crossguardsaber wired up and everything except the side blades is functioning properly. I just want to make sure I understand the wiring. I currently have the side blades neopixel data lines connected together with a 330ohm resistor going to A0 on the nano, is this correct? I also have the main blade data line connected to D13 with a 330ohm resistor and that one is functional. I might be misunderstanding the pixel accent data.
I also made sure to define it as a crossguardsaber, and put the main length at 120 and crossguard at 12
Sorry to ask, but I couldn't find an answer in the forums, issues or wiki.

So far you did all right, except for one thing: the sideblades form one stripe with the main blade. Therefore the side blades are connected first to D13, and the output of one side blade to the input data line of the main blade. A0 us not needed

Thanks for the quick response. I will update my wiring.


Gendocs failing CI

There are a couple files which are failing gendocs in PRs, but for which running the gendocs script in master produces no output:

cluster/addons/dns/kube2sky/RELEASES.md
examples/limitrange/README.md

Example:
http://travis-ci.org/GoogleCloudPlatform/kubernetes/builds/63049476

To be clear, the bug here is that these files aren't generated with the analytics link in master, but they are in branches.

PR #8343 was merged even though it failed the check.
cc @nikhiljindal @caesarxuchao @derekwaynecarr

Or should have failed, anyway.

The analytics change was merged this morning: #8296
It looks like the other changes were tested without that change (and vice versa).
cc @vmarmol

This should be fixed.


renderToNodeStream implementation

Are you planning on implementing async rendering anytime soon?
maybe for v8.0 release( I hope )?

Duplicate of #3342


Add ".fgb" extension to vector trim list in  main.cpp

Added the extension ".fgb" to the vector trim list to avoid including the extension in the layer id creation by default, when an mbtiles is generated from several flatgeobuf files.

What is the release proces of this change?


Issue 31: Support trivy scan2html k8s --report summary cluster interactive_result.html

closes https://github.com/fatihtokus/scan2html/issues/31


Link the Rose database browser to Trac

resolution_fixed   |    by<EMAIL_ADDRESS>
Changing this to minor since there exists a work around for updating the trac page via a cron job.
The syntax error produce by rosie web server is reported in https://accessdev.nci.org.au/trac/ticket/83

Issue migrated from trac:4 at 2024-01-31 17:22:18 +1100

<EMAIL_ADDRESS>commented

We've added a link to the rosie database on the front page of the wiki.
Going the other direction there appears to be a problem with rose - see https://github.com/metomi/rose/issues/1044

ibc599 commented

Need to update the MET office that a fix still needs to be done regarding URLs. eg rosie produces trunk@HEAD, when we expect trunk?rev=HEAD

<EMAIL_ADDRESS>changed owner from `` to ibc599

ibc599 commented

Looks like a fix https://github.com/metomi/rose/pull/1097 will be in the next version of rose

ibc599 commented

Syntax problem is a separate issue to the title of this ticket https://accessdev.nci.org.au/trac/ticket/83

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>commented

Replying to [comment:3 ibc599]:

Looks like a fix https://github.com/metomi/rose/pull/1097 will be in the next version of rose

This fix is not relevant to problem.
Ian/Robin

<EMAIL_ADDRESS>changed reviews which not transferred by tractive

<EMAIL_ADDRESS>commented

Fixed by changeset:f9e4c132f4dba90a6391dbd6ec5429fe9f21df86/puppet

<EMAIL_ADDRESS>changed owner from ibc599 to saw562

<EMAIL_ADDRESS>commented

Taking a look in  branch saw562/rose-refactor
Added a setgid script that updates Trac & can be added in a post-commit

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>changed resolution from `` to fixed

<EMAIL_ADDRESS>changed _comment0 which not transferred by tractive

<EMAIL_ADDRESS>removed milestone (was accessdev v1.0)

<EMAIL_ADDRESS>commented

Milestone accessdev v1.0 deleted


[JSDoc] Add missing documentation for botbuilder-ai-orchestrator

Fixes # 2602
Description
This PR adds all the missing JSDoc documentation for botbuilder-ai-orchestrator based on the errors thrown. Some documentation is ported and adapted from botbuilder-dotnet.
Specific Changes

Ported documentation from botbuilder-dotnet
Added @private for private methods.
Added JSDoc comments in 3 files.

Testing
In the following image there is a sneak peek about the ESLint Report with no JSDoc errors.

@santgr11 @ceciliaavila @iscaltritti-sw, applied feedback, thanks

promoted