Can not bring up networking When stop a VM and start it again

What happened:
stop a VM by virtctl stop xxx and start it again, an error happened Failed to start LSB: Bring up/down networking. and interface eth0 is down
What you expected to happen:
eth0 is up
How to reproduce it (as minimally and precisely as possible):

create a dv
create a vm using a pvc generated by the dv as boot device
start it, eth0 is up
stop it and start it again, eth0 is down

Additional context:
dv.yaml
apiVersion: cdi.kubevirt.io/v1beta1
kind: DataVolume
metadata:
  name: "block-pvc"
spec:
  source:
      http:
         url: "https://cloud.centos.org/centos/7/images/CentOS-7-x86_64-GenericCloud.qcow2"
  pvc:
    storageClassName: xxx
    volumeMode: Block
    accessModes:
      - ReadWriteOnce
    resources:
      requests:
        storage: "10Gi"

vm.yaml
apiVersion: kubevirt.io/v1
kind: VirtualMachine
metadata:
  labels:
    a: b
  name: block-pvc
spec:
  running: false
  template:
    metadata:
      labels:
        a: b
    spec:
      domain:
        devices:
          disks:
          - disk:
              bus: virtio
            name: filepvcdisk
          - disk:
              bus: virtio
            name: cloudinitdisk
        resources:
          requests:
            memory: 1G
      terminationGracePeriodSeconds: 0
      volumes:
      - name: filepvcdisk
        persistentVolumeClaim:
          claimName: block-pvc
      - cloudInitNoCloud:
          userData: |-
            #cloud-config
            chpasswd:
              expire: false
            password: fedora
            user: fedora
        name: cloudinitdisk

Environment:

KubeVirt version (use virtctl version):

Client Version: version.Info{GitVersion:"v1.1.1", GitCommit:"689c0e66cc6893f311dd648ff32e247203b6c96a", GitTreeState:"clean", BuildDate:"2023-12-25T10:58:20Z", GoVersion:"go1.19.9", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{GitVersion:"v1.1.1", GitCommit:"689c0e66cc6893f311dd648ff32e247203b6c96a", GitTreeState:"clean", BuildDate:"2023-12-25T11:56:11Z", GoVersion:"go1.19.9", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes version (use kubectl version): N/A
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): N/A
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

It seems that a new MAC address will be generated on every VM startup, but field HWADDR in /etc/sysconfig/network-scripts/ifcfg-eth0 is never changed, which causing the mismatch, and the network refuses to start

Similar issues:
https://github.com/kubevirt/kubevirt/issues/11415
https://github.com/kubevirt/kubevirt/issues/11577
https://github.com/kubevirt/kubevirt/issues/11113#issuecomment-1963675046

/cc @orelmisan

this also happen in ubuntu 22.04. Any solutions for this?

a workaround is removing field HWADDR in /etc/sysconfig/network-scripts/ifcfg-eth0

It seems that a new MAC address will be generated on every VM startup, but field HWADDR in /etc/sysconfig/network-scripts/ifcfg-eth0 is never changed, which causing the mismatch, and the network refuses to start

Yes, this is indeed correct.
Unless you specify explicitly an interface with a mac address, it will not be persisted between stop/start.
The preferred option is to use something like KMP that allocates a mac address from a pool and assures there are not collisions.

In my case, i'm using Ubuntu families (20.04, 22.04), i remove mac-address and set-name in cloud-init process.
Because I cannot found /etc/sysconfig/network-scripts/ifcfg-eth0 or similiarities in the OS

I believe that on the Linux side,
https://www.freedesktop.org/wiki/Software/systemd/PredictableNetworkInterfaceNames/
is addressing this problem.
MACADDR based syscfg scripts are indeed error prone in our flow.

Message ID: @.***>

It seems that a new MAC address will be generated on every VM startup, but field HWADDR in /etc/sysconfig/network-scripts/ifcfg-eth0 is never changed, which causing the mismatch, and the network refuses to start

Yes, this is indeed correct. Unless you specify explicitly an interface with a mac address, it will not be persisted between stop/start. The preferred option is to use something like KMP that allocates a mac address from a pool and assures there are not collisions.

It's too complex to me, can we just clear the field HWADDR or something similiar?

Oh yes, you can clear the HWADDR field in sysconfig, however, the solution depends on your guest operating system/distribution.

Oh yes, you can clear the HWADDR field in sysconfig, however, the solution depends on your guest operating system/distribution.

I mean can kubevirt do it automatically?

You can set the mac address explicitly on the VM spec. Then it will persist between boots.
But you need to be careful not to define duplicate addresses in your cluster.
https://kubevirt.io/api-reference/v1.2.0/definitions.html#_v1_interface
The field name is macAddress.

You can set the mac address explicitly on the VM spec. Then it will persist between boots. But you need to be careful not to define duplicate addresses in your cluster.
https://kubevirt.io/api-reference/v1.2.0/definitions.html#_v1_interface The field name is macAddress.

@EdDev  Sorry, that's not what I want. Say I have 100 vms, I just want to make sure they are still work after restart, I don't want to carefully pick 100 valid MACs and assign them one by one

You can set the mac address explicitly on the VM spec. Then it will persist between boots. But you need to be careful not to define duplicate addresses in your cluster.
https://kubevirt.io/api-reference/v1.2.0/definitions.html#_v1_interface The field name is macAddress.

@EdDev Sorry, that's not what I want. Say I have 100 vms, I just want to make sure they are still work after restart, I don't want to carefully pick 100 valid MACs and assign them one by one

As mentioned in https://github.com/kubevirt/kubevirt/issues/11585#issuecomment-2094793474 , you can use KMP.
At the moment, there is no other option.

@EdDev , ok, I see. thanks


[GSSoC'23] Flame Fire Animated Project completed.

Title and Issue number

Title - Candle Flame
Issue Number - 2819
Close #2819

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @pranjay-poddar @Saurabh-kumar09 Sirs, please review this pr


permute has different syntax for Tensor and UniTensor

permute has a different syntax for Tensor and UniTensor. Example:
Tensor: t.permute(1,2,0)
UniTensor: uT.permute([1,2,0])
This is confusing. I would suggest that we agree on one syntax for both (the Tensor syntax is similar to other libraries I guess, while the second is more flexible if the rank of the tensor is flexible). Or we provide both ways for Tensors and UniTensors. The decision should then also be applied to reshape (which uses the syntax (int, int, int, ...) for both tensors at the moment).

There are some ambiguous for this, because UniTensor can accept string label, as well as integer.
On python side it should be possible, but on C++ side the API needs to think carefully to support this


fix(blockifier): update compiler version

This change is

crates/blockifier/src/execution/syscalls/syscall_tests/keccak.rs line 29 at r3 (raw file):
Previously, meship-starkware (Meshi Peled) wrote…
@ilyalesokhin-starkware, Can you make sure this gas change looks logical after the compiler version update?

Looks fine


fury serialization & deserialization response times are erratic

@lize1516 Could you share more details about this issue? Like we did for bug report:

Which java version you used
Which fury version you used
How do you create Fury
What scenario will this happen? Did it happen for every time serialization? Or only for the first time? Or for the first time a new thread?


Add da-avail feature flag

When set, this flag disables the eip-4844 blob consistency checks that we currently do.

Maybe I'm not running the tests correctly. Some tests encountered issues when da-avail feature flag is turned on. Other parts look good to me. In addition to adding the avail feature, it's an effective refactor as well.

The tests won't pass when the da-avail flag is set, but this is intended. The tests will pass once the da-avail circuits are added.


Introspective Functions / Testing?

I'm browsing the source, seems like some introspective methods would be useful for testing.  In defining a Dry::Struct, and wanting to write unit tests that I have defined them as expected,
I'm writing the following:
      class BaseStruct < Dry::Struct

        def self.includes_attribute?(attr)
          attr_names.include?(attr.to_sym)
        end

        def self.attr_names
          # schema part of Dry::Struct
          @attr_names ||= schema.keys
        end

        ...

seems like methods like these would be useful within Dry::Struct itself?

Also - any thoughts on shoulda-like helpers -
expect(SomeStruct).to have_attribute(:attr_name).of_type(SomeType)

would this be desirable? or do you have other thoughts?
ftr, willing to contribute if desired.

@MissingHandle check out rspec-dry-struct, I think this does exactly what you want.

Ah - thanks @AMHOL ! That indeed takes care of the testing need.
What about the introspective functions - any thoughts there? I'm coming across a scenario where I have a specification in a JSON file. Within my code, I want to make sure that values in the JSON are valid according to various structs i've defined. For me, the introspective functions seem unavoidably necessary - would you want them within dry-struct?

@MissingHandle Struct.attribute? and Struct.attribute_names have landed in master :) I'll  push a new release next week

I'd actually suggest moving it to an optional module. This type of
functionality shouldn't be typically needed at runtime. I would even say
this should be a rare case that somebody needs this for testing. It's much
better to test behavior versus configuration of some classes.
On Fri, 23 Jun 2017 at 7:15 PM, Nikita Shilnikov<EMAIL_ADDRESS>wrote:

@MissingHandle https://github.com/missinghandle Struct.attribute? and
Struct.attribute_names have landed in master :) I'll push a new release
next week
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dry-rb/dry-struct/issues/45#issuecomment-310723187,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAEKq17TcFX-yhQAjZQ_pS2l5EGmn17ks5sG_K_gaJpZM4OCvbz
.

@solnic tbh I added this specifically for use at the runtime. FWIW using attribute? is more semantic-full than a plain respond_to? call and attribute_names can be useful for metaprogramming. I wouldn't write a test for a struct, for my taste, it's a waste of time :)

I also wanted the functions for runtime. Though I was asking about it, I might agree with you on the testing.


Add ability to add custom actions to GridField record

Mainly I see this as useful for ModelAdmin, but could be useful elsewhere as well.
Currently GridField only adds Create/Cancel or Save/Delete buttons when editing an object. The record controller now checks if the Model class provides a getCMSActions() method, and if so, uses that.
Also added some extra methods for Publishing and UnPublishing content.
Still rudimentary, but think it would be quite a useful feature.

I'd really like to get this feature formalised and, at the very least, put into some kind of tutorial that others can follow to add custom actions. I see the lack of core support for DataObject::getCMSActions() to be an oversight. It's a common use case which shouldn't require that much custom code to get working.

+1

+1 too ;)

And +1  (or have I missed something?)

