Removal of get_or_create profile

is it possible for you guys to implement this back in?
one of the reasons I use klaviyo is so that I don't have to manage and track who is subscribed to what - with the new changes I now have to either run multiple calls to find out who a user is and what they are subscribed to, rather than just sending an email to an endpoint
also - to be clear, without this get_or_create profile - for me to add someone to a list I need to:

run a get query to retrieve all profiles with an associated email
write logic to determine if the profile is correct or not
if no profile, create a new with required params
otherwise request to add to the list with the id returned

Hi @wdifruscio
We do have an upsert profile API that is a "create or update" with a single call, and returns the profile that was created/updated.  (Docs)
Is this what you mean?  This API should still be available

I'd second that a get_or_create method would be helpful. I ended up implementing my own method which calls klaviyo.Profiles.get_profiles(filter=f'equals(email,"{email}")') and if no profile is returned will call klaviyo.Profiles.create_profile(data). Recently, we've had 4 or 5 errors with the method as the get_profiles returns none but the create_profile call throws and ApiException with A profile already exists with one of these identifiers.. I'm assuming there is some kind of race condition happening so I'll be rescuing it and retrying the get_profiles call but would be helpful if this was handled within the library itself and I didn't need to implement the custom logic myself.

@nburt sorry for the delay in reply. would the create_or_update as @jason-myers-klaviyo  method not work for you in this scenario?

Hi all,
sorry  - I've been meaning to reply here
Yes I did implement this, but I found the entire migration to this new system to be pretty irritating, and the documentation to be sub-par.  It is a considerable cost to run these services, if this was open source I would be more forgiving and probably would not be publicly whining
I totally get why things were moved into the system that they are now - it is standardized and basically follows a known convention - which in theory is very nice - but documentation and path to upgrade could have been well thought out - the emails to upgrade were nice, but actual information was not great
This change cost a considerable amount of hours, which really should have been done within an afternoon, as the original implementation was, so I am bit dissapointed in everything. I use this sdk with several companies and had to painfully line by line the new arduous implemention with multiple developers who were equally annoyed and confused

@sanfordj our use case has been that we're trying to find or create a profile prior to sending events for them (viewed product, started checkout, placed order) so that we can send the profile ID along with the event. I suppose we could do a create_or_update with empty values but we aren't actually updating the profile so I think that a find or create API would be more appropriate.

We appreciate your feedback, and apologize for any frustration you experienced here. We're always trying to improve our SDK documentation.
@wdifruscio, for your use case, we'd recommend using create_or_update_profile
for the relevant identifier (e.g. email address). This will create a new profile if one doesn't exist, or return the profile if it already exists. Then you can add that profile to a list via create_list_relationships.
@nburt, for your use case, we'd recommend one of the following options, with (1) being the simpler route:

Call create_event with the relevant  profile identifier (e.g. the email). Profile ID is an optional field; it is not required. This will create the profile if one doesn't exist already or find the relevant profile prior to creating the event.
2)Use create_or_update_profile for the provided email address. This will create the profile if one doesn't exist; if not it will return a profile that already has that . Then you can call the create event endpoint using the profile ID.


Unhandled Exception: System.IO.FileNotFoundException

I never used nor coded F# and I only know C# and VB.NET so I don't what any of this means. I understand that it couldn't find a file, but I don't know what file.
 The system cannot find the file specified.    at System.ModuleHandle.ResolveType(RuntimeModule module, Int32 typeToken, Int Ptr* typeInstArgs, Int32 typeInstCount, IntPtr* methodInstArgs, Int32 methodInst Count, ObjectHandleOnStack type)    at System.ModuleHandle.ResolveTypeHandleInternal(RuntimeModule module, Int32 typeToken, RuntimeTypeHandle[] typeInstantiationContext, RuntimeTypeHandle[] met hodInstantiationContext)    at System.Reflection.RuntimeModule.ResolveType(Int32 metadataToken, Type[] ge nericTypeArguments, Type[] genericMethodArguments)    at System.Reflection.CustomAttribute.FilterCustomAttributeRecord(CustomAttrib uteRecord caRecord, MetadataImport scope, Assembly& lastAptcaOkAssembly, Runtime Module decoratedModule, MetadataToken decoratedToken, RuntimeType attributeFilte rType, Boolean mustBeInheritable, Object[] attributes, IList derivedAttributes, RuntimeType& attributeType, IRuntimeMethodInfo& ctor, Boolean& ctorHasParameters , Boolean& isVarArg)    at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule decora tedModule, Int32 decoratedMetadataToken, Int32 pcaCount, RuntimeType attributeFi lterType, Boolean mustBeInheritable, IList derivedAttributes, Boolean isDecorate dTargetSecurityTransparent)    at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeAssembly asse mbly, RuntimeType caType)    at System.Attribute.GetCustomAttributes(Assembly element, Type attributeType,  Boolean inherit)    at System.AppDomain.GetTargetFrameworkName()

FSharp.Core.dll is not in your current working directory (or in one of the directories in your $PATH %PATH%), so the DLL can't be loaded. Copy it to, for example, your current working directory. (It should be in the output directory if you built GMSD from source, though...)
And sorry for the potential creepiness of my quick response time.

 I installed language support for F#. I tried to compile and getting the error then when trying to run it like described above. Where I get the DLL to run it?

Okay, figured it out. Set to latest supported F# version in VS


BPN issuing optimization

Description
The general BPN (business partner number) is structured that way:

E.G.: BPNL 1234 5678 90ZZ
This is defined in the standard: CX - 0010 Business Partner Number
But its not defined how the checksum is / should be generated.
The current reference implementation generates the checksum solely based on the 'identifier', in accordance with ISO 7064 MOD 1271-36. This feature should be enhanced to include both the prefix and the classification, in addition to the identifier, for checksum generation.
Acceptance criteria:

[x] BPN checksum will be generated by prefix+classification+identifier

Dependencies:
Just inside the BPDM product
Additional information

[x] I'm willing to contribute to this feature with development resources (@nicoprow / @SujitMBRDI / +more)

This would invalidate any BPNs issued by the operating company and is thus a breaking change. Have we talked to the operating company about that?

Yes they are aware and support this feature.
Would be nice if we could adjust / extend the standard with this guidance. I will bring this to the expert group.

To adapt the standard I will create a change request in Houston and I will create a issue in the GitHub of the Catena-X association to track it.
@Sebastian-Wurm will contribute to do the changes in the standard.

Hello @nicoprow, @maximilianong,
In a few weeks, we want to publish Tractus-X Release 24.08. For this, we need a clear status about the features (which will be included in the release and which will be cancelled or postponed to the next release).
Currently, the feature is still in status Inbox. Please update the status by 09.07.24 eob.
If the feature is not ready for release 24.08, or it is possibly not planned at all, please set the status to Not in release. Please remember that in this case, any dependent components must be informed.
Thank you and best regards.

Hi @stephanbcbauer this is one of those features we discussed in our last committee meeting. We did not present this in the open planning meeting for 24.08 but we saw this during the PI as a relevant feature, developed it and merged it. So it will be in the release 24.08.

This is part of the 24.08 release.


Cannot compile kernel on kirkstone

bitbake virtual/kernel results in 1977 errors, mostly
pinctrl-tegra194.c with the structure tegra_pingroup no member named <...>
a little extract of log.do_compile:
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:430:18: error: 'const struct tegra_pingroup' has no member named 'parked_bit'; did you mean 'pad_bit'?
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:439:18: error: 'const struct tegra_pingroup' has no member named 'e_io_hv_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:441:18: error: 'const struct tegra_pingroup' has no member named 'gpio_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:446:18: error: 'const struct tegra_pingroup' has no member named 'pbias_buf_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:447:18: error: 'const struct tegra_pingroup' has no member named 'preemp_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:448:18: error: 'const struct tegra_pingroup' has no member named 'rfu_in_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:453:18: error: 'const struct tegra_pingroup' has no member named 'pwr_domain'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:916:8: error: initialization of 'signed char:6' from 'char *' makes integer from pointer without a cast [-Werror=int-conversion]
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:916:8: error: initializer element is not computable at load time
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:430:18: error: 'const struct tegra_pingroup' has no member named 'parked_bit'; did you mean 'pad_bit'?
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:439:18: error: 'const struct tegra_pingroup' has no member named 'e_io_hv_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:441:18: error: 'const struct tegra_pingroup' has no member named 'gpio_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:446:18: error: 'const struct tegra_pingroup' has no member named 'pbias_buf_bit'
/data/workspace/kirkstone/build/tmp/work-shared/guidance-jetson-xavier-nx-specs/kernel-source/nvidia/drivers/pinctrl/pinctrl-tegra194.c:447:18: error: 'const struct tegra_pingroup' has no member named 'preemp_bit'

Really hard to say based on the little data provided.  The machine name isn't one of ours, so I suspect you have other layers in your build that could be breaking things.

@intera91 a suggestion is to attempt to reproduce at https://github.com/OE4T/tegra-demo-distro and if you can't use this to bisect differences in your custom setup.

Slighthly off-topic but @madisongh , I see the kirkstone branch has 35.5.0 support and is being maintained. Any plans to have 36.3 on kirkstone too? Just the kernel, oot drivers and the uefi firmware would be fantastic to have
Thank you

@acostach No plans for an R36-based kirkstone branch. We do have an R35-based scarthgap branch we'll be maintaining to help stage migration between LTS branches and between R35 and R36 BSP releases.

Solved in kernel 5.10.192 (JP 5.1.3)


module not working

Hi,
I want to test your module electron-scroller, I'm following steps listed in your description but it does not work for me.

I just tried it and it works.  Unless you can be more specific, I'm afraid I cannot help you.

Actually, I'll make one change to the browser.js code in the example.  The call to scroller.add() may need to go inside a function:
const scroller = require('electron-scroller')

onload = function() {
  scroller.add()
}


Agregar imagenes

An implementation is performed to replace the template image.
The steps to relate it were those provided in the Issue  #3

Coverage decreased (-1.6%) to 96.815% when pulling 1e130f0037f75673edb3e2afa66f246b401c9495 on JonathanLSignot:agregar-imagenes into 883c55a17bbe2262c3aaeb1b64ad72976c1a1a6e on guigrpa:master.

First of all, thanks for the PR. I think it adds a really interesting functionality.
Before merging, I think the API could be polished a bit. What I had in mind was something like this:
replaceImages: {
  'image1.png': '/absolute/path/to/newImage1.png',
  'image3.png': '/absolute/path/to/newImage3.png',
},

