different result between raw metric and metric of record rules

when I used metric generated of record rules and raw metric, there is a gap, about 10%.
this is my record rules:
rules:
- record: sum:instance_cpu_quota:cell:product:marker:functional_marker
expr: sum(instance_cpu_quota)by(cell,product,marker,functional_marker)
- record: sum:instance_mem_quota:cell:product:marker:functional_marker
expr: sum(instance_mem_quota)by(cell,product,marker,functional_marker)
- record: sum:instance_mem_quota:cell:product:marker:functional_marker
expr: sum(instance_mem_quota_bytes)by(cell,product,marker,functional_marker)
- record: sum:instance_ssd_quota:cell:product:marker:functional_marker
expr: sum(instance_ssd_quota_bytes)by(cell,product,marker,functional_marker)
- record: sum:instance_hdd_quota:cell:product:marker:functional_marker
expr: sum(instance_hdd_quota_bytes)by(cell,product,marker,functional_marker)
- record: sum:instance_cpu_used:cell:product:marker:functional_marker
expr: sum(instance_cpu_used)by(cell,product,marker,functional_marker)
- record: sum:instance_mem_used:cell:product:marker:functional_marker
expr: sum(instance_mem_anon_bytes)by(cell,product,marker,functional_marker)

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


global time series failed with index out of bounds

Hi @xylar ,
I'm Jianfeng Li from PNNL. I am running zppy for an 11-year coupled simulation. However, I encountered the below error for the global time series diagnostic (other diagnostics ran successfully). Does anyone have any suggestions about it? Is it because the simulation is too short (only 11 years)? Thanks.
Setting reference time units to days since 1986-01-01 00:00:00
Traceback (most recent call last):
File "/global/common/software/e3sm/anaconda_envs/base/envs/e3sm_unified_1.8.1_pm-cpu/bin/cdscan", line 1843, in
main(sys.argv)
File "/global/common/software/e3sm/anaconda_envs/base/envs/e3sm_unified_1.8.1_pm-cpu/bin/cdscan", line 1309, in main
vartime[0],
File "/global/common/software/e3sm/anaconda_envs/base/envs/e3sm_unified_1.8.1_pm-cpu/lib/python3.10/site-packages/cdms2/axis.py", line 2212, in getitem
raise IndexError('index out of bounds')
IndexError: index out of bounds
zppy_config_for_B_compset_v2_1_LR_historical.cfg.txt

@forsyth2 or @golaz, can you take a look at this.
@jli628, I don't have much expertise with zppy.

@jli628 I'm running my own version of your cfg, but it usually takes a while to get compute nodes on Perlmutter. Can you open up permissions on /pscratch/sd/j/jli628/E3SMv3atm/zppy_climo_B_compset, so I can look at your output files?

@forsyth2 Thank you for testing it. Perlmutter was unstable yesterday evening. I changed the permission of the whole directory of /pscratch/sd/j/jli628/E3SMv3atm. Could you please try it again? Thank you for your time and help.

@forsyth2 I have another question regarding the cfg variables. For the below variables, can I set ref_start_yr=1986 and ref_final_yr=1996 to match the model simulation period? Will it produce the same results as the current setting? Thank you!
[e3sm_diags]
active = True
years = "1986:1996:11",
ts_num_years = 11
ref_start_yr = 1979
ref_final_yr = 2016
walltime = "08:00:00"
qos = "regular"

I changed the permission of the whole directory of /pscratch/sd/j/jli628/E3SMv3atm.

I'm looking at my own completed run and the logs in your directory. The only thing I can think of is that it doesn't like the 11-year increment, but I was fairly certain that was resolved a while ago (#208). I also tried running a version of your cfg on 10 years but the ts task failed. I'll see if I can find anything else.

can I set ref_start_yr=1986 and ref_final_yr=1996 to match the model simulation period?

In many of the diagnostics sets for E3SM diags, the top panel is test, the middle panel is ref, and the bottom panel is diff. I believe you should be able to set the ref parameters accordingly based on what data is available for whatever you have chosen as ref (in your case /global/cfs/cdirs/e3sm/diagnostics/observations/Atm/climatology/).

Thank you, @forsyth2 , for the explanation. I understand the meanings of ref_start_yr and ref_final_yr now.

$ cd /pscratch/sd/j/jli628/E3SMv3atm/zppy_climo_B_compset/post/scripts

# Check for errors
$ grep -v "OK" *status
global_time_series_1986-1996.status:ERROR (3)

$ emacs global_time_series_1986-1996.bash
# Looking in this file, `ERROR (3)` is the exit status if `cdscan -x glb.xml mpaso.glb*.nc` fails in `${case_dir}/post/ocn/glb/ts/monthly/${ts_num_years}yr`

$ cd ../ocn/glb/ts/monthly/11yr/
$ ls
mpaso.glb.198601-199612.nc

# Load Unified
$ source /global/common/software/e3sm/anaconda_envs/load_latest_e3sm_unified_pm-cpu.sh

# Look at contents
$ ncdump -h mpaso.glb.198601-199612.nc
netcdf mpaso.glb.198601-199612 {
dimensions:
	time = UNLIMITED ; // (0 currently)
	nbnd = 2 ;
variables:
	double time(time) ;
		time:long_name = "time" ;
		time:units = "days since 1986-01-01 00:00:00" ;
		time:calendar = "noleap" ;
		time:bounds = "time_bnds" ;
	double time_bnds(time, nbnd) ;
		time_bnds:long_name = "time interval endpoints" ;
	double ohc(time) ;
		ohc:long_name = "total ocean heat content" ;
		ohc:units = "J" ;
	double volume(time) ;
		volume:long_name = "sum of the volumeCell variable over the full domain, used to normalize global statistics" ;
		volume:units = "m^3" ;
}

Notice time = UNLIMITED ; // (0 currently). This error also occurred in #391: https://github.com/E3SM-Project/zppy/discussions/391#discussioncomment-4719545. This could be because short-term archiving wasn't done first.


[BUG] connection with environment configuration doesn't work, with custom Provider instance it does

Laravel Version: 5.7.*
Adldap2-Laravel Version: ^4.0
PHP Version: 7.2
LDAP Type: OpenLDAP

Description:
When creating a custom instance of \Adldap\Connections\Provider I'm able to connect to the OpenLDAP server. Using the environment variables with the same configuration doesn't work and I receive Invalid DN syntax for all attempts. openldap within the controllers is a docker container with the same hostname which can be accessed from my laravel docker container.
Not working example
.env
ADLDAP_ACCOUNT_PREFIX=""
ADLDAP_ACCOUNT_SUFFIX=""
ADLDAP_CONTROLLERS="openldap"
ADLDAP_PORT=389
ADLDAP_TIMEOUT=5
ADLDAP_BASEDN="dc=company,dc=com"
ADLDAP_ADMIN_ACCOUNT_PREFIX=""
ADLDAP_ADMIN_ACCOUNT_SUFFIX=""
ADLDAP_ADMIN_USERNAME="cn=admin,dc=company,dc=com"
ADLDAP_ADMIN_PASSWORD="secret"
ADLDAP_USE_SSL=false
ADLDAP_USE_TLS=false
ADLDAP_FOLLOW_REFERRALS=false
ADLDAP_ELOQUENT_USERNAME=username

ADLDAP_PASSWORD_SYNC=true
ADLDAP_LOGIN_FALLBACK=true

AuthController.php
public function login(Request $request, AdldapInterface $ldap)
{
    // does not work - "Invalid DN syntax" error appears
    dd($ldap->search()->all());
}

Creating a custom instance of the Provider - working example
AuthController.php
public function login(Request $request)
{
        $config = [
            // Mandatory Configuration Options
            'domain_controllers'    => ['openldap'],
            'base_dn'               => 'dc=company,dc=com',
            'admin_username'        => 'cn=admin,dc=company,dc=com',
            'admin_password'        => 'secret',

            // Optional Configuration Options
            'account_prefix'        => '',
            'account_suffix'        => '',
            'admin_account_suffix'  => '',
            'port'                  => 389,
            'follow_referrals'      => false,
            'use_ssl'               => false,
            'use_tls'               => false,
        ];

        $schema = new OpenLDAP();
        // this works
        $provider = new \Adldap\Connections\Provider($config, null, $schema);
        // this also works
        $provider->auth()->attempt('cn=mySecondUserIveCreated,dc=company,dc=com', 'secret');
        // and yes, this also works
        dd($provider->search()->all());
}

I figured out that the default provider always contains the admin username "admin". Prefixes/Postfixes are also not set at all.
public function login(Request $request, AdldapInterface $ldap)
{
    // does not work - "Invalid DN syntax" error appears:
    var_dump($ldap->search()->all());
    // does work:
    var_dump($ldap->connect('default','cn=admin,dc=company,dc=com', 'secret')->search()->all());
}

Hi @renepardon,
In your adldap.php config, do you have auto_connect set to true or false?

Hi @stevebauman
auto_connect is set to true.
I'm not sure right now but it may be a caching issues with my docker container. After removing and re-building it, it seems to work. So maybe the bound volume has had a problem. But I will double check this.


SelfLearningModel with basemodels having no predict_proba method

It seems like
self.plattlr.predict_proba
Is called (line 63, within self.predict_proba) before self.plattlr is set on line 78.
This causes models like a generic SVC to fail:
AttributeError: 'SelfLearningModel' object has no attribute 'plattlr'

if you use SVC, you only have to do is use probability=True as an argument,  like SVC(..., probability=True).
see this document: http://scikit-learn.org/stable/modules/generated/sklearn.svm.SVC.html


Bug storing cards (AccessControl.ino)

Step 1: Describe your environment

Arduino IDE version: 1.8.5
MFRC522 Library version: master
Arduino device: independed
MFRC522 device: independed

Step 2: Describe the problem
Affected file(s) or example(s):

AccessControl.ino

Steps to reproduce:
Copied from #389 :
In the code when i = 1 and the count =1 (which means there is a card registered) the '<' makes the condition not to be true at that time. ' <=' resolves the issue.
Observed Results:

storing card id does not work correctly

Other relevant issues/pr
Solved a issue/ introduced current issue #346
Fix of current issue but introduce old issue #389
Next steps
We need a good fix for both issues. Means correct handling of extreme/special values like 0x00 0xff.

What #346 really fixed?
It is not clear.
Could someone explain what was the bug?
It says that fixes an infinite loop, but changing i <= count to i < count will never solve an infinite loop problem.
What about 0x00 0xff? When are they causing issues?
Without further information, looks like it fixed a issue to a very specific scenario to always make the last inserted record unreachble.

For #346 lets ping autor @mads256c .
The example is not from me. So my suggestion is just a suggestion ;).
Yes I think the 255 aka 0xff is the problem.
I found a suspicious line of code:
https://github.com/miguelbalboa/rfid/blame/master/examples/AccessControl/AccessControl.ino#L373
//////////////////////////////////////// Read an ID from EEPROM //////////////////////////////
void readID( uint8_t number ) {
  uint8_t start = (number * 4 ) + 2;    // Figure out starting position
  for ( uint8_t i = 0; i < 4; i++ ) {     // Loop 4 times to get the 4 Bytes
    storedCard[i] = EEPROM.read(start + i);   // Assign values read from EEPROM to array
  }
}

