Default texture in advancement tab

I installed the Data pack to my server, I also installed the texture pack. It seems that the data pack works fine except the tabs are not all side by side with each other but instead mixed in. Not a big deal there. But the texture ack does not seem to work on version 1.19.3 so every fish in the advancements just has the default tropical fish texture. Not sure if this is because I don't have optifine but instead a mix of different mods since optifabric hasn't been updated to 1.19.3 or something but if the pack could be updated to fix this on 1.19.3 that would be great since my friends are obsessed with catching fish

Hi, I just did the development to generate the pack, on the technical side, I'll let you contact: https://www.curseforge.com/members/eclairdefeu360

Ok thanks I will try and message them

Ok thanks I will try and message them

Okay, I talked to him, I made the changes. The links on curseforge have been updated, can you tell me if the changes fix the problem?


[E2E] Update the clearEditor util to use the ControlOrMeta key

Description
We previously had to use browserName in clearEditor to determine which key we need to press in editor simulation.
https://github.com/freeCodeCamp/freeCodeCamp/blob/59498e5b091fdcb17092f77c154aeb1886b6ec70/e2e/utils/editor.ts#L34-L39
But we now can replace that with just ControlOrMeta:
await page.keyboard.press('ControlOrMeta+a');

Changes needed

Update the clearEditor function to use ControlOrMeta
Remove browserName from the function signature (this also requires updating places where the function is called)

This may not work just as is, seeing how #55326 broke the webkit tests


Microsoft standard terminology.

Throughout Azure UI (Load Balancer and Network Watcher/Connection Monitor),  frontend and backend are 1 word.

Load Balancer and
https://cloudblogs.microsoft.com/opensource/2018/06/03/javascript-node-js-ruling-the-web-john-papa-interview/

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

ID: bbc69b4f-7d0b-f81c-b326-fa8786ec3654
Version Independent ID: 7955b3b8-e03b-6245-4366-1a588cb5cc33
Content: front end, front-end - Microsoft Style Guide
Content Source: styleguide/a-z-word-list-term-collections/f/front-end.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Is the query/suggestion to update about Azure UI or to update this style guide? Just asking.


RFM - colorconsole.portable

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details

Package URL: https://chocolatey.org/packages/colorconsole.portable
Package source URL: https://github.com/dtgm/chocolatey-packages/tree/master/automatic/colorconsole.portable/

Date the maintainer was contacted (in YYYY-MM-DD): 2021-12-13
How the maintainer was contacted: email

Closing due to the following comment.
Only the meta-package should be opened as an RFM request unless there are different maintainers for the install/portable package.


Broken constraint in transformed parameters block should generate a warning

Summary:
The manual suggests that constraints in transformed parameters are designed for error checking. If this is really the case, unsatisfied constraints should not cause a silent rejection, but also a clear warning message.
Description:
Stan manual, section 6.6 (on transformed parameters block) says

Like the constraints on data, the constraints on transformed parameters is meant
to catch programming errors as well as convey programmer intent. They are not
automatically transformed in such a way as to be satisfied. What will happen if a
transformed parameter does not match its constraint is that the current parameter
values will be rejected.