Although I forgot about this issue, I haven't stopped working on it.
In 4.0 it'll be a lot easier to inject custom request handlers to gridfield for extra actions.
Please see https://github.com/open-sausages/silverstripe-framework/tree/8276b4e6bd28c952e1886eb74a0012637a5b673f/model/versioning for a proof of concept; This example lets you do versioning actions in gridfield for versioned dataobjects. :)


Collapse component should match image width on Benefits Navigator page

Collapse component on Benefits Navigator page should be the same width as the preceding image
Instead of always spanning the width of the page, the Collapse components on the Benefits Navigator page should match the image size responsiveness.
Test Instructions

Go to Benefits Navigator page
See that Collapse components takes up the same width as it's preceding image on small and large displays

Check out a preview of your pull request here: https://alphasite-appservice-dev-pr-preview-1452306092.azurewebsites.net


Added Build Podspec Script

Added build_podspec.sh

@swift-nio-bot test this please

@weissi Good catch! Pushed the changes 👍

@swift-nio-bot test this please


How to clean ressources ?

Any way to clean resources using this library. I would like to open a socket using suspense but when the Host change, i need to reload a new socket and would like to clean effect the previous.

I too am wondering about this. I asked on Stackoverflow.

you have clear, eg: https://github.com/pmndrs/drei/blob/4aa04c93e4e86711f0a5a6777482a92d52988253/src/core/FaceControls.tsx#L350

@abernier so is the answer "use an effect"?

that's how I did yes -- that way if a dep changes, it will be cleaned up and cleared

@abernier That's not ideal because it de-couples the effect from it's clean-up.
Suspend-react is aware of when the dependencies change, and so is best-positioned to run clean-up effects at the right time.

I thought like you primarily, I even asked about that on Discord
From @drcmda's answer, I guess this decoupling is intentional:

suspense is a cache, unmount means nothing to a cached entry, it will continue to be available

which makes sense to me, it's a cache, nothing more
If you want more, like a side-effect when your cache-key changes, then you have useEffect

I see.
Sounds like we should be wrapping suspend in our own hook.
Something like this.
const useSuspense(cb, deps) {
  const { value, cleanup } = suspend(cb)

  useEffect(() => {
    () => cleanup()
  }, deps)

 return value;
}

...but then we get the pitfall from a lack of reference-counting if two components use the same suspended keys.
useEffect(() => {
    () => {
      if (lastRef) evictFromCache();
      cleanup()
   }
  }, deps)

I found an implementation of a "useMemoCleanup" hook from here
that performs cleanup.
i've been using it in my own project like so:
const texture = useMemoCleanup( () =>
		[
			suspend(async () => {
                            // your code here
                            return texture;
                        }, []),

			// cleanup func
			() => {
				texture .dispose();
			}
		], [] );