For 255

uint8_t start = (number * 4 ) + 2;
(255 *4) +2
1020 +2
1022

So start at 1022 and then we read 4 byte -> end at 1026, but we only have 1024 on arduino.
I do not have the time to review and fix the complete example. For me it is just an example. So if you @rafaelmalheiros can fix the code I appreciate that :+1: .

Hi @Rotzbua,
uint8_t can't hold numbers > 255.
So, the result of the (255*4) + 2 = 1022 = 0x3FE will be truncated to 0xFE = 254.
Regards,
Liviu


chore: organize all of the use statements

Summary
No functional changes are introduced in this patch.
I noticed that some of the files were following the community convention of grouping + organizing the use ... statements according to ~ the following:
mod foo;

use std::{foo, ...};

use external_lib::{foo, ...};

use workspace_lib::{foo, ...};

use super::{foo, ...};
use crate::{foo, ...};

while others were not.
This patch organizes all of the use statements so they're all consistent and organized.  IDEs should now be able to auto-arrange and alphabetize any dependencies added in the future.

There's no way to lint for this community convention or auto-format in my editor?
I don't mind adding these blank lines as a one-time thing. It's harmless and agree it's a little nicer for readability. But it's another story to expect new outside contributors to follow a convention that isn't linted for or automatically applied. It also doesn't seem like the kind of thing we can expect reviewers to reliably look for, or a good use of finite review time.

There's no way to lint for this community convention or auto-format in my editor?
I don't mind adding these blank lines as a one-time thing. It's harmless and agree it's a little nicer for readability. But it's another story to expect new outside contributors to follow a convention that isn't linted for or automatically applied. It also doesn't seem like the kind of thing we can expect reviewers to reliably look for, or a good use of finite review time.

I am absolutely not going for nor caring about perfection here.  If they're mostly in order than its easy enough to maintain honestly. Its not something even I might notice or call out in a PR review.
Some 'set and forget' options we could do is:

add cargo fix to the recommended dev toolchain,
add merge_imports = true to the rustfmt.toml and rustfmt should auto-take-care-of-it.

add imports_granularity = "Crate" to the rustfmt.toml and rustfmt should auto-take-care-of-it.

@tuddman Nice! I like my imports pretty, I just want to expend exactly 0 effort getting there.


remove picwriter depedency

15% of gdsfactory module load time is on picwriter
scalene gdsfactory/__init__.py

we need to rewrite the picwriter based components natively into gdsfactory
TODO:

[x] disk.py https://github.com/gdsfactory/gdsfactory/pull/606
[ ] spiral.py
[x] dbr_tapered.py https://github.com/gdsfactory/gdsfactory/pull/601
[x] coupler_full.py  https://github.com/gdsfactory/gdsfactory/pull/600
[x] coupler_adiabatic.py
[ ] grating_coupler_circular.py
[x] cdc.py

the issue with the picwriter components is that it does not use the Path and CrossSection from gdsfactory, so it cannot be extruded with arbitrary cross-sections.
disk.py
spiral.py
dbr_tapered.py
coupler_full.py
coupler_adiabatic.py
grating_coupler_circular.py
cdc.py

Advantages of porting picwriter components:

make gdsfactory module faster to load
components can have arbitrary cross-sections

@joamatab,
I tried moving the import inside the functions, but that doesn't seem to make a huge difference. I guess rewriting the picwriter-based components natively would be the better option. Would you like me to take a look into that?
Before moving import statements:

After moving import statements:

Hi Skandan,
Yes,
I think porting the few components that we use would be the best solution

https://github.com/gdsfactory/gdsfactory/pull/600

https://github.com/gdsfactory/gdsfactory/pull/601
We are half way done
Thank you Skandan

2 more to go :)
thank you Skandan!

Down to 9% now :)

Nice,
I think we can skip the circular grating coupler
Once we implement the spiral we can remove picwriter


Polishing README after v2.0 merge

@kalaninja please take a look and merge if looks good.

I like the changes that you've done. You definitely write docs much better than I do.


remove snapshot() path from Container

Only summaries should be used.
There are some places (DDS) where we still use "snapshot" terminology & types - we should change that to use summaries.
As part of it, we should remove / move to r11 driver(s) all GIT specifics like FileMode and remove them from protocol-defintions.

@arinwt - FYI

I'm sure there are duplicates of this item :)
The most challenging part here would be fixing tooling, i.e. replay tool. It's a must to maintain it (tests would fail otherwise). We either need to move it to summaries, or do a conversion somewhere for it to still get snapshot when runtime talks summaries.

Given we have completed some of the work around old snapshots, it may be good to time redefine scope of this ticket. And close it if needed.

Adding @scarlettjlee to this since she is working on related topic - https://github.com/microsoft/FluidFramework/issues/8460