However, when constraints in the transformed parameters block are not met, the sampler silently rejects and causes a divergence, which may be hard to understand for users (see eg. this discourse thread: http://discourse.mc-stan.org/t/imposing-constraint-in-transformed-parameters-block/3921). Instead, the sampler should generate a clear warning message.
 Increasing adapt_delta above 0.8 may help. See
http://mc-stan.org/misc/warnings.html#divergent-transitions-after-warmup
2: Examine the pairs() plot to diagnose sampling problems

Expected Output:
A clear warning message, something like:
Warning: Attempt to assign out-of-bounds value to transformed parameter "real<lower=0> b"  Count: 2008.

Current Version:
v2.17.3

Thanks.
This repo isn't the immediate culprit here---it's the interfaces.  So I'm going to move this to an RStan issue.  I tried this model
parameters {
  real<lower = -1, upper = 1> scale;
}
transformed parameters {
  real<lower=0> scale2 = scale;
}

and reproduced your lack of message in RStan.
In CmdStan, here's what you see:
method = sample (Default)
  sample
    num_samples = 1000 (Default)
    num_warmup = 1000 (Default)
    save_warmup = 0 (Default)
    thin = 1 (Default)
    adapt
      engaged = 1 (Default)
      gamma = 0.050000000000000003 (Default)
      delta = 0.80000000000000004 (Default)
      kappa = 0.75 (Default)
      t0 = 10 (Default)
      init_buffer = 75 (Default)
      term_buffer = 50 (Default)
      window = 25 (Default)
    algorithm = hmc (Default)
      hmc
        engine = nuts (Default)
          nuts
            max_depth = 10 (Default)
        metric = diag_e (Default)
        metric_file =  (Default)
        stepsize = 1 (Default)
        stepsize_jitter = 0 (Default)
id = 0 (Default)
data
  file =  (Default)
init = 2 (Default)
random
  seed = 714361321
output
  file = output.csv (Default)
  diagnostic_file =  (Default)
  refresh = 100 (Default)

Exception: bar_model_namespace::write_array: scale2 is -0.0758982, but must be greater than or equal to 0  (in '/Users/carp/temp2/bar.stan' at line 5)

Meaningful warning message, which is good, but it shouldn't be terminating.  So I'll create a new issue for CmdStan.

This issue was moved to stan-dev/rstan#521


add autoscaler for ingesters

What this PR does:
Adds a HorizontalPodAutoscaler for the ingesters.
https://cortexmetrics.io/docs/guides/capacity-planning/ describes that ingester memory grows as series are added, so this autoscaler tracks memory usage to make scaling decisions.
https://cortexmetrics.io/docs/guides/ingesters-scaling-up-and-down/#scaling-down says that "adding more ingesters to a Cortex cluster is considered a safe operation", so this autoscaler behavior is configured to allow adding one ingester pod at a time, but does not automatically scale down.
Checklist

[x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Oops, you beat me to the merge/rebase.


rafactor timeslot/availablities

term accuracy only defined the availabilites accuracy.
add 'strict' mode which is what we have now
for timeslots, accuracy is default calendar accuracy but can be changed from 0.2 to 4.

how to display timeslot that doesnt fit the selected accurary on the calendar ?

rename 'creneau' to 'calendarSlot'
todo

conf-maker

assignment term

add 'non-strict' mode where timeslot can be created freely
'strict' mode is default, timeslot will have to match accuracy
'strict' to 'non-strict' allowed, not the other way around

task-maker

timeslot

calendar

strict mode

???

non strict mode

same as now

start/end date

timeslot cannot overlapp but accuracy check depends on strict mode

assigment

timeslot calendar

strict mode

userToTask

same as now

taskToUser

??

non strict mode

same as now

user-maker

nothing change, availability calendar uses term accuracy

#Tests remaining

non-stricte with timeslot from previous day (midnight thing)

margin-top should not even be concerned


Support http ^0.13.0

Looking for a version that depends on http 0.13.0 as there have been some breaking changes. Many packages have migrated to version 0.13.0 and they can no longer be used if you also depend on artemis.

Looks like this will be handled in #259


[4.6.1] CLOUDSTACK-9015 - Redundant VPC Virtual Router's state is BACKUP & BACKUP or MASTER & MASTER

This PR closes #1064
All the details can be found in the original PR, which won't be merged because it was created agains master. Once this PR is closed, the original one will be also closed.

Ping @remibergsma
PR against 4.6 has been created. No we need people to review/test it.
Cheers,
Wilder

@wilderrodrigues Starting my tests..

Ping @remibergsma
Are the tests still running?
Cheers,
Wilder

@remibergsma and @miguelaferreira faced problems related to SSH timeout whilst testing this PR. I retested it and did not face the same issue:
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 3 tests in 4882.851s

OK
/tmp//MarvinLogs/test_vpc_redundant_6MQWZS/results.txt (END)

We tested against the same virtualised environment, but running different steps. It makes it trick to find the root cause, so this PR will standstill for a while until I have time to retest and, hopefully, get an error as well so I can dig deeper.
Cheers,
Wilder

Although it’s based on an assumption, enforced by good pattern recognition, what happened to @miguelaferreira  and @remibergsma was:

the code does “delete_nat_rules()” and tries to SSH (expected failure). that works fine
The code does “add_nat_rules()” and tries to SSH again, which should pass - and did all the times I tried, and I’m not lying

so, what’s the problem then?
the “delete_nat_rule()” method is deleting the nat_rule, as it says, but not deleting the public IP that was acquired. So, when the code calls the “add_nat_rule()” method, it acquires a new IP and adds a nat rule to that IP.
what does it mean? that you guys got a situation where the test tried to SSH via the IP that doesn’t contain a NAT (PF) rule. that explain the ssh error.
I fixed it, but first need to test before pushing.
Cheers,
Wilder

Ping @miguelaferreira @remibergsma
Tests are fine, but of course we can only be sure if they work with you as well.
There was also a last sleep of 5 seconds that I removed. It now works as it should: no sleeps required!
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 3 tests in 5040.173s

OK
/tmp//MarvinLogs/test_vpc_redundant_8WKYY7/results.txt (END)

@miguelaferreira @remibergsma @wilderrodrigues  LGTM :+1:
nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-ISOLATED.cfg -s -a tags=advanced,required_hardware=true component/test_vpc_redundant.py

Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 3 tests in 3522.654s

OK

@remibergsma: some time to re-run the tests? Need a 2nd LGTM.

LGTM based on these tests:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py \
smoke/test_network.py

Result:
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 33 tests in 14431.271s

OK

And:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_privategw_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycle.py \
smoke/test_vpc_vpn.py \
smoke/test_service_offerings.py \
component/test_vpc_offerings.py \
component/test_vpc_routers.py

Result:
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
test_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_privategw_acl | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test for delete account ... === TestName: test_delete_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for admin account ... === TestName: test_public_ip_admin_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for user account ... === TestName: test_public_ip_user_account | Status : SUCCESS ===
ok
Test for release public IP address ... === TestName: test_releaseIP | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 42 tests in 8631.243s

OK


Silent failures are generally bad news

Pass the wrong type of second argument to addEventListener() and get a silent no-op. No warning (better), no error (best), not even any logging to indicate the caller is wrong. Also, the third equal sign is pointless, as both sides of the comparison are the same type. Using strict comparisons for everything makes the code less clear.
EventSource.prototype.addEventListener = function addEventListener (type, listener) {
  if (typeof listener === 'function') {
    // store a reference so we can return the original function again
    listener._listener = listener
    this.on(type, listener)
  }
}

Whenever I come across code like this, I wonder if it is just trying to be more "robust" for beginners (and doing them no favors) or is trying to cover up for inherent issues in the module.
So there is only one internal call to this method:
set: function set (listener) {
      this.removeAllListeners(method)
      this.addEventListener(method, listener)
}

No internal problem with this one, so:
EventSource.prototype.addEventListener = function addEventListener (type, listener) {
  if (typeof listener != 'function') {
     throw new Error('Listener must be a function!');
  }

  // store a reference so we can return the original function again
  listener._listener = listener;
  this.on(type, listener);
}

...allows the caller to know when they did something wrong (e.g. set onerror or onmessage to something other than a function), rather than failing silently. That way the developer can debug their code at the first point that it failed, rather than trying to figure out why things are not working as expected later, which can be very difficult with code like this. Currently, setting on* to anything other than a function ends up removing all listeners. Was this really the intention?
Another example:
this._close = function () {
    if (readyState === EventSource.CLOSED) return
    ...

Really? It's weird as some other module that wraps this one calls this method in its onerror callback, specifically if readyState == 2 (CLOSED), which is essentially a no-op. In any event, this is never a good way to start a function. :(
https://github.com/fanout/reconnecting-eventsource/issues/48
How about this?
this._close = function () {
    if (readyState == EventSource.CLOSED) {
        throw new Error('Connection is already closed!');
    }
    ...

If that one is supposed to be for your own code's benefit, then the above will help you find the problem with it. If it supposedly to help those who download this thing, then you may well "break" their code (quotes indicate it was already broken, trying to close an already closed connection). So may be too late to fix this one (a familiar refrain for libraries that become wildly popular before they are ready).
Think I am going to cross this one off the shopping list, as it is just too aggravating. What exactly is supposed to happen in an onerror callback? From what I've seen, emitted error events have either a status, a message, both or neither.
That "reconnecting" module that wraps this one has some interesting code in its onerror, but it hardly seems trustworthy (see the ticket linked above). I've traced through the code enough times at this point (defeating the purpose of using a third-party library) to know I don't want it, at least not in its current state.
In any event, good luck! :)

Here's another one, similar to the second one above:
function onConnectionClosed (message) {
    if (readyState === EventSource.CLOSED) return

Unfortunately, this one is called all over the place internally, with one case failing to pass a message (so message: undefined). Another "clue" anyway. :) Would love to see an attempt to document how to use onerror to do anything practical (and without slogging though all of this code).

Also note that this creates a self-referencing property called "_listener". Why?
// store a reference so we can return the original function again
  listener._listener = listener;

The caller passed this Function reference (as always in JS, by value), so what is the point of augmenting the object in this way?
The following (and only other) line in the containing function is:
this.on(type, listener)
Perhaps it may augment the Function object as well (hope not), but it sure won't change the reference, so will still have it in the end.
The method that removes listeners sets the created property to undefined. Also a question mark.
That leaves this, the getter for the on* properties:
get: function get () {
      var listener = this.listeners(method)[0]
      return listener ? (listener._listener ? listener._listener : listener) : undefined
}

So presumably the - listeners() method is returning the same reference with the same _listener property, which both reference the same thing. If it isn't returning that, but some other object with a "_listener" property, then why is it being called in this circumstance? Here we just want the Function reference that was set (or passed to addEventListener).
None of this makes sense to me. Seems like this module should just store these listeners somewhere (e.g. a Map) and return the references in the getter. It isn't doing that and what it is doing makes no sense at a glance.
445 lines with white space and comments. Wonder how many years it has sat in this state. Certainly I've used it in the past and it is probably part of many thousands of projects. I have seen a lot of discussions about how to keep it alive, but will be glad not to revisit them at this point. Only asking about this _listener thing out of curiosity.
Thanks!

While you make valid points, here is a counterpoint for the close function. This package focuses on W3C compliance and according to the docs, the close function should do nothing in the case the connection is already closed.
EventSource.close()
"Closes the connection, if any, and sets the readyState attribute to CLOSED. If the connection is already closed, the method does nothing."

As mentioned, we try to stay in line with the EventSource specification. The new eventsource@^3 has been rewritten in more modern style - if you have specific ideas/thoughts about improvements to it, feel free to open a new issue with specific suggestions.


Does this work with Expo?

I'm stil using react-native-scripts and have not ejected, so I can't add custom gradle scripts. Should react-native-config still work, and do I need to set up something so that it works with the Expo app? I'm currently just getting an empty object for Config.

I get the same thing. It doesn't seem to be working.

I've spent a couple of hours trying to get @jdrydn's solution above to work, and to no avail. Seems everyone else has managed to though, so no idea why it's not working for me.

Starting the expo local server with exp start as needed for notification setup
Have installed both babel-plugin-inline-dotenv and dotenv (and tried without dotenv installed too), both options haven't worked
Have deleted node_modules and package_lock.json
Have installed first with yarn, then removed and installed with npm
Have tried adding inline-dotenv to .babelrc inside env.development as well as at the root
Restarted the server and closed and re-opened Expo app at every turn

Current .babelrc:
{
  "presets": ["babel-preset-expo"],
  "env": {
    "development": {
      "plugins": ["transform-react-jsx-source","inline-dotenv"]
    }
  },
  "plugins": ["inline-dotenv"]
}

Are there any steps I'm missing that aren't listed above? I'm at the end of myself here, I just want to use environment variables I can't believe I've spent half a day on this and it still doesn't work 😞

I ejected from Expo. It works on the Java side, but the Javascript Config object remains empty

Fixed it by adding resValue "string", "build_config_package", "host.exp.exponent" per the documentation.

inline-dotenv plugin is working for me as described above. However every time I change the .env file I have to restart server and clean the cache by passing -c flag (otherwise the configuration won't change):
yarn start -c

@jdrydn Solution worked for my Expo Web Preview, but not on Expo Android Preview.
Only got undefined when accessing env keys

I can't seem to get it to work with a local config file like ".env.local" to contain the config. I get an exception saying "Your API key is invalid, please check you have copied it correctly". Even though I have the right one and trying to access it with: process.env.REACT_APP_FIREBASE_API_KEY
I suppose that process.env is wrong when having the key in .env.local but I can't figure out what else to do

Has anyone tried with Expo unimodules? The two seem to conflict for me and I can't figure it out for now

spent a lot of time screwing around with this. i found react-native-dotenv worked out of the box, but our team is using this with expo. Not sure what step I'm missing:
add babel plugin
npm install --save-dev babel-plugin-inline-dotenv
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  plugins: [
    "inline-dotenv",
    'react-native-reanimated/plugin'
  ],
};

add typings etc.
// types for .env vars
declare module "react-native-config" {
  export interface NativeConfig {
    API_BASE: string;
  }
  export const Config: NativeConfig;
  export default Config;
}

Nothing is found in the imported  Config
(or on process.env)
am I missing a step?

spent a lot of time screwing around with this. i found react-native-dotenv worked out of the box, but our team is using this with expo. Not sure what step I'm missing:
add babel plugin
npm install --save-dev babel-plugin-inline-dotenv
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  plugins: [
    "inline-dotenv",
    'react-native-reanimated/plugin'
  ],
};

add typings etc.
// types for .env vars
declare module "react-native-config" {
  export interface NativeConfig {
    API_BASE: string;
  }
  export const Config: NativeConfig;
  export default Config;
}

Nothing is found in the imported Config (or on process.env)
am I missing a step?

This is what mine looks like
/* eslint-disable @typescript-eslint/no-var-requires */
const fs = require('fs');
const path = require('path');

// eslint-disable-next-line func-names
module.exports = function (api) {
  api.cache(true);
  const envName = process.env.NODE_ENV || 'development';
  const dotEnvPath = path.resolve(__dirname, `.env.${envName}`);
  if (!fs.existsSync(dotEnvPath)) {
    throw new Error(`Babel config couldn't find dot env file path: ${dotEnvPath}`);
  }
  return {
    presets: ['babel-preset-expo'],
    plugins: [['inline-dotenv', { path: dotEnvPath }], 'react-native-reanimated/plugin'],
  };
};


expose markets page route externally without need to connect a wallet.

pave the path for shareable links to specific categories and potentially on boarding users
for example a tweet about TC opps with a link to
https://develop.shapeshift.com/#/markets/category/thorchainDefi

Reference: https://github.com/shapeshift/web/blob/9b47fbdecefce840f4c9d3683fdc789bf2f0d737/src/Routes/Routes.tsx#L110-L123

Exposing externally hould just be about not redirecting for the specific case of certain routes (e.g markets)
/category/ routing is already implemented so nothing to do here 🎉

If a use attempts to navigate to outside the markets page, we throw this modal in the same way we do for the demo wallet


Add getters for gelf appender data

I want to test if all parameters were set correct by my
programmatically created gelf appender

could you please rebase on current master and resolve the conflict?

of course :)

Now it should be ok.


[FEATURE] vGPU Support

Is your feature request related to a problem? Please describe.
Customers with NVIDIA cards that support the vGPU feature currently can't use it with VMs created by Harvester. This is preventing them from using the acceleration features in their workflows, adding cost by making their work take longer.
Describe the solution you'd like
Adding support for NVIDIA vGPU to the host, including the drivers, configuration, and a UI to manage the configuration. Specifically, the UI should allow the user to enable virtual GPUs and then assign them to VMs. This is different from #992, since there can be multiple vGPUs for a single physical GPU. The multiple vGPUs can be flexibly shared so that multiple VMs use the same GPU. This is different from #992, which allows a single VM to control a single GPU, in a one-to-one manner.
Describe alternatives you've considered

#992 but it's not sufficient, because there are NVIDIA devices and workloads that can't possibly max out the device. The solution is to add support for this vGPU (in Kubernetes it's called a Mediated Device), and then multiple VMs can share the underlying hardware and make maximum use of it.

Additional context
vGPU is analogous to SR-IOV, in that they both involve drivers (that will be in the host) which expose virtual devices, with their own PCI Addresses. Those virtual PCI addresses can then be fully owned by VMs. The PCI passthrough work in #992 is necessary to begin working on this issue.

Reopen. QA should verify the issue before closing it.

vGPU testing scenarios
For testing all testing was done on focal cloud images with cuda and a nVidia a102 GPU
[] Enable vGPU addon PASS
[] Select server side driver PASS
[] Try to add vGPU profiles to card that doesn't support vGPU PASS
[] Add vGPU devicePASS
[] Remove device PASS
[] Add two vGPUs to vm FAIL #5289
[] add one vGPU to VM that already has active vGPU FAIL #5289
[] Try to add already used device UNSURE The functionality on this
[] Try to add too many devices in advanced vGPU devices PASS
[] Try to remove vGPU that is in use FAIL #5293
Upgrade
[] Add vGPU on server that has PCI passthrough addon enabled after upgrade from v1.2-head to v1.3.0

validated in master-a2c98e96-head. Closing as validated. We need to test that upgrade scenario separately.


Most probable fix for python invalid argument error

This is the probable fix for that random problem of python: invalid argument error.
Last time when I'd seen that error I had noticed that the error was coming from the python present at $PREFIX/bin. Also, when I tried running the same executable, it gave me the same error whereas python at /opt/conda/bin was working right. Since yesterday, I've not seen the error again after this change. Also, this error was observed mostly during TF builds, rest all the recipes have been working fine. But since the error was randomly reproducible, I'll wait for 2 more days to confirm that this change will fix that error. Even if it doesn't, I think this change should anyway be merged as it is necessary.
@jayfurmanek - If you have a container where the same error is being recreated, could you please try this fix?

Yes, good find. This looks like a problem. I think without this change, a build of TensorFlow can occur with any version of Python, but at runtime it will try to use whatever version is specified.


Improve derivations

refactor derivations
move package files from $out to $out/node_modules/package_name

addresses https://github.com/thomashoneyman/purescript-overlay/issues/37

@thomashoneyman, please, review.

Please don’t rush me — it’s not been 24 hours since this was opened. I’m quite busy at the moment and will most likely be able to review today, but I can’t promise a timeline.


where to get tmLanguage and language configuration files

First, thank you so much for this library and the other libraries you maintain for Monaco! I'm trying to set up a simple TypeScript editor in Next.js with TextMate support and wondering where to get the tmLanguage and language configuration files?

Hello, you can get them from https://github.com/microsoft/vscode/tree/main/extensions/typescript-basics

Wonderful, thank you!

@CGNonofr do you happen to know why I'd be getting this error when calling setGrammars?

This is what I'm trying to do, full example here:
setGrammars(
  [
    {
      language: 'typescript',
      scopeName: 'source.ts',
      path: './typescript.tmLanguage.json',
    },
    {
      language: 'typescriptreact',
      scopeName: 'source.tsx',
      path: './typescriptreact.tmLanguage.json',
    },
  ],
  async (grammar) => {
    switch (grammar.scopeName) {
      case 'source.ts':
        return (await import('./TypeScript.tmLanguage.json')).default as any
      case 'source.tsx':
        return (await import('./TypeScriptReact.tmLanguage.json'))
          .default as any
    }
    throw new Error('grammar not found')
  }
)

You need to set the languages first by using setLanguages
Take a look at https://github.com/CodinGame/monaco-vscode-api/blob/main/demo/src/setup.ts

Hm, I forked that file and am doing that above here like this:
setLanguages([
  {
    id: 'typescript',
    extensions: ['.ts', '.tsx'],
    aliases: ['TypeScript', 'ts', 'typescript'],
    configuration: './typescript-configuration',
  },
])

Maybe I missed something else since there isn't any highlighting that's applied and the callback to setGrammars is never called.

Ok sorry I didn't go to see your example.
I don't know about the typescript language but the typescriptreact language is missing for sure, maybe it's linked

No worries! Yeah, not really sure what's going on. Even if I just try to load a TypeScript version without any of the React portion it still throws that error in the console.

I've just tried taking your files (grammar+config) and adding the language/grammar registration into the demo here and everything seems fine. Please provide a simple project that reproduces it

Weird, I wonder if there was something in the monorepo causing that. I made a minimal repo here with a simple Next app that showcases a new issue with the configuration file not loading even though it's being loaded through setLanguageConfiguration. Also, do I need to load TypeScriptReact grammar? Hoping to at least get TypeScript working first and then I can look into React. I really appreciate all of your help so far! I'm pretty lost on most of this and just looking to get textmate grammar highlighting 😄

I don't have the same error as you on your repo, but I have others

missing .json on the language configuration
the webpack worker loader was transforming some monaco internal files
You were not setting the theme after installing it
The worker wasn't configured to allow jsx syntax

here's the patch that makes it work: fix.zip

Woot, THANK YOU SO MUCH! This is working perfectly now 🎉 again, greatly appreciate all of your help! 🙏

Hello again, I tried upgrading recently to the latest monaco-vscode-api version and I'm running into issues again trying to get syntax highlighting working properly 😞. I'm not entirely sure what's going, but I tried following the recent changes to the demo in my PR here, but it just renders an editor with no theme. It looks like none of the files ever try to load so assuming it may be some configuration I'm missing in the new API.

Ok there is a bug: if you don't use the vscode api, the extensions are not loaded
Use any vscode api and it'll work, the bug will be fixed in the next release
for instance:
import * as vscode from 'vscode'

vscode.languages.registerColorProvider('java', {
  provideColorPresentations() {
    return []
  },
  provideDocumentColors: function (document: vscode.TextDocument, token: vscode.CancellationToken): vscode.ProviderResult<vscode.ColorInformation[]> {
    return []
  }
})

You have also an issue with the oniguruma wasm loading, you must add assetPrefix: 'http://localhost:3000/', in your next config

Ah ok good to know, thank you, that looks like it fixed it! 🙏  Is there any way to load oniguruma in this new configuration without needing to provide a specific asset prefix like previously? If possible, I'd like to automate this for the end user. I could potentially read the package json and see if they have set a custom port, but then I'd also need to know the production url 🤔

Not anymore, it's related to the way vscode is loading assets


Source of Kitura callback info

It's not clear where to find the Kitura server callback info when configuring local.env as, unlike the the other "credentials" this info is not found in the Connections tab of the app.
I assume I am right in thinking that the KITURA_host, KITURA_port, and KITURA_schema can each be deduced from the URL of the page you get from clicking View app button on the app's page of the (new) Bluemix console.

Thanks, this is referring to where  the kitura server is running. This is the route of the CloudFoundry application if you're running on Bluemix.  You can see details in the "Service Credentials" section under this readme (last paragraph in that section): https://github.com/IBM-Swift/BluePic/blob/master/Docs/OpenWhisk.md

@iainhouston We made a few minor updates to the docs, hoping that would make things a little clearer. Yes, you are correct, those values can be deduces from the URL. Updated docs: https://github.com/IBM-Swift/BluePic/blob/master/Docs/OpenWhisk.md

Thanks @rolivieri that's clear now


LIN-2518: Step1 Paywall Configuration

Added the ability to configure Step1 title, background colour, and image based on the Offering.metadata.
Proposed metadata structure:

Ticket:
https://vectornator.atlassian.net/browse/LIN-2518

@myeyesareblind We had to separate imageName into two properties - iOS and macOS. Please, have a look here: https://github.com/LinearityGmbH/purchases-ios/pull/45/commits/ee4fafb831f527319d6c60b7667a87be7ea5f56b


Add a Typescript definition file

I'm new to React, Typescript, etc but have managed to use this component with Typescript by creating a custom type definition file.
declare module 'react-hls-player' {

	export class ReactHls extends React.Component<DetailedHTMLProps<VideoHTMLAttributes<HTMLVideoElement>, HTMLVideoElement>> {
		url: string;
		hlsConfig?: Partial<Hlsjs.Config>;
		controls?: boolean;
	}

	export default ReactHls;
}

I'm unsure if it's correct but it got the basics working for me.   Hopefully this helps someone else until someone more experienced than myself could perhaps improve the definition and submit a pull request.

Thanks for the feedback @hastarin. My knowledge in Typescript is fairly limited, but I agree that proper Typescript definitions will definitely make a great addition. I originally forked this project as it looked like it was abandoned and was one I needed. My goal was to fix issues that I was running in to, while also maintaining the package and adding new features. I am currently looking in to the possibility of a full rewrite and releasing as version 1.
I will do more research in to this and as time allows will add Typescript compatibility. I'll leave this issue open for now so others can see a possible solution until then.

@hastarin I've released a V1 of this package. It now includes a typescript definition file by default for all that intellisense goodness! API should be the same, this was just a rewrite a long with some improvements.
7b8979cafb8242b9fd4794fff498b7969dd92b55

Thanks @devcshort I will have to make time to update my little test site I am using this in.
At a glance though something closer to the definitions I used would have worked a lot better?
You're actually extending existing controls and I would have thought it's easier to use their definitions than redefine them.
I'll have to try it to see, but would think that with what you've gone with some of the magic of Typescript, such as Intellisense for various properties, will be unavailable as you've defined them as objects.
Again, I'm no expert, my language of choice is C# which is why I've been using Typescript for a little side project in the first place.

Thanks @devcshort I will have to make time to update my little test site I am using this in.
At a glance though something closer to the definitions I used would have worked a lot better?
You're actually extending existing controls and I would have thought it's easier to use their definitions than redefine them.
I'll have to try it to see, but would think that with what you've gone with some of the magic of Typescript, such as Intellisense for various properties, will be unavailable as you've defined them as objects.
Again, I'm no expert, my language of choice is C# which is why I've been using Typescript for a little side project in the first place.

You’re completely right. I shouldn’t have extended it the way I did. My knowledge of Typescript is minimal and I was fumbling with it earlier. I was too focused on some of the extras that I completely missed the proper typings for some of the things I did as objects. I’ll push out a fix as soon as possible to have better typings. Thanks for the feedback!

@hastarin I tried to update my Typescript definition so that it could extend the HTML Video Element, but I started running in to issues from that. It would tell me that all of the attributes are considered required (width, height, autoplay, etc). Did you run in to this at all? I don't want to force this requirement if I'm to have the hls video player be an extension of that. I also ran in to issues when trying to overwrite the style attribute to use React's style interface.
I'm going to reopen this in case anyone else may be able to help as well. I want to try and get all the Typescript definitions working properly.

@hastarin I tried to update my Typescript definition so that it could extend the HTML Video Element, but I started running in to issues from that. It would tell me that all of the attributes are considered required (width, height, autoplay, etc). Did you run in to this at all? I don't want to force this requirement if I'm to have the hls video player be an extension of that. I also ran in to issues when trying to overwrite the style attribute to use React's style interface.

No I did not.  The only required attribute in the definition I used is url.  Make sure if you're doing something like the following:
hlsConfig?: Partial<Hlsjs.Config>;
You need to use Partial so it doesn't require all attributes from the referenced type.

No I did not. The only required attribute in the definition I used is url. Make sure if you're doing something like the following:
hlsConfig?: Partial<Hlsjs.Config>;
You need to use Partial so it doesn't require all attributes from the referenced type.

I had tried the Partial based on your example, which worked for the Hlsjs Config, but I was also trying to have the base extend from the html video component which is where I was running in to issues. Essentially I want all html video component props to be accessible as root props rather than being a separate object. As soon as I extended HTMLVideoElement I received errors that all video element props are required. Maybe I should replicate your example completely rather than just making an interface. 🤔

Just released version 1.1.0. I updated the Typescript declaration to use your example. I made one small change to how the types extend as I was getting class based component methods showing up in intellisense for props. It seems to be working as it should now. Let me know if you think there are any other changes I should make to it.
Thanks again!


[Bug] Handled Clearing access count for a dictionary column in compaction and limit case

Code Modifications done:

Modified code to handle clearing of access count in compaction operation and query with limit case.
Corrected bug in forward and reverse dictionary to clear dictionary access count in case of failure to load dictionary.

LGTM, Merged to master.


Adding add table row and column rest actions

Description

I prepared table row and column rest actions. I'm not completely sure that my approach is the best one to perform these actions, so I open this PR to get feedback.
I intend to commit web actions in a different PR.
Adding table row is pretty straightforward, I just move up to the parent node and add a new row node and place it to the correct position with orderBefore.
Example call: http://localhost:8080/content/bulma/pages/Personal/jcr:content/pagecontainer/hero/table/tablehead/tablerow.bulma-table-service.add-table-row.action?insertBefore=true
Adding table column is a bit more complex. Since we have no table column component, only rows and cells, the rest action here is expected to be called on the table cell resource, and it iterates through all the table rows and inserts a cell for each row before or after the selected cell. So we expect a custom action to pop up when the user selects a table cell to add column before or after that table cell.
Example call: http://localhost:8080/content/bulma/pages/Personal/jcr:content/pagecontainer/hero/table/tablebody/tablerow1/tablecell2.bulma-table-service.add-table-column.action?insertBefore=true
Motivation and Context

Screenshots (if appropriate)
Upgrade notes (if appropriate)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue) labeled with bug
[x] New feature (non-breaking change which adds functionality) labeled with enhancement
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code standards of this project.
[ ] My change requires updating the documentation. I have updated the documentation accordingly.