and it seems to be working so far
(even though i haven't heavily tested it)
hope this helps someone

@nassosyian How does this work? Wouldn't the memorized value be lost when suspense throws to the boundary?


Update hdinsight.md

Docs Build status updates of commit 9fb4145:
:clock10: Full build: the average full build time is 70 min(s) 1 sec(s), this is based on the last 203 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 9fb4145:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/hdinsight.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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.


[red-knot] intern types separately from type inference result

Initial exploration of what it might look like to intern types independent of the type inference result, so the granularity of type inference and the granularity of type interning don't have to be coupled.
The goal of this is to make it possible to add definition_ty queries, but this PR doesn't do that yet.
This PR also doesn't work yet :) Just for illustration, I'm using Arc::get_mut in the places where we need to mutate the FileTypeStore, but we can't actually do this because Salsa is still holding on to a reference to the Arc also. To make this work, we'll need to introduce interior mutability to the FileTypeStore, which is a bit tricky along with wanting to return references to type structs stored inside it. The old TypeStore in the old red_knot crate shows one approach to this, but I'm not sure it's the best option.
Having the FileTypeStore returned by a salsa query at all might require the Salsa "mutable arena" feature discussed in https://github.com/salsa-rs/salsa/pull/490#discussion_r1641000198 -- if so, for now we could instead just keep our our own dashmap of VfsFile to FileTypeStore on the Db, but not actually in Salsa.
Some nice things about interning types ourselves: a lot fewer lifetimes (no lifetime on Type enum anymore), and we can eliminate the TypingContext indirection layer and just use Db in all APIs.

As reference, this is how rustc interns types:
The data type:
https://github.com/rust-lang/rust/blob/42add88d2275b95c98e512ab680436ede691e853/compiler/rustc_middle/src/ty/adt.rs#L97-L106
A reference to a type (bound to the TyCtxt lifetime)
https://github.com/rust-lang/rust/blob/42add88d2275b95c98e512ab680436ede691e853/compiler/rustc_middle/src/ty/adt.rs#L172-L174
The definition of Interned
https://github.com/rust-lang/rust/blob/42add88d2275b95c98e512ab680436ede691e853/compiler/rustc_data_structures/src/intern.rs#L13-L25
And this is used in TyCtxt which is rustcs arena where it interns all data.
https://github.com/rust-lang/rust/blob/42add88d2275b95c98e512ab680436ede691e853/compiler/rustc_middle/src/ty/context.rs#L194-L197

I do like splitting out the type internet from the type inference result. I think it makes sense to have them separate and also allows more effective reuse.
The part that's unclear to me and that you already mentioned on your write up is how to collect unused types. How can we prevent that this is an ever-growing arena?
We could try to track type references by using an AtomicUsize (not an Arc because that would heap allocate each type!) It would give us very fine-grained reclamation of types, at least if we use a slab like arena that supports reusing IDs. An other possibility could be to rely on Salsa by using one interner per file, package or some other granularity, and the interner would be retrieved by calling a salsa-query. This won't give us type level collection, but salsa would evict the entire type store if e.g. the file gets deleted (or all files in a package).

This now uses Salsa interning, which eliminates a lot of our own interning and ID boilerplate, and actually works!

Some things in this PR that might require further attention:

I think you mentioned you don't want to expose Salsa Db directly to lint rules, but I'm not sure how this will work with the need for lint rules to call methods on types. Previously you exposed typing context, which is just a wrapped db and is passed to all methods on types. Now methods on types take a db, so if we want to avoid exposing db to lint rules, they won't be able to call methods on types.
Interning UnionType and IntersectionType in Salsa requires that all their fields be hashable, which means I can't use IndexSet. For now I just switched to Vec, but I need to do a bit more careful analysis of what properties we need here (do we want ordering? do we need fast contains checks? etc) in order to settle on the right data structure.

The message says I'm supposed to acknowledge the regression on CodSpeed, but when I login to CodSpeed with GitHub, it says I'm not allowed to acknowledge the regression...

You know have the permissions to acknowledge regressions. Use it responsibly :P

After offline discussion with @MichaReiser, going to switch to OrderSet for unions and intersections (from https://crates.io/crates/ordermap/0.5.0 ), which is a newtype wrapper around IndexSet that also adds order-respecting Eq and Hash. If we care about respecting order in unions (for UX reasons only, not because it's meaningful in the type system), which I think we do, and we care about fast contains checks (I think we also do), then this seems like the best option. It means we will intern "duplicate" unions that differ only in order separately, but this is pretty much unavoidable if we really want to respect order for UX. We will have to see how much of a cost this is in practice, and if it's worth paying for the UX.

The updated perf results are a bit better; not sure if this is just noise or if not interning ModuleType actually helped. I don't think Vec vs OrderSet should help, since we aren't currently doing any contains checks on unions or intersections.


Cu 21prury pull regs

Task linked: CU-21prury Retrieve the metainformation about the consolidated acts and regulations


THRIFT-4018 corruption after ApplicationException in Ruby server

This fixes an issue where a Ruby Thrift server can corrupt connections by putting ApplicationException in the middle of a response. See https://issues.apache.org/jira/browse/THRIFT-4018 for more information.

Would you be able to rebase this on the current master and push?  This would kick a new build, and I would like to see it pass before merging anything.


Adds Multivariate Hawkes benchmark.

This PR adds the Multivariate Hawkes benchmark for comparing jump algorithms that accept variable rates.
The document contains a section that produces QQ-plots which are useful to check the correctness of the simulations before benchmarking them.
The benchmark also benchmarks a Python library. The document installs and sets up the required Conda environment. However, I don't know whether that is the recommended way to do that.
The document is quite long as I aimed to keep it self-contained. I am looking forward to some feedback on how to improve it.

It looks like that despite the fact that the Build Kite processes finished successfully, the markdown file did not compile correctly.
...
Error: InitError: could not load library "/cache/julia-buildkite-plugin/dep
ots/5b300254-1738-4989-ae0a-f4d2d937f953/conda/3/x86_64/lib/libpython3.10.s
o.1.0"
/cache/julia-buildkite-plugin/depots/5b300254-1738-4989-ae0a-f4d2d937f953/c
onda/3/x86_64/lib/libpython3.10.so.1.0: cannot open shared object file: No
such file or directory
during initialization of module PyCall

It seems like the Conda environment was generated sucessfully but then PyCall could not be loaded.

Hmm getting Python to work in this might be hard.

An option is to skip the Python program when running on cloud. What do you think?

Let's go with that to start and then improve it in steps.

PR #585 turns off the Python benchmark on the cloud.


Fix zip deploy logic of if preparing resource is required

Fix issue #361

Codecov Report

Merging #363 into develop will decrease coverage by 0.03%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #363      +/-   ##
===========================================
- Coverage    55.51%   55.48%   -0.04%
===========================================
  Files           62       62
  Lines         1823     1824       +1
  Branches       210      211       +1
===========================================
  Hits          1012     1012
- Misses         743      744       +1
  Partials        68       68

Impacted Files
Coverage Δ

.../maven/artifacthandler/ZIPArtifactHandlerImpl.java
61.53% <0%> (-2.47%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@57f67b7). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #363   +/-   ##
==========================================
  Coverage           ?   55.01%
==========================================
  Files              ?       62
  Lines              ?     1823
  Branches           ?      209
==========================================
  Hits               ?     1003
  Misses             ?      753
  Partials           ?       67

Impacted Files
Coverage Δ

.../maven/artifacthandler/ZIPArtifactHandlerImpl.java
66.66% <0%> (ø)

.../maven/webapp/handlers/JarArtifactHandlerImpl.java
0% <0%> (ø)

Continue to review full report at Codecov.

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


Add zerotier-one

I think that zerotier is already available in entrare but it misses the systemd integration.

I think that zerotier is already available in entrare but it misses the systemd integration.

It looks like you are correct. I'll have to check to see what all they include, but it is a couple of minor versions behind. They probably don't have the manpages or selftest packages either.

https://github.com/Entware/entware-packages/tree/master/net/zerotier
It looks like they have the same config and a couple of other small patches that I should take a closer look at. They also have picked an odd name for the package. Most distributions call it zerotier-one since that's what the software is called.
I'll have to adjust my package to take into account that the entware one exists, and add some of the patches they use.

Tested on my rM1 and it's working. The selftest succeeds as well:

The selftest also passes on my rM2, as well as running it.

Tested with linux-stracciatella and this works as expected.


Implemented Event Card with placeholder images

Issue ID
COLOS-6
What has been done?

[x] Made the basic Event Card
[x] Placeholder images
[ ] Actual images in place of the placeholder images
[ ] Implemented Active Layout around the Event Card

Sceenshots/Videos

Closed the PR because of a small miss-click, wanting to remove an unwanted comment...
Reopened again, sorry


Unresponsive content when sidebar is collapsed

What version of mystjs are you using?
latest
How did you install myst?
npm
What operating system are you using?
Mac
Which area is this feature request for?
Website User Interface
Describe the Bug
When the sidebar is collapsed on a narrow screen, the content that is close to the left side of the window becomes unresponsive, as can be seen in this screen recording:
https://user-images.githubusercontent.com/44469195/221351897-1a5836ad-64f0-45bf-9db3-b5561c84d20b.mov

Here is a sample project for testing media-content:
https://github.com/kolibril13/mystjs-media-example
Expected Behavior
No response
To Reproduce
No response

Thanks for this bug report -- I made a change to the mobile nav, but that is now open + not visible on all variants! I will get that removed in @myst-theme.

Also tracking here:
https://github.com/executablebooks/myst-theme/issues/25

Bug is fixed and merged, just needs a release.

Have released and updated the website. To use this locally, myst clean --templates and then myst start!
Thanks again for reporting this, silly mistake -- however, it is a prelude to some very cool upcoming features for article layout. 🚀


fix: future ownership task grant (#1954)

This Pull Request corrects the OWNERSHIP role revocation to be on the FUTURE TASKS GRANT rather than the already created TASKS. This fixes #1954
Test Plan
Additional tests have been added for unit testing in future_grant_test.go and task_grant_acceptance_test.go

/ok-to-test 9b81a4e

/ok-to-test sha=9b81a4e

this looks good to me.


Added build-grcov command

This adds a build-grcov command described in https://github.com/rust-fuzz/honggfuzz-rs/issues/28.
Please note that this feature was hacked in a short time and there might be some better way to do that or there might be a better set of flags to get profiling.
I am happy to update this PR according to the comments.
This can also be further updated by e.g. adding a run-grcov which would run such built binary against all crashes and inputs and generate a zip with grcov profiling data (that can later be used to generate coverage report for given sources/package).

Thanks @disconnect3d ! (and sorry for answering lately)
Would you mind very quickly explaining how you then use the resulting binary so that maybe I'll also create a run-grcov command too :-)


Update Select.php

Same as pull #864 for master.
Show placeholder if Select field is empty. If placeholder value is not set then show field label to prevent php
error.

@z-song Is it possible to merge this request into the 1.4 branch?
This goes hand in hand with #873.


chore: remove unused dependencies

Couldn't find any usages of these dependencies. I have rebuilt and ran Storybook. All works as before.

Something broke when I fixed conflicts in https://github.com/warp-ds/react/pull/118 so I create this one and closing the first one


Added additional initWithFrame methods to allow for chart options

Added additional initWithFrame methods to allow for choice of shadowColor, displayCountingLabel BOOL and overrideLineWidth option.
Also change the UIBezierPath radius calculation so circle uses the full frame space allotted.

Thanks Kevin!
That's actually my first ever contribution to a GitHub project. It feels
good to help out. :)
My work keeps me so busy, I usually never have a moment to consider it.
I thought you would like to see how we are using your PNChart:
[image: Inline image 1]
Thanks again,
Craig
On Fri, Jan 9, 2015 at 5:43 AM, Kevin<EMAIL_ADDRESS>wrote:

Merged #138 https://github.com/kevinzhow/PNChart/pull/138.
—
Reply to this email directly or view it on GitHub
https://github.com/kevinzhow/PNChart/pull/138#event-216460962.

--
[image: --]
Craig McLeod
[image: http://]about.me/dredful
http://about.me/dredful?promo=email_sig


Tap service errors

Describe the bug
The Tap service will not run (not even on tutorial notebooks, for example tutorial notebook #2)
To Reproduce
Steps to reproduce the behavior:

Go to tutorial notebook 'Intermediate TAP Queries for DP0 catalogs'
Run the notebook until section 1.2 Schema discovery
See error "DALServiceError: 401 Client Error: Unauthorized for url: https://data.lsst.cloud/api/tap/sync"

Expected behavior
Normally this should run without errors.
Desktop (please complete the following information):

OS: Mac OS desktop
Browser Chrome

Additional context
I first noticed this error while working on my own notebooks. I checked the tutorial notebooks, and indeed the same problem occurred. I erased my erased my tutorial-notebooks directory and re-'git'ed it, but still the same problems. Perhaps there has been a change to how we are supposed to be calling on the Tap from inside Jupyter notebooks?

Thanks for reporting this issue!
I attempted to recreate the issue, but the TAP service runs OK for me with tutorial-notebooks 1 and 2, with the most up-to-date versions and the recommended image Weekly 2021_49.
@ledwar04 does the issue persist for you, or was it maybe just temporary?
When I ran Section 1.2 of NB 02 just now, these are the same cells where you get that error?

Thanks, Melissa!
Unfortunately, the issue still persists.
I double-checked and indeed am also running the Weekly 2021_49 versions. Neither tutorial notebook 1 nor 2 are working. Here's a longer version of the error message:

E19                                       Traceback (most recent call last)
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/pyvo/dal/query.py in execute_votable(self, post)
241         try:
--> 242             return votableparse(self.execute_stream(post=post).read)
243         except Exception as e:
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/astropy/utils/decorators.py in wrapper(*args, **kwargs)
535
--> 536             return function(*args, **kwargs)
537
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/astropy/io/votable/table.py in parse(source, columns, invalid, verify, chunk_size, table_number, table_id, filename, unit_format, datatype_mapping, _debug_python_based_parser)
165             _debug_python_based_parser=_debug_python_based_parser) as iterator:
--> 166         return tree.VOTableFile(
167             config=config, pos=(1, 1)).parse(iterator, config)
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/astropy/io/votable/tree.py in parse(self, iterator, config)
3600                 else:
-> 3601                     vo_raise(E19, (), config, pos)
3602         config.update(self._get_version_checks())
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/astropy/io/votable/exceptions.py in vo_raise(exception_class, args, config, pos)
111         config = {}
--> 112     raise exception_class(args, config, pos)
113
E19: None:1:0: E19: File does not appear to be a VOTABLE
During handling of the above exception, another exception occurred:
DALServiceError                           Traceback (most recent call last)
/tmp/ipykernel_16457/4172518829.py in
----> 1 results = service.search("SELECT ra, dec, mag_g, mag_r, mag_i "
2                          "FROM dp01_dc2_catalogs.object "
3                          "WHERE CONTAINS(POINT('ICRS', ra, dec), CIRCLE('ICRS', 62.0, -37.0, 0.5)) = 1 ", maxrec=10)
4 results_tab = results.to_table()
5 results_tab
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/pyvo/dal/tap.py in run_sync(self, query, language, maxrec, uploads, **keywords)
244         TAPResults
245         """
--> 246         return self.create_query(
247             query, language=language, maxrec=maxrec, uploads=uploads,
248             **keywords).execute()
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/pyvo/dal/tap.py in execute(self)
940            for errors parsing the VOTable response
941         """
--> 942         return TAPResults(self.execute_votable(), url=self.queryurl, session=self._session)
943
944     def submit(self, post=False):
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/pyvo/dal/query.py in execute_votable(self, post)
242             return votableparse(self.execute_stream(post=post).read)
243         except Exception as e:
--> 244             self.raise_if_error()
245             raise DALFormatError(e, self.queryurl)
246
/opt/lsst/software/stack/conda/miniconda3-py38_4.9.2/envs/lsst-scipipe-0.7.0/lib/python3.8/site-packages/pyvo/dal/query.py in raise_if_error(self)
251         if self._ex:
252             e = self._ex
--> 253             raise DALServiceError.from_except(e, self.queryurl)
254
255     @property
DALServiceError: 401 Client Error: Unauthorized for url: https://data.lsst.cloud/api/tap/sync

Hmmm, OK, that error traceback looks like it's referring to the execution of the cell in Section 2.3 of NB 01 -- did the calls to service.search earlier in the notebook produce the same or similar error messages?
I'm not sure what the problem is, just trying to collect more information before bringing this to the attention of others.

OK! Maybe it was temporary? I've just tried again, and now everything works. I logged out and back in (which I also did this morning...) but that seems to have cleared it up. Thanks!
l.

Hmm, ok. I'll still bring it to the attention of the RSP team, just in case it's indicative of an underlying problem. I am going to close this issue, but, definitely report again if it happens again. Thanks Louise!


feat(TFD-1765) : add AWS Redshift support to JDBC

What is the current behavior? (You can also link to an open issue here)
JDBC component doesn't support AWS Redshift as database.
What is the new behavior?
Add support for AWS Redshift support.
Note :
Redshift provides 2 JDBC drivers, a light weight one redshift-jdbc42-no-awssdk which supports basic username/password authentication and a uber jar redshift-jdbc42 which in addition to simple authentication supports also the AWS IAM authentication (https://docs.aws.amazon.com/redshift/latest/mgmt/generating-iam-credentials-configure-jdbc-odbc.html).
The Uber jar contains nested jars so it isn't compatible with sandbox class loader (https://forums.aws.amazon.com/thread.jspa?threadID=263573), so we had to add separately the AWS SDKs to the extra artifacts in addition to the redshift-jdbc42-no-awssdk in order to support Simple & IAM authentication https://jira.talendforge.org/browse/TFD-3748.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results

Pull request test result: success :star:
Sonar results


Set minimum protocol version to TLSv1.2_2019

what

Set minimum protocol version to TLSv1.2_2019

why

This is the most secure option as it only supports tls 1.2 and 1.3

references

https://aws.amazon.com/about-aws/whats-new/2020/07/cloudfront-tls-security-policy
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/secure-connections-supported-viewer-protocols-ciphers.html#secure-connections-supported-ciphers

/test all


Rewrite: Fix templates helpers loading

Loading templates/markdown/helpers.ts fails for me with "syntaxerror: Cannot use import statement outside a module" as "import" call is used targeting the ".ts" file, not the build ".js" file.
I have fixed this issue by copying templates ".hbs" files to "dist" using copyfiles package and changing used templates directory to the one in dist.

Thank you @cytadela8, but I thought I had fixed this in https://github.com/OpenZeppelin/solidity-docgen/commit/7a988820e1393a1edf57581ae202bd6d31c8d263.
Can you share the error with stack trace?

I can confirm it's fixed.


setup.py fails on Arch Linux x64

Traceback (most recent call last):
  File "setup.py", line 72, in <module>
    requirements = [str(ir.req) for ir in parse_requirements('requirements.txt')]
  File "/usr/lib/python2.7/site-packages/pip/req/req_file.py", line 19, in parse_requirements
    "parse_requirements() missing 1 required keyword argument: "
TypeError: parse_requirements() missing 1 required keyword argument: 'session'

This is all I get when I run
 python2.7 setup.py install

Being root doesn't make a difference, I have pip installed so I'm not sure what I'm doing wrong. The repository was just cloned today. Running it with just
python setup.py install

defaults to using python 3.4. Contents of uname -a below:
amelia@artemis~/g/i/Ice> uname -a
Linux artemis 3.18.2-2-ARCH #1 SMP PREEMPT Fri Jan 9 07:37:51 CET 2015 x86_64 GNU/Linux

Huh, thats really odd. Can you confirm that requirements.txt exists in that folder?

Yup, got it right here.
amelia@artemis~/g/i/Ice> ls
config.txt    emulators.txt    icon.icns  LICENSE    requirements.txt  src/
consoles.txt  ice-example.png  icon.ico   README.md  setup.py          tools/
amelia@artemis~/g/i/Ice> cat requirements.txt
psutil==2.1.1
appdirs==1.2.0
pysteam>=0.1.3

Ok, thanks. Not sure what the issue is, looks like I need to get myself an Arch Linux install.
Thanks for the report, I'll look into it.

This issue should be fixed, but would you mind pulling the most recent version and checking to confirm?


Define behavior of add/remove during iteration.

Should it be explicitly undefined? Should it throw an error? Can we handle mutation during iteration (probably not)?

Closing due to inactivity. Also, I'm curious about the use case?


Updating warbler version to fix build failures

What:
This pull request updates warbler version in the Gemfile and Gemfile.lock to fix issues with bundler >10.1
Why:
Otherwise, build fails due to old warbler not being compatible with new bundler (bundler is not locked in version, since it is downloaded separately)
More:
See: https://github.com/bundler/bundler/issues/3691 and https://github.com/rubygems/rubygems/issues/505 for details.

Lipstick-pull-requests #99 FAILURE
Looks like there's a problem with this pull request

Lipstick-pull-requests #100 FAILURE
Looks like there's a problem with this pull request


addressed issues with test error in api

Substitute this line for a meaningful title for your changes
Overview
Please write a description. If the PR is hard to understand, provide a quick explanation of the code.
Designs
Link to the related design prototypes (if applicable)
Testing instructions
Please explain how to test the PR: ID of a dataset, steps to reach the feature, etc.
Feature relevant tickets
Link to the related task manager tickets

Checklist before submitting

[ ] Meaningful commits and code rebased on develop.
[ ] If this PR adds feature that should be tested for regressions when
deploying to staging/production, please add brief testing instructions
to the deploy checklist (docs/deployment-checklist.md)
[ ] Update CHANGELOG file

as i put in the previous one there are a couple of things i want to do on those test but will take me some time to and i prefer to open another pr to handle those.


Ben weinstein

Updates to vignettes.
Coded Vazquez 2009 mgen wrapper (descriptive method, no predict possible)
Multinomial bayesian model with a Dirichlet prior, tested and runs clean.

Current coverage is 32.40% (diff: 16.91%)

Merging #11 into master will decrease coverage by 1.10%

@@             master        #11   diff @@
==========================================
  Files            17         18     +1
  Lines           385        537   +152
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            129        174    +45
- Misses          256        363   +107
  Partials          0          0

Powered by Codecov. Last update abc3fa0...df92ea3

@bw4sz this needs to be tested before merging.

@tpoisot
Can you provide a bit more detail? When it says the code coverage has decreased, does it mean that some dependencies are undefined, or that there are global variables missing? I'm not totally following what is being evaluated.

As an example I add a test for OccupancyToJags in the previous commit !
I simply test that the file is indeed created.

@bw4sz sure -- it's counting the number of lines that are executed when you run the tests. It's expected that every possible situation is tested. For example the test by  @KevCaz is one, but it's not sufficient since it's not telling anything about whether the file is correct.

Tim is right, what I have done is not enough. I guess the best is to apply your models (which requires jags files) on simple cases for which results are known.

The test implemented by @KevCaz for the functions (binomialToJags,  interceptToJags, multinomialToJags, poissonToJags, OccupancyToJags) are fine. These functions are supposed to create files.
@bw4sz, is there a way to send a fake dat object  to your fit_bayesreg function and know (a priori) the expected results/values (or at least the range) of this function? In that case, you compare the output of the function fit_bayesreg with the expected result.
If you need further details or examples have a look at http://r-pkgs.had.co.nz/tests.html. We know that implementing unit tests is more jobs but that practice is well suitable to prevent bugs and errors in the code.

Looking at the names of these functions, I wonder: shall we start the name by a lowercase letter or a uppercase one?

@KevCaz, lower !

Looks like steve updated the camelCase, and the namespace will need to be rebuilt. I will not get to testing these functions this week, i suggest we move forward with getting functions together.

Ben, I have stried to solve the issues, can you check if everything is Ok on your side??

@KevCaz , i pulled, merged, and pushed, everything looks okay on first pass. Will run my vignette in a sec.


Clockwork CDT

Describe the bug
Sometimes when you encounter gilded reanimation CDT will happen. Happens at least 20 times for me while I'm completing the story of Clockwork. At some point its become unavoidable for me to complete the quest without CDT. Fixed with disabling gilded reanimation in the MCM menu.
To Reproduce
Steps to reproduce the behavior:

Go to the Hall of The Elements
Clear location
Wait till Gilded are reanimated
Try to run through the location again, 100% you will encounter CDT.

If you use my save - all you need is run to the exit of the current location and you will have CDT shortly after.
Expected behavior
No CDT.
Save
saves.zip
Crash log
crash-2022-08-04-08-56-16.log
Desktop (please complete the following information):

OS version:  Windows 10
RAM: 16gb
CPU: ryzen 3700
GPU:  RTX 1080 ti
Storage type Yggdrasil VR is installed on: SSD
Antivirus Windows Defender

Additional context
Needs to be fixed CDT's with gilded reanimation, or disabled gilded reanimation by default.

I made an MCM recording to automatically turn the Gilded respawn off from the start.  Should prevent the CTD error.
0001_Clockwork .zip

I trust that mcm recorder script to work, if the bug reappears this can be reopened.


Add version: antonreshetov.massCode version 1.3.1

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

InstallerUrl contains Manifest version instead of PR version:
InstallerUrl: https://github.com/massCodeIO/massCode/releases/download/v3.7.0/massCode.Setup.3.7.0.exe
(Automated message - build 601)


"World" at the top of each page should read "Planet" in this game!

Hab den issue mal zu lean4game kopiert.

(fixed on dev)


test: coverage for foreign key violation err

[X] Do only one thing
[X] Non breaking API changes
[X] Tested

What did this pull request do?
Test coverage for ErrForeignKeyViolated, related issue: https://github.com/go-gorm/gorm/issues/6345

Currently, MySQL PR is not merged https://github.com/go-gorm/mysql/pull/122, and it is not implemented for sqlserver; I will do the PR for sqlserver.

Mysql driver giving some errors, I will fix that.

Broken tests fixed


Adding symbol for sqrt

Oh, sorry, one least thing, needs a release note, see dimod's README. Something like this one.


D435i IMU problem

Required Info

Camera Model
D435i

Operating System & Version
Win 10

Language
python

Issue Description
As I have taken IMU data by using following code :
accX = frames[0].as_motion_frame().get_motion_data().x
accY = frames[0].as_motion_frame().get_motion_data().y
accZ = frames[0].as_motion_frame().get_motion_data().z

gyroX = frames[1].as_motion_frame().get_motion_data().x
gyroY = frames[1].as_motion_frame().get_motion_data().y
gyroZ = frames[1].as_motion_frame().get_motion_data().z

The Data which I am getting is highly fluctuating and from that I also not be able to derive accurate YAW , PITCH , ROLL etc values .
Is there any method available in python to derive these data ( YAW, PITCH , ROLL ) directly ?

@TanmayPathak hello, IMU raw data inherently exhibits noisy characteristics.
Please take a look at rs-motion demo that continuously integrates IMU data to produce orientation angles/matrix.
Duplicate of #4356 .

@ev-mp from the rs-motion example, we can get angles of two axis only. Isn't it ?

@malith1992, that is correct. In an inertial system affected by G-Force only you can extract the actual Pitch and Roll angles. You also track the left-right rotation, but the initial Heading angle is arbitrary defined.

I wrote a code to calculate the YAW, PITCH and ROLL in python by referring rs-motion example and a IMU data gathering example . I used the same logic. Please tell me if there is any wrong in my code @ev-mp
`import pyrealsense2 as rs
import math
def initialize_camera():
# start the frames pipe
p = rs.pipeline()
conf = rs.config()
conf.enable_stream(rs.stream.accel)
conf.enable_stream(rs.stream.gyro)
prof = p.start(conf)
return p
p = initialize_camera()
first = True
alpha = 0.98
try:
while True:
f = p.wait_for_frames()
    #gather IMU data
    accel = f[0].as_motion_frame().get_motion_data()
    gyro = f[1].as_motion_frame().get_motion_data()

    ts = f.get_timestamp()

    #calculation for the first frame
    if (first):
        first = False
        last_ts_gyro = ts

        # accelerometer calculation
        accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
        accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
        accel_angle_y = math.degrees(math.pi)

        continue

    #calculation for the second frame onwards

    # gyrometer calculations
    dt_gyro = (ts - last_ts_gyro) / 1000
    last_ts_gyro = ts

    gyro_angle_x = gyro.x * dt_gyro
    gyro_angle_y = gyro.y * dt_gyro
    gyro_angle_z = gyro.z * dt_gyro

    dangleX = gyro_angle_x * 57.2958
    dangleY = gyro_angle_y * 57.2958
    dangleZ = gyro_angle_z * 57.2958

    totalgyroangleX = accel_angle_x + dangleX
    totalgyroangleY = dangleY
    totalgyroangleZ = accel_angle_z + dangleZ

    #accelerometer calculation
    accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
    accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
    accel_angle_y = math.degrees(math.pi)

    #combining gyrometer and accelerometer angles
    combinedangleX = totalgyroangleX * alpha + accel_angle_x * (1-alpha)
    combinedangleZ = totalgyroangleZ * alpha + accel_angle_z * (1-alpha)
    combinedangleY = totalgyroangleY

    print("Angle -  X: " + str(round(combinedangleX,2)) + "   Y: " + str(round(combinedangleY,2)) + "   Z: " + str(round(combinedangleZ,2)))

finally:
p.stop()`

@malith1992
I try to your source test. but  why result Angle(x, y, z) data different?

@AhnHyeongHyun What kind of different you mean ?

@malith1992, in the above code you use hard-coded indexes for accel and gyro frames, which is not the correct approach:

#gather IMU data
accel = f[0].as_motion_frame().get_motion_data()
gyro = f[1].as_motion_frame().get_motion_data()

You need to iterate over the frames in the frameset to establish their affinity.
Also the orientation calculation is wrong - the acceleration values refer to the linear acceleration, i.e. they affect the device's movement in 3D space (x,y,z)  but they do not affect the camera orientation. The orientation is tracked exclusively by integrating  gyro velocities.

totalgyroangleX = accel_angle_x + dangleX
#accelerometer calculation
accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
accel_angle_y = math.degrees(math.pi)

In certain situations you can extract orientation roll and pitch from the acceleration values and use them to compensate for the gyro drifts. But this is only possible if (a) you have the IMU unit perfectly calibrated, and (b) the sum of forces applied to the camera equals G (e.g the device's is either completely static or moves with a constant linear velocity)

@TanmayPathak , do you have follow-up questions on this topic ?

Hello, I'm new to sensors like IMU, and I've got some questions about using D435i IMU and converting it to angles.

I tried the rs-motion, and here's my result.

As you can see, roll and pitch works fine, but yaw is not, which should be fine quoting this.

@malith1992, that is correct. In an inertial system affected by G-Force only you can extract the actual Pitch and Roll angles. You also track the left-right rotation, but the initial Heading angle is arbitrary defined.

My question is, why the gif shown at the rs-motion sample can track left-right (yaw) rotation?

Referencing from the imutest, rs-motion, and this issue source code, I tried to implement my own rs-motion in python and it looks similar to @malith1992's implementation. I notice that my Y value (yaw) is only fluctuating a bit from the initial value of PI or not changing at all (tried the above source code I am referencing above and got the same result). Is there something wrong with my device or do I need to implement another function to track the left-right rotation? If my device is normal, I'd be happy if someone could explain to me how to track the left-right rotation (it's okay even if the initial heading angle is arbitrary defined, as long as I know the degree of rotation from the initial position).

@magnusbarata , it seem that the gyro is not working properly.
With gyro streaming normally there should be no issue of tracking left/rigth. Actually gyro has no notion of "left/right" it just tracks attitude in 3D inertial frame.
Did you try to run Reaselse-Viewer and check the IMU framerates and data ?
As for the demo - please verify that this is not a host issue ( Release build/ Opengl drivers/ Remove Desktop) by running the same demo on a different PC

@malith1992, in the above code you use hard-coded indexes for accel and gyro frames, which is not the correct approach:

#gather IMU data
accel = f[0].as_motion_frame().get_motion_data()
gyro = f[1].as_motion_frame().get_motion_data()

You need to iterate over the frames in the frameset to establish their affinity.
Also the orientation calculation is wrong - the acceleration values refer to the linear acceleration, i.e. they affect the device's movement in 3D space (x,y,z) but they do not affect the camera orientation. The orientation is tracked exclusively by integrating gyro velocities.

~totalgyroangleX = accel_angle_x + dangleX
#accelerometer calculation
accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
accel_angle_y = math.degrees(math.pi)~

In certain situations you can extract orientation roll and pitch from the acceleration values and use them to compensate for the gyro drifts. But this is only possible if (a) you have the IMU unit perfectly calibrated, and (b) the sum of forces applied to the camera equals G (e.g the device's is either completely static or moves with a constant linear velocity)

can you correct the mistakes in the codeabove @ev-mp , or provide a clear way to calculate the following values？

Hey, thanks for the code. It is working very well, however, except the angle_Y. I made a few change so that angle_Y could be estimated by integrating gyro_Y (tested, working well ignoring the shift after long-time running).
    alpha = 0.98
    totalgyroangleY = 0
    try:
        while True:```

        ...
        ...

        totalgyroangleX = accel_angle_x + dangleX
        # totalgyroangleY = accel_angle_y + dangleY
        totalgyroangleY = accel_angle_y + dangleY + totalgyroangleY
        totalgyroangleZ = accel_angle_z + dangleZ

        #accelerometer calculation
        accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
        accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
        # accel_angle_y = math.degrees(math.pi)
        accel_angle_y = 0

        #combining gyrometer and accelerometer angles
        combinedangleX = totalgyroangleX * alpha + accel_angle_x * (1-alpha)
        combinedangleZ = totalgyroangleZ * alpha + accel_angle_z * (1-alpha)
        combinedangleY = totalgyroangleY

Please look at the commented line and the amended line below it if you wanna get the angle_Y

Thanks so much @roobooot for sharing your solution for calculating Y  :)

Thanks to @malith1992 and his brilliant solution.

Hey, thanks for the code. It is working very well, however, except the angle_Y. I made a few change so that angle_Y could be estimated by integrating gyro_Y (tested, working well ignoring the shift after long-time running).
    alpha = 0.98
    totalgyroangleY = 0
    try:
        while True:```

        ...
        ...

        totalgyroangleX = accel_angle_x + dangleX
        # totalgyroangleY = accel_angle_y + dangleY
        totalgyroangleY = accel_angle_y + dangleY + totalgyroangleY
        totalgyroangleZ = accel_angle_z + dangleZ

        #accelerometer calculation
        accel_angle_z = math.degrees(math.atan2(accel.y, accel.z))
        accel_angle_x = math.degrees(math.atan2(accel.x, math.sqrt(accel.y * accel.y + accel.z * accel.z)))
        # accel_angle_y = math.degrees(math.pi)
        accel_angle_y = 0

        #combining gyrometer and accelerometer angles
        combinedangleX = totalgyroangleX * alpha + accel_angle_x * (1-alpha)
        combinedangleZ = totalgyroangleZ * alpha + accel_angle_z * (1-alpha)
        combinedangleY = totalgyroangleY

Please look at the commented line and the amended line below it if you wanna get the angle_Y

totalgyroangleY = accel_angle_y + dangleY + totalgyroangleY makes my angle_Y grow larger over time, even if my camera doesn't move.


Optional: Generate results in Latex

Many researchers prefer the way that software supports generating results in Latex. So it is easy
to include results in research papers. If possible, please add this functionality.
A word of warning: I advise you not to use any of existing software packages for generating Latex from Python.
From my past experience, many upstreams abandoned the development of their packages after a couple of releases.

Have a good day @firefly-cpp. I was asking myself what the expected output would be like.
For this purpose I created an Overleaf document which I am sharing here:
https://www.overleaf.com/read/dhvbqthmxpfy
Would that be the expected format like? If not, how should it be?

@LaurenzBeck , @firefly-cpp , does the current NiaAML package support any other kind of output format that is easily readable, such as Markdown?

@sisco0, not really.  What about generating a diagram using the Graphviz package?


check for existing service name

added a check if the service name mapping doesn't exist
update service names as discussed @scbedd

@KarishmaGhiya Please prefix the PR title with [EngSys] or [Tooling] or something to indicate that this is related to tooling.
We are trying to follow the format where the prefix tells which service the PR is touching so that at a glance, we know which one to pick up for reviewing based on current priority

Please prefix the PR title with [EngSys] or [Tooling] or something to indicate that this is related to tooling.

Shouldn't we use labels for that?

Shouldn't we use labels for that?

Either label or prefix works. Labels are better for queryablity


Failed to load the ide-haskell-repl package

[Enter steps to reproduce:]

...
...

Atom: 1.53.0 x64
Electron: 6.1.12
OS: Microsoft Windows 10 Pro
Thrown From: ide-haskell-repl package 0.10.0
Stack Trace
Failed to load the ide-haskell-repl package
At ENOSPC: no space left on device, write

Error: ENOSPC: no space left on device, write
    at Object.writeSync (fs.js:588:3)
    at Object.writeFileSync (fs.js:1216:26)
    at Proxy.writeFileSync (<embedded>:11:269877)
    at LessCache.writeJson (<embedded>:14:1161448)
    at LessCache.putCachedCss (<embedded>:14:1162931)
    at LessCache.cssForFile (<embedded>:14:1163907)
    at LessCompileCache.cssForFile (<embedded>:14:14087)
    at ThemeManager.loadLessStylesheet (<embedded>:11:392106)
    at ThemeManager.loadStylesheet (<embedded>:11:391463)
    at <embedded>:11:3704898
    at Array.map (<anonymous>:null:null)
    at Package.loadStylesheets (<embedded>:11:3704870)
    at <embedded>:11:3697550
    at Package.measure (<embedded>:11:3696677)
    at Package.load (<embedded>:11:3697450)
    at PackageManager.loadAvailablePackage (<embedded>:11:382096)
    at PackageManager.loadPackage (<embedded>:11:380824)
    at PackageManager.activatePackage (<embedded>:11:383396)
    at <embedded>:11:916621
    at r (<embedded>:11:911227)
    at s (<embedded>:14:1134640)
    at <embedded>:14:1134728
    at Socket.<anonymous> (<embedded>:14:1133396)
    at Socket.emit (events.js:205:15)
    at Pipe.<anonymous> (net.js:586:12)

Commands

Non-Core Packages
atom-clock 0.1.18
atom-material-syntax 1.0.8
atom-material-ui 2.1.3
autocomplete-python 1.16.0
build-platformio 1.2.4
emmet 2.4.3
file-icons 2.1.46
hey-pane 1.2.0
Hydrogen 2.14.6
ide-cpp 0.1.0
ide-css 0.3.5
ide-haskell-repl 0.10.0
ide-json 0.2.1
kite 0.197.0
minimap-pigments 0.2.2
pigments 0.40.6
platformio-aia 0.0.5
script 3.29.3
scroll-through-time 0.3.1
teletype 0.13.4
zentabs 0.8.9

Literally the first line in the trace: ENOSPC: no space left on device. Free up some space on your drive, then try again.

Literally the first line in the trace: ENOSPC: no space left on device. Free up some space on your drive, then try again.


Make 5 decks into 1 deck

First of all, I wanna say thank you for making these Anki decks.
can I suggest something? maybe the 5 decks would be better if you put them together in one deck, so I can learn N4 while reviewing N5 (for example).
Thanks

I guess you could import all 5 decks in Anki and then move them all into one deck from there? I am pretty sure the tag would not be erased in the process.

As @jeromegrosse has pointed out, you could manually import into one deck.
I'll think about making a mega-deck of all 5. However, for most learners I think they'll be targeting a particular level.


Add missing padding

Only the first commit "Add missing padding" was meant to go into this PR. I am trying to figure out how to remove the other two commits from this PR.

Removed the other two commits. Now this PR contains exactly one correct commit.

thanks!

Now greentea directory is out-of sync with caffe directory, due to files duplicated in greentea directory (vgg_a.prototxt, alexnet.prototxt, etc). I can submit a PR removing those and other duplicate files, and using files in ../caffe instead, but I am not sure if @naibaf7 plans on changing the copy of the files in the future with greentea-specific things.

@ozabluda
for the close future it is safe to assume Greentea can always use the same protobuf files as original Caffe.


Make UUID a non-default feature

Currently if the UUID feature is not needed it has to be removed with default-features = false. Temporary files can be created with just a name, so it makes sense for UUID to be a non-default feature instead.

Thanks for the observation. It'll be in the next release.

Published in 0.5.0.


Optional disable outline rendering for PDF document

Description
There may be scenarios where the outline feature in typst is not required, for example, when embedding a typst document rendered in PDF into an HTML document tree.
Use Case
In my scenario, the typst exporters are compiled into individual wasm modules.
When I upgrade the typst from 0.1.0 to 0.2.0, the code size of typst::export::pdf module increased from 700KB to 7.1MB, since a complete library in global should be packaged into module.
https://github.com/typst/typst/blob/b79121d4cf8cdddb7977cbb22b627ac4706434f0/src/export/pdf/outline.rs#L14-L15

The code is related to outline feature.
A sudden increase in code size could indicate that the new feature potentially causing code coupling, which can negatively impact code quality.

But how would you even use Typst at all without the library? It can't do anything without it.

I'm doing this by first serializing the output of typst::doc::Document in backend and then deserializing it in frontend. It's been working well so far.


Feature/customer orders test

DBT Build Status 📖failure
Show DBT Build Output

Pushed by: @abhr1994, Action: pull_request


Text length classification

PR checklist:

[x] Tested by creator on localhost:8000/docs
[x] Tested by creator on refinery
[ ] Tested by reviewer on localhost:8000/docs
[x] Tested by reviewer on refinery
[x] (If added) common code tested in notebook/ script
[ ] Conforms with the agreed upon code pattern

wrong base branch?

branch names should be with -

A bit unsure about the split behavior. Curranty we would split on every whitespace and count the resulting array. Since the name is Text length classification i'd assume something different (since this-is-a-long-text-but-i-dont-know-how-to-whitespace! is short and a, b, c &  d are chars is medium)
Maybe wordcount to make it a bit more clear?


Modify update_notification_email_address keyword so that it doesn't stop test execution on errors

When we are using Update Notification Email Address keyword if keyword execution fails then it will stop whole robot execution du to sys.exit(1)  process chnaging it so that it return status
Signed-off-by: Tarun Kumar<EMAIL_ADDRESS>

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/796/console

could you test it again?
Is the failure expected?


hca-util upload progress shown on new lines

When uploading files using the command hca-util upload <path to file> the progress is printed on a new line each time rather than overwriting the same line, this appeared when I recently upgraded to 0.2.16. I was using an earlier version a couple of days ago to upload multiple files and it had a new line for each file but progress for each file stayed on one line until complete, although some uploads failed for no apparent reason.

I was able to recreate the problem on my laptop.

The problem does not happen in hca-util 0.2.15. Seems there has been a progress bar implementation change between 0.2.15 and 0.2.16.
@yusra-haider could you please look at it?


2015 07 06 triton r1

This PR implements the clustered Couchbase in Docker containers demo (in git) in Mesos.

Consul is promoted up to the level of Mesos as a shared resource. This makes discovery problems in Mesos easier to resolve.

TODO: the linking of Docker containers to Consul is janky. I'd love to see this work as cleanly as it does in Docker Compose + Triton, but it does't (today). The next best solution would be to use DNS, but that requires injecting DNS settings into the container. I'd love for Mesos + Marathon to offer better support for this. Investigation into options is needed for both cases.

The Couchbase container and a simple client app are defined as separate Marathon tasks. I'd originally defined them as an application group, but then realized that they need separate lifecycles. We don't want Mesos + Marathon to redeploy Couchbase (and lose data from the filesystem in the running containers) when we release a new version of the client, for example

Unfortunately, it's not safe to scale down Couchbase purely in Mesos + Marathon as this demo works now. The issue is that Couchbase needs to rebalance data into a smaller set of nodes before destroying excess nodes

The repo includes a start.sh script that adds prompts for everything. Get started using the instructions in the existing README.md

LGTM

The couchbase demo on Mesos script gets stuck for me. #4 However, I think I forgot to test this branch.


adding oauth to gogs

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/426/
Test PASSed.


template fails cause crash

This happens when using requiredEnv inside of a values file and the env variable is not set.
2018/08/26 08:33:34 [bug] this code path shouldn't be arrived: helmfile is expected to exit from within the `cleanup` func in main.go: template: stringTemplate:41:41: executing "stringTemplate" at <requiredEnv "HELM_AC...>: error calling requiredEnv: required env var `HELM_ACCOUNT_ID` is not set
panic: [bug] this code path shouldn't be arrived: helmfile is expected to exit from within the `cleanup` func in main.go: template: stringTemplate:41:41: executing "stringTemplate" at <requiredEnv "HELM_AC...>: error calling requiredEnv: required env var `HELM_ACCOUNT_ID` is not set

goroutine 1 [running]:
log.Panicf(0x144bde3, 0x75, 0xc42015b728, 0x1, 0x1)
	/usr/local/Cellar/go/1.10.3/libexec/src/log/log.go:333 +0xda
main.main()
	/Users/sstarcher/go/src/github.com/roboll/helmfile/main.go:388 +0x17d8

using latest master as of today


Fix nginx.conf to block recursive calls and identity endpoints

These changes reflect https://github.com/kubernetes/kubernetes/pull/51302.
We should make these changes here and get rid of the configmap in https://github.com/kubernetes/kubernetes/pull/51302, because configmaps have no good update strategy.  Specifically, if the master version changes, the configmap will change, but pods making up the daemonset won't get restarted (unless the version of the container changed as well), so they won't update their config (the configmap volume may change, but nginx won't reload it).
cc @ahmetb @destijl @mikedanese @Q-Lee.

This change is

Xref https://github.com/kubernetes/kubernetes/pull/51730


unvalidated user defined headers dangerous

After looking at #4245 I started wondering if we could set up user defined headers which would tickle our new release assert (and formerly OOM).  At least in unit test (coming soon) we can.
While hopefully there's sanity checking of user defined config before it hits Envoy, I also think we need some sane limits on user defined headers to reduce the likelihood of malicious users configuring themselves an Envoy Query of Death.
By default I think I'd like a smallish limit on number of user defined headers, length of individual header lines and maybe another one on total header size.  I'd lean towards starting restrictive and allowing configuring permissiveness as folks need it but this could break existing behavior so needs some discussion.
@envoyproxy/maintainers

@curiouserrandy @PiotrSikora (for folks looking at user defined headers, and not breaking Istio respectively.)

@alyssawilk is there more to do here?

Yes, I believe we should limit the both the number of headers we are willing to alter and the size we are willing to alter.    It's still trivial to configure a user-defined-header config which will cause Envoy crashes and better to put limits in Envoy rather than making everyone do their own.   I think we can get most of the way there by adding max_bytes and max_items validation on the proto, but there may be other expensive corner cases.  I can take those two at least

Sure sounds good. No rush.


[question] support for realsense 2 js ?

librealsense2 support
Description
after seeing support for leapmotion and kinect, i thought that there are some modern devices that could be used in interesting way in webGL, mainly the realsense series of rgbd cameras and even the experimental tracking camera t265 cause it would add instant slam for almost any device with chromium build.
As for leapmotion being in hiatus for some time and loosing ground for other companies like google with their media pipeline with headtracking and 3d handtracking, also i think oculus released api for handtracking in webxr.
Version

all of them

Platform

Web Version [X]
Windows [X]
Linux [X]

Hello
I intend at some point to remove booth leap motion and kinect support from the IDE want to move it to extensions when i have those available.
Realsense could also be added as an extesion when that happens.
Thanks a lot!

I have released a new version where booth kinect and leap have been removed. I will start working on extensions for the editor soon.
Support for realsense should be attainable as a extension later on. Thanks a lot!


Dark mode research

Summary
Research various approaches and patterns for smoothly implementing a new color-scheme into our component library that would enable dark mode support.
💬 Description
Typically when implementing dark mode, colours tokens that are used in the overall styles come in pairs (light and dark variant), e.g. light and dark action colour. When we eventually have these pairs of colours, we will need to determine the most efficient way of adding them to our component styles.
💰 Use value
Determine the ideal approach early will reduce design time for the actual implementation, and create less technical debt since we will have a solution that hopefully won't require much adjustment.
Dark mode allows greater accessibility and is eco-friendlier, should a user chose to activate.
📝 Outcomes
Items to document and consider:

[ ] A clearly defined baseline description of what dark mode means and what it means for the implementation within ICDS (e.g. will we also be implementing a pattern/means of switching dark mode on and off too?) @GCHQ-Developer-094
[ ] Variants - conversely also consider the bigger picture of the 'light', and 'default' (if necessary?), as well as dark mode
[ ] Desk research - various presentations of where it's implemented in applications and sites elsewhere
[ ] Review of existing 'light' and 'dark' themes in current ICDS components: which ones and what do they look like? Are they consistent? Good baseline for dark mode work, or do they need a re-think? @GCHQ-Developer-465 (refer to internal docs)
[ ] HOW will we implement this - will we be utilising design tokens? Will this lend itself well to future work on theming? @ry15701

Whether this is all achieved individually or not, a summary of the above will be presented back to the wider team to prompt discussion and agree the approach going forward.
Additional info:
See this original related ticket for more context (some of it is repeated in this ticket).
https://github.com/mi6/ic-ui-kit/issues/1654
Also -https://github.com/mi6/ic-ui-kit/issues/1496

Completed. Artefacts internally stored. Subsequent dark mode ticket work created.


Out Of Memory - drrun.exe

When running drrun.exe with WinAFL DLL I get this error and no log file is generated:
Application C:\Storage\WinAFL\POC\WinAFL_POC.exe (2104).  Out of memory.  Program aborted.  Source I, type 0x0000000000000001, code 0x00000000c000001c.
Need your help..

Can you show the command line you used?
Are you using the latest versions of everything (DynamoRIO, winafl)? What OS?
Does running it without drrun.exe work, or also crash?
Does your program use up a lot of memory? Did you enable PageHeap?
Lastly - is there anything interesting in the event log (eventvwr.exe) about the crash?

The command is:
C:\Storage\WinAFL\POC> ..\DynamoRIO\bin64\drrun.exe -msgbox_mask 0x0 -c ..\build64\bin\Release\winafl.dll -debug -target_module WinAFL_POC.exe -coverage_module SketchUpAPI.dll -target_method fuzz -fuzz_iterations 100 -nargs 2 -- WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg"
Yes I took the latest version of both tools.
I didn't try executing winafle.exe yet, I want to try the harness I wrote first.
The harness load simple DLL and use exported function as target (DLL is 64bit, OS is Win64), it is very simple executable so I guess it is not consuming a lot of memory..

I meant you should try running it directly without drrun OR winafl, i.e.: C:\Storage\WinAFL\POC> WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg". Does that work fine? crash? You can also try echo %errorlevel% to see if it returned a success return code. eventvwr.exe's Application tab might also provide useful info and show if there was a crash and if so - why (and possibly the offending module, etc.).
Another attempt worth trying is to run it with drrun only, i.e.: C:\Storage\WinAFL\POC> ..\DynamoRIO\bin64\drrun.exe -- WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg"
Also, not sure if this is strictly necessary, but I had some trouble in the past with passing a path in the -c param - I now run it only from the folder containing winafl.dll, and specify -c winafl.dll. This could be due to something else I got wrong or just random voodoo, but worth a shot ¯\_(ツ)_/¯

Thanks! I see an error using the Event Viewer.

I solved the error in the harness, but I still get the same failure in drrun.exe.

So this works without crash/error/failure/event log?
C:\Storage\WinAFL\POC> WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg"
And what about this:
C:\Storage\WinAFL\POC> ..\DynamoRIO\bin64\drrun.exe -- WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg"?
And the original command line shows the exact same error? Or did something change?
Also, could you please share the contents of the event log that shows the crash?

C:\Storage\WinAFL\POC> WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg" - works fine, (DLL function returns with no errors or crashes.
C:\Storage\WinAFL\POC> ..\DynamoRIO\bin64\drrun.exe -- WinAFL_POC.exe "Files\id_000082,src_000000,op_flip1,pos_179,+cov.jpg" - SAME error. (Out of memory.  Program aborted.  Source I, type 0x0000000000000001, code 0x00000000c000001c.)
And the original command line shows the exact same error? Or did something change? - SAME
Also, could you please share the contents of the event log that shows the crash? - the crash was irrelevant, bad use of the DLL exported function.

I saw this answer for the exactly same problem:
It seems that your error is due to failure to reserve memory at init time.
The code (0x00000000c000001c) is a bit weird though, but it seems to be for unreachable special heap and stack. Try using a lower init value for settings like vm_heap, stack_size and prof_pcs_heap_size (which uses special heap) at https://github.com/DynamoRIO/dynamorio/blob/89841a1463ee5348d34ed6d0b6bb296b868da630/core/optionsx.h#L1582.
It suggests to change some memory settings which I didn't see any documentation about.

Since ..\DynamoRIO\bin64\drrun.exe -- WinAFL_POC.exe doesn't work, unless you managed to fix it with the answer you found, I would advise to ask in DynamoRIO repo, because it seems the problem is reproduced without winafl.


Support snake-case and kebab-case words in strings.ToCamel et al

Is your feature request related to a problem? Please describe.
I have strings in legacy schemas that I need to convert from kebab case ("card-session-request") to TitleCase ("CardSessionRequest")
Presently:
foo: strings.ToTitle("card-session-request")
>  foo: "Card-session-request"

Describe the solution you'd like
foo: strings.ToTitle("card-session-request")
>  foo: "CardSessionRequest"

Describe alternatives you've considered
_tmp1: "card-session-request"
_tmp2: [for k in strings.Split(_tmp1, "-") { strings.ToTitle(k) } ]
foo: strings.Join(_tmp2, "")
> foo: "CardSessionRequest"

Additional context
Maybe cue-lang could add a dependency on strcase?

I asked about this a while ago (can't find the issue or discussion), and iirc, @mpvl had preference for handling conversions internally rather than adding a dependency

https://github.com/codemodus/kace might be a good option
only 500 lines, no deps, looks to be doing unicode things


Kickoff the fork from a codebase point of view

What are the next actions to take in the codebase and in what order
Right now, what I have in mind is:

upgrade tower to v0.5 and lsp-types to v0.97

add support for new LSP features (e.g. notebook support)
See https://github.com/ebkalderon/tower-lsp/pulls?q=is%3Apr+is%3Aopen

address issues from the original repository and merge PRs

What should also be done is updating the CI actions to their latest version and remove or replace the unmaintained action-rs actions.


Vertical swipe on iOS

When testing flipster on iOS (iphone5/ipad), It is difficult to scroll vertically when the screen is filled with the flipster widget.
You need to find an area not used by the flipster widget to scroll down.
Is there any way to force flipster not to capture vertical swipes?
Many Thanks

Good catch @SamHoltby! I'll make sure your fix makes it into the default behavior.

Thanks again, fixed in the v0.4 branch!

No worries!
From: Adrien Delessert<EMAIL_ADDRESS>Sent: 18 March 2015 04:58
To: drien/jquery-flipster
Cc: Sam Holtby
Subject: Re: [jquery-flipster] Vertical swipe on iOS (#31)
Thanks again, fixed in the v0.4 branch!
—
Reply to this email directly or view it on GitHubhttps://github.com/drien/jquery-flipster/issues/31#issuecomment-82739828.


Add customizable skip button color

Add property skipButtonTextColor in onboardingViewController

The skip button and page indicator were recently exposed in the header so you are able to access it directly to customize it on the OnboardingViewController itself, like:
onboardingVC.pageControl.pageIndicatorTintColor = [UIColor yourColor];
[onboardingVC.skipButton setTitleColor:[UIColor yourColor] forState:UIControlStateNormal];


What we do when entities moved to another module after persist init command

Description:
In the Persist init command, we do following

Check whether the user is already defined entities. If so, discover the module in which the user defines entities.
Check whether the user is defined entities in multiple modules. If so, give an error. if not, get the module that defined entities as the entity module.
If no entities are defined, set the default/root module as the entity module.
Generate a directory corresponding to the entity module and add database-configuration.bal with configurable variables.
If entities are defined, Generate clients inside the module directory in the generated directory.
Add/Update the Config.toml file with the config parameters.
Generate persist directory with the Persist.toml file.

If the user defines entities in a different module or moves entities to a different module after executing persist init. those scenarios are still valid,
This task is to discuss,

If entities are moved to another module after persist init, the generated configs in the Config.toml file are invalid now. Can we change it in generate and push commands? If so, we need to get consent from the user to do a re-init.
If entities are moved to another module after persist init or persist generate commands, how do we clean up already created modules and clients which are now invalid?

We need to capture the following scenarios,

The user defines entities after persist init, not in the default module, but in a different module.
The user moves the entities to a different module after generating the clients. Either after persist init, or persist generate.

For the second scenario, the user will get compilation errors from the previously generated clients. We can ask the user to remove already generated clients inside the generated directory and its usages before regenerating the new clients.
The issue is handling the database-configuration.bal file inside the generated directory. we can generate a new config bal file inside the new module and update the Config.toml file. but the previous config bal file may remain and give error at runtime.

Closing this issue. as this is not valid with the new design


Data Use Agreement  -  Update Data Set License and Data Set Policy for DCP2

https://data.humancellatlas.org/about/data-use-agreement

So I think for MVP, this is fine as is
That said we do need to look at this post MVP as we shouldn't relicense datasets we have imported from GEO or ArrayExpress with a different set of terms than come from their own service but I think that is a more complex issue than can be done in time for I think for MVP this is sufficient

@theathorn can we close this based on the comments above?

This shouldn't stop the ticket being closed but as an FYI, was in a recent conversation with people from the Ethics working group and they are drafting an open data use agreement for all HCA data which once it becomes final we should adopt. I will let you know if/when I hear more but if you can also keep an eye out that would be great

@NoopDog If we are not changing anything for the DCP/2 MVP, perhaps we should move this ticket into the backlog and use it to monitor for a new data use agreement from the HCA Ethics WG.

Cane we use this ticket to note that we have done the review and make a new one for the ethics review etc... ?

Created #782 and closing this ticket as complete.


JSON converter work

This will add the pontojson script as well as format the documentation for the repo

:+1:

:+1: I verified the rules-wordings, and I reviewed the python as well as I could.


Focus Matrix-All In One Page

Experience
Focus Matrix-All In One Page.zip
Title
Focus Matrix-All In One Page
Screenshots

Description
A compact, all-in-one tool for daily reflections, task prioritization, and personal growth, keeping everything organized on a single page.
Categories
Work, Personal Growth, Other
Language
en-US
Terms

[X] I agree that this Experience is licensed under the MIT License.
[X] I have intellectual property rights and/or permission to contribute the Experience and its content.
[X] I confirm that the Experience does not contain any harmful or malicious content.

/gallery approve


WFLY-5221 getTimers() will return all timers created in the current t…

…ransaction, even for other EJB's

Linux Build 7906 is now running using a merge of f25aa36a7b22fbda82b593af064f86eccf17a008

Windows Build 2947 is now running using a merge of f25aa36a7b22fbda82b593af064f86eccf17a008

Linux Build 7906 outcome was SUCCESS using a merge of f25aa36a7b22fbda82b593af064f86eccf17a008
Summary: Tests passed: 3056, ignored: 358 Build time: 00:49:10

Windows Build 2947 outcome was SUCCESS using a merge of f25aa36a7b22fbda82b593af064f86eccf17a008
Summary: Tests passed: 3056, ignored: 358 Build time: 00:51:06


Is the library broken?

Issue
Seems that many of the libraries functions returns null, even on the demo page:

e.g. md.userAgent()
Is a bug on the library or is a change on Chrome/Firefox that broke it?

Is it possible to extend its feature to desktop as well?

Probably not. There is not much contribution to this project and the "mother"-project written in PHP. It's kind of starving, I'm afraid.


Add DelayFromTimer for when SYST is unavailable

Fixes #169

Thanks! Would you consider adding some documentation and/or an example for this? I think it's quite difficult to discover DelayFromTimer exists, either adding some documentation at the top of delay.rs or an example would really help. We don't currently have an example for the RTIC scheduler, this could be a great excuse to add that.
This abstraction breaks down for durations longer than a second (think about what happens for delay.delay_ms(2_000)). I don't think that's stopping us merging this, but it would be an improvement to fix it. Since I wrote that snippet we now have the set_timeout method on the timers, if that method was part of a trait it could be used here. Something to experiment with if you're interested!

Ah right... I didn't look too closely at the implementation. I'll have a look at set_timeout and see where that takes me. I'll definitely add some documentation, good point.

@richardeoin I pushed a new commit and would like some quick feedback (still WIP):
I extracted some common timer configuration methods into a trait TimerConf (maybe TimerControl is better, Conf doesn't really make sense with some methods like pause()), which allows for a delay implementation using the set_timeout method. I tested with a delay of 5s and seems to work as expected.
Should I try to grab all the methods currently defined on Timer and put it in the new trait? Or should I prioritize to see if it can be generalized over LpTimer as well? I figure the trait could be more useful then.
Does it make sense to keep around an implementation based on CountDown? If the new trait can't be used for an LpTimer it might still be useful.
Also I spotted now that clear_irq() and clear_uif_bit() methods are identical, I can fix that.

Ok thanks for this. I understand the approach of putting enough functionality in the TimerConf trait implementation and then making DelayFromTimer generic over T: TimerConf.
My first comment would be that if TimerConf is part of the external API, it should abstract as much hardware-specific detail as possible. So I would go in the direction of prioritizing functionality and making it generic over LpTimer as well. However LpTimer currently doesn't have any methods for delays > 1s (there's no set_timeout method) so you would need to add that.
Another thing to note is that CountDown comes from the embedded-hal crate, and therefore is important for writing device drivers that are portable between HAL crates. I realise that in practice the DelayMs/DelayUs traits are in much more commonly used, but I think we should keep implementing as much of embedded-hal as possible - including CountDown.
Would another approach be to write DelayFromCountDownTimer so that it loops over the block! calls if t > 1s? That would allow it to use the existing CountDown<Time = Hertz> interface.

Ok thanks for this. I understand the approach of putting enough functionality in the TimerConf trait implementation and then making DelayFromTimer generic over T: TimerConf.
My first comment would be that if TimerConf is part of the external API, it should abstract as much hardware-specific detail as possible. So I would go in the direction of prioritizing functionality and making it generic over LpTimer as well. However LpTimer currently doesn't have any methods for delays > 1s (there's no set_timeout method) so you would need to add that.
Another thing to note is that CountDown comes from the embedded-hal crate, and therefore is important for writing device drivers that are portable between HAL crates. I realise that in practice the DelayMs/DelayUs traits are in much more commonly used, but I think we should keep implementing as much of embedded-hal as possible - including CountDown.
Would another approach be to write DelayFromCountDownTimer so that it loops over the block! calls if t > 1s? That would allow it to use the existing CountDown<Time = Hertz> interface.

Ok great idea about looping over block!, got it working for both Timer and LpTimer. There was one hitch though, for some configurations the maximum timeout for LpTimer is <1s, so I couldn't just loop on a 1.hz() timeout. In my case where I was testing it the clock used was 120Mhz (PCLK1 I think), and I've set the looping timeout to work with this. I'm not sure what the maximum clock configuration is though as the manual is quite dense, do you happen to know?
It would be great if the CountDownTimer trait had some way to signal the maximum supported value for the start() method.
This implementation solves the original issue, so I could remove the other implementation as this is more generic now since it supports LpTimer as well.
There might be some value to gain from a shared trait, I can see if I can clean that up a bit and what be made generic over LpTimer.

Ok great idea about looping over block!, got it working for both Timer and LpTimer. There was one hitch though, for some configurations the maximum timeout for LpTimer is <1s, so I couldn't just loop on a 1.hz() timeout. In my case where I was testing it the clock used was 120Mhz (PCLK1 I think), and I've set the looping timeout to work with this. I'm not sure what the maximum clock configuration is though as the manual is quite dense, do you happen to know?
It would be great if the CountDownTimer trait had some way to signal the maximum supported value for the start() method.
This implementation solves the original issue, so I could remove the other implementation as this is more generic now since it supports LpTimer as well.
There might be some value to gain from a shared trait, I can see if I can clean that up a bit and what be made generic over LpTimer.

Cool, nice that you got it to work! The arithmetic looks good to me, could you just add an if statement to protect against division by zero when time_left = 0?
120MHz is the maximum PCLK for most parts (the HAL has asserts here), but it is possible to clock timers at up to 480MHz depending on the part. In the case of the LPTIMs this is through the kernel clock multiplexers (eg. RM0399 Fig. 65). So maybe you can just increase that divisor to 60?

It would be great if the CountDownTimer trait had some way to signal the maximum supported value for the start() method.

Traits can have associated constants, so that would be an obvious solution. You could propose adding it to CountDown in embedded-hal. It would be a non-breaking change to embedded-hal if the trait sets a default value.

This implementation solves the original issue, so I could remove the other implementation as this is more generic now since it supports LpTimer as well.

Sure. If there's a shared trait that's useful, that would be great to have. Otherwise DelayFromCountDownTimer is also useful on its own.

Cool, nice that you got it to work! The arithmetic looks good to me, could you just add an if statement to protect against division by zero when time_left = 0?
120MHz is the maximum PCLK for most parts (the HAL has asserts here), but it is possible to clock timers at up to 480MHz depending on the part. In the case of the LPTIMs this is through the kernel clock multiplexers (eg. RM0399 Fig. 65). So maybe you can just increase that divisor to 60?

It would be great if the CountDownTimer trait had some way to signal the maximum supported value for the start() method.

Traits can have associated constants, so that would be an obvious solution. You could propose adding it to CountDown in embedded-hal. It would be a non-breaking change to embedded-hal if the trait sets a default value.

This implementation solves the original issue, so I could remove the other implementation as this is more generic now since it supports LpTimer as well.

Sure. If there's a shared trait that's useful, that would be great to have. Otherwise DelayFromCountDownTimer is also useful on its own.

I think extracting into a generic trait might warrant more careful consideration, especially given the Enabled/Disabled semantic of LpTimer, so I backtracked on that. I did leave some changes where the CountDown impl dogfoods some of the public methods on Timer though.
I have one more question though since I'm already touching these things: The pre-existing methods clear_uif_bit() and clear_irq() on Timer are identical. Should I remove one of them to avoid confusion? LpTimer also has a clear_irq() method but uses a different bit. I guess clear_irq() would make sense for a generic trait over both types, but for now maybe clear_uif_bit() is more transparent.

I think extracting into a generic trait might warrant more careful consideration, especially given the Enabled/Disabled semantic of LpTimer, so I backtracked on that. I did leave some changes where the CountDown impl dogfoods some of the public methods on Timer though.
I have one more question though since I'm already touching these things: The pre-existing methods clear_uif_bit() and clear_irq() on Timer are identical. Should I remove one of them to avoid confusion? LpTimer also has a clear_irq() method but uses a different bit. I guess clear_irq() would make sense for a generic trait over both types, but for now maybe clear_uif_bit() is more transparent.

Yes, I think it would be clearer if one of them was removed. It doesn't terribly matter which, but I would have a small preference for keeping clear_irq. UIF is the only interrupt flag that is common among all timers afaict.

Yes, I think it would be clearer if one of them was removed. It doesn't terribly matter which, but I would have a small preference for keeping clear_irq. UIF is the only interrupt flag that is common among all timers afaict.

Alright, I removed the clear_uif_bit() method. I squashed the commits into two, one with the DelayFromCountDown implementation and the other with the Timer refactorings. If you think it looks ok I'd be ready for merging.

Alright, I removed the clear_uif_bit() method. I squashed the commits into two, one with the DelayFromCountDown implementation and the other with the Timer refactorings. If you think it looks ok I'd be ready for merging.

Hm, there's still the question of documentation for purpose of discoverability, I can see if I can add a short example at the top of delay.rs

Hm, there's still the question of documentation for purpose of discoverability, I can see if I can add a short example at the top of delay.rs

Ok! Thanks for the example too
bors r+

Ok! Thanks for the example too
bors r+


Quantile Remapping Fix

This PR includes the following:

Update to DDSketch 0.1.3 which fixes an issue mapping quantiles to rank.
Updates Summary to return the min and max values when a request for the 0.0 or 1.0 quantile is made. This is convenient as described in 5274708.
Resolves an issue with quantile mapping in Summary::quantile(), where the Summary could previously return the value that belongs to the adjacent rank. Also resolves incorrect assertions in a test, that now pass with the changes to quantile mapping.

Released in<EMAIL_ADDRESS>


Add insert-bb tool to utils

This is a little script I wrote to help with SIL diffing. It prints a modified version of its SIL input which adds a new basic block containing only “placeholder”—but more importantly, it updates all references to existing higher-numbered basic blocks. This can help to reduce the amount of noise in SIL diffs without removing any information from the file.

Can you use python instead of perl? I don't think we have any other perl scripts in the codebase.

We do have one in utils/, but point taken. I'll rewrite it when I have a little time.

Thanks! What is the one in utils? (If I get bored, I may just fix that).

or I guess I can look myself = p. Thanks!

@brentdax Is this something you still want to pursue, or should I turn it into an SR and let somebody else take it?

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


[Addition]: Kerlink cluster is cheating all the time

Hotspot b58 Addresses
11aeEYRcCyqHyRQJNpLGmbb8uYMiqBjiHcFMbkjdtkU77Wxg7hL
112LdgL9HdaZBx3HKpCX1WcY76LEJAK6HVC7YT8qdp8uvXh5Q4Vp
Discord Handle
No response
Reason(s)

This is a cluster of many hotspots witnessing each other only.
Hotspots only witnessing each other
I am living in this area, there is no possibility to install hotspots there

@loidef Please stop cheating... you don't live in chyprius, in the US and in asia at the same time... (cf : https://github.com/helium/denylist/issues?q=+author%3A"loidef"+living ).


on trigger watchdog run call triggerwatchdog

This make shure that the correct code is run on trigger from outside

Done.

Close