The DDS work is done and I created a separate task for moving git code from protocol-definitions (#9003). So this issue should just be for removing snapshot from Container.
@ssimic2, do you want to keep this open until you do the removal after the deprecation? Otherwise we can close.


NULL pointer dereference in httpGetSubField2()

Hi,
I spotted a bug in http_add_field() (cups/http.c) while fuzzing the HTTP server of CUPS 2.3.0 that causes a NULL pointer dereference in httpGetSubField2().
Look at this snippet from _httpUpdate():
    else if ((field = httpFieldValue(line)) != HTTP_FIELD_UNKNOWN)
    {
      http_add_field(http, field, value, 1);

      if (field == HTTP_FIELD_AUTHENTICATION_INFO)
        httpGetSubField2(http, HTTP_FIELD_AUTHENTICATION_INFO, "nextnonce", http->nextnonce, (int)sizeof(http->nextnonce));
    }

It firstly add the Authentication-Info field and then calls httpGetSubField2().
In this routine at the beginning there is a for loop that iterates over http->field[field] assuming that it is a valid pointer.
  if (!http || !name || !value || valuelen < 2 ||
      field <= HTTP_FIELD_UNKNOWN || field >= HTTP_FIELD_MAX)
    return (NULL);

  end = value + valuelen - 1;

  for (fptr = http->fields[field]; *fptr;) // HERE
  {
   /*
    * Skip leading whitespace...
    */

Now looking at http_add_field() you can see that if valuelen is 0 the routine simply set the first byte of http->_fields[field] to 0 and leave http->fields[field] to NULL.
  valuelen = strlen(value);

  if (!valuelen)
  {
    http->_fields[field][0] = '\0';
    return;
  }

So it's clear that with an empty Authentication-Info field we can crash the application when  httpGetSubField2() is called.
The crashing testcase from the fuzzer is:
GET http://localhost:631/index.html HTTP/1.1
Authentication-Info:
Authentication-Info:   7.0

Best Regards,
Andrea
Edit: I'm on Ubuntu 18.04.2 x86_64

[master c9b65dc4d] Fix NULL pointer dereference in httpGetField2 (Issue #5598)


Pods occasionally get killed (exit code 93) since upgrade to 3.3.6

Checklist

[x] Double-checked my configuration.
[x] Tested using the latest version.
[x] Used the Emissary executor.

Summary
We recently upgraded from 3.2.3 to 3.3.6. Since the upgrade we are occassionally experiencing pods getting killed. We start ~180 pods all executing a python script, which trains and predicts some ML model with some variable arguments. We do this every day. Since the upgrade 1-3 pods get killed with following message:
/argo/staging/script: line 10:    93 Killed                  pyenv exec python script/execute.py "${args[@]}"
However the killed pods are not being deleted immediately. Instead it is being rerun after 75m (active deadline seconds 4500)  with respect to our configured retry policy. The second try has always succeeded for now.
In other occassions the return code might differ. In rare cases we get:
/argo/staging/script: line 10:    97 Killed                  pyenv exec python script/execute.py "${args[@]}"
First of all I am interested in what might cause this kills and what the return codes 93 and 97 mean?
Is it likely to be caused by the upgrade or some other component (k8s, Google Big query, scikit-learn, lightgbm, python version)?
What version are you running?
-> 3.3.6
Diagnostics
The pasted example configuration is not runnable without the python source code, which I am not able to provide. Nevertheless I hope every important information is provided with the partial configuration.
    - name: train-predict-script
      inputs:
        parameters:
          - name: subcategory_partition_id
          - name: config_filename
          - name: cpu_request
            default: 2
          - name: memory_request
            default: 12Gi
          - name: expost
            default: "false"
          - name: job_timeout  # to be passed as activeDeadlineSeconds
            default: 4500  # 1 hour 15 minutes, must be lower than maxDuration of retry strategy
          - name: expost_mode_monthly
            default: "false"
      script:
        image: eu.gcr.io/{{workflow.parameters.projectId}}/psf:{{workflow.parameters.buildId}}
        command: [bash]
        source: |
          args=('-p={{workflow.parameters.projectId}}' \
          '-s={{inputs.parameters.subcategory_partition_id}}' \
          '-f={{inputs.parameters.config_filename}}' \
          '-b={{workflow.parameters.buildId}}')

          [[ {{inputs.parameters.expost}} == 'true' ]] && args+=( '-e' )
          [[ {{inputs.parameters.expost_mode_monthly}} == 'true' ]] && args+=('--expost-mode=monthly')
          echo "arguments: ${args[@]}"

          cd python/psf && ls -la && pyenv exec pip list && pyenv exec python script/execute.py "${args[@]}"
        imagePullPolicy: Always
        resources:
          requests:
            cpu: 2
            memory: 12Gi
      retryStrategy:
        limit: 2
        retryPolicy: "Always"
        backoff:
          duration: "5s"
          factor: 5
          maxDuration: "180m" # two activeDeadlineSeconds terminated runs are possible
      podSpecPatch: '{"activeDeadlineSeconds": {{inputs.parameters.job_timeout}}, "containers":[{"name":"main", "resources":{"requests":{"cpu": "{{inputs.parameters.cpu_request}}", "memory": "{{inputs.parameters.memory_request}}"}}}]}'

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I think your main container is being killed, but the wait container remains running.  I think exit code 93 is a red herring.
Can you try the emissary executor if you’re not using it, if you are can you try the pns executor?
After that, you please try running the :latest version of executor?

We are using the emissary (default) executor in the :latest-version. Today we have been able to have a look in the container logs of one of the failed pods. It seems to be an OOM error, which is not argos fault at all. Nevertheless argo does not terminate the failing pod properly and the pod idles till the active deadline seconds thresh is reached. Besides a meaningfull error code and/or message would be highly appreciated.

Can you attach your controller logs and the full pod yaml?

I attached 3 files:

the log of one failing pod. The pod log logs an OOM error (exit code 137), that does not show up in the argo gui. And the pod does not terminate but runs until it is restarted by configured active deadline seconds/retry policy.
full pod yaml
controller logs with some filters. The filters are:
resource.type="k8s_container"
resource.labels.cluster_name="flash"
resource.labels.namespace_name="argo"
labels.k8s-pod/app="workflow-controller"
-(textPayload=~"warning msg="workflow uses legacy/insecure pod patch") -> legacy rbac patching
-(textPayload=~"warning msg="Non-transient error") -> artifactory not found, but it works
-(textPayload=~"Get workflowtemplates 200")
-(textPayload=~"Enforcing history limit")
-(textPayload=~"info msg="Update leases 200")
-(textPayload=~"info msg="Get leases 200")
-(textPayload=~"info msg="Create events 201")

This appears to be the same root cause as https://github.com/argoproj/argo-workflows/issues/9083, as mentioned by Alex above. In that bug, you can see my comment - I was able to show that the fix [here](this resolves this. Feel free to re-test with v3.4.0rc3 (using instructions on our Release page) or otherwise using latest from master. Or we can just close this.


Add Mud Runner/Snow Runner support

Might work with a LUA file hack.
Download this software to see lua changes
https://en-motionhouse.imweb.me/

🤔  basic suspension data would be a lot of fun on snowrunner.
Would like to help on this if I can, got any "getting started" guide for contributing?

🤔 basic suspension data would be a lot of fun on snowrunner. Would like to help on this if I can, got any "getting started" guide for contributing?

GenericTelemetryProvider.sln is the solution
Using Balsa as an example
Duplicate/rename an existing provider and UI
BalsaTelemetryProvider.cs
and
BalsaUI.cs
So, basically, you decide how you want to get the data in every frame, then call ProcessTransform. Providers run on their own thread separate to the main thread and each ui (dialog) has it's own thread also. (there's thread safety functions for the ui all over the place)
Each provider is based on GenericProviderBase, the main function for all the processing is 'ProcessTransform'
The intention is for the developer to pass a transformation matrix for the frame to ProcessTransform, let the base class control the order of the function calls and then do any provider specific extraction of data from this transform within overridden functions.
So you shouldn't need to change anything inside GenericProviderBase and do everything inside your provider code.
The base class can do most things for you as long as the transform provided is in a left handed coordinate system. (have a look at the Matrix4x4.Identity value).
You can also completely override GenericProviderBase.ProcessTransform and do your own thing as long as you fill out the 'filteredData' structure and call SendFilteredData() at the end of the function.
Put a breakpoint inside GenericProviderBase.ProcessTransform and step into the function calls and it should be pretty obvious what's going on.
Add an instance of the UI to MainForm code (BalsaUI balsaUI;)
Add a button and a click callback to the MainForm ui. (balsaButton_Click)
Duplicate/rename a default config file for the new provider to the Configs subfolder (BalsaConfig.txt these config files hold all of the filter data.)
When you're happy, do a pull request and I can merge.
If you have any more questions ping me on discord Pezza#6155

@PHARTGAMES welp, I spent a few hours trying to figure out how to get the data but I can't figure this out...
I tried CheatEngine and drilled down to try and get useful values like the wheels, suspension or just the player position as a starting point, but I either end up with 4k+ values that are always changing slightly but not enough to figure out what part exaclty, and when I try to freeze values it messes up the physics and the truck goes crazy :p
got any other tools you recommend using or any technique to find the data needed for this?

@mchamplain Sorry it took a while to reply, It will be a pain to do with cheat engine, you can do it with a lua hack
Extract the Media.zip file in the mudrunner folder
find classes/truck.lua
attached is the lua file that MHServicer uses to extract data.
You just need to figure out how to get that data from the lua file into spacemonkey, probably via udp.
truck.txt
l

seems specific to MudRunner/SpinTire...
in SnowRunner there's no media.zip file, only a media folder, but no lua files anywhere, looks like they got rid of lua scripts in that version


Shipping address is not sent to paypal

Hi!
When you fill in a shipping address different from the billing address, it is not used in the Paypal screens anywhere.
On the "Final" Paypal screen, you see your shipping address before clicking to return to the site, and it is actually the Billing Address
Looking into the Checkout Controller Decorator, it looks like there is only one [:address] option, so there doesn't appear to be a way to submit them both to PayPal
Our client is saying this is confusing their customers, and I am desperate for a solution here!
This is not resolved by unchecking the "No Shipping" checkbox in the Admin Payment Options configuration... that option actually causes your Logged-In-Paypal-Account's shipping address to OVERWRITE the Shipping Address used during Spree Checkout.
Thanks for your time & consideration, guys <3

+1


--dirs-first flag does nothing if no SortType is specified

for example: et --dirs-first will not sort directories above files, but if sort type is specified: et --dirs-first -s name it works as expected.
i was able to produce the expected behavior by adding a None enum variant to SortType and adding:
} else if ctx.dirs_first() {
	Order::from((SortType::None, true))
		.comparator()
		.map(|func| current_node.sort_children(func));
}

to this line, but there are probably "cleaner" approaches.
also, i noticed that clippy suggests a bunch of stylistic refactors all over. do you have an opinion against such stylistic approaches?

I do not, but if you plan to open a PR if you could pass on running clippy that'd be awesome. Someone has another PR that's currently waiting to be merged and they ran clippy and I'd be worried about really gnarly merge conflicts that could cause headaches. Thanks for looking into this btw!


Add variable object examples

Start to a variable object summary.

@mcoughlin SO MANY PRS!! which one do we review, #44 or #45?

Ok. Looks fine


Add the ARGOCD_ENV prefix in the helm with args example

Description

The example that shows how avp can be invoked to render with the cm configmap has been updated to:

include the added prefix in env vars (https://argocd-vault-plugin.readthedocs.io/en/stable/config/#argocd-240-environment-variable-prefix)
respect the namespace of the application
Fixes: <! -- link to issue -->

Checklist
Please make sure that your PR fulfills the following requirements:

[] Reviewed the guidelines for contributing to this repository
[ ] The commit message follows the Conventional Commits Guidelines.
[ ] Tests for the changes have been updated
[ ] Are you adding dependencies? If so, please run go mod tidy -compat=1.17 to ensure only the minimum is pulled in.
[ ] Docs have been added / updated
[ ] Optional. My organization is added to USERS.md.

Type of Change

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

Other information

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

Codecov Report

Merging #431 (4a5deb8) into main (a71224e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #431   +/-   ##
=======================================
  Coverage   77.17%   77.17%
=======================================
  Files          23       23
  Lines        1485     1485
=======================================
  Hits         1146     1146
  Misses        252      252
  Partials       87       87

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


typo descedant should be descendant in build-a-survey-form.md

Describe the Issue
the word descendant is misspelled multiple times as "descedant" int the file build-a-survey-form.md
https://forum.freecodecamp.org/t/small-typo-in-freecodecamp/561536/3
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/build-a-survey-form-project/build-a-survey-form
Your code
N/A
Expected behavior
correct the spelling
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
list of what appears to be 2 files that need to be fixed:
hbar1st@LAPTOP-KKSHJUG1:~/fcc/freeCodeCamp/curriculum/challenges/english$ grep -r descedant
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #name should be a descedant of #survey-form.
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #email should be a descedant of #survey-form
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #number should be a descedant of #survey-form.
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #name-label should be a descedant of #survey-form.
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #email-label should be a descedan
of #survey-form.
01-responsive-web-design/responsive-web-design-projects/build-a-survey-form.md:Your #number-label should be a descedant of #survey-form.
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #name should be a descedant of #survey-form.
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #email should be a descedant of #survey-form
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #number should be a descedant of #survey-form.
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #name-label should be a descedant of #survey-form.
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #email-label should be a descedan
of #survey-form.
14-responsive-web-design-22/build-a-survey-form-project/build-a-survey-form.md:Your #number-label should be a descedant of #survey-form.

Thanks for opening this issue.
This looks like something that can be fixed by "first-time" code contributors to this repository. Here are the files that you should be looking at to work on a fix:
List of files: see the first comment of the issue.
Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or our forum if you need help contributing; our moderators will guide you through this.
Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.

Hi @hbar1st, I am first time contributing to Hacktoberfest. could assign this issue to me.

We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue.
If you would like to fix this issue, please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.


HLS playback crash on Android 6 and below.

Version
Media3 main branch
More version details
Version 1.3.1
The crash is caused by Introducing a Java 8 java.util.Stream api here: https://github.com/androidx/media/pull/1054/files#diff-3895326de0c5e6ffdc885f40d9160501824ffaa155f4fb6593db8d648a83e1b0R982
You can reproduce it by playing the HLS streams on the demo app on an Android 6 device.
Devices that reproduce the issue
Any Android OS 6 or below.
Devices that do not reproduce the issue
Android 7 and up.
Reproducible in the demo app?
Yes
Reproduction steps
You can reproduce it by playing the HLS streams on the demo app with ExoPlayer 1.3.1 on an Android 6 device.
Expected result
No crash
Actual result
Crash
2024-05-29 09:57:16.877 22666-22786 AndroidRuntime          androidx.media3.demo.main            E  FATAL EXCEPTION: ExoPlayer:Playback                                                                                Process: androidx.media3.demo.main, PID: 22666                                                                                java.lang.NoSuchMethodError: No interface method stream()Ljava/util/stream/Stream; in class Ljava/util/List; or its super classes (declaration of 'java.util.List' appears in /system/framework/core-libart.jar)                                                                                at androidx.media3.common.Format.<init>(Format.java:982)                                                                                at androidx.media3.common.Format.<init>(Format.java:128)
Media
Reproducible on the HLS streams in the demo app (Apple 16x9 basic stream TS)
Bug Report

[X] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Hi @goranch,
We have reverted the mentioned usage of Java 8 above. This change will be included in our next release.


Generate citation strings from citation.cff file as part of build process

@jodeleeuw When you updated the dependencies, including esbuild from 0.15.14 to 0.23.1 in 59ce0b3, did you read the changelogs? There seem to be a lot of breaking changes in esbuild and we need to be confident that they are not breaking for @jspsych/config users (unless you are planning to release a config v4 for any other reason :thinking: ). I didn't get suspicious about this earlier because your commit message didn't make it clear to me that you were updating dependencies :see_no_evil:

@jodeleeuw I double-checked and committed the rollup dependency updates in #3396 – they're all welcome and independent of the .json issue. Re that issue: Because we are using esbuild via the rollup plugin, the final bundling is performed by rollup. esbuild never sees @citation-js/core/package.json, hence can't run its JSON loader on it. Your workaround with the additional Rollup JSON loader is good for now, although the more desirable solution might be to inline the version string right in the citation-js build (like most other packages do too). I'm too busy to contribute there ATM though :confused:

@bjoluc just FYI, we ended up solving this by using @rollup/plugin-replace. We explored the esbuild define, but it required adding some declare const statements at the top of the plugin ts templates and that felt unnecessarily confusing for rookie devs.

Have you thought about declaring the constants globally in a .d.ts file in @jspsych/config, referenced by the exported tsconfigs?

For some reason I thought this kind of strategy wouldn't work, but can't reconstruct my reasoning now. We can explore it!

Also, what about replacing a single CITATIONS object? That would allow to add other formats at a later point without changing all the plugins, while further reducing boilerplate.

This was something that crossed my mind. I was thinking that the current boilerplate makes it easier for someone building with the JS template to modify because they won't have to know the syntax, but then again we could just have a different template for plugins that aren't going to go through the build process. So that does seem better.

@cherriechang I'd like to get this merged soon. We need some docs still I think.
@jadeddelta we could also include the CFF files in the v8 update for the contrib repo?

Hi all, I just received the notification mail and had a quick look at the diff. I think I would much prefer replacing a __CITATIONS__ reference (with a global TS definition) over replacing a '__CITATIONS__' string as IMO with the former variant the intention is clearer and the typing more consistent. Just posting the wish here, guessing that you have most likely had good reasons to rule that path out...


Update FAQ 2.19 #rat_tan_hotline: Mention rapid PCR tests

Internal Tracking ID: EXPOSUREAPP-12118

@Ein-Tim Good point at this stage. We will remove last sentence for the time being. Thanks for the review.

ok, lets hope it will not result in a merge conflict....

@dsarkar I think it should not as I identically copied the content from the 2.19 branch.
I opened #2535.


Ignore not existing file on travis-env-load

it's not a problem since yq get executed in a subshell so main script exit gracefully anyway


Loading custom datasets

Hello everybody :)
I am working with a custom pipeline for performing link prediction in a graph. I construct this graph through processing of csv data but in the end I obtain an object of class pytorch_geometric.Dataset (the same class as the default ones used in this repo like cora, protein, email...).
I would like to know what part of the code of this repo should I modify to load my custom dataset object for performing link prediction.
Thank you very much!

You may need to create a folder of your custom dataset in the data/ folder similar to the CiteSeer folder under data/.
Then pass the name of your custom data folder as an argument in the cmd line when calling the train.py script.
For example: if I save my custom dataset files under data/ as: data/Custom-Data, then I can point to this new dataset folder by calling:
python train.py --dataset Custom-Data

I also started working on loading custom datasets, however I don't know how I could prepare my data into the files in the format acoording to data/CiteSeer folder. Moreover if I use the python train.py --dataset CiteSeer, I ended up with the error
Traceback (most recent call last):   File "train.py", line 117, in <module>     loss = deal.default_loss(inputs, labels, data, thetas=theta_list, train_num=int(X_train.shape[0] *args.train_ratio)*2)   File "/Users/liuqi7/repos/2021_complete_the_look_hackathon/deal/model.py", line 466, in default_loss     dists = data.dists[nodes[:,0],nodes[:,1]]  TypeError: 'NoneType' object is not subscriptable

@lajd I looked at your code. Looks like you have used datasets available in the pytorch geometric datasets. I want to run in on my data but I don't know how to prepare the data into the format that is used in the code for example those numpy zip files and sparse matrix. I am facing problems in understanding what those files represent and on what basis they are made.

@lajd Is it possible that the dists file contains the normalized shortest path distances between each pair of nodes?

@lajd I looked at your code. Looks like you have used datasets available in the pytorch geometric datasets. I want to run it on my data but I don't know how to prepare the data into the format that is used in the code for example those numpy zip files and sparse matrix. I am facing problems in understanding what those files represent and on what basis they are made.

same question. have you found the answer?

@lajd Is it possible that the dists file contains the normalized shortest path distances between each pair of nodes?

Yes, it is the shortest path between each pair of nodes.


Link samples to the migration patterns

Add the migration samples from GitHub to the article

@rukmani-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @rukmani-msft -- This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We'll keep this PR open for review and merge, but in the future, would you make content updates in the private repo? Thanks!

#sign-off

Thanks for this guidance Kristine, can you please help advise if there is a corresponding private repo I need to work on, or if I need to fork a private repro for myself and then submit a PR?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Kristine Toliver<EMAIL_ADDRESS>Sent: Thursday, October 15, 2020 9:48 AM
To: MicrosoftDocs/azure-docs
Cc: Rukmani Gopalan; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Link samples to the migration patterns (#64453)
Hi @rukmani-msfthttps://github.com/rukmani-msft -- This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We'll keep this PR open for review and merge, but in the future, would you make content updates in the private repo? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/64453#issuecomment-709453375, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKLIUEXIXXK35QOMC6Q6R3LSK4RTZANCNFSM4SSDJJ4Q.


Just set max width default in TextView.h to 0.0 instead of negative o…

…ne. Addresses issue #102
@benjaminbojko simple change of the defaults here.

Thanks! Will merge this in


Allow lazy-read of netCDF-4/HDF5 files

At this point, this represents more an aspiration that a plan, but there has been some discussion (see PR #849) of how to enable lazy reads of netCDF-4 file metadata.
Files with a very large amount of metadata take a long time to load because netCDF reads all metadata at file open. For classic files, this doesn't seem to bother people much. But for netCDF-4/HDF5 files, it does. Perhaps this can be explained by the use of netCDF-4/HDF5 for some really complex and large datasets, which end up with tens of thousands of attributes, variables, dimensions, and/or groups. Or perhaps the classic formats, having all their metadata in a block at the beginning of the file, just load faster.
This has already cost us satellite users - the NPP uses netCDF-4, but the follow-on JPSS spacecraft switched to HDF5 without netCDF, due to the slow load times. I was told a similar story about a ESA satellite system by a very active netCDF user in the Netherlands. (Satellite L2 data files generally contain a very large number of attributes, some of which may be reasonably large arrays.)
One idea I suggested is to read each group only as needed. This would be pretty easy to implement I think. It would help where there's lot of groups. @DennisHeimbigner points out that this will not help with files that contain lots of vars. He indicates a known use case with a very large number of vars, all in the root group.
 ;-)
In order to do lazy reads as Dennis suggests I think much of the libsrc4 code would have to be rewritten. (The good news is that with #849 soon to merge, and #856 to follow, the libsrc4 code will be a fair bit smaller than it is now.)
For example, if we open a file and read nothing, and then the user does an nc_inq(), we need to find out how many variables there are. In the current code, we count our list, because we have already read them. In the lazy-read code, we would rsee if there's a way we can get the numbers we need without reading every variable's metadata. That is probably possible in HDF5, but not how the code is currently written.
Handling dimensions in a lazy-read is going to be particularly tricky. They may be in different groups from a variable. So if the user opens a file and does a nc_inq_var() on a var deep in the group structure, we will have to have code smart enough to find all the dimensions in whatever group they are in. All this information is in the HDF5 file, but the code to read it and use it properly remains to be written.

I should note that the metadata read speedup that I already have
still reads all the metadata at one time (i.e., not lazily). It gets
its speedup from pervasive use of hashing. I just put up a PR
for the initial integration of this speed up. Waiting for the netcdf-4
rename stuff to be completed before I do the final step.

Another issue I discovered was that if we do lazy eval, then the various integer
ids (dimid, varid, etc)  will probably change depending on the order in which
the meta-data elements are read. Specifically, any user code that assumes anything
about the ids is potentially going to fail. Issue https://github.com/Unidata/netcdf-c/issues/851 is relevant here.

Maybe an easy way to do this would be a special attribute that contains a table of metadata. So instead of reading each object in a lazy way, we read the table. Then all our data structure code continues to work.
Files that do not have the table of metadata can not be read quickly. But they could add the table and then quick reads would work on the file.
We could store the metadata in various ways, like a file level attribute that lists all the dims in the file, a group-level table that lists all vars in a group, etc. Or we can package it up as one big metadata element (perhaps just store the output of ncdump -h as a char attribute).

Not a bad idea. The problem is that if we have large amounts of metadata, then
that extra attribute itself becomes very large. We would need some very
good encoding method.

Even a very large ncgen -h output, stored as a char array, would read hugely faster than opening all the objects in HDF5 and querying them each. The whole thing would be 1 disk access, instead of tens or hundreds of thousands. Essentially it would be O(1) instead of O(N), right?
The more I think about it, the more I like it. Instead of re-writing all the libsrc4 code, it all will still work just fine.

Would it be at all possible/practical to put the existing ncgen/ncdump parsing code into the library, so that we could literally use the output of ncdump -h?
Or is that just nutty?

That is just nutty :-)
It also occurs to me that we need to allow for non-lazy (bulk) read of the
metadata when the caller is e.g. something like ncdump.

Let us say we have an optional text attribute in the root group with a protected name which contains some compressed, easy to parse representation of the metadata in the file.
Each time any metadata is added or altered, the attribute is updated so it is always correct.
We add a function nc_put_metadata() which does the bulk read of metadata and (re-)creates this attribute.
Then why wouldn't ncdump be able to use the metadata too? Seems like that would be useful and a lot faster.

But this does not solve anything; innstead of reading all the metadata at once
as we do now, we are reading the equivalent in that single attribute. Plus
we have the overhead of parsing it, which is potentially a big cost. This
reminds me of the HDF-EOS solution.

It does remind me of the HDF-EOS thing too, and that is certainly not a compliment to the idea. So perhaps not the way to go.

Before this goes much further, we need to have
a software architecture and design document to
codfy the API and the critical implementation
details. Something in markdown like a cross between
this https://github.com/Unidata/netcdf-c/blob/master/docs/filters.md
and this https://support.hdfgroup.org/HDF5/doc/Advanced/DynamicallyLoadedFilters/HDF5DynamicallyLoadedFilters.pdf

Having worked with this code a bunch, I do see an optimization which would be pretty easy, and could result in significant speed-up of file opens.
In NC_VAR_INFO_T we we add a field atts_read.
When opening the file we don't read any variable atts.
When the user asks for a variable att, we check atts_read. If zero, we read all atts for that variable and set it to 1.
Anyone have any thoughts or opinions on this approach?

Developers, have you considered no attribute cacheing as an alternative strategy to cacheing and lazy reads, in the case of netCDF4?  HDF5 offers efficient attribute access by name and by index number.  It seems to me that a thin wrapper approach for attributes would result in less memory demand and simpler library code, with little impact on performance in most real cases.
This approach may also be valid for other alternative storage formats such as cloud.

An update on this ticket:

Lazy att reads were part of the last release. Atts for a group or var are all read when any information is requested about any atts in that group/var. That is, if there are 10 atts in a var, all 10 will be read into memory if you ask about any of them.

Lazy var reads are more of a problem. I have worked out a faster way to read vars, but am now trying to make it less impactfull on the code and the data format. (That is, I am looking for a lazier and easier way to obtain the same speedup.)

Here's some gprof output for the following code:
    int ncid;
     int i;
     for (i = 0; i < 1000; i++)
     {
        if (nc_open("wrfbdy_d01", NC_WRITE, &ncid)) ERR;
        if (nc_close(ncid)) ERR;
     }

The file used is from a WRF model run, it's got lots of vars and attributes, but it is a real data file which is read a gadjillion times a day, all over the world, by anyone using the WRF model.
(I have cut off the call graph where times dropped below .1 s.)

I am surprised to see how slow it is to close the file.

I am going to close, as the changes discussed here have all be merged.


Bug: Watching video from Week 2 but button shows Week 01.

This happens in every video, I was watching Week-11, it still shows Week-1 selected
The code where this bug is potentially happening is this part in SideBar.jsx, But there is no way to test it, because only one Course is available in the development db


Living GMC Document

I think we would also need to update the Status before merge.

I believe that covers the requested changes plus needed RPIP editor number assignment (went with 29 cuz of a parallel thing)
@xrchz @shfryn please give a thumbs up let's git'r merged.
This is likely to be a somewhat active RPIP with monthly edits or more -- editors will try to keep mostly out of the way here.


[ENH, MRG] Add citation for intracranial JOSS paper

Happy to change the wording if it doesn't fit with MNE style but I think it would be nice to promote the addition of new modules to MNE with additional publications. I tried to be subtle about it but I think it would be nice to advertise a tiny bit.

Failure looks unrelated, couldn't pip install pandas.

Okay to merge or should I try and run the CIs that failed again?

thx @alexrockhill


Fixing problem with links inside of < > brackets in safe mode.

There's a test case that better explains what was going wrong.
Also, a fix for an issue introduced with the logic inversion in commit 35930e0928e19...

Just added another commit - a2377e1 - that fixes an issue where unclosed block-level html elements would stop further processing of markdown. This seems like correct behavior when passing-through html, but not when escaping it. The change turns off html processing when safe mode is set to 'escape'. In that case I think we just want to escape the text and be done with it.
Feedback?

Hey mdrilf,
Thanks for the work on this. However, in the future, could you keep unrelated issues in separate pull requests? Some of the stuff here is a no-brainier, but others I need to look at closer. Everything all grouped together makes that harder to do. I guess what I'm trying to say is that less work for me gets your requests merged faster.
I'll need to play with 425fde1 and compare with other implementations. Babelmark makes this fairly easy.
Regarding 542324b, is that just a documentation edit? Was the actual logic changed previously, Should it not have been?
a2377e1 is an interesting solution I hadn't considered. IIRC, the current behavior is there for historical reasons. We didn't always use ElementTree (which gives us escaping for free). No reason why we can't take advantage of that now. Question though, based on your commit message, is sounds like your patch is just masking a bug in the raw html parser which will still manifest itself when we don't use safe_mode='escape'. If so, we need to fix that. Could you provide more info (preferably in a separate bug report).
I'll probably just merge your commits on the fenced code blocks, but those really should be part of issue #53. At least those bugs will be fixed while waiting for the refactor and the tests will help in doing the refactor as well.

Hi Waylan,
Thanks for all of your hard work on Python-Markdown. We are using it
at https://fiesta.cc
Replies are inline:

Thanks for the work on this. However, in the future, could you keep unrelated issues in separate pull requests? Some of the stuff here is a no-brainier, but others I need to look at closer. Everything all grouped together makes that harder to do. I guess what I'm trying to say is that less work for me gets your requests merged faster.

Yes I will try to - GH was yelling at me for trying to create a second
pull request but maybe I just didn't search around hard enough.

I'll need to play with 425fde1 and compare with other implementations. Babelmark makes this fairly easy.

I think the test case is definitely demonstrating proper behavior
here, but not sure that my fix is correct. I was fumbling around a bit
trying to figure out the best way to accomplish it.

Regarding 542324b, is that just a documentation edit? Was the actual logic changed previously, Should it not have been?

That is a bug fix: commit 35930e0928e19e37f81c906d5d11dfcc1087092b changed

   if netloc != '' or scheme in locless_schemes:

   if netloc == '' or scheme not in locless_schemes:

In order to properly reverse the logic it needs to be an and I
think. There was actually a failing test on my machine (python 2.6)
when I checked out that change.

a2377e1 is an interesting solution I hadn't considered. IIRC, the current behavior is there for historical reasons. We didn't always use ElementTree (which gives us escaping for free). No reason why we can't take advantage of that now. Question though, based on your commit message, is sounds like your patch is just masking a bug in the raw html parser which will still manifest itself when we don't use safe_mode='escape'. If so, we need to fix that. Could you provide more info (preferably in a separate bug report).

I'm not sure that it's masking a bug. The problem is basically that if
I have the following markdown:
<div>
*bold*

In the non-safe-mode case, I'm trying to inline html - so the bold
shouldn't get processed since it's inside of an unclosed . In the
safe-mode case, the  is basically meaningless, so bold should
turn into bold. Does that help explain the problem?

I'll probably just merge your commits on the fenced code blocks, but those really should be part of issue #53. At least those bugs will be fixed while waiting for the refactor and the tests will help in doing the refactor as well.

Yeah sorry for the poor protocol - we needed all of those changes fast
so I took some shortcuts in reporting them back to you. Will try to be
better in the future.

On Wed, Jan 18, 2012 at 4:40 PM, Mike Dirolf  wrote:

Great - sounds like all of the commits are fine except one still needs
discussion, see below:

Regarding 542324b, is that just a documentation edit? Was the actual logic changed previously, Should it not have been?

That is a bug fix: commit 35930e0928e19e37f81c906d5d11dfcc1087092b changed

if netloc != '' or scheme in locless_schemes:

if netloc == '' or scheme not in locless_schemes:

In order to properly reverse the logic it needs to be an and I
think. There was actually a failing test on my machine (python 2.6)
when I checked out that change.

No, there were 2 changes to that line. Both sides of the or were
negated. It is still an or.  Besides, I see no code change in your
commit so  this a documentation edit - an inaccurate one unless I'm
missing something (like your failing test??). I'll need more info
before doing anything with this.

Yeah but this is boolean logic, right? To negate (A or B) we need (!A
and !B) - your change made it (!A or !B). The failing test case wasn't
a new one I wrote, but one of the existing tests - can you try running
them on Python 2.6?

Right. I was thinking I changed (!A or B) to (A or !B). Anyway, your
patch fixes it. Not sure how I missed that.

Okay glad we got that figured out - was worried I had done something
wrong there.
On Wed, Jan 18, 2012 at 20:38, Waylan Limberg
<EMAIL_ADDRESS>wrote:

On Wed, Jan 18, 2012 at 4:40 PM, Mike Dirolf  wrote:

Great - sounds like all of the commits are fine except one still needs
discussion, see below:

Regarding 542324b, is that just a documentation edit? Was the actual logic changed previously, Should it not have been?

That is a bug fix: commit 35930e0928e19e37f81c906d5d11dfcc1087092b changed

if netloc != '' or scheme in locless_schemes:

if netloc == '' or scheme not in locless_schemes:

In order to properly reverse the logic it needs to be an and I
think. There was actually a failing test on my machine (python 2.6)
when I checked out that change.

No, there were 2 changes to that line. Both sides of the or were
negated. It is still an or.  Besides, I see no code change in your
commit so  this a documentation edit - an inaccurate one unless I'm
missing something (like your failing test??). I'll need more info
before doing anything with this.

Yeah but this is boolean logic, right? To negate (A or B) we need (!A
and !B) - your change made it (!A or !B). The failing test case wasn't
a new one I wrote, but one of the existing tests - can you try running
them on Python 2.6?

Right. I was thinking I changed (!A or B) to (A or !B). Anyway, your
patch fixes it. Not sure how I missed that.

Reply to this email directly or view it on GitHub:
https://github.com/waylan/Python-Markdown/pull/72#issuecomment-3556589


Broken link to libvirt RPC knowledgebase

KB link has been updated and the old one now 404s.
https://libvirt.org/internals/rpc.html --> https://libvirt.org/kbase/internals/rpc.html
PR Link: https://github.com/digitalocean/go-libvirt/pull/152

Fixed in #152


How to convert dateTimePicker format in winforms to match StartBoundary

How to convert dateTimePicker format in winforms to match StartBoundary
Originally posted: 2015-01-14T01:15:33

Are you asking how to use the DateTimePicker control in your own application so that it can be used to edit a StartBoundary? If so, you really can't as it forces you to either supply a date or a time. If you need just the date or just the time, the Value property can be assigned directly to the StartBoundary.
Optionally, the TaskSchedulerEditor assembly contains a FullDateTimePicker control that you can just add to your form and set the UTCPrompt property to the string you want to follow the time picker, set the TimeFormat property to LongTime and then set the Value property to the StartBoundary value. This control mimics the one found in the native Windows Task Scheduler app.
Originally posted: 2015-01-14T16:38:30


Add documentation for Address.transactions()

Was looking over the documentation for the Address endpoint and noticed that there is no API documentation for the Address.transactions() endpoint. Here is the code that governs this endpoint.

Documentation for Address.transactions() has been added.


doesn't work with javascript-typescript-langserver

Commands to reproduce:
$ git clone https://github.com/PurpleMyst/sansio-lsp-client
$ cd sansio-lsp-client
$ echo console. > playground.js
$ sed -i s/playground.py/playground.js/g sansio_lsp_client/__main__.py
$ sed -i 's/languageId="python"/languageId="javascript"/g' sansio_lsp_client/__main__.py
$ npm i javascript-typescript-langserver
$ node node_modules/javascript-typescript-langserver/lib/language-server.js

Then, in another terminal:
$ cd sansio-lsp-client
$ PORT=2089 python3 -m sansio_lsp_client

sansio_lsp_client fails with this output:
File URI: file:///home/akuli/mystbug/sansio-lsp-client/playground.py
Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/__main__.py", line 103, in <module>
    main()
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/__main__.py", line 43, in main
    events = list(client.recv(data))
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/client.py", line 109, in recv
    messages = list(_parse_messages(self._recv_buf))
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/io_handler.py", line 95, in _parse_messages
    assert set(headers.keys()) == {"Content-Type", "Content-Length"}
AssertionError

For a very quick and dirty fix, I tried adding this line before the failing assert:
if 'Content-Type' not in headers: headers['Content-Type'] = 'application/vscode-jsonrpc; charset=utf8'

Then it failed with this output in Python:
File URI: file:///home/akuli/mystbug/sansio-lsp-client/playground.js
Initialized!
Server capabilities:
{'codeActionProvider': True,
 'completionProvider': {'resolveProvider': True, 'triggerCharacters': ['.']},
 'definitionProvider': True,
 'documentSymbolProvider': True,
 'executeCommandProvider': {'commands': []},
 'hoverProvider': True,
 'referencesProvider': True,
 'renameProvider': True,
 'signatureHelpProvider': {'triggerCharacters': ['(', ',']},
 'textDocumentSync': 1,
 'typeDefinitionProvider': True,
 'workspaceSymbolProvider': True,
 'xdefinitionProvider': True,
 'xdependenciesProvider': True,
 'xpackagesProvider': True,
 'xworkspaceReferencesProvider': True}
{'code': -32001,
 'data': {},
 'message': "Cannot read property 'ensureReferencedFiles' of undefined"}
Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/__main__.py", line 103, in <module>
    main()
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/__main__.py", line 43, in main
    events = list(client.recv(data))
  File "/home/akuli/mystbug/sansio-lsp-client/sansio_lsp_client/client.py", line 124, in recv
    raise RuntimeError("Response error!")
RuntimeError: Response error!

and this output in node:
DEBUG Spawning 2 workers
DEBUG Worker 1 (PID 27753) online
DEBUG Worker 2 (PID 27754) online
INFO  Listening for incoming LSP connections on 2089
INFO  Listening for incoming LSP connections on 2089
DEBUG Connection 1 accepted
ERROR Handler for textDocument/completion failed: TypeError: Cannot read property 'ensureReferencedFiles' of undefined
    at TypeScriptService.textDocumentCompletion (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:906:14)
    at Object.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:199:48)
    at Generator.next (<anonymous>)
    at /home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:3:12)
    at MessageEmitter.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:127:47)
    at MessageEmitter.emit (events.js:214:15)
    at StreamMessageReader.callback (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:51:18)
    at StreamMessageReader.onData (/home/akuli/mystbug/sansio-lsp-client/node_modules/vscode-jsonrpc/lib/messageReader.js:183:18)
Message: {
  jsonrpc: '2.0',
  method: 'textDocument/completion',
  params: {
    textDocument: {
      uri: 'file:///home/akuli/mystbug/sansio-lsp-client/playground.js'
    },
    position: { line: 5, character: 12 },
    triggerKind: 1,
    triggerCharacter: null
  },
  id: 1
}
ERROR Handle textDocumentDidOpen: TypeError: Cannot read property 'ensureReferencedFiles' of undefined
    at TypeScriptService.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:1202:39)
    at Generator.next (<anonymous>)
    at /home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:3:12)
    at TypeScriptService.textDocumentDidOpen (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:1199:16)
    at Object.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:199:48)
    at Generator.next (<anonymous>)
    at /home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:7:71
    at new Promise (<anonymous>)
ERROR Stream was closed without shutdown notification
TypeError: Cannot read property 'dispose' of undefined
    at TypeScriptService.shutdown (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/typescript-service.js:212:29)
    at MessageEmitter.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:285:21)
    at MessageEmitter.emit (events.js:209:13)
    at /home/akuli/mystbug/sansio-lsp-client/node_modules/javascript-typescript-langserver/lib/connection.js:57:18
    at CallbackList.invoke (/home/akuli/mystbug/sansio-lsp-client/node_modules/vscode-jsonrpc/lib/events.js:62:39)
    at Emitter.fire (/home/akuli/mystbug/sansio-lsp-client/node_modules/vscode-jsonrpc/lib/events.js:120:36)
    at StreamMessageReader.fireClose (/home/akuli/mystbug/sansio-lsp-client/node_modules/vscode-jsonrpc/lib/messageReader.js:111:27)
    at Socket.<anonymous> (/home/akuli/mystbug/sansio-lsp-client/node_modules/vscode-jsonrpc/lib/messageReader.js:151:46)
    at Socket.emit (events.js:214:15)
    at TCP.<anonymous> (net.js:588:12)

