[RFC]: add support for testing whether a value equals the current year

Description
This RFC proposes adding support for asserting whether a provided integer value equals the current year.
The implementation can use the package https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/time/current-year to dynamically resolve the current year (local time) and then perform a comparison with a provided input value.
For example,
var currentYear = require( '@stdlib/time/current-year' );

function isCurrentYear( value ) {
    return ( value === currentYear() );
}

Package name: @stdlib/assert/is-current-year.
Alias: isCurrentYear.
Related Issues
No.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Hmm, should we change the implementation to first check whether the supplied value is a Date object and if so, extract the current year from it? Seems one may want this package to both accept numbers representing the years and also Date objects.

Yeah, I think you are right. Checking for a Date object to support that use case makes sense to me. Obv, a user could also do
var d = new Date();

var bool = isCurrentYear( d.getFullYear() );
// returns <boolean>

However, an advantage to us handling this is that we can use @stdlib/time/current-year to use the correct API, which might be a source of user errors otherwise.
In short, I'm in favor of internally branching based on value type.

Implementation finished and pushed.


Added derive macro prototype and test example

Using cargo expand, we get the following output for the #[derive(StringSet)]:
struct Top {
    a: u32,
    b: u8,
    c: [u8; 3],
}
impl Top {
    pub fn string_set(
        &mut self,
        mut topic_parts: core::iter::Peekable<core::str::Split<char>>,
        value: &str,
    ) -> Result<(), ()> {
        let field = topic_parts.next().ok_or(())?;
        let next = topic_parts.peek();
        if let Some(_next) = next {
            match field {
                _ => Err(()),
            }
        } else {
            match field {
                "a" => {
                    self.a = serde_json_core::from_str(value).map_err(|_| ())?.0;
                    Ok(())
                }
                "b" => {
                    self.b = serde_json_core::from_str(value).map_err(|_| ())?.0;
                    Ok(())
                }
                "c" => {
                    self.c = serde_json_core::from_str(value).map_err(|_| ())?.0;
                    Ok(())
                }
                _ => Err(()),
            }
        }
    }
}

Note that the case where recursion is needed (if Next is Some(T)) is not yet implemented. We need to populate the match arms representing the fields that are structs. Also needed is populating a match statement where the field is an array to allow for setting a single array element.

I'm currently trying to figure out how to determine the type of each field (Is it a struct? Is it an array?) to create match statements for these other special cases, and I'm not sure how to do that in a proc macro since it doesn't have an understanding of the types, it only sees the raw syntax. As a short-term solution, we could manually add annotations on the fields that the proc macro could use to determine which fields go in which match statements, but I'm sure there is a better way.

I'm currently trying to figure out how to determine the type of each field (Is it a struct? Is it an array?) to create match statements for these other special cases, and I'm not sure how to do that in a proc macro since it doesn't have an understanding of the types, it only sees the raw syntax. As a short-term solution, we could manually add annotations on the fields that the proc macro could use to determine which fields go in which match statements, but I'm sure there is a better way.

As discussed out-of-band, the current POR is to implement StringSet recursively until the end of the input topic is reached. Currently, arrays/tuples etc. individual elements are not required, although would be beneficial in the future.

As discussed out-of-band, the current POR is to implement StringSet recursively until the end of the input topic is reached. Currently, arrays/tuples etc. individual elements are not required, although would be beneficial in the future.

As discussed out-of-band, the current POR is to implement StringSet recursively until the end of the input topic is reached. Currently, arrays/tuples etc. individual elements are not required, although would be beneficial in the future.

In order to do this, I needed to create implementations of the StringSet trait for the primitive types. Arrays are the annoying part until const generics are stabilized, you can't specify an implementation for an array of an arbitrary length, you need to implement for every length you plan to use.

As discussed out-of-band, the current POR is to implement StringSet recursively until the end of the input topic is reached. Currently, arrays/tuples etc. individual elements are not required, although would be beneficial in the future.

In order to do this, I needed to create implementations of the StringSet trait for the primitive types. Arrays are the annoying part until const generics are stabilized, you can't specify an implementation for an array of an arbitrary length, you need to implement for every length you plan to use.

In order to do this, I needed to create implementations of the StringSet trait for the primitive types. Arrays are the annoying part until const generics are stabilized, you can't specify an implementation for an array of an arbitrary length, you need to implement for every length you plan to use.

Correct, and most things in Rust provide a macro to implement traits up until 32 elements long as well.

In order to do this, I needed to create implementations of the StringSet trait for the primitive types. Arrays are the annoying part until const generics are stabilized, you can't specify an implementation for an array of an arbitrary length, you need to implement for every length you plan to use.

Correct, and most things in Rust provide a macro to implement traits up until 32 elements long as well.

Also, please pass it through a cargo fmt --all run

Also, please pass it through a cargo fmt --all run

Merging as-is with follow-on improvements

Merging as-is with follow-on improvements


Palette for command-mode?

I've tweaked lightline colorscheme to my taste, and noticed, while there are normal, insert, visual, etc. palette components, there seems to be no command component. Is it possible to change colors in the command/search mode?

It's definitely possible, you can define your command colors in your lightline colorscheme.

Yes it's possible. Use
let s:p = {
    \ 'normal': {},
    \ 'inactive': {},
    \ 'insert': {},
    \ 'replace': {},
    \ 'visual': {},
    \ 'tabline': {},
    \ 'command': {},
    \}

let s:p.command.middle = [ [ '#ff0000', '#00ff00', 9, 10 ] ]

Thanks!


Unittests excecuted during circleCI testing

unit-test upon circleCI excecution
Found that the cutting of a new release has been challenging.
We should add a set of basic unit test that gets executed with circleCI
See here for an example done by @leiyangleon for the RAiDER package

https://github.com/leiyangleon/RAiDER/blob/dev/.circleci/config.yml.
https://github.com/leiyangleon/RAiDER/tree/dev/test

We started in the past with a cricleCI integration relying on docker, which we could just replace with the conda. We need to update the .circleci/config.yml file to run a set of test and build the code. We should include a test folder with the following test suite added.
Code build
We should build conda with support for:

aria-Tools
MintPy

Testing of code install correctly:

Unit test count
Program
Test
Outcome

1
test executable for ariaDownload, ariaExtract, ariaTSprep,
e.g.; ariaDownload.py -h
provide help not error out

2
test executable for MintPY
prep_aria.py -h
provide help not error out

Testing of code runs:

Unit test count
Program
Test
Outcome

1
product count
ariaDownload.py -t 124 -w products --ifg 20180408_20180502 -v -o count
2

2
product url
ariaDownload.py -t 124 -w products --ifg 20180408_20180502 -v -o url
file: products/download_products_1240.txt

