Fonts are missing

I'm running server on arm64 debian. Fonts work fine, but after intalation of ttf-mscorefonts-installer and successful font generation on the onlyoffice side, the fonts are missing on the server.
What should I do?

Seems this is issue with DocumentServer
How did you installed DocumentServer on arm64? Currently we have no official support of this platform, did you use some third-party port?


pass char[] as the parameter to c++ function

thanks for this crate, i am faced with a problem,   i want to call a C++ function in rust,  the c++ code detail is like bellow,
typedef char TThostFtdcBrokerIDType[11];
//struct
struct CThostFtdcFensUserInfoField
{
TThostFtdcBrokerIDType	BrokerID;
TThostFtdcUserIDType	UserID;
TThostFtdcLoginModeType	LoginMode;
};
// function
virtual void RegisterFensUserInfo(CThostFtdcFensUserInfoField * pFensUserInfo) = 0;
my question is, did the cxx support the struct like char[11]?  i didnt see this in the official support list.  or can help advise this ?

Yes, you'd need to give some Rust type which has a matching layout as TThostFtdcBrokerIDType. For example:
use std::os::raw::c_char;
use cxx::{type_id, ExternType};

#[repr(transparent)]
pub struct TThostFtdcBrokerIDType([c_char; 11]);

unsafe impl ExternType for TThostFtdcBrokerIDType {
    type Id = type_id!("TThostFtdcBrokerIDType");
    type Kind = cxx::kind::Trivial;
}

#[cxx::bridge]
pub mod ffi {
    struct CThostFtdcFensUserInfoField {
        BrokerID: TThostFtdcBrokerIDType,
        //...
    }

    extern "C++" {
        include!("path/to/header.h");

        type TThostFtdcBrokerIDType = crate::TThostFtdcBrokerIDType;
    }
}

(You need at least cxx version 1.0.66 for this.)

i didnt see this in the official support list. or can help advise this ?

You were probably looking at the list of built-in bindings. Something doesn't need a built-in binding in general for you to use that type. You can fill in your own ExternType impl instead for any type, as above.


[hotfix]fix chatglm rmsnorm

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

This PR fixes the import err of chatglm rmsnorm
💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ ] I have performed a self-review of my code
[ ] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

before replace rms_norm

after rms_norm


Off-by-one error in clear sky mid-month calculation for Linke turbidity

Description
On line 260 at https://github.com/pvlib/pvlib-python/blob/0781787b7efa381ce26d226dd35dddf6a2791502/pvlib/clearsky.py#L260, we see that _interpolate_turbidity uses time_utc.dayofyear which is 1-indexed. When the middle of the month is calculated, this uses 0-indexing.
Usually in other parts of the code, this is corrected by an offset of 1, but in this case, it is not.
Reproduction
For instance, this year, February 15th 12:00 PM was the midpoint of February. This is the 46th day of the year, so the index is 45.5 which is correctly calculated by _calendar_month_middles.
Expected Behavior
This can be seen for some midpoints which you would expect to be exactly what's in the HDF5 file, but are not. For example, we would expect $\frac{57}{20} = 2.85$ for these coordinates at this time:
>>> pvlib.clearsky.lookup_linke_turbidity(pd.DatetimeIndex([datetime.datetime(2024,2,15,12)]), 38, -93)
2024-02-15 12:00:00    2.861667

I imagine this is of very low priority, since these calculations only have the resolution of a single day, but I noticed it and thought I should make an issue.

pvlib.__version__ == '0.11.1'
pandas.__version__ == '2.2.3'
Python 3.13.0

Maybe the issue is a lack of clarity in what to expect. What value should be used to represent the entire month in these cases?
February 2024: the middle of the month day is noon of Feb 15 (14.5 days before, 14.5 days remaining). The returned index is 45.5, which is what I would expect (31 + 14.5). But starting from day of year (46 for Feb 15, 2024) I can understand why seeing 45.5 appears to be off.
February 2023: the middle of the month is midnight on Feb 14. The returned index is 45. The day of year for that datetime is 46, but 1 second earlier would be 45.
import datetime
import pandas as pd
from pvlib import clearsky

dt = pd.DatetimeIndex([datetime.datetime(2024,2,15,12)])
print(dt.dayofyear)

middles = clearsky._calendar_month_middles(2024)
print(middles[2])

dt = pd.DatetimeIndex([datetime.datetime(2023,2,15,0)])
print(dt.dayofyear)
dt = pd.DatetimeIndex([datetime.datetime(2023,2,14,23,59,59)])
print(dt.dayofyear)

middles = clearsky._calendar_month_middles(2023)
print(middles[2])

The lookup and interpolation methods originated with pvlib code, so we aren't bound to follow a reference here and can choose how to do this.

I don't really have too much stake in this; I just thought it was weird. My own suggestion would be to use actual times instead of dayofyear calculations, so the interpolated value is a continuous function of datetime. You still need to get the middle of the month in epoch time or something like that.
I haven't tested the sensitivity of a final PV power output with respect to turbidity, so I'm not sure how much this matters, especially given how the input data is in uint8.


add document validation

@jyemin bump

bump

if there are no other objects, i'll merge to master.

What are all 19 commits from August 31?

rebased against master since it'd been a while.  trying to get this prepped for merge.

something clearly went wonky so i'm going to close this one and open a new so that the diffs are obvious.


How to make FP16 quantization on gpt/xl?

How could I fix this error?
ValueError: Message tensorflow.GraphDef exceeds maximum protobuf size of 2GB:<PHONE_NUMBER>

Hi @Archelunch, what is the script you use? At which step are you getting this error?

output_graph = "frozen_graph.pb" output_graph_def = tf.graph_util.convert_variables_to_constants(                         sess,                         tf.get_default_graph().as_graph_def(),                         ["sample_sequence_2/while/Exit_3"]                     )  with tf.gfile.GFile(output_graph, "wb") as f:     f.write(output_graph_def.SerializeToString())
`---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
 in
8     # serialize and dump the output graph to the filesystem
9 with tf.gfile.GFile(output_graph, "wb") as f:
---> 10     f.write(output_graph_def.SerializeToString())
ValueError: Message tensorflow.GraphDef exceeds maximum protobuf size of 2GB:<PHONE_NUMBER>`

Are you working with a big model? It seems you're reaching the limit size (2GB) supported by protobuf which is used internally by TensorFlow to serialize the graph. It doesn't have any relation with quantization, it's a more general TF limitation it seems.

Is there any way to loop through the .pb file in a similar way to this stackoverflow question/answer so as to cut down on the amount of information that is loaded in memory?


Add trailing commas

Signed-off-by: Dmitry Chigarev<EMAIL_ADDRESS>
What do these changes do?