@paweljozwicki I just pushed the following changes based on your suggestion:

moved some logic (find table, find rows etc.) to rest models, so perform method in rest action is less complex
general refactoring, structuring the code better overall
added support for direct rows under the table
added unit tests for the rest models (needed to register the RequestParameterInjector to make it work (context.registerService(new RequestParameterInjector());), was not easy to figure out :) )

@piotr-szewczul-ds & @paweljozwicki - I just pushed my latest changes which includes all the intended changes for rest actions and models including the unit tests for them. Please take a look and let me know your feedback. thanks!

Closing as I opened https://github.com/websight-io/bulma/pull/122 which contains these changes as well.


Templates problems with vsphere datastore

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
There are a number of minor problems with vsphere node templates and datastores/datastore clusters.

On a vsphere node template, 1. when a datastore cluster is chosen, 2. a specific host is chosen, 3. the disk in the cluster with the lowest usage is not attached to that specific host, rancher will try and provision with the correct host, but the disk with lowest usage will be selected from the cluster.  If that disk is not on the host selected, provisioning will fail because the disk cannot be found by host. The fix is to provision the disk from the cluster that matches with the host.
On vsphere node template 1. when a specific host is selected, 2. a datastore is selected and that disk is actually part of a datastore cluster.  The node will not prevision.  No errors are supplied.  Rancher will just wait for the node to become available and when it doesn't (because it was never provisioned).  It will attempt it again.  (Note:  this was tried because the previous selection was trying to provision the wrong disk for the selected compute resource)