I don't know whether this is a sansio-lsp-client bug or javascript-typescript-langserver bug, but I think javascript-typescript-langserver shouldn't be quite this broken.
Versions:
$ python3 --version
Python 3.7.3
$ git rev-parse HEAD
b9e01a0f6be1b196f8613ace0d90fe36a3ed87aa
$ node --version
v12.9.1
$ npm --version
6.10.2

Searching for the exact error message on Google yields this GitHub issue https://github.com/sourcegraph/javascript-typescript-langserver/issues/372
It seems that this issue stems from a missing rootUri in the initialize request. Currently Client.__init__ has it as an "optional" parameter, and the spec says that it's optional. It's probably best to set it though (in an editor) if some langservers don't work without it.
What do you think we should do?

Also I think I should add logging to this crate. Maybe I could finally use structlog, it seems so interesting..

Maybe I should just set rootUri to wherever the first .editorconfig file or .git directory is found?

Does porcupine have no concept of an open folder? (a la VS code)

There is no concept of an open folder, really only open files. I wrote some code to look for editorconfig, .git or README for now.
Let's keep this issue open for now. I'll let you know whether I get this to work or not.

Alright, let me know. I think rust-analyzer also needs a rootUri.

The headers thing is also weird..

Seems like the header thing also affects clangd

