Implements JSON parser in pure scopes.

Description
Attempts to resolve #66.
This adds a JSON parser written in scopes, will need to implement solidus / reverse solidus but all the other core features are available. More testing needs to be written and need to write better access methods, @aiverson's implementation uses a json-match function I believe.
Implementation Overview
See my write-up here.
Summary
Needs some further review and better testing but core functionality is present.

You broke the entire API and removed every convenience feature while adding a broken parser. It heap allocates tiny objects on the heap multiple times per character of input and segfaults on reasonably sized input. The Lexer and parser architecture is very bad. This needs to be rewritten completely.

Firstly I'm sorry, the point of this was not to merge yet I thought I set this as a merge draft, I wanted to get the lexer parser implementation done before I moved on to readding the previously implemented features. I had tested on a variety of inputs but I suppose not enough. I'll look at it again and send some questions. Thank you.


Add capability for only the whitelisted ips to connect to the subscriber

Since the subscriber is closely tied to the vm node and if the subscriber stops running, the vm node will as well. On top of this, anyone can connect to the subscriber right now. It makes sense to add some kind of whitelisted ips options so only the authorized ips can connect to it.

This is now enabled


DOC: more detail for input shapes and numpy

This is a small doc update to clarify the byte ordering and indexing of the input data files.
Perhaps more controversial: I point out that unless you are running one of the configs where you need to share files, there is really no need to use big-endian.  I'm sure the performance hit is minimal, but swapping across 2read/write cycles isn't nothing, and it makes the analysis code more verbose than it needs to be. Happy to take that out, but I've run little-endian for 15 years with no actual issues.