[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #1546
[ ] tests added and passing

see #1524

@devin-petersohn can we extend this style to the whole modin, and not just to this file modin/pandas/dataframe.py?

@anmyachev Yes we should do it everywhere.
Is black failing for others on other files? Why is this the only file affected?

@devin-petersohn trailing commas in black
the last paragraph is interested for us, black adds extra comma only if file for Python 3.6+, if .py file has some traits of 3.6+ (like format strings, which was added at dataframe.py in #1524) black suppose to add it.
It seems that some files doesn't have traits of 3.6+ that's why only dataframe.py was affected

Thanks @dchigarev, I have just realized a serious issue and we cannot do this.
We are supporting also python 3.5, which doesn't support trailing commas: https://github.com/modin-project/modin/blob/master/setup.py#L68
We must support all python versions that pandas supports, so I do not think we can update the trailing commas until we move to only python 3.6+.

@devin-petersohn, pandas 1.0.3 (which is required to use modin) seems to works only at python versions that greater than 3.6.1: https://github.com/pandas-dev/pandas/blob/master/setup.py#L774
so It seems no problems to update modin to 3.6+ only

I see, good catch. I must have been going on an older version and did not update it.
Let me fix a couple of things in the setup.py, then we can add this. We need to update the versions of some dependencies as well.
We should wait until we fix the python version in setup.py so we do not have a commit that breaks for Python 3.5.

done it at #1551

Great, left a comment on that one.

@devin-petersohn since #1551 are merged, this PR is ready to merge


Fix Solax X3 modbus template battery power scale

By using the current Solax X3 battery template in config, battery power is provided with a wrong sign. This fix will correct this.

Vielen Dank!


[Bug] Curve overlap in version 5.4.1

Version
5.4.1
Link to Minimal Reproduction
https://codepen.io/KCheng-sun/pen/RwBGobz
Steps to Reproduce
When the render is over, change all the values of a row to 0 and render again
Current Behavior
When the render is over, change all the values of a row to 0 and render again
Expected Behavior
Do not create curve overlay
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

Just to let you know - if bug description is not clear, most likely there wont be any replies.
In your case "render", "row", "curve overlay" are not defined, people do not understand the problem.


Allow a logger to be passed to Git.clone

Your checklist for this pull request

[X] Ensure all commits include DCO sign-off.
[X] Ensure that your contributions pass unit testing.
[X] Ensure that your contributions contain documentation if applicable.

Description
Add new :log option log to Git.clone which logs the clone operation and returns the new Git::Base object with its logger set to the value passed in :log.

The original discussion for this PR was on PR #496.


Add browser badges

added in https://github.com/inttter/md-badges/commit/ea8b97a69fa8e158c7570c1dd6f44519980368a1


Decrease StyleBoxFlat corner detail in editor theme

Decreases the corner detail of editor theme StyleBoxFlats from corner_radius * 1.5 to corner_radius * 0.5. This improves the performance of the editor because the hundreds of StyleBoxes require generating and rendering far fewer points while having the same appearance.
The corner detail of the GraphNode panel StyleBoxes is set to corner_radius so that they still look good when the GraphEdit is zoomed in.
Corner Detail Comparison:

Corner Radius
Before
After

3 (default)
5
2

4
6
2

6
9
3

Marking as a draft for now because this is currently untested.

This improves the performance of the editor because the hundreds of StyleBoxes require generating and rendering far fewer points while having the same appearance.

You sure this holds true for editor scales over 100%? Maybe the multiplier should scale with the editor scale? 0.75 for 100%, 1.5 for 200%?

@YuriSizov It does use the editor scale as a multiplier. I should have mentioned that.
Math::ceil(0.5 * p_corner_width * EDSCALE)

It might be worth chechking if beveled corners (detail = 1) look good enough for the editor's purposes.

It might be worth checking if beveled corners (detail = 1) look good enough for the editor's purposes.

I tried this and it didn't look good. It makes the corners look more sharp than rounded.

Thanks!


Replace missing abstract enum types

The abstract enum types in OpenFL (such as openfl.display.BlendMode) need to be replaced with simple static values for hand-written JavaScript to use them. This might be best done as a step after generating our ES5 modules from the original OpenFL source, since abstract enums are a nice approach to getting the best features of enums while still compiling into static string values

This is done :wink:


Unable to reproduce the results using GPU

When trying to run the code using GPU on local machine, the code executes successfully without any errors but the kernel-out.txt is coming out to be all zeros and does not match the output from the CPU method. Are there any parameters that need to be changed to get the code working as expected? The GPU on my machine is GeForce GTX 950M(compute capability 5.0) and CUDA version is 10.1

Works on a workstation GPU, closing the issue.


Use compiler function in nightly recipe, pin to Rust 1.62.1

switches back to using the compiler function to set our Rust build requirement, which should be unblocked with https://github.com/conda-forge/rust-activation-feedstock/pull/22 in
specifies the Rust build requirement with a conda build config file, as suggested by @jakirkham
pins our CI to this Rust version (1.62.1) to match with the nightly builds

Codecov Report

:exclamation: No coverage uploaded for pull request base (datafusion-sql-planner@9d4b9d1). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #687   +/-   ##
=========================================================
  Coverage                          ?   66.82%
=========================================================
  Files                             ?       73
  Lines                             ?     3635
  Branches                          ?      747
=========================================================
  Hits                              ?     2429
  Misses                            ?     1061
  Partials                          ?      145

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


Remove QuackAnimationSpec#isSnapshotMode

There is no need to change AnimationSpec to snap as we are not needed snapshot testing now.

INVALID -> CLOSE
https://github.com/duckie-team/duckie-quack-quack/issues/199


Add referrer meta tag to website

Since moving to SSL only on the site, I would like to add this meta tag.  Not high priority.
<meta name="referrer" content="origin">

I'd say rel="canonical" should be used.
https://yoast.com/rel-canonical/

@jdorfman Seems legit, I'll let you make the call.

@jdorfman Any opinions on this?

@XhmikosR create a PR =)

Not sure how to do it with Jade. I still have a long to do list for this
repo....
On Jul 21, 2016 00:49, "Justin Dorfman"<EMAIL_ADDRESS>wrote:

@XhmikosR https://github.com/XhmikosR create a PR =)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/issues/641#issuecomment-234094338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtYAuKG9CH4WPuri0EO5nCZ1qin28ks5qXpfegaJpZM4HhKfR
.

Here's an example of a meta tag: https://gist.github.com/xtian/888338#file-html5boilerplate-jade-L5.
That said, I don't mind doing it if you don't have the bandwidth.

The canonical tag needs to get the current page's URL. Since you are more
familiar with Jade than me it should be easier for you.
On Jul 21, 2016 00:52, "Joshua Mervine"<EMAIL_ADDRESS>wrote:

Here's an example of a meta tag:
https://gist.github.com/xtian/888338#file-html5boilerplate-jade-L5.
That said, I don't mind doing it if you don't have the bandwidth.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/issues/641#issuecomment-234095243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtYQw-vPDi-JFN5DWU8NGzQR3c4Myks5qXpingaJpZM4HhKfR
.

@jdorfman @XhmikosR Is the suggestion canonical over referrer or both?

Never mind, referrer is already there.

My personal choice is definitely only canonical.
On Jul 21, 2016 00:56, "Joshua Mervine"<EMAIL_ADDRESS>wrote:

@jdorfman https://github.com/jdorfman @XhmikosR
https://github.com/XhmikosR Is the suggestion canonical over referrer
or both?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/issues/641#issuecomment-234096298,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtR8FUnP2WwWpBQoE7e7HH9-wnkJJks5qXpmlgaJpZM4HhKfR
.


Text Explainer Ops

Need to add documentation on how to use the explainer into operators

Now part of SDK 390 2


Python Memory Profiling

Hi team,
I have tried to lookup many docs, but haven't found the way how to profile python memory with Parca.
Is it possible (ideally I want to have smth like https://github.com/bloomberg/memray but with continuous profiling).
Could you please help me with this case?
Thanks!

We don't officially support memory profiling with Python at the moment.
Something that might work is using https://github.com/timpalpant/pypprof, which then needs to be scraped by Parca. We might investigate these things in the future.


As Device Owner I want to be able to install thin-edge also without Debian Package Manager

Extract installation logic from debian maintainer scripts into one installtion-script and one removal-script per each thin-edge package. Therefor consider topics below:

All lines from maintainer scripts that not necessarily need to be located in maintainer scripts shall be moved to those installation/removal-scripts from above.
If any rest was kept in some maintainer scripts reason to be stated (for later alignment).
Those installation/removal-scripts shall be added to according Debian packages and shall be called as appropriated in maintainer scripts.
Finally those installation/removal-scripts shall allow to easily create packages for other Package Managers than Debian.
Moreover those scripts shall even allow to provide the installation as ZIPed file-tree, for installation without any Package Manager at all.

Installation procedure: (1) Copy filetree to „/“; (2) Execute installation script.

NOTE: See also ticket (https://github.com/thin-edge/thin-edge.io/issues/761) as second step, to move script-logic into Rust code.
AC:

Installation-scripts and removal-script per thin-edge package do exist and are added to according debian packages and are called in according maintainer scripts.
For each lines that was kept in Maintainer scripts a statement in that tickets comment section states the reason.

As of version 0.11.0, tarballs are now included containing the thin-edge.io components.
The tarballs can be downloaded, and the initialization can be done by just running the <binar> --init command.

tedge_0.11.0_x86_64-unknown-linux-musl.tar.gz
tedge_0.11.0_aarch64-unknown-linux-musl.tar.gz
tedge_0.11.0_armv7-unknown-linux-musleabihf.tar.gz


Make reference fully atomic

Hi Guillaume,
I have worked a bit with AtomicReference and think came up with a solution that resolves this retain count problem you've explained me earlier. The idea is just that AtomicReference by default retains its pointee and releases it again once the pointer is changed. This way, another thread cannot kill a variable without the other thread knowing.
Note that this is WIP in that I didn't quite understand all the code. To be more precise, I didn't know what the assert in load is supposed to mean and why it's necessary. I assumed it was necessary because of the spin load but wanted to be sure anyways. Apart from that I wasn't a 100% sure about the semantics of the load method either, or at least I got a bit confused. I assume load is not supposed to delete the reference to the object right? Maybe you could clarify this assertion real quick.
I have also written a test to ensure AtomicReference correctly releases the references again. Let me know if you think any other tests are necessary.

You're not addressing the case illustrated in testDemonstrateWhyLockIsNecessary.
Imagine an object with ref count of 1, whose reference/unmanaged is held in an AtomicReference. Two threads try to read it: thread A wants to use the reference, while thread B wants to lower the ref count. If thread B reads the pointer and lowers the count in between thread A reading the pointer and then using it, then you have a crash. You should write a test that makes two threads race and have it succeed a large number of times in order to show that it works. I'm pretty sure this won't.

I added a new race test that exercises timing between a load and a deinit. It crashes on your branch. (It crashes as I expected with swift test, but for some reason doesn't crash in Xcode; go figure.)

I wasn't a 100% sure about the semantics of the load method either

The difficulty with load is that it must increase the reference count by 1, unlike the other methods that return a reference. This is because it is a copy.  take and swap don't change the ref count because they are not copies; they change the reference that's being pointed at.
By the way, the CAS itself is fine. I'd never thought about it properly. I would suggest doing the reference counting only on the success path: since you have both references as parameters the objects cannot die before the end of the function. Right now both paths involve 1 retain and 1 release. However the failure path has both on the same object, and its count is already >0.

I added a new race test that exercises timing between a load and a deinit. It crashes on your branch. (It crashes as I expected with swift test, but for some reason doesn't crash in Xcode; go figure.)

Aah, now it clicked. I thought you mean something like:
Thread A:
        while let thing = r.load()
        {
          let id = thing.id
          _ = id
        }

Thread B:
         var thing = r.load()
         thing = nil

Which should obviously be possible. But now I see the problem. But I guess I'll revert most of the changes but keep CAS, since that should be possible, right? You're of course right that we can wait with retaining it until after the CAS returned successfully.
Thanks anyways for your comments!

agreed. thanks!
maybe closing this pr and opening a new one with just the CAS would be preferable

One more thing -- i wonder whether the CAS should just fail when the reference is locked, or spin until it becomes unlocked. In the case of a weak CAS obviously the former is better; for a strong CAS, I'm not so sure.

In my opinion both should fail. This way both are guaranteed to be lock free.
I'm not sure I can write this pull request though. I looked at the code and I don't understand all of it well enough, I'm afraid.

I added OpaqueUnmanagedHelper.CAS on the devel branch. It should be a good basis for the AtomicUnmanaged version


Made jobs.<job_id>.steps[*].shell a separate ##

Why:
Separated jobs.<job_id>.steps[*].shell into its own ## which increases discoverability.

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Ukaž kozičky prosím


Middleware to deep-freeze all state

It'd be cool to have a middleware that automatically makes all items in the store read-only. It'd be similar to the JS package 'deep-freeze', but probably throw errors on mutations instead of silently ignoring them.
Speed of implementation doesn't really matter here, since it'd only be a development tool.
Implementing this in Lua 5.1 might be tricky. Normally for a read-only table, I'd create a new table with __index and __newindex metamethods acting as a proxy, but then you wouldn't be able to iterate (or shallow copy) the table! In 5.2, Lua gained __pairs and __ipairs which would've helped here, but alas.
There might be one edge case in this implementation. Way back when I first used Redux, I would keep JS Promise objects in the store as a way to manage request status. Promise objects have internal mutability, which we might consider to be breaking the Redux paradigm, but because it's not observable outside of attaching a handler to the promise, I don't know if it is. If we wanted to support a use case like that, we'd need to make sure not to deep-freeze some objects, or force Promise objects to be userdata instead.

I don't think we need to support Promise objects in the store.
I haven't gained any insight into how to actually deep-freeze tables. 😦

I'm not sure if deep-freezing tables is even possible in Lua without __pairs and __ipairs. Deep-freezing in Lua 5.1 must either:

Allow mutations of existing keys (bad)
Not allow iteration over the frozen table (also bad)

table.freeze is now out, so this is doable.

I don't think we need to support Promise objects in the store.

In my own systems, I check getmetatable(table) ~= nil as a heuristic for whether or not to freeze or not. Not sure how good it is as a generality.


Expired Credentials (mitmproxy-ca-cert.pem)

Expired on 2024/02/19. When is new update?
Downloaded from http://mitm.it

If you delete your local ~/. mitmproxy folder, a new CA certificate will be
generated on startup.
On Fri, Mar 8, 2024, 22:11 maguro1kan @.***> wrote:

Expired on 2024/02/19. When is new update?
—
Reply to this email directly, view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/6727, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHY2PX5XY4N2GWWUXXJ763YXISPVAVCNFSM6AAAAABENOB6V6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TMNZXGU4TMNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I got it. Thanks


Command line switch for rules

It would be nice, if the command line interface would allow to en-or disable checks for certain rules.

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


Lazy load modules not being bundled

- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Angular CLI version 1.0.0
MACOS Sierra
Repro steps.
Simplified Scenario:
A. User Logs in
B. Server will authorise the user and also send back the accessible routes i.e. :

{
  new {
    itemID = 1,
      name = "Users",
      parent = "Administration",
      icon = "??",
      path = "user",
      loadChildren = "./adm-user/adm-user.module#AdmUserModule"
    //"../adm-user/adm-user.module#AdmUserModule"
  },
  new {
    itemID = 1,
      name = "Roles",
      parent = "Administration",
      icon = "??",

      path = "role",
      loadChildren = "../adm-user/adm-useaar.module#AdmRoleModule"
  }
}

C. My Angular code uses the returned values to manipulate the routerConfig and add the returned routes.
Issue: It looks like Angular CLI (WebPack) ng server and ng build module are not creating any bundles for the AdmUserModule and AdmRoleModule. I guess this is because there is no reference to these in the code and it's being created dynamically.

Note: when I hard code the routes and build I get more bundles created and everything works fine

Desired functionality.
Cli/Webpack giving the functionality to either create bundles for all modules available or if this is not possible. Being able to create individual bundles per module manually.

@ashkank83 can you still reproduce this issue? The lazy-loaded modules should be bundled into separate chunks.

@mgechev Thanks a lot for the response. TBH, I gave up on Angular since and recreated the same in React (Please don't get me wrong, I'm Not saying one is better than the other). I know they're both (Create React App & Angular-CLI) using webpack and maybe as you said the issue is solved here as well now. Thanks

If there is solution for it, might be helpful to Angular community to post it here as well.
I had a few messages and comments asking for solutions

Thanks for the quick follow up!


Feats not working with Inventor Armorer Thunder Gauntlets

The versions of Solasta and this mod.
Game 1.5.92
Mod <IP_ADDRESS>
A list of other mods you have installed [you shouldn't have any].
None
A short description of the bug.
The following feats do not appear to be working with the Thunder Gauntlets:
Dual Flurry: Should grant an extra bonus attack for your offhand gauntlet, but any combination of either 2 main hand or 1 main hand 1 bonus action is not granting it like it does with other weapons
Twin Blades:  When hit does not prompt a reaction to increase your AC by 3 even though you are wielding two weapons.
Save File:
Solasta Save.zip
I couldn't locate the JSON file for the campaign, its not a custom campaign just ah PoI starting campaign.  If you need that let me know where I can find it.  Only found one JSON for a campaign and its my custom map for testing.

New save, party is loaded into Baraz HQ, Inventor has the Dual Flurry and Twin Blades feats, can load mobs to check.
Solasta Save.zip


Remove the concept of an experimental networks

We haven't really cared about experimental networks for a while, this
just finishes removing the leftover mentions of them in our code.

This change is

awesome thanks :+1:


Problems with VQA finetuning

Hello! I am trying to finetune OFA-large on VQA using Visual Genome dataset, using the finetuning instruction in the repo. Unfortunately, I have encountered a bug that I have some difficulties indentifying. I preprocessed the data exactly like in an example, but during training my gradients overflow and model does not train.
slice_id 0 seek offset 0
2022-03-28 02:29:07 - trainer.py[line:703] - INFO: begin training epoch 1
2022-03-28 02:29:07 - train.py[line:296] - INFO: Start iterating over samples
2022-03-28 02:29:09 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 64.0
2022-03-28 02:29:11 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 32.0
2022-03-28 02:29:14 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 16.0
2022-03-28 02:29:15 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 8.0
2022-03-28 02:29:17 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 4.0
2022-03-28 02:29:19 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 2.0
2022-03-28 02:29:22 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 1.0
2022-03-28 02:29:23 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 0.5
2022-03-28 02:29:26 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 0.25
2022-03-28 02:29:28 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 0.125
2022-03-28 02:29:28 - trainer.py[line:922] - INFO: NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 0.0625

I narrowed the issue to the answers column. If I replace this column in my dataset with the column in the dataset provided in the repo, everything works fine. However, if I change the answers in the column, or even modify them in any way I get the same issue. I suspected that my procedure of changing the column could be a problem, but if I "modify" the column with empty string, it still works. Any other symbol added to the column again concludes in an overflow. I also tried modifying not the whole column, but single elements, and found out that changing certain answers does not lead to an overflow, while changing others does. I was unable to further narrow the issue or find any pattern in it.
I train on single server with 1 GPU.

Hi Markov, for your custom answer candidate set, please also prepare a custom trainval_ans2label.pkl file (a pickled python dict mapping the answer-text to label-id) to replace the provided one. This file is used in training & inference which constrains the output space from the full vocabulary to only the answer candidate set. It should be conformed with your dataset, otherwise the overflow problem will arise during training if an answer unseen in the trainval_ans2label.pkl is encountered in your dataset.

Doesn't the existence of such file cripples the zero shot capability?

Hi, since we have utilized various sources of VQA samples during pretraining, for zero-shot (open-domain) VQA, we directly turn to the pretrained OFA-Large, which do not set this constraint. For more details of zero-shot VQA, please refer to the open-domain VQA Colab (url). The VQA fine-tuning process is specifically targeted for the VQAv2 challenge, whose answer is more restricted into the 3,129 candidate answer set. To achieve higher accuracy on this specific challenge, we use trie-based constrained training & inference using the trainval_ans2label.pkl file.

Hi @yangapku , please provide a given example for trainval_ans2label.pkl . I don' understand what a pair answer-text to label-id is?
Thanks

@phanxuanphucnd You can refer to the trainval_ans2label.pkl file we provided for VQAv2.

Hi @yangapku
A given sample of trainval_ans2label.pkl file such as:
{     "": 0,     "boats": 835,     "not at all": 2421,     "name": 1,     "harley davidson": 78,     "plain": 2,     "20 ft": 2379,     "museum": 3,     "parking": 1710,     "behind": 1590,     "steeple": 4,     "turning": 2380,     "tent": 836,     "no parking": 1995,     "tulip": 1568,     "low": 452,     "muffin": 1172,     "9:55": 846,     "hair": 453 }
I don't understand what does it mean? By what mechanism are indexes assigned?
Can you help me explain?
Thanks

Hi, it's a python-dict which does mapping from candidate answer text to its index (starting from 0). The indexes can be assigned just by random with no specific rules. Just make sure that each candidate answer is assigned with a unique index and the indexes are assigned continuously from 0. All the ground-truth answers of training and validation samples should be included in this candidate answer set.

Yes, i understand it such as:
All answers in the question-answer pair of the train and valid dataset must be included in this file, right?
THanks @yangapku

@phanxuanphucnd Yes. In our practice on VQAv2 dataset which has a long-tailed distribution of all the appeared ground-truth answers, we follow the common practice which uses the most frequent 3,129 answers as the candidate set to build this dict. Then we filtered the original training and valid split, only the question-answer pair whose answer is in this candidate set is kept for finetuning OFA.


Disable articles

Is there a way to disable articles? because I don't use them
Thanks

i rewrite the funciton in menus.php to solve this issue
create another menu by myselft, and in the sidebar.blade.php , I use my own menu.


partial update to json field

@a8m is it possible to update JSON partially using ent? we are planning to store settings as JSON field and wondering on how to do partial updates to the JSON structure?

Hello (again) @dilipkk-foyernet 😃
We don't support this feature at the moment, but it's definitely in our TODO list. I'll keep this issue open.


Make it easier to use test cafe with DevExtreme control

What is your Scenario?
At this moment I'm writing test cafe test which test DevExtreme control, I found it extremely hard and wondering if you guys from same company test it with test cafe? For example testing select box recipe is just html select box, try to test cafe DevExtreme select box, you will be surprised how difficult it is to implement
What are you suggesting?
Obviously we can't complain to DevExtreme to change how they render the component
I did try to get the instance of the component from test cafe, but does not seems to work
What alternatives have you considered?
Keep hacking but not sure anymore if its the right direction
Additional context
No response

Hello @andrew-vdb,
The DevExtreme team uses TestCafe to test their components. They also face similar challenges, so they create page models and helpers.
I think you're right, and it would be great to make it easier to test DevExtreme components with some kind of adapter. We will discuss this with the DevExtreme team.
Do you have any specific suggestions on how this might look like?

in the end, within my test cafe, i can call my vue component method (therefore it call devextreme method component)
So for filling the form and assert the value, i use vue component method
but to trigger the submit, still using test cafe click button
that make my life hundred times easier
would be nice if your team can also share the helper right?

I discussed this issue with the DevExtreme team.
The release of such a tool will require a lot of effort from the DevExtreme team and we do not expect it to appear anytime soon.
However, you can see how the DevExtreme team uses TestCafe here. You can use their helpers and models to write your own tests.


The MPI plugin should provide a configuration to transtype exposed communicators between C, Fortran and python

In GitLab by @jbigot on Oct 8, 2018, 15:14

closed via merge request !184


Deleted credential is still being used

v0.22.0-rc2
Steps:

Create a registry for DockerHub with wrong credentials (wrong username/password)
Delete and purge the registry
Try to create a docker:mysql container

Results:
Container won't create because it says I don't have authorization.
Expected:
Should create
Comments:
Looks like even thou the registry is deleted, it still is pulling those credentials

v0.23.0-rc2
Verified Fixed


Display Content Overlaying the Video: Profile name, Captions, Audio track name

This project is unique and amazing, thank you for your efforts! It would be great if, in addition to the video itself, part of the content that accompanies it on TikTok/Reels could also be displayed, such as:

Description/caption
Profile photo/name/link to that originated the post
Name of the audio track used

Replicating part of the app's experience, and allowing you to use Erin as a backup. It may be possible to make Erin, even without a backend, search for a .txt/.json/.yaml file with the same name as the video and display the metadata in the interface.

Hey,
Thanks a lot for the kind words, along with the detailed explanation of the feature.
I have been thinking about that feature for a long time, and gave up because I didn't know how to handle the "metadata" part. But you've just opened my eyes with the txt/json/yaml file. What a great idea.
I would like to ask your confirmation on the following flow before I implement it :

The user opens Erin and logs in
A video is played, the file's name is "video-345.mp4".
In the background, the client (via JS) sends a request to Caddy, trying to fetch "video-345.json".
Caddy replies with a file that has the following (made up) structure : https://pastebin.com/Sk367R2M
The metadata (in JSON) is parsed by JS, and displayed on the user interface, trying to reproduce Tiktok's design.
In the case that "video-xxx.json" doesn't exist, the client would simply keep playing the video, ignoring the "not found" error.

How does that look?
Thanks again!

Re,
Thanks for your confirmation.
Regarding your questions :

Data placeholder while loading : I think this is doable, but by default I had thought of the data just popping out into the UI. Since the UI is static (read, there's no layout reflow), the data would certainly just transition from invisible to visible, smoothly when it's loaded, and not trigger any reflow / movement of other parts.

Files organization : I think any organization will work, whether it's one video per folder, or all videos in one. Erin will, anyway, recursively scan all the directories. The most important thing should be to give a unique name to every video, and have the matching metadata file somewhere (with the exact same name, just a different extension), with the exact same name, so they can be associated internally.

Captions : The web client in Erin uses UTF-8 so, if I'm not mistaken, it should all work as expected with non-Latin characters, emojis, etc. However, user mentions and hashtags should certainly be encoded as raw HTML if you need the features you've mentioned (links, different colors, etc.). I believe the best way to satisfy all needs (present and future) is to allow raw HTML in the captions, so anyone can use their own formatting and style. And when no HTML is provided, the raw text will just be wrapped in a paragraph.

What do you think?
Thanks again for the kind words, along with the brainstorming!

Recursive search, independent folder organization, raw HTML for captions - it couldn't be any better. Thank you for clarifying, looking forward for the feature!

Alright, it's released.
You should be able to pull the latest image, and have it working.
I've added a little paragraph in the README about it, but else, just create a matching .json file for your videos, using the following structure : https://github.com/will-moss/erin/blob/master/examples/video-metadata.json and it should work.
Let me know if you encounter any issue.

Wow, that was pretty fast!
I just tested it, and it looks really cool!
Overall, I only have two concerns: Some captions can be very large and end up overlapping a large part of the video. I don't know if it would make sense to limit the height of the container and add a scroll bar to see the entire text.

I'm also a little concerned that the TikTok/Reels interface adds a transparent dark overlay to increase the contrast between the text and the video, and since we don't have it here, it might make the captions a little harder to read.

Thanks a lot for your feedback, and thanks too for the screenshot with your concerns.
I have just published a new release that should address both issues you've mentioned.
Now, the bottom layer with metadata has a semi-opaque background (10% black, 90% transparent), and the caption has a maximum height of 50 pixels with a scrollbar when the content overflows.
Is it better now?
Thanks a lot once again, I'm so happy with that feature.

Hi ! Just passing by to ask if I can close the issue, and most importantly, if everything works fine on your end.


feat: lu shell api

Description
update latest BF-LU parser, add section operate api in composer shell.
Task Item
refs #867
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code refactor (non-breaking change which improve code quality, clean up, add tests, etc)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Doc update (document update)

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have functionally tested my change

Screenshots
Please include screenshots or gifs if your PR include UX changes.

@liweitian  maybe you can check, can the create new dialog wizard use this?

And @alanlong9278, please take a look, make sure it satisfies the usage scenario.

@liweitian maybe you can check, can the create new dialog wizard use this?
Maybe you mean create new trigger wizard?
Talked with @zhixzhan on some code issue.
Tested on my side. An error happened when updating intent on an empty lufile. It should be a bug in @bfcomposer/bf-lu/lib/parser package.

@liweitian maybe you can check, can the create new dialog wizard use this?
@boydc2014 Maybe you mean create new trigger wizard?
Talked with @zhixzhan on some code issue.
Tested on my side. An error happened when updating intent on an empty lufile. It should be a bug in @bfcomposer/bf-lu/lib/parser package.

Talked to Fei, he will update it.


Add Inherited Render Point failing test

Hi @ruudk,
I added failing test with inherited template and render point.
As you can see, if render point inject title in child template, but the variable is use in parent template also, the variable.undefined error thrown.

Thanks for the failing test. The problem is clear, only I'm not sure yet about the solution.
I created an issue here with details:

https://github.com/twigstan/twigstan/issues/57

Will be solved with https://github.com/twigstan/twigstan/pull/135


Add support for ACF expiration dates up to 9999-12-31

ASN1_TIME_set did not work well with arbitrary large dates. This caused
input "current time" dates to fail, resulting in the ASN1_TIME_compare
to also fail.
Since the yyyy-mm-dd to unix time conversion used to convert expiration
time still functions properly for dates through 9999-12-31 remove the
extra converstion from unix time to ASN1_TIME and do the comparison
using unix time values.

This may not be that critical to fix. In my testing the input unix time starts failing sometime between the year 8051 and 8052. I only ran into this because I was testing the upper limit of an 8-byte unix time.
YEAR: 8051 | RC: 0
YEAR: 8052 | RC: 44


mitigate intra-channel upgrade

We will try to delete CSV ibm-common-service-operator-4.0.0 or ibm-common-service-operator-v4.0.1 when all following requirements are fulfilled

User pass in a channel which is not v4.0, so this is not a fresh installation on v4.0
Those CSVs are not in Subscription's installedCSV

This is used to solve following issue:

User would like to do following upgrade patch: v4.0.0 -> v4.0.1 -> v4.1.0
Common Service operator is stuck at upgrading v4.0.0 -> v4.0.1, see following screenshot for details

As a result, NamespaceScope operator could not be upgraded v4.0.0 -> v4.1.0 due to this error

constraints not satisfiable: @existing/test-mitigation//ibm-common-service-operator.v4.0.0 and @existing/test-mitigation//ibm-common-service-operator.v4.0.1 originate from package ibm-common-service-operator, subscription ibm-common-service-operator requires @existing/test-mitigation//ibm-common-service-operator.v4.0.1, subscription ibm-common-service-operator exists, clusterserviceversion ibm-common-service-operator.v4.0.0 exists and is not referenced by a subscription

How to test

Install a 4.0.0 base version with SOD topology, CatalogSource need to be pinned at 4.0.0 tag

./cp3pt0-deployment/setup_tenant.sh --license-accept -c v4.0 --operator-namespace test-mitigation --services-namespace test-miti

Update CatalogSource to cd tag
Execute setup_tenant.sh to perform the upgrade to v4.1

./cp3pt0-deployment/setup_tenant.sh --license-accept -c v4.1 --operator-namespace test-mitigation --services-namespace test-miti

...
[✗] Delete CSV ibm-common-service-operator.v4.0.0 in test-mitigation for upgrade mitigation

clusterserviceversion.operators.coreos.com "ibm-common-service-operator.v4.0.0" deleted
# Checking whether Namespace Scope operator exist...
[✗] There is an ibm-namespace-scope-operator subscription already deployed

The both CS and NamespaceScope operator should be upgraded to v4.1.0 successfully.

Test both passed with -c v4.0 and -c v4.1
./cp3pt0-deployment/setup_tenant.sh --license-accept -c v4.1 --operator-namespace test-upgrade --services-namespace test-upgrade-service -n test-upgrade -s opencloud-operators-mitigation -v 1

# Start to validate the parameters passed into script...
...

#  Checking whether Cert Manager exist...
[✔] Found only one Cert Manager exists in namespace test-upgrade

[✗] Delete CSV ibm-common-service-operator.v4.0.0 in test-upgrade for upgrade mitigation

clusterserviceversion.operators.coreos.com "ibm-common-service-operator.v4.0.0" deleted
# Checking whether Namespace Scope operator exist...
[INFO] Creating following Subscription:


guild.EveryoneRole.Mention returns incorrect string

Calling Mention on the everyone role returns <@&SERVERID> instead of @everyone.
While the everyone role's ID is the same as the server's, and <@&ID> is otherwise how you mention a role, it's not how you mention the everyone role. Mentioning the everyone role is literally @everyone.
<@&SERVERID> will produce an inexplicable non-existent phantom @@everyone role that doesn't highlight anyone.

comparing role IDs on every IRole.Mention access feels gross to me.

What would feel gross to me as a consumer would be seeing this IEveryoneRole for no real reason.
Fortunately, befitting of an implementation detail, it can all be made internal. This means it could all be done as a patch release, and we can delegate the decision to expose this stuff if it comes up for the next minor release.
internal interface IEveryoneRole : IRole
{
}

public class Role : IRole
{
    public ulong Id { get; }
    public virtual string Mention => $"<@&{Id}>";
    public bool IsEveryone => this is IEveryoneRole;
}

internal class EveryoneRole : Role, IEveryoneRole
{
    public override string Mention => "@everyone";
}

You guys would prefer making 2 classes and one interface over adding a server id check? o.O

Fox is the one against comparing IDs, so I'm putting in what can be done to compromise.

Personally, it's a bit overkill to not just compare ids.

Yeah, I don't like the idea of adding new classes when only one behavior changes.


Improve comments for more clarity/grammar/spelling

Modified the comments per James' feedback.

James already went in and changed the comments appropriately for me. Thx.


Triggering Settings Profile Quickpick using keyboard spacebar from status bar item pre-fills filter with space character

Issue Type: Bug

Use Tab to move focus to status bar.
Use right-arrow to move focus to Settings Profiles item.
Press spacebar to click the item (instead of Enter).

:bug: quickpick opens pre-seeded with a single space character, causing the placeholder text not to appear.
VS Code version: Code - Insiders 1.69.0-insider (0b3574dcef8f35fec4ee4f83dc958c1f16ef6fce, 2022-07-01T05:17:04.343Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

/assign @sandy081

This bug has been fixed in the latest release of VS Code Insiders!
@gjsjohnmurray, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 052d5b0027f6c9d64c8ca35955cb5117ba94d5d7 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified


the scroll don't work when I don't use container as ' body'

when i use body's children as the scroll container, it doesn't move. but when i use 'body' as container, it works. I am not sure what kind of problem cause it. css style ? or bug?

i have same problem

Please provide code/reproduction when opening an issue. I can't help without seeing what's going on.


New package: Thunks v0.1.0

Registering package: Thunks
Repository: https://github.com/tbenst/Thunks.jl
Created by: @tbenst
Version: v0.1.0
Commit: eb75df63278aaab9306ef0ab04a3ea4a5bd77d18
Reviewed by: @tbenst
Reference: https://github.com/tbenst/Thunks.jl/commit/eb75df63278aaab9306ef0ab04a3ea4a5bd77d18#commitcomment-54446486

[noblock] This is my first registered package, so if anyone has feedback it's much appreciated! thanks!


getFocusedBuilding()

It would be great to have access to this since indoor functionality isn't available on the javascript api yet.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/4942842-getfocusedbuilding?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github).

map. getFocusedBuilding() has implemented in the test branch. (#442)


Sorting doesn't work in Server Side Paging Example

In the below example sorting and Search functionality doesn't work. Could you please provide some working example?
http://josebalius.github.io/ngReactGrid/examples/serverSide.html

Hi
I am begginer  in angularjs and implementing the below example
http://josebalius.github.io/ngReactGrid/examples/serverSide.html
and using the code as mention in above comment
$scope.grid = {
...
getData: function() {
var grid = this;
// sorting info (field and direction) - grid.sortInfo.field, grid.sortInfo.dir
// search info - grid.search
// pagination info - grid.currentPage
mySpecialAPIService.getData(grid.sortInfo.field, grid.sortInfo.dir, grid.search, grid.currentPage).then(function(data) {
$scope.grid.data = data;
});
}
...
};
I am getting problem during cancel after click on edit button it shows No record found. I am using the below code.
$scope.cancel = function () {
$scope.editingGrid = false;
$scope.grid.cancel();
};
or i have to call my service  on cancel button press also. Please provide the solution for this if possible and if localMode: false then column filter is not showing, Could you please provide  the solution for this.
one more thing if any function available in this grid by which i can get edit only those rows which are checked and i want to get $scope.selections value for those rows because this time i am calling my service and again uploading the grid and assigning $scope.selections=[] in cancel function and then clicking on checkbox is showing blank.
$scope.cancel = function () {
// calling my service to load grid from server
$scope.selections=[];
$scope.editingGrid = false;
$scope.grid.cancel();
};
Any kind of help is really appreciable.


Add ApplicativeLocal and FunctorTell loggers

This PR adds two new loggers:

A logger that uses a value of cast.mtl.ApplicativeLocal as an additional MDC;
A logger that writes LogMessage entries into effect context using cats.mtl.FunctorTell;

These changes related to the https://github.com/ChristopherDavenport/log4cats/issues/143.

My only concern with having this in the main project is that cats-mtl is still pre-1.0 and we are post-1.0. Would you mind if I created a project for this that we included prominently that could be sub 1.0 versioned? Otherwise we risk a major on a change to mtl which makes me nervous.

@ChristopherDavenport I am completely fine with a separated project until the major release of cats-mtl.

Here is where I have the baseline of the project setup. https://github.com/ChristopherDavenport/log4cats-mtl
I can either move your work over, or a PR over there would be appreciated for us to get this rolling. The deploy is all setup, so it should just be a matter of getting the code in place.

I will create a PR

Done https://github.com/ChristopherDavenport/log4cats-mtl/pull/1


Change dependabot configuration

Check monthly for gh-actions
Ignore k8s dependencies included with apiextensions-apiserver

The new commit title isn’t great :-/. I’m fine with merging this as-is, but “Reduce dependabot frequency, target k8s bumps” would be more informative for the short log.


cached data internally in order to prevent multiple api calls on each filter modal mount

… filter modal mount

First thing, PLEASE READ THIS: ReactPlay Code Review Checklist

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots or example output

@Angryman18 Build Error:

@atapas review it


fix(EMI-2133): fix collections signals tracking

The type of this PR is: Fix

This PR solves EMI-2133
Description
Fixes the tracking signals for the collections, including using the hideSignals prop to only track signals we render to the user.

This PR should be reverted - there's some weirdness that crept in with this, auction pages aren't using the right kind of grid layout. Mentioned in Slack https://artsy.slack.com/archives/C02JHHHKP5K/p1729737714932469 but posting here for posterity.


Used p2p peers in p2p gossip protocol, big refactor.

Motivation
Closes #2017
Changes
Used p2p/peers in p2p/gossip protocol, instead of custom implementation. Rewrote gossip protocol unit tests. Added some cosmetic changes.
Test Plan
Ran p2p package unit tests, peers test and rewrote gossip tests

Will fix tests

Was some sort of CI bug :) @y0sher  ready for review

bors merge

@y0sher may we restart?)

@justplesh , yes sorry we had some ci issues. I hope it will be better now.
bors merge

Hm, out of range looks very strange, failing on shuffle.
https://github.com/spacemeshos/go-spacemesh/blob/5282bba4ccce31865e5b4902bde6a6af7e48ecdb/p2p/peers/peers.go#L52

let me know if there's trouble and you need help.

@y0sher so, I did some investigations
I found that specemesh rand package uses rand.NewSource as rand.New argument
https://github.com/spacemeshos/go-spacemesh/blob/b9fdeb9b230a607fa36e6e83225971de07cf1087/rand/rand.go#L12
As the official documentation says:

Unlike the default Source used by top-level functions, this source is not safe for concurrent use by multiple goroutines.

https://golang.org/pkg/math/rand/#NewSource
As a result, all concurrent usages of spacemesh rand package have race conditions, which luckily happened during the last unit tests.
If I am wrong - please correct me.

I can see the problem you are talking about with concurrently using rand. though I don't see how this would've caused the error we are seeing.

bors try

@y0sher

though I don't see how this would've caused the error we are seeing.

For example, rand.Shuffle is used concurrently, and the case is accessing indexes from other concurrently randomized value, that is out of bound of an array. This is a very huge bug and we need to fix it everywhere with rand. Rand package can't be used in such a way when concurrent access is possible. I guess the logic must be changed. We can implement it in another issue, but it is very important, we can't just ignore it.

I see. you are right this is a broader issue.
from golangs github -
If you need to get randomness from multiple goroutine,
the solution is to instantiate a random generator for
each goroutine.

so to solve current situation you can create a new rand and source for each Peers instance. this is the recommended usage pattern and it'll prevent these issue at least in Peers so we can conclude this PR.

Exactly, but I guess we need to create an issue and validate all usages of rand for sequential rand usage.

@y0sher used separate rand with Mutex - check please and let's run bors

I don't think the mutex is needed since Peers is created for each component and there's no use as of now that we do of GetPeers concurrently. let's drop the mutex and I'll run bors

hm, yep you are right, but let's leave a comment, that method is concurrently not safe

bors merge


Stop using bluebox gem mirror for now

It's under a lot of pressure and needs some time off to sort through its
life.

:+1:


Need help with $auth->getUser($uid);

https://stackoverflow.com/questions/58961336/unable-to-get-the-user-form-kreait-firebase-php-using-auth-getuseruid

https://stackoverflow.com/a/58961590/284325
Please don't cross-post Stack Overflow questions here.


Implementation commit.

Acabat el fifth assignment topic

The method  getMapByBrand() is wrong. Tip: use a Set to filter new brands that can be added in te future...


AutoLoad fails to load in an unintuitive way

Godot version
4.1.1 and 4.1.2 (v4.1.2.stable.official [399c9dc39])
System information
Windows 10 (Compatibility)
Issue description
I was trying to figure out why, at random intervals, my game started crashing and complaining about one of my AutoLoad not inheriting from a Node. after a day and a half of trial and error (I could not find someone with the same problem online) I managed to get it to work by just shuffling the AutoLoad order. It did not make any sense as one of the AutoLoad that was giving me an error was quite literally empty and could not possibly depend on other singletons. Then, i realized it wasn't random, the fix was specifically to move the AutoLoads that were causing the crash before my scene manager, which preloads every scene in the game and that's when it clicked. changing preload to load fixed the problem regardless of the order.
Conclusion: When an AutoLoad preloads scenes that use other singletons that haven't been loaded yet, it causes a crash and gives you some error message about the AutoLoad not inheriting from Node. If this cannot be fixed, changing the error message would be nice.
Steps to reproduce

Create a project.
Add two AutoLoad.
Create a scene that calls a function from AutoLoad2.
Preload the scene in AutoLoad1.
You should get an error that AutoLoad2 doesn't inherit Node and the program will crash when the function is called as AutoLoad2 is nil.

Note that you won't get a crash if you only refer to the AutoLoad, you have to call a method or refer to a variable as if it were a null pointer.
Minimal reproduction project
ErrorReport.zip

For the record, yes, all the AutoLoads involved had "extends Node" at the beginning.

I ran your MRP :

in 4.1.2 it does not crash on my side, but it writes an error "start: Script does not inherit from Node"
in 4.2.dev6 it works fine without crash or error

Did you test it on 4.2? I believe that many fixes where made on autoloads and gdscript side.

I did not test it on 4.2 but I'm happy to hear this. My bandaid fix on my end was to move around my AutoLoads and in one case to load scenes on the fly instead of with preload. At least I can carry on until 4.2 comes out.

Also, the project i sent does not crash because it does not use a method from the AutoLoad that fails to load. It does, however, fail to load and would act as a nullptr.

4.2.beta1 comes out soon, so I suggest trying it on your project after making a backup.

I'll wait for 4.2 stable to come out. Changing preload to load does the job and if I really need to, I can always change the AutoLoad order in the meantime. I'm just glad I figured out why my game was crashing.

4.2 stable is out. Did you confirm it's fixed?


GroupEmail issues

** problem is all fields get inherited from parent content definition. Try to uninherit with the tool and it comes back checked.
** symptom

condition fields should not show.
do not send before and do not send after show a date when not set (1/1/0001 12:00:00 AM)

fixed. cdef load into dictionary loads parent first, then overlays cdef


Rework scanner reset behavior

If Scanner returns a token, there is only one state where it cannot simply be reset with no resulting subsequent token after the caller is done with the token: pi_maybe_end. It would be a great simplification opportunity if this could be omitted, and the scanner could just be cleanly reset before starting on the next token in TokenReader.
This is also a reminder to me to double-check on the similar CDATA ending states, which look like they may be handled incorrectly, but I need to confirm (and add a comment explaining why the current behavior is correct if it turns out to be right).

No longer relevant with #36.


Laplacian Filtering.

Implementation of the same.

Implementation of the same.

@geekquad I am GSSoC'21 participant and I want to work on this issue. Kindly assign this to me.


Mount Updates

baseMounts->boxMounts
Cleaned up some of the echo statements - removed some pure debugging ones.
Updated to show the predefined shapes and masks in the console with the other global variables.

Fixed minor fillet issues with buttons (wasn't suppressing them if told to)
Mounts:
Renamed vector name.
Noticed mount positioning wasn't consistent - fixed.
Saw that with roundRadius > mount thickness fillet didn't touch case properly - fixed.
Fillet could extent into screw opening - fixed.
Added option for negative mount width.  This rotates the slot 90 degrees so it is perpendicular to the case instead of parallel.

See new boxMounts demo for examples.

@rosenhauer,
I have found something in RidgeExtDemo_v30:
ECHO: "YAPP=========================================="
[WARNING: Unable to convert translate([undef, 75, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 85, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 98, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 25, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 45, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 38, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 75, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 85, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 98, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 25, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 45, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
[WARNING: Unable to convert translate([undef, 38, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949](https://github.com/mrWheel/YAPP_Box/pull/1949,/Users/WillemA/Documents/openSCAD/YAPP_Box/library/YAPPgenerator_v30.scad)
Compiling design (CSG Products generation)...

I removed some old files and added some new files to the repo
Also:
I sometimes want to add pcbStands and/or connectors from an image like this:

As you can see all measurements are done from pcb[0,0,0] (as I have learned at school), but it is no longer possible to enter the coordinates for the stands with these values ... (or am I doing something wrong?)
In pré v3.0 I could enter this:
pcbStands =
[
     [3.6,  3, 2, yappBoth, yappPin]                     // back-left
   , [3.6,  pcbWidth-3, 2, yappBoth, yappHole]           // back-right
   , [pcbLength-3,  7.5, 2, yappBoth, yappHole]          // front-left
   , [pcbLength-3, pcbWidth-3, 2, yappBoth, yappPin]     // front-right
 ];

but with v3.0 I have to enter this:
pcbStands =
[
    [3.6, 3, 2, yappBoth, yappPin, yappBackLeft]      // back-left
   ,[3.6, 3.4, 2, yappBoth, yappPin, yappFrontRight]  // back-right
   ,[3.6, 7, 2, yappBoth, yappHole, yappFrontLeft]    // front-left
   ,[3.6, 3.4, 2, yappBoth, yappHole, yappBackRight]  // front-right

];

You have to calculate all stands as if the origen is pcb[0,0,0].
In this example it is not that difficult to translate, but there is no more relation with the original drawing.

On the RidgeExtDemo_v30 that is the old version.  There is a new
YAPP_RidgeExtDemo_v30.scad that should be used.
on the pcbStands.  The 6th param (yappBackLeft,yappBackRight...) tells it
what to use as the origin or origins. So you can do it as before if you
specify the prime origin (yappBackLeft) as the origin for each stand:
pcbStands =
[
[3.6,  3, 2, yappBoth, yappPin, yappBackLeft]
// back-left
, [3.6,  pcbWidth-3, 2, yappBoth, yappHole, yappBackLeft]
// back-right
, [pcbLength-3,  7.5, 2, yappBoth, yappHole, yappBackLeft]
// front-left
, [pcbLength-3, pcbWidth-3, 2, yappBoth, yappPin, yappBackLeft]
// front-right
];
I changed it to have the default the prime origin instead of
yappAllCorners.  Now it will work like before.  Will push update in a few
min after I test it.
On Tue, Dec 12, 2023 at 8:06 AM mrWheel @.***> wrote:

@rosenhauer https://github.com/rosenhauer,
I have found something in RidgeExtDemo_v30:
ECHO: "YAPP=========================================="
WARNING: Unable to convert translate([undef, 75, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 85, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 98, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 25, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 45, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 38, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 75, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 85, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 98, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 25, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 45, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
WARNING: Unable to convert translate([undef, 38, 0]) parameter to a vec3 or vec2 of numbers in file library/YAPPgenerator_v30.scad, line 1949
Compiling design (CSG Products generation)...
I removed some old files and added some new files to the repo Also:
I sometimes want to add pcbStands and/or connectors from an image like
this:
image.png (view on web)
https://github.com/mrWheel/YAPP_Box/assets/5585427/6c771f30-85a4-41f2-b518-95f4f84ab103
As you can see all measurements are done from pcb[0,0,0] (as I have
learned at school), but it is no longer possible to enter the coordinates
for the stands with these values ... (or am I doing something wrong?)
In pré v3.0 I could enter this:
pcbStands =
[
[3.6,  3, 2, yappBoth, yappPin]                     // back-left
, [3.6,  pcbWidth-3, 2, yappBoth, yappHole]           // back-right
, [pcbLength-3,  7.5, 2, yappBoth, yappHole]          // front-left
, [pcbLength-3, pcbWidth-3, 2, yappBoth, yappPin]     // front-right
];
but with v3.0 I have to enter this:
pcbStands =
[
[3.6, 3, 2, yappBoth, yappPin, yappBackLeft]      // back-left
,[3.6, 3.4, 2, yappBoth, yappPin, yappFrontRight]  // back-right
,[3.6, 7, 2, yappBoth, yappHole, yappFrontLeft]    // front-left
,[3.6, 3.4, 2, yappBoth, yappHole, yappBackRight]  // front-right
];
You have to calculate all stands as if the origen is pcb[0,0,0].
In this example it is not that difficult to translate, but there is no
more relation with the original drawing.
—
Reply to this email directly, view it on GitHub
https://github.com/mrWheel/YAPP_Box/pull/45#issuecomment-1852104116, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBTUKBBMRCGUMQW3IDXX6TYJBQFLAVCNFSM6AAAAABAQLR2NOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJSGEYDIMJRGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dave

I found this strange artifact that I can't get rid of...

I've not seen that before.  I messed around with different wallThickness
and RoundRadius and can't reproduce it.   Can you send the full script that
generated that?
Could you add a value

default = undef;
deflt = undef;
So, instead of using undef we can write default in parameter list.

To be consistent would you want to use yappDefault?
On Wed, Dec 13, 2023 at 8:42 AM mrWheel @.***> wrote:

@rosenhauer https://github.com/rosenhauer,
I found this strange artifact that I can't get rid of...
boxMounts   =
[
[0,           3.5, 15, 3, yappLeft, yappCenter]
,[shellLength, 3.5, 15, 3, yappLeft, yappCenter]
,[0,           3.5, 15, 3, yappLeft, yappLid, yappCenter]
,[shellLength, 3.5, 15, 3, yappLeft, yappLid, yappCenter]
];
Screenshot.2023-12-13.at.11.21.04.png (view on web)
https://github.com/mrWheel/YAPP_Box/assets/5585427/c87d7f10-883d-4f6a-b305-4267798eeb24
default = undef;
Could you add a value
default = undef;
deflt   = undef;
So, instead of using undef we can write default in parameter list.
—
Reply to this email directly, view it on GitHub
https://github.com/mrWheel/YAPP_Box/pull/45#issuecomment-1854044819, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBTUKF26TG5VXNVCE5EOULYJG5F3AVCNFSM6AAAAABAQLR2NOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJUGA2DIOBRHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dave

@rosenhauer,
You are amazing!
Problem is gone!


Merge modularization-main into modularization/frontend

Feature or Bugfix

Merge request

Detail

Merge modularization-main into modularization/frontend

Relates

#398

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The description is misleading:
Merge modularization-main into modularization/frontend but it's actually vice versa:
Merge modularization/frontend into modularization-main.

For the future: 9cfc07577d560593b8a9813606d63a37a8647678 deserves a dedicated PR

frontend/src/api/MLStudio should be deleted

I checked all the changes. The branch contains all the recent changes from the modularization-main.
All frontend changes from main were correctly translated to a new frontend structure from modularization/frontend (though we should address linting errors for those changes).
No business logic is lost
There are a few small issues in this PR. When they are addressed I will gladly approve this PR.
Great job, @itsmo-amzn! The merge was big, so you can be sure your effort deserves a medal 🥇


Fixed parsing of @import stylesheets with Firefox

Firefox doesn't allow javascript to access styles that were included into a css file with @include.
As a fix I wrapped the code into a try-catch block.

Duplicate of #51


Migrate Travis CI from Precise to Trusty

We want to use Travis CI default
https://docs.travis-ci.com/user/precise-to-trusty-migration-guide/
Related to #558

Deferring to post-4.0.0 (see https://github.com/SOCI/soci/pull/575#issuecomment-327534271)


Remove overlap between dotLess.Core and dotLess.ClientOnly

Imagine a scenario like this:
Third-party library A references dotLess.Core
Third-party library B references dotLess.ClientOnly
In this scenario, libraries A and B can't be used together in the same project, as dotLess.Core and dotLess.ClientOnly overlap in terms of namespaces and classes. Ideally this overlap should be removed, and dotLess.Core should have a dependency on dotLess.ClientOnly, just adding the additional functionality on top of it.
Also see andrewdavey/cassette#379

I too think this should be cleaned up. One package should reference the other.  However, I would add that while that change is being made, The package names should be cleaned up/reworked.  Core implies to me that it should be what is in ClientOnly, but a better name than Core could be chosen.  The name ClientOnly is totally opaque to me and implies nothing useful.  Also, I see no reason not to ship the command line compiler with the equivalent of the ClientOnly package.
Perhaps things should be further factored out, issue #328 suggests adding build targets which might make sense as its own package.

The naming is probably a source of much confusion. dotless.Core.dll and dotless.ClientOnly.dll are basically the same thing, with the difference that dotless.Core.dll has dotless.AspNet.dll ILMerge'd in (both assemblies have Pandora, ServiceLocation and whatnot ILMerged too).
We already have separate projects for Core, Compiler and AspNet -- we just do away with the distinction as a part of the build process. In the future, the package layout might reflect that:
dotless.Core - compiler infrastructure only
dotless.Commandline/dotless.Compiler - command line compiler tools
dotless.AspNet - equivalent to the current dotless package
And possibly dotless.Owin for projects that target ASP.NET 5 and the like.

with the difference that dotless.Core.dll has dotless.AspNet.dll ILMerge'd

Why do they need to be ILMerge'd? Seems like it'd be simpler if a NuGet dotless.AspNet package only included dotless.AspNet.dll, and had a dependency on the "core" package to pull in the core stuff.

And possibly dotless.Owin for projects that target ASP.NET 5 and the like.

For what it's worth, ASP.NET 5 doesn't use OWIN. Its design was inspired by OWIN and middleware you write for OWIN can be used with minimal changes (eg. see https://github.com/reactjs/React.NET/blob/master/src/React.AspNet/JsxFileSystem.cs and https://github.com/reactjs/React.NET/blob/master/src/React.AspNet/JsxFileMiddleware.cs) but they're two separate libraries.

with the difference that dotless.Core.dll has dotless.AspNet.dll ILMerge'd

Why do they need to be ILMerge'd? Seems like it'd be simpler if a NuGet dotless.AspNet package
only included dotless.AspNet.dll, and had a dependency on the "core" package to pull in the core
stuff.

You'll have to ask the earlier maintainers for that -- I wasn't around for that decision. :(

And possibly dotless.Owin for projects that target ASP.NET 5 and the like.
For what it's worth, ASP.NET 5 doesn't use OWIN.

Strictly speaking that's 100% correct -- http://owin.org/ lists ASP.NET vNext as an OWIN implementation. ;-)

The build process and this decision was made when there was no NuGet yet.
At the time I was a firm believer that it's more convenient to users to
have just one .dll to be referenced from inside their project. And any
project that had dotless as it's dependency was usually shipping it as a
.dll alongside their own so you would not get any issues.
In case the project came with the ClientOnly dll it also worked with the
Core anyways.
So we internalized all the stuff that's internal (Pandora Container was
used and written mostly because nobody uses it and it's essentially the
internal IoC of dotless exclusively) to make it more convenient and not
scare users by a ton of dlls that all have to be referenced and then
clutter their namespace.
Cleaning this up ( and the naming ) should be quite simple. And I'm all for
it. Now that probably 99% of the users just install the dotless NuGet we
can remove this.
greetings DAniel
Lauri Kotilainen<EMAIL_ADDRESS>schrieb am Fr., 15. Mai 2015 um
19:24 Uhr:

with the difference that dotless.Core.dll has dotless.AspNet.dll
ILMerge'd
Why do they need to be ILMerge'd? Seems like it'd be simpler if a NuGet
dotless.AspNet package
only included dotless.AspNet.dll, and had a dependency on the "core"
package to pull in the core
stuff.
You'll have to ask the earlier maintainers for that -- I wasn't around for
that decision. :(
And possibly dotless.Owin for projects that target ASP.NET 5 and the
like.
For what it's worth, ASP.NET 5 doesn't use OWIN.
Strictly speaking that's 100% correct -- http://owin.org/ lists ASP.NET
vNext as an OWIN implementation. ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/dotless/dotless/issues/304#issuecomment-102463271.

...nobody uses it...
Until now.
I need to inject parameters. You suggested at https://groups.google.com/forum/embed/#!topic/dotless/Q2_YQSKdC2c to provide an IParameterSource.
The infrastructure for it is there. All I need to do is create an Engine that takes a ContainerFactory instance that registers my IParameterSource.
Except. I can't make a ContainerFactory that registers anything, due to the Pandora parts being inaccessible.
I can see where the decissions leading to this come from. I'm sympathetic to it.
But I do think it is a shame that something was made that can be extended so nicely, only to break it by making an essential part internal.
When is the cleanup going to be?
Until then I will resort to getting the code and including it in my project.

 For what it's worth, you can use the ParameterDecorator implementation of ILessEngine to inject an IParameterSource to a wrapped ILessEngine -- that's how  QueryStringParameterSource is used, too.
As for the cleanup -- I don't have a timeframe for you, unfortunately. I had high hopes for working on dotless this autumn, but between work and house renovation, coming up with the energy for that has been a bit of a challenge.
Version-wise, I'd say 2.0 is where it is most likely to happen, as that's the perfect time for breaking changes. I was planning on a roadmap that only includes one minor version (1.6) prior to 2.0.

Errr....QueryStringParameterSource/AspNetContainerFactory also depends on access to Pandora.
Not going to subtype ParameterDecorator to tweak it. Good code needs to be left in peace. And considering that ILessEngines also need to be registered using Pandora, I am sceptical about getting that working anyway.
I am actually using a ContainerFactory subtype and an IParameterResource implementation just like QueryStringParameterSource/AspNetContainerFactory do,
I just had to pull the source for the dotless.Core project into my own solution, and put the Pandora dll in it separately. No code changes needed. Just making Pandora public. Then it worked like a charm. Just shows how good the code already is.
I'll keep an eye out for updates. Until then, the current solution is good enough.


ConnectionError: HTTPSConnectionPool(host='flagsapi.skype.com', port=443)

I'm getting this while login with this code
from skpy import Skype, SkypeChats
sk = Skype("email", "pwd")

Same here.
I've commented main.py:280 and it helps... (Yes, I know this is a bad solution, but I need a hot fix ASAP)
And the first time I've got this error was October, 12. Since that day I can't establish connection to flagsapi.skype.com (I've tried many geolocations). So I've tried to parse new address (in case if they changed it) from web.skype.com, I've tried to google something about it - nothing. Maybe it can help to solve the problem.
Thank you for SkPy and I hope you'll fix this in a new version (:

@tehdoorsareopen should i fork it and then install it by git path with commented 280 line from main.py?

@pankajPizone I've just commented it in my venv (site-packages/skpy), but I'd say it's one of the worst solutions you can provide (I did it only because of lack of time).
Another (better) solution is adding this code:
import requests
from skpy import Skype, SkypeAuthException, SkypeChats, SkypeConnection, SkypeContacts, SkypeSettings, SkypeTranslator

class FixedSkypeSettings(SkypeSettings):
    def __init__(self, skype=None, raw=None):
        super(SkypeSettings, self).__init__(skype, raw)

class FixedSkype(Skype):
    def __init__(self, user=None, pwd=None, tokenFile=None, connect=True):
        super(Skype, self).__init__(self)
        self.conn = SkypeConnection()
        if tokenFile:
            self.conn.setTokenFile(tokenFile)
        if user and pwd:
            self.conn.setUserPwd(user, pwd)
        if connect and ((user and pwd) or tokenFile):
            try:
                self.conn.readToken()
            except SkypeAuthException:
                self.conn.getSkypeToken()
        self.contacts = SkypeContacts(self)
        self.chats = SkypeChats(self)
        self.settings = FixedSkypeSettings(self)
        self.translate = SkypeTranslator(self)

class FixedSkypeEventLoop(FixedSkype):
    attrs = FixedSkype.attrs + ("autoAck",)

    def __init__(self, user=None, pwd=None, tokenFile=None, autoAck=True, status=None):
        super(SkypeEventLoop, self).__init__(user, pwd, tokenFile)
        self.autoAck = autoAck
        if status:
            self.setPresence(status)

    def cycle(self):
        try:
            events = self.getEvents()
        except requests.ConnectionError:
            return
        for event in events:
            self.onEvent(event)
            if self.autoAck:
                event.ack()

    def loop(self):
        while True:
            self.cycle()

    def onEvent(self, event):
        pass

class MySkype(FixedSkypeEventLoop):
    def __init__(self, user=None, pwd=None, tokenFile=None, autoAck=True, status=None):
        super(FixedSkypeEventLoop, self).__init__(user, pwd, tokenFile)
        self.autoAck = autoAck
        if status:
            self.setPresence(status)

    def onEvent(self, event):
        **Your Code Here**

See #113.


wait until all the images are done loading in About page

Wondering if its possible to wait  until all the contributors imgs are fully loaded. Atm, it will show orange dots until the image loads

I can implement a view more or scroll to reveal/load feature in this section.
Please assign the issue to me.

Thanks for assigning it to me.

@iswarmondal are you working on this still?


[FR] Add to quicklisp

Hi @NightMachinary . I appreciate the support, unfortunately I'd like to avoid putting this on Quicklisp until it's more complete. At a minimum I wish to have:

A roundtripping reader/writer (output JSON exactly as it was read)
Object coercion mechanism for reading (eg read JSON as a CL class/structure)

Until these are done, I don't want to pollute QL with yet another JSON library.

+1 for adding to Quicklisp.
IMHO, jzon is already the best JSON library for rendering JSON views in web app.
So, I'd like to switch from Jonathan in my libraries, but can't because jzon is not in Quicklisp.

@fukamachi Thanks for the support.
Unfortunately I haven't been doing much programming in my free time as of late.
But if you think the library is useful as-is, then it's fair to add it to quicklisp, and I can add my other desired features over time as I can.
I just don't want to continue to contribute to the 100 JSON library meme on Quicklisp unless I offer some substantial benefit

@Zulu-Inuoe I think that jzon is a new generation JSON parser taking into account advantages and disadvantages of previous generation of parsers. And it has full coverage of JSON standard verified by third-party test suite. This is already good reasons for releasing v1 on quicklisp.

Yes, can you please add this to quicklisp, the pretty print feature is exceptionally useful :)

Throwing my hat into the pile. I'd like to start using this.

Pretty please?

Oh my gosh. Let me see what I can do this weekend and I'll add v1 on Monday no matter what

haha jk did I say that weekend? I meant this one.
In all seriousness I've been doing some work and thinking over the week and will definitely have something solid here soon.
Thanks all

Unfortunately I am a busy tortoise :(

Do you need any support with that - as in, what would happen to me if I did things the naïve way and made a quicklisp-projects issue pointing to the current master branch?

I honestly don't remember the status of what things are WIP. I believe there's still a couple of issues with stringify, including the floating point printer, the string replacer, and the auto-coercion stuff in general.
As before it's ultimately: I still don't know if it offers anything truly above and beyond the other json libs on QL already.
The other issue is that if we make available a beta version as-is, it'll be hard to fix things without breaking users - the auto-coercion stuff especially scares me.

Alright this is actually finally happening. All my remaining stuff for v1 are resolved and I've decided what to leave out for a v2.
Code-wise, the true final final stuff is whether or not I keep the alist/plist heuristics in write-value which I am awaiting review.
The other changes are things I can do without breaking backwards-compat, like updating docs and optimizing my string-on-string parser

https://github.com/quicklisp/quicklisp-projects/issues/2256

Great, thank you!

Added and available; huge thanks! I think this issue can now be closed.

Thanks for the heads up, @phoe


Pin/Autohide option in panes

I'll appreciate it if you add a option to pin/autohide panes. e.g. By debugging I use the debug console en de code pane. so if i'm searching in the code, i want to autohide the debug console and vice versa. In summary: the same autohide functionality in visual Studio but then implemented in Visual Studio Code.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


net: Add buffer pool to replace connection allocation

Summary
Patches included:

net: Add buffer pool to replace connection allocation
net: Optimize ipfwd and wrbuffer by buffer pool

Our net socket connection allocations are powerful but redundant because they're implemented once in each protocol.  This is not good for further optimizing and extending to other allocations, so maybe we can add a common implementation for the usage.
Impact

We add a struct net_bufpool_s as pool descriptor, which may use a little bit more memory than previous implementation (~28Bytes).
We share same functions between pools, so code size may shrink under some scenarios.

Testing
Mainly on SIM.

tcp/udp/icmp/devif_callback tested with different configs, both conn and wrbuffer pools.
Other pools share exactly the same logic, not tested very carefully.

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

Already merged as https://github.com/apache/nuttx/pull/15285


Eng 4192 artemis testing

Description
Ensured compatibility with artemis api server and it's api gateway be ensuring proper headers are set and routes are updated.

What Changed and Why? :

routes updated to refer to artemis routes

Type of change :

[ ] New feature

will add the version prefix /api/v1/ to the routes in terraform, the corresponding route should be updated in krakenD gateway


minor fixes

Summary

[x] Properly reveal compilation error for relational property mapping (like what we did for M2M and FlatData)
[x] temporarily disable grammar roundtrip test for persistence DSL
[x] fix broken test introduced by #1010
[x] revert the workaround introduced in #1162
[x] Temporarily disable broken DSL Persistence grammar roundtrip test
[x] Temporarily hide global test runner from activity bar

How did you test this change?

[ ] Test(s) added
[ ] Manual testing (please provide screenshots/recordings)
[x] No testing (please provide an explanation)

now this is a minor fix PR :)


Need method name highlight in code outline.

Issue Type: Feature Request
It will be great if we can select a line in a method and this method name is automatically selected in the Code outline. If user selects the Code outline then user is already having the method Name, which I think is great.
VS Code version: Code 1.25.0 (0f080e5267e829de46638128001aeb7ca2d6d50e, 2018-07-05T13:02:28.333Z)
OS version: Darwin x64 16.7.0

/duplicate of #52981


gitAndTools.git-recent: 1.0.3 -> 1.0.4

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
/tmp/tmp.Laok6v3TeR /home/ryantm/.cache/nixpkgs
/home/ryantm/.cache/nixpkgs
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/git-recent -h got 0 exit code
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/git-recent --help got 0 exit code
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/git-recent help got 0 exit code
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/.git-recent-wrapped -h got 0 exit code
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/.git-recent-wrapped --help got 0 exit code
ran /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/.git-recent-wrapped help got 0 exit code
found 1.0.4 with grep in /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4
directory tree listing: https://gist.github.com/a5c51e3598313240f56c2e95a8ddf85e

cc @jlesquembre for review

Success on x86_64-linux (full log)
Attempted: gitAndTools.git-recent
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin
patching script interpreter paths in /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4
/nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4/bin/.git-recent-wrapped: interpreter directive changed from "/bin/bash" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4...
/nix/store/0qpj62pcv8gvlyimr7bmmhnq2y4kq59c-git-recent-1.0.4

Success on x86_64-darwin (full log)
Attempted: gitAndTools.git-recent
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
strip is /nix/store/vb6mgjqgd9h11nbv60fvrp1ls9nxck2y-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/lm40ay7zsaa5nvmab07ldw534nn3mbi3-git-recent-1.0.4/bin
patching script interpreter paths in /nix/store/lm40ay7zsaa5nvmab07ldw534nn3mbi3-git-recent-1.0.4
/nix/store/lm40ay7zsaa5nvmab07ldw534nn3mbi3-git-recent-1.0.4/bin/.git-recent-wrapped: interpreter directive changed from "/bin/bash" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/bash"
/nix/store/lm40ay7zsaa5nvmab07ldw534nn3mbi3-git-recent-1.0.4

Success on aarch64-linux (full log)
Attempted: gitAndTools.git-recent
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4/bin
patching script interpreter paths in /nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4
/nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4/bin/.git-recent-wrapped: interpreter directive changed from "/bin/bash" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4...
/nix/store/69y6g563pj7gk27hxfnlqy63181zz60d-git-recent-1.0.4

Good for me


fix(deployment): wrong labeling templating

The previous PR added a regression in the Helm Chart that was preventing its render.
NOTE: the issue was due to editor auto-formatting

TBR!