header thing fixed in #8

can we close this?

can we close this?

I haven't used javascript-typescript-langserver in a long time. I'll reopen this or make a new issue if I want to later use it and it doesn't work.

I haven't used javascript-typescript-langserver in a long time. I'll reopen this or make a new issue if I want to later use it and it doesn't work.


Collection manager, reviewer, and admin should be able to edit items submitted for review

Collection manager, reviewer, and admin should be able to click on the pencil icon and edit items submitted for review and then choose to deposit them or return them. Currently when I open an item to review and click on the pencil icon I'm taken to the start page and I see this message

I believe this is a duplicate of #445

@jcoyne This seems like a specific issue that might not have been covered by that issue. I don't want this point to get lost, so I'm happy to move my comment over to that issue and close this one.

duplicate of #442


Make output of multi input hybrid block unique

Description

fixes issue - https://github.com/apache/incubator-mxnet/issues/12795
Problem: If we have multiple inputs passed to the same block, then output symbol will have same name leading to conflicts.
Solution: append input symbol name to the block prefix, making output names to be unique per input symbol.

Example code of having 2 inputs to same block. The same Dense block is applied twice (2 inputs). Without this change, both a and b will have same name.
class MyBlock(mx.gluon.HybridBlock):
    def __init__(self):
        super().__init__()
        with self.name_scope():
            self.model = mx.gluon.nn.Dense(units=5)

    def hybrid_forward(self, F, x, y):
        a = self.model(x)
        b = self.model(y)
        return a + b