Configuration option is called replaceImages rather than images, which I believe is more descriptive of what the new functionality is about.

A plain object, rather than an array: simpler, and it prevents (upredictable) duplicate substitution of a single file.

Original file names, rather than id: the id values you propose rely on a certain file ordering coming out of readdir() (and the assumption that this ordering matches what the user sees in Windows Explorer, the OS X Finder, or his unzipper utility). Better use the filename itself, which is unambiguous.

New files: I see you are passing the contents of the files themselves, but in a prior commit (#62323b5) you included the target file paths. Why did you switch? I think it's better to use target paths and leave loading the target file to docx-templates; this way you can use fs-extra's copy() or copySync() utilities, and don't need to load the whole image in memory.

I look forward to your feedback!

Hi @guigrpa , this is fine I understand the changes, I only have one question, I need it to be base64 can I leave it as an option for basea64 or path?
something like that:
base64 : false,
replaceImages: {
  'image1.png': '/absolute/path/to/newImage1.png',
  'image3.png': '/absolute/path/to/newImage3.png',
},

Coverage decreased (-2.2%) to 96.226% when pulling dbcee656e35930d534407e284491ef34917794ba on JonathanLSignot:agregar-imagenes into 883c55a17bbe2262c3aaeb1b64ad72976c1a1a6e on guigrpa:master.

I will merge it now, thanks!
In any case, some names may change in the final API. Watch out when you upgrade!


Optional UIBlurEffectStyle with default value is inconsistent with one constructor

I think UIBlurEffectStyle should have a default value of NIL for all Init Signatures instead of .Dark, the reason is that there is one Init constructor with a default value of NIL but then all of the other Init's have a default value of .Dark and it is not consistent.
Also because the default value for backgroundColor? is also NIL for all the constructors.
Thanks

yeah, new version moves to an enum with associated value for the background https://github.com/JanGorman/Agrume/blob/4e015235962d0fbe5ca85549e2c871aff1b47ab7/Agrume/Agrume.swift#L8


feat(Datatable): table header

Contributes to #1931
WKC data table does not include Carbon for IBM Products PAL table headers.
What did you change?

title and description props(supplied via datgridState) added in <TableContainer /> component.
<DatagridToolbar /> moved inside <TableContainer />
Fix overflow scroll of table container

How did you test and verify your work?
Storybook

@matthewgallo Thanks for spotting the issue in the left panel. I have fixed that one in the last commit and now it looks like the following screenshot.

I have moved the left panel inside TableContainer to fix that issue.

Hey @Ratheeshrajan, the fix for the left panel story/placement of the toolbar looks great. But my only reservation now is that there seems to be double scrollbars on some cases which is something we should fix.
This screenshot is from the Batch actions story:

@matthewgallo fixed the double scrollbar issue. Please have a look.

Also just noticed that the data table header now blends into the table. Might have been a conflict when you merged from main?

@elycheea can you please review @Ratheeshrajan response?

@davidmenendez We discussed via Slack. Ratheesh will add the dense header variant and will follow up with design on clarification since the styles are coming from the Carbon default.

@elycheea @matthewgallo For dense header, I added one data table header variant. I'm also waiting for Marion's response on the grey10 header background.

Discussed with both @elycheea and @Ratheeshrajan over slack but there seems to be some layout issues with the pagination appearing at the bottom of the page in the deploy preview. After that is fixed, I think we'll be ready to merge this one!


Use Selinux policy for GPU workloads

Summary
The GPU workloads requesting the plugin created resource gpu.intel.com/i915 need access to /dev/ folder of the host node. Addresses https://github.com/intel/intel-technology-enabling-for-openshift/pull/104
Detail
GPU Workload pods requesting gpu.intel.com/i915 resource cant be executed- until they have access for /dev/ on the GPU node.
This can be achieved by setting- setsebool container_use_devices on on host node. This is not feasible to implement if a cluster has multiple GPU nodes and this permission has to be set on each node manually.
Solution
We can use user-container-selinux- container_device_t selinux option for running the workloads.
seLinuxContext:
  seLinuxOptions:
    type: container_device_t

To use this selinux option in pod/workload yaml's, we need to create a corresponding custom SCC  and bind it to a service account. This service account and selinux option can now be used in the workload.

@vbedida79 which milestone is this targeted for, please add the milestone. Thanks!

Solution  with container_device_t option,  fails with
type=AVC msg=audit(1693240354.718:400): avc:  denied  { getattr } for  pid=1634424 comm="ls" path="/dev/sda1" dev="devtmpfs" ino=65545 scontext=system_u:system_r:container_device_t:s0:c17,c28 tcontext=system_u:object_r:fixed_disk_device_t:s0 tclass=blk_file permissive=0 type=AVC msg=audit(1693240354.718:401): avc:  denied  { getattr } for  pid=1634424 comm="ls" path="/dev/sda" dev="devtmpfs" ino=65544 scontext=system_u:system_r:container_device_t:s0:c17,c28 tcontext=system_u:object_r:fixed_disk_device_t:s0 tclass=blk_file permissive=0 type=AVC msg=audit(1693240354.718:402): avc:  denied  { getattr } for  pid=1634424 comm="ls" path="/dev/log" dev="devtmpfs" ino=22631 scontext=system_u:system_r:container_device_t:s0:c17,c28 tcontext=system_u:object_r:devlog_t:s0 tclass=lnk_file permissive=0 type=AVC msg=audit(1693240473.700:403): avc:  denied  { map } for  pid=1637667 comm="clinfo" path="/dev/dri/renderD128" dev="devtmpfs" ino=360931 scontext=system_u:system_r:container_device_t:s0:c17,c28 tcontext=system_u:object_r:dri_device_t:s0 tclass=chr_file permissive=0
We need to check for right permissions, or missing options in custom SCC. For v1.0.1, this PR workaround needs to be followed- https://github.com/intel/intel-technology-enabling-for-openshift/pull/104

The device plugins give access to the devices the workload requested. So workload can run without custom policy/permissions. But on the node SELinux blocks access anyway because it has no concept of containers. That's why they designed that setsebool container_use_devices . So for plugin workloads to have access to the devices plugins exported, The Node must have container_use_devices set to on. its off by default. The alternative is to use custom Selinux Label and SCC.

@rhatdan We have a question about the how the SELinux label translation between device files volume mounted in a pod and on the node. We have a workload pod that requests resource from device plugin and as the result that device is shared with the workload. The device file is visible to the pod as container_file_t and on the Node its device_t . The pod can access the file without getsebool container_use_devices on. The translation works in this case.
But in another case (GPU), the device file visible to the pod is container_file_t and on the node its dri_device_t. But in this case SELinux blocks the access and we see the following error.
Is this a bug that GPU devices dri_device_t are treated differently.
type=AVC msg=audit(1694192866.178:142): avc:  denied  { read write } for  pid=2741395 comm="clinfo" name="renderD128" dev="devtmpfs" ino=124588 scontext=system_u:system_r:container_t:s0:c17,c28 tcontext=system_u:object_r:dri_device_t:s0 tclass=chr_file permissive=0
Thanks.

We are currently testing a fix from Dan. https://github.com/containers/container-selinux/pull/268

This issue is fixed in 1.2.0 release https://github.com/intel/intel-technology-enabling-for-openshift/releases/tag/v1.2.0


.

Hello @williamcruzme
I don't know why when i set persistent to true in nuxt js an errors appears like this 'Error _classPrivateFieldGet(...) is null', is there any solution to solve this problem?

Facing the same error


Wen restart send heaviest fork earlier when first hearing from supermajority.

Problem
In the previous implementation, we only send out HeaviestFork when ready to exit or every 30 minutes, this is a problem:
Imagine 4 validators in wen_restart, each holding 20% stake.

When they restart roughly at the same time, they will send out first HeaviestFork where observed_stake is close to 0, because Gossip message propagation takes time
Then they all get the HeaviestFork from others, so now if they send out HeaviestFork, observed_stake will actually be 80%, and we should be able to exit. But we don't send out HeaviestFork until 30 minutes later, so everyone just sits idle for 30 minutes.

Summary of Changes
There are two ways to fix this:

We can send out HeaviestFork the first time observed_stake passes 80%
We can just send out HeaviestFork more frequently, let's say every minute or even every 10 seconds.

1 means fewer messages, but the logic is more complicated and error-prone. What if the validator just died in the middle of sending that important HeaviestFork, we probably would send out another HeaviestFork upon restart according to current code, but if the code changes in the future this might break.
So I slightly prefer 2, just make it every 10 seconds, then we don't need any additional conditions. Worst case after 10 seconds we will get another Gossip refresh. Since HeaviestFork is ~100 bytes, and at this stage everyone is doing nothing else other than sending and receiving HeaviestFork, I think HeaviestFork being a bit spammy is not a big concern.
I'll add tests if we decide on 1 or 2.

What if the validator just died in the middle of sending that important HeaviestFork, we probably would send out another HeaviestFork upon restart according to current code, but if the code changes in the future this might break.

This seems like the right move, we could even log a timestamp into the log everytime we send this so that on a restart of the validator we don't resend this.

We can just send out HeaviestFork more frequently, let's say every minute or even every 10 seconds.

This validator crashing problem seems like it can happen here as well, so option 1 seems strictly better.

What if the validator just died in the middle of sending that important HeaviestFork, we probably would send out another HeaviestFork upon restart according to current code, but if the code changes in the future this might break.

This seems like the right move, we could even log a timestamp into the log everytime we send this so that on a restart of the validator we don't resend this.

We can just send out HeaviestFork more frequently, let's say every minute or even every 10 seconds.

This validator crashing problem seems like it can happen here as well, so option 1 seems strictly better.

I think if validator crashes then you are right. But Gossip messages can also be lost. What if the Gossip messages are lost, then we have to wait for 30 minutes, which seems too much.
How about lowering the wait time to 5 minutes at least, that shouldn't be too spammy.

What if the Gossip messages are lost

Gossip was designed specifically to handle redundancy and delivery of these messages

What if the Gossip messages are lost

Gossip was designed specifically to handle redundancy and delivery of these messages via push and pull

I don't know, I've seen Gossip messages getting lost in controlled environment like Google Cloud.
Anyways, can we change every 30 minutes to every 5 minutes then? Each Gossip message here is less than 100 bytes, so it's not really spammy.
And these Gossip messages should be dropped by others not in restart.

Seems ok to send out  the HeaviestFork the first time observed_stake passes 80% and then at 5 minute intervals after

Seems ok to send out the HeaviestFork the first time observed_stake passes 80% and then at 5 minute intervals after

Sounds good, changed to send every 5 minutes and added unittest.


driver.close() should be driver.quit()

fixes issue-311 https://github.com/kubernetes/examples/issues/311

/assign @sebgoa

/lgtm
thanks for filing a detailed issue and providing a fix.


Security check: pending buddies

Before adding a new buddy to a trip, check to see if they are in the pending buddies list

go to the div and use data striping to get the date that was clicked.


Modify maximum allowed tag length to 50 characters

Sometimes it becomes necessary to have tags with larger names for proper identification of objects.
Example, A tag that contains names of the Openstack Provider and OpenStack Tenant . Currently
the maximum allowed character length for tag is 30, This commit changes it to 50 characters.
Reference : http://talk.manageiq.org/t/cant-create-a-tag-with-more-than-30-characters/963
Signed-off-by: Prasad Mukhedkar<EMAIL_ADDRESS>

Looks good to me.  @gtanzillo Please review as well.  Does this get affected by any prepend of classifications?

:+1: This looks good. @Fryguy I don't see and issues with increasing the length of name.


Is there a way to change the output of where the video is stored?

I want the videos to directly show up in my gallery, is there a way to do that?

First
nano .yt-dlp/config/config on Termux
Second
then change -o ~/storage/shared/Youtube/%(title)s.%(ext)s to -o ~/storage/shared/DCIM/Camera/%(title)s.%(ext)s


Why is there limitation at the steering when using waypoint to control vehicle in scenario_runner?

I'm trying to create a scenario for a vehicle to turn right or left at the intersection using waypoint.
So i had to understand the code at atomic behaviors.py (https://github.com/carla-simulator/scenario_runner/blob/master/srunner/scenariomanager/scenarioatomics/atomic_behaviors.py) and especially the class ChangeActorWaypoints.
However, the scenario does not work as expected because of the code below:
class ChangeActorWaypoints(AtomicBehavior):

      ,,,

        def initialise(self):

     ,,,

                        heading_delta = math.fabs(new_heading - last_heading)
                        if math.fabs(heading_delta) < 0.5 or math.fabs(heading_delta) > 5.5:
                            route.append(wp_tuple[0].transform)

       '''

The scenario i created using Waypoint gives the value of heading_delta between 0.5 and 5.5. Thus, it is not added to the array route.
Why is there limitation to the range of heading_delta? and what does the value 0.5 and 5.5 means??

Hello @holymoly-JihyeSeo, from comments inside the code:
# The router sometimes produces points that go backward, or are almost identical
# We have to filter these, to avoid problems

What you checked that this is not the case for your waypoints?

@holymoly-JihyeSeo Hey did you find a solution for this issue?


AtmoOrb - improvements

Added AtmoOrb send delay in settings (defaults to 50ms).
AtmoOrb only connects if user has setting enabled.

Should help reduce the network load mentioned in issues #342

I changed the way devices are handled now. Take a look at the latest dev build. Would a solution like that suffice your needs?


[WIP] Diagnose master branch failing on Travis

…yarn/eslint-plugin-node-9.1.0"
This reverts commit 1b7dd99ec1858b8621d3e41659c58b2b5e2f8c25, reversing
changes made to a4ecc0a8277bf867674da7910a61981bed9da58d.

It seems to be a problem with travis/ember-try 😞


PickPhotosAsync Crashes after selecting photos and pressing Done in iOS

Hello, I just recently updated Xamarin and now this part of my code is crashing after I press Done after selecting my photos
var _mediaFiles = await CrossMedia.Current.PickPhotosAsync(new Plugin.Media.Abstractions.PickMediaOptions                     {                         PhotoSize = PhotoSize.Medium                         //,RotateImage = false                     }, new MultiPickerOptions                     {                         MaximumImagesCount = 5                     });
Its break at the AppDelegates file with an exception message saying "Value cannot be null. Parameter name: cgiImage".

Seems to be an issue with iOS 13 on the simulator. I have a co-worker that has a iPhone 7 plus running iOS 13 but not on his simulator.

Issue with iOS 13 (Unable to uplaod photos)

This has been fixed up in latest beta


BoxBrowseSDKResources is missing

UIImage+BOXBrowseSDKAdditions.m line 52, resourcesBundleURL is null and app crashed
    NSURL *resourcesBundleURL = [[NSBundle mainBundle] URLForResource:@"BoxBrowseSDKResources" withExtension:@"bundle"];

I used use_frameworks! config inside podfile, does it cause this issue ?
I'm using BoxPreviewSDKResources for temporary now.

Are you seeing the same issue the sample app?

@nhattieunhatkiem Please let us know if you are still experiencing issue or need additional help.


README: make a gif of app to show in readme

@championswimmer  should there be 1 gif that walks you through the whole app, or multiple gifs for different sections?
Cheers.

I have created a 40 sec gif containing 400 frames at 10 fps, 360px*640px, its size is 17 mb. This is the best size that i have obtained, is it ok ?

Here is mine, 320x569 @ 10FPS,
Fossasia.gif
Cheers.

Or lets embed youtube rather. :P
This is getting too big
On 10-Apr-2016 14:11, "Harshit Dwivedi"<EMAIL_ADDRESS>wrote:

Here is mine, 320x569 @ 10FPS,
Fossasia.gif
https://github.com/the-dagger/WallPoopers/raw/master/fossasia.gif
Cheers.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/fossasia/open-event-android/issues/332#issuecomment-207946407

Ok, cool.
I'll make a sample video and post it.
Cheers.

@championswimmer here is the video.

@the-dagger looks fine. You can add this to README.

@mananwason Should i remove the screenshots from Readme?

We can close this I guess @championswimmer
Cheers.


Sub sections like tocify

Hello,
Thanks for this lib. Works great !
I look for subsections like http://gregfranko.com/jquery.tocify.js/
Could angular-scroll do that ?
Thank you,
Philippe

Hi,
If you use the du-spy-context directive it should be possible with the following markup (that way both the parent and children may have the active class simultaneously):

    Section 1

Sub Section 1

Sub Section 2

Sub Section 3

Sub Section 4

Section 2

Section 3

Hello,
Thank you for this quick answer.
du-spy-context is what i need.
But there are two issues :
1 - Subsections are not handling duScrollOffset
See http://plnkr.co/edit/C9CcAsWI6K0qOYHDoADs?p=preview
It's possible ti highlight Section 2 and Subsection 1-2.
The problem does not occur if you don't set duScrollOffset
2 - When you scroll to the end,  Subsection 1-2 and  Section 3 are highlighted.
Did I something wrong ?
Philippe

Hi,

It actually does, but the spies only use it for activation not deactivation. Will look into changing this. It's also possible to expand/collapse sections by listening to scroll spy events.
Yes noticed that as well. Made the bottom spy configurable and off by default. If you check out the repo and build it should work (will publish a release eventually)

No 2 is solved as of 0.7.0.


compute on aws batch dev env

Problem to solve
compute on aws batch is written on python but there is no direnv that automatically enters the development env.
Intended users
Permissions and Security
Proposal

setup env dev with makePythonVscodeSettings

Test plan
What does success look like, and how can we measure that?
Links / references
/label ~"type::feature"

As discussed in #502, specific-purpose builtins will no longer be supported.


real time collaboration via y.js

Hi,
I came across airboardgame earlier via the y.js discord and wanted to say hi, but I struggled to find an email beyond the<EMAIL_ADDRESS>so I thought a github ticket might be appropriate.
I saw you are looking at using y.js for syncing user state across games and wondered if I could speak to you more about this. I am currently building a backend with persistence for y.js, https://github.com/firesync-org/firesync , I was wondering if you would be open to helping me understand your usage of y.js a little bit, I am trying to work out exactly what features and functionality people are looking for out of the box. Your input would be really useful to me. I think it’s still early days for you and y.js so your thoughts would be extra useful to me.
I've spent a fair bit of time working in the real time collaboration space, my previous company sharelatex (overleaf.com now)  is a collaborative editor for academics, but built on Operational Transformation, not CRDT’s.
If you are up for a little chat some time it would be amazing.
Thank you and sorry for spamming your issues.

Answered by mail.


Expose FILE_SAVING event to extensions

Just allows extensions to use workspace.onSavingTextDocument to bind to FileEventType.FILE_SAVING events.
Not sure if there was a reason this event was not exposed to extensions.
My use case was for an extension that needs to know when a user tries to save a file, even if it fails (Write Protected).
This change may be superseded by a Pre-Save hook that allows an extension to block saving ( #239 )

Hi @jel-massih, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@jel-massih, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

I think having such an event is fine, however extensions will likely want to modify the text before the save completes and thus that requires some ugly coupling of the save call on the main side to the event handling on the plugin host side...

If we expose this to extensions it should be possible to make changes. However, then maybe it should not be an event but rather a notion of "save participant".

Or we provide a rich event which allows to express participation. That way, we enable the simple case of just knowing when something changed, and the complex case of delaying saving because other work must finish first.
A SaveEvent ala
interface SaveEvent {
    resource: Uri;
    addPrecondition(task:() => Thenable<any> | any):void;
}

@jel-massih Are you planning to do more work on this? Iff so I will reopen this PR


Bug on Nvidia with shader and "uniform sampler2D texture;"

On Nvidia (GeForce GT 620M) the default name of the texture passed as uniform variable "uniform sampler2D texture;" cause a conflict with the name of the fonction "texture". In fact the function GLSL texture2D, texture1D or texture3D are deprecated. So impossible to use them:

'texture2D' : function is deprecated and not available

And when I use the new texture() glsl function I have a beautiful

error C1105: cannot call a non-function

The bug is present only on my Nvidia graphic card, not on intel integrated or on ATI they seems to accept texture as variable name and function name at the same time.
The bug is not-critical it can be fixed with a call to

Shader::setParameter(const std::string& name, CurrentTextureType)

and renaming the variable. But it could be interresting renaming the default name of the uniform sampler because somes examples will not run.

Did you define any specific shader version in your shader source? Could you provide some minimum example?

You haven't said on what operating system (and what version of SFML) this occurs.
The example fragment shaders omit any #version directive which means that they must be parsed as version 1.10 according to the specification. In 1.10, texture was not yet a built-in function and could be used as a name.
GLSL 1.30 specification with changes highlighted.
The examples don't request a 3.0+ context, so in that case 1.30 cannot be forced anyway. This must be one of those many annoying Nvidia GLSL bugs.
I guess renaming the variable from texture to the_texture or something should fix the problem.

yep the bug appear both on

#version 330

and

#version 140

I didn't try other version
The code is the pixelate.frag example from the sfml (just replaced the "texture2D" by "texture").
uniform sampler2D texture;
uniform float pixel_threshold;

void main()
{
    float factor = 1.0 / (pixel_threshold + 0.001);
	vec2 pos = floor(gl_TexCoord[0].xy * factor + 0.5) / factor;
	gl_FragColor = texture(texture, pos) * gl_Color;
}

Texture2D is deprecated since version GLSL 1.3 (130)

Operating system Windows 7.

Just to clarify, in case you didn't manage to read my previous comment before posting yours: This is an Nvidia bug, not an SFML bug. They don't follow the specification which requires the shader to be parsed as 1.10. We know that it won't work with versions 1.30+, but we aren't asking for it either.

And before I forget: Did you modify the shader source and added a #version directive yourself or not?
If you did, then it isn't a bug at all. The source won't work with #version 130 or higher. If you want to add the #version directive yourself, you will also have to make the required modifications to the shader source and example code as well.

I added #version because Nvidia seems compile the shader with a higher version of GLSL (> 140) when no #version is specified.  Because I get errors: that some things are not available (ex: "global variable gl_Vertex is removed after version 140") when I didn't specify the #version.
And yes my drivers are up-to-date...

So... there are other things (gl_Vertex) that wouldn't work as well if you don't specify a #version?
Can you paste the GLSL error log that is printed to the console when running the example unmodified (nothing added or removed)?

Ok, there is no error with fresh SFML compiled example without doing anything... I don't understand why... May be it's due to my configuration on my app with Nvidia Control Panel.
However, the name texture remain an issue if you use more recent GLSL.

However, the name "texture" remain an issue if you use more recent GLSL.

If we use more recent GLSL, which we don't.
We are aware that the examples (and even SFML shaders themselves) won't work with newer GLSL, which is why we don't request it in any of the examples. As an example, a lot of the API would have to be changed just to support user-defined vertex attributes.
We will approach this problem at a later time. For now, just remember that SFML's shaders aren't supposed to work with newer GLSL, and this is intentional.


fix: delete failed fetches

Hi,
If a fetch fails because of permissions or file not found, react-native-file-access still creates a file on the file system. This will cause issues because if a later valid url that matches the same cache key is used, the corrupted file is used instead since it is a cache hit, so we need to make sure files created out of failed fetches are removed

@efstathiosntonas Could you please take a look at the open prs please?


Fixed issues from build report

Docs Build status updates of commit 5d096b4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/input/font-support.md
:bulb:Suggestion
View
Details

globalization/input/text-input.md
:white_check_mark:Succeeded
View

globalization/locale/time-formatting-and-time-zones-in-dotnet-framework.md
:white_check_mark:Succeeded
View

globalization/localizability/string-handling.md
:white_check_mark:Succeeded
View

globalization/input/font-support.md

Line 116: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 155: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 194: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 265: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 279: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 301: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.
Line 329: [Suggestion] Illegal link: Top of page -- missing bookmark. The file input/font-support.md doesn't contain a bookmark named content.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


When marking a job as orphaned, append a message to the job log

Makes sense, right?
        {
            "msg" : "The job did not report in for a long time and was canceled. You could try running the job again.\n",
            "fd" : -1
        },

This is the functionality driven by api/jobs/reap ?

I believe so, yes. There may be multiple entry points to the orphaning logic.


predict_generator function does not support samples with different lengths

Hi, I am doing a sequence labelling task(like PosTagging) using Keras + theano. The model I use is BiLSTM. For training, I did not do padding. Instead, I just used the batch size of 1, and did not specify the input_length parameter. It works well for training and evaluation(for both I use the generator version to deal with the memory problem). However, when I want to do prediction, I have the following errors:
Predicting...
Traceback (most recent call last):
  File "predict.py", line 81, in <module>
    main()
  File "predict.py", line 55, in main
    val_samples=samples)
  File "/home/user/.local/lib/python2.7/site-packages/keras/models.py", line 1001, in predict_generator
    pickle_safe=pickle_safe)
  File "/home/user/.local/lib/python2.7/site-packages/keras/engine/training.py", line 1749, in predict_generator
    all_outs[i][processed_samples:(processed_samples + nb_samples)] = out
ValueError: could not broadcast input array from shape (1,42,1310) into shape (1,10,1310)

I think the reason is the first sentence have a length of 42 words, while the second sentence have a length of 10 words.
My question is, why it works well for training and evaluation, but does not work for prediction?
The code I think relevant is here
    print('\nPredicting...')

    samples = 1 # only 1 work for now
    prob = model.predict_generator(data_generator(X_test, X_test_feats, y_test, tag_size, shuffle=False),
                                   val_samples=samples)
    result = prob.argmax(axis=-1)

Could you kindly have a look at this problem? @fchollet

This still seems to be a problem in Keras using Tensorflow 1.12.0.
When I run model.predict_generator I get results only if each X returned by the generator has the same  length. Otherwise it throws this error:
ValueError: all the input array dimensions except for the concatenation axis must match exactly


Web UI not listing packages because of 'time' property unavailability in package.json in storage dir

Describe the bug
We're trying to migrate from sinopia to verdaccio for our internal npm registry, and noticed that the internal packages that were published using our existing setup (using sinopia) aren't showing up in the Web UI with the latest verdaccio (version: 3.1.2), even though we're able to install them. This is happening because of the some of the packages that were published earlier seem to be missing the time property in the package.json
To Reproduce
Ensure a module is already published once to verdaccio. Remove the time member from the package.json inside the storage directory. Then try to load the Web UI home page.
Results:

The published package doesn't show up in the Web UI
The below exception shows up in the verdaccio.log during server start-up

TypeError: Cannot read property '2.1.0' of undefined
 at ../node_modules-v8/lib/node_modules/verdaccio/build/lib/storage.js:389:37
 at storage.readPackage (../node_modules-v8/lib/node_modules/verdaccio/build/lib/local-storage.js:616:7)
 at ../node_modules-v8/lib/node_modules/verdaccio/node_modules/@verdaccio/local-storage/lib/local-fs.js:167:9
 at <anonymous>

Additional context
There has to be a isNil check for the time property in https://github.com/verdaccio/verdaccio/blob/master/src/lib/storage.js#L374, similar to the fix done in https://github.com/verdaccio/verdaccio/commit/e977d50c8a1b005acf81bf3b2bc5d71e0c6d2755 to support backward compatibility.

Please try with<EMAIL_ADDRESS>

Verified, thank you! @juanpicado
One related minor UI (alignment) issue in the home page I noticed when there are packages with/without the time details in their package.json, attaching the screenshot below:

The license div is not right aligned, for the older packages, because of the missing time info div on the left side.

Ohh !! I must be blind. I'll create a ticket. Thanks 👍

@kumarvi @juanpicado I am creating a separate issue for this. 🥇

@kumarvi I confirm the License alignment issue. It will be solved soon. Thanks 😀
Ref #761


People are reporting an error message when they press “send letter” at BattlefortheNet.com

Looking into this to see if I can reproduce...

confirmed, i can reproduce in FF latest.

Confirmed I can reproduce in FF latest.

Getting that error:
"There was an error submitting the form, please try again."
Firefox  54 on Ubuntu 17.04. Comcast business service.

Console:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://queue.fightforthefuture.org/action. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).

@acabey

Comcast business service

Well there's your problem. 😄
Jokes aside...CORS all the things!

@as-com
I didn't want to rule anything out, but it looks like it's a configuration issue anyways. Hopefully this can be fixed earlier rather than later

It appears fixed now for me. We think this is fixed.

@holmesworcester
Confirmed here

Interesting, we didn't make any change to CORS policy this morning. It's possible that CORS headers aren't being set on non-2xx responses.

Also giggling at seeing Cloudflare's version of this widget displaying at https://enable-cors.org/ @as-com 😄

Closing now, as this issue wasn't related to the widget.


Is there a way to get dynamic colors (by their names) in Inline embedded style?

Is there a way to get dynamic colors (by their names) in Inline embedded style? For example, in React within a style tag.
style={{backgroundColor: '#FF4E4F', boxShadow: 'rgba(255, 78, 79, 0.4) 0px 0px 5px 0'}}
Is there a way to get the color # FF4E4F by its name?

Maybe this will help if you're working with React https://github.com/bradlc/babel-plugin-tailwind-components

Thanks


Karan Bir's profile added

I've updated the ReadMe and added my personal page with photos. Please accept.

There're multiple commits above, can you please squash them and rebase against upstream? This might be helpful: http://code.dblock.org/2015/08/31/getting-out-of-your-first-git-mess.html

Can be closed


Add conda Jenkinsfile

The text is definitively an improvement. Here's my proposal, make a text saying "PLACEHOLDER" for where the template Jenkinsfile is, then we can merge this and create a new ticket to work on the Jenkinsfile using the docker plugin next sprint.


web: clean up the UI a bit on narrow widths

Hello @jazzdan, @yuindustries,
Please review the following commits I made in branch nicks/wrap:
e5c9754cc8c1878d7935feb04af997761cbabec4 (2019-05-20 12:09:55 +0200)
web: clean up the UI a bit on narrow widths

still not perfect, but at least it doesn't look totally broken


CStruct with bitfields

Hi, I know it's early days, but I wanted to report this in case there's already a solution. Below is a diff to your examples/{foo.c,ffi.cy} that shows the issue. In short, how to handle C structs that use bit fields?
diff --git a/examples/ffi.cy b/examples/ffi.cy
index 0d8aedf..4f03827 100644
--- a/examples/ffi.cy
+++ b/examples/ffi.cy
@@ -7,7 +7,18 @@

 import os 'os'

+object bitStruct:
+  flag uint32
+  other uint32
+
+
 lib = os.bindLib('./libfoo.so', [
     os.CFunc{ sym: 'add', args: [#int, #int], ret: #int }
+    os.CFunc{ sym: 'getBitStruct', args: [#uint], ret: bitStruct }
+    os.CStruct{ fields: [#uint, #uint], type: bitStruct}
 ])
-lib.add(123, 321)
+print lib.add(123, 321)
+
+
+print lib.getBitStruct(22).flag
+
diff --git a/examples/foo.c b/examples/foo.c
index 082fc1e..22909ff 100644
--- a/examples/foo.c
+++ b/examples/foo.c
@@ -1,3 +1,16 @@
+#include <stdint.h>
+
+typedef struct bitStruct {
+    uint32_t flag: 1, other: 31;
+} bitStruct;
+
+bitStruct getBitStruct(uint32_t a) {
+   bitStruct b;
+   b.flag = 1;
+   b.other = a;
+   return b;
+}
+
 int add(int a, int b) {
     return a + b;
-}
\ No newline at end of file
+}

This hasn't been implemented yet but I did check that it's possible with libtcc. It would look something like this in Cyber:
import os 'os'

lib = os.bindLib('./libfoo.so', [
    os.CFunc{ sym: 'getBitStruct', args: [#uint], ret: bitStruct }
    os.CStruct{ fields: [os.CuintBits(1), os.CuintBits(31)], type: bitStruct }
])

object bitStruct:
  flag number
  other number

s = lib.getBitStruct(22)
print s.flag
print s.other

I see. Thanks for the response.
For that example, I also don't mind bitshifting myself. I do have a related question. I was just trying out wrapping htslib. The first function is hts_open with this signature:
htsFile *hts_open(const char *fn, const char *mode);

where htsFile is:
typedef struct htsFile {
    uint32_t is_bin:1, is_write:1, is_be:1, is_cram:1, is_bgzf:1, dummy:27;
    int64_t lineno;
    kstring_t line;
    char *fn, *fn_aux;
    union {
        BGZF *bgzf;
        struct cram_fd *cram;
        struct hFILE *hfile;
    } fp;
    void *state;  // format specific state information
    htsFormat format;
    hts_idx_t *idx;
    const char *fnidx;
    struct sam_hdr_t *bam_header;
    struct hts_filter_t *filter;
} htsFile;

so I'd prefer not to specify that in cyber. In fact it can be completely opaque.
I tried this:
import os 'os'

hts = os.bindLib('libhts.so', [
    os.CFunc{ sym: 'hts_open', args: [#charPtrZ, #charPtrZ], ret: #ptr}
])
print hts

h = hts.hts_open("../htslib/test/range.bam")
print h

but I see:
BindLib53175d61490b23df
panic: Missing method symbol `hts_open` from receiver of type `BindLib53175d61490b23df`.

/home/brentp/src/cyber/bam.cy:8:5 main:
h = hts.hts_open("../htslib/test/range.bam")
    ^

Is there a way to do this without writing a separate shared lib containing a wrapper function that returns a void *?

You've set up bindLib correctly, but you called hts_open with 1 arg instead of 2. Since Cyber currently has function overloading by number of params, it say's it's missing a symbol. I'll make the error message more descriptive.

Is there a way to do this without writing a separate shared lib containing a wrapper function that returns a void *

That's the intended purpose of #ptr, I think it will be renamed to #voidPtr in the near future.

You've set up bindLib correctly, but you called hts_open with 1 arg instead of 2. Since Cyber currently has function overloading by number of params, it say's it's missing a symbol. I'll make the error message more descriptive.

Oh, wow. Thanks for noticing this. Indeed it's working now.


Use task status version for task status long polling notifications

rebased, ping @dain

addressed @dain

rebased


Change ports?

Is there a way to change the port numbers for SMTP and the HTTP server?

Yes there is, eg mailpit --smtp <IP_ADDRESS>:1025 --listen <IP_ADDRESS>:8025 (those are the default values, but you can change them). Please note that if you are running it as a non-root user then you cannot listen on ports lower than 1024 on Linux/Mac (I don't know about Windows).
Please see https://github.com/axllent/mailpit/wiki/Runtime-options


Pgsql has gone away

Receiving this occasional error
General error: 7 server closed the connection unexpectedly \tThis probably means the server terminated abnormally \tbefore or while processing the request.

Seems to be coming from the same backend though, so may be db server settings specific
ref: https://keboola.slack.com/archives/C09U3R1J4/p1543481464025800

This isn't getting retried?

should be resolved by 2.5.7


Adding chat component

Pull Request Test Coverage Report for Build 1884

2 of 55 (3.64%)  changed or added relevant lines in 8 files are covered.
761 unchanged lines in 70 files lost coverage.
Overall coverage decreased (-14.7%) to 12.927%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/academy/grading/GradingEditor.tsx
0
1
0.0%

src/components/academy/grading/GradingWorkspace.tsx
0
2
0.0%

src/components/assessment/AssessmentWorkspace.tsx
0
3
0.0%

src/containers/ChatContainer.ts
0
4
0.0%

src/components/chat/MessageList.tsx
0
6
0.0%

src/components/chat/Input.tsx
0
13
0.0%

src/components/chat/ChatApp.js
0
24
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/reducers/session.ts
1
91.18%

src/reducers/academy.ts
1
72.73%

src/components/commons/Markdown.tsx
1
62.5%

src/components/dropdown/Help.tsx
1
85.71%

src/reducers/workspaces.ts
1
96.36%

src/components/dropdown/Profile.tsx
1
84.62%

src/utils/castBackend.ts
1
0.0%

src/components/dropdown/About.tsx
2
69.23%

src/utils/paramParseHelpers.ts
2
42.86%

src/components/workspace/side-content/ResultCard.tsx
3
0.0%

Totals

Change from base Build 1876:
-14.7%

Covered Lines:
641

Relevant Lines:
4418

💛  - Coveralls

A minor gripe is that it is a little slow to load. (however, that cant be helped)

This is probably because the api call is made when the chat component is mounted (i.e. when you open the chat tab)
A suggestion is to move the api calls to the parent component (AssessmentWorkspace), and pass down the results to the chat component

I think its better to have it isolated for now! Let us not couple it with the parent just yet (till we are sure this is the route we want to take in the long term). The API calls are done by the chatManager and it may be difficult to shift it out to the parent.
The app is still usable for an end-user and I feel its not a major hindrance.


v1_21_R1ChunkDataProviderImpl  NullPointerException

Stack trace
Hi, I’m working on one of my personal projects and I needed to find a repo that could save me from the burden of creating my own Pathfinder, and that’s when I found you. After a tough struggle to understand the API since the documentation and code have changed, I managed to make it work—or at least I think I did. However, after structuring the event for my ArmorStand, I got a 'NullPointerException', but I’m not sure if it’s my fault or if there's something wrong in the 'v1_21_R1ChunkDataProviderImpl' class.
I’m sharing the link with my code and the error log below. I also forgot to include how I initialize the API, so I’ll leave that here as a quote.

My main class
class ItemGuard : JavaPlugin() {

     internal val pathfinder = {
         PatheticMapper.newPathfinder(
             PathfinderConfiguration.createConfiguration()
                 .withAllowingFailFast(true)
                 .withAllowingFallback(true)
                 .withLoadingChunks(true)
                 .withBStats(true)
         )
     }

     override fun onEnable() {

         PatheticMapper.initialize(this)

         println("[ItemGuard] initialize pathfinder ${Pathetic.isInitialized()}")

         server.pluginManager.registerEvents(PlayerListener(), this)
     }

     override fun onDisable() {
         HandlerList.unregisterAll(this)
         server.scheduler.cancelTasks(this)
         Pathetic.shutdown()
     }
 }

The code of my event and the log referring to the error

Actions to reproduce (if known)
No response
Other
No response

Update: I found the error, and it seems it was my mistake. I sincerely apologize for creating the issue. I spent over 30 minutes trying to figure it out until I stopped to carefully review everything. In the source, I saw that you had instantiated the "PathEnvironment" by passing the UUID and the world name, and I hadn’t done it that way. I’ll leave the reference code below, and again, I apologize for the inconvenience.

Update SupportEvent
private fun pathPosition(item: Item): Map<String, () -> PathPosition> {
        return mapOf(
            "start" to {
                val location = armorStand.location
                PathPosition(
                    PathEnvironment(ownWorld.uid, ownWorld.name, location.blockY - 20, location.blockY + 20),
                    location.x,
                    location.y,
                    location.z
                )
            },
            "end" to {
                val location = item.location
                PathPosition(
                    PathEnvironment(ownWorld.uid, ownWorld.name, location.blockY - 20, location.blockY + 20),
                    location.x,
                    location.y,
                    location.z
                )
            }
        )
    }

hi, all good, no worries. glad you figured it out! :)
we appreciate very much that you found to us in the first place, even if the mistake was not with us.
you said you had trouble understanding the API, can you maybe go a little bit more into detail here? we would like to improve and any feedback is welcome and would help us alot!
sincerely,
metaphoriker

Thank you! I’ll leave comments if I find anything. For now, I’m testing and building the logic I need, so along the way, if I find something that could be useful to implement, I’ll mention it. Thanks for the source, so far it seems quite complete.


send weekly MoonDAO message on discord

just need to get proper permissions in 📢│governance-alerts channel

working


Added support for OwnNamespace

Signed-off-by: thejasn<EMAIL_ADDRESS>

/docs-approvd
/px-approved
/qe-approved

/label tide/merge-method-squash

/label px-approved docs-approved qe-approved

/label px-approved

/label docs-approved
/label qe-approved

/lgtm
/approve


Add Submit a request form

Description

added UI for Submit a request form
see changes in /modules/new-request-form and /styles/main.css
[TODO] add logic and UI for the related articles in the form - asking the zendesk support team about how to display the field

Type(s) of changes

[ ] Bug fix
[x] New feature
[ ] Update to an existing feature

Motivation for PR

Closes #6

How Has This Been Tested?

Tested on local preview: https://uniswaplabs1730152595.zendesk.com/hc/en-us/requests/new or https://uniswaplabs.zendesk.com/hc/en-us/requests/new
Applicable screenshots

https://www.loom.com/share/d5919715e0784cd8906ed992e3cb2620?sid=1c2afc77-b5cf-4458-9e55-1b5df79ae548
Follow-up PR

when are you going to handle [TODO] add logic and UI for the related articles in the form - asking the zendesk support team about how to display the field? Looks like they want the original approach implemented!

when are you going to handle [TODO] add logic and UI for the related articles in the form - asking the zendesk support team about how to display the field? Looks like they want the original approach implemented!

handling now using the original approach, i just need to figure out how to show the RA field! UI is also implemented in module/related-articles. waiting on the Zendesk support team's response about to show the field.

One of the issue we have with file difference is we don't have a prettier setup to run before we commit.
I think Nahee and I have different, prettier setups locally vs. code. So whenever we save, it updates each other's code lol
I have been working more widely across the project, so Nahee's PR has more change 🤔
maybe we should add precommit hook to make sure we run prettier before


correction to change the 'Get' in 'Get Geocoding Batch' and 'Get Geocoding Reverse Batch' to 'Post'.

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Once merged, ensure that the redirection changes in PR6512 are merged.

#label:"PublishToCustomers"

API change check
APIView has identified API level changes in this PR and created following API reviews.
preview


Access content from Chrome Legacy Window

Hello,
I am trying to use this library, to get the content of a page, loaded inside a CefSharp component.
See this link for the exact use case: https://stackoverflow.com/questions/57890068/automation-for-chrome-legacy-window-chromium-using-winium
The problem is, as described there also, I cannot get to the controls inside the "Chrome Legacy Window". It gets this AutomationElement, but cannot go inside it. Is there any way of getting controls from inside this type of control?
Thank you.

How dows the Chrome Legacy Window look in an inspect tool (like FlaUInspect)? Do you see content in there?

Hello,
This is what it shows:

You could give UIA3 a try. But if the Chrome Legacy Window does not expose anything to UIA, there's nothing FlaUI can do except working with mouse/keyboard/screenshots. Also as far as I remember, to automate Chrome with UIA, accessibility needs to be activated in Chrome itself, maybe it is somehow the same with the Chrome Legacy Window.

UIA3 did not do the trick either. Thank you for the suggesting FlaUInspect.

Can you provide me a sample application with that Chrome Legacy Window?

I was able to fix this, by forcing Cef Sharp (Chromium) to start with accessibility on:
cefSettings.CefCommandLineArgs.Add("force-renderer-accessibility", "true");
Seem something on that PC, did not tell the browser to expose Accessiblity. But giving it that param at startup, fixed it.

Great that you figured out and many thanks for the feedback!


Fixed player synchronization with more than two instances open

First of all thanks for the tutorial.
I noticed this issue was also mentioned by youtube user TommyPin95 (1 month ago, in the GDQuest video https://www.youtube.com/watch?v=JAtaeP3Nf74&t=1s - Intro to Multiplayer in Godot (3): Rifle and UI (tutorial)).
The issue happens when you open more than two instances. The first is the server, the second is a joined player, and the third joins and does not appear in the second. The following changes aim to fix that issue. I tried to keep as simple as possible.
I changed the Network a little-bit to be able to proper send player info.
By studying how the Godot networking works and reading the current docs of version 3.0.x, the signal "network_peer_connected" of the SceneTree is triggered every time any player connects, and it is triggered in every instance of the game.
My logic was, when a player connects to the game, you will receive first his ID, which is used to request all of his info.
It will be requested to the the server instance only:
remote func _request_player_info(request_from_id, player_id):
	if get_tree().is_network_server():

There is no need anymore to loop every player and send their info. They are requested to the server when needed.
The function remote func _request_players(request_from_id): at plural is not used, as the comments says it will be useful only if you want to get all the players info in the current session for some reason.
Hope it helps, it was an exercise for me to understand how Godot's networking works.
Thanks.

Thank you very much!
@guilhermehto  can you check this out? The code looks good to me but you're the one who has experience with networking in Godot. you can merge directly if you think it's good

It's working fine and you even made the code clearer to read.
Thank you! 😃


Writing re-usable, decoupled models (multi-app architecture)

I am writing a complex web application that consists of a multiple flask apps, as well as non-flask Python apps (internal scheduler, etc).
I need to write models that I can re-use across multiple Flask apps, as well as non-flask python code using pure SA. However, I don't want to lose any benefits that Flask-SQLAlchemy gives you (when possible).
My first approach was to use declarative_base and it somewhat works:
from flask_sqlalchemy import declarative_base, DefaultMeta

Base = declarative_base(bind=user_db_conn_string, metaclass=DefaultMeta)

class User(Base):
    __bind_key__ = "user_db"
    name = Column(String(128), nullable=False)
    ....

However, multi-database binds don't seem to work properly. When I run
db.session.query(User).all()

It ignores the bind/bind key and runs it against the default database (and fails).
What should be correct approach here?

Personally I just use Flask-SQLAlchemy everywhere. Even if it's not a webapp, you can simply wrap your code with a flask app context - all you need is to call db.init_app(app) and previously instantiate the app and set the sqlalchemy-related config.

See #250 for previous work on this and https://stackoverflow.com/a/33127999/400617 for an example of what ThiefMaster is talking about.

Hi @davidism
I am really sorry but unfortunately, neither of those two links answer my question.
Example given in the end of #250 discussion does not work with a multi-database setup. Specifically, models do not bind to the correct engine using with __bind_key__ and I could not figure out how to make it bind properly.
Second link and solution provided by ThiefMaster allows to use models across multiple flask applications but it does not solve for non-flask environment. Shipping flask and flask-sqlalchemy with non-flask applications is barely a workaround and does not scale.
Imagine the following scenario - you have a non-flask codebase, developed by another team using SQLAlchemy, and you are developing a simple Flask-based frontend for it. Asking them to couple their architecture to flask-sqlalchemy and flask seems like a really bad idea.

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Some style attributes don't parse correctly

Script trips on certain styles:
Input:
<svg viewBox="0 0 93.87 436.96">
    <rect style="fill:#ccc" />
</svg>

Output (note the misplaced #):
import React from 'react';

export default function WineBottle({width = '50px', height = '50px'}) {
  return (
    <svg viewBox="0 0 93.87 436.96" height={height} width={width}>
  <rect style={{fill:#"cc}} />
</svg>

  );
}

WineBottle.propTypes = {
  width: React.PropTypes.string,
  height: React.PropTypes.string
}

Happy to submit a PR.
Could consider this: https://github.com/harrison/react-magic/tree/add-html-svg-attributes to do the heavy lifting. I've seen it used quite well here: https://github.com/gocardless/splash-pages/blob/dev/script/generators/render-icons

Thanks for catching that. A PR would be great. If that project produces a more consistent output, then I'm all for it.
I should probably add unit tests to this project in the near future so we can catch these issues easier.

Great I will look at a PR over the next few days 👍

This is closed with PR #9


cluster needs to be created before the domain

When applying quick-start yaml, domain fails with below error until it gets resolved with next retry happens in 120 seconds.
The reason is that domain gets created before cluster. It must be the other way around.
`
% kubectl describe domain sample-domain1 -n sample-domain1-ns
Message:                 Cluster resource 'sample-domain1-cluster-1' not found in namespace 'sample-domain1-ns'. Will retry next at 2024-06-26T11:24:46.003016707Z and approximately every 120 seconds afterward until 2024-06-27T11:22:46.003016707Z if the failure is not resolved.
Events:
Type     Reason   Age    From               Message

Normal   Created  3m22s  weblogic.operator  Domain sample-domain1 was created.
Warning  Failed   3m21s  weblogic.operator  Domain sample-domain1 failed due to 'Domain validation error': Cluster resource 'sample-domain1-cluster-1' not found in namespace 'sample-domain1-ns'. Update the domain resource to correct the validation error.
`

@gvnc, as you probably noticed after the bot response, we are required to verify that external contributors have signed the OCA prior to merging commits.
You can either do that and then I can accept your change or we can switch this to an issue and I can merge a similar change (that I would author).
What would you like to do?

@rjeberhard I guess it would take some time for me to get over the bot check. Let's cancel this PR and you take care of it in an issue.

Done


Cannot assign uploader from one model to another to copy files

This code used to work for us (we mount our Image uploader on the media field, and images are stored on S3 using the fog gem):
copied_asset = Assets::Image.create(
  media: original_asset.media,
  internal_name: "published_#{original_asset.internal_name}"
)

Now it fails to save because of image type protection here:
https://github.com/DarthSim/carrierwave-bombshelter/blob/master/lib/carrierwave/bombshelter.rb#L41
Which generates
copied_asset.errors.full_messages
# => ["Media Image has an unsupported type"]

I may be completely on the wrong track, but when I look at the line that feeds the type checker
https://github.com/DarthSim/carrierwave-bombshelter/blob/master/lib/carrierwave/bombshelter.rb#L36
def protect_from_image_bomb!(new_file)
      image = FastImage.new(new_file.path || get_real_file(new_file.file))

and try to pass in
new_file = original_asset.media,
then since new_file.path resolves to image_assets/3bb19772-4350-4f96-9279-688740bb7628.png, I get
image = FastImage.new(new_file.path || get_real_file(new_file.file))
# => #<FastImage:0x007f9f0a75dcc0
@uri="image_assets/3bb19772-4350-4f96-9279-688740bb7628.png",
@options={:type_only=>false, :timeout=>2, :raise_on_failure=>false, :proxy=>nil, :http_header=>{}},
@property=:size,
@parsed_uri=#<Addressable::URI:0x3fcf853aebcc URI:image_assets/3bb19772-4350-4f96-9279-688740bb7628.png>
>
image.type
# => nil

If I skip the path option I get
image = FastImage.new( get_real_file(new_file.file))
ArgumentError: wrong number of arguments (1 for 0)
  from .../gems/carrierwave-0.11.1/lib/carrierwave/storage/fog.rb:225:in `read'
  from .../gems/fastimage-2.0.0/lib/fastimage.rb:327:in `block in fetch_using_read'

(which suggests I am sending in the wrong object; though new_file.file responds_to?(:read), which makes FastImage think it can call read(LocalFileChunkSize) on it--which is one too many arguments for Fog )
If I use new_file = original_asset.media.file I still get
image = FastImage.new(new_file.path || get_real_file(new_file.file))
# => #<FastImage:0x007f9f0a6d5c08 @uri="image_assets/3bb19772-4350-4f96-9279-688740bb7628.png",
@options={:type_only=>false, :timeout=>2, :raise_on_failure=>false, :proxy=>nil, :http_header=>{}},
@property=:size,
@parsed_uri=#<Addressable::URI:0x3fcf8536ac10 URI:image_assets/3bb19772-4350-4f96-9279-688740bb7628.png>
>
image.type
# => nil

So backing all the way out, if I try at the top level to use
asset = Assets::Image.create(
  media: original_asset.url,
  internal_name: "published_#{original_asset.internal_name}"
)

It does actually save, but with no image:
asset.url
# => nil
asset.media
=> #<ImageUploader:0x007f9f0a67f178
@model=#<Assets::Image id: 37848, created_at: "2016-05-13 15:45:10", media: nil, type: "Assets::Image", updated_at: "2016-05-13 15:45:10", internal_name: "published_3bb19772-4350-4f96-9279-688740bb7628_378...">,
@mounted_as=:media,
@storage=#<CarrierWave::Storage::Fog:0x007f9f0a674cf0
@uploader=#<ImageUploader:0x007f9f0a67f178 ...>>
>

Is this a use case that was not considered before?
Or, at the very least, is there a way to do what we are doing and pass protect_from_image_bomb! ?

Thanks for discovery! Fixed in 0.2.1.

:+1: Thanks! Bundling the new version now


MediaRouteProvider Not visible in Wifi network

MediaRouteProvider created in this sample is visible in the same device to other apps but its not visible to apps on other device. Can you please help us to get MediaRouteProvider visible in wifi network. Not so sure if I should have posted this in StackOverFlow

Please help me. I have same problem.

Facing the same problem. Anyone, need help!

Any news, could I expect this will be updated?


Questionable GUI decisions

Window is too tall on my laptop per default. And I cannot read the text very well.

The window is resizable but we can change the default width and height in src/GUI/main.py. (TODO: these variables should be put in settings.py).
GUI elements are disabled before opening image(s), which is why the text shows up like that. However, it shouldn't be that invisible even when disabled. Shows up fine on my laptop.
Check https://github.com/COMP523TeamD/BreezeStyleSheets (this is a link to our fork) for a JSON field that determines the color of disabled GUI elements, if it exists. Instructions for using exporting from BSS to the HCT repo are here, if you want to test it in our GUI.
I'm finishing up a few small changes FYI so will commit in a few.

This is solved. But still need to fix the aspect ratio of the image or something similar.


feat: add offline sprite support

Closes #4
Notes:

Questions:

what happens when multiple styles point to the same upstream sprite url? right now, the implementation more or less assumes a 1-to-1 relationship between a sprite and its style

@gmaclennan see https://github.com/digidem/mapeo-map-server/pull/42#discussion_r921376116 for more context, but wondering if we need to store etags for both the json and png assets. right now the sprite table only has a single etag column and the implementation stores the json etag in it, but it seems like it would be useful to have both etags (assuming each one can have its own)

@gmaclennan see #42 (comment) for more context, but wondering if we need to store etags for both the json and png assets. right now the sprite table only has a single etag column and the implementation stores the json etag in it, but it seems like it would be useful to have both etags (assuming each one can have its own)

I think we would be ok storing just the sprite etag. The sprite could change without the json changing, but I can't imagine a scenario where the json would change without the sprite changing, so monitoring the sprite for changes is enough to capture the change in the json too.


Proposal: Series security Solidity contract on Kaia

Description

This submission proposes the development of a Series Security Solidity contract on the Kaia blockchain. The contract will focus on enhancing the security and integrity of smart contracts within the Kaia ecosystem by implementing advanced security measures and following best practices in Solidity development.

Bounty track

Education

Approved. I like security stuffs. keep watch @kaia-dev-intern


Add Keyboard Shortcuts

I try to use copy (ctrl + c) or undo (ctrl + z) but not working, any one know if is this a bug o this feature is not available?

It should work now. I updated to BPMN-js 18.0.0 and they added the support for keyboard shortcuts.


Build from source doesn't work whereas supplied minified version does.

I built from source but resulting calendar doesn't respond to mouse clicks (i.e. it pops up, but I can't select dates or change months. I can't even close it with the "Close" button).
The supplied minified version does work.
Any suggestions..?

In the directives.js, remove:
element.pickadate('picker').set('select', element.val(), {format: options.format});
From the function: function onOpen(e)

Thank you!


I dont like text which is strike/line through. This is fixing it

I honestly think its easier to see that a registration have been clicked with the line though.
Maybe we should change the color even more?

I honestly think its easier to see that a registration have been clicked with the line though.
Maybe we should change the color even more?

Just tested it in the preview. The color was more prominent in my local test. I will change it

Could also be that the preview got overwritten by another pr 🤷♂


Add PromQL to json for Robusta UI

Combine promql requests for different pods into one, pass query to result data

looks good, i would run a test comparing the table of the previous version to this to make sure nothing changed with the prometheus query

some comments, looks good, i would run a test comparing the table of the previous version to this to make sure nothing changed with the prometheus query

Actually I got some rows to differ, I will investigate further


Pull requests

Is there any chance we could get all the pull requests merged so that this can be used with 1.6? Would really like to use it.

Yep, afaik there are currently three pull requests providing 1.6 support, it would be nice to have one merged into a release :-)

@m00s any news about this?

+1 we need it

@bramski


Benchmarking comparison of Alevin-fry configurations to cellranger

Closes #92. In this PR, I'm adding a notebook to compare the different configurations of alevin-fry to Cellranger with both single-cell and single-nuclei samples. Specifically, we were interested in answering the following questions about use of Alevin-fry:

Should we be using the splici index for single-cell samples rather than the cDNA index based on recent data showing a false increase in gene expression in pseudoaligners - https://combine-lab.github.io/alevin-fry-tutorials/2021/improving-txome-specificity/
Should we be using the cr-like or the full resolution - Use of the splici index with cr-like warrants use of the USA mode by alevin-fry which allows counts from only spliced reads and ambiguous reads to be counted and excludes and reads that are intronic.
Is use of pseudoalignment (sketch) still a viable option as we consider these new conditions?

I have tested all of these alevin-fry conditions on 3 single-cell samples and 4 single-nuclei samples. I then created three dataframes with combined colData from each of these analysis, combined rowData from each of these analysis, and associated information about the configurations for each run. These dataframes were created in #99 and used to generate the plots in this PR to look at the following metrics:

mitochondrial content
UMI/cell
genes detected/cell
correlation of UMI/cell
correlation of mean gene expression

The analysis done here is very similar to the previous benchmarking since we have added new conditions, but if there is interest in looking at additional metrics than we can add those as well. In general, alevin-fry performs very similarly to cellranger in all of the conditions, with the following observations:

Alevin-fry tends to detect more cells than cellranger, and in those additional cells the counts tend to be lower, this is probably a filtering problem and less of an Alevin-fry problem
the full resolution shows higher UMIs/cell and genes detected/cell than both the cr-like and cellranger - I'm curious if others have thoughts on choosing a resolution type
splici performs similarly to the cDNA index in single-cell samples, but it does come at a slightly higher cost than the cDNA

This PR is also stacked on #99 and that will need to be merged first.

Hi @allyhawkins,
These are great!  I really do enjoy seeing the thorough analyses that you are doing across these samples.  I will dig through this more deeply later and give any other thoughts, but one thing I just wanted to point out, since I wasn't sure of your index construction procedure itself, is that we find using the sparse index reduces the index size with little to no cost in mapping time.  It has no bearing on the mappings themselves (dense and sparse indices give identical mapping results).  To build the sparse index you simply pass the --sparse flag to salmon when constructing the index.
The other point worth raising is that I'm not certain what tool versions are used here (since we've been bumping alevin-fry quite quickly while working on the manuscript), but in the pre-print we used salmon v1.5.1 and alevin-fry v0.4.0.  Compared to older versions these do have a few small tweaks that were designed to slightly improve accuracy by increasing the counts for a handful of genes (under the cr-like rule, if UMIs map to > 1 gene with the same read count, they are discarded); these new versions added a few heuristics to minimize those scenarios where feasible.  Also, --cr-like-em will probabilistically allocate (rather than toss) these "ties"; though I'm not sure if you want to add yet another variable into the mix.
Finally, one question I had was how did you use the --full resolution with the splici index?  Alevin-fry should check the resolution mode being passed, and should only allow either --cr-like or --cr-like-em when the splici index is being used (i.e. when a 3-column tsv is passed to the -m parameter).
Best,
Rob

Hi @rob-p, thanks for your feedback and help as always! To answer your question about the version, this is with samples that were run using alevin-fry 0.3.0 and salmon 1.4.0, so as we move forward with alevin-fry we will update the workflows to use the newer versions.
Thanks for the tip about the index construction as ours were not generated using the --sparse construction. In general, the increase with splici is still much smaller than the memory and time usage to use cellranger though, especially with the single nuclei samples.
As for your question about using the splici index with the full resolution. I am referring to using the splici index as any configuration that used the index that was built with both the reads from spliced cDNA and the intronic reads included. We have generated both the 2 column and 3 column tx2gene.tsv file from this index, so in the case of using the full resolution, we are passing the 2 column file through. For the cr-like, we pass the 3 column one through. What that also means is that the outputs are slightly different and if we have used the full resolution with splici, then we grab all of the rows in the counts matrix corresponding to intronic reads and collapse those counts with the corresponding spliced cDNA from that gene to make sure the matrix is summarized to gene level. It's a similar approach to how you guys are doing USA mode where you collapse the reads for S + A reads. Here, we are either removing intronic reads from the counts matrix (in the case of single-cell) or combining reads from intronic reads with the spliced cDNA reads (in the case of single-nuclei).
Based on reading through the alevin-fry paper, it appears to me that there is a strong argument for using the cr-like or cr-like-em approach rather than the full resolution as we were doing. In our hands, it appears that the cr-like is producing more similar UMIs/cell and genes/cell as cellranger, which makes sense as it has the same approach of throwing out multi-mapped reads like cellranger.
I'm curious if you tested whether or not use of the full resolution or any of the other resolution strategies had an effect on detection of the false gene expression, similar to what is identified in the absence of using the splici index? I know the full resolution uses EM to resolve multi-mapped reads, probably explaining the increased numbers, but that doesn't necessarily seem like a bad thing to me. Is there any evidence for why increased UMIs and genes with the other resolution strategies other than cr-like could be a bad thing? I'm just trying to get a better sense of the resolution strategies and why we might want to pick one over the other.

Hi @allyhawkins,
Thanks for your detailed response!  My thoughts are below:

Hi @rob-p, thanks for your feedback and help as always! To answer your question about the version, this is with samples that were run using alevin-fry 0.3.0 and salmon 1.4.0, so as we move forward with alevin-fry we will update the workflows to use the newer versions.

Awesome!  I realize that things on the alevin-fry side in particular have been changing quickly as we converged on getting the pre-print out.  While I anticipate we will still continue to add/improve things, I think there should be some more stability now.

Thanks for the tip about the index construction as ours were not generated using the --sparse construction. In general, the increase with splici is still much smaller than the memory and time usage to use cellranger though, especially with the single nuclei samples.

Right; even with the dense index, alevin-fry should be pretty memory frugal.  In all of our testing everything could be done comfortably for all of the datasets within 16G of RAM.  However, with the sparse index, all data in the paper could be processed in <8G of RAM, and fraction of them even under 4GB.  Interestingly, this is without any consistent slowdown in runtime.  Of course, if the dense index serves your needs sufficiently, then this capability is just a bonus.

As for your question about using the splici index with the full resolution. I am referring to using the splici index as any configuration that used the index that was built with both the reads from spliced cDNA and the intronic reads included. We have generated both the 2 column and 3 column tx2gene.tsv file from this index, so in the case of using the full resolution, we are passing the 2 column file through. For the cr-like, we pass the 3 column one through. What that also means is that the outputs are slightly different and if we have used the full resolution with splici, then we grab all of the rows in the counts matrix corresponding to intronic reads and collapse those counts with the corresponding spliced cDNA from that gene to make sure the matrix is summarized to gene level. It's a similar approach to how you guys are doing USA mode where you collapse the reads for S + A reads. Here, we are either removing intronic reads from the counts matrix (in the case of single-cell) or combining reads from intronic reads with the spliced cDNA reads (in the case of single-nuclei).

Ahh, that makes sense; you are doing post-processing of the quantification matrix to extract the intended values.  I think what you are doing here makes sense, though we haven't tested this type of "post-quantification" processing of the values derived under these different resolution strategies with the splici index.  More on this below.

Based on reading through the alevin-fry paper, it appears to me that there is a strong argument for using the cr-like or cr-like-em approach rather than the full resolution as we were doing. In our hands, it appears that the cr-like is producing more similar UMIs/cell and genes/cell as cellranger, which makes sense as it has the same approach of throwing out multi-mapped reads like cellranger.

Right, so there are certainly technical differences between all of our resolution methods and exactly what cell ranger does (of course, we also have some points in the paper that suggest that the cell ranger resolution method may not always be ideal).  The cr-like method will discard gene multimappers (if the fragment frequencies are tied), but the cr-like-em method will actually resolve these gene multimapping reads probabilistically.  The main difference then between something like full and cr-like-em are largely in the exact procedure used to determine how UMIs should be allocated in the case when fragments with the same UMI map between genes or when fragments map among distinct transcripts within a gene.

I'm curious if you tested whether or not use of the full resolution or any of the other resolution strategies had an effect on detection of the false gene expression, similar to what is identified in the absence of using the splici index? I know the full resolution uses EM to resolve multi-mapped reads, probably explaining the increased numbers, but that doesn't necessarily seem like a bad thing to me. Is there any evidence for why increased UMIs and genes with the other resolution strategies other than cr-like could be a bad thing? I'm just trying to get a better sense of the resolution strategies and why we might want to pick one over the other.

So these are great questions.  We are working on expanding the full suite of resolution methods to work natively with splici, so that if you want to use those methods you should be able to do that without having to do the post processing yourself.  To see how much of the difference in observed UMI count is due to discarding multimappers, you can test against cr-like-em.  This almost universally (as expected) produces slightly higher counts than cr-like, though we haven't extensively compared this to full on the splici index, as that is not natively implemented yet.
Finally, while I don't have a lot of data to offer on this, but I strongly doubt that the increased count you see in full is due to the types of false positives that result from not using the splici index.  Generally, those false positives arise from spuriously attributing intron-derived reads to spliced transcripts (either of the same or of a different overlapping gene), and the UMI resolution method itself shouldn't move the needle too much on that.  In this case the difference is likely due more to a combination of (a) retaining gene multi-mapping reads where cr-like and CellRanger are discarding them and (b) splitting some UMIs within a gene when they cannot be uniformly attributed to a consistent set of compatible isoforms.  I think throwing cr-like-em in the mix may give some idea of the relative contribution of both of these causes.
Let me know if you have any other questions; I'm happy to chat more about any of this!
Best,
Rob

This was really helpful, thanks for the clear explanation @rob-p!

In this case the difference is likely due more to a combination of (a) retaining gene multi-mapping reads where cr-like and CellRanger are discarding them and (b) splitting some UMIs within a gene when they cannot be uniformly attributed to a consistent set of compatible isoforms. I think throwing cr-like-em in the mix may give some idea of the relative contribution of both of these causes.

This makes complete sense to me, and my naive interpretation of our results would show me that we are increasing our read count/cell by allowing those multi-mapped reads to get resolved rather than throwing them out. I think the question we will have to answer on our end is whether or not we want to use a resolution that throws out those multi-mapped reads or not which is to be determined. Thank you again for your help on this!

Thanks for the feedback @jashapiro! I went ahead and simplified some things, but also modified the plots just slightly. I realized I had forgotten to address the differences between using the pseudoalignment and selective alignment. Although in the single-cell samples the difference seems minor, there does seem to be a difference in the single-nuclei samples so I included it in the visualization of each of the plots so we can see how each of the variables we are testing impacts the metrics.
I would agree that alevin-fry is the way to go and I think a quick update of the versions and test of the cr-like-em would be a good move. I'm curious about your thoughts on the alignment modes.


Clean up Sprite Dialogue loading and handle routine discrepancies

Currently, the Sprite Dialogue editor doesn't do much to tell the user about discrepancies between the routines listed in the dialogue INI file and the routines found in the actual game (ROM or assembly).

Add some handling to detect routine discrepancies
Provide an option in Sprite Dialogue Settings to either warn or prevent the editor from loading
Ensue that sprites tied to unknown handlers are disabled from editing

Fixed with PR #2


why do u use RockDB

Hi, why do you choose RockDB instead of other DB like MongoDB, MySQL......thank u so much

We are currently not using RocksDB, so I'm not in a good position to answer your question. This repository only contains JRuby bindings for RocksDB, not RocksDB itself, so I think you will find a better answer over on the official homepage: http://rocksdb.org/.


new piece registrar validity

disable ability to complete form unless filled out properly

This was implemented during the big "collections" project


Add a UI loader instead of profile picture while setting the DID.

The DID creation is a complex action that require several seconds in the blockchain environment.


SameFace-Swift5.1.3- problem

Hello,
with the new Xcode 11.3 again the SameFace cannot be integrated, because the Xcode now uses Swift 5.1.3.
Would you please recompile again ?
Is there a way that this can be made independent of the Swift- version to avoid such problems?
There were some hints in the previous tickets.
Thanks
Ante

I am also facing the same error. How to solve this?

Has anyone solved this problem?

Hey everyone, I've recompiled the framework for Swift 5.2 which now provides module stability. If @BohdanNikoletti accepts my pull request, we'll soon have an updated version available on CocoaPods.


Update rbperf slides' links

And the title, as it never got updated
cc @xmulligan
Signed-off-by: Francisco Javier Honduvilla Coto<EMAIL_ADDRESS>

Thanks!


Looks like :dirs can only be one

(reload/init {:dirs ["src" "dev" "test"]})
Execution error (IllegalArgumentException) at clj-reload.core/files->namespaces$fn (core.clj:56).
Don't know how to create ISeq from: java.io.File

This works:
(reload/init {:dirs ["src"]})

Anyways, 0.1.3 should fix that


Missing Armor

Adding armor on cards.

@dreuse Before updating the service do we need to reimport JSONs into the database?
⚠ The database currently does not have armor information!

@dreuse Before updating the service do we need to reimport JSONs into the database?
⚠ The database currently does not have armor information!

I think it will not break because of this missing attribute, but I'll reimport them asap.


python grpcio is installed but grpc can't not be imported

I have already reinstall grpcio

pip freeze screen shot, grpc is missing so I cannot use it in python code

grpc is missing import but the script still run

same issue with me

i have installed the grpcio==1.30.0 and grpcio-tools==1.30.0 in docker environment for python (3.7, 3.9)
grpcio and grpcio-tools  installed in the docker environment but now in my file it is throwing the error that No Module name grpc

@AshfaqWanclouds can you try if our latest release still have the problem on your environment? Also, please open a separate issue, the cause might not be the same.


Handle different different simulation lengths when LSODA returns early

Sometimes in stiff equations (e.g. Rossler), the return from scipy.integrate.solve_ivp() contains fewer points than t_eval.
x
Do we reject these samples?  Or adapt the code to not stack the arrays and allow different simulation lengths (and then bias the regression to the blow-up trajectories?)
For instance, the final line of the following raises ValueError: all input arrays must have the same shape :
import numpy as np
from pysindy.utils.odes import rossler
from scipy.integrate import solve_ivp

y0_1 = np.array([ 4.50040013, -1.72136214, -5.54952269])
y0_2 = np.array([ 0.41888012, -5.95648243,  2.85360632])
t_train = np.arange(0, 4, .01)

y_train = []
for y0 in (y0_1, y0_2):
    y = solve_ivp(
        rossler,
        y0=y0,
        t_span=(0.0, 3.99),
        t_eval=t_train,
        **{"rtol": 1e-12, "method": "LSODA", "atol": 1e-12},
    ).y.T
    y_train.append(y)

np.stack(y_train)

I'm thinking we drop them with a warning, and raise an error if we drop all trials.

Shows up in gridsearch.py:
plot_test_trajectories(grid_data["x_test"][sim_ind], model, grid_data["dt"])
Raises
ValueError: x and y must have same first dimension, but have shapes (1600,) and (1441,)


Captions not working with HLS

Current Behavior
The track element appears in the DOM, but is not visible on the page.
Expected Behavior
Captions appear over the video.
Steps to Reproduce

Load an HLS video into react-player
Add subtitles:

{
    kind: 'subtitles',
    src: 'http://localhost:8080/captions/caption.vtt',
    srcLang: 'en',
    default: true
}

Play video.

Environment

URL attempting to play: HLS video file
Browser: Chrome
Operating system: MacOS

Other Information
DOM output:

Hi @naheller.
I have the same issue.
Did you find some solution?

same issue

Hi @naheller Did you know the solution?

Why on earth is it closed? It is still an issue!

yeah not working with HLS... guess will have to make an overlay for now