3
product download
ariaDownload.py -t 124 -w products --ifg 20180408_20180502 -v
 2 products in: products/*.nc

4
extract of 3D data
ariaExtract.py -f "products/*.nc"  -l azimuthAngle -d download
file: azimuthAngle/20180502_20180408

5
extract of mosaic data
ariaExtract.py -f "products/*.nc"  -l coherence
file: coherence/20180502_20180408

6
extract of unwrapped data - overlap
ariaExtract.py -f "products/*.nc"  -l coherence
file: coherence/20180502_20180408.vrt

7
extract of unwrapped data - overlap
ariaExtract.py -f "products/*.nc"  -l unwrappedPhase
file: unwrappedPhase/20180502_20180408

8
tsprep
ariaTSsetup.py -d download -m download -f "products/*.nc"
 3 files under stack folder: stack/cohStack.vrt; stack/connCompStack.vrt; stack/unwrapStack.vrt

9
run loading of aria products in mintpy
prep_aria.py -w mintpy -s stack/ -d DEM/SRTM_3arcsec.dem -i incidenceAngle/20180502_20180408.vrt -a azimuthAngle/20180502_20180408.vrt --water-mask mask/watermask.msk
h5 files in mintpy/input folder

@yunjunz I am trying to implement unit-tests for "prep_aria.py" as so: "prep_aria.py -d ../DEM/SRTM_3arcsec.dem -i ../incidenceAngle/20151111_20150912.vrt -a ../azimuthAngle/20151111_20150912.vrt -s ../stack"
Traceback (most recent call last):
  File "/data2/ssangha/conda_installation/stable_feb14_2020/MintPy/mintpy/prep_aria.py", line 533, in <module>
    main()
  File "/data2/ssangha/conda_installation/stable_feb14_2020/MintPy/mintpy/prep_aria.py", line 465, in main
    inps = cmd_line_parse(iargs)
  File "/data2/ssangha/conda_installation/stable_feb14_2020/MintPy/mintpy/prep_aria.py", line 125, in cmd_line_parse
    fnames = glob.glob(fname)
  File "/data2/ssangha/conda_installation/stable_feb14_2020/envs/ARIA-tools/lib/python3.6/glob.py", line 20, in glob
    return list(iglob(pathname, recursive=recursive))
  File "/data2/ssangha/conda_installation/stable_feb14_2020/envs/ARIA-tools/lib/python3.6/glob.py", line 40, in _iglob
    dirname, basename = os.path.split(pathname)
  File "/data2/ssangha/conda_installation/stable_feb14_2020/envs/ARIA-tools/lib/python3.6/posixpath.py", line 107, in split
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

Can you please advice me here? I think I just may not be calling "prep_aria.py" appropriately.

Hi @sssangha, I guess this is a bug we introduced during the last update. Could you try to change prep_aria.py#L125 from:
fnames = glob.glob(fname)

to
if fname:
    fnames = glob.glob(fname)
else:
    fnames = []

@yunjunz thanks, this works now!
I will have to change the part where the nodata values are parsed as it tries to get these values from the second IFG in the stack.
E.g. since we only pass one IFG in these unit tests, it errors out when trying to index "dsUnw.GetFileList()[2]". This is fixed when I change this to "dsUnw.GetFileList()[-1]".
I will issue a PR to address this.
Thanks again!

Hi @bbuzz31, I randomly chose 2 instead of 0 or -1 in https://github.com/insarlab/MintPy/pull/155, apparently for no reason. Your fix sounds good to me!

@yunjunz @sssangha ok great, the circleCI test pass through till the last test. That should pass as well once the PR is updated https://github.com/insarlab/MintPy/pull/329
Closing this issue.


Feature/transforms speedup

Speeds up two slow parts of the generate_commodity_groups transform by replacing nested for loops with vectorised groupby calls.
Also adds unit tests for the added functions and small test data files (from austimes) for verification.
Pytest runs and coverage reporting are now integrated into CI as well.
CSV-only regression test results show decent overall speedup:
Total runtime: 286.65s (main: 310.07s)
Change in runtime (negative == faster): -23.42s (-7.6%)
Change in correct rows (higher == better): +469 (+0.6%)
Change in additional rows: -12 (-0.0%)

But not sure whether the additional row metric increasing or decreasing is good, can you advise?
Update: full GAMS CI run here didn't show any change to correct/additional rows  :)
Total runtime: 293.08s (main: 314.38s)
Change in runtime (negative == faster): -21.30s (-6.8%)
Change in correct rows (higher == better): +0 (+0.0%)
Change in additional rows: +0 (+0.0%)

Thanks @SamRWest, looks great! I left some minor comments. I guess, we'll need to merge the benchmark PR first?

But not sure whether the additional row metric increasing or decreasing is good, can you advise?

It really depends. :-) Some of them could be additional, because casing is not the same as in the benchmarks - doesn't matter for GAMS; some need extra work - e.g. a dimension is inccorrect; some should have not been generated...

Thanks @SamRWest, looks great! I left some minor comments. I guess, we'll need to merge the benchmarks PR first?

Yep.  Looks like they're both ready to merge now, just need a couple of approvals.
But yes, do #184 first please.

@siddharth-krishna please go ahead merging if you don't have any further comments.
It is fun collaborating across the time zones!


Release: Gateway <IP_ADDRESS>

Description
Sourced from https://github.com/Kong/kong-ee/blob/next/3.4.x.x/changelog/<IP_ADDRESS>/<IP_ADDRESS>.md
Addresses https://konghq.atlassian.net/browse/DOCU-3719 and https://konghq.atlassian.net/browse/DOCU-3713.
Testing instructions
Preview link:
https://deploy-preview-7076--kongdocs.netlify.app/gateway/changelog/#3435
https://deploy-preview-7076--kongdocs.netlify.app/gateway/latest/breaking-changes/34x/#tls-changes
Checklist

[x] Review label added
[x] Conditional version tags added, if applicable.

For example, if this change is for an upcoming 3.6 release, enclose your content in {% if_version gte:3.6.x %} <content> {% endif_version %} tags (or if_plugin_version tags for plugins).
Use any of the following keys:

gte:<version> - greater than or equal to a specific version
lte:<version> - less than or equal to a specific version
eq:<version> - exactly equal to a specific version

You can do the same for older versions.

Note for Docs Reviewer:
Asked for clarification on two items on https://github.com/Kong/kong-ee/pull/8443, and noted with comments, so please skip those in your review. The question about where TLS content belongs will impact the breaking changes page as well, but planning to copy the content already published for https://docs.konghq.com/gateway/latest/breaking-changes/#tls-changes

Please ignore vale, it's doing that thing again where it randomly triggers on text in backticks.

This PR is also:

Adding all of the features that are in 3.4 that are also in either 3.5 or 3.6 already (no new docs, only adjusting version conditionals)
Generating 3.4 kong.conf and splitting the kong.conf files into maintainable separate versions

This is a lot of added lines, I know. But we decided as a team that this is a more maintainable way to handle generated docs, and for this release, it's less work for me to split them than to try and figure out where the diffs are between each version.


Improving details

During the OH the students need the SQL Server details (including server name) and the acr details, so making them available upfront, will make their life easier.

Looks great! Thanks for the update Tibi!


Include localized message files in the typescript npm package

In order to support https://github.com/Microsoft/vscode/issues/18634#issuecomment-273280003, we need to publish localized messages with the TS package.

What will the process be for contributing changes that add or touch existing messages? Who will translate them?

We already have translated messages that are shipped as part of VS. I will need to figure out the process of getting these into the main TS npm package..


Add default for SDKToUse so that solution builds in Visual Studio

PR Summary
build.psm1 was updated awhile back to conditional build with and without WindowsDesktop SDK depending on target.  Since this is only defined in build.psm1, it doesn't build in VisualStudio using the solution file.  Fix is to add in powershell-win-core project file to define SDKToUse to default to WindowsDesktop SDK if it isn't already defined.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

Thanks, I haven't had the time to test it but it is very appreciated that the team tries to keep the VS support as quite a few community members like me, Staffan and others like to use VS for some code navigation/debugging where vs-code sometimes is too simplistic or lacks some of the enhanced features of VS (despite VS being a bit more heavy)


RMVPE Functionality in Colab

I just updated RVC on my local setup (which is only powerful enough to handle inference) and noticed the RMVPE option. I wanted to try it out on the training side, but the Colab doesn't appear to have that functionality enabled. I've noticed that the Colab does seem to be behind what the local version has in terms of features (no 48k v2 support, no Crepe option, no RMVPE option, etc.). Is there a specific reason for this, or am I missing something?

Because WebUI is udpated by me, and colab is updated by some other persons.
Somebody to pr the updated colab version is needed.

Got it. Well, until there's an official update, I did some tinkering and got it working in a copy of the v2 Colab Notebook.

Added torchcrepe to the Dependencies list
Updated Clone Repository to fetch from the "Format Code (#1011)" commit (from the day of the latest version release)
Added the RMVPE.pt file to the Weights Download list
Enabled all the v2 weights (since those all seem to be functional now) and disabled all the v1 & v2 32k weights (since there isn't a 32k option in the current webUI).

I ran a small model through it and everything seemed to work fine. A few DEBUG notifications in the terminal during training, but nothing that ground things to a halt. I'm sure there are some back end improvements that would streamline it a bit more, but it's functional enough for me right now.

Hi, can you give me link please? Thanks

Got it. Well, until there's an official update, I did some tinkering and got it working in a copy of the v2 Colab Notebook.

Added torchcrepe to the Dependencies list
Updated Clone Repository to fetch at Commit 56917db (from the day of the latest version release)
Added RMVPE.pt to the Weights Download list
Enabled all the v2 weights (since those all seem to be functional now) and disabled all the v1 & v2 32k weights (since there isn't a 32k option in the current webUI).

I ran a small model through it and everything seemed to work fine. A few DEBUG notifications in the terminal during training, but nothing that ground things to a halt. I'm sure there are some back end improvements that would streamline it a bit more, but it's functional enough for me right now.

Hi, can you give me link please? Thanks

Sure, no problem. I'm new to the whole Github thing when it comes to etiquette and whatnot, and I know just enough about coding to be dangerous.
Retrieval_based_Voice_Conversion_WebUI_v2_RMVPE.txt
(Obviously change the file extension back to .ipynb. Couldn't attach that file format.)

Sure, no problem. I'm new to the whole Github thing when it comes to etiquette and whatnot, and I know just enough about coding to be dangerous.
Retrieval_based_Voice_Conversion_WebUI_v2_RMVPE.txt
(Obviously change the file extension back to .ipynb. Couldn't attach that file format.)

HI, I change colab code as same as "Retrieval_based_Voice_Conversion_WebUI_v2_RMVPE.txt" you provide
but I find the issue is that
!cp /content/Retrieval-based-Voice-Conversion-WebUI/logs/{MODELNAME}/added_.index /content/drive/MyDrive/
!cp /content/Retrieval-based-Voice-Conversion-WebUI/logs/{MODELNAME}/total_.npy /content/drive/MyDrive/
doesn't work.
Also the main function don't produce ".index file" and ".npy file".
How to adjust that?

Sure, no problem. I'm new to the whole Github thing when it comes to etiquette and whatnot, and I know just enough about coding to be dangerous.
Retrieval_based_Voice_Conversion_WebUI_v2_RMVPE.txt
(Obviously change the file extension back to .ipynb. Couldn't attach that file format.)

Thank you for your work. Is it working smoothly so far, for the RMVPE part?

So far so good. Haven't had any issues with it thus far.


Parsing no major code errors

For 'Which academic year did you complete your initial teacher training?'
This question originally used a select field that was later replaced with a
set of radio buttons. Radio buttons should be marked-up with
and  rather than a  so the html parser error is resolved.
For the address question:

'region' is not valid autocomplete value, use 'address-level-1'
see below
Town or city is the second most specific after County so is
'address-level-2'

See guidance:
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete#Values

 4.1.1 – Parsing	No major code errors


move basedomain to rawVals

keeping it consistent where we set the other values we override in the chart.

@bacongobbler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


shebang needs to be changed to /usr/bin/env python

I had a problem with rosella when it tried to run flight, as it could not import skbio. I thought the problem was with the conda recipe, but it turns out it still doesn't work in conda environments that clearly have scikit-bio installed.
Turns out that the main script of flight uses #!/usr/bin/python3 as a shebang, thereby simply ignoring the conda environment alltogether.
This needs to be changed to #!/usr/bin/env python in order to access the python modules of the conda environment

Perhaps I'm misinterpreting what you are saying, but all of the main files of flight appear to have the correct shebang: #!/usr/bin/env python3 e.g. https://github.com/rhysnewell/flight/blob/master/flight/flight.py
Can you point to where you found the incorrect shebang?

Yeah, seems like something truly bizarre is happening with your conda. Have you checked that your .condarc is set up correctly and you've got your channels configured correctly?

.condard looks ok and although i have added bioconda to my channel list, I have the channel-priority set so that python packages are preferrably installed from conda-forge rather than bioconda (which is the recommenden channel-priority according to the miniconda and the bioconda documantation). I never had these problems before...
However it seems after some googling, that some versions of skBio not working together with the newest versions of scipy is a relatively common problem... Just for testing, could you tell me which specific scipy and scikit-bio versions you are using successfully with rosella & flight?

ok the python version problem was something to do with my conda setup. But with the current scikit-bio version scipy must be older than >1.9. So during intallation scipy=1.8.1 needs to be specified, currently


begin work on the proposed "lookup free quantization"

the heart of the improvement seen at MagViT2
should lift up all generative models
this would potentially validate the approach set forth by finite scalar quantization

works! code utilization rockets to over 90%, recon goes down


DeprecationWarning: screen_size_px and minimap_size_px are deprecated.

Hi! I tried to run the train_mineral_shards.py using pycharm on the windows 7,  the error is :
Traceback (most recent call last):
File "D:\Anaconda3\lib\multiprocessing\process.py", line 258, in _bootstrap
self.run()
File "D:\Anaconda3\lib\multiprocessing\process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\Administrator.SC-201608140925\Desktop\pysc2-examples-master\common\vec_env\subproc_vec_env.py", line 20, in worker
minimap_size_px=(32, 32)) as env:
File "D:\Anaconda3\lib\site-packages\pysc2\env\sc2_env.py", line 182, in init
"screen_size_px and minimap_size_px are deprecated. Use the feature "
DeprecationWarning: screen_size_px and minimap_size_px are deprecated. Use the feature or rgb variants instead. Make sure to check your observations too since they also switched from screen/minimap to feature and rgb variants.
I hope someone can answer my question, thank you！

I got the same issue

@YuWentao95
You should use the new way：
AGENT_INTERFACE_FORMAT = sc2_env.AgentInterfaceFormat(
feature_dimensions=sc2_env.Dimensions(screen=32, minimap=32)
)
pass this into the SC2Env constructor：
with sc2_env.SC2Env(
map_name="DefeatZerglingsAndBanelings",
step_mul=step_mul,
visualize=True,
game_steps_per_episode=steps * step_mul,
agent_interface_format=AGENT_INTERFACE_FORMAT) as env:

  mconv1 = layers.conv2d(tf.transpose(minimap, [0, 2, 3, 1]),                          num_outputs=16,                          kernel_size=8,                          stride=4,                          scope='mconv1')


CI: cache mingw, azure pipelines

caches the mingw32 chocolately install in the Azure Pipeline.
Also adds whitespace to make it easier to visually parse the file.

The pipeline on my fork demonstrates that it's working. Whether it fixes our issues remains to be seen.

I thought of doing this and tried it on my own fork back in May--https://github.com/tylerjereddy/scipy/pull/35
My experiments didn't really find much consistent benefit

I thought one of the issues was download time? I may look into GitHub actions, they may be a little more stable W.r.t run time.

Sorry, I meant that it would download from some kind of cache service (maybe Amazon S3 or something?) and it seemed to be about the same amount of time in my hands. Absolutely feel free to give it a test though, maybe it'll work better for you and my iterations were just off the mark.

It may also reduce the number of failures because downloading doesn't work - that'd be a larger benefit than a speedup.
Let's give this a try I'd say.

Thanks @andyfaff and @tylerjereddy

Hmm, it seems unhappy:
Starting: Cache choco packages
==============================================================================
Task         : Cache
Description  : Cache files between runs
Version      : 2.0.1
Author       : Microsoft Corporation
Help         : https://aka.ms/pipeline-caching-docs
==============================================================================
Resolving key:
 - choco      [string]
 - Windows_NT [string]
Resolved to: choco|Windows_NT
ApplicationInsightsTelemetrySender will correlate events with X-TFS-Session 796f886d-bfa0-476e-b7db-6d1ab7ed1bcf
Getting a pipeline cache artifact with one of the following fingerprints:
Fingerprint: `choco|Windows_NT`
There is a cache miss.
tar: D\:\\a\\1\\s\\$(CHOCO_CACHE_DIR): Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
ApplicationInsightsTelemetrySender correlated 1 events with X-TFS-Session 796f886d-bfa0-476e-b7db-6d1ab7ed1bcf
##[error]Process returned non-zero exit code: 2
Finishing: Cache choco packages

From here

Probably best to revert for now. I've seen that before when I tested it--the CI in the initial PR is not sufficient testing--it has to go through the cache production and usage cycles to be thorough of course.

Let's see it fail at least once more - possible this just happened when there was no cache right after merging.

Okay more cache misses, reverted.

My uneducated guess based on the drive path is that the drive mounting changes, and the cache is not located in the place where Chocolatey is looking for it.


fix: update to eye v10.7.13

fix: update to eye v10.7.13

:tada: This PR is included in version 16.4.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v16.4.15

Your semantic-release bot :package::rocket:


Additional src folders?

Although we have no need for it now, in a future version it may be useful to have additional src folders. The .gitignores are setup to ignore any strict subfolders of src/it/unimi/dsi/fastutil. But sometimes it may be useful to have non-generated classes for specific cases. For example, specifically tuned implementation of boolean collections abusing the minuscule element state space (2). Or non-type specific code that we don't want to be public api could be in a it.unimi.dsi.fastutil.internal package. We can't put those in the existing src folder as the .gitignore would exclude them.
I think the cleanest solution to this would be to have a separate source folder for non-generated classes. Ant supports multiple source folders, so they would all live in the same classpath. Something like src-nongen or something (terrible name but I can't think of anything better at the moment). With some sort of ABOUT.txt or something describing why it is there.

I think a more canonical approach would be to move to the src/main/java / src/test/java and src/main/generated etc. structure. This then also would be understood by IDEs I think

I think a more canonical approach would be to move to the src/main/java / src/test/java and src/main/generated etc. structure. This then also would be understood by IDEs I think

I like that idea better, but I wonder if
src/main/java for handwritten code and
src/generated/java would be even more canonical

I like that idea better, but I wonder if
src/main/java for handwritten code and
src/generated/java would be even more canonical

IIRC, canonical is src/main and src/test for different source / compilation sets (main is everything that is part of the distribution, test is part of testing, you could have src/test-fixtures or similar for, well, fixtures). So the first level basically refers to where the code belongs to, second level to its type. So we could have src/main/c for the generated c code, src/main/generated for the generated java code and src/main/java for the actual implementations.
This is at least the structure that gradle, IntelliJ, and (I think) maven generates by default.

IIRC, canonical is src/main and src/test for different source / compilation sets (main is everything that is part of the distribution, test is part of testing, you could have src/test-fixtures or similar for, well, fixtures). So the first level basically refers to where the code belongs to, second level to its type. So we could have src/main/c for the generated c code, src/main/generated for the generated java code and src/main/java for the actual implementations.
This is at least the structure that gradle, IntelliJ, and (I think) maven generates by default.

This is at least the structure that gradle, IntelliJ, and (I think) maven generates by default.

And, I hate Maven 😂

This is at least the structure that gradle, IntelliJ, and (I think) maven generates by default.

 We already aren't using a "conventional" folder structure for the code generation.
We can convince, forcefully if needed, Ant (the current fastutil build system for the Java code) to accept whatever folder structure we want. 😆
That said, something resembling conventional would still be a good idea.


document use for creating multiple presentations

Not sure if is possible to use reveal-hugo to create multiple presentations using a single source repository to host them.
I would like to avoid having to create a new repository for each presentation I create and publish and instead to use a single repository to host multiple presentations.

Yes, this is possible. Each folder in the content directory is turned into its own presentation with _index.md as the root. If you look at the exampleSite folder of this repository,  you can see multiple presentations co-existing. Hopefully that's instructive as to how to get that set up.

After cloning the repo, cd exampleSite, hugo serve there is no list of presentations, but a single presentation is shown instead. I found its content in /home. Does home have a special meaning? Or how can we reach e.g. custom-theme-example? By knowing the url http://localhost:1313/custom-theme-example/ ? One paragraph documentation would be nice. Thanks for your great work - a single-presentation-repo already works great for me!
Is it also possible to have a reveal-hugo presentation inside some other hugo site? Having a separate git repository and build for every presentation would be too heavy. And having one hugo site for documentation, sort of wiki etc. and separate hugo site just for presentations (even multiple) feels cumbersome. hugo seems to support multiple hugo themes per site but one example how to install reveal-hugo and activate just for a single content/ subfolder would be great, provided you use that approach too. Or do you have a separate git repo / hugo site just for all your presentations?

Finally found the source of all my trouble: while the hugo modules supposed to be modular, in our case - provide revealjs type of output, and most parts in https://github.com/joshed-io/reveal-hugo/tree/master/layouts/_default try to be a good citizen (they contain reveal in file name  and only take effect for sections with output: reveal, some nasty ones take high priority (according to the order in https://gohugo.io/templates/lookup-order/#single-templates)
The hack https://github.com/joshed-io/reveal-hugo/commit/d3e297d5ccf21e24d91a7bf61b2513f8d4b02841 just nukes all the other layout definitions you may have in your repo, provided by other theme. This is why I was getting empty pages all the time!


Add D3D12 pixel history depth bounds test.

Copied from the Vulkan implementation.

Description

Before:

After:

Figured I'd sneak in a last one of these for 2023, according to my time at least. Happy new year!

Thank you again for another missing piece of D3D12 Pixel History.
I will enable the D3D12 Pixel History automated test for Depth Bounds Test (which before this change failed and passes with this change)


The buttons on the toolbar of react queue explorer show as regular size when setting the toolbar size to small

Storage Explorer Version: 1.32.0-dev (93)
Build Number: 20231007.7
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Sonoma 14.0(Apple M1 Pro)
Architecture: x64/x64/arm64
How Found: From running test case
Regression From: Previous release (1.31.1)
Steps to Reproduce

Open 'Settings' -> Click 'Data Explorers -> Toolbar Size'.
Set the toolbar size to 'Small'.
Expand one storage account -> Queues.
Right click one queue -> Click '[NEW] Open'.
Check whether the buttons on the toolbar show as small size.

Expected Experience
The buttons on the toolbar show as small size.

Actual Experience
The buttons on the toolbar show as regular size.

Verified this issue on the main build 20231013.4. Fixed. So, we are going to close it.


Call to a member function isConsultant() on null

Error
Call to a member function isConsultant() on null
:dart: /app/resources/views/projects/my-projects.blade.php
:page_facing_up: /app/resources/views/projects/my-projects.blade.php
12       </div>
13   </x-slot>
14*  @if (($user->context === \App\Enums\UserContext::Organization->value &&
15       ($user->organization->isConsultant() ||
16           $user->organization->isConnector() ||

Open in Flare

Looks like this is coming from an organization user. The condition checks if the user has an 'organization' context, and if so proceeds to check the roles of the user's organization. However, it is possible for a user to have an organization context without the organization being attached yet. This happens for newly created organizations. The user account is created with the 'organization' context, but the organization isn't created until the organization name has been defined. This happens after the user submits the e-mail verification and selects the organization type. However the projects page is accessible to the user at this point, either from a direct link to the projects page, using the available top navigation link to the projects page, or if they had tried to access the page before verifying the e-mail and were redirected to it.

I spoke with @cherylhjli about possible solutions. We've decided to redirect from the project page back to the org type selection page if the organization isn't defined for a user with an organization context.


Windows debugging

This PR was created in an attempt to further debug the Windows issues that we are running into for wasmcloud.
TL;DR: use WSL
Re: #60, the libloading library automatically appends .dll which I confirmed in their documentation, so we have to extract the provider with the .dll extension or windows machines will not be able to work.
Refactored the libloading section to have a linux, windows, and "other" section for loading capability providers. I was able to discover that the .dll section is not required so long as you use the libloading::os::windows module for the library loading portion. I was not able to resolve https://github.com/wasmCloud/wasmCloud/issues/59, and windows still can segfault upon initialization of the capability provider. I was able to confirm with dumpbin that the downloaded httpserver provider (I'm running the echo example) does export __capability_provider_create, but regardless the code panics with a LoadLibraryExW error. From what I can read, this is due to missing shared libraries elsewhere on my system, but as far as I know I have a well functioning Windows development environment.
I've used depends to try and track down the shared module that is failing, however my depends lists hundreds of missing .dlls.
I've used ProcessMonitor to track syscalls for the wasmcloud program, but I can't find anything out of the ordinary other than a failure to CreateFileMapping. I've attempted to open the library with the specified protections, but unfortunately this doesn't fix the issue.
Here's my current error:
[2021-03-30T18:14:31Z DEBUG] GetProcAddress { source: Os { code: 126, kind: Other, message: "The specified module could not be found." } }
[2021-03-30T18:14:31Z ERROR] Failed to extract plugin from provider: GetProcAddress failed

Interested if anyone else on windows can reproduce, if you have nats-server and I stole the manifest for the program from https://github.com/wasmCloud/examples/blob/main/echo/manifest.yaml
Reference:
https://docs.microsoft.com/en-us/windows/win32/memory/memory-protection-constants
https://docs.rs/libloading/0.7.0/libloading/os/windows/struct.Library.html

w/r/t missing shared files - have you installed a windows equivalent of the C runtime? On linux we need that for libloading to work.

@autodidaddict Unfortunately I've spent hours trying to make sure my runtime environment is correct, and I'm like 90% sure that my configuration is the correct one. Originally I started on x86_64-pc-windows-gnu, only to find out that the recommended way is to install visual studio (not code) and get all of the build tools through there. Which I was able to do, and I even use the whole cross tools command prompt, but its still possible I don't have the proper toolchain installed. If there's any easy resource to check that I'm happy to try.

Thanks for the diligence on this. I think you're right in that, at least for now, we recommend people use WSL when working on windows.

Thanks for the diligence on this. I think you're right in that, at least for now, we recommend people use WSL when working on windows.

This does narrow our scope, but as you mentioned (potentially in slack) this only really affects users that intend to use windows in production as WSL is a small change for a development machine. We'll just have to take this stance in our documentation somewhere as far as OS/platforms that we officially support

Since https://github.com/wasmCloud/wasmCloud/issues/60 is closed, closing this PR. Picking up any related docs in https://github.com/wasmCloud/wasmcloud-dev-site/issues/28


Bluetooth: tests: bsim: host/adv/resume2

This test is an attempt at formalizing at least part of the behavior described in commit 6a79c3deae7d3a09965a6e71fc7da863aaa721fa.

Here is the range-diff:
git fetch http://github.com/zephyrproject-rtos/zephyr pull/70670/head && git range-diff f0ef5147e84^! 836b540ac6^!


Review mapshader docstrings

I asked Sphinx team about an issue (Suppress variable expansion when using Sphinx autosummary) that I'm having.

@brendancol @rencesar , could you take a look at this PR?


Clicking cancel button jump to Initial ViewController on ios5.x simulator

i write a Demo for this issue on :https://github.com/zhoujie903/PXAlertViewIssue.git
i modify some code in PXAlertView for running on ios5.x(xcode4.6.3)
I found a solution to this problem:
in [- (void)dismiss:(id)sender]
modify this line code [self.mainWindow makeKeyAndVisible] to [self.mainWindow makeKeyWindow]

I encountered the same problem on an iOS 8 iPhone 6 simulator (Xcode 6.3.1). The fix still works though.


Ping notifications

Hello,
Could you add ping notifications if ping fails x amount of times send out mail to notify someone.

Hello @NicoCG,
I'll take a look at adding this immediately. @bryan5989 and I were toying around with this idea at work for several days, and I think its a good idea.
Let me get in touch with him, I believe we can add an event based system for notifications. Let me get his opinions first.
Thanks,
Johnny

Thanks @RedRoyalDog and @bryan5989 I am excited to see future functionality!


Closing window through taskbar context menu closes the app completely.

I have 7+ Taskbar Tweaker (Win7) and it allows me to close things in my taskbar by middle-clicking the entity on it.
The problem is: all apps that live in tray don't terminate when I close their window like this, but TogglDesktop does.
To replicate this error without that software - right click Toggl Desktop and choose "close window":

Expected result: same if I'd click a cross icon in the window - Toggl is minimized to tray.
Actual result: Toggl quits.
Other software doesn't get killed if I do this. (such as: Spotify, Telegram, Workrave and more)

I'm able to replicate this issue. I will look more into it to find a fix.
Thank you for reporting this.


LLVM 20. KW 2022

@b-studios @phischu Should we now delete the singlestack branch?

Should we now delete the singlestack branch?

If it only consisted of 50c234c and aad1553, then sure.


Open AI chat completion API "Retry-After" header is not exposed to the browser

Hi, I'm trying to use stay within the tokens per minute limited enforced by the Open AI Chat Completion API.
When server responded with 429 error, I'm expecting the Retry-After field in the header exposed to the browser so the caller can retry after the server suggested period of time. The documentation here specially calls for such retry strategy

General best practices to remain within rate limits
To minimize issues related to rate limits, it's a good idea to use the following techniques:

Implement retry logic in your application.
...

The problem is that the response header is set such that the Retry-After header field is hidden from the browser. See screenshot below:

This makes it difficult for the client to set the right timeout. Please consider adding Retry-After to the Access-Control-Expose-Headers field.
References:

Understanding rate limit
General best practices to remain within rate limits

@chuanqisun Apologies for the late reply. Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

Adding service team to look into this.


Camera Rotation

Hi, I'd like to ask about camera rotation. I'm trying to figure it out, but programming is new to me. I would like to make a rotation adjustment for a 6-camera scheme. It should work so that the cameras in the minimized windows are static. After a certain time, all 5 cameras would rotate in the enlarged window. The scheme would be with 6 windows, but there would be only five cameras. Instead of the sixth, all the cameras would take rotation. How to do it?

I have no idea. It was difficult getting the current rotation to work
without an object oriented language. RPiSurv may do what you want.
Thanks
On Mon, Aug 24, 2020, 2:10 PM IkaSGC<EMAIL_ADDRESS>wrote:

Hi, I'd like to ask about camera rotation. I'm trying to figure it out,
but programming is new to me. I would like to make a rotation adjustment
for a 6-camera scheme. It should work so that the cameras in the minimized
windows are static. After a certain time, all 5 cameras would rotate in the
enlarged window. The scheme would be with 6 windows, but there would be
only five cameras. Instead of the sixth, all the cameras would take
rotation. How to do it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Anonymousdog/displaycameras/issues/76, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIYIKJIVCTT4IINPL6XE6LSCKUJBANCNFSM4QJYUZSQ
.


Activation fails on ABAP 7.02

Hi,
When trying to activate the latest build of the abapGit report on ABAP 7.02 (SP 14) one gets the following error message:
Unexpected word "EXPORTING" in functional method call

This does not occur with later versions (I tested with ABAP 7.4).  The error is triggered by the following call in the method popup_package_overwrite of the class ZCL_ABAPGIT_SERVICES_REPO:
    zcl_abapgit_ui_factory=>get_popups( )->popup_to_select_from_list(
      EXPORTING
        it_list               = ct_overwrite
        iv_header_text        = |The following objects have been created in other packages.|
                             && | Select the objects which should be overwritten.|
        iv_select_column_text = |Overwrite?|
        it_columns_to_display = lt_colums_to_display
      IMPORTING
        et_list               = lt_selected ).

In ABAP 7.02 a functional method cannot have an EXPORTING parameter. Apparently at compiling time the ABAP compiler erroneously identifies the EXPORTING keyword as part of the zcl_abapgit_ui_factory=>get_poups() functional method call.
I solved the problem as follows:

DATA: lr_popups TYPE REF TO zcl_abapgit_popups.
    lr_popups ?= zcl_abapgit_ui_factory=>get_popups( ).
    lr_popups->popup_to_select_from_list(
    EXPORTING
        it_list               = ct_overwrite
        iv_header_text        = |The following objects have been created in other packages.|
                             && | Select the objects which should be overwritten.|
        iv_select_column_text = |Overwrite?|
        it_columns_to_display = lt_colums_to_display
      IMPORTING
        et_list               = lt_selected ).

After the correction, the problem is solved and the report is compiled successfully. I'm submitting a patch with the solution.
Best,
Hector

thanks, hope to implement https://github.com/larshp/abaplint/issues/221 sometime, so we can more easily discover errors like this

You're welcome. BTW amazing work with abapGit overall 👍 👍

fix is merged, guess this can be closed, closing


Create tests cases for profile pic upload and errors, setup github actions

Please comment on whether this approach is desired for testing the code or some other approach is favored instead.
I would like to use the library in a commercial product and it would make the team a whole lot more comfortable using it if the code were even partially covered by tests.

Thanks for this PL, I am not sure it fits the library since if we want to add something like UnitTest, then we have to cover all aspect of the project, not just one functionality which will be shady.
I have to close this, so we can focus on the project more.
Habby Christmases.


Space X project: Rocket page, Mission page, Profile page

In this pull request, @Hassaanjbaig-code and I have been able to:

solve issue 24 issue 23 issue 22:

Setup the react-redux repository
Setup the linters
Add Navbar
Add logo to the project

solve issue 21 issue 17 issue 13:

Add styling on the active Link of the Navigation bar
Add a directory for all Redux state slice files.
Add the fetchRockets method to fetch the list of rockets from the API
Add ListRocket component to dispatch fetchRocket only once and display the list for request

solve issue 15 issue 18 issue 20

Basic structure of the mission
Deploy the mission in the list -render
Fetch the data from the API

solve issue 5, issue 12 issue 8:

Add rocket booking feature
Add rocket booking cancelation
Add conditional rendering for 'reservation' badges
Add testing for Rocket.jsx, ListRocket.jsx, Navbar.js and App.js

solve the issue 3:

Add the feature of displaying a list of the reserved rockets on the profile page
Add test for profile pages

solve issue 2 issue 9 issue 10

Implement mission joining
Implement mission leaving
Switch badges for Missions
Display joined missions in My profile

Here is the Live Demo

@DammyShittu Thank you for review


Added generation of unique identifiers

Problem
Currently there are several problems related to the unique ids:

There is no support for getting and storing unique id attribute from Basic cluster
There is no support for getting unique id for rotating device id calculation purpose. That unique id is different one from
the Basic cluster unique id and is utilized as an input key for rotating device id generation algorithm
Rotating device id is generated using serial number what is not compliant with the spec (it should be unique id)

Change overview

Added methods to get and store unique id utilized in Basic cluster
For all platforms added generic method generating random unique id utilized in Basic cluster
Added GetRotatingDeviceIdUniqueId method for getting unique id utilized by rotating device id
Replaced using serial number with unique id for generating rotating device id.
Disabled CONFIG_FPU for nrfconnect platform, as it turned out to lead to MPU_FAULT and system crash on BLE C3 characteristic read

Testing
Did some manual testing including:

Unique ID verification with Python CHIP controller:

The assigned random unique ID is available.

zclread Basic UniqueID 1 0 0
Data = "BDA74374A48427AE",
AttributeReadResult(path=AttributePath(nodeId=1, endpointId=0, clusterId=40, attributeId=18), status=0, value='BDA74374A48427AE')

After reboot unique ID didn't change.

zclread Basic UniqueID 1 0 0
Data = "BDA74374A48427AE",
AttributeReadResult(path=AttributePath(nodeId=1, endpointId=0, clusterId=40, attributeId=18), status=0, value='BDA74374A48427AE')

After factory reset unique ID is different

zclread Basic UniqueID 2 0 0
Data = "793AE7D2D94648CC",
AttributeReadResult(path=AttributePath(nodeId=2, endpointId=0, clusterId=40, attributeId=18), status=0, value='793AE7D2D94648CC')

Rotating device ID value is added in C3 BLE characteristic and commissionable node service, as before, but the value is generated using unique ID.

Also I've added automated test verifying storing and getting Unique ID value from the storage.

Rebased

Rebased

@andy31415 @bzbarsky-apple I refactored the AdditionalDataPayloadGenerator to not use rotating device ID only parameters, but rather generic structure type. I believe that will allow to extend this module to use not only rotating device ID in the future. Please see the last commit: https://github.com/project-chip/connectedhomeip/pull/15031/commits/da4ee7cd84600202424f420353d10320acdb4158 and review if those changes are applicable.

Rebased


Having the notification appear for wait

How can I have the notification appear as long as I am waiting for the current operation to return status like an ajax call. All the examples have a predefined time which we can set. Is there any setting where I want the notification to appear as long as I want and when I have result from the ajax call then I can hide the notification.

@zeeshanjan82 you could give the sticky notifications a try. - https://github.com/matowens/ng-notify#sticky-notifications
Set sticky to true, and then when your promise is resolved, you could call the dismiss() method to close the notification when you're ready to.

@matowens in iOS, I'm using this statregy however the notifications isn't fading out. I opened an issue #48 for this.


feat(vite-aliases): Added Vite 3 Support + Export Default

Vite 3 Support

Added Support for Vite 3.

Turned to "type": "module"
Changed imports in package.json around to support new files

Added export default to index.ts -> to import as import ViteAliases from '....' or import { ViteAliases }  from '...'

If I need to change something or test with Vite 2, I can add those test.
Thanks

@Subwaytime

Hey there, thanks for the PR!
Will take a look later today :)

Hey again! Finally was able to check the changes in this pr! These changes are already part of the upcoming changes i have made locally, but i wasnt able to get it working like you in the pr (as i moved to ESM entirely) while your Option still covers cjs, so i will merge it soon together with my changes :) :+1:


offset is different for btcPrice and btcPriceList queries

Description
Can be observed by calling this query:
query btcPriceList($range: PriceGraphRange!) {
  btcPrice {
    base
    offset
    currencyUnit
    formattedAmount
  }
  btcPriceList(range: $range) {
      timestamp
      price {
      base
      offset
      currencyUnit
      formattedAmount
    }
  }
}

Result:

Task
We should change this offset value to be consistent (advised by @nicolasburtey) and determine if any follow-on changes are needed in galoy-mobile or galoy-pay.

Hello, I'm new to the project and contributing to open source, took a quick read of README, CONTRIBUTING, and DEV.
These are my first impressions, I haven't set up a development environment yet but just following the assignments, this is what I'm seeing/expect.
Just following through what's assigned where, the offset of 4 vs. 12 is assigned in the config. The 8 delta between the two is the SATS/BTC conversion.
Lines 17-18
SAT_PRICE_PRECISION_OFFSET is assigned for BtcPrice
BTC_PRICE_PRECISION_OFFSET is assigned for BtcPriceList
Looking at the use of each constant in the codebase, BTC_PRICE_PRECISION_OFFSET appears only in this one instance, so likely for wider consistency, the SAT_PRICE_PRECISION_OFFSET should be used in BtcPriceList.
In galoy-pay, the conversion is performed on the fly from offset and base, so should be fine in both cases.
https://github.com/GaloyMoney/galoy-pay/blob/main/lib/use-sat-price.tsx
In galoy-mobile, this is similarly done here
Lines 127-133
Lines 166-167
Line 53
Line 142

... so likely for wider consistency, the SAT_PRICE_PRECISION_OFFSET should be used in BtcPriceList.

Hi @McLarney, sorry for the delay in getting back to you on this. I think this sounds about right. Would you like to try tackling this change in a PR? If you join our Slack, we can also help with any issues you have getting the dev environment setup to test against.


Get Acknowledgment on sink from source to commit offsets manually

Please see the following Stack Overflow post for details on issue:
http://stackoverflow.com/questions/39042880/how-to-manually-commit-offsets-on-kafka-source-module-on-getting-acknowledgment

I have answered on SO.


Adding documentation for TEMT6000X01

Resolves #789
Resolves #802
Resolves #832

Coverage increased (+0.02%) to 79.931% when pulling 2c33d170742880f02e08581657ee6416c02e64de on HuangDave:temt6000x01_documentation into f04e5900c0d8fdb3ef36c8cd194c3955c7a0fb54 on kammce:master.


Filter Overheads does not filter overhead messages.

Messages that are blocked in "public" are not blocked in overhead chat.
Spam still displays in overhead.
Included image showing blocked messages in public, while displaying on overhead.

My Settings if needed.

I think this could be a dupe of https://github.com/jackriccomini/spamfilter-plugin-runelite/issues/10. The issue there was that whitespace at the start of a message would cause it to be processed inconsistently by the chatbox and overhead filters. I have fixed that, however in your screenshot the message from e.g. "Coin PN54" doesn't seem to have any extra whitespace compared to other messages which would suggest it's a different underlying issue.
I will keep this issue open. After 1.0.1 releases (track it here) could you comment whether you still experience the issue? If you do then it must be a different problem

I'm decently new to git, has the 1.0.1 version been released as of today?
Tried to keep up with the link you posted, but unsure what I'm really looking at.

I'm decently new to git, has the 1.0.1 version been released as of today?
Tried to keep up with the link you posted, but unsure what I'm really looking at.

It released yesterday. If you check the link you will see it now says "merged" with a purple icon

There does still seem to be an issue with overhead filtering. Most of the unfiltered overheads used chat effects, that might be contributing to the issue?

I think I figured out the issue. Whenever the Chat Filter plug-in is turned on, the overhead messages are not filtered.

I think I figured out the issue. Whenever the Chat Filter plug-in is turned on, the overhead messages are not filtered.

Thank you! A had a branch for this last year which I never ended up merging but which I think should still work. When I have my dev environment set up again I will test it and hopefully be able to close this issue

Hey, ik this issue is pretty old but has any workarounds been found?

Hey, ik this issue is pretty old but has any workarounds been found?

IIRC this has been fixed in the master branch. The version containing the fix has not yet been released to the plugin hub. I made a PR here to release to the plugin hub at the time I fixed the issue. The checks failed because this plugin was originally written to use APIs that have since been deprecated. I made a mental note at the time to fix it when I next had a chance and it's been on my mental todo list ever since


Interaction caching for non-commands

Firstly I want to thank you for all the work that you do on this. I really love it!
Is your feature request related to a problem? Please describe.
I wanted to use buttons for my automation and enabled the raw message option. I tried to answer to an interaction by a button I created using the id in the raw interaction, but noticed that I get an error. Looking through the code I saw that only command interactions are cached and can be responded to.
Describe the solution you'd like
Please implement the option to cache interactions that do not relate to commands.
Describe alternatives you've considered
I could just check for changes in the raw interaction and respond with a normal message, but I will always get the error when pressing the button.

Thank you for submitting this issue. 👍
I haven't used buttons in discord yet.
Can you provide a detailed example of what you have done including some example code?

Thank you for your immediate response.
This is the code I was using to test what I wanted to do:
`function sendNotificationWithButton(notification) {
sendTo('discord.0', 'sendMessage', {
serverId: '',
channelId: '',
content: {
content: notification,
components: [
{
"type" : 1,
"components": [
{
"type": 2,
"label": "Click me!",
"style": 1,
"custom_id": "test_id"
}
]
}
]
}
}, (ret) => {
console.log(ret);
});
}
function sendInteractionReply(interactionId) {
sendTo('discord.0', 'sendCustomCommandReply', {
interactionId: interactionId,
content: {
content: "Response"
}
}, (ret) => {
console.log(ret);
});
}
function getInteractionIdFromString(interactionString) {
var interactionJson = JSON.parse(interactionString);
return interactionJson.id;
}
on({id: "discord.0.raw.interactionJson", change: "ne"}, async function (obj) {
var interactionId = getInteractionIdFromString(obj.state.val);
console.log(interactionId);
sendInteractionReply(interactionId);

});
sendNotificationWithButton("test");`
This is the error I am getting after clicking the button:
script.js.Notification.Test: { error: 'Error sending reply: Error: No current interaction with ID<PHONE_NUMBER>159496202 found for reply!', interactionId: '1258860476159496202', content: { content: 'Response' } }

👍 I'll have a look at this in the next days.

@Babywombat Can you try the current GitHub version? This should work with your provided example.

Internally all unknown interactions are now cached and should be usable by sendCustomCommandReply for 15 minutes after they are created. The reply to these interactions is always deferred, so the user has some time to send the reply.

@crycode-de I just tested it out and it works like a charm. Thank you for the quick addition.

Great, thank you!
I'll release a new latest version probably tomorrow.

Version 2.4.0 released containing this changes.


Problem on compilation

Hey! I am using realm-js v1.1.1
Error
/Users/ovr/projects/interpals/frontend/native/node_modules/realm/src/object-store/src/object.cpp:42:20: error: no viable overloaded '='
        m_notifier = std::make_shared<_impl::ObjectNotifier>(m_row, m_realm);
        ~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /Users/ovr/projects/interpals/frontend/native/node_modules/realm/src/object-store/src/object.cpp:19:
In file included from /Users/ovr/projects/interpals/frontend/native/node_modules/realm/src/object-store/src/object.hpp:22:
/Users/ovr/projects/interpals/frontend/native/node_modules/realm/src/object-store/src/impl/collection_notifier.hpp:256:13: note: candidate function not viable: no known conversion from 'typename enable_if<!is_array<ObjectNotifier>::value, shared_ptr<ObjectNotifier> >::type' (aka 'std::__1::shared_ptr<realm::_impl::ObjectNotifier>') to 'const realm::_impl::CollectionNotifier::Handle<realm::_impl::ObjectNotifier>' for 1st argument
    Handle& operator=(const Handle& other)
            ^
/Users/ovr/projects/interpals/frontend/native/node_modules/realm/src/object-store/src/impl/collection_notifier.hpp:265:13: note: candidate function not viable: no known conversion from 'typename enable_if<!is_array<ObjectNotifier>::value, shared_ptr<ObjectNotifier> >::type' (aka 'std::__1::shared_ptr<realm::_impl::ObjectNotifier>') to 'realm::_impl::CollectionNotifier::Handle<realm::_impl::ObjectNotifier>' for 1st argument
    Handle& operator=(Handle&& other)
            ^
1 error generated.

My ENV
clang -v
clang version 4.0.0 (tags/RELEASE_400/final)
Target: x86_64-apple-darwin16.4.0
Thread model: posix
InstalledDir: /usr/local/opt/llvm/bin

Thanks

Thanks for filing an issue! This repository is for the object store, which is an internal component of Realm. I've cross-filed your bug to the realm-js repository and will be closing this ticket.

Hey! Thanks for move this issue, but src/object-store/src/object.cpp:42:20 looks like it's object-store problem and posted here :)

This repo isn't intended for public issue tracking. Our bindings engineers triage bugs and then reassign them as necessary. Some bugs that appear to originate in object store code may be caused by a bug in the binding code.


Quad Nodes leak memory

Operating system or device - Godot version:
Tested and observed in both 2.0.1 on Windows 7 and 2.1.1 on Windows 10
Issue description (what happened, and what was expected):

This here number goes up when instancing a Quad, but never goes down when freeing it.
Steps to reproduce:
Create a quad. Try to free it.
Link to minimal example project (optional but very welcome):
Quad Test.zip

Here is a bunch of errors generated from that test project on exit after doing the instance/free operation a few times:
ERROR: finish: Condition ' instance_dependency_map.size() ' is true.
   At: servers/visual/visual_server_raster.cpp:7742.
ERROR: ~MemoryPoolDynamicStatic: DYNAMIC ALLOC: ** MEMORY LEAKS DETECTED **
   At: core/os/memory_pool_dynamic_static.cpp:249.
ERROR: ~MemoryPoolDynamicStatic: DYNAMIC ALLOC: 168 bytes of memory in use at exit.
   At: core/os/memory_pool_dynamic_static.cpp:250.
ERROR: ~MemoryPoolDynamicStatic: DYNAMIC ALLOC: Following is the list of leaked allocations:
   At: core/os/memory_pool_dynamic_static.cpp:252.
ERROR: ~MemoryPoolDynamicStatic:        52 bytes - core/dvector.h:403, type: DYNAMIC
   At: core/os/memory_pool_dynamic_static.cpp:258.
ERROR: ~MemoryPoolDynamicStatic:        52 bytes - core/dvector.h:403, type: DYNAMIC
   At: core/os/memory_pool_dynamic_static.cpp:258.
ERROR: ~MemoryPoolDynamicStatic:        36 bytes - core/dvector.h:403, type: DYNAMIC
   At: core/os/memory_pool_dynamic_static.cpp:258.
ERROR: ~MemoryPoolDynamicStatic:        28 bytes - core/dvector.h:403, type: DYNAMIC
   At: core/os/memory_pool_dynamic_static.cpp:258.
ERROR: ~MemoryPoolDynamicStatic: DYNAMIC ALLOC: End of Report.
   At: core/os/memory_pool_dynamic_static.cpp:262.
INFO: dynmem - max: 354226, 168 leaked.
**ERROR: STATIC ALLOC: ** MEMORY LEAKS DETECTED **
**ERROR: STATIC ALLOC: 1008 bytes of memory in use at exit.
**ERROR: STATIC ALLOC: Following is the list of leaked allocations:
        0x48a38d0 - 16 bytes - core/vector.h:283, memalloc.
        0x47fa9e0 - 24 bytes - core/array.cpp:63, memnew type: ArrayPrivate
        0x39f2cc0 - 24 bytes - core/array.cpp:63, memnew type: ArrayPrivate
        0x48a4b70 - 400 bytes - drivers/gles2/rasterizer_gles2.cpp:1904, memnew type: Surface
        0x3b733f0 - 256 bytes - core/vector.h:283, memalloc.
        0x47fc280 - 16 bytes - MID::Data
        0x3eb51f0 - 28 bytes - core/os/memory_pool_dynamic_static.cpp:86, memalloc.
        0x4850480 - 16 bytes - MID::Data
        0x3f05470 - 36 bytes - core/os/memory_pool_dynamic_static.cpp:86, memalloc.
        0x48a9c40 - 16 bytes - MID::Data
        0x3eb5190 - 52 bytes - core/os/memory_pool_dynamic_static.cpp:86, memalloc.
        0x48023a0 - 16 bytes - MID::Data
        0x43bc180 - 52 bytes - core/os/memory_pool_dynamic_static.cpp:86, memalloc.
        0x48a8640 - 56 bytes - drivers/gles2/rasterizer_gles2.cpp:1848, memnew type: Mesh
**ERROR: STATIC ALLOC: End of Report.

Looks like the Quad node makes some mesh in VisualServer, but fails/forgets to free it.

The mesh is created here: https://github.com/godotengine/godot/blob/master/scene/3d/quad.cpp#L228
Seems like VisualServer doesn't clean meshes of freed instances (which is very, very good :laughing:), but, this mesh specifically is used only by this Quad node (not shared). That's why it should free this RID in the yet nonexistent Quad::~Quad destructor, like this (probably):
Quad::~Quad() {
  VisualServer::get_singleton()->free(mesh);
}


Ll 6594/testing

Add 100% unit testing coverage to SDK.
CONTEXT
LL-6594

@IAmMorrow @achabrolin-ledger We now have to decide what % of code coverage is required (actually 100%) so I can update the yarn test to fail if the % is under that 👌

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@32cb303). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #13   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         7
  Lines           ?       151
  Branches        ?        24
========================================
  Hits            ?       151
  Misses          ?         0
  Partials        ?         0

Continue to review full report at Codecov.

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

Looks good to me 🚀
Regarding % of test coverage needed, we can keep 100% as of today and adjust as we go if need be

👍
Squashed your typo fixes to avoid a "fix blablabla" commit, hope you don't mind !


add flexbug 15.

Why would I use clearfix on flex parent you may ask.
I won't but this issue mainly arises when using flexbox with the CSS frameworks like Bootstrap.
Since they have clearfix applied to the row class and when we try using row as flex container we get inconsistent behavior across browsers. Chrome and Firefox don't seem to have any issue but Safari has this issue.
Apologies for the bad grammar.
Sources: https://danisadesigner.com/blog/flexbox-clear-fix-pseudo-elements/

I'm going to close this PR as there's been no response and it's now out of date. Feel free to open a new PR if you still want to add this.


docs(layout): Add additional layout examples

Add layout examples showcasing the usage of different parameters akin to
how we do for Grid.

Just documentation changes, merging


Move Call Function

I try to use this code to mimic the transaction.
def mint_nft():
    function_name = "mintNFT"
    module = "collection"
    package = "0x04c784fea238ea44d8b01c8f9e67857dc16b91e857dc7204c5800367f1694275"
    args = ["Input(1)", "NestedResult(0, 0)"]
    test_address = "0x6c83210160289da5a92b20cc1461d5d7470a06394d5bba41ac432355438863f0"

    call_sui_contract_function(test_address, package_object_id=package, module=module, function_name=function_name,
                               arguments=args, type_arguments=[], gas="1200")

mint_nft()

def call_sui_contract_function(signer: str, function_name: str, arguments: List[str],
                               type_arguments: SuiArray[SuiString],
                               package_object_id: str, module: str, gas: str = "1000"):
    """
    Call SUI contract function.
    """
    move_call_builder = MoveCall(
        signer=SuiAddress(signer),
        package_object_id=ObjectID(package_object_id),
        module=SuiString(module),
        function=SuiString(function_name),
        arguments=SuiArray([SuiString(p) for p in arguments]),
        type_arguments=type_arguments,
        gas=ObjectID(gas),
        gas_budget=SuiInteger(int(gas))
    )
    result = sui_client.execute(move_call_builder)
    print(result)

But error: 'byte deserialization failed, cause by: AccountAddressParseError'
Not sure how to work with this case.

Ok, so this is not a pysui error but in how you are calling the function.  In explorer it is saying that the first argument should be an address and the second argument to the entry point should be a vector of coin objects addresses and you are passing two strings in.

I see. We are ETH devs.
Trying with Move.
May I ask more questions?

For the explorer, I should call a batch transaction including SplitCoin and MoveCall?
For the Inputs of explorer, what is this and how we call it with pysui?

Thank you.

You should ask that in the Sui discord Move channel. I'm not sure what you are trying to achieve?

Very simple. We want to build a minting page like this: "https://suitestnet.souffl3.com/" for our NFT project. Rightnow, we did not find the source code, just first phase integrate with entry points, move function and see how it goes.

So, pysui is better suited to server-side execution or desktop/command line applications. For full web (web 3,0) you may be better off with the Sui Typescript library or Wallet extension which offer all the same (perhaps renamed) capabilities of pysui

I understand what you mean. But we need some works with pysui also. I have updated my latest code:
wallet_address = "0x6c83210160289da5a92b20cc1461d5d7470a06394d5bba41ac432355438863f0"
    module = 'TOKEN'
    function = ["mint_test_token_usdt"]
    gas = 1000
    package = "0xe158e6df182971bb6c85eb9de9fbfb460b68163d19afc45873c8672b5cc521b2"
    arg_0 = "0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12"

    for f in function:
        move_call_request = MoveCallRequestParams(
            package_object=ObjectID(package),
            module_str=SuiString(module),
            function_str=SuiString(f),
            arguments=SuiArray([SuiString(arg_0), SuiString(wallet_address)]),
            type_arguments=SuiArray([]),
        )
        batch = BatchTransaction(
            signer=SuiAddress(wallet_address),
            transaction_params=SuiArray([move_call_request]),
            gas=ObjectID(""),
            gas_budget=SuiInteger(int(gas))
        )

        result = sui_client.execute(batch)
        print(result)

But still error: "But error: 'byte deserialization failed, cause by: AccountAddressParseError'" - There are some error within _validate_builder() function. Can you check this use case?

What version pysui? I performed the following using the pysui repo (currently at v 0.16.0)
I did not get the error you are getting but there is an error from the runtime which is a Sui error reporting on the shared object. See end of message.
I have found that I can 'dry run' the routine without error. Herei s the source
def debug_issue():
    """."""
    cfg = SuiConfig.default()
    client = SuiClient(cfg)
    # wallet_address = "0x6c83210160289da5a92b20cc1461d5d7470a06394d5bba41ac432355438863f0"
    wallet_address = cfg.active_address.address
    module = "TOKEN"
    function = ["mint_test_token_usdt"]
    gas = 10000000
    package = "0xe158e6df182971bb6c85eb9de9fbfb460b68163d19afc45873c8672b5cc521b2"
    arg_0 = "0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12"

    for f in function:
        move_call_request = MoveCallRequestParams(
            package_object=ObjectID(package),
            module_str=SuiString(module),
            function_str=SuiString(f),
            arguments=SuiArray([SuiString(arg_0), SuiString(wallet_address)]),
            type_arguments=SuiArray([]),
        )
        batch = BatchTransaction(
            signer=SuiAddress(wallet_address),
            transaction_params=SuiArray([move_call_request]),
            gas=ObjectID("0xb4bdf98b6d754ed1a1538604e13d97f67b678651023a30937170beb662a63843"),
            gas_budget=SuiInteger(gas),
        )

        # result = client.execute(batch)
        result = client.dry_run(batch)
        if result.is_ok():
            print(result.result_data.to_json(indent=2))
        else:
            print(result.result_string)

Here are the results:
{
  "effects": {
    "status": {
      "status": "success",
      "error": null,
      "succeeded": true
    },
    "messageVersion": "v1",
    "gasUsed": {
      "computationCost": "1000000",
      "storageCost": "4180000",
      "storageRebate": "1813297"
    },
    "transactionDigest": "233WsorZx2VXHJdZQdD6CwaT2oUQJ1TMA6pABrt2Hefx",
    "gasObject": {
      "reference": {
        "objectId": "0xb4bdf98b6d754ed1a1538604e13d97f67b678651023a30937170beb662a63843",
        "version":<PHONE_NUMBER>854775807,
        "digest": "9yci7offWfhsAQ3uCeeco1zw9ZNgoe9YZRRz38Lh1d8"
      },
      "owner": "0x112c6fe78f3eb2ad1829ffdd7939d6e06be3e67bf5a7675101a7497b7d42ff20"
    },
    "modifiedAtVersions": [
      {
        "objectId": "0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12",
        "sequenceNumber": 2723281
      },
      {
        "objectId": "0xb4bdf98b6d754ed1a1538604e13d97f67b678651023a30937170beb662a63843",
        "sequenceNumber": 14793
      }
    ],
    "dependencies": [
      "8eKoCbFBabQf2NqLxbVZ16zEE4pjMe9xeSZttzM9NZvm",
      "Fr3HiKskZhvDcBwGt148MsyKJ4qxqKcZ1PPYpzGfEi78",
      "GhxzmJfZokYqwb9xrMQZBp1TDRSpjCbKaQrXpsNpjDJ8"
    ],
    "mutated": [
      {
        "reference": {
          "objectId": "0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12",
          "version":<PHONE_NUMBER>854775807,
          "digest": "58uSbcuovf4391Hw9YKoV7VHgLZnYsNjjFaEtXjC3X22"
        },
        "owner": {
          "initial_shared_version": 14220
        }
      },
      {
        "reference": {
          "objectId": "0xb4bdf98b6d754ed1a1538604e13d97f67b678651023a30937170beb662a63843",
          "version":<PHONE_NUMBER>854775807,
          "digest": "9yci7offWfhsAQ3uCeeco1zw9ZNgoe9YZRRz38Lh1d8"
        },
        "owner": "0x112c6fe78f3eb2ad1829ffdd7939d6e06be3e67bf5a7675101a7497b7d42ff20"
      }
    ],
    "created": [
      {
        "reference": {
          "objectId": "0x6f8a1780965f363874d1a8d3f501bd6dddd986d4bdbfaa70fc07299736a30dcb",
          "version":<PHONE_NUMBER>854775807,
          "digest": "Brwbr5YiVa4hmvsjQwk3WujgJfpnNHiBGNo45q5ZFm1z"
        },
        "owner": "0x112c6fe78f3eb2ad1829ffdd7939d6e06be3e67bf5a7675101a7497b7d42ff20"
      }
    ],
    "deleted": [],
    "wrapped": [],
    "unwrapped": [],
    "unwrappedThenDeleted": [],
    "sharedObjects": [
      {
        "objectId": "0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12",
        "version": 2723281,
        "digest": "E89j1v5SmiGPBL6rUPHA4nnS43gjzBRek9eDhendkgn7"
      }
    ],
    "executedEpoch": "752",
    "eventsDigest": 0
  },
  "events": []
}

However with a client.execut(...) I get
{'code': -32000, 'message': 'Error checking transaction input objects: ObjectSequenceNumberTooHigh { object_id: 0x4d511e8e21f0b2d2f8ae3aa4dc0ea1efb41e65d78ab1f42869f13d527effae12, asked_version: SequenceNumber(2723405), latest_version: SequenceNumber(2723402) }'}

I use 15.0. After updating and using your code, I understand why it show AddressParserError.
The reason is I set gas = ObjectId("").
And "" is not a address.
Such a great help. Thank you.
Last question, for this sample:

I need to send a batch transaction with SplitCoins and MoveCall right?
For NestedResult(0,0) -> what should I input in PySui? (SuiString(""))?

Oh I just found that there is no "SplitCointsRequestParams" implemented yet.

There is no plan from MystenLabs to add one. Instead they're pushing Programmable Transactions which are much more flexible than the unsafe Batch.
I release 0.16.0 of pysui today which has a working Programmable Transaction capability.


Add option to show desktop navigation toolbar on desktop click

Allow the desktop navigation toolbar to come to the top if the desktop is clicked, but make it a preference disabled by default.

This is annoying so I don't think I will add this.


mypy: 0.812 → 0.910

types-toml: Init at 0.1.5
mypy: 0.812 → 0.910

Motivation for this change

types-toml: Init at 0.1.5
type annotations for toml, a new dependency of mypy.

mypy: 0.812 → 0.910
New versions of mypy include significant changes:

Switch to modular typeshed
TypeGuard support
Configuration via pyproject.toml

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests) N/A
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking

[x] Fits CONTRIBUTING.md.

Issues
@lnl7 @martingms I am having issues getting this to build, as mypy apparently
doesn't find the type hints for the toml module, even though it should be in
$PYTHONPATH.  Any hint (^^') how to proceed?

Related: https://github.com/NixOS/nixpkgs/pull/130173

@mweinelt Thanks, not sure how I missed that.
@veehaitch, would you like shared maintainership of types-toml and types-typed-ast, if/when I figure out what's broken here?

@veehaitch, would you like shared maintainership of types-toml and types-typed-ast, if/when I figure out what's broken here?

Very much, yes!

I see that here and in https://github.com/NixOS/nixpkgs/pull/130173 it looks like change is completed and appears the checks have passed, but PR is still listed as a draft? Is there some kind of issue that prevents this or https://github.com/NixOS/nixpkgs/pull/130173 to be merged?

Just noticed the comment that repo maintainers can update a draft to a PR, @lovesegfault or @jonringer I saw that you merged prior updates to this package. Can you please look at this or https://github.com/NixOS/nixpkgs/pull/130173 and merge it? Recent changes to mypy don't allow to automatically bump version anymore.

I think this can well stay a draft until the issues the author noticed themselves are resolved.

Closes https://github.com/NixOS/nixpkgs/issues/139319

I see that here and in #130173 it looks like change is completed and appears the checks have passed, but PR is still listed as a draft? Is there some kind of issue that prevents this or #130173 to be merged?

As @mweinelt noted, there's a couple of remaining issues which are why I left this as draft for now.
I've been delving in mypy annotation-finding code to try and debug this, but no success so far.

@nbraud Opened a new PR. Hope you don't mind. Check if issues are resolved. I've integrated @jnetod additions. Feel free to add yourself as maintainer (in a new PR).

Closing in favor of #140627


Copy bags between disks fix

copy_bags_between_disks.py: code consolidation #4775

Don't hard code upon python interpreter version
Use capital letter for consistent value

Thanks for the great work! Could you sign the CLA so that we can merge the code?


Change margin for linenumber

Using non-defaults fonts doesn't show whole label for some reason. If change margin for label from 3(default label value) to 0 this behaviour change. Default font also keeps look good.
Before :

After :

But I don't see any reason keep use this overloading and would prefer apply this change to base class in mixins since there aren't any significant differences.
Type

[ ] Bug Fix
[ ] New Feature
[x] Quality of Life Improvement

Tests

I have tested locally against:

[x] Windows
[ ] linux
[ ] OS X
[ ] BSD

I have included automated tests:

[ ] Unit tests
[ ] Integration tests
[ ] End to end tests

I have no idea why the first test run failed; this looks good to me.


Sharing a group using "New role from Selection" is faling

Steps

Create a group
Share the group by selecting "New role from selection"

Issue
The operation fails to complete. The below message is shown in the UI.

Labels - 3.1.0-M7, bug blocker


shfmt: 2016-06-15 -> 2016-06-16

Motivation for this change
Update to a version that fixes an annoying bug
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Among other improvements the author fixed mvdan/sh#13

In case anyone was worried, I don't plan on updating this every day. This particular bug fix makes the tool able to parse code of mine that it was previously choking on.


Preemptive Priority Process Scheduling algorithm | C

Title - Preemptive Priority Process Scheduling algorithm
what will change - Adding an Operating Systems scheduling algorithm- Preemptive Priority Process Scheduling algorithm implemented using C programming.
Type of Issue - Adding Preemptive Priority Process Scheduling algorithm.
Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Code
[x] Improving Documentation
[x] Bug Fix

Programming Language

[x] C

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding,
Please assign it to me. I have started working on it. Thank You :)

Please assign it to me. I have started working on it. Thank You :)


[dailymotion] unable to download playlist

[X] I've verified and I assure that I'm running youtube-dl 2018.06.25

Before submitting an issue make sure you have:

[X] At least skimmed through the README, most notably the FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones
[X ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

latest version doesn't want to download any playlist from dailymotion, shows "downloading 0 videos"
$> youtube-dl --version
2018.06.25
$> youtube-dl -v "https://www.dailymotion.com/playlist/x42uxl"
[debug] System config: []
[debug] User config: [u'--format', u'best', u'--external-downloader', u'aria2c', u'--external-downloader-args', u'-x 5']
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.dailymotion.com/playlist/x42uxl']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.06.25
[debug] Python version 2.7.12 (CPython) - Linux-4.4.0-128-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 3.4.2-1, ffprobe 3.4.2-1
[debug] Proxy map: {}
[dailymotion:playlist] x42uxl: Downloading webpage
[download] Downloading playlist: SURF
[dailymotion:playlist] x42uxl: Downloading page 1
[dailymotion:playlist] playlist SURF: Downloading 0 videos
[download] Finished downloading playlist: SURF

Same issue here, no error message displayed.

Same issue with:
$ youtube-dl --version
2018.07.21

$ youtube-dl https://www.dailymotion.com/playlist/x57eub
[dailymotion:playlist] x57eub: Downloading webpage
[download] Downloading playlist: Liste de Lionel
[dailymotion:playlist] x57eub: Downloading page 1
[dailymotion:playlist] playlist Liste de Lionel: Downloading 0 videos
[download] Finished downloading playlist: Liste de Lionel

But the playlist is not empty. The same command (with version 2018.02.08 of youtube-dl) on the same playlist worked in March, 2018.


Platforms - UI: learn more links

Issue:

bcgov/entity/issues/24508

Description of changes:

add learn more links
downgrade nuxt (3.14 seems to have some issues)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://strr-platform-dev--pr-404-sna2zjrw.web.app


[test] TCP echo test does not test if echoes were actually completed

Description
It is possible for the tcp echo test to not actually echo any packets. If the client successfully connects to the server kernel but then the kernel closes the connection, either the test will exit successfully or it will hang if the kernel does not close all connections. In either side the server side tcp echo application will likely hang because it did not get any connections from the OS kernel.
How to Reproduce
Steps to reproduce the behavior:

OS kernel is up but not passing connections on to the application
TCP echo client gets successful connects to all clients
Server-side kernel closes the connections
TCP echo client exits if all client connections are closed, otherwise hangs.

Documentation
This line in the TCP echo test is the problem (same between sequential and concurrent):

We are not seeing this line print out on the server and we should if the client actually connects/exits without error:

We should probably add some more application level traces, for example in each of the issue_* and handle_* functions.
Expected Behavior
If the server-side application is not echoing data back, the test should detect that.

@iyzhang could you help me elaborating on the following topics:

Why would the OS kernel not be passing connections to the applications?
Why would server-side kernel close connections, even without receiving any data?
If the server is not echoing back packets, how would TCP-echo client exit?
What is the buggy line that you wanted to highlight?


[DNM] ci: test trigger ci

What problem does this PR solve?

Issue Number: close #xxx
Problem Summary:
What is changed and how it works:
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

/test pull-mysql-test

/test ?

/test ?

/lgtm

/lgtm

/test ?

/test ?

/test pull-mysql-test

/test pull-mysql-test

/test ?

/test pull-integration-common-test

/test pull-integration-jdbc-test

/test pull-mysql-test

/retest

/retest

/test ?

/test pull-integration-ruby-orm-test

/test pull-integration-common-test

/test pull-integration-jdbc-test

/test pull-integration-jdbc-test

/test pull-integration-jdbc-test

/test pull-mysql-connector-test

/test pull-integration-ruby-orm-test

/test pull-mysql-connector-test

/test pull-mysql-connector-test

/test ?

/test ?

/test all

/test pull-integration-python-orm-test

/test pull-integration-prisma-test

/test pull-integration-sequelize-test

/test pull-integration-python-orm-test

/test pull-integration-prisma-test

prisma test will be fixed in https://github.com/pingcap/tidb/pull/46428

/test pull-integration-sequelize-test

/test pull-integration-sequelize-test

/test pull-integration-sequelize-test

/test pull-integration-python-orm-test

/test pull-mysql-test


Latent Consistency Models support would be great in this project

In the last week, there has been a lot of talk about a new type of model Latent Consistency Models that significantly improves the performance and generation of stable diffusion with fewer steps.
It apparently works with a LoRA adapter that can be applied to any existing model. I'm not sure if there are any specific changes to the UNet architecture that need to be made, but what needs to be done is adding a new sampler LCM Solver.
After completing the CUDA acceleration support, which is almost finished, I will see if I can work on adding LoRA support. This will require a complete change in the current project structure. Following that, I'll add the new solver and conduct the necessary tests.

Nice to know we're thinking alike! I just added support for the Latent Consistency Models Sampler, as mentioned here: https://github.com/leejet/stable-diffusion.cpp/discussions/76#discussioncomment-7600422. Currently, I'm working on implementing support for lora, similar to sd-webui. Once that's done, lcm-lora will also be available. It's almost complete, and I expect to finish it in the next day or two.

Nice to know we're thinking alike! I just added support for the Latent Consistency Models Sampler, as mentioned here: https://github.com/leejet/stable-diffusion.cpp/discussions/76#discussioncomment-7600422. Currently, I'm working on implementing support for lora, similar to sd-webui. Once that's done, lcm-lora will also be available. It's almost complete, and I expect to finish it in the next day or two.

I just wanted to talk to you about changing the project structure to fully support the gguf format and applying the structure used by other projects that use ggml, like llama.cpp and whisper.cpp. This should allow having a general struct to decouple each component.
I have also conducted research on LoRA that could be helpful to you for both LoRA and SDXL support. Please review the code of my pull request.

@FSSRepo I've replied within your pull request. Sorry for the delayed response, I've been busy with work recently.

lcm-lora will also be available. It's almost complete, and I expect to finish it in the next day or two.

I just added LoRA support, now we can specify the use of LCM-LoRA through the prompt. Any SD1.x model should be adaptable.
https://github.com/leejet/stable-diffusion.cpp#with-lora
Here's a simple example:
./bin/sd -m ../models/v1-5-pruned-emaonly-ggml-model-f16.bin -p "a lovely cat<lora:lcm-lora-sdv1-5:1>" --steps 4 --lora-model-dir ../models -v --cfg-scale 1

I just added LoRA support, now we can specify the use of LCM-LoRA through the prompt. Any SD1.x model should be adaptable.
https://github.com/leejet/stable-diffusion.cpp#with-lora

I will adapt your code for the PR

I will adapt your code for the PR.

Since you opened the PR, there have been several updates in my master branch. I'd appreciate it if you could rebase your branch onto my master to integrate these changes, rather than straightforwardly copying the code. This will streamline conflict resolution when merging your PR.

I will adapt your code for the PR.

Since you opened the PR, there have been several updates in my master branch. I'd appreciate it if you could rebase your branch onto my master to integrate these changes, rather than straightforwardly copying the code. This will streamline conflict resolution when merging your PR.

No problem!

@leejet Why use these asserts? Since sizeof(nb[0]) measures the size of the type, which is size_t and is 8, it is not the same as float.
static void asymmetric_pad(struct ggml_tensor* dst,
                               const struct ggml_tensor* a,
                               const struct ggml_tensor* b,
                               int ith,
                               int nth,
                               void* userdata) {
        assert(sizeof(dst->nb[0]) == sizeof(float));
        assert(sizeof(a->nb[0]) == sizeof(float));
        assert(sizeof(b->nb[0]) == sizeof(float));

Closed by #75 (full tested LoRA Loader)

Closed by https://github.com/leejet/stable-diffusion.cpp/pull/75 (full tested LoRA Loader)


improve hpa config

Fixed and extended example/links regarding HPA config

pushed a fix

Many thanks @yurymuski !


Custom resources don't appear in "quick load"

Godot version
v4.0.beta2.official [f8745f2f7]
System information
Windows 11, GLES3, NVIDIA GeForce GTX 1650, i5-11300H
Issue description
Quick load doesn't detect custom resources
Steps to reproduce

Create a custom resource with a class name
Create a resource of the type you created
Create a new node and attach a script. To that script add an export variable of the same type as the resource you created
Right click -> Quick load, you'll see the list doesn't contain your resource

Minimal reproduction project
Example.zip

This is expected for now as a result of #66215, which disabled QuickOpen custom resource loading because a refactor is needed for editor data structures to make it work without being prohibitively slow.

Oh I see. Should I close this issue then?

No the issue can stay open as a reminder that this feature is missing.

This issue was fixed few betas ago.

This issue was fixed few betas ago.

I'm still experiencing this issue in both 4.0 and 4.0.1

@Cratesmith Please provide a minimal reproduction project.
I tested the one attached in this issue and Resources need a script_class field to appear in Quick Load. If you have resources created before this issue was fixed, you need to re-save them.

It was fixed for me but I noticed that when I deleted the .godot folder the resources disappeared again

I'll make a minimum project for it tomorrow, but I've actually never had the resources appear in quickload. In my case, I haven't deleted the .godot folder to get this effect.
Sent from Proton Mail mobile
-------- Original Message --------
On 26 Mar 2023, 12:46 am, ben wrote:

It was fixed for me but I noticed that when I deleted the .godot folder the resources disappeared again
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Same issue here, in Godot 4.1 custom resources still don't show in "Quick Load"

In Godot 4.1 this still happen randomly, it seem like quick load does not recognize old custom resource, and I have to recreate some resource to make it work, not sure what is the cause of this issue maybe some cache file system

addition info, let's say we have 2 custom resources in the project "A" and "B"

when I recreate "A" the quick load will only recognize "A" ("B" is disappear from search result, you cannot find them event type the name)
then I recreate "B" now quick load will only recognize "B" and "A" is gone from search result

These issues will gone when removing .godot folder and then re-import the project

From my experience it will re-appear.

Same here. Godot 4.2.1
Custom resources:

Quick load menu:

It seems that .godot/editor/filesystem_cache8 doesn't reference the rest of the resources. Deleting this file doesn't solve the problem.
Why is this issue closed?

So I made an editor plugin with a button that resave all my custom resources, so they appear again in the quick load menu:
@tool
extends Button

func _on_pressed():
	var file_paths = get_all_file_paths("res://resources")
	resave_resources(file_paths)

func get_all_file_paths(path: String):
	var file_paths = []
	var dir = DirAccess.open(path)
	dir.list_dir_begin()
	var file_name = dir.get_next()
	while file_name != "":
		var file_path = path + "/" + file_name
		if dir.current_is_dir():
			file_paths += get_all_file_paths(file_path)
		else:
			file_paths.append(file_path)
		file_name = dir.get_next()
	return file_paths

func resave_resources(file_paths):
	for path in file_paths:
		if ResourceLoader.exists(path):
			var resource = ResourceLoader.load(path)
			ResourceSaver.save(resource, path)
			print("Re-saved: " + path)

I'm having the same issues as @luislodosm, only some of the custom resources are shown in the Quick Load. I think this issue should be reopened.


Feature/blazor client aspnet identity

[x] Create Blazor client app with ASP.NET Core Identity Support
[x] Make User entity inherit IdentityUser
[x] Separate login views for Student/Teacher
[x] Add GetAllCourses endpoint to enable students to pick course when registering account
[x] Persist custom attributes for Student/Teacher with custom Userstore
[x] Authorize all route views except Account Pages (Login/Register)

Currently a user is not automatically logged upon registration, but I thought that might be out of scope for this - already quite big - PR.
THE PR also uses the Identity default UI (stylings, layout etc.), as that is also better addressed in a UI/UX related task.