Before:
{
      "op": "FullyConnected",
      "name": "myblock0_dense0_fwd",
      "attrs": {
        "flatten": "True",
        "no_bias": "False",
        "num_hidden": "5"
      },
      "inputs": [[0, 0, 0], [1, 0, 0], [2, 0, 0]]
    },
    {
      "op": "null",
      "name": "data1",
      "inputs": []
    },
    {
      "op": "FullyConnected",
      "name": "myblock0_dense0_fwd",
      "attrs": {
        "flatten": "True",
        "no_bias": "False",
        "num_hidden": "5"
      },
      "inputs": [[4, 0, 0], [1, 0, 0], [2, 0, 0]]
    },

After
{
      "op": "FullyConnected",
      "name": "myblock0_dense0_data0_fwd",
      "attrs": {
        "flatten": "True",
        "no_bias": "False",
        "num_hidden": "5"
      },
      "inputs": [[0, 0, 0], [1, 0, 0], [2, 0, 0]]
    },
    {
      "op": "null",
      "name": "data1",
      "inputs": []
    },
    {
      "op": "FullyConnected",
      "name": "myblock0_dense0_data1_fwd",
      "attrs": {
        "flatten": "True",
        "no_bias": "False",
        "num_hidden": "5"
      },
      "inputs": [[4, 0, 0], [1, 0, 0], [2, 0, 0]]
    },

Created this PR for initial feedback.
@zhreshold @safrooze @szha

Will this also fix #12783 ?

Will this also fix #12783 ?

No. That requires separate fix using F.broadcast_* operations like Sheng and lostella are working on other PR.

LGTM, but a small concern is if this affect the save/load of existing models

@sandeep-krishnamurthy Thanks for making this change. Can you please fix build?

Change the code above to
class MyBlock(mx.gluon.HybridBlock):
    def __init__(self):
        super().__init__()
        with self.name_scope():
            self.model = mx.gluon.nn.Dense(units=5)

    def hybrid_forward(self, F, x, y):
        return self.model(self.model(x))

and you get these symbol names:
data0
myblock0_dense0_weight
myblock0_dense0_bias
myblock0_dense0_data0_fwd
myblock0_dense0_myblock0_dense0_data0_fwd_fwd
myblock0_dense0_myblock0_dense0_myblock0_dense0_data0_fwd_fwd_fwd

Something doesn't sit right with these names.

@sandeep-krishnamurthy ping, please try to rebase on latest master and see if CI pass.

@sandeep-krishnamurthy could you take a look at gluon rnn unit tests failing?