These worked:

selecting a host and a matching disk that is not in a datastore cluster
selecting "any" for the host and the datastore cluster

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5.6

Installation option (single install/HA): single install

Docker version (use docker version):

Client:
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.13.8
 Git commit:        afacb8b7f0
 Built:             Fri Dec 18 12:15:19 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.13.8
  Git commit:       afacb8b7f0
  Built:            Fri Dec  4 23:02:49 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.3.3-0ubuntu2.2
  GitCommit:
 runc:
  Version:          spec: 1.0.1-dev
  GitCommit:
 docker-init:
  Version:          0.18.0
  GitCommit:

I do not know if the problem still exists, because my builds are mor sophisticated and use ansible templates for builds.  My ansible templates allow me to pick either a cluster disk or standalone disk as well as a specific node(s). So I never run into it anymore.


[QUERY] HDInsight error message is not clear enough when putting wrong storage account name

Query/Question
I'm getting below message when creating cluster via HDInsight service.
The storage account name passed:
https://<storage_account_name>.blob.core.windows.net/
Get the error message:
{"error":{"code":"GatewayTimeout","message":"The gateway did not receive a response from 'Microsoft.HDInsight' within the specified time period."}}
It misled that the issue was from server end.
After I update the name to:
<storage_account_name>.blob.core.windows.net
Everything works fine with the updated value.
Why is this not a Bug or a feature Request?
It might be more like enhancement. It would be better if the service could return error message with clear info or link to some help-doc so that the users could investigate whether they need to update the request configurations.
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE : [e.g. IntelliJ]
Version of the Library used

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Query Added
[ ] Setup information Added