thx @jklymak LGTM, this is really helpful.
Can you try doing a merge master (following procedure at end of doc section 5.2.1? I'm not sure why the readthedocs check is not going through (@jahn ?)

@jrscott Looks like all the CI tests passed, including the readthedoc.

@jm-c that's a relief, I was  impatient :)
I think we can merge.

Thanks for the extra commits - this seems good to me.

Will merge this PR tomorrow, unless someone wants to add something.


AllowDebug false in policy.toml

Description
When building in release mode for hardware mode, we should generate a policy.toml file that does not allow SGX debug mode.
This probably requires changes to the rust code in order to launch the enclave in non-debug mode.
Motivation and Context
The hardware docker image we publish on dockerhub has no reason to have SGX debug mode on.
We should add a build option / environment variable to generate allow-debug policy files.
Test plans
Either

wait for the SGX-enabled CI runner and write a test
run on our own SGX-enabled runner, write a test for hardware mode, which will get ignored by the non-SGX runner

This is a good opportunity to add the following tests:

client does not allow debug mode enclave if allowDebug = false in policy.toml

Additional Information
none
Checklist

[ ] This issue concerns BlindAI Client
[x] This issue concerns BlindAI Server

Noted and accepted. Applying this won't ask for much changes. As we are only supporting SGX+FLC and higher, we can disable debug without drawbacks.


Fix name resolution issue

Motivation
GitLab enforces ipv6 now, however LocalStack has issues with it. So we need to work this around by getting the IPv4 address of the docker service.
Changes

use getent to get IPv4 address

By testing the change in GitLab and turning the pipeline green proves it's correctness. Merging without review.


Difference or commonality with gpcs4 ?

Hello, I find it difficult to express my thought in English without an interpreter, (because I am Russian), but I would like to ask,

are you referring to the GPCS4 code?

Is it possible to integrate the GPCS4 code to speed up your software development?

Is it possible to integrate your project with spine ?

I apologize if my questions seem inappropriate to you.
I'm not a developer.
But I even made a github for the sake of my question.
Why did you name your project Kyty?
Is it related to "whales" (animals)?
or is it a reference to whales - "rich people" ?

Close this issue and move to discussions.


Can one CL file be mapped into several different binary file name?

Hello,
We would like to load *.cl source files at runtime, save the compiled binaries into specific files, and load the binaries when our app is run next time.
This feature seems important since we have experienced long compilation delay when we first run a complex model file.
If we can save and load binaries, we can just load them when our app is run. This, I hope, will improve user experience a lot.
One issue I am thinking while looking into ACL code is that current ACL uses only one binary name for one cl source code. (e.g., fill_border.cl for cl source code, and fill_border.clbin for cl binary file)
https://github.com/ARM-software/ComputeLibrary/blob/master/src/core/CL/CLKernelLibrary.cpp, line 904, 52ba29e936b8e711e8acdfe819e36f884d4f3fe1)
However, depending on compilation option, one cl source code can be compiled into many binary files.
For example, while we run a model, we found that fill_border.cl is compiled with the following options and they, I guess, will be compiled into two different binaries.
-DBORDER_SIZE_BOTTOM=1 -DBORDER_SIZE_LEFT=0 -DBORDER_SIZE_RIGHT=1 -DBORDER_SIZE_TOP=0 ...

-DBORDER_SIZE_BOTTOM=0 -DBORDER_SIZE_LEFT=0 -DBORDER_SIZE_RIGHT=10 -DBORDER_SIZE_TOP=0 ...

So, I wonder how I can load these different binary files with current implementation of CLKernelLibrary.
Of course, we can modify ACL for our app but ACL runtime in our platform could be common runtime for many apps. So we would like to use original ACL.
If this feature is not available for the current version, would you add this for later version?
Thank you!

Hello @labsurde,
Yes this is valid and important requirement. We will have a look into adding support for this as soon as possible. Thanks for raising.

@GeorgeARM, Thank you! Hope to see these features soon!

@labsurde : What you can do is: configure all the functions you're planning to run, then save the CLKernelLibrary cache to file. Then the following time you can just re-load the cache in the CLKernelLibrary and the kernels shouldn't be recompiled:
Here is how to save the cache to file:
            printf("Saving kernels in cache\n");
            std::ofstream out("cache.bin");
            for(auto it: CLKernelLibrary::get().get_built_programs())
            {
                std::vector<std::vector<unsigned char>> binaries = it.second.getInfo<CL_PROGRAM_BINARIES>();
                ARM_COMPUTE_ERROR_ON( binaries.size() != 1);
                out << it.first.length() << "," << binaries[0].size() << "," << it.first;
                out.write( (const char*) binaries[0].data(), binaries[0].size());
            }

And here is how to re-load it:
            printf("Loading kernels from file\n");
            std::ifstream in("cache.bin");
            while(!in.eof())
            {
                size_t name_len=0, binary_len=0;
                char comma;
                in >> name_len >> comma >> binary_len >> comma;
                if( name_len == 0)
                    break;
                std::vector<char> tmp(name_len);
                std::vector<unsigned char> binary(binary_len);
                std::string name;

                in.read(tmp.data(), name_len);
                name.assign(tmp.data(), name_len);
                tmp.resize(binary_len);
                in.read((char*)binary.data(), binary_len);
                cl::Context context = CLScheduler::get().context();
                cl::Program::Binaries binaries { binary } ;
                std::vector<cl::Device> devices = context.getInfo<CL_CONTEXT_DEVICES>();
                cl::Program program( context, devices, binaries);
                program.build();
                CLKernelLibrary::get().add_built_program(name, program);
            }

Please note this is just a proof of concept, in a production environment you would probably want to use a  binary file format with some checksums, etc.
Hope this helps,

Hi @labsurde
Please take a look at this patch implementing the solution described above: https://review.mlplatform.org/#/c/ml/ComputeLibrary/+/1488/
Hope this helps.


The type RegCloseKeySafeHandle generated for RegLoadAppKey() is not compatible with RegistryKey.FromHandle()

Discussed in https://github.com/microsoft/CsWin32/discussions/440

Originally posted by efraimbart November  5, 2021
The type RegCloseKeySafeHandle which is generated as the out for RegLoadAppKey() does not inherit from SafeRegistryHandle and is therefore not compatible with RegistryKey.FromHandle().
I'm kinda new to the whole Win32 API, so any help to figure where I should be going from here would be greatly appreciated!

I see that SafeRegistryHandle is implemented to call RegCloseKey to release the handle, so that meets the purpose. We should be able to reuse that .NET type rather than generating our own.


Revert "Static alloc for hybridblock"

Reverts apache/incubator-mxnet#11313
My veto has been ignored and thus I'm allowed to revert this PR. See https://www.apache.org/foundation/voting.html for reference.

it seems p

My as well as Thomas' concerns have not been addressed.

If you have specific concerns, raise them. Your concern of lack of test coverage has been addressed, and you are abusing your right as committer to revert good changes back...

https://github.com/apache/incubator-mxnet/issues/11171 has not been addressed
Pedros review in https://github.com/apache/incubator-mxnet/pull/10817 has not been addressed
Thomas comment https://github.com/apache/incubator-mxnet/pull/10817#issuecomment-396701926 has not been addressed

I think I have raised specific concerns (twice) that have gone unanswered, just in case they got lost in the flow, let me write them here again:

"good changes", I'm sure they are, but the problem with the previous PR was that it was actually making things slower due to a bug, hopefully now fixed. I think we cannot just rely on blind trust, being data driven is beneficial for everybody. Could we have a runtime of maybe just a MNIST training to show that the speed is actually improved and rejoice accordingly?
The new API says that static_shape cannot be used without static_alloc. Can we assert that static_alloc is set if users set static_shape to True to avoid undefined behavior?

Bonus ask:

a description of the PR to help reviewers, and users alike to appreciate the changes and guide them through the 1400 lines of code, that have barely any comments.
better documentation on the impact of the bulk_size arguments in the hybridize() function

This PR is already fixing the problems and added test coverages. If we just block this, how can we move forward?

This PR is already fixing the problems and added test coverages. If we just block this, how can we move forward?

@marcoabreu I have been intensively using CachedOp with static memory allocation this week. I also read the code this week. Other than the small bug reported previously, I don't find problems and it works very well with me in my PR. https://github.com/apache/incubator-mxnet/pull/10451
I observe the memory allocation behavior and it works exactly as I expected.
What are you looking for? If you are looking for test code, my entire PR can serve as test code. If you are looking for review, I have been reading the code for the entire week. I don't think it's reasonable to block other people's work for no reason.

@marcoabreu I think you should at least give people who understand the PR the opportunity to review it.

Exactly, that's all I'm asking for. Give the community enough time to review the PR and address their concerns before merging it.


Rewrite simplify_boundary plsql function as to avoid errors on pg14

I propose this rewrite of plsql simplify_boundary function as to avoid errors I got when using pg14 or pg15

Looks good, thanks.


🐞[BUG] Missing horizontal scrollbar in tables

Description

Occured when using Google Chrome at specific resolutions/zoom levels on E2E:
The horizontal scrollbar in tables disappears, which makes it difficult to scroll horizontally.
Current behaviour
Happens in parts table and quality notifications tables. Please check for other tables as well.
At 100% zoom level at 1920x1080 in Google Chrome:

At lower than 80% zoom level the horizontal scrollbar appears again:

Expected behavior

When it's possible to scroll horizontally in the tables, the horizontal scrollbar is always visible in every table in major browsers at regular zoom levels and resolutions.
Steps to reproduce the Bug

Use Google Chrome
Open Trace-X
Go to parts / quality notifications
Zoom in or out to see the effect of the scrollbar appearing/disappearing

Additionally the bottom of the table is cut off at a certain window size:

I can't scroll down further here and the last part is cut off.


With bootstrap config disable lastresort by default

When device is installed using a single-use EVE installer, the intended network configuration is available from the very first boot, even before device onboarding. It therefore does not make much sense to still keep using last-resort network config as a fallback.
In fact, lastresort is often causing issues, especially when network outage is outside of the EVE control (external network goes down). Changing network configuration from the intended state to lastresort not only cannot help in such cases, but it can also have negative side effects, such as:

prolonged outage: device will stay with lastresort if all DPCs are failing and it will only retest the intended config once per 10 minutes
confusing logs and events - customer might get a wrong impression that outage was caused by EVE when it unwantedly changed applied config (customer will blame EVE instead of troubleshooting outside networks)
break NIC pass-through - EVE will mark all ethernets for management which will prevent them from being assigned to apps (this can be triggered by a race between domainmgr deploying app and NIM falling back to lastresort at around the same time)

Instead of changing the default value of lastresort all across the board, the proposal is to at least change it for the case of device being installed using a single-use EVE installer (which is a new feature so this is not changing behavior for older workflows).
Signed-off-by: Milan Lenco<EMAIL_ADDRESS>

This change is definitely OK, but I wonder if we should also go the extra mile and change the default once we get controller connectivity (the same way that debug.enable.usb and debug.enable.vga are enabled until we get the configItems from the controller, when we apply 1) what's in the configItems and 2) of configItem is missing set to false/disabled.
Need to get feedback if folks thinks that is OK.


No difference in startuptime when profiling with dstein64/vim-startuptime

Now, let me prefix this by saying, that I might have misunderstood the aim of this project, or misconfigured something. What I'm seeing is the following:
When I take a look at LuaCacheProfile after consecutive startups after LuaCacheClear, This is the first startup after clearing:
    Resolve                  Load
   12.850ms │           │   36.882ms │           │ Total                                         │
...

and this is after the cache has been built (after second startup):
    Resolve                  Load
    3.816ms │           │    4.391ms │           │ Total                                         │
...

So there definitely seems to be a difference between cached and uncached startup. However, I didn't notice any difference in the startuptime, so I installed dstein64/vim-startuptime and took a look at StartupTime in between cached and uncached startups. The result: I'm averaging around 160mS and it doesn't seem to matter if I have a built cache or not.
I've installed impatient via packer. I set up impatient before any other code is run.
Looking at #31, I might lack some information on how to actually set this up or how the plugin works together with packer. Can you help me out here? Or is this simply not how it works?

Just ran it myself and saw a significant decrease in startuptime:

Looks like most of my time decrease is coming from packer_compiled.lua being cached.

 I misinterpreted the output of vim-startuptime. I thought it would reuse the current session. Thanks for looking into it! I Uninstalled impatient and took a look at StartupTime again. The jump wasn't as significant for me (190mS to 160mS), but still, it proves, that it actually has an effect on my startup time.
Thanks for the support and thanks for packaging this up!


The "default command experience" is not great

$ podman run --pull=newer ghcr.io/osbuild/otk
usage: otk [-h] [-v] [-W WARN] command ...
otk: error: the following arguments are required: command

Compare this with podman:
$ podman
Manage pods, containers and images

Usage:
  podman [options] [command]

Available Commands:
  attach      Attach to a running container
  auto-update Auto update containers according to their auto-update policy
  build       Build an image using instructions from Containerfiles
  commit      Create new image based on the changed container
  compose     Run compose workloads via an external provider such as docker-compose or podman-compose
...

We should at least show a brief summary of what otk does, and show the available commands. Basically mimic what https://github.com/spf13/cobra does by default.

I think you crossed #277 which does this change as a driveby.


Loading non-context-aware native module in renderer

(node:82420) Electron: Loading non-context-aware native module in renderer: '/Users/foo/repos/app/node_modules/usb/build/Release/usb_bindings.node'. This is deprecated, see https://github.com/electron/electron/issues/18397.

This should be resolved once #399 is merged.

I believe this issue is fixed, please reopen if it persists with the latest release.


.gitignore not being copied by extension creator

The /.gitignore file from the extension boilerplate doesn't get copied to the created extension, leading node_modules to be added to git.
This is because npm removes .gitignore files when publishing packages, meaning that even when globOptions.dot is true when calling copyTpl, the file won't be copied.
The recommended solution appears to be to rename the files to something else, such as gitignore and renaming them to .gitignore after copying them.

darn 😬


How to get the ABRP user token ?

Hello,
I'm unable to get or generate the user token following the following from iternio: https://github.com/iternio/ovms-link?tab=readme-ov-file#obtain-live-data-token (no Live Data button as explained).
The goal here is to fill the env var ABRP_USER_TOKEN.
How to get this token ?

You might need to request it by Email from ABRP team for your account. I sent an Email to<EMAIL_ADDRESS>with my Login email address and they provided me with the Key.

you can fetch a token yourself from under your car configuration go to live data from the car and fetch a generic token

I don't this live data button :/

@Chuckame if you are using an OBD dongle, you need to de-configure it first.
You can always re-add it later after you get the car token


Autosave

Currently, the content of a note is only saved when toggling the editor pane visibility.  This doesn't make as much sense now that the editor pane doesn't completely replace the rendering, so it's reasonable to leave it permanently visible.
Instead, the current content should be saved on the following events:

when navigating away from a particular note
when closing the application
when editing a note (at most every few seconds)

Fixed by a108624d49a987ff9c6708f096ae930a3e000630.


Wrong Attendee kit after december 2022 update

after priad update to december 2022 release (at the time of writing)
attendee localized kit is wrong:

docx instead of pdf

only documents not solutions

mysolution, presentation, setup folders are missing

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

ID: e0439d49-76b1-380b-f226-317e96ca0397
Version Independent ID: 2dd9f620-5244-936f-76f9-1f32f5ac5a23
Content: Power BI Paginated Reports in a Day course - Power BI
Content Source: powerbi-docs/learning-catalog/paginated-reports-online-course.md
Service: powerbi
Sub-service: powerbi-resource
GitHub Login: @kfollis
Microsoft Alias: kfollis

Thanks for your comment, @R1k91. This issue has been assigned to the doc author for review.

Assigning to the right writer for that type of content.

I removed the assignment. I don't own paginated reports @davidiseminger

Hi Maggie - might not be you, but dealing with paginated reports. If not you, maybe you know someone who could fix?

Hi, @R1k91 -- Hoping to find the current owner of this content. I'll let you know.

Thanks, it appears this issue has been resolved.


Update what-is-discovery.md

Microsoft should be changed to Contoso

@danielledennis
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


不能几个校园网用户同时连接服务器，当一个校园网用户连接后，其他用户就不能连接了。使用流量却可以多个用户连接。已经采用了多用户模式，也不行。

Please read the instruction and answer the following questions before submitting your issue. Thank you.
Please answer all the questions with enough information. All issues not following this template will be closed immediately.
If you are not sure if your question is truely a bug in V2Ray, please discuss it here first.

What version of V2Ray are you using (If you deploy different version on server and client, please explicitly point out)?

What's your scenario of using V2Ray? E.g., Watching YouTube videos in Chrome via Socks/VMess proxy.

What did you see? (Please describe in detail, such as timeout, fake TLS certificate etc)

What's your expectation?

Please attach your configuration file (Mask IP addresses before submit this issue).

Server configuration:
    // Please attach your server configuration here.

Client configuration:
    // Please attach your client configuration here.

Please attach error logs, especially the bottom lines if the file is large. Error log file is usually at /var/log/v2ray/error.log on Linux.

Server error log:
    // Please attach your server error log here.

Client error log:
    // Please attach your client error log here.

Please attach access log. Access log is usually at '/var/log/v2ray/access.log' on Linux.

    // Please attach your server access log here.

Other configurations (such as Nginx) and logs.

If V2Ray doesn't run, please attach output from --test.

The command is usually /usr/bin/v2ray/v2ray --test --config /etc/v2ray/config.json, but may vary according to your scenario.

If V2Ray service doesn't run, please attach journal log.

Usual command is journalctl -u v2ray.
Please review your issue before submitting.

这里面有一个疑问，你们的出口是同一个IP吗？

学校内网，出口应该是一个ip吧


Snippets does not work

Reproduce:
type input, click from menu Packages -> Emmet -> Expand abbreviation, it expands to <input></input> likewise with html:5 or others.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Apparently only non html files are affected, I use TypeScript and JSX it's pretty important to get the html snippets to work in all files where Emmet works too.

Please check DevTools console for possible errors

@sergeche there is no errors. Just a warning I don't have C:/Users/username/emmet directory. Btw, this error may be duplicate of #104

Can you send me package names you’re using for JSX and TypeScript?

@sergeche I don't think this has anything to do with TypeScript or JSX. The snippets just doesn't work right in non-html files. such as something.json, something.js or something.ts.
Also I don't have any special jsx package, I use atom-typescript and newly introduced JavaScript feature templating string (backticks) basically a multiline string for JSX.

I'm getting this same problem as well now.
My console:
Loading Emmet extensions from ~/emmet
/Users/spentacular/.atom/packages/emmet/lib/emmet.coffee:150 Emmet: no such extension folder: /Users/spentacular/emmet

I currently unable to expand anything besides the basic class or id structure. a results in , etc.

Same problem here.
Using for JSX syntax: https://atom.io/packages/react
Console warnings:
Loading Emmet extensions from ~/emmet
/Users/username/.atom/packages/emmet/lib/emmet.coffee:150 Emmet: no such extension folder: /Users/username/emmet```

Emmet version: 2.3.10
Atom: 0.201.0

I am also seeing this same behavior when using non-html grammers. When I am in HTML (Ruby - ERB) mode i can only expand basic htmls taga like a or input but more complex things like a.button fail to expand.
I am also seeing the same errors as other users. If I switch to just the plain HTML grammer everything works fine but then I lose my syntax highlighting for ERB.
Loading Emmet extensions from ~/emmet
/Users/patricklocal/.atom/packages/emmet/lib/emmet.coffee:150 Emmet: no such extension folder: /Users/patricklocal/emmet
``

Atom: 0.211.0
Emmet: 2.3.10

When I am in HTML (Ruby - ERB) mode i can only expand basic htmls taga like a or input but more complex things like a.button fail to expand.

See https://github.com/emmetio/emmet-atom#tab-key

Warnings like
Loading Emmet extensions from ~/emmet
/Users/patricklocal/.atom/packages/emmet/lib/emmet.coffee:150 Emmet: no such extension folder: /Users/patricklocal/emmet

...simply tells you where Emmet looks for custom extensions, it doesn’t break anything.

This problem has nothing to do with tab key, as I demonstrated, I clicked from a menubar: Packages -> Emmet -> Expand abbreviation - not using the shortcut (which btw is ctrl+enter on my mapping at the moment)
Either way, it still expands incorrectly "input" to "" in normal text file or anything else than HTML.

Should be fixed in v2.3.11

Passed two years, same problem.

@lid3rs it’s not a problem, it’s just a warning telling that you have no custom Emmet extensions

=)) Hi guys.... We can read last paragraph of readme.md this plugin
Extensions support
You can easily extend Emmet with new actions and filters or customize existing ones. In Preferences > Emmet, set Extensions path to folder with Emmet extensions. By default, it’s ~/emmet, e.g. emmet folder in your system HOME folder.
So...
This plugin try find this folder (~/emmet = :\Users<CurrentUser>\emmet - in Windows)
You can just create this folder (empty) ;)

=)) Hello guys! You can read the last paragraph readme.md of this plugin:

Support extension
You can easily extend Emmet with new actions and filters or customize existing ones. In the "Settings"> "Emmet" section, set the extension path to the folder with the Emmet extensions. The default is ~/emmet. emmet in the home directory of your system.

Hence the conclusion
This plugin tries to find this folder "~ /emmet"  ("<WinDrive(C)>:\Users<CurrentUser>\emmet" - on Windows)
And the solution to this problem.

You just need to create this folder (empty) ("<WinDrive(C)>:\Users<CurrentUser>\emmet" - on Windows)
OR
If you want to extend emmet


File upload, HTML capture attribute not working

Hi,
using capture="camera" not working properly on Android (not tested on iOS). Normally, the camera app should automatically start, but with the react-native-webview I have to "choose an action" (between "camera app" and "my files").
Thanks,
Hugo

I see this as being a nice enhancement.
The capture attribute has 3 possible values, as described in the standard https://www.w3.org/TR/html-media-capture/#the-capture-attribute:
<input type="file" accept="image/*" capture>
<input type="file" accept="image/*" capture="user">
<input type="file" accept="image/*" capture="environment">

Also, the capture attribute is not very well supported at the moment:
https://caniuse.com/#feat=html-media-capture

This would be nice, for sure.

Tested on ios, upload btn always opens the camera. Would love to see it works on android.

Running into this issue also. I know nothing about Android development but isn't it simply because the capture method is missing? Looks like there are only file picker methods:
https://github.com/react-native-webview/react-native-webview/blob/816c4a403814bbb4014efeccfc818d90cf3dea64/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java#L1173

Hi Folks,
Seems like this issue still happening in android

this issue still happening in android

Hi, any updates on this? The issue still happening in android.

Does #2954 solve this?

Does #2954 solve this?

Yes, but user or environment is not supported, it's blocked by FileChooserParams - no capture value provided.


IntanRawIO: Fix memmap order for Intan one-file-per-stream

Fixes #1556
Data before fix:

Data after fix:

Basically, since Intan was designed to be easily read with Matlab (their example work is to use fread which is column-major reading) we should make our memmap with order='F'.
@h-mayorquin want to look over this quickly and @alejoe91 I think this can be a quick review as long as everyone is agreed. See the linked issue for more information.
Can you test this @Tevin-yue?
NOTE: this is simulated data so it should look like a repeating motif of simulated "spikes".

I don't have your experimental expertise so this is for my learning. How do you know based on your plots (before and after fix) that your fix is right?

I don't have your experimental expertise so this is for my learning. How do you know based on your plots (before and after fix) that your fix is right?
The values are really weird on the axis (almost like we are summing spikes ie the samples and channels values are mixed up). And for intan simulated data it should like a regularly spiking (regular in the non FS/RS way) neuron so it should have a nice downward followed by upward deflections on a background of some level of noise. The "bad" graph looks to sinusoidal (again from in appropriate summing of the voltages).
so this meant either we scaled wrong. I double checked and we weren't. Or we loaded the data in wrong (easy to mix up samples and channels when making the memmap). Since we did the documentation from intan (I think this part was me so mea culpa), I did the F-major style but didn't specify. Sam's tip was just flip the sample and channel to allow us to read the C style.
So to summarize I would accept amplifier/wideband data to be noisy with spikes in it. Not weird sinusoidal things.
Or do you have an additional question related to this?

Maybe this will help you @h-mayorquin if I plot all channels instead of one channel.

You see we have sinusoidal noise (because this is simulated data.

Honestly dude you should propose that to Numpy! They just link to the wikipedia to explain this, but at least with your example you show what is actually happening with Numpy code. That's great!
@alejoe91 or @samuelgarcia. this is good to go now :) Tested and everyone is happy!


feat(Vs Code): Added Routes & Fixed Issues For Vs React Merge Run

Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Please Include Screenshots or Videos of the intended change:

Tested designer & data mapper working on the build. This will be the last PR except removing the react folders (vs-code-data-mapper-react, vs-code-designer-react)


update: collection of minor cleanups across the codebase

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area driver-kmod
/area libscap
/area libsinsp
Does this PR require a change in the driver versions?
NO
What this PR does / why we need it:
This is a collection of minor cleanups across the whole codebase, mostly relate to missing cmake includes, nullptr checks (peculiarly important after the recent refactor from @Andreagit97), and resolution of compilation warnings.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

/milestone 0.14.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


No conversion routine for select options

Hi,
thanks for that tool. First I thought it didn't work, because the entered material number in my try with the table mara wasn't fetched at all. I discovered that you have to enter the material number in the internal format. So I wrote a little piece of code, that calls the conversion routine from the domain of the select option in progress at the end of your method update_sel_row and it worked as expected.
      " Get and execute domain conversion routine
      IF c_sel_row-domain IS NOT INITIAL.
        DATA ls_dd01v TYPE dd01v.

        CALL FUNCTION 'DDIF_DOMA_GET'
          EXPORTING  name          = CONV ddobjname( c_sel_row-domain )
          IMPORTING  dd01v_wa      = ls_dd01v
          EXCEPTIONS illegal_input = 1
                     OTHERS        = 2.

        IF sy-subrc = 0 AND ls_dd01v-convexit IS NOT INITIAL.
          DATA(lv_conv_exit_name) = |CONVERSION_EXIT_{ ls_dd01v-convexit }_INPUT|.
          DO 2 TIMES.
            ASSIGN COMPONENT COND string( WHEN sy-index = 1 THEN 'LOW' ELSE 'HIGH'  ) OF STRUCTURE <range> TO <field>.
            IF <field> IS INITIAL.
              CONTINUE.
            ENDIF.

            CALL FUNCTION lv_conv_exit_name
              EXPORTING input  = <field>
              IMPORTING output = <field>.
          ENDDO.
        ENDIF.
      ENDIF." c_sel_row-domain IS NOT INITIAL.

Thanks! The program is not finished yet. Maybe I will go on next year.


2.0.0-beta6 and higher - autoTransition="animateToStart"  not work

when I click 'trigger'  and MotionLayout finish start to end,  expect it can back to start. unfortunately, animateToStar not working.
I find some differ in MotionLayout. there is the condition that this.mCurrentTransition == transition in  MotionScene#autoTransition(...),   can u explain what's the meaning of it?

I suggest that remove the condition in MotionScene.java:
if (transition.mAutoTransition == Transition.AUTO_NONE) {
    continue;
}

That was add to prevent what you are trying to do because others saw it is a bug. We messed up on making the API ambiguous. We will add another flag to support your use case.


UpsertGraph returning trick doesn't return all data if graph has no changes

Hey 👋
Using upsertGraph and returning * trick:
Book.query()
  .upsertGraph({
    id: 20,
    title: 'new title'
  })
  .returning('*')

When any field is changed, full model is returned (title is changed in example above):
Huddle {
  id: 20,
  title: 'new title',
  description: 'nice book',
  // rest fields
}

However, if nothing is changed, only provided fields from graph are returned (using same query above):
Huddle {
  id: 20,
  title: 'new title'
}

I believe it's because only graph fields are selected from database to check if update is needed:
SELECT "books"."id", "books"."title" FROM "books" WHERE "books"."id" IN (20)

If new data is added, returning * trick gets the rest of data:
UPDATE "books" SET "title" = 'new title', "updated_at" = '2021-05-27T18:38:58.998Z' WHERE "books"."id" = 20 returning *

Possible fix is to select all fields if returning * trick is present.
objection - 2.2.15

returning is only supposed to add a returning statement. It shouldn't add a select statement, let alone a separate select query.
You can use upsertGraphAndFetch.

@koskimas I didn't mean adding separate select query. When using upsertGraph there is already a select query which fetches provided graph fields. From example above (id, title):
SELECT "books"."id", "books"."title" FROM "books" WHERE "books"."id" IN (20)

As you can see, id and title will be returned. What I propose is if returning is present, this select would take all columns from database:
SELECT "books".* FROM "books" WHERE "books"."id" IN (20)

This way, if graph data is unchanged and update query is not executed, all data will be still present and upsertGraphAndFetch won't be needed.
Makes sense?

returning should only add a returning statement.

Okay, makes sense. Maybe an option in upsertGraph can be used to select all fields for the model 🤔
I'm just trying to find a way skip using upsertGraphAndFetch as it produces extra queries when data is almost already there.


General: Fix Model Participation Test Flakiness

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.

Motivation and Context

From time to time the ModelingPartizipationTest failed due to a #start-exercise-{id} element not being present on the exercise page. After a lot of testing locally (since the bug did not seem to appear that often locally, only after about 150 manual tests), I found that sometimes the exercise element was not yet present after creating it for some seconds. For some reason, the test failed a lot more often on bamboo than it did on my local instance.
Description

The fix is rather small. Cypress just reloads the page until the element is present. With this little fix and another 200 local tests, the issue did not appear again on my local instance.
Steps for Testing

Prerequisites:

Bamboo access
Or local cypress/artemis instance

Run cypress tests locally or on bamboo to see, if the tests fails or not

Review Progress

Code Review

[x] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Codecov Report

Merging #6017 (e6d051c) into develop (180772c) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #6017   +/-   ##
==========================================
  Coverage      79.92%   79.92%
  Complexity     12380    12380
==========================================
  Files           2126     2126
  Lines          84523    84523
  Branches       12678    12678
==========================================
+ Hits           67556    67558    +2
+ Misses          9431     9429    -2
  Partials        7536     7536

Impacted Files
Coverage Δ

...in/www1/artemis/service/connectors/GitService.java
71.39% <0.00%> (+0.47%)
:arrow_up:

Flag
Coverage Δ

client
75.77% <ø> (ø)

server
84.54% <ø> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

I need to investigate the issue further, since it does not seem to fix the flakiness on bamboo


Support for modifying labels

With the structure property it is quite easy to modify most values, by first finding the value to modify, then replacing the lines (known from startline and endline) in the lines property.
For example, for CMD:
for insn in parser.structure:
  if insn['instruction'] == 'CMD':
    content = 'CMD my-new-cmd-value\n'
    lines = parser.lines
    del lines[insn['startline']:insn['endline'] + 1]
    lines.insert(startline, content)
    parser.lines = lines
    break

However, for the LABEL instruction it is much more complicated as firstly there are two forms of LABEL, secondly there is quoting to deal with, and thirdly a single LABEL instruction can contain many labels.
It would be nice to provide support for modifying a single label, handling all these cases.

@twaugh  Please check #4, the new code might be familiar to you ;-)

Closed via #4.


Autojump to @inner from @outer

In Vim, when you type a motion outside a textobj, it will auto jump forward then do the motion.
This is super usefull, mean you don't need to move the cursor to the textobj part.

In treesitter-textobjects, it only do the motion when cursor in right position.
For example below, the @inner is inside @outer.
When do the motion like ca, da, ya. It works good.
Cause cursor always in @outer
But the @inner part, have to move cursor it, then do the motion right.

And here's the config, but I think has nothing to do with the config.
; import
(import_statement
  (import_clause
    [(named_imports) (identifier)] @import.inner)) @import.outer

I think this already implemented with lookahead = true option
      textobjects = {
        select = {
          enable = true,
          lookahead = true,
        }
   }

However the documentation is missing. Would you mind creating a PR that explains this feature. I guess your figures are perfect to put them into the README.

https://github.com/nvim-treesitter/nvim-treesitter-textobjects/pull/76 was already merged, thanks!


[FEATURE] Modify integration tests to run with OpenSearch Serverless

Is your feature request related to a problem?
Currently, integration tests are implemented to run with OpenSearch server running in docker.
As we want to make Flint compatible with OpenSearch Serverless, we should make integration tests be able to run against Serverless so that we can verify its functionality easily.
What solution would you like?
We need following changes

Mock metadata storage since we don't store them to serverless (otherwise, we need to come up a way to avoid issues caused due to WAIT_FOR refresh policy not supported)
Modify FlintSparkSuite and OpenSearchSuite to switch the OpenSearch to serverless collectio (Since we cannot run serverless within docker, we might not be able to run tests in Github workflow)
Fix verification method to not depend on queryIndex

OpenSearch Serverless does not provide stats API, and usage of DataFrame causes issue due to it. (Another approach is eliminating the usage of stats API)

What alternatives have you considered?
n/a
Do you have any additional context?
n/a

Looks like you want to run open source tests against a vendor-specific distribution.
We definitely want to continue running tests here against dockerized open-source OpenSearch.
For vendor-specific features we do have some precedent. There's an issue for running tests against vendor-specific distributions or instances in https://github.com/opensearch-project/opensearch-clients/issues/47 for SigV4.
One thing to be mindful of is that blocking PRs because of a vendor-specific solution being broken doesn't seem right.
[Catch All Triage - 1, 2, 3, 4]

@ykmr1224 thanks for the Feature request!
I am alarmed at the possibility that the vendor-neutral project will be tied down in any way by a vendor-specific one.
The following scenario is what i am afraid of -

Contributor wishes to make changes to opensearch-spark
Works on code and submits a PR
PR fails tests. Changes are required in OpenSearch Core (open source) and OpenSearch Serverless (closed source)
Changes are made to Core (now the feature is compatible with the open-source project)
PR still fails tests because the change breaks when running on OpenSearch Serverless
Contributor must ask folks maintaining OpenSearch Serverless to make changes in their code
Feature is stuck until vendor makes changes (IF they agree to the changes)

This enables the vendor to take control of large portions of open-source code de facto. @ykmr1224 @dblock I am curious if there are positive precedents for this type of dependency out there.
This may not be 100% true for this feature, but could be the case for other vendor-owned integrations, or might start to happen once we open the possibility and push a gray line further and further.

Opened up https://github.com/opensearch-project/.github/issues/229 to discuss the issue at large

As mentioned above, this issue opensearch-project/.github#229 discusses a better structure. Until that is achieved, there may still be desire to run tests against Spark EMR. This should be a stop gap measure until the OpenSearch Serverless release is better aligned with the OSS release.
With some initial testing, I was able to run SQL queries against the Spark EMR docker image.
https://gallery.ecr.aws/emr-serverless/spark/emr-7.2.0
Here is what I needed to do:

Create a directory to hold logging configuration. This really just needs two specific shell scripts. In my testing, these were empty shell scripts.

run-adot-collector.sh
run-fluentd-spark.sh

Create a directory that holds the Spark app
Get a local copy of the /etc/spark/conf directory from the Spark EMR Docker image.
 I tested with a Scala app for Spark.
There is still a step missing for including the OpenSearch PPL extension.

Another consideration is how to actually run tests on the docker image. I have a proposal:
Structure the tests with several files:

Data files
Setup queries
PPL query
Expected results

Create a Spark app. It can do some initial setup, such as creating tables. For each test:

Run the setup queries for the test
Run the PPL query
Write the results to a JSON file that is on a mounted volume of the docker image

After the Spark app finishes, go over each test and verify that the expected results match the actual results.
This means that there will be two components for running the tests:

A test running that is running on the host machine and can be started from the SBT build
A Spark app that runs the queries and captures the results for each of the tests

Create a Spark app. It can do some initial setup, such as creating tables. For each test:

@normanj-bitquill thanks for the suggestions
I agree we need a dedicated spark job to run the test and create a report regarding the success and performance of the tests being executed.
In addition I was hoping for using OpenSearch's Dashboard workbench PPL via a Spark data-source to access a local docker spark cluster which is running the Flint job - can you experiment if this is achievable ? and what are the prerequisites for this ?

As mentioned above, this issue opensearch-project/.github#229 discusses a better structure. Until that is achieved, there may still be desire to run tests against Spark EMR. This should be a stop gap measure until the OpenSearch Serverless release is better aligned with the OSS release.
With some initial testing, I was able to run SQL queries against the Spark EMR docker image. https://gallery.ecr.aws/emr-serverless/spark/emr-7.2.0
Here is what I needed to do:

Create a directory to hold logging configuration. This really just needs two specific shell scripts. In my testing, these were empty shell scripts.

run-adot-collector.sh
run-fluentd-spark.sh

Create a directory that holds the Spark app

Get a local copy of the /etc/spark/conf directory from the Spark EMR Docker image.

 I tested with a Scala app for Spark.
There is still a step missing for including the OpenSearch PPL extension.

@normanj-bitquill this look good - lets create a tutorial page with all the needed scripts for anyone to experiment with

@ykmr1224 thanks for the Feature request!
I am alarmed at the possibility that the vendor-neutral project will be tied down in any way by a vendor-specific one. The following scenario is what i am afraid of -

Contributor wishes to make changes to opensearch-spark
Works on code and submits a PR
PR fails tests. Changes are required in OpenSearch Core (open source) and OpenSearch Serverless (closed source)
Changes are made to Core (now the feature is compatible with the open-source project)
PR still fails tests because the change breaks when running on OpenSearch Serverless
Contributor must ask folks maintaining OpenSearch Serverless to make changes in their code
Feature is stuck until vendor makes changes (IF they agree to the changes)

This enables the vendor to take control of large portions of open-source code de facto. @ykmr1224 @dblock I am curious if there are positive precedents for this type of dependency out there.
This may not be 100% true for this feature, but could be the case for other vendor-owned integrations, or might start to happen once we open the possibility and push a gray line further and further.

@AmiStrn thanks for the feedback
I do believe that opening our code for multi-vendors contribution in a well managed way would actually benefit the Community - there are some open source projects that utilize this approach allowing vendor specific code contributions in their own rep (similar to OpenTelemetry - contrib repository)
We do need to avoid any direct build or runtime dependencies in these contrib components
We need an additional abstract layer to allow the specific vendor implementation without explicitly depending on it (the next example is NOT a good separations of concerns IMO )


Lage crashes (EPIPE) when total byte size of project paths or hash output exceeds 64KB (on MacOS).

Describe the bug
// ./src/getPackageDeps.ts: line 137
    const result = execa.sync(gitPath || "git", ["hash-object", "--stdin-paths"], {
      input: filesToHash.map((x) => path.resolve(packagePath, x)).join("\n"),
    });

The buffers execa uses to execute this command have an upper limit.  I don't know if that limit is specified in execa, NodeJS, or by the operating system, but on MacOS Ventura 13.3 it appears to be 64KB.
I am attempting to break up a medium-sized monolith into individual packages.  With the existing node_modules collection, that monolith's list of 56,038 hashable files is 3.3MB, far exceeding the 64MB limit.
To Reproduce:

Create a simple zero-package project for lage.
Into the empty packages folder, place a large and complex project.
Configure the root package.json to recognize the new workspace, and lage to build it.
Attempt to build the project.
See the error:

Running with 11 workers
node:internal/errors:496
    ErrorCaptureStackTrace(err);
    ^

Error: Command failed with EPIPE: git hash-object --stdin-paths
spawnSync git EPIPE
fatal: Unable to hash /Users/ken/projects/dev/web/node_modules/@goauthentik/web
c6ea78f6224fcba40713d28c20319873aa4733e3
06fccc32b1939efb64b92d9ae37bc21fb9118051
552cb0764a8c0cf7a937b25a04f55f0b70ea51cb
31479ed80f5dc571782e0a4d31658ba78adf0108
1b2db5bd40698e7f8467eb0cfe901cf38517f48d
... (many lines of hash output, ending unexpectedly and with an incomplete last line)

Expected behavior
I expected the build to complete.
Environment/Troubleshooting
  System:
    OS: macOS 13.3
    CPU: (12) arm64 Apple M2 Max
    Memory: 273.66 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.18.0/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
  Managers:
    Cargo: 1.71.0 - ~/.cargo/bin/cargo
    Homebrew: 4.2.0 - /opt/homebrew/bin/brew
    pip3: 23.1.2 - ~/Library/Caches/pypoetry/virtualenvs/authentik-BoGTmjdo-py3.12/bin/pip3
    RubyGems: <IP_ADDRESS> - /usr/bin/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 14.0.3 - /usr/bin/gcc
    Git: 2.39.2 - /usr/bin/git
    Clang: 14.0.3 - /usr/bin/clang
    FFmpeg: 6.0 - /opt/homebrew/bin/ffmpeg
    Curl: 7.87.0 - /usr/bin/curl
  Virtualization:
    Docker: 23.0.5 - /usr/local/bin/docker
  IDEs:
    Emacs: 29.1 - /opt/homebrew/bin/emacs
    Vim: 9.0 - /usr/bin/vim
    Xcode: /undefined - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Go: 1.20.4 - /usr/local/go/bin/go
    Perl: 5.30.3 - /usr/bin/perl
    Python: 3.12.0 - /Users/ken/Library/Caches/pypoetry/virtualenvs/authentik-BoGTmjdo-py3.12/bin/python
    Python3: 3.12.0 - /Users/ken/Library/Caches/pypoetry/virtualenvs/authentik-BoGTmjdo-py3.12/bin/python3
    Ruby: 2.6.10 - /usr/bin/ruby
    Rust: 1.71.0 - /Users/ken/.cargo/bin/rustc
  Databases:
    PostgreSQL: 14.9 - /opt/homebrew/bin/postgres
    SQLite: 3.39.5 - /usr/bin/sqlite3
  Browsers:
    Chrome: 120.0.6099.129
    Safari: 16.4
  Monorepos:
    Yarn Workspaces: 1.22.19

Closing this in favor of Lage hasher crashes trying to run git hash-object on a symlink.  This appears to have been a consequence of that bug, and the fact that it crashed within ~40 bytes or so of processing exactly 64KB was a red herring.  It seemed a reasonable assumption that 40bytes was about the length of a path, but further investigation revealed the symlink to have been the real issue.


Issue with Tag table in grapher step

I'm running into this issue when running grapher steps:

[SQL: SELECT tags.id, tags.name, tags.slug, tags.createdAt, tags.updatedAt, tags.isBulkImport, tags.parentId, tags.specialType
FROM tags
WHERE tags.name IN (%s) AND tags.slug IS NOT NULL]
[parameters: ('Tuberculosis',)]
(Background on this error at: https://sqlalche.me/e/14/e3q8)

Tuberculosis is an existing tag and I am pretty sure I have pulled the latest masters on owid-grapher, etl and ops.
An example would be:
poetry run etl grapher/tuberculosis/2023-11-27/expenditure --grapher on tb-expenditure branch.

@spoonerf could you try running make refresh from etl directory? There were some changes to MySQL structure yesterday.

Ah okay, I tried that and it's taking a while (not sure if that is unexpected or not), it's been stuck on this for the last few mins:

haha, I should ask Santa for more patience, that refreshed and the issue is solved. Thanks @Marigold!


Update Get-HTMLTable

These are cool, could you please add an example for this and the Import-Html in the Examples folder.
Plus, either update the README or leave a comment, just a line about these additions.
Feel free to leave a twitter id or blog link too.
Thanks for the PR!


String Interpolation

Hello friends,
I had some time this Sunday and decided to hack a bit on the self hosted jacket compiler to add a feature I was always thinking would really contribute to readability when watching videos of JT of Andreas: String Interpolation!
The current syntax uses curly braces without any prefix so you can write
let interpolation = "there"
println("Hello {interpolation}")

The part between the braces is parsed as an expression so you can also do stuff like:
let part_of_the_answer = 13
println("The answer is {29 + part_of_the_answer}")

The expressions are also type checked. For the code generation I currently check if there are any expressions in the string that need to be interpolated and only then generate a call to String::formatted, otherwise the basic String() constructor is used. I'm sure there are many edge cases I have forgotten but I thought before I put much more time into this I ask the community for comments. I'm sure there are lots of improvements since this is my first time contributing to the project and also hacking on a compiler, so be gentle :^)
Some stuff I'm unsure about:

How to handle old style format string when mixed with interpolated expressions? I think it would be best to check that only one style of formatting is used
Currently during lexing a list of subtokens is parsed and attached to the Token. Would it be nicer to keep the "Token stream" flat and not embed the information? What would be the best factoring here?

Have a nice Sunday everyone and let me know what you think?

Hello!
One or more of the commit messages in this PR do not match the Jakt code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

Currently these are indistinguishable from normal strings, so cosegen emitting braces or json generation become needlessly difficult.
And I remember discussions on Stringinterpolations running against a wall on discord, maybe discuss this there again
Second

Write the commit message subject > line in the imperative mood ("Foo: Change the way dates work", not "Foo: Changed the way dates work").
Write your commit messages in proper English, with care and punctuation.

You are missing a verb

This looks pretty interesting. I'm wondering if it makes putting literal { in strings impossible now? there would have to be a way to escape the curly brace. There's some finer syntax things to think about, e.g. should the string be prefixed with something? (e.g. python f-strings f"foo {some_var}" or perhaps "foo <some fancy character>{some_var}".
It's probably worth hopping onto discord to discuss in the #jakt-design channel to see if folks are ok with moving forward with this approach to iterate on, if there's fundamental problems, if there's minor changes requested, etc.

Thanks for your comments,

Currently these are indistinguishable from normal strings, so cosegen emitting braces or json generation become needlessly difficult.

Good point. I did not think about this. This should be fixable in many different ways (see below)

And I remember discussions on Stringinterpolations running against a wall on discord, maybe discuss this there again

That's exactly what I thought although I'm wasn't very active on Discord so I didn't see this discussion. But I thought instead of bike shedding in the Idea stage I first come up with a possible solution as a basis for further discussion

You are missing a verb

Thanks, I hope I fixed this now. Let me know if this is still wrong, as I said this is my first contribution to the project :)

I'm wondering if it makes putting literal { in strings impossible now?

Well, due to a bug it actually was, but that is fixed now, see the example. Its still a bit weird because of different escaping characters in the FormatParser and Jakt lexer but as I wrote in the commit message, this will probably fix itself when we switch to a more "elaborate" format like ${ something }.
As for how to fix your objections, here is an (incomplete) list of possible solutions with (incomplete) advantages and disadvantages I see.
Current implementation "This will be {interpolated}"
Examples in the wild: PHP (kind of?, I admit I'm going by wikipedia here)
Advantages:

clean
easy
possible (as proven by this PR)

Disadvantages:

Makes it harder to generate text with lots of curlies (json/code)
maybe some scenario where it would be ambiguous?

Special interpolation prefix "This value is ${interpolated}"
Examples in the wild: Swift, Kotlin
Advantages:

Easy to lex/parse
unambiguous
fixes the concern of generation code/json mostly (wll, except you want to generate interpolated strings :) )

Disadvanatges:

Very slight chance of backward compatibility issues, but easy to grep for

Special string prefix: f"This is an {interpolated} String"
Examples in the wild: Pythons f-String or Nims fmt Strings
Advantages:

No backward compatibility issues
Easy to lex/parse
unambiguous

Disadvanatges:

Special syntax
(I guess) many languages that chose this way did so because they started without string interpolation and needed to be fully backward compatible
Probable source of bugs (forgetting the special identifier at the start of the string)

Special string quoting: This is an {interpolated} String
Examples in the wild: JavaScript / TypeScript
Advantages:

No backward compatibility issues
relatively easy to lex / parse
unambiguous

Disadvanatges:

Yet another quoting style
In the case of backticks like JS/TS: Backticks are in some locales hard to type since they are used as a combinatory key by default
(I guess) many languages that chose this way did so because they started without string interpolation and needed to be fully backward compatible

No String Interpolation "This makes me {sad}"
Advantages:

Less code to manage
no possible source of bugs in the compiler
Disadvantages:
Splitting the format string from the expressions to fill the placeholders is harder to read, especially if many placeholders are filled
Jakt is a modern language that should have modern features :)

I hop on discord now and try to start a discussion there.

I just pushed a new commit that implements the proposed "This is ${interpolated}" syntax that got the most upvotes on discord.
I'm still happy to get any suggestions on the code itself.
All tests pass for me with the current implementation

Just implemented support in the interpreter:
> let foo = 42
> "The answer is ${foo:o} in octal"
= "The answer is 52 in octal"

Can you squash your commits?

Can it handle "${ "${ 0 }" }" (nested interpolated strings)?

Can you squash your commits?
Sure, done :)

Can it handle "${ "${ 0 }" }" (nested interpolated strings)?
No, but I'm also honestly not sure how you could parse this in any reasonable way. Currently in this case there would be an error that the opening curly is unmatched after the second quote.

I you really want to do something like this for a good reason I think it would be way more readable to embed a format() call into the expression like "${ format(fmt_str, 0) }", however you still can't nest the quotes.

No, but I'm also honestly not sure how you could parse this in any reasonable way.

You could just look for the matching close curly (simple paren counting), and then parse whatever is in between as an expression.  This would handle the case properly
Not that the example in question  is necessarily useful - but to me this seems like a more robust way of parsing anyway.

but to me this seems like a more robust way of parsing anyway.

Well, I need to refactor the lexing & parsing anyway, so I will give it a go. Guess you are right although it probably makes some things harder (e.g. syntax highlighting). Anyway, I'll give it a go.

I just did a refactor of the lexing & parsing steps. Notable changes:

Parsing now supports nested interpolated strings ("${ "${ 0 }" }") which isn't something you should necessarily do but as Mustafa said that's a more robust way of parsing
Escaping of an interpolated expression now works using double curlies instead of a backslash. This is how the StringFormatter class does it and makes things much easier to implement since \$ is not a valid escape sequence in C++ and thus needs to be specially treated during lexing and code generation
QuotedString and InterpolatedString are now distinct types so its easier to special case type checking and code generation for any of the two types
Lexing yields a 1-D stream of tokens again instead of embedding an array of Tokens into a QuotedString

I think this is now ready for a first code review :)

You have a typo in your commit message: "forat"

You have a typo in your commit message: "forat"

Thanks, fixed

I added syntax highlighting support in the vscode extension just now.
Other editors are not yet supported and just render interpolated blocks as part of the string but that's at least a start.
I have this addition in a separate commit for now since I think it makes sense to have those 2 parts separated but let me know if you disagree.
From my testing the code works pretty solid now, any comments appreciated.

I'd be more comfortable with this if the syntax looked a little different than strings. "{foo} bar" should always just be a string so you know what it is when you're looking at it without having to read the whole thing.
Requiring a $ in front of the string to interpolate gives that immediate indication that it's an interpolated string.
let x = 100
let y = 200
let result = $"my result is {x + y}"

Hi jt,
I just started a 2 week vacation so I won’t be able to work on this immediately.

"{foo} bar" should always just be a string

well it is, the syntax that was most liked from the alternatives above is "${foo} bar" and wouldn’t even that be a string after interpolation? :)
I‘m not quite sure I understand your concerns. From a developer perspective isn‘t
"${x + y} is the result " nicer to read than any of format("{} is the result ", x + y) or result_str + " is the result "? The latter is a nice example where you actually have to read the whole line to know that the result of the concatenation is a string. Also the syntax highlighting I implemented for the vscode extension also helps when reading to parse the interpolation part
Anyway, I have absolutely no problem changing the syntax, just want to understand the concern better to find a good solution.
CxByte also had the suggestion of supporting tagged strings someday so I would propose fmt"${x + y} is the result ", then if tagged strings land someday fmt can simply be a runtime comptime function, although that would be possible with the current syntax as well if you just use fmt as the default (Like JS kindof does). In this case $ would be a wierd identifier :^)

@dangrie158 - my main concern is readability.
As one example, think of a long string
"this is a very very long string and this string may have places that get converted to something but you don't know ${hi}"

In your version, you don't know that's a string interpolation until you read the entire string from start to finish. You need to know this bit of info during reviews and updating existing code because there are cases, namely the kernel, where you must know if you're going to allocate. Having to read the whole string to figure out if its an interpolation slows you down and makes the ergonomics worse if you think of how the language is used for different purposes.
$"this is a very very long string and this string may have places that get converted to something but you don't know {hi}"

Having an indicator on the front of the string tells you immediately if it's an allocating interpolation or a string literal.

Your right, fair point.
I thought about the tagged strings again, if we would allow any comptime function it would probably make sense to also support any return value, then using double quotes makes no sense.
If you are interested I could try to implement tagged templates closely  following the JS implementation. This would mean changing the quotes to backticks (``) which should also adress your concerns? It would also make a clear separation from string both in the compiler and for the reader. (Plus support for macros which could be nice for some usecases :) )

hmm as someone who got badly bitten by forgetting to add the f prefix to what I wanted to be a format string in python, would it be ok if we forbid interpolations in non-format strings? I.e. parse them in normal strings also and then throw if the string is not marked as a format string?

I'm closing this in favour of #1195


Add file dialogs

One of the items blocking #58 (the conversion to xi-win-ui).
Adds API surface to both xi-win-shell and xi-win-ui. Just one option is supported (hidden files).

Note: there's a bit of Zulip discussion here: https://xi.zulipchat.com/#narrow/stream/147926-druid/topic/File.20dialogs


Indent submenu elements

Hi.
It would be nice if submenu elements are indented a little bit. This would give a better overview about what is a topmenu element and what is a submenu element.
The different color is good, but not sufficient.
Maybe you could provide a css variable as well as the colors, so that everyone could adjust the depth of the indention separately.
Thank you very much.

It would be nice if submenu elements are indented a little bit. This would give a better overview about > what is a topmenu element and what is a submenu element.
The different color is good, but not sufficient.

I guess this depends very much on personal preference, design guidelines and the usecase.
The material guidelines are quite specific here, for a very deep nesting they don't encourage to use indention. For menus with a very flat tree it doesn't encourage to use collapsing menus but therefore indention.
It is quite easy to get the effect you want since you only need to add some padding:
.app-layout-submenu-container {
    padding-left: 30px;
}

Maybe you could provide a css variable as well as the colors, so that everyone could adjust the depth of > the indention separately.

I guess I can do that. But I haven't ever heard of a css variable 😛. But maybe you mean a sass or a scss variables 🙂.

I guess I can do that.

Remember when the menu is small
this
.app-layout-submenu-container {
    padding-left: 30px;
}

change to
.app-layout-submenu-container {
    padding-left: 0px;
}

I suppose.

@luisrafaelinf thanks for the hint I took that in account by adding two variables to set the indention for "small" menus and the expanded menus separably.

Since I will need more time to add the documentation / JavaDocs I added an additional release before 1.0.0 to roll out the new features to you.


Move slaves to autoscaling group

We should be able to place a single slave inside a each AZ in a private VPC and add it to autoscaling group

closing in favour of #65


Android Pay Button

Hi,
I tried running your project in android studio, I didn't see any errors but why can't I see the Buy with Android Pay button? Pleasee see attached.

Hope you can help me out. Thank you!

Do you have the Android Pay app on your device and is it loaded with a
valid credit card?
On Tue, Jul 12, 2016, 5:11 AM cyerro<EMAIL_ADDRESS>wrote:

Hi,
I tried running your project in android studio, I didn't see any errors
but why can't I see the Buy with Android Pay button? Pleasee see attached.
[image: capture]
https://cloud.githubusercontent.com/assets/20377672/16766455/ce114408-486c-11e6-8579-6695737fe029.PNG
Hope you can help me out. Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/android-pay/androidpay-quickstart/issues/18, or mute
the thread
https://github.com/notifications/unsubscribe/AIEw6imyxrwZ-TbXuM5FzWTyEixWy9Acks5qU4SIgaJpZM4JKU_N
.

Hi,
Thanks for your prompt reply! I do have android pay installed in my mobile
tho my card is not accepted since my issuing bank is not a participated
bank.
Is there some test cards to be used with android pay? And is that the
reason I can't see the Buy with Android Pay button?
Thanks again!
On Tuesday, July 12, 2016, Sam Stern<EMAIL_ADDRESS>wrote:

Do you have the Android Pay app on your device and is it loaded with a
valid credit card?
On Tue, Jul 12, 2016, 5:11 AM cyerro<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Hi,
I tried running your project in android studio, I didn't see any errors
but why can't I see the Buy with Android Pay button? Pleasee see
attached.
[image: capture]
<
https://cloud.githubusercontent.com/assets/20377672/16766455/ce114408-486c-11e6-8579-6695737fe029.PNG
Hope you can help me out. Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/android-pay/androidpay-quickstart/issues/18, or
mute
the thread
<
https://github.com/notifications/unsubscribe/AIEw6imyxrwZ-TbXuM5FzWTyEixWy9Acks5qU4SIgaJpZM4JKU_N
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/android-pay/androidpay-quickstart/issues/18#issuecomment-232066456,
or mute the thread
https://github.com/notifications/unsubscribe/ATbwSNjNyT0P0Uoo5Zj81pda_SOFJa04ks5qU6XRgaJpZM4JKU_N
.

Sorry right now you can't use any test cards with Android Pay.  That's the reason you can't see the button, you can see in the sample code that the button is hidden when there are no cards on the device.
Going to close this issue since the sample is working as intended.  I hope you can find a card that works!

Hi,
Thank you very much for your fast answers. Last question, have you tried
integrating Android Pay via CyberSource?
Thanks a lot!
On Wednesday, July 13, 2016, Sam Stern<EMAIL_ADDRESS>wrote:

Sorry right now you can't use any test cards with Android Pay. That's the
reason you can't see the button, you can see in the sample code that the
button is hidden when there are no cards on the device.
Going to close this issue since the sample is working as intended. I hope
you can find a card that works!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/android-pay/androidpay-quickstart/issues/18#issuecomment-232095708,
or mute the thread
https://github.com/notifications/unsubscribe/ATbwSCtkggF56T1HXkpNga0-ZsdqU8MFks5qU7tNgaJpZM4JKU_N
.

@cyerro nope sorry, I don't have any experience with CyberSource.

Hi,
No worries, thank you very much again!
Have a good night. ☺
On Wednesday, July 13, 2016, Sam Stern<EMAIL_ADDRESS>wrote:

@cyerro https://github.com/cyerro nope sorry, I don't have any
experience with CyberSource.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/android-pay/androidpay-quickstart/issues/18#issuecomment-232109131,
or mute the thread
https://github.com/notifications/unsubscribe/ATbwSP9EBRTZo1q2Rnj1IW-lB20LC9RPks5qU8eAgaJpZM4JKU_N
.


Change Dropdown box height

hi. thank you for this great library. is there any way to change the dropdown box height? I almost use everything and the height does not change.

I'm glad the library has been useful. The fourth example in the sandbox should solve this issue. Please let me know if you need any further assistance.

thank you the fourth example solved my problem


Unable to Install in Mac

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
OSX Mavericks

Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
node v6.2.2
npm v3.9.5

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

The log given by the failure.

Thanks! We'll be in touch soon.

＋1

Tried to force install as said by CanKer in #2161

npm install -g -f angular-cli

but got the following errors:
uid must be an unsigned int
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>
npm ERR! Darwin 13.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "-f" "angular-cli"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.8

npm ERR! uid must be an unsigned int

You seem to have some npm problems on your system. @angular/tsc-wrapped is definitely a valid name, maybe you're using a npm proxy or something.
I'm sorry I can't help you more, but this problem doesn't seem related to the CLI itself.


Terminal View Has Poor Visual Feedback To Indicate That It Has Focus

Issue Type: Feature Request
The visual feedback of whether the terminal view has focus is poor. The only indication is if the cursor is blinking (if one has chosen a blinking cursor) and if the cursor happens to be visible. One often thinks he or she is typing in the code pane when the typing is actually in the terminal pane and vice versa. The terminal pane should have some easily noticeable visual feedback when it has focus such as a colored bounding rectangle.
VS Code version: Code 1.63.0 (7db1a2b88f7557e0a43fec75b6ba7e50b3e9f77e, 2021-12-07T05:15:48.091Z)
OS version: Darwin x64 15.6.0
Restricted Mode: No

you can set your theme to high contrast and then it does

meganrogge, I'm sorry but I don't see from your screen shot what features besides a blinking cursor make indicate that the terminal pane has focus. The terminal has a box drawn around it but the box does not change appearance when the terminal has or does not have focus. And changing the theme to solve the problem probably won't change this behavior and in any case is impractical.

there's an orange border around the active terminal as you can see in my gif.

Hmmm.... There is no decoration like this on my computer. Is there a setting for this to make it more visible?

you could also set 	"workbench.colorCustomizations": {"terminalCursor.foreground": "#ff0000"} to make the cursor more noticeable

Hmmm.... There is no decoration like this on my computer. Is there a setting for this to make it more visible?

All I did was change the theme to High Contrast

I'm trying to find the color customization key for that border now so that you could keep whatever theme you have and use that

Thank you. High contrast theme hurts my soul.

It doesn't look like there's a way to set that

I'm trying to find the color customization key for that border now so that you could keep whatever theme you have and use that

I tried terminal.tab.activeBorder but it didn't seem to have any effect.

This feature request is now a candidate for our backlog.

Since it is now a feature request, I will mention that the orange border that appears in the High Contrast theme, despite its orangeness, is a little subtle. Perhaps a thicker line or a box that extends around the pane would be better.

I tried terminal.tab.activeBorder but it didn't seem to have any effect.

Oh.


Update to r0 1.0

We have now released r0 v1.0. Please do update to this stable version.

Fixed in v0.2.4.


[enhancement]: Persistent Model On Restart

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
Currently is it only possible to specify a singular default model on start with --model <model name> in the init or leave it blank (this usually loads SD 1.5). Almost everything else in Invoke returns to the last-used state on restart (prompts, settings, even the mask on the unified canvas), but not the model. This can lead to mistakes when one sees everything else is the same and proceeds without realizing the model has reset. Also, on systems that have a harder time changing models (lower ram, low caching. HDD), it may be a significant improvement having the model you are using persist until you are ready to move on to using another. An option to persist the model on next open would aid in user experience consistency.
Alternatives
A warning that pops up (with an "do not remind me again" option) the first time you click invoke after a restart, reminding you that the model has been reset to default (and list what the default is). This does not address the model change difficulty issue.

In the next release, the initial model selection will effectively be managed by the UI and work like you suggest here. In fact when you use the settings from a past generation, it will automatically use that model. We don't have an ETA on the next release just yet.

I wasn't going to put it here since it was a bit different, but I was going to suggest adding a "use model" and "use all and model" to the gallery right click (distinct from use all since changing model is a significant action and a common thing to do is to try the same gen parameters but on a different model).

That's something we will be able to do on the new system 👍

If I can pipe in here, I would love to have a "use model" button in addition to the "use seed", "use prompt" and "use all" buttons. In addition, it would be very handy to have a checkbox that automatically loads all previous settings when scrolling around within the gallery so that I can quickly see the prompt and other settings used to generate an image.


Update issue templates

:white_check_mark: Build pagarme-sdk-dotnet 1.0.119 completed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/cfb7486670 by @guibranco)

:white_check_mark: Build pagarme-sdk-dotnet 1.0.124 completed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/c5744705a4 by @guibranco)


Add a NewWithContext function to the non appengine build

The switch to google.golang.org/appengine means that app engine projects can be be built using the standard go tools, this is useful as goapp dosn't work well with oracle, gocode etc.
The issue is there is no NewWithContext function on the non appengine build. If this was added GitKit could could be used in app engine projects and use the go tools.

Thanks for the contribution. We can merge your pull request and close this issue once you sign the CLA.

Google CLA has been submitted for github account jonprice

Great, your pull request has been merged. Thanks for the contribution!


--export-plist command in CodeChecker quickcheck

The changes in #556 are missing because of another pull request where the relevant parts were refactored.

Wasn't these changes discussed to be completely deprecated before even being introduced, because analyze already handles this feature?

We might deprecate it but there is a bug fix in that pull request which is still valid and should be in v5.9.
We will discuss the other parts because there will be multiple changes in the next releases.

@whisperity could you handle this issue?

If you specify what was the bug fix in that pull request which is still valid (I can't really find it, that pull request was very long ago and I don't remember), then sure.

This was the bug fix.

--keep-tmp is a deprecated argument in quickcheck.
@gyorb The linked bugfix doesn't make sense anymore, as the log and analyze part invoked by quickcheck does not remove any "workspaces" anymore.

Okay so basically. There was an issue, that even if you supplied --keep-tmp, it would still remove the temporary folder. The bugfix fixed this, so if --keep-tmp was given, it would actually not delete the folder. The fix was implemented because --keep-tmp's help mentioned its expected behaviour.
Earlier in the restructuring, we decided that (seeing that it never actually worked) the argument would be deprecated from the valid argument list. Thus, the fix is also obsolete.
Current behaviour is that the temp folder (/tmp/codechecker-qc-[some random stuff]) is automatically deleted, and specifying --keep-tmp is indifferent to the invocation.

Ok, I've just wanted to make sure we do not loose any bug fixes.


Is there any macosx_xxx_arm64.whl in any previous release?

It's about #6. Don't know why it's closed. I can't install it in my Mac Silicon. I still have libaec.h not found issue and no other version recommended had worked.
I'm browsing https://pypi.org/project/imagecodecs/#files and few previous versions and I haven't seen any macos arm64 wheel there.

#6 was closed because "pip3 install imagecodecs" works on macOS 10.15.
There are no binary wheels for ARM64, any OS. I don't have the resources to support those platforms and no one contributed build scripts, binaries, or hardware.
My understanding is that conda-forge provides binaries for some ARM64 platforms, including macOS.

Ran into this issue too. Let me know if there's any way I can help


Plugin not able to find schema from a generated JSON file

Describe the bug
I get an error while loading schema:
error [graphql-eslint] Error while loading schema
 Error:
      Unable to find any GraphQL type definitions for the following pointers:

          - ./graphql.schema.json

This makes no sense because graphql.schema.json is in the root dir and is generated by graphql-code-generator.
Here's my config:
module.exports = {
  // ...
  overrides: [
    // ...
    // GraphQL
    {
      files: ['*.graphql', '*.gql'],
      parser: '@graphql-eslint/eslint-plugin',
      plugins: ['@graphql-eslint'],
    },
    {
      files: ['api/**/*.{gql,graphql}'],
      extends: ['plugin:@graphql-eslint/schema-recommended'],
    },
    {
      files: ['web/**/*.{gql,graphql}'],
      extends: ['plugin:@graphql-eslint/operations-recommended'],
    },
  ]
}

Adding parserOptions.schema doesn't work either. It seems like this plugin is just not listening to my parserOptions at all because when I try to set it to a bogus value I don't see my change reflected in the error message.
To Reproduce

Install graphql-eslint
Apply above config
Run graphql-eslint
💥

Sidenote: I'm not seeing any difference when using this with graphql-config.
Expected behavior
graphql.schema.json is picked up by this ESLint plugin and I don't have to muck about with configuration.
Environment:

OS: macOS Monterey
@graphql-eslint/...: 3.7.0
NodeJS: 14
ESLint: 7.32.0

Hi @tubbo, could you create reproduction repo?

@B2o5T I tried to but I couldn't reproduce. Turns out my repo had a rogue .graphqlconfig file that I wasn't aware of, and that was screwing things up.
Now I'm dealing with a different problem, turning off any rule seems to not work at all.
added '@graphql-eslint/require-description': 'off', to my rules and that totally works fine in the bare reproduction repo, but of course does not work in my actual repo :(


Rename acceleration files for H_GRONINGEN

If the H_GRONINGEN package (https://doi.org/10.5281/zenodo.3552507) on Zenodo needs an update, it would be useful to rename the acceleration files from H_GRONINGEN-acc-yyyy.csv to H_GRONINGEN-acceleration-yyyy.csv, to be consistent with other bird-tracking datasets.
Update should happen in data and metadata.

Done in https://doi.org/10.5281/zenodo.3828298


Source logos should be exposed through the cccatalog-api

/v1/sources should expose a link to the logo image for each source
They should be dynamically controllable through a database table exposed through Django admin interface

@makkoncept will work on this.


New package: SDPLR v0.1.0

Registering package: SDPLR
Repository: https://github.com/jump-dev/SDPLR.jl
Created by: @blegat
Version: v0.1.0
Commit: 29552b8da0982d61fe286f73d11f766cfb7361d6
Reviewed by: @blegat
Reference: https://github.com/jump-dev/SDPLR.jl/commit/29552b8da0982d61fe286f73d11f766cfb7361d6#commitcomment-136136835
Description: Julia wrapper for SDPLR

The naming follows the name of the library that is wrapped: https://github.com/sburer/sdplr. The closeness in name with other SDP solvers is also to be expected.

[noblock]
Just :+1:ing this.
JuMP owns two of the similar packages, https://github.com/jump-dev/SDPT3.jl and https://github.com/jump-dev/SDPA.jl, and the other, https://github.com/probcomp/SPPL.jl,  hasn't been updated in two years.
All the JuMP packages start with "SDP" because that's the problem class they are solving.


SEDkit rewrite not processing dictionary on initial input

I am trying to run SEDkit using photometry only, but I am getting a Nonetype error.
I have tried running the code in two ways:

a = sed.MakeSED(225, db)

a_dict = {'photometry':'*', 'parallaxes':4}
a = sed.MakeSED(225, db, from_dict=a_dict)

Both methods give me the same error:
/Users/elliannaschwab/codes/SEDkit/SEDkit/sed.py in __init__(self, source_id, db, from_dict, wave_units, flux_units, SED_trim, SED_split, name, verbose, **kwargs)
    225                 dummy = db.query(qry, fmt='table')
    226                 print(dummy)
--> 227                 dummy.remove_row(0)
    228                 setattr(self, table, at.QTable(dummy))
    229

AttributeError: 'NoneType' object has no attribute 'remove_row'

I added a print statement to see dummy, but it is empty and prints out as None

Ooooh, I know what this is. This was a terrible hack I added for tables that had no data. The problem with SQLite is that is returns None if a query has no results rather than an empty table. I'll fix it.
Joe Filippazzo, Ph.D.
Senior Research and Instrument Analyst
Space Telescope Science Institute
(410) 338-6871

From: Ellie Schwab Abrahams<EMAIL_ADDRESS>Sent: Saturday, March 3, 2018 10:54:38 PM
To: hover2pi/SEDkit
Cc: Subscribed
Subject: [hover2pi/SEDkit] SEDkit rewrite not processing dictionary on initial input (#9)
I am trying to run SEDkit using photometry only, but I am getting a Nonetype error.
I have tried running the code in two ways:

a = sed.MakeSED(225, db)

a_dict = {'photometry':'*', 'parallaxes':4}
a = sed.MakeSED(225, db, from_dict=a_dict)
Both methods give me the same error:
/Users/elliannaschwab/codes/SEDkit/SEDkit/sed.py in init(self, source_id, db, from_dict, wave_units, flux_units, SED_trim, SED_split, name, verbose, **kwargs)
225                 dummy = db.query(qry, fmt='table')
226                 print(dummy)
--> 227                 dummy.remove_row(0)
228                 setattr(self, table, at.QTable(dummy))
229
AttributeError: 'NoneType' object has no attribute 'remove_row'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hover2pi/SEDkit/issues/9, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AACHf8XRsMapK1qK-paZ595WPeHt1Qhzks5ta2V-gaJpZM4SbMDl.

@hover2pi just checking in on this. How is the fix going?

Ok, this is changed in the master branch. Now you can make an SED by just specifying the database object and the id from each table.
from astrodbkit import astrodb
db = astrodb.Database('$BDNYCdb/bdnycdev.db')
s = SED()
s.from_database(db, photometry=[1,2,3,4], parallax=23, spectral_type=2, spectra=[2], source_id=2)
s.results```


[WIP] bootstrap-table

#32

Codecov Report

Merging #34 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           40        40
  Lines         1081      1081
  Branches        83        83
=========================================
  Hits          1081      1081

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 d509293...73ff884. Read the comment docs.


VSOL 2802RH O5 status no internet connection

Hello, im trying to replace Nokia G-240W-C ONT to VSOL 2802RH and after set all configurations, i could get a VLAN return on omcicli mib get 84 as of:
AP#omcicli mib get 84
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
VlanTagFilterData
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
=================================
EntityID: 0xe102
FilterTbl[0]: PRI 0,CFI 0, VID 1107
FwdOp: 0x10
NumOfEntries: 1
=================================
=================================
EntityID: 0xe103
FilterTbl[0]: PRI 0,CFI 0, VID 1107
FwdOp: 0x10
NumOfEntries: 1
=================================
=================================
EntityID: 0xe10b
FilterTbl[0]: PRI 0,CFI 0, VID 1107
FwdOp: 0x10
NumOfEntries: 1
=================================

AP#omcicli mib get 171
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
ExtVlanTagOperCfgData
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
=================================
EntityId: 0xe01
AssociationType: 10
ReceivedFrameVlanTagOperTableMaxSize: 0
InputTPID: 0x8100
OutputTPID: 0x8100
DsMode: 0
ReceivedFrameVlanTaggingOperTable
INDEX 0
Filter Outer : PRI 8,VID 220, TPID 5
Filter Inner : PRI 8,VID 4096, TPID 0, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 1
Treatment Inner : PRI 9,VID 1107, TPID 3
INDEX 1
Filter Outer : PRI 15,VID 4096, TPID 0
Filter Inner : PRI 8,VID 220, TPID 5, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 1
Treatment Inner : PRI 8,VID 1107, TPID 2
INDEX 2
Filter Outer : PRI 8,VID 210, TPID 5
Filter Inner : PRI 8,VID 4096, TPID 0, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 1
Treatment Inner : PRI 9,VID 1107, TPID 3
INDEX 3
Filter Outer : PRI 15,VID 4096, TPID 0
Filter Inner : PRI 8,VID 210, TPID 5, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 1
Treatment Inner : PRI 8,VID 1107, TPID 2
INDEX 4
Filter Outer : PRI 15,VID 4096, TPID 0
Filter Inner : PRI 15,VID 0, TPID 5, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 3
Treatment Inner : PRI 15,VID 0, TPID 2
INDEX 5
Filter Outer : PRI 14,VID 4096, TPID 5
Filter Inner : PRI 14,VID 4096, TPID 0, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 3
Treatment Inner : PRI 15,VID 4096, TPID 3
INDEX 6
Filter Outer : PRI 15,VID 4096, TPID 0
Filter Inner : PRI 14,VID 4096, TPID 5, EthType 0x00
Treatment Outer : PRI 15,VID 0, TPID 0, RemoveTags 3
Treatment Inner : PRI 15,VID 4096, TPID 2
AssociatedMePoint: 0x601

after configuring WAN with 1107 vlan still no internet connection, any1 know what i could do?
Edit: when im connect with Vlan 1107, i can get a strange ip in the range of 10.52.15.XX without internet connection

Similar Issue
https://github.com/Anime4000/RTL960x/issues/121

Try setting FwdOp to 0x02 for each entityID Alone with the following commands then check for Internet then reboot after no connection
omcicli mib set 84 0xe102 FwdOp 0x02
Then check for connection on VLAN 1899 and 210
No connection reboot
then
omcicli mib set 84 0xe103 FwdOp 0x02
Then check for connection on VLAN 1899 and 210
No connection reboot
then
omcicli mib set 84 0xe10b FwdOp 0x02

Tried all, still no connection :(

I also experimenting

may I know, did your ISP have forced specific LAN port of ONT for Internet Access?
here in Malaysia (TM Unifi), this ONT doesn't wok when have LAN Port Provisioning (aka: 4 LAN port ONU Emulation)

No, just wan port and 4 lan ports to connect

What I can guess is, the FwdOp: 0x10 is very problematic,
Your ISP uses FwdOp: 0x10 for special bridging, since VoIP only available that mean VSOL accept that and ignore the rest.
my fiber wholesale set their OLT to force ONU LAN 2 and disable other ONU LAN, only can hack the bridge is ODI SFU firmware
sad to say, I can't reverse engineering much on VSOL V2802RH.
you need to use ODI Stick with SFU Firmware + Media Converter

You get a 10.xx.xx.xx IP means your ONU is working perfectly. You just need the correct VLAN. Maybe you can get it from your ISP or ask a service engineer to get it from their backend.

another user experiencing the same problem said that if he set:
omcicli mib set 84 0xe102 FwdOp 0x02
omcicli mib set 84 0xe103 FwdOp 0x16
omcicli mib set 84 0xe10b FwdOp 0x02
he gets internet via IpoE but no connection on bridge

You get a 10.xx.xx.xx IP means your ONU is working perfectly. You just need the correct VLAN of internet. Maybe you can get it from your ISP or ask a service engineer to get it from their backend.

yeah, aparently provisioning is OK. The problem is that this IP range is for VOIP. i confirmed this because if i check directly on ONT provided by ISP, VOIP is coming from a Ip from that range on Vlan 220.

updating
after executing
omcicli mib set 84 0xe102 FwdOp 0x02
omcicli mib set 84 0xe103 FwdOp 0x16
omcicli mib set 84 0xe10b FwdOp 0x02
i could get internet conection trough IpoE routing.

updating 2:
setting FwdOp 0x15 and 0x00 on entity 0xe103 bridge kinda works...
Vsol isnt get any IP trough bridge, returning only <IP_ADDRESS>, but i can connect to internet and some websites like youtube, google works, but other websites like speedtest.com, waveform bufferbloat test doesnt connect.
also download speed seems ok, but upload speed stuck at 0.08mb

updating 3:
after almost a month using Vsol like this, i has to reboot, and after that i cannot connect again, everytime i set the right FwdOp 0x15 im getting only ipv6 connection and no ipv4.
Any1 know what may be causing this?


[sealer-app-images] add flannel images

[sealer-app-images] add flannel images

Maybe you can provide a CloudImage address in readme, so others just pull and use it, not care about how to create a flannel CloudImage~
We also will sync the flannel CloudImage to the official registry.

Maybe you can provide a CloudImage address in readme, so others just pull and use it, not care about how to create a flannel CloudImage~
We also will sync the flannel CloudImage to the official registry.

Ok, let me modify it！


Design new swagger capabilities like filtering

V2 Design => https://apiview.dev/Assemblies/Review/46373cdd5bff4c86aeef03d3b8f79d50?diffRevisionId=bdba2e7c72b0480baec03266a26037bb&doc=False&diffOnly=False

Include in https://github.com/Azure/azure-sdk-for-net/issues/22658


Fix for [Android] Error: Duplicate resources

Summary

I keep getting error Android Duplicate Resources when building on Android, then I found #22234, i saw a comment that fix my problem by patching react.gradle with code like in this commit, this works on my machine.
Changelog

Android Fixed - Fix assmbleRelease build
Test Plan

AssembleRelease basic react-native project

don't run react-native bundle --platform android --dev false --entry-file index.js --bundle-output android/app/src/main/assets/index.android.bundle --assets-dest android/app/src/main/res
If the assets(--assets-dest android/app/src/main/res) is not extracted, it is not duplicated.

Thanks for sending the PR. I'm closing this as the logic is completely rewritten now and uses AGP artifacts API


Empty list in filters shouldn't be treated as null values

What I'm trying to achieve
To filter with empty list of specifiers.
Steps to reproduce the problem

attributes(first: 5, filters: { ids: [] }) {
  edges {
    node {
      id
    }
  }
}

What I expected to happen
Definitely not to omit filter. It should return empty list.

As well, null should keep its behavior, so returning "something".

The problem with this one is that django-filters is treating empty values as a skipped filter.
docs
We could work around it by adding the MAGIC value that indicates that filter should be applied in this case.
Any thoughts?

From the example above. ids filter is an instance of GlobalIDMultipleChoiceFilter. You can extend the current implementation to handle the given case.

That was my first attempt.
Unfortunately in the filter, I will get value = [] either when the ids will be skipped in the query or set to id: []. So I can't distinct if this was intentional or not.
Changing it to return empty queryset on [] will work globally and will basically make that field required in query filters to have any result.