This fix is wrong as pointed by sina. Closing for now as I try to work on the fix.


Clickhouse readonly user error

Hello,
When I'm trying to connect Clickhouse DB with a readonly user DBeaver gives error because of trying to set some settings which is not allowed for readonly user.
How can we solve it?

Do you mean user with some specific permissions or "Read-only connection" option in DBeaver?
Please post exact error message/stacktrace.

Yes, this totally makes sense. But DBeaver doesn't update/delete/create anything during connection.
Although it reads some database metadata and maybe one of such reads identified by Clickhouse as update query.
That's why I need exact error message and stacktrace (you can get it in Error Log view).

The error message is:
Unexcepted driver error occured while connecting to database     ru.yandex.clickhouse.except.ClickHouseException: ClickHouse exception, code: 164, host: <IP_ADDRESS>, port:8123; Code: 164, e.displayText() = DB::Exception: Cannot override setting (extremes) in readonly mode, e.what() = DB::Exception
Maybe its related to driver. But from the error message, it looks like while DBeaver is trying to connect its trying to modify extreme setting (maybe with a default value).

Thanks.
But please post error stacktrace (main menu Window->Show View->Error Log or from log file https://github.com/serge-rider/dbeaver/wiki/Log-files)

I've checked log files (both .log and debug log), unfortunately there aren't logs after this connection error. There weren't any logs for the day of testing (I tested again and no logs created)

This issue is no longer reproducible with last versions of Clickhouse driver.

This issue is no longer reproducible with last versions of Clickhouse driver.
DBeaver 5.2.3, Clickhouse  driver 0.1.42

log in clickhouse-server.log
Trace> HTTPHandler: Request URI: /?compress=1&max_result_rows=200&password=password&user=user&extremes=0&database=default&result_overflow_mode=break
{}  HTTPHandler: Code: 164, e.displayText() = DB::Exception: Cannot override setting (max_result_rows) in readonly mode, e.what() = DB::Exception, Stack trace:
delete DBeaver.app & config folder, reinstall  did not work


Swipe action to manage favorite tags on iOS

I keep trying to do this. Putting into v1 to help things feel more functional.


AUI-2076 Improving the way to reference the move field button

Just started reviewing :) :octocat: Sent from GH.

Pull request submitted to https://github.com/mairatma/alloy-ui/pull/301. :octocat: Sent from GH.


use the right index when selecting gradient kernel

Codecov Report

Merging #795 (0cf15d1) into develop (2f03f71) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #795   +/-   ##
========================================
  Coverage    94.86%   94.86%
========================================
  Files          140      140
  Lines         8982     8982
========================================
  Hits          8521     8521
  Misses         461      461

Impacted Files
Coverage Δ

src/serac/numerics/functional/domain_integral.hpp
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

For reference, this bug showed up when adding the feature to let the integrals specify which arguments they required. Before, serac::Functional essentially represented calculations like:

where $f$ is the serac::Functional and $g_i$ are the integral calculations registered to it. After letting each integral specify which arguments it wants, then it looks more like:

So, now the $i^{th}$ argument of $f$ is not the same as the $i^{th}$ argument of $g$, so there needs to be a way to translate between them. I wrote that index translation thing once, and then copied it to the 6 different places it was required, but neglected to actually use the translated index in 2 places, and that produced the bug that Kenny saw.

Side note: while we're in src/serac/numerics/functional/domain_integral.hpp, I noticed that the argument_indices member variable doesn't have the trailing underscore.

I made these changes on my local branch and confirmed that it fixed the bug and that the gradient calculations are correct.


Linker errors when building on Windows

Building Roc on Windows is not very fun at the moment. Thank you to @folkertdev for all the help on Zulip. Currently, building Roc as-is on Windows just does not work. All of the errors encountered are linker errors. The errors are the following:
  = note: lld: error: undefined symbol: roc_alloc
          >>> referenced by C:\Users\Ross\Documents\roc\crates\roc_std\src\roc_list.rs:70
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(roc_std::roc_list::RocList$LT$T$GT$::elems_with_capacity::h2483f6d70c7c27a7)
          >>> referenced by C:\Users\crs20\Documents\roc\crates\roc_std\src\lib.rs:51
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(roc_std::roc_alloc_refcounted_help::h08260161a777fba6)

          lld: error: undefined symbol: roc_realloc
          >>> referenced by C:\Users\Ross\Documents\roc\crates\roc_std\src\roc_list.rs:174
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(roc_std::roc_list::RocList$LT$T$GT$::reserve::hb395b2048ff799d4)
          >>> referenced by C:\Users\crs20\Documents\roc\crates\roc_std\src\roc_list.rs:270
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(roc_std::roc_list::RocList$LT$T$GT$::extend_from_slice::h7698bf583e077d5e)

          lld: error: undefined symbol: roc_dealloc
          >>> referenced by C:\Users\Ross\Documents\roc\crates\roc_std\src\roc_list.rs:220
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(roc_std::roc_list::RocList$LT$T$GT$::reserve::hb395b2048ff799d4)
          >>> referenced by C:\Users\Ross\Documents\roc\crates\roc_std\src\roc_list.rs:464
          >>>               libroc_std-c05e5e25b8ffcb50.rlib(roc_std-c05e5e25b8ffcb50.1rxfp4xdyhetnw4k.rcgu.o):(_$LT$roc_std..roc_list..RocList$LT$T$GT$$u20$as$u20$core..ops..drop..Drop$GT$::drop::h00aee7c542f8a77b)

Note that have set the linker to be LLD (RUSTFLAGS = "-Clinker=lld"), as that is my preferred linker, but that same error occurs with MSVC's LINK.exe
Following the instructions in BUILDING_FROM_SOURCE.md got me partially there. Using the build LLVM in the 7z file, and setting the LLVM_SYS_130_PREFIX environment variable did work. Then run:
cargo +1.61.0-msvc build
And you will see the error above (or the equivalent LINK.exe error).
The error is occurring because it is trying to link in the roc_alloc, roc_realloc, and roc_dealloc symbols in the following crates:

roc_load
roc_cli
roc_docs
roc_docs_cli
repl_test

The first one hit is roc_load, so that was foxused on at first. Setting the ROC_SKIP_SUBS_CACHE environment variable did not let it build. folkertdev created a branch that removed a lot of things in this crate to try and work on Windows here. That commit as-is still did not work, and my solution was to just delete the "build.rs" file completely. Then the other crates had errors, and folkertdev suggested to remove them from the top-level cargo.toml. After this I was about to build the project. But this removes all the CLI executables so I'm not sure how useful it is.
The steps to build are:

rm .\crates\compiler\load\build.rs
Apply this patch

diff --git a/Cargo.toml b/Cargo.toml
index 014589c6a..045b6c8df 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -37,20 +37,16 @@ members = [
     "crates/glue",
     "crates/editor",
     "crates/ast",
-    "crates/cli",
     "crates/code_markup",
     "crates/highlight",
     "crates/error_macros",
     "crates/reporting",
     "crates/repl_cli",
     "crates/repl_eval",
-    "crates/repl_test",
     "crates/repl_wasm",
     "crates/repl_expect",
     "crates/test_utils",
     "crates/utils",
-    "crates/docs",
-    "crates/docs_cli",
     "crates/linker",
     "crates/wasi-libc-sys",
 ]

This only allows for cargo +1.61.0-msvc build to run successfully. Running any executables still does not work, and cargo +1.61.0-msvc test has the same linker failures, except you can't just remove things until it works because for any useful tests to run, they depend upon those symbols existing.
Here is a branch with the above changes:
https://github.com/RossSmyth/roc/commit/e06a8efcf8f058ee3a8052f461cdb639c0fb47d6

Looking it I think that these symbols are supposed to come from the host language and managed by the Roc compiler.

One thought that I have is that it is because of transitive dependencies.
There is a note here:
https://github.com/roc-lang/roc/blob/603f6b6613d7d8380a5690700918a3e64b5ae013/Cargo.toml#L65-L66
About the issues above, as most of them original from here:
https://github.com/roc-lang/roc/blob/0fab77af99f970245f04da85a6fd645caa7063cb/crates/roc_std/src/roc_list.rs#L16
Or elsewhere in roc_std is used, it will still try to pull it in, as this will not stop cargo from attempting to build it if it is a transitive dependency.

And to make sure I'm not insane, I can compile it fine in WSL2.

we concluded earlier that the problem is insufficient dead-code elimination. The roc_alloc function and its friends are not actually used by the compiler code. On *nix systems that means the roc_alloc function is DCE'd and it's not a problem  for the linker. On windows this is clearly not the case.
That may mean we need a feature flag that either gives just the types in roc_std, or both the types and implementations

to get the cli to compile you now need to disable repl_expect and add some definitions to cli/src/lib.rs to satisfy the linker
use core::ffi::c_void;

/// # Safety
/// The Roc application needs this.
#[no_mangle]
pub unsafe fn roc_alloc(size: usize, _alignment: u32) -> *mut c_void {
    libc::malloc(size)
}

/// # Safety
/// The Roc application needs this.
#[no_mangle]
pub unsafe fn roc_memcpy(dest: *mut c_void, src: *const c_void, bytes: usize) -> *mut c_void {
    libc::memcpy(dest, src, bytes)
}

/// # Safety
/// The Roc application needs this.
#[no_mangle]
pub unsafe fn roc_realloc(
    c_ptr: *mut c_void,
    new_size: usize,
    _old_size: usize,
    _alignment: u32,
) -> *mut c_void {
    libc::realloc(c_ptr, new_size)
}

/// # Safety
/// The Roc application needs this.
#[no_mangle]
pub unsafe fn roc_dealloc(c_ptr: *mut c_void, _alignment: u32) {
    libc::free(c_ptr)
}

fixed by #3901
cargo build works on that commit. It's not in CI yet so some things may break again. We'll try to at least get a cargo check CI job set up soon. Also testing probably still fails because of missing symbols in some places.
in any case, we're working on it, and the workarounds in this issue should no longer be needed.


initialising a disabled single-item select

Hello. My use-case is creating a single-item select, passing in a disabled <select> html element.
I would have expected the created Choices select to be disabled, but instead only the options are disabled, i.e. you can still see the dropdown but just can't change the selection.
The solution I have is to run this.disable() inside the callbackOnInit option.

Q: Wouldn't it make sense for the whole select to be disabled automatically if a disabled html element is passed?

@mattlub I agree I have just made this change in version 4.0.6 👍


update-sample-distribution-analysis (1/N)