hi @xccc-msft could you pls help to take care of this? thanks

Adding Service Attention as it's required by service ask.

Thanks for your feedback, I will investigate it.

I have transferred this feedback to the service's backend team.


On Darwin changing 'base64 -w0' into 'base64 -b0'.

On Darwin base64 has no option named '-w0' but uses the option '-b0'.
This PR checks if it is or is not on Darwin and uses the current base64 option according to the used platform.

Is there an alternative way we can make it work on Darwin, @robshelly?

This has now been resolved in the this commit.
As of that commit, the Ansible playbook creates the default configuration for Alertmanger during installation, which is already Base64 encoded. If users want to create a their own configuration the documentation describe this process  and provides a sample config file.


Generics can't be used in SFC with both script and script setup blocks

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I'm using eslint-plugin-vue.
[X] I'm sure the problem is a parser problem. (If you are not sure, search for the issue in eslint-plugin-vue repo and open the issue in eslint-plugin-vue repo if there is no solution.
[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.41.0
What version of eslint-plugin-vue and  vue-eslint-parser are you using?

<EMAIL_ADDRESS><EMAIL_ADDRESS>
What did you do?

Configuration
parser: 'vue-eslint-parser',
  parserOptions: {
    parser: '@typescript-eslint/parser',
    sourceType: 'module'
  },

<script lang="ts>
...
</script>
<script setup lang="ts" generic="T">
defineProps<{ prop: T }>();
</script>

What did you expect to happen?
Generic type argument doesn't trigger the no-undef rule when being used in the <script setup lang="ts">...</script> block alongside a normal <script lang="ts">...</script> block
What actually happened?
When using a normal <script lang="ts">...</script> block alongside a <script setup lang="ts" generic="T">...</script>, generics trigger the no-undef rule from eslint's core ruleset.
Link to Minimal Reproducible Example
Minimal reproduction: https://github.com/ferferga/vue-eslint-parser-repro
Original repository affected with the issue: https://github.com/jellyfin/jellyfin-vue
Additional comments

I'm sure this problem belong to this repo because the faulty rule is from eslint core itself
In my repo, I found out that mixing both types of script blocks means that imports could only be inside the non-setup script block
While creating the reproduction repository, I also found out that vue-eslint-parser thinks that <script setup> is part of the template when mixed with the <script> tag. I use prettier in my original repo, so these issues were turned off by eslint-config-prettier, but if you run eslint on the reproduction repository (which doesn't have prettier), besides the no-undef rule being triggered in the Triggered.vue file, additional problems will be applied to the block itself, like vue/no-parsing-error and vue/html-closing-bracket-newlines.

TLDR: It looks like vue-eslint-parser isn't build towards considering <script>...</script> and <script setup>...</script> can be mixed together

Thank you very much @ota-meshi! It's working wonderfully now!

Thank you for letting me know.  I'm glad it worked.


Fix tests and make the path normalisation work on windows

The tests run fine for me locally using npm test, so I have no idea wtf is going on with circle

Ok, updating ts-node seems to have fixed it


Reading emails from inbox

I could not find any API to read email. It would be great if we have this feature.

Hi, this is a different protocol, IMAP/POP3. Simple Java Mail deals with the SMTP protocol for sending emails.
However, this is a planned feature for the future, but not the near future.


Power toyz run: It seems that it doesn't  start on startup

Microsoft PowerToys version
0.70.1
Installation method
GitHub, PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Settings:
Launch PowerToyz  on startup ON
Always launch as admin ON
Simply power on the computer
✔ Expected Behavior
After powering on the computer PowerToyz should launch
PowerToyz Run should be active and using the shortcut should pop it on the screen ready to be used
❌ Actual Behavior
PowerToyz seems to start with the startup but PowerToyz Run cant be activated with the shortcut
you have to manually relaunch the PowerToyz so you can use the PowerToyz Run
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/24339


Helper to easily obtain a response and its body

We added ExampleRetrieveHTML, which did this with network.GetResponseBody and a dozen lines of Go code, but apparently it's not that simple. We also need to wait for network.EventLoadingFinished on top of the response's network.EventResponseReceived.
At this rate, the user would probably need over twenty lines of non-trivial Go code. This probably calls for a simple helper to deal with the simple use cases.

给大家推荐一个全球性的网站导航网站
https://887d.com
它可以找到全球可式可样的网站。通过语言，内容分类，语种，国家等，按你的需要找你的网站。
在全球有很多优秀的网站不被我们所知晓，他们可以提高我们的知识，丰富我们的生活，让我们能了解一些前所未闻的信息。这个网站通过详细的分类，精准的定位。能方便让你找到需要的网站。
如果你注册帐号，还能长期收藏常用的网址。
它就是 887d.com
建议你收藏本站，方便打开你需要的网站。

Perhaps, I'mn a nitwit for these solutions, but how can the document title be retrieved in the easy way that you have now, and a full source only with a dozen lines of code? How come that fill html source needs a EventResponseReceived kind of solution, while Title is just Title (I never looked what it does under the hood).  I can imagine, the source needs to be in memory to have access to the title. But as I said: I'm the nitwit here ;-)

Btw. Now I did some extra digging.  Title is evaluated like a javascript command. so I was looking if there is such a thing for all of the html. I found document.all to contain that information.
When I do document.all[0] or document.all[0].innerHTML I get more info in the console. Perhaps this is usable?

We already have InnerHTML and OuterHTML as actions you can use. This is different from obtaining the original HTML response sent by the HTTP server, though. That requires waiting for a couple of events and executing an action, so it's not as simple.

This is doable, but the there's a reason that Chrome/ium does a secondary retrieval when doing view-source:. Will make a point of implementing this along with HAR capture ability.

Would love to see this feature added having to create a listener and capture the response body is more painful then it needs to be for any XML or JSON requests this would be preferable


"Is Exactly" filter still uses LIKE

Why does is the "Is Exactly" filter using LIKE? This removes any ability to rely on indexes in the list view. Some of my tables have millions of rows making a like of that nature impossible...

This is still an issue with the latest version 0.8.1. As @ehoch mentioned this is horribly inefficient, especially for indexed columns. I have put a small patch in that makes it use the '=' operator, and we saw a huge improvement in performance for "Is Exactly" Filters. I simply added this to build_statement_for_string_or_text:

I actually see that when using "is exactly" filter the query built with ILIKE.

When could this be fixed?

Fixed by #3000.


make cp print usage correctly

Noticed that cp displays usage twice. Possibly because clap did not support custom usage at the time.
Custom usage looks like this:
cp [OPTION]... [-T] SOURCE DEST
cp [OPTION]... SOURCE... DIRECTORY
cp [OPTION]... -t DIRECTORY SOURCE...

versus autogenerated
cp [FLAGS] [OPTIONS] [--] [paths]...

Thanks!


net/http: test panic on linux-386

While cutting 1.7.5:
panic: Fail in goroutine after TestIdleConnH2Crash has completed

goroutine 6665 [running]:
panic(0x83f0620, 0x18ae4330)
        /tmp/workdir/go/src/runtime/panic.go:500 +0x331
testing.(*common).Fail(0x18e30300)
        /tmp/workdir/go/src/testing/testing.go:412 +0x10d
testing.(*common).Error(0x18e30300, 0x1894ac14, 0x1, 0x1)
        /tmp/workdir/go/src/testing/testing.go:478 +0x61
net/http_test.TestIdleConnH2Crash.func2(0x84460d3, 0x3, 0x18c513c8, 0xf, 0x0, 0x0, 0x0, 0x0)
        /tmp/workdir/go/src/net/http/transport_test.go:3545 +0x1d9
net/http.(*Transport).dialConn(0x18b140a0, 0x85d8680, 0x18c51460, 0x0, 0x18c513c0, 0x5, 0x18c513c8, 0xf, 0x0, 0x0, ...)
        /tmp/workdir/go/src/net/http/transport.go:949 +0x3c5
net/http.(*Transport).getConn.func4(0x18b140a0, 0x85d8680, 0x18c51460, 0x18c514a0, 0x18b4a500)
        /tmp/workdir/go/src/net/http/transport.go:885 +0x48
created by net/http.(*Transport).getConn
        /tmp/workdir/go/src/net/http/transport.go:887 +0x3a0
FAIL    net/http        16.102s

Yeah, test looks like it could use work. It's also full of typos. Will fix, thanks.

Just saw this failure on OpenBSD:
--- FAIL: TestIdleConnH2Crash (0.06s)
	transport_test.go:3742: write tcp <IP_ADDRESS>:42762-><IP_ADDRESS>:4536: write: socket is not connected
FAIL
FAIL	net/http	5.203s

https://storage.googleapis.com/go-build-log/b809bbbc/openbsd-amd64-60_156a748f.log

Timed out.


feat: support rss2 and multi-format feed

hexo-generator-feed has long support generating RSS2-formatted feed (in addition to the default Atom), so this PR address that.
hexo-generator-feed 2.1+ now supports generating both atom and rss2 at the same time (if enabled, currently still defaults to atom-only).
2.1+ also insert the autodiscovery tag by default, so it's no longer necessary for theme to insert it. However, user might be using older version, so it's up to the theme dev whether to support it or not. There is also a slight performance advantage if theme inserts it, instead of plugin doing it.
More details: https://github.com/hexojs/hexo-theme-unit-test/pull/25

that sounds awesome, thanks


[doc] Deprecation of data- attributes

In release notes, it is stated that data- attributes have been deprecated in favour of "React"-style attributes. However, this doesn't apply to the data-simplebar attribute that is required to apply simplebar on a specific element, just simplebar doesn't work.

Hi,
It's only for the simplebar-react package. It does not apply to the main SimpleBar library. It's only if you use React.

@Grsmto: OK, would it make sense to make this more clear in the docs of the other packages?


File notification platform

This PR adds a simple way to store notifications in a plain text file.

+1


cli: add fitdump for dumping bytes segments

For low-level debugging purpose.

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

Project coverage is 100.00%. Comparing base (e609368) to head (330a2d7).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #199   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3572      3572
=========================================
  Hits          3572      3572

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


Allow combining of text property highlights and regular syntax hl

Is your feature request related something that is currently hard to do? Please describe.
Currently highlights from text properties completely override regular syntax highlighting,
so it is hard/impossible to them in combination, with properties as an auxiliary source.
Describe the solution you'd like
Add option 'highlight_combine': v:true to prop_type_add(), which will combine highlight attributes from syntax and text property when drawing.
Alternatively, always combine these attributes and require 'highlight_combine': v:false to turn it off.
Additional context
I use a Language Server Protocol plugin and would like to have errors and warning underlined through textprops, but otherwise keep the syntax.

Good idea.


"/anything" route only allows access by GET method

> curl -X POST -d 'a=1&b=2' https://httpbin.org/anything

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>405 Method Not Allowed</title>

Method Not Allowed

The method is not allowed for the requested URL.

> curl -X PUT -d 'a=1&b=2' https://httpbin.org/anything

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>405 Method Not Allowed</title>

Method Not Allowed

The method is not allowed for the requested URL.

notes:

code
should also output: 'data'

maybe something like..
@app.route('/anything', methods=['GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'TRACE'])
@app.route('/anything/<path:anything>', methods=['GET', 'POST', 'PUT', 'DELETE', 'PATCH', 'TRACE'])
def view_anything(anything=None):
    """Returns request data."""

    return jsonify(get_dict('url', 'args', 'headers', 'origin', 'method', 'form', 'data', 'files', 'json'))

Because the /:verb routes don't allow path data, (e.g. /get/123) the combined effect with this issue is that there's no way (that I can see) to use httpbin to simulate a RESTy 'collection'.
If the /anything route allowed all methods, then one could POST /anything; GET /anything/123; DELETE /anything/123; etc. (Note I'm not suggesting that httpbin actually create resources, that's clearly out of its scope.)

Because the /:verb routes don't allow path data, (e.g. /get/123) the combined effect with this issue is that there's no way (that I can see) to use httpbin to simulate a RESTy 'collection'.

Wrong: /status/:code allows all methods, so valid HTTP status codes can be used as resource IDs.

Though it isn't my place to comment (for the author) on the purpose of this tool, it feels to me that it's entirely out of scope for the backend to allow the user to define their own custom RESTful routes and then parse data out of them.. to report that data in the response. None of which has anything at all to do with the issue at hand.
GET works fine (sans any special parsing logic):
curl -X GET https://httpbin.org/anything/foo/bar/baz

{
  "args": {},
  "headers": {
    "Accept": "*/*",
    "Connection": "close",
    "Host": "httpbin.org",
    "User-Agent": "curl/7.47.0"
  },
  "method": "GET",
  "origin": "...",
  "url": "https://httpbin.org/anything/foo/bar/baz"
}

..still, my concern is being able to echo headers and data for all of the standard request methods. (PS: GET should also echo data.. there's nothing in the spec that says a server should ignore it)
..sorry for speaking out of turn; I just want this issue to stay on point.

I just want this issue to stay on point.

I'm sorry if you feel I was leading it astray: I only meant to offer an example use-case for why one might want to use the various methods on /anything.

it feels to me that it's entirely out of scope for the backend to allow the user to define their own custom RESTful routes and then parse data out of them.. to report that data in the response.

I'm not sure I quite understand what you mean here. All I want to do is POST /anything; {METHOD} /anything/:anything. I explicitly said I was not suggesting any creation or tracking of the 'resources' created, and that that would be "clearly out of [httpbin]'s scope".

Apologies if I misunderstood your comments. Didn't mean to offend.
As for a concrete use-case. I found this tool the other day when I wanted to run a few tests on an HTTP client library.. to make sure that it was sending the requests properly.
The default path was chosen to be "/anything". The issue described here is the reason that each test request is configured to override this value (with a specific ":verb"). It would've been preferable to not do so for 2 reasons:

less configuration
the response would repeat the requested method, which would be nice to include in the output log file. The workaround was to embed this information in the data stream, so it would appear in the log.

again: additional configuration
doesn't work for the /get route

workaround: embedded in querystring
again: additional configuration


Update gpg-suite to 2017.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Duplicate of https://github.com/caskroom/homebrew-cask/pull/41702
Thanks for the PR nonetheless.


Service: Imaginary

Also see #24
I do found some issues with this tho, following the instructions in the readme (and using the install.sh) I don't seem to have pecl available. I followed https://stackoverflow.com/questions/54521005/pecl-command-not-found-on-macbook to get this working, not sure how we can manage that in the install script.

Closing some old issues that don't seem relevant anymore, please re-open if you still encounter this issue.


UX Researcher: Research user roles and team needs

Looking to take a deeper dive into understanding what user roles and permissions need to be in place, but also to understand the dynamics of teams working together and how that might affect the CMS in the future.
Feature: Research user roles and team needs
  As a UX Researcher,
  I want to interview the users of the WOX CMS about user roles, managing users, and working in teams
  So that the appropriate user roles and permissions exist
    And to ensure that the software is conducive to teams

  Scenario: Define users current role
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about their current role
     Then I should understand all the roles we may need for WOX

  Scenario: Define users permissions (adding, editing, requesting access)
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about permissions and access
     Then I should be able to match roles with permission
       And have an understanding of user expectations around accessing content

  Scenario: Discuss current team structure and role
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about their role in the team they work in
     Then I should be able to understand team roles and structure
       And how it may differ from the roles and responsibilities within the CMS

  Scenario: Discuss team communication around content
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about working in teams
     Then I should be able to understand how the team communicates around content
       And the role the CMS may have in providing ways to communicate within the team

  Scenario: Discuss visibility of what they are working on
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about visibility of the content they work on
     Then I should be able to understand how team members make sure other members know what they are working on
       And what visibility may be needed to certain team members to make sure they are aware of what everyone is working on

Research user roles and team needs

[ ] Define users current role
[ ] Define users permissions
[ ] Discuss current team structure and role
[ ] Discuss team communication around content
[ ] Discuss visibility of what they are working on

Notes:
Managing users (feature)

User roles
User permissions
Adding new users
Editing existing users
Requesting access

Working with teams (feature)

What is everyone working on/content team view
Team specific roles? (Differ from system access roles)
Team communication? (Could potentially be related to user workflows)

@mkiang3051 - Wrote out the features and scenarios for this story. Please take a look and let me know if you have any thoughts. Will work on #296 next.

@ineedsubstance
This looks good! 👍
I was a little confused at first because the title had changed to be a bit more broad, so I thought this was going to be a story for ALL user needs. But I understand it's for the basic user needs and team structure, and then we still have the other stories for more particular workflows.

@mkiang3051 - I think you make a good point that it was too open. I updated the title to hopefully match the story better. Thanks.

Sweet! Awesome possum.

Blocked by #321

Made this an epic! Moving the features and scenarios into the comments here so we can use it for reference (I don't want to lose the good work @ineedsubstance put in on these stories as they are valuable and provide more clarity around the research topic).
Feature: Research user roles and team needs
  As a UX Researcher
  I want to interview the users of the WOX CMS about user roles, managing users, and working in teams
  So that the appropriate user roles and permissions exist
    And to ensure that the software is conducive to teams

  Scenario: Define users current role
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about their current role
     Then I should understand all the roles we may need for WOX

  Scenario: Define users permissions (adding, editing, requesting access)
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about permissions and access
     Then I should be able to match roles with permission
       And have an understanding of user expectations around accessing content

  Scenario: Discuss current team structure and role
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about their role in the team they work in
     Then I should be able to understand team roles and structure
       And how it may differ from the roles and responsibilities within the CMS

  Scenario: Discuss team communication around content
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about working in teams
     Then I should be able to understand how the team communicates around content
       And the role the CMS may have in providing ways to communicate within the team

  Scenario: Discuss visibility of what they are working on
    Given that, we are building a CMS for WOX for content entry
       And that I want to understand and document our user's needs
     When I go to interview our WOX CMS users about visibility of the content they work on
     Then I should be able to understand how team members make sure other members know what they are working on
       And what visibility may be needed to certain team members to make sure they are aware of what everyone is working on

Research user roles and team needs

[ ] Define users current role
[ ] Define users permissions
[ ] Discuss current team structure and role
[ ] Discuss team communication around content
[ ] Discuss visibility of what they are working on

@mkiang3051 -all the stories in the EPIC are closed, can we close this EPIC?

@ineedsubstance I am cool with closing this epic - we can reopen in the future if we need, but at this moment I don't foresee doing more research with our internal users to get more data.

Right, doesn't seem like this research will continue.


[Feature] Sortable Environments

Environments should be sortable just like the items in the request sidebar.

Are you thinking of a click and drag?

@FlyteWizard possibly in the dropdown, but it might be easier to implement in the "Manage Environments" sidebar. However, if we got dropdown sorting working then it could also be used to sort workspaces in the main dropdown.

Awesome! I'll start working on this.

[ ] "Manage Environments" Sidebar
[ ] Dropdowm Menu

Click and drag reordering would be really nice.

@jroyal If you would like to work on this, it is now free for the taking :D

@FlyteWizard I didn't realize you edited your previous message (GitHub doesn't notify of edits apparently). I can also take this on if I can find some time.

I'll give this a shot as I get some free time this weekend and upcoming week.

Awesome, @jroyal! Let me know if you need any guidance 😄

Omg thx so much @gschier, this has been killing me now I got dozens of envs

😄


Cost for AWS PrivateLink VPC Endpoint not per AZ

Thank you first of all for this awesome tool!
We are testing it and we noticed one potential bug:
When counting the PrivateLink VPC Endpoint it does not include the amount of AZs this endpoint would be instantiated.
So it shows me it will cost 730 hours for my endpoint with the pricing of https://aws.amazon.com/privatelink/pricing/.
However, it is instantiated in two AZs and hence is costing me 2*730h of usage.
Any idea how I can reflect this in infracost?
It was first originally introduced with #264.
Many thanks in advance!

@icereed thank you for reporting the issue! I'm struggling to find a way to detect the number of AZs for the vpc_endpoint from https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/vpc_endpoint, any ideas? Would we be able to tell if we follow the service_name link, fetch that resource's AZ, then count the total number of unique AZs that refer to the vpc_endpoint?
@robh007 cc'in you in case you have any ideas too.

@icereed thank you for reporting the issue! I'm struggling to find a way to detect the number of AZs for the vpc_endpoint from https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/vpc_endpoint, any ideas? Would we be able to tell if we follow the service_name link, fetch that resource's AZ, then count the total number of unique AZs that refer to the vpc_endpoint?
@robh007 cc'in you in case you have any ideas too.

Yea it needs to count the subnets that the resource is deployed into. It's not AZ specific, albeit you'll more than likely deploy different subnets into separate AZs. I can take a look later.

Yea it needs to count the subnets that the resource is deployed into. It's not AZ specific, albeit you'll more than likely deploy different subnets into separate AZs. I can take a look later.

Thanks @robh007! I've assigned it to you in case you have time to dig deeper, please let us know if we need to make changes to enable the lookups.

Thanks @robh007! I've assigned it to you in case you have time to dig deeper, please let us know if we need to make changes to enable the lookups.

This will ship in v0.7.13, this should be released on Sunday.

This will ship in v0.7.13, this should be released on Sunday.

@icereed I think I saw a comment on the PR mentioning you'd prefer a patch release so you can test it?
Here you are https://github.com/infracost/infracost/releases/tag/v0.7.13 :)
BTW, if your project has a lot of resources, it would be good to hear your feedback on how long it took for infracost to run (my email is<EMAIL_ADDRESS>if you prefer that).

@icereed I think I saw a comment on the PR mentioning you'd prefer a patch release so you can test it?
Here you are https://github.com/infracost/infracost/releases/tag/v0.7.13 :)
BTW, if your project has a lot of resources, it would be good to hear your feedback on how long it took for infracost to run (my email is<EMAIL_ADDRESS>if you prefer that).

I will immediately test it. We are still in the evaluation phase of this tooling. In the end we plan to scan resources of over 50 AWS accounts. But this will take some months to integrate for us. For this current fix I may be able to deliver feedback in a few hours.

I will immediately test it. We are still in the evaluation phase of this tooling. In the end we plan to scan resources of over 50 AWS accounts. But this will take some months to integrate for us. For this current fix I may be able to deliver feedback in a few hours.

@icereed thank you! That's awesome - happy to help with any issues.

@icereed thank you! That's awesome - happy to help with any issues.

I can confirm it works now as expected 🎉

I can confirm it works now as expected 🎉

@icereed great! Thanks for the quick turnaround.

@icereed great! Thanks for the quick turnaround.

In the end we plan to scan resources of over 50 AWS accounts.

@icereed just FYI that in v0.7.15 (that we just released), users with big projects should see infracost run much faster as we've parallelized the Cloud Pricing API calls. Would be good to know if that makes a difference for you.

In the end we plan to scan resources of over 50 AWS accounts.

@icereed just FYI that in v0.7.15 (that we just released), users with big projects should see infracost run much faster as we've parallelized the Cloud Pricing API calls. Would be good to know if that makes a difference for you.


how to sync audio out?

Currently, I have your library caged inside some timings so I can detect a buffer overflow.  I have to wait until a problem happens before I can take action.  If I had visibility into the circular buffer then I could fill it up to-the-byte without much effort at all.  Time complexity could be managed perfectly and Ruby is plenty capable of sub-millisecond accuracy with the right interface.  The unreliable blocking detection I'm currently using is very likely to cause compatibility problems outside my laptop and I'm not sure how to make a solution that works for everyone unless I can manage the buffer myself.
What I need is the opposite of #dropped_frame.  You can see what I'm doing now on my project:
https://github.com/ham21/radio/blob/master/lib/radio/signals/coreaudio.rb#L111-120

Sorry for late reply.
I'm not confident to grasp your requirement well.
Maybe you mean you should call CoreAudio::OutputBuffer#<< without blocking when ring buffer is full, right?
If I misunderstand, the followings are just a page filler.
I think there are several approaches about this issue.

Add a method like OutBuffer#space to tell frame size you can append without blocking.

Add some `mode' of OutBuffer to you can call non-blocking version of OutBuffer#<< or add another method.
It should return size of stored frame, 0 (or nil) when there's no space in ring buffer.

Callback model. You can register callback Proc object to be called when OutBuffer has space to append.
For this strategy, I have to implement inner thread to handle callbacks.
I think this is most modern and smart way, and most difficult to implement.

Are these fit for your requirement?

You got it!  #1 is the supremely best option.  Your circular buffer interface also needs to report a similar value.
I'm going to demo my DSP technology with your gem to the Ruby user group tomorrow!

I got it. I'll add CoreAudio::AudioBuffer#space method soon.
But I still wonder if you understand correctly what OutputBuffer#dropped_frame means.
dropped_frame is incremented when ring buffer is empty when CoreAudio's callback is invoked,
not when OutputBuffer#<< call blocked. When OutputBuffer#<< blocked, it wait CoreAudio's callback
consume audio data from ring buffer, and it ensure all passed data are passed to buffer.
Why don't you increase buffer size of OutputBuffer same as InputBuffer?

Everything I'm doing right now is because I don't have #space.  My buffer sizes and how I use #dropped_frame is all workaround for not knowing the status of the ring buffer.  I can't really defend the choices except to say it gives me results I need.  But #space will give me the results I want.  Thanks for checking into this.

I see.
I've committed to add AudioBuffer#space method and push gem package to rubygems.org just now.
Please check it out.
In fact, I found a trivial bug about treatment of ring buffer and OutputBuffer use (buffer_size - 1) frames.
So device.output_buffer(1024).space returns 1023.
I'm ashamed... I'll fix it in the near future.
Thank you for your request! I am appreciated much.

This works great!  Seeing it work, I am still convinced it was the best fix.  Thanks!


abcm2ps: 8.13.21 -> 8.13.22

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/abcm2ps/versions.
These checks were done:

built on NixOS
/nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22/bin/abcm2ps passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 8.13.22 with grep in /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22
directory tree listing: https://gist.github.com/5a223ca34ee0df856ba6919ac9a3f929
du listing: https://gist.github.com/8a98226e349bfc9d005c36768be801c8

meta.description for abcm2ps is: '"abcm2ps is a command line program which converts ABC to music sheet in PostScript or SVG format"'.
cc @dotlambda for testing.

Success on x86_64-linux (full log)
Attempted: abcm2ps
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
moving /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22/doc to /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22/share/doc
shrinking RPATHs of ELF executables and libraries in /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22
shrinking /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22/bin/abcm2ps
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22/bin
patching script interpreter paths in /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22
checking for references to /build in /nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22...
/nix/store/p5sx9sz4nxcijlz57dmys7gxhwqglmhg-abcm2ps-8.13.22

Success on x86_64-darwin (full log)
Attempted: abcm2ps
Partial log (click to expand)
for f in ./*.abc ./*.eps ./*.txt; do \
        install -m 644 $f /nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22/doc/abcm2ps; \
done
glibPreFixupPhase
post-installation fixup
moving /nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22/doc to /nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22/share/doc
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22/bin
patching script interpreter paths in /nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22
/nix/store/dblzxz20pywg3ch032x315kbfcgy5vvj-abcm2ps-8.13.22

Success on aarch64-linux (full log)
Attempted: abcm2ps
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
moving /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22/doc to /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22/share/doc
shrinking RPATHs of ELF executables and libraries in /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22
shrinking /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22/bin/abcm2ps
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22/bin
patching script interpreter paths in /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22
checking for references to /build in /nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22...
/nix/store/qjai0khrlrd3l4inv3r7ng8r4h9fmqqq-abcm2ps-8.13.22


MAPREDUCE-7279. Adds RM host name to history server web page

2028

 Quality gate: Failed
1 Issues

 1 Code issues

Details

 Quality gate: Passed
 Everything looks good
Details


Add RSA-OAEP-256 support

Adds support for RSA-OAEP-256 algorithm. Tested in RSpec and also with our company's testing RSA-OAEP-256 key pair.

Unfortunately, this is not a valid implementation. It seems that Ruby OpenSSL does not currently support changing the hash function for OAEP to SHA-256. We could use this dependency to get around that but would want input on that first.


Markdown snippets are not accessible in .ipynb notebooks

Type: Bug
Create a snippet for markdown and try to import it in markdown (in notebook).
VS Code version: Code 1.95.0 (912bb683695358a54ae0c670461738984cbb5b95, 2024-10-28T20:16:24.561Z)
OS version: Linux x64 5.15.0-124-generic
Modes:

Are you in a markdown cell? I see my custom markdown snippets showing up just fine. You have to manually trigger them using ctrl+space since markdown doesn't have quick suggestions enabled by default

Tried it and with Ctrl+Space, the snippets appeared, but till now i was trying to get it by quick suggestions and it worked only for first time after starting or Reloading VSCode. Is there some way to enable quick suggestions?

Thank You


Added option page Rust->General

Added option page Rust->General with settings: Path to racer.exe and path to source of rust
fix #111

Nice, there are some small things (both text labels have text "Path to Racer.exe"), but I'll take fix them in next commit or two.


replace some c headers by std c++ headers

I was compiling acpp on some tight nixos configuration, which resulted in some c headers not being accessible even though c++ headers were working. This highlighted several c std includes that could be replaced by their c++ counterpart.
This PR does exactly that.

I think for the bitcode libraries, we deliberately decided that we didn't want to use the C++ headers. C++ standard library headers may pull in some system configuration files which may fail to compile since effectively in the bitcode headers. We are kind of in a cross-compilation scenarion.
For the host bitcode libraries this is somewhat different, but it could still be argued that we should align it with the others. @fodinabor what is your opinion?
But good catch with the group functions header! :)

For host, I don't see any strong reason for one or the other at the moment... using these C++ headers should be okay, as in most scenarios, they're mostly wrappers/pulling the math/io functions into the std namespace.


Enable custom chef-client config for delivery-cmd

We are having issues using this template since it hard codes the Chef Config path to /etc/chef/client.rb. I am just trying to provide a mechanism to enable us to easily customise this. Let me know if you have any other ideas :).

Hey @mjohno .. Thank you for your PR. I am going to take a look at this. Sorry for the late response.


Detect and shut down stuck server pods

This code looks for stuck pods, which is detects by comparing the current time with any deleted time stamp, allowing for the specified grace period. If any pods managed by the operator is terminated, but are still present after the grace period, the operator will force-delete them and run a new make-right on their domain(s).
Note: we need to do some code cleanup on the async calls. Adding new parameters to be checked by unit tests is increasingly difficult. It is time to introduce an object rather than passing in a list (see Clean Code, chapter 3, regarding function parameters).

@russgold, I've tested a build from your branch and it does detect and delete a stuck pod. Can you please add an INFO level message indicating that the operator found and is deleting such a pod? Presently, I only see a message when the operator subsequently creates the replacement pod.

Per discussion, merging this now.


Jest 26 fixes

Why was this change made?
Upgrade testing libraries.
How was this change tested?
Unit and feature tests.
Which documentation and/or configurations were updated?
NA

👏

Looks like a rebase is needed to fix a conflict.


StreamSyncTensor implicit pipelining for TorchRec [Prototype]

Provide implicit pipelining in TorchRec by using streams. StreamSyncTensor will continue on its current stream if all the incoming tensors are StreamSync tensors on that Stream.
Otherwise it will sync to the main stream and continue.

Copying over comments from diff from @ezyang
I'm thinking about whether or not we can implement this class without use of record_stream (which can lead to suboptimal memory usage in the CUDA caching allocator.) The key ingredient is that we have to track uses of the tensor from other streams and ensure we don't deallocate our tensor before those uses are done. If we assume that the internal elem tensor of a StreamSyncTensor is private (no access that is not mediated through our subclass), it seems to me that we could have 100% accurate information on stream usage. We could then generate an event for each external stream usage representing events that must be complete before deallocation. We would then need to extend the CUDA caching allocator with an API that allows the user to add its own CUDA events instead of forcing a fresh event to be created at deallocation time:
  void insert_events(Block* block) {
    int prev_device = 0;
    C10_CUDA_CHECK(c10::cuda::GetDevice(&prev_device));

    stream_set streams(std::move(block->stream_uses));
    AT_ASSERT(block->stream_uses.empty());
    for (auto& stream : streams) {
      C10_CUDA_CHECK(c10::cuda::SetDevice(stream.device_index()));

      EventPool::Event event =
          create_event_internal(static_cast<int>(stream.device_index()));
      C10_CUDA_CHECK(cudaEventRecord(*event, stream.stream()));

      block->event_count++;
      cuda_events[stream].emplace_back(std::move(event), block);
    }

    C10_CUDA_CHECK(c10::cuda::MaybeSetDevice(prev_device));
  }
This would allow for more prompt deallocation, as the amount of time we have to wait would be solely until the known uses are complete, rather than whatever the state of the main stream was at the time of deallocation.

Ah, so this is a policy decision. I suppose it's not unreasonable to propagate these policy decisions via data-dependent values (e.g., similar to requires_grad), but I would ask if it makes more sense to have a context manager control the policy decision instead. For example, you could imagine:
with no_sync_to_main():
    x = all2all(...)
    y = post_all2all(x)
and this would result in recordStream calls on all the tensors post_all2all accessed that weren't stream tensors.

I do wonder if it is good UX to implicitly setup these waits. You could imagine that what if the SST is far ahead of the main stream, then this implicit wait results in a big bubble. But the user side code change could have been quite innocuous; just happening to add a dependency on some tensor result. I suppose this is what you were asking me about having lots of streams; there's no way to save you if you put everything on the same stream, but if you had a lot of streams you could potentially perform this wait on yet another stream, and let work that didn't depend on this particular operation continue to go ahead.
In any case, we should consider whether or not it would be better to force an explicit wait. Something like stream_sync_tensor.wait_on(plain_tensor) which makes it clear that we're waiting on plain tensor on the side stream in user code. I don't know how much this goops up user code in your example.

cc @ngimel