Purpose/implementation Section
What scientific question is your analysis addressing?
Here, we explore longitudinal data in the PBTA cohort.
What was your approach?
What GitHub issue does your pull request address?
Update input and output files.
Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.
Which areas should receive a particularly close look?
Please review the script logic.
Is there anything that you want to discuss further?
Is the analysis in a mature enough form that the resulting figure(s) and/or table(s) are ready for review?
Yes.
Results
What types of results are included (e.g., table, figure)?
There are three tables generated for cases with matched longitudinal samples:

genomic_assays_matched_time_points.tsv
paired_assays_matched_time_points.tsv
transcriptomic_assays_matched_time_points.tsv

What is your summary of the results?
There are 2248 patient cases in total with longitudinal information. In cases with only genomic assays, there are 1997 patient cases with 119 having matched longitudinal samples. In cases with only transcriptomic assays, there are 1839 patient cases with 106 having matched longitudinal samples. In cases with both paired genomic and transcriptomic assays, there are 1541 patient cases with 70 having matched longitudinal samples.
The five cancer types with the highest number of tumor samples (>100) therein are: Low-grade glioma, Medulloblastoma, Diffuse midline glioma, High-grade glioma, and Ependymoma.
Reproducibility Checklist

[x] The dependencies required to run the code in this pull request have been added to the project Dockerfile.

Documentation Checklist

[x] This analysis module has a README and it is up to date.
[x] This analysis is recorded in the table in analyses/README.md and the entry is up to date.
[x] The analytical code is documented and contains comments.

Looks good to me! I made an additional comment on line 127 regarding a suggested change, but I don't need to review again.

That was meant for Line 227, correct?

This script has been updated based on suggestions by @rjcorb.
@jharenza and I decided to save also the pbta df in results_dir, because this file will be needed in other modules across the project. See e14cbd35c900d6273007ef0a2cb2ea0c5202ca8d
Please run this one last time to ensure that all files are included in the results_dir. @rjcorb @naqvia

Looks good! Can confirm pbta-tsv is saved to results_dir


Bato.to: Update page script selector

First contribution
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio

🎉


Fix the Travis builds for Ruby 2.0.

Pin net-http-persistent to version 2.9.4 as versions >= 3.0 require
Ruby >= 2.1, causing bundle install to fail.
Fixes #161

Thanks for this fix, @oliverklee!
I think we're just going to remove the travis pre-commit hook, as it doesn't provide a huge amount of value, and pinning net-http-persistent to an old version results in our Gemnasium badge showing us as having "out of date" dependencies, which isn't strictly true.


Add __len and __atindex methods on storage, and a capacity field.

Note: __atindex will deprecate mget.

Closed, there wasn't much utility and __atindex can't really replace mget since it uses multi returns.


GPT-4o mini for Advanced Paste

Description of the new feature / enhancement
Upgrade Advance Paste's "Paste with AI" from GPT-3.5 Turbo to GPT-4o mini.
Scenario when this would be used?
Advanced Paste's "Paste with AI" feature.
Supporting information
GPT-4o Mini is significantly more advanced/knowledgeable and cheaper than 3.5 Turbo.

Any update? Advanced Paste still uses GPT-3.5 Turbo Instruct model. Is this intentional?


Fix inline cache assert with BoundFunction

When we create a bound function, we set its prototype to match the prototype of the target function. Typically this is the Function prototype. But if it isn't, we can't set the prototype of the new instance without changing the prototype of all the other bound functions that are sharing the type. If the type is shared, and the new prototype doesn't match, create a new type for the new instance.

@abchatra, this is the one we discussed this afternoon.

:+1:


Fixed variant list in cls.process_checkout_lines

Used the same method found in complete_checkout.py to itemize line quantities and variants.
This will ensure both lists refer to the same line/variant when doing stock availability calculations
This will also allow multiple lines to point to the same variant, since the lists will now be of the same length
see check_stock_quantity_bulk in saleor/warehouse/availability.py for the place where this change will be apparent.
for variant, quantity in zip(variants, quantities):

I want to merge this change because...

This fixes the error identified in issue #11056

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

LGTM 🎉 @sixthgear could you also add a test case for the described situation?

@fowczarek I'd love to do that. I've been deep-diving the saleor architecture for a project.
Unfortunately, I'm currently in crunch mode to release our new storefront and I'm still getting acquainted with your testing setup and conventions. I was hoping my step-by-step in the original issue would help someone from the core team write a test case for now.
Alternatively, if you're willing to wait a bit I'll loop back to this in a few days and try to provide a test.

@fowczarek I've added a test case for you. I've confirmed that it fails on tag 3.8 but passes on my PR.

@fowczarek I've added a test case for you. I've confirmed that it fails on tag 3.8 but passes on my PR.

@sixthgear greater to here that you test it this way. I'm passing it to the team review and QA process after review.

Don't forget this one review team! In my opinion, this is actually a pretty significant bug for anyone using the track inventory feature.

Passed to QA.

Thanks for contributing to Saleor @sixthgear 🎉


Can you add an option to disable all RT from the .ini file?

Feature Summary:

Is your proposal related to a problem?

Describe the solution you'd like

Describe alternatives you've considered

Dupe


Precision on parameters names

Hello,
Can you please add the information that if you use multiple underscore in parameter names (example 'kikou____lol') then the binding expression wont't be resolved and your function won't start at all ?
Thanks !
[Enter feedback here]

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

ID: 77fbaa31-cfcc-4b8c-abe0-4efeaee3e4f0
Version Independent ID: db65fc3d-d8b3-1f19-91a0-4f3696fbca69
Content: Azure Functions bindings expressions and patterns
Content Source: articles/azure-functions/functions-bindings-expressions-patterns.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Hello @jonathanantoine, Thank you for your feedback! We will review and update as appropriate.

Hello @jonathanantoine, the requested information above is documented in the Local development guide for function app.

Setting names can't include a colon (:) or a double underline (__). Double underline characters are reserved by the runtime, and the colon is reserved to support dependency injection.

Please let me know if there are any additional concerns.

Hello @jonathanantoine, we will now proceed to close this issue out. Please tag me in your reply if there are any additional concerns, I will be glad to continue with our discussion.


Utilize the others and the evicted information in new statement table

Feature Request
Is your feature request related to a problem? Please describe:
Now we don't know how inaccurate the statement data is.

Describe the feature you'd like:
Utilize the others row and the evicted counter of the statement in the UI to notify users about how inaccurate the data is and how to make them accurate.

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

This has already been supported.


Is there any way to retrieve container sequence number as ${CONTAINER_SEQ} and use it to pass through env variables to container

Hi there,
This is mostly the question if there any way to retrieve container sequence number as ${CONTAINER_SEQ} and use it to pass through env variables to container.
For example: when I have service in terms of rancher and many scaled containers across different hosts, in a list the of containers each one is named as
{container}-1
{container}-2
{container}-3
....
When the container is launched, is there any way to get access to that sequence id 1, 2, 3 or use some variable in ranched to pass it through the container configuration > environment variables?
Looking forward to hearing from you soon.
Thank you,

No, but it's in metadata (create_index)

Hi @vincent99,
Thanks for the answer.
If I understand correctly, it means that there is a possibility to have a rest call to rancher rest meta data api to retrieve this information somehow knowing some input parameters. It looks a bit complicated, we would need anyway to pass the service name, container name to container configuration or maybe also something like container id to retrieve this kind of information.
Would be great if other more simple way exists to retrieve this kind of data.
Thank you,

You don't need to know those, metadata knows what container is asking for it and has a secton fro "self".  So you want curl http://rancher-metadata/2016-07-29/self/container/create_index

Thanks for the suggestion! I will try this option!

It works and solves my current case! Thanks again for your assistance!


Board recruiting tweaks v3

Don't close the banner after clicking the button (only close when clicking the "X")
Add "Board" to the menu

Merging without review as I need this live in 10 minutes. Sorry!


After update there are two (or more) system tray icons

Microsoft PowerToys version
v0.49.0
Running as admin

[X] Yes

Area(s) with issue?
System tray interaction
Steps to reproduce
Install old version, use update to install newer version. See multiple icons in system tray.
✔ Expected Behavior
Only one icon for PowerToys
❌ Actual Behavior

Other Software
No response

After updating from 0.49.0 to 0.49.1 this was not happening anymore, The icon disappear correctly.
Case can be closed.


Is this the latest code version ？

Hi,
I'm trying to reproduce your code, but I found some bugs,such as this

(probably my pytorch version problem, I use 0.4.0).
Is this the latest version of your code? If there is an update, please update it. Thanks
liu

Hi,
Have you tried to print(e.shape) and print(bgn)?

Best wishes,
Qiuqiang

From: liushenme<EMAIL_ADDRESS>Sent: 14 January 2020 09:53
To: qiuqiangkong/sound_event_detection_dcase2017_task4<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [qiuqiangkong/sound_event_detection_dcase2017_task4] Is this the latest code version ？ (#1)
Hi,
I'm trying to reproduce your code, but I found some bugs,such as this
[image]https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F49808596%2F72333414-b97c3080-36f6-11ea-872d-88d39ae3d171.png&data=02|01|q.kong%40surrey.ac.uk|9b031196932641c832d508d798d7a344|6b902693107440aa9e21d89446a2ebb5|0|0|637145924234554645&sdata=xfenoE%2BJX%2BlxCwUUnHxHIkT80H0NffBTKQWsdn7R0Xw%3D&reserved=0
(probably my pytorch version problem, I use 0.4.0).
Is this the latest version of your code? If there is an update, please update it. Thanks
liu
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fqiuqiangkong%2Fsound_event_detection_dcase2017_task4%2Fissues%2F1%3Femail_source%3Dnotifications%26email_token%3DADFXTSNW25AXLWP63YCSUHDQ5WDSLA5CNFSM4KGQCDGKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IF77ARQ&data=02|01|q.kong%40surrey.ac.uk|9b031196932641c832d508d798d7a344|6b902693107440aa9e21d89446a2ebb5|0|0|637145924234564640&sdata=L3TV6SV2aQqiGoF95Vl0b0EFDz6mKZYTALZ9VT1Rh6w%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADFXTSJDCJ5W6LKI7UWKG5LQ5WDSLANCNFSM4KGQCDGA&data=02|01|q.kong%40surrey.ac.uk|9b031196932641c832d508d798d7a344|6b902693107440aa9e21d89446a2ebb5|0|0|637145924234574633&sdata=%2B8GcxHp1%2BrmocfYwnAZXo5JWt%2BBNQmcr8pxhabLjt2o%3D&reserved=0.