Adding SIMD to dot product intrinsics sample

Description
Added a SIMD dot product function to the intrinsics sample as per github issue: https://github.com/oneapi-src/oneAPI-samples/issues/63
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Command Line

@pmpeter1  Requesting your review as this is linked to an external request.

Should follow the parts of the coding guidelines that are applicable to OpenMP.

Ethan fixed the sample, by using clang


Add Logout button to the menu bar

I added to the frontend the following changes:

added logout button to the menu bar
added some text to the login page

Preview:
NOTE: normal button

NOTE: When hovering over the button

I added the email next to the Logout link.
Preview:


Relocate ROS publisher creation in vectornav node

Rationale
The vectornav node sets up ROS publishers and subscribers before it has successfully connected to the device, which may involve several attempts. As a result, we often get spurious publisher staling faults on bringup.
We have to defer publisher creation till after we've managed to connect.
Definition of Done

[x] Rearrange vectornav's node run function to avoid these faults.
[x] Bump pose estimator staling checks to cope with these setup delays.
[x] Validate all changes manually on a vehicle.

Fixed in #139


[hi] Localize 'API Gateway' to Hindi

Describe your changes
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[ ] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[ ] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

I've made the changes you told @abhay-raj19. Will you please check and tell me if is there anything I should do?

@AniketNS Please change the target branch of this PR to dev-hi, not main.

@AniketNS Please change the target branch of this PR to dev-hi, not main.

+1

@AniketNS Its always a good practise to create your local branch significant
you should signoff your commit as well to avoid DCO failure! use git commit -s -m "your message here" in this way

Hey @aj11anuj and @kumarankit999, I don't How this new PR got created. As I'm a beginner, I don't actually how it is happening. Really sorry for that.

No Problem @AniketNS, you can work on that PR Only!

Approvers, please check for duplication before starting review process.
And contributors are also advised to check if there's already an open PR or not, before making a new PR
Closing this one as it's a duplicate


Use async/await, remove some manual serde-impls

Are you interested in these changes? I think there are still a few spots in the code-gen where I'm missing the async/await annotations. I'll have to fix those.

Thanks!
Only had a quick skim though, will have a good check in a few days.
First off though is there any chance we can have both blocking and async options like request? Does it seems not useful?

You certainly could have both. We would have to duplicate the main trait that you implement.
One thing I meant to ask, what's the purpose of the string-concatenation code gen stuff that's in the tests?

@leshow I'm interested in pushing this over the finish line, is there anything left to be done and if so what's left?

It depends what you want to do with it. If you don't care about maintaining sync compatibility then it currently works.
I don't understand the approach taken for code generation in this library, so there may be some stuff left to do there also.

sorry for letting this slide. things have been 'different' lately! I've put aside some time to try and port my existing apps to this version and see how that works out.
One thing i note is the message subtype deserialising used to fall back to Message::Standard, but it wouldn't now. Would need to investigate if slack can send messages without subtype

after further investigation i don't think that change should go in, also it wasn't in the generator so let's make that separate from async changes

thanks @leshow

No problem!


US BLS Jolts data refresh

PR request is created for US BLS Jolts .
The existing dcid's are retained by modifying the script.
If any new SV introduces in future , that should be added to the dictionaries that are in map_config.py & mcf_config.py .

Instead of creating a separate sub directory for updated files, can you modify the original set of files. Also include unittest for this.

Creating a new PR as the old one was pushed incorrectly.


fix quiver length bug in trplot func

When the translation component in the homogeneous transformation matrix is not all zero, setting the quiver length (defined in trplot) to a value other than 1 will output an incorrect plot.

When the translation component in the homogeneous transformation matrix is not all zero, setting the quiver length (defined in trplot) to a value other than 1 will output an incorrect plot.

The PR allows setting the length of the quivers.  Can you explain more the comment above, it doesn't seem to be related.

Thank you for your reply. I have a problem using the trplot function. For example, if I use the bellow code to visualize a transform matrix:
T = transl(1, 2, 3) @ trotx(30, 'deg')
trplot(T, dims=[-5, 5], length=3)
plt.show()
then the three axes of the visulized coordinate frame will not be orthogonal. This does not happen if the 'T' contains only the rotation part, or if the value of 'length' is set to 1. I think the problem is here:
https://github.com/petercorke/spatialmath-python/blob/471790437dd76394359aa51a87f73467b0cbf33c/spatialmath/base/transforms3d.py#L1471-L1475
https://github.com/petercorke/spatialmath-python/blob/471790437dd76394359aa51a87f73467b0cbf33c/spatialmath/base/transforms3d.py#L1484-L1486
I think the correct calculation should be:
o = T @ np.array([0, 0, 0, 1])
x = T @ np.array([1, 0, 0, 1])
...
ax.quiver(o[0], o[1], o[2], (x[0] - o[0])*length, (x[1] - o[1])*length, (x[2] - o[2])*length, arrow_length_ratio=wtl, linewidth=width, facecolor=color, edgecolor=color)
...
or just use the 'length' parameter defined in 'quiver' function of 'pyplot' module:
o = T @ np.array([0, 0, 0, 1])
x = T @ np.array([1, 0, 0, 1])
...
ax.quiver(o[0], o[1], o[2], x[0] - o[0], x[1] - o[1], x[2] - o[2], length=length, arrow_length_ratio=wtl, linewidth=width,               facecolor=color, edgecolor=color)
When the code is modified to one of the above two ways, the axes of the coordinate frame will be orthogonal to each other. Looking forward to your advice.

Hello everybody,
and first of all thank you for the wonderful toolbox. It will make our life way easier!
Currently I noticed the same issue with the length argument of the function trplot.
The issue arises whenever the passed pose T in SE3 is not in the origin @ [0;0;0] and the length != 1.
The reason is that the length is not just defining the length of the unit-vector in the x,y,z-direction, but also the scaling the whole Pose (it is rather a SIM3 Matrix than a homogeneous matrix).
The fix is rather simple and works both for  arguments rviz=True and if arrow==True.
Just these changes need to be applied in the file spatialmath/base/transforms3d.py @ line 1772
    # create unit vectors in homogeneous form
    o = T @ np.array([0, 0, 0, 1])
    x = T @ np.array([length, 0, 0, 1])
    y = T @ np.array([0, length, 0, 1])
    z = T @ np.array([0, 0, length, 1])

Tested it on a trajectory (both rviz=True and if arrow==True) wuth a length==0.5 and width==0.1:

I will provide a commit with the modification and a pull request.
Best wishes,
Roland

Hello everybody,
and first of all thank you for the wonderful toolbox. It will make our life way easier!
Currently I noticed the same issue with the length argument of the function trplot.
The issue arises whenever the passed pose T in SE3 is not in the origin @ [0;0;0] and the length != 1.
The reason is that the length is not just defining the length of the unit-vector in the x,y,z-direction, but also the scaling the whole Pose (it is rather a SIM3 Matrix than a homogeneous matrix).
The fix is rather simple and works both for  arguments rviz=True and if arrow==True.
Just these changes need to be applied in the file spatialmath/base/transforms3d.py @ line 1772
    # create unit vectors in homogeneous form
    o = T @ np.array([0, 0, 0, 1])
    x = T @ np.array([length, 0, 0, 1])
    y = T @ np.array([0, length, 0, 1])
    z = T @ np.array([0, 0, length, 1])

Tested it on a trajectory (both rviz=True and if arrow==True) wuth a length==0.5 and width==0.1:

I will provide a commit with the modification and a pull request.
Best wishes,
Roland

Hi Roland,
Thanks for your interest and I look forward to processing a PR.
They are not the most beautiful axes in the world but I’m not sure we can do much better using Matplotlib.
Peter

On 29 Jan 2021, at 4:35 am, JungR<EMAIL_ADDRESS>wrote:
Hello everybody,
and first of all thank you for the wonderful toolbox. It will make our life way easier!
Currently I noticed the same issue with the length argument of the function trplot.
The issue arises whenever the passed pose T in SE3 is not in the origin @ [0;0;0] and the length != 1.
The reason is that the length is not just defining the length of the unit-vector in the x,y,z-direction, but also the scaling the whole Pose (it is rather a SIM3 Matrix than a homogeneous matrix).
The fix is rather simple and works both for arguments rviz=True and if arrow==True.
Just these changes need to be applied in the file spatialmath/base/transforms3d.py @ line 1772
# create unit vectors in homogeneous form
o = T @ np.array([0, 0, 0, 1])
x = T @ np.array([length, 0, 0, 1])
y = T @ np.array([0, length, 0, 1])
z = T @ np.array([0, 0, length, 1])

Tested it on a trajectory (both rviz=True and if arrow==True) wuth a length==0.5 and width==0.1:
https://user-images.githubusercontent.com/25152516/106181848-5ee93180-619e-11eb-9f72-8796b8894656.png
https://user-images.githubusercontent.com/25152516/106182943-dc617180-619f-11eb-80be-c30365717e91.png
I will provide a commit with the modification and a pull request.
Best wishes,
Roland
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/petercorke/spatialmath-python/pull/2#issuecomment-769289584, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2BIUQRHGXRGWV3BCLO6UTS4GU7NANCNFSM4OQJAXBA.

Hi Roland,
Thanks for your interest and I look forward to processing a PR.
They are not the most beautiful axes in the world but I’m not sure we can do much better using Matplotlib.
Peter

On 29 Jan 2021, at 4:35 am, JungR<EMAIL_ADDRESS>wrote:
Hello everybody,
and first of all thank you for the wonderful toolbox. It will make our life way easier!
Currently I noticed the same issue with the length argument of the function trplot.
The issue arises whenever the passed pose T in SE3 is not in the origin @ [0;0;0] and the length != 1.
The reason is that the length is not just defining the length of the unit-vector in the x,y,z-direction, but also the scaling the whole Pose (it is rather a SIM3 Matrix than a homogeneous matrix).
The fix is rather simple and works both for arguments rviz=True and if arrow==True.
Just these changes need to be applied in the file spatialmath/base/transforms3d.py @ line 1772
# create unit vectors in homogeneous form
o = T @ np.array([0, 0, 0, 1])
x = T @ np.array([length, 0, 0, 1])
y = T @ np.array([0, length, 0, 1])
z = T @ np.array([0, 0, length, 1])

Tested it on a trajectory (both rviz=True and if arrow==True) wuth a length==0.5 and width==0.1:
https://user-images.githubusercontent.com/25152516/106181848-5ee93180-619e-11eb-9f72-8796b8894656.png
https://user-images.githubusercontent.com/25152516/106182943-dc617180-619f-11eb-80be-c30365717e91.png
I will provide a commit with the modification and a pull request.
Best wishes,
Roland
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/petercorke/spatialmath-python/pull/2#issuecomment-769289584, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2BIUQRHGXRGWV3BCLO6UTS4GU7NANCNFSM4OQJAXBA.

Dear @petercorke,
the pleasure is on my side and I am honored to contribute. The PR can be found here.
Regarding the axes style: I like that simple style and making them even look better comes just at the cost of lower plot refresh rates in live simulations.
One thing that could be changed is that the reference frame name also appears in the the axis description.
For instance, if one wants to display the reference frame and the frame counter or current timestamp, the 3D plot will drown in text. But I think this might be discussed in an issue as an feature request/suggestion.
Best wishes,
Roland

Dear @petercorke,
the pleasure is on my side and I am honored to contribute. The PR can be found here.
Regarding the axes style: I like that simple style and making them even look better comes just at the cost of lower plot refresh rates in live simulations.
One thing that could be changed is that the reference frame name also appears in the the axis description.
For instance, if one wants to display the reference frame and the frame counter or current timestamp, the 3D plot will drown in text. But I think this might be discussed in an issue as an feature request/suggestion.
Best wishes,
Roland


Modified the recipes/Unit_Tests_Generation/Unit_Tests_Generation.ipyn…

…b code to use the Granite 3.1 8b model
PR Checklist
Model Interaction

[ ] Flexible LLM platform support The platform should be easily switchable. Use LangChain or LlamaIndex.
[ ] Use prompt guide corresponding to the model  For example for Granite 3.x Language Models

Data

[ ] Example data: Follow the example data guidance.

Notebook requirements

[ ] Notebook outputs cleared: Ensure all notebook outputs are cleared.
[ ] Automated testing: Add the recipe to the automated tests as described here
[ ] Test in Google Colab:

[ ] Test that it works in Google Colab (Python 3.10.12).
[ ] Colab has its own package set and Python version, so ensure compatibility.

[ ] Test locally:

[ ] Ensure the code works in a fresh Python virtual environment (venv).

[ ] Standard access to secrets and variables Include !pip install git+https://github.com/ibm-granite-community/utils in the first code cell in order to make get_env_var available to accessing secrets and variables in the recipe.

Incoming References

[ ] README.md updates:

[ ] Add a link to the recipe in the Table of Contents (ToC).
[ ] Include a Colab button after that link.

GitHub

[ ] Commits signed: All commits must be GPG or SSH signed.
[ ] DCO Compliance: Developer Certificate of Origin (DCO) applies to the code, documentation, and any example data provided. Ensure commits are signed off.

Is it still the policy to clear the cell outputs?


Unable to run a Validation node

SUMMARY

ISSUE TYPE

Bug Report

COMPONENT NAME

OASIS NODE VERSION

21.1.1

OS / ENVIRONMENT
UBUNTU 18.4

STEPS TO REPRODUCE

error when starting a node

oasis-node --config /serverdir/etc/config.yml

ACTUAL RESULTS

"incompatible: peer is on a different network.", after that "Couldn't connect to any seeds"

The process retry, but same results

EXPECTED RESULTS

I expected the connection to the Network

Are you using the correct genesis file? Are you connecting to the Mainnet or Testnet? You need to provide some more log entries from oasis-node.

I think the genesis file is ok, connecting to mainnet
File internal.sock is empty, please indicate log files to attach

After investigating the issue it seems that the seed I am using may be inactive, is it possible to have additional peers/seeds?


Use scoped slots defined by components (other than default)

Describe the feature
At the moment it's not possible to use custom slots inside of a FormKitSchema. Here is only the "default" slot supported.
It would be great when the support will be added because it's also possible with the usage in the template, so i think this should be aligned.

This is definitely something we need to do. Do you have a suggested API for the schema node?
{
  $cmp: 'MyComponent',
  children: [
    // -- this is the default slot already --
  ],
  slots: {
    myNamedSlot: [
      {
        $el: 'h1',
        children: 'hello world'
      }
    ]
  }
}

Something like that perhaps?

Yes, this looks good from my side.

@AustinGrey yeah I think something like that is about right. basically we just need a place where we can define what "variable" receives the scope data. We already do this (variable "assignment") in one other place and thats the for loop:
{
  $el: 'li',
  for: ['item', '$items']
}

Which is intended to read (in the developers mind) "for item in items" or at least as  best as possible in JSON format that is :)
Another API proposed by @GustavoFenilli is:

```js
{
  $cmp: 'MyComponent',
  children: [
    {
      $slot: 'scopedSlotName',
      children: ['Hello world']
    }
  ]

We could pretty easily add scoped data to that as a top level key — although I do think it is a bit clearer to have "slots" as a top level feature of components nodes as in the first API. However, I'm not totally sold on using the semicolon scopedSlot:scopedData — would be nice if the syntax was a bit closer to the assignment in for for loop.

I like the idea of passing possible scoped variables as items inside the array and appending the slot name to it so it is indeed scoped and easy to find.
Having the slots at top level wouldn't matter for vue, as slots are passed to the dom correctly using the <slot> inside the component, but what about React/Svelte and others that do not have this feature, what order and where would you show thoses slots? if its inside children, we know the order and where.
Slots are basically in concept just a function with scoped variables that renders to the dom, where its rendered is the trick part, but we could also just the top level named slots defined to render inside the children just like $slots.default.
{
  $cmp: 'MyComponent',
  slots: [
    {
      name: 'userDefined',
      scope: ['var1', 'var2'],
      schema: []
    }
  ],
  children: [
    '$slots.userDefined'
  ]


Bulkloading

Added page on bulk loading.

Alex,
Thanks for suggestions.
Updated PR to add using symlinks as an alternative, and fix the nodetool import syntax.


Allow_customizing_DocService

Hi I'd like to solve this issue #2185 , I made some changes but I don't know if what I'm doing is the right choice, could you help me?

I think I need to think briefly about this.
If DocServiceBuilder exists to be injected, use it as is, otherwise, create a new instance.
WDYT?

I think I need to think briefly about this.
If DocServiceBuilder exists to be injected, use it as is, otherwise, create a new instance.
WDYT?

Yes that was my first intention now I understand the Optional, it's very interesting.
Then I checked the ArmeriaServerConfigurator which extends Ordered and I copied it for the DocServiceBuilder.

If DocServiceBuilder exists to be injected, use it as is, otherwise, create a new instance.

I'm not sure what's the advantage of this. Any specific use case that's impossible with DocServiceCustomizer?

I agree to create a class like DocServiceCustomizer. This will give you the flexibility to deal with another problem later.

Hi I made some changes but I didn't know how to add the spring dependencies on this module, could you tell me please?

Hi I made some changes but I didn't know how to add the spring dependencies on this module, could you tell me please?

I think core doesn't need the spring dependencies.
How about moving to boot-autoconfigure/com.linecorp.armeria.spring?

Thank you, @Charliocat and reviewers!

Thank you all!


Solved 0881-boats-to-save-people in c++

File(s) Modified: 1-Two-Sum.py, 2-Add-Two-Numbers.py, etc...
Language(s) Used: python, javascript, etc...
Submission URL: _https://leetcode.com/problems/[problem-name]/submissions/xxxxxxxxx/_

This is my first pull request and I will appreciate any feedback.
Thanks in anticipation.

Thank you sm :)


Add threadLocal reusable byte array for VarByteChunkReaderV4

Intuition: Reduce repeated intermediate byte array allocation on jvm heap when QPS is high. Without this change, there will be an additional allocation of intermediate byte array in ReaderContext per record read. When qps is relatively, the YoungGC are likely to be triggered frequently. With this change, no allocation of intermediate byte array if the entry size is than the configurable _maxReusableByteSize.
Is it thread safe? Yes, since the ReaderContext is created per ProjectionOperator per segment per query. A non combine operator per segment would only be executed by a query worker.
Additional Permanent Memory cost: number of query workers *  _maxReusableByteSize (should be set as a fieldConfig per table). In the future, it will be number of query workers *  min(_longestLengthOfEntry, _maxReusableByteSize) if we can add _longestLengthOfEntry in the V5 segment header.
Potential proof demonstrating the benefits of reusable byte array on V2 / V3: https://github.com/apache/pinot/pull/12242#issuecomment-1932695513
This PR is rebased on top of https://github.com/apache/pinot/pull/12978

Codecov Report
Attention: Patch coverage is 0% with 72 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (bf5e1fc).
Report is 345 commits behind head on master.

Files
Patch %
Lines

...ders/forward/VarByteChunkForwardIndexReaderV4.java
0.00%
72 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12977       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2427        -9
  Lines        133233   132897      -336
  Branches      20636    20581       -55
=============================================
- Hits          82274        0    -82274
- Misses        44911   132897    +87986
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Have you found any issues from these allocations? In our experience w/ V4 we have not seen GC issues even when using lower throughput GC algorithms (ZGC/generational ZGC).
I remember there was a discussion/choice to avoid reusing the bytes previously: https://github.com/apache/pinot/pull/7661#discussion_r745500379

Have you found any issues from these allocations? In our experience w/ V4 we have not seen GC issues even when using lower throughput GC algorithms (ZGC/generational ZGC).
I remember there was a discussion/choice to avoid reusing the bytes previously: #7661 (comment)

I am concerning the scenarios of high qps against large payload column. for small byte[], we can set the _maxResuableByteSize to 0 in the table config so that we can leverage modern ZGC.


Fix mybinder

Issue: This fixes the mybinder integration. It was broken from  952d15d, because adding a Dockerfile on the top directory, causes mybinder to build the environment from it, resulting in this case in a tensorflow environment missing the covid19 notebooks.
Compare: https://mybinder.readthedocs.io/en/latest/config_files.html
Solution: The Dockerfile and related scripts are simply moved to the new 'docker' directory.
In the README.md, the link to mybinder is updated so that it starts showing the directory content.

Thanks @SchuhMichael!


Make bash autocomplete instructions easier to find

See original issue on GitLab
In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 11:17
Background
Buildstream autocompletion instructions are quite deep within the install documentation.
Task description

[ ] Make instructions easier to find either by moving or linking to from elsewhere.

Acceptance Criteria
Autocomplete instructions should be easy to find for newcomers to the project.

In GitLab by [Gitlab user @tristanvb] on Aug 6, 2018, 11:21
It appears that the install guide has evolved in such a way that the bash completion steps are hard to find.
Would you like to repurpose this issue into a documentation issue ?

In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 11:39
changed title from {-Terminal autocompletion for buildstream-} to {+Make bash autocomplete instructions easier to find+}

In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 11:39
changed the description

In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 11:40
Regarding the lack of autocompletion support for other shells, should these be raised as separate tickets?

[ ] Fish
[ ] Zsh

In GitLab by [Gitlab user @tristanvb] on Aug 6, 2018, 11:43

Regarding the lack of autocompletion support for other shells, should these be raised as separate tickets?

Yes they should (this will let us treat them separately and close the appropriate issue when one of them is implemented). Please mark them as related.

In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 13:23
marked this issue as related to #558

In GitLab by [Gitlab user @Qinusty] on Aug 6, 2018, 13:23
marked this issue as related to #557

In GitLab by [Gitlab user @jennis] on Aug 9, 2018, 09:57
[Gitlab user @Qinusty], I agree, I think we need to move the 'BASH completions' section so that it is no longer living under the "Installing from source" section.
What did you have in mind?
I would like to contribute my own thoughts on this. I think we need only make a minor modification to the install_linux_distro.rst file to have the following sections:
Installing BuildStream on a Linux distro
========================================

Installing from source (recommended)
------------------------------------

Installing from distro packages
-------------------------------

Enabling auto-completion
------------------------

Thoughts?


replace the default conhost with new terminal

we should build newhost to forward execute

build and copy to system32 dir.

when the console program starts. the condrv.sys will create conhost process and modify the process command line.

we should modify the path to our newhost.

build the driver and load.

set HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\Windows\ErrorMode to val 2. ignore the error windows diag.

end

Thank you. We are working on an official solution to this in /dup #492.

I do not recommend patching OS files, especially the console driver, to accomplish this.


File mode passed to uhyve is given in hex instead of octal.

The mode in the open hypercall is passed in hex, but it should be passed as octal. This results in strange permissions of files which are created by uhyve.
https://github.com/hermitcore/libhermit-rs/blob/44eaa9f693566413884f505552f74f2e25863e78/src/syscalls/interfaces/mod.rs#L58-L69
Example: 1991 = 0x777 is passed but it should be 511 = 0o777 instead. The flags field  is passed correctly.
@tlambertz pointed out that this has to be fixed in the stdlib

Fixed by https://github.com/rust-lang/rust/pull/109368.


Silent Aim issues/improvement idea

Idk what is directly causing it, but silent aim either rarely works, or it auto-fires.
Should have options of whether you want it to fire automatically or not.
One option for Silent Aim, another to tick w/ auto fire
Silent aim should work just by spraying and it hitting whatever is in the FOV (it does not do it currently, unless I am missing an option).
The biggest bug is that I get hit with my own bullets when using Silent Aim. I don't know if it's caused by a ricochet, or what; but my wallshoot is turned off so ricochet should be off.

It works fine on my side. We often saw users reporting "I get hit with my own bullets" using extra mods doing incompatible stuff. Is it the case on your configuration ?

We often saw users reporting "I get hit with my own bullets" using extra mods doing incompatible stuff. Is it the case on your configuration ?

I"ve had that issue whilst turning out the FOV for aimbot to an absurdly high number

I"ve had that issue whilst turning out the FOV for aimbot to an absurdly high number

ah interesting

Just thought of that, but you just need to tweak SilentAimNextShotDelay, the lower it is the faster the aimboit will re-shoot.


Error installing module in windows PC

hey,
First of all very nice work done.
but I have problem installing via npm.(tried on with bower and loved your work)
though I reported problem in npm and npm-windows-update
Error is something like
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\alfanso\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "perspective-transform"
npm ERR! node v4.2.3
npm ERR! npm  v3.5.2
npm ERR! code ENOSELF
npm ERR! Refusing to install perspective-transform as a dependency of itself
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     E:\node\perspective-transform\npm-debug.log

Sorry you're having issues installing with npm. By any chance is "perspective-transform" the name of your module in your package.json? Take a look at a solution to this issue on SO:  http://stackoverflow.com/questions/27267707/npm-warn-install-refusing-to-install-hapi-as-a-dependency-of-itself


feat: Allow non-unix newlines in XML input and output

In this PR I added support for non-unix newlines in XML input and output.
Example for input
prettify-xml now accepts non-unix-newlines in input string:
import prettify from 'prettify-xml';
const input = '
\r\n
foo
\r\n
bar
\r\n
'

const expectedOutput = [
  '
',
  '
foo
',
  '
bar
',
  '
',
].join('\n')

const options = {indent: 2}
const output = prettifyXml(input, options)

assert(output === expectedOutput)

Example for output
Output newlines can be specified by passing a newline option:
import prettify from 'prettify-xml';
const input = '

foo

bar

'

const expectedOutput = [
  '
',
  '
foo
',
  '
bar
',
  '
',
].join('\r\n')

const options = {indent: 2, newline: '\r\n'}
const output = prettifyXml(input, options)

assert(output === expectedOutput)

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is 100%.

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


chore(main): release 1.1.1

:robot: I have created a release beep boop
1.1.1 (2022-11-22)
Bug Fixes

Specify readonly attributes (#158) (0347f83)
Update content example keys to match accept & content-type header (#155) (1521bee)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/whispir/openapi/releases/tag/v1.1.1 :sunflower:


Fix MarkLogic array concatenation and enable all MapFunc tests

Fixes array concat to work with strings and arrays in MarkLogic. Also enables all the StdLibSpecs for MarkLogic and adds a few new ones exercising ArrayConcat.
In the process, also relaxed various array operations to work on any XML element by not requiring child elements to have a particular name.
Fixes #1503.

Current coverage is 63.67% (diff: 9.52%)

Merging #1702 into master will decrease coverage by 0.19%

@@             master      #1702   diff @@
==========================================
  Files           350        351     +1
  Lines         16592      16640    +48
  Methods       16310      16358    +48
  Messages          0          0
  Branches        142        142
==========================================
- Hits          10596      10595     -1
- Misses         5996       6045    +49
  Partials          0          0

Powered by Codecov. Last update c8f8ef5...760bbe7

@alissapajer I'd forgotten about the typing + constant folding for ConcatOp, I've remedied that and its ready for another look.

LTGM - merging.


xTASKSTATUS Runtime counter type (IDFGH-10757)

Is your feature request related to a problem?
FREERTOS task runtime counter overflow.
Describe the solution you'd like.
configRUN_TIME_COUNTER_TYPE ulRunTimeCounter; to xTASK_STATUS in task.h
Describe alternatives you've considered.
tracking overflow in expense of RAM
Additional context.
No response

Hi @EnariSaarBolt
I seems like the changes you are requesting is already a part of FreeRTOS v10.5.1, which we are in-progress of updating to.
I'll link to this issue so that we finish the update you'll get a notification here.

4d85cbff6d30cac120e00d53aa1984e026f64a09


Fixed typo in comment

Not really a typo
https://en.wikipedia.org/wiki/Phablet

The phablet is a class of mobile devices combining or straddling the size format of smartphones and tablets. The word itself is a portmanteau of the words phone and tablet.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/30881

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/42565

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/43251

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/51669

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/61921

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108668


Added some missing locals, removed redundant initializer, moved one local closer to its usage

Just some small tweaks I did when I read the source. Nothing to write home about :-).

Thanks!


Detection macros for libcpp

False-positive warning generated for _LIBCPP_VERSION == 10000
boost-1_68_0/boost/hana/config.hpp:99:9: warning: "Versions of libc++ prior to the one shipped with Clang 3.5.0 are not supported by Hana." [-W#warnings]
#       warning "Versions of libc++ prior to the one shipped with Clang 3.5.0 are not supported by Hana."
        ^

I believe this should have been fixed by https://github.com/boostorg/hana/commit/8f935a710b5c30544da5c89f65bed6f0b7b7e8cf. Can you please confirm whether you're still able to reproduce the problem with Boost 1.71.0?

I just ran into this. The fix shows up in 1.72.
Confirmed here: https://godbolt.org/z/9QCn-d
Thank you for the report and the fix.
I'll go ahead and close this.


mailsend is deprecated, add mailsend-go

Announcement on mailsend on 2/11/2019:

If you use mailsend, please try mailsend-go. Your feedback will be appreciated.

Consider adding support for: https://github.com/muquit/mailsend-go

Awesome, thank you @niheaven 👍


New extension (Jazzmidi)

Closes https://github.com/TurboWarp/extensions/issues/105

I reformatted this extension to match the guidelines -- can you see if it still works? I do not have the necessary hardware to test it myself

you don't need hardware, you just need some software and an extension for your browser, download the software from jazzmidi
[https://jazz-soft.net/]

What is jazzmidi

What is jazzmidi

Jazz midi is a module which allows you to use midi to play notes on your computer and midi devices

closed due to inactively, FluidSynth will be used


Rack 2.0 support

Keeping the changes in the project as small as possible.  This PR is not going to be merged, instead this is going to become the mainline branch for Invoca which we will continue to use for releasing private updates to this gem as needed until we phase out the need for its use.

Closing since this version has been released


Add list images in CLI

Going to put this PR into https://github.com/rancher/rio/pull/669 for sake of testing, closing for now.


Setup Intune + Autopilot two articles, unclear how to do both?

There appear to be multiple sections for endpoint management articles and I'm starting to wonder if we have intune and autopilot setup correctly.
This article talks about just enrolling devices into intune using GPO
Docs > Windows > Client management > Mobile Device Management has this article
https://docs.microsoft.com/en-us/windows/client-management/mdm/enroll-a-windows-10-device-automatically-using-group-policy
And then we have this article that talks about enrolling in autopilot using GPO, but not in intune...
Docs > Enterprise Mobility + Security > Microsoft Endpoint Manager > Autopilot
https://docs.microsoft.com/en-us/mem/autopilot/windows-autopilot-hybrid#create-and-assign-an-autopilot-deployment-profile
We are trying to enroll existing devices into intune, and also enroll them in autopilot moving forward. However, the instructions in these two articles are not exactly similar...
We opened service ticket 23906196, to troubleshoot Prt issues and are still trying to understand the best way to configure things correctly.

Hello @jonwbstr and thank you for raising this question. Is your feedback primarily for this particular article: https://docs.microsoft.com/en-us/windows/client-management/mdm/enroll-a-windows-10-device-automatically-using-group-policy

there are two areas in docs for intune articles. That's confusing.
there are 3 scenarios. Intune, autopilot, both. I found articles for 2
of 3 scenarios. Would be nice to have the 3rd scenario covered somewhere.

The feedback is not on the article specifically, it's that the article
doesn't acknowledge there are 3 scenarios, or have a link to the other
article which is related to deploying intune via autopilot.
P.s. I'm honestly not even sure why someone would enroll devices in
autopilot via gpo. Once enrolled they have to be wiped again. Just weird.
On Wed, Feb 3, 2021, 11:28 AM Denise Vangel-MSFT<EMAIL_ADDRESS>wrote:

Hello @jonwbstr https://github.com/jonwbstr and thank you for raising
this question. Is your feedback primarily for this particular article:
https://docs.microsoft.com/en-us/windows/client-management/mdm/enroll-a-windows-10-device-automatically-using-group-policy
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9067#issuecomment-772684166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXTPAECHVFW5FVHRO5ZTTDS5GBVTANCNFSM4W7QZN2A
.

I realized I may have been mixing up my terminology. Microsoft Endpoint Manager Admin Center encompasses both Intune and Autopilot. Autopilot and Intune seem to be joined at the hip. The current layout for documentation having intune and autopilot in completely separate areas is confusing given the admin center is trying to put them under the same umbrealla.
Feel free to close this issue whenever, I don't really expect any changes to happen, just wanted to pass on feedback on how the documents are organized incase it hasn't been brought up by someone already.

@jonwbstr Thank you for coming up with further clarifications about the nature of the issue. Closing based on your suggestion.


prep for release

relax checkstyle a bit
remove -SNAPSHOT
upgrade truth
fix metadata

@tswast FYI
@jabubake FYI


refactored auth code

moved schema validation out of verifyAuth() into message parse()
separated verifyAuth() into authenticate() and authorize()
differentiated between 400 and 401 errors
improved CollectionsQuery authorization logic on recipient filter
removed not-so-useful interfaces and improved auth code coverage
various trivial renames and test code refactoring

@csuwildcat, this code is now merged. Please pay attention to the CollectionsQuery authorization code added:
https://github.com/TBD54566975/dwn-sdk-js/blob/main/src/interfaces/collections/messages/collections-query.ts#L68


[NSE-237] Add ARROW_CSV=ON to default C++ build commands

In future, we may be able to modulize oap-project/arrow's cpp code to let compilation pass no matter ARROW_{FORMAT} is specified.


Client server article

This PR contains an article added to the ZeroMQ docs that describes the construction of a workflow running a simple client-server network architecture.
Loosely related to issue #15.

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

Hi @glopesdev I would like to update the zeromq to the new docs theme but there's this article that hasn't been merged in yet, can you review and merge it in before I do the update?


bug: accessibility enhancement for footer component

Describe the bug
The Footer component in the app currently lacks essential accessibility features, making it challenging for users with disabilities to navigate and comprehend the content. This issue is created to address these accessibility concerns.
The primary accessibility issues in the Footer component are as follows:

The brand logo lacks an accessible text alternative (alt).
Some links do not have descriptive text, making their purpose unclear to screen reader users.
Social media icons are missing accessible descriptions.
The colour contrast in the footer does not meet accessibility standards.

Proposed Solution
To make the Footer component more accessible, the following actions should be taken:

Add an alt attribute to the brand logo's Image component with a descriptive text alternative.
Provide descriptive text for links to make their purpose clear to screen reader users.
Include accessible descriptions for social media icons using aria-label attributes.
Improve colour contrast by adjusting text styling or background colour.

Expected Outcome
After implementing the proposed accessibility improvements, we expect the Footer component to be more inclusive and user-friendly.
Steps to reproduce

Go to 'https://app.opensauced.pizza/user/notifications?filter=all'
Scroll to the bottom.
Check the social icons.

Browsers
Chrome
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

All yours @RitaDee!


Chrome adding strange pop-up to screenshots

Hi,
We have this strange issue when we run wraith.
I tested the browser on our two machines that we use for capturing and in the both places chrome version is up to the latest version. Currently 63. ....
I validate my config and it is OK.
We try to add user agent like this:
module.exports = function (phantom, ready) {
    // make Wraith wait a bit longer before taking the screenshot
	page.settings.userAgent = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36';
    setTimeout(ready, 1000); // you MUST call the ready() callback for Wraith to continue
}

but with no result.

About the check list. I can not perform "wraith info" command. Here is all available commands which I have: Out wraith project is hosted on Ubuntu instance in AWS cloud.

Regards, Svetlin.

Reporting a problem? Please describe the issue above, and complete the following checklist so that we can help you more quickly.
Issue checklist:

[x] I have validated my config file against YAML Validator to make sure it is valid YAML.

[ ] I have run the wraith info command and pasted the output below:

paste results here

[ ] I have run the command in verbose mode (by adding verbose: true to my config) and pasted the output below:

paste results here

[x] I have pasted the contents of my config file below:

##############################################################
##############################################################
# This is an example configuration provided by Wraith.
# Feel free to amend for your own requirements.
# ---
# This particular config is intended to demonstrate how
# to use Wraith in 'capture' mode, which is best suited to
# comparing a test and live version of the same website.
#
# `wraith capture capture.yaml`
#
##############################################################
##############################################################

# (required) The engine to run Wraith with. Examples: 'phantomjs', 'casperjs', 'slimerjs'
browser: "phantomjs"

# (optional) JavaScript file to execute before taking screenshot of every path. Default: nil
before_capture: 'javascript/wait--phantom.js'

# (required) The domains to take screenshots of.
domains:
  current:  "http://www.ruukki.com"
  new:      "http://ruukki-com.rc.sitefinity.fi"

# (required) The paths to capture. All paths should exist for both of the domains specified above.
paths:
  global-b2b: /b2b
  global-b2b-services: /b2b/services

# (required) Screen widths (and optional height) to resize the browser to before taking the screenshot.
screen_widths:
#  - 320
#  - 600x768
#  - 768
#  - 1024
  - 1366
  - 360

# (required) The directory that your screenshots will be stored in
directory: 'shots'

# (required) Amount of fuzz ImageMagick will use when comparing images. A higher fuzz makes the comparison less strict.
fuzz: '5%'

# (optional) The maximum acceptable level of difference (in %) between two images before Wraith reports a failure. Default: 0
threshold: 1

# (optional) Specify the template (and generated thumbnail sizes) for the gallery output.
gallery:
  template: 'slideshow_template' # Examples: 'basic_template' (default), 'slideshow_template'
  thumb_width:  200
  thumb_height: 600

# (optional) Choose which results are displayed in the gallery, and in what order. Default: alphanumeric
# Options:
#   alphanumeric - all paths (with or without a difference) are shown, sorted by path
#   diffs_first - all paths (with or without a difference) are shown, sorted by difference size (largest first)
#   diffs_only - only paths with a difference are shown, sorted by difference size (largest first)
# Note: different screen widths are always grouped together.
mode: diffs_first

I would recommend to try switching the browser in your config to chrome
While it looks like there has been some recent activity in the phantomJS github, the author has stepped down from maintaining it.


Launcher: Added Launcher model with more abilities to filter

Brief description
Launcher has global model with single access to docs used in all widgets. Also gives ability to easily change filters for all components in launcher tool.
Changes

created LauncherModel which cares about refreshing of data (projects, assets and tasks)

single access to mongo database for most of launcher components
cares about filters for all components (filters are not added as real widgets just prepared backend for easy change of it)

task can have assignee which is stored to task items in tasks widget
assets widget in launcher has information about all children task types and all assignees on all tasks (to help filter them)

Testing notes:

Launcher should work the same and maybe a little bit faster

Task linked: OP-2416 Launcher tool model

What kind of data would the Launcher's task model or asset model have that the Asset model must never have. There might be data that is less important for the loader or alike but still it wouldn't get in the way?
E.g. I'd love to also allow filtering the loader assets based on assigned tasks. Or on "related to X". It seems to me that in essence the models are 99% related and where we improve one the other should too. At the very least they should inherit each other?
It will be much more work keeping functionality in sync if we separate plus there is more code to maintain.
Basically what I am asking: are we sure it's easier if we duplicate into a unique model?
Looking at this PR it only adds assignee data. I'd say that's just as worth it in the Loader or even better the Work Files browser.

Main goal of LauncherModel is to keep basic data used across multiple widgets at one place and prequery data that would be queried anyway.
For example with LauncherModel both AssetsWidget and TasksWidget do not query assets from database on their own but use LauncherModel methods to get it and listen to it's changes which gives much more clean way to decide when Asset or Task widget should change their context.
If you would want to do filters on task level you should filter on asset level too to avoid showing irrelevant assets at the same time you have to keep filtering of all widgets in sync and tell them if any has changed in a maintainable way and you have to get values that can be used for filtering (query all tasks for whole project and get them).
With all of this and threaded refresh of assets widget it was horror to make it work and read the code afterwards to understand what is happening why and understand sequence of events and avoid querying of all assets from whole project 4times on first project selection.

At some point when something must be unique to function we can likely inherit from the other to only add slightly more data into it and still use a shared base model.

That is exactly what is happening here. LauncherAssetModel inherits from AssetModel. Changes how assets docs are passed to the model and adds 2 roles for filtering. Both roles require to query also tasks which is not case of default asset model because tasks contain a lot of data which is massive slowdown for other tools on bigger projects. At the same time tasks are used for tasks widget so tasks widget does not have to query them all the time when asset changes and at the same time you have already collected all values that can be used for filtering.

It will be much more work keeping functionality in sync if we separate plus there is more code to maintain.

For example with LauncherModel both AssetsWidget and TasksWidget do not query assets from database on their own but use LauncherModel methods to get it and listen to it's changes which gives much more clean way to decide when Asset or Task widget should change their context.
If you would want to do filters on task level you should filter on asset level too to avoid showing irrelevant assets at the same time you have to keep filtering of all widgets in sync and tell them if any has changed in a maintainable way and you have to get values that can be used for filtering (query all tasks for whole project and get them).

This makes sense. Thanks for the information. Should have probably spent more time investigating the whole code. 👍

task can have assignee which is stored to task items in tasks widget
assets widget in launcher has information about all children task types and all assignees on all tasks (to help filter them)

How would I access or test this? I can't seem to find any complex filtering other than the name field.

I can't seem to find any complex filtering other than the name field.

I didn't implemented widgets for the filtering yet, just prepared base to be able do it. This changes how data in launcher widgets are handled which is already a lot of changes just to have at least the same functionality.
PR adding the widgets will be probably stuck for some time as discussion about "how", "where", etc. will be raised...


Add MacOS CI

Is your feature request related to a problem? Please describe
These tools and jobs are running on MacOS workers.
Describe the solution you'd like
Add MacOS GHA CI.
Describe alternatives you've considered
No response
Additional context
No response

@gaiksaya please review my PR.

Closing this issue as PR is merged now.
Thanks!


No env.R_VERSION

Base on pharmaverse:devel branch
Branch name compatible with SD

@cicdguy same errors as here https://github.com/pharmaverse/admiralophtha/pull/45


Update Swedish translation to v1.3

Hi!
I've added the missing Swedish translations to Sniffnet, and removed the warning of Swedish not being up to date.
Thank you for maintaining Sniffnet!

Thank you so much!

And thank you for a great piece of software!


Add support for R4 type Canonical => Reference Searches

User story
Add support for the R4 primitive type Canonical.
Acceptance criteria

Support is visible at the /metadata endpoint
Canonical urls are indexed correctly
Verify resources can be searched by QuestionnaireResponse.questionnaire, including ?questionnaire=http://url|version

Related to #882

Related to #882

AB#76717

AB#76717

Hi there, are there any updates in regards to this?
This feature is a dealbreaker for us moving from an AWS solution over to Azure, as our API clients must be able to search for certain resources by their canonical identifier (e.g. Questionnaire resources by canonical url). At present attempting this on a Saas health data services, an OperationOutcome with
        {
            "resource": {
                "resourceType": "OperationOutcome",
                "id": "574e8fd763dac8a410252a26cfe07e7d",
                "issue": [
                    {
                        "severity": "warning",
                        "code": "not-supported",
                        "diagnostics": "The search parameter 'questionnaire' is not supported for resource type 'QuestionnaireResponse'."
                    }
                ]
            },
            "search": {
                "mode": "outcome"
            }
        },

Is returned


Add consent request mailer

This will be the email sent out by the scheduled job to request that parents submit consent for their children. That code is to come, this email isn't being sent yet.
The template in GOVUK Notify has been updated and set to live.

Squashed commits


Icon switch frequency

Adds a configurable constant, which determines the amount of days to wait between cycling icons within a single season. See my conversation with lemon: https://github.com/python-discord/organisation/issues/191#issuecomment-590086579 and below.
@lemonsaurus I liked your suggestion with using itertools.cycle, but I decided to just keep internal state as an int. The reason here is that we can easily reset it back to 0 if we switch seasons.
I tried to keep the implementation as simple as possible. It can be tested by manually setting sleep_time to a few seconds. Running this loop with a sleep_time forced to 3 seconds and logging set to DEBUG produces the following output.
02/23/20 20:03:41 - bot.seasons.season DEBUG: Days since last icon change: 1
02/23/20 20:03:41 - bot.seasons.season DEBUG: Waiting 2 more days to cycle icon
02/23/20 20:03:44 - bot.seasons.season DEBUG: Days since last icon change: 2
02/23/20 20:03:44 - bot.seasons.season DEBUG: Waiting 1 more days to cycle icon
02/23/20 20:03:47 - bot.seasons.season DEBUG: Days since last icon change: 3
02/23/20 20:03:50 - bot.seasons.season DEBUG: Days since last icon change: 1
02/23/20 20:03:50 - bot.seasons.season DEBUG: Waiting 2 more days to cycle icon
02/23/20 20:03:53 - bot.seasons.season DEBUG: Days since last icon change: 2
02/23/20 20:03:53 - bot.seasons.season DEBUG: Waiting 1 more days to cycle icon
02/23/20 20:03:56 - bot.seasons.season DEBUG: Days since last icon change: 3
02/23/20 20:03:59 - bot.seasons.season DEBUG: Days since last icon change: 1
02/23/20 20:03:59 - bot.seasons.season DEBUG: Waiting 2 more days to cycle icon

Considerations
Have I chosen a good place for the constant? Should it be configurable via an environmental var, like the other constants in Client?

Do we care that this counter is going to be reset every time the bot restarts?

@sco1 I don't think it's a tragedy. The bot already doesn't persist state across sessions w.r.t. the index variable, which controls which icon to choose when cycling. So the cycle is reset to index=0 on every restart. I'm not trying to use this as an excuse, of course, ideally we'd do something smarter for that as well, but I see this fix as a stop-gap solution until #329 is ready.
A logical solution would be taking current day in month, and moduloing it by the configured delay:
>>> delay = 3
>>>
>>> for day in range(1, 31):
...     print(day, "cycle" if day % delay == 0 else "wait")
...
1 wait
2 wait
3 cycle
4 wait
5 wait
6 cycle
7 wait
8 wait
9 cycle
...

I don't think this really improves the situation that much without also changing the way icons are drawn though.  Or we could always just draw randomly. I'm open to suggestions.


Allow AsyncAPI mocking and contract-testing support

Reason/Context
As of today (version 0.1.4 of this library) we just support essential features of Microcks provided by the main Microcks container. The list of supported features is the following:

Mocking of REST APIs using different kinds of artifacts,
Contract-testing of REST APIs using OPEN_API_SCHEMA runner/strategy,
Mocking and contract-testing of SOAP WebServices,
Mocking and contract-testing of GraphQL APIs,
Mocking and contract-testing of gRPC APIs.

Asynchronous API mocking and ASYNC_API_SCHEMA contract-testing strategy are not supported at the moment as it would require an additional microcks-async-minion container as well as a Kafka broker to ensure communication with the main container.
Description
We'd like to extend the notion of MicrocksContainersEnsemblewe described in #24 to also allow the usage of Microcks asynchronous features.
Implementation ideas
Ideally, we should be able to manage 2 different modes:

the one we're providing a Kafka broker because the user doesn't care (typically, we'll use a RedPanda testcontainer here)
the one we're reusing a Kafka broker that the user has initialized. In that case, we may want/need to reuse or share the same docker network and also have loose-coupling just reusing a broker bootstrap URL to connect everything

Now that we have https://github.com/microcks/microcks/issues/1020, we can tackle this one more lightweight and efficiently!


Added support for installer authentication.

Great project, thank you for your effort!
For the automated configuration of inverters I thought a support for installer login would be helpful and extended the code of the lib respectively. I didn't touch the CLI though.  Also I removed configuring the user name. There doesn't seem to be a choice. It is 'master' for installers and 'user' else.

Thank you for contribution! I think I need some time until I can take a look at it.
But until then: What can you do with the installer login in contrast to the normal user?

@stegm The installer can change some basic settings, which either require electrical engineering know-how or are subject to regulations. For your reference please have a look at the items with footnote 1) on pages 124 and following in the manual.

Created new pull request with changes for backwards compatibility of the login method.


updates readme.md with information of switch case statment in js-basi…

added switch case statement in js-basics/making-decisions

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

heyy!!!can I contribute??

I want to contribute. I have some changes to be suggested.

Why are u telling me?? 🥲🥲 I am not a maintainer, I'm a contributor.

@alfredodeza Hey, can you review and suggest changes in my pull request.


Install Another Version... takes a really long time for the C# extension only

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: 1.63.2
OS Version: Windows 10 Enterprise, Version 20H2, OS Build 19042.1415

I raised an issue regarding this in OmniSharp-VSCode's repo, but they suggested that the issue should be raised here.
Steps to Reproduce:
For weeks, i've noticed that among the all extensions i've installed, the C# one is the only one has this issue. Go to the Extensions view. Click Install Another Version... on the menu for the C# extension. Observe that after hanging for a long time (it took me 56 seconds), the expected list of versions finally shows up.

Getting the same behavior with the Gitlab Workflow extension, also takes forever but eventually loads.
Seems it happens with extensions that have many versions -> long list to produce

/duplicate

@sandy081 could you please mention which one this duplicates?

There is a PR - https://github.com/microsoft/vscode/pull/136992 for this and not an issue tracking it. I will use this issue for tracking that PR


Use/Migrate to Open Source softwares to manage the MOOC

This OpenScience initiative is great! Keep the good work!
I'd like to brainstorm a bit on how to select a tool to use. In order to achieve an Open Everything status, the tools/services used should be Open Source also. For example, git is an free and open-source tool, but GitHub is a non-paid proprietary service. GitLab on the other hand uses only open-source software on their services. An open-source alternative to Slack is Riot.

Aha, this is an excellent discussion to have. I think @dannycolin has been looking into some alternatives for this too.
While I love the idea, we have to consider the impact of moving the whole project to different platforms, especially with large existing communities in them. For example, if we could do a simple migration from GitHub to GitLab, I wouldn't object to that. This is something I'd love to hear about from others too though. :)

I :heart: Open but would be very cautious with moving away from Github (slack to a lesser extent) - I'd be happy with just making clear to learners that this is a proprietary platform, and that there are OSS alternatives.
My main justification for the worry would be that for educational purposes I want to:

have the lowest possible barrier to entry (IMO github UI and docs are >>> for beginners than anything else)
give people the skills that will be the most transferable to other projects (alas there are still many more people developing openly on github, in science and elsewhere, than anywhere else)

On Mon, 12 Nov 2018 03:55:42 -0800
Alexander Morley<EMAIL_ADDRESS>wrote:

I :heart: Open but would be very cautious with moving away from
Github (slack to a lesser extent) - I'd be happy with just making
clear to learners that this is a proprietary platform, and that there
are OSS alternatives.
My main justification for the worry would be that for educational
purposes I want to:

have the lowest possible barrier to entry (IMO github UI and docs
are >>> for beginners than anything else)
give people the skills that will be the most transferable to other
projects (alas there are still many more people developing openly on
github, in science and elsewhere, than anywhere else)

indeed. GitHub is the biggest repository of repositories. But GitLab is
growing a lot. When Microsoft bought GitHub some time ago, several
projects migrated to GitLab.
This idea of having a note saying the tool/service/platform is
proprietary and a list of alternatives is a good idea.

As a Free and Open Source user myself (and also developer), I'm really for having
more open tools but like others already said we need to be carefully when it
comes to migrate to a new platform.
So here's my opinion on the subject and some informations on the tools proposed.
GitLab
It's true that GitLab is more free than GitHub but the instance running on
GitLab.com isn't Open. GitLab offers two versions that are :

Community Edition
100% Free and Open Source but you have to host it yourself. That mean you
need to pay for a server and you need to maintain it (keep it up-to-date).
Enterprise Edition
Not 100% Open and this is the version running on GitLab.com

In other word, moving to GitLab.com wouldn't make us more free. And at the
moment, we don't have the people nor the financial resources to self-host an
instance of GitLab.
A solution for now could be to create a task in the module 5 based
on GitLab. It would give people a free alternative to GitHub.
We could also add this as a prerequiste to every tools we teach in the different modules.
What do you think about that?
Riot
I've tested it and it works great but I have to admit that newcomers will
probably face a bigger learning curve with this platform than they have to face
with Slack. The interface is a bit less intuitive and like others pointed out
it's not an easy task to move people to a new platform.
So my opinion on that would be that in order to move on Riot we would definitely
need to do a guide to help people migrating. We would also need to give people
time. I think if we go that way we should open an issue just for that migration.
Finally,

I'd be happy with just making clear to learners that this is a proprietary platform, and that there are OSS alternatives.

I'm also voting for that. I think it should even be an obligation to make clear if a software that we propose is Open or not.

This idea of having a note saying the tool/service/platform is proprietary and a list of alternatives is a good idea.

So since it's not the first time someone on a thread think that a list of alternative would be a good idea, I have added to my todo list a task of creating a small website à la alternativeto.net (nothing to fancy) where people could enter a software name and in return a list a Free and Open Source alternative will pop up. If others here would be interested to help I could create a new repository to coordinate it.

In other word, moving to GitLab.com wouldn't make us more free. And
at the moment, we don't have the people nor the financial resources
to self-host an instance of GitLab.

That's a very strong argument.

We could also add this as a prerequiste to every tools we teach in
the different modules. What do you think about that?

You mean for every tool in the modules to be open source?

I've tested it and it works great but I have to admit that newcomers
will probably face a bigger learning curve with this platform than
they have to face with Slack. The interface is a bit less intuitive
and like others pointed out it's not an easy task to move people to a
new platform.

Riot devs are making a new interface to solve lots of issues, some
initial comments about it are
here.
I also needed some time to understand Slack and feel comfortable with
every concept there. If the tool is Slack, Riot or something else,
newcomers will face difficulties for a time.

This idea of having a note saying the tool/service/platform is
proprietary and a list of alternatives is a good idea.

So since it's not the first time someone on a thread think that a
list of alternative would be a good idea, I have added to my todo
list a task of creating a small website à la alternativeto.net
(nothing to fancy) where people could enter a software name and in
return a list a Free and Open Source alternative will pop up. If
others here would be interested to help I could create a new
repository to coordinate it.

That would be great! I don't have experience with web development but I
am quite good at opening issues at repositories ^^

We could also add this as a prerequiste to every tools we teach in
the different modules. What do you think about that?

You mean for every tool in the modules to be open source?

I mean if there's a task with a non-free software, we should at least have a alternative task (if there's a alternative ofc) with a FOSS software. In that case we could have a task with GitLab and a small list of instance that run the Community Edition (maybe also GitLab.com with a note saying that it's not 100% free but better than GitHub etc.).

That would be great! I don't have experience with web development but I am quite good at opening issues at repositories ^^

It's not a problem there's more than just code ;)
Btw here's a list of FOSS gitlab that you can use for your project:

https://framagit.org (hosted by framasoft a French non-profit that promote FOSS software especially cloud services).
https://salsa.debian.org (it's Debian but if you respect the Code of Conduct you can use it for your personal projects)
https://about.0xacab.org/ (it's hosted by Riseup.net)

@dannycolin We can coordinate on alternativeto.net if you want, they key step is to create a reasonable database of the correspondence between the FOSS and alternatives at the moment AFAIK none exists. I need to create one for a separate project (https://github.com/alexmorley/Back-Your-Scientific-Stack/issues/2) so we can collab on making that database if you fancy :smile:

Alexander Morley, 13/11/18 10:32:

FOSS and alternatives at the moment AFAIK none exists.

There's plenty of alternatives:
https://en.wikipedia.org/wiki/Comparison_of_source_code_hosting_facilities
https://www.gnu.org/software/repo-criteria-evaluation.html
(See also
https://about.gitlab.com/press/releases/2017-11-01-gitlab-transitions-contributor-license.html.)
IMHO it's not that hard, a freedom-friendly course can recommend to use
GitLab while noting that directions exist to self-host or seek an (even
more) ethical hosted solution, including nongnu.org and
https://framagit.org.

@nemobis I think @alexmorley was talking about an alternative to alternativeto.net :).
If I do a recap of the discussion, we pointed out 3 different issues (see list below). I'll create a new issue for 2 and 3. So this is one could focus on 1.

Use/Migrate to Open Source softwares to manage the MOOC (#17)
Suggest alternative tasks that use Open Source softwares (#18)
Create a OpenSource softwares database à la AlternativeTo.net (#19)

To follow up on this issue, here's a proposal for a [possible] future migration.

I strongly believe that promoting free and open source softwares is a core principle of open science and I know that people here think the same.
I also know that for the time being we don't have the resources to self-host our own services.
Finally, it's easy to migrate code but it's harder to do the same with users. But the more we wait the more it'll be hard to do the migration.

I propose that we flag this issue as a bug and act accordingly. This means that :

we will not open new collaborative platforms that aren't open source;
we will investigate, compare alternatives to GitHub and Slack.;
we will put in place a migration plan.

Of course, there's no rush for the moment to put that plan in action. The goal of this proposal is to acknowledge that we all agree on the principle of using as much free and open source softwares as we can.

1. we will not open new collaborative platforms that aren't open source;

I'd suggest as open as possible and as closed as needed. As you mentioned, you'll need resources to host your own services and that might not be easy. GitHub is not open-source, but allows you to be transparent and show your source.

2. we will investigate, compare alternatives to GitHub and Slack.;

I mentioned Riot as alternative to Slack because:

It uses an open protocol: Matrix
It's open source
There's an Open Science room there :) (and it was there that I found out this OpenScience MOOC)
If you don't like Riot, there are other softwares that are in the Matrix ecosystem and you can use them to access the same chats/rooms/users
There's a redesign of the user interface coming out soon.

I'd suggest as open as possible and as closed as needed.

@heitorPB I've edited my comment. Thanks for the suggestion! It think the as open as is sufficient tho.
If everyone agree with the proposal, I think we could chat about alternatives in the platform channel on Slack. :)

Hey all, I've created a table of software alternatives in the Wiki https://github.com/OpenScienceMOOC/Main/wiki/Default-to-Open:-Migration-plan-for-the-OpenScienceMOOC
If you have other suggestions don't hesitate to add them even if you can't find all the informations :).

I would add this note for the Riot entry:
Riot is one client for the Matrix protocol, a federated open protocol for real-time communication. There are several other clients available. You can also create your own server with one of the available server softwares.

@heitorPB Yes, you're right. I think I'll change it for Matrix and add a link to the clients suggested on the Matrix site.


Consider not following symlinks when removing outputs

Before 6.0 we used to follow symlinks when removing task outputs before loading from cache, removing stale outputs, removing local state on cache load, running the Delete task etc.
We might want to reconsider how we want this to work. One negative side effect is when someone has for example a node_modules directory with symlinks to globally installed modules that can get the global modules removed this way. This is surprising if not outright incorrect behavior.

Actually, the Delete task (and hence clean tasks) defaults to not following symlinks.


Restructure configuration.toml for new Writeable section and Abstract Registry

Once the Abstract Registry integration into device-sdk-go is merged the go.mod file needs to be updated to pull latest device-sdk-go and then the following changes need to be made to the following files:
    - cmd/res/configuration.toml
    - cmd/res/docker/configuration.toml

Add the Writable section as:
[Writable]
LogLevel = 'INFO'

Modify the Logging section to be:
[Logging]
EnableRemote = false
File = "./device-simple.log"

Also add following to Registry section
Type = "consul"

sivakrishnan-intel will make these changes

Addressed by PR #32


db_owner is not working

Module doesn't show db_owner

Thanks for the bug report! Should be fixed now, please reopen if the issue persists


Fixed scrolling when verticalScrollHeight is 0

Open the basic example:
http://ui-grid.info/docs/#/tutorial/101_intro
Hover above the grid and try to scroll up - it doesn't scroll the page.
Scrolling down, however, works.
This fixes it

Duplicates: #3666 #3531 #3340 (for reference)

@500tech-user Can you tell me is this still an issue in master?  I think this was fixed in some other commits.

ok, I see that mousewheel is still broken for it

Thanks for PR.
Rather than putting the logic where you have it, we would rather see the verticalScrollLength property added to scrollEvent and then the atBottom, atTop, atLeft, and atRight can use that number to determine if it is at bottom, top, etc.
I'll look at making this change and will close this PR when complete.

Should be fixed with
https://github.com/angular-ui/ng-grid/commit/e582174a826bb232ddbb4fda8001b64c3273df0d .  Thanks for the PR

Awesome
BTW, I also noticed that when zooming in or out the calculation is wrong.
Did you ever encounter that?
--

Adam Klein http://il.linkedin.com/in/adamklein500
CTO @ 500Tech.com http://500tech.com/ - Web and Mobile development
hackademy.co.il http://hackademy.co.il/angularjs/about - AngularJS,
Ruby on Rails training
(+972) 52 747 5633

On Fri, Jun 19, 2015 at 5:50 AM, Shane Walters<EMAIL_ADDRESS>wrote:

Closed #3690 https://github.com/angular-ui/ng-grid/pull/3690.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ng-grid/pull/3690#event-334830703.

The problem persists in v.3.0.1.


When child is focused, container gets scrolled in DOM as well as _childFocused firing

Leads to content being permanently clipped off the top of the scroller.

Actually, I'm not even seeing _childFocused being called...


Show current version and availability of upgrades

Might need to wait for a proper v1 - but the version of the system should be shown, along with a subtle warning that there's updates, perhaps less subtle if the current version has been flagged for some reason, like vulnerabilities. Obviously being docker based, can't really make the system self up-gradable, but a solid warning would go a long way.

@thecoshman
This will have to wait until we have v1.0.
This will require some kind of usage of REST api from github to get versions and compare the one that's running.
This will soon go out of hand as there might be channels of versions too
stable versions - coming from master branch
dev versions - coming from develop branch
And then option to move between channels - people may choose to move from stable to dev or dev to stable.
For now, just let the users pull the right image and use it.

@thecoshman - it is done (hopefully). Download v1.2.13 and let me know.


Vanitygen option

It would be cool if this had a vanity option so that the z-address matched a requested format.
You mentioned that you had some of this in branch. If you could copy-paste this into a gist or a different branch, I can try and get it working.

Done in #5


Update installations_doc.md

Added a description of the solution to common Redis issue to Troubleshooting: 7.3 Could not connect to Redis server
Revised chapter  7.2

*** Then I updated installations_doc.md.
Please update the installation guide DIRECTLY here: https://qto.fi:442/qto/view/installations_doc
after that checkout the latest dev ( should be v0.8.2 ) branch. that is git diff dev..v0.8.2 should not return anything

run the generate-md-docs AGAINS this site: https://qto.fi:442/
that is configure first the app section of the configuration file for the qto.fi:442 siteand run the bash ./src/bash/qto/qto.sh -a generate-md-docs

after that check the difference between the devby git diff
and than create the merge request to merge into dev ...


CLI bug: Interactive mode -> Unpacker -> Input file path -> 'Input does not exist' when using prefix/**/*.js glob pattern

Trying the new CLI in interactive mode, it seems to have issues when I give it an input file path like unpacked/**/*.js, even though that path definitely exists (running from the root dir of this checked out repo (Ref))
⇒ npx @wakaru/cli

┌   Wakaru CLI v0.0.2
│
│  Run "wakaru --help" for usage options
│
◇  Select features to use (Use <space> to select, <enter> to submit)
│  Unpacker - Unpacks bundled code into separated modules
│
└  Selected features: Unpacker

┌   Unpacker
│
▲  Input file path (Supports glob patterns)
│  unpacked/**/*.js_
└  Input does not exist

If I change the input path to something like unpacked/, then the error message changes to Input is not a file.
While I haven't looked too deeply into it, I think there might currently be a check to ensure that the provided input is a file; but that would then break the ability to import folders and globs.

While it's OOS of this issue, I also noticed that it's not possible to tab complete files/paths while using the interactive mode. That may be by design, but it would probably make it more user friendly. I can open this as a separate issue if desired.

While I haven't looked too deeply into it, I think there might currently be a check to ensure that the provided input is a file; but that would then break the ability to import folders and globs.

Looking at the code, this seems to be implemented here:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/cli.ts#L203-L218
It seems to be using the existSync / statSync functions from fs-extra:

https://github.com/jprichardson/node-fs-extra/tree/11.1.1

https://github.com/jprichardson/node-fs-extra/blob/11.1.1/docs/pathExists-sync.md

An alias for fs.existsSync(), created for consistency with pathExists().

https://nodejs.org/api/fs.html#fs_fs_existssync_path

???

https://github.com/jprichardson/node-fs-extra/blob/acf55855580db460fd5532b22d73c4543798cd7d/lib/util/stat.js#L20-L33

I'm not sure if the CLI is designed to be able to be passed a folder without a glob, but if so, then these checks should accept a folder as the path.
For checking globs, I think maybe a function like isGlobPath or similar should be added to:

https://github.com/pionxzh/wakaru/blob/main/packages/cli/src/path.ts

It seems globby is the glob matching library used, which seems to already expose a method that could be used for this isDynamicPattern:

https://github.com/sindresorhus/globby

https://github.com/sindresorhus/globby#isdynamicpatternpatterns-options

Returns a boolean of whether there are any special glob characters in the patterns.

I'm not sure if the CLI is designed to be able to be passed a folder without a glob, but if so, then these checks should accept a folder as the path

The CLI calls unpacker here:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/cli.ts#L275-L278
unpacker has a paths arg, which seems to be processed as a list of full file paths; so folders aren't currently supported at this level:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/unpacker.ts#L15-L25

That said, if we wanted to enhance things to be able to specify a folder path (that would then apply a default glob, eg. $folderPath/**/*.js), that should be fairly easy to do. Off the top of my head, we would:

check if the provided path is to a folder
if so, check if the folder is within the 'allowed working directory'
if so, append the default glob pattern to it (eg. **/*.js)
allow the normal glob expansion code to process it

I think currently, the isPathInside check will fail here when given a glob/etc as well:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/cli.ts#L211-L214
At least, depending on how that glob is defined I suppose:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/path.ts#L6-L12

I also noticed that it's not possible to tab complete files/paths while using the interactive mode. That may be by design, but it would probably make it more user friendly. I can open this as a separate issue if desired.

This one is on my todo list. Can open in another one.

The path validation in unpacker is not correct because it forgot to calculate the glob.
And this is path validation in unminify:
https://github.com/pionxzh/wakaru/blob/3758f4810965ddb7b7b7dcf7fc02653350867a0a/packages/cli/src/cli.ts#L314-L316

My expectation on the path input would be only "file(s)" or "glob to files". I'm also not sure about the use of a passing folder...
If we don't change the design, we can improve the error message to be Input is a folder, if you want to specific files in the folder, please use {folder}/**/*.js.

This one is on my todo list. Can open in another one.

@pionxzh Done:

https://github.com/pionxzh/wakaru/issues/83

My expectation on the path input would be only "file(s)" or "glob to files". I'm also not sure about the use of a passing folder...
If we don't change the design, we can improve the error message to be Input is a folder, if you want to specific files in the folder, please use {folder}/**/*.js.

@pionxzh I don't have strong feelings either way. The first way I tried to do it was just passing the input folder path, but it's not much extra effort for me to be explicit and use a glob.
If a folder path on it's own isn't allowed, then I definitely think the improved error message is a good idea.
For simplicity sake/reducing the number of options, I would probably suggest just forcing the user to use a glob + showing the improved error message.

Whichever way is chosen though, the validation should probably be shared between passing args via CLI flags, versus using interactive mode, as currently it looks like it (at least somewhat) works to do this:
⇒ npx @wakaru/cli unpacker unpacked --unpacker-output stage2-unpacked --perf

┌   Wakaru CLI v0.0.2
│
└  Selected features: Unpacker

┌   Unpacker
│
◇  Unpacking...
SyntaxError: Leading decorators must be attached to a class declaration. (1:5)
    at constructor (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:18018:23)
    at FlowParserMixin.raise (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:20914:23)
    at FlowParserMixin.parseDecorators (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30573:22)
    at FlowParserMixin.parseStatementLike (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30370:29)
    at FlowParserMixin.parseStatementLike (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:22799:28)
    at FlowParserMixin.parseModuleItem (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30349:21)
    at FlowParserMixin.parseBlockOrModuleBlockBody (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30976:40)
    at FlowParserMixin.parseBlockBody (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30969:14)
    at FlowParserMixin.parseProgram (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30248:14)
    at FlowParserMixin.parseTopLevel (/Users/devalias/.npm/_npx/adacf4dfcf214674/node_modules/@wakaru/cli/dist/chunk-KOQHKJQW.cjs:30238:29) {
  code: 'BABEL_PARSER_SYNTAX_ERROR',
  reasonCode: 'UnexpectedLeadingDecorator',
  loc: Position { line: 1, column: 5, index: 5 },
  pos: 5
}

Technically even if the user is forced to use a glob, they could write a glob that only matches a folder, so we would also need to make sure that the expanded glob path is filtered down to the list of files (assuming that isn't already being done).


Add typeguards

This looks based on some old code, because I did replacement of isPromise with isPromiseLike some 10 hours ago.

Just fixed that up

This looks like a lot of code to replace something small and trivial. What's the justification for such a change?

The majority of the code is just function overloading (most of which probably won't be needed in the end and is just included for completeness); there's very little actual code.
The two new functions hasOwn and hasOwnOfType will likely be used quite a lot in future changes. I've set this PR to draft for now as there's quite a bit that will probably go into it.
I'm on GMT +13

Codecov Report

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

@@           Coverage Diff            @@
##             main       #96   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        41
  Lines           ?      1252
  Branches        ?       260
========================================
  Hits            ?      1252
  Misses          ?         0
  Partials        ?         0

Continue to review full report at Codecov.

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

I've done a lot of performance benchmarking here first, for generators, to conclude that in the latest Chrome and NodeJS, generators are about 1.4x times slower than native JavaScript iterators. And async/await iterators are about 2x times slower than native JavaScript iterators. That's why I'm not using those in this library, which has a strong focus on performance.

I tend to take a different approach with my projects. I usually prioritize readability and maintainability over small performance improvements.
I wounder if we can get the best of both worlds by writing the source code with generators and then using the build process to replace them with more performant code.
regenerator is one such library that can transform generator functions into non-generator functions. But as it is general purpose, it has its own runtime and thus adds its own overhead which isn't ideal. If prepack ever continues development and gets into a more useable state, running everything though that would potentially remove all that overhead.
nodent is another library that could potentially work but in my short play with it I couldn't really get it to do anything useful.
Unless there's another miracle library out there that does everything we want, I guess we'll have to stick with the current way for maximum performance. Luckily however I've got more ideas on how to improve readability and maintainability without sacrificing performance.
I'm probably going to wipe this PR's current work and start again with it.

I'll mark this PR as ready for now. Best to keep it small.
Let me know what you think of it.

This PR should be rebased on #117.

@vitaly-t This one is now ready.
A lot of the previous comments seem to be irrelevant now as the PR does less than it initial was intended to do.


HTTP tutorial doesn't make it clear how load additional http library

I am not clear on how to get a new dependency elm-lang/http  loaded by elm-reactor and the tutorial doesn't mention workflow to do so.
I see related platform issue  https://github.com/elm-lang/elm-platform/issues/162 "Error Installing The Elm HTTP Library"  I had the same issue with Elm 0.18 and the tutorial at https://guide.elm-lang.org/architecture/effects/http.html.  I threw away ~/.elm to no avail, but got it to work running
elm-package install elm-lang/http
Perhaps this is normal, but I'm not sure the tutorials makes it clear when elm-reactor won't grab a dependency.  Thanks.

With the latest version of https://guide.elm-lang.org/ there is an Edit button for each example, that takes you to the online editor, like this. So you wouldn't need to use elm install for that path.
This repo also has all the necessary dependencies listed in elm.json, so running based on the examples in README.md should be working.
Finally, the guide now has a section (on this page) that introduces how to use elm install explicitly.
So hopefully these changes address the root issue here!


Watchdog timer set to garbage

It looks like watchdog is configured incorrectly in init_clocks_and_plls
https://github.com/rp-rs/rp-hal/blob/ffa97842e295b19a01fb6438f700c683e9d90ce1/rp2040-hal/src/clocks/mod.rs#L320
in datasheet it is that it should work at 1MHz (and alarms are using the same clock), therefore this should be set here to:
(xosc_crystal_freq / 1_000_000) as u8
currently it is trying write very big number (12MHz) to 8bits and as it is truncated it sets it to garbage value

Nice catch. This logic checks out.
This is also essentially how the C SDK does it - they pass XOSC_MHZ which is declared as #define XOSC_MHZ 12 to their equivalent of this function.


flake.nix: allow inclusion of nixpkgs as path:/.../

When adding nixpkgs as flake-input using the path-fetcher, you
currently get the following error since neither lastModifiedDate nor
lastModified are stored in flake.lock for paths:
error: --- EvalError --------------------------------------------------------------------------- nix-build
at: (48:71) in file: /nix/store/147clg8svaxyj7pl80ra9kmmm72mdg94-source/flake.nix

    47|                   system.nixos.versionSuffix =
    48|                     ".${final.substring 0 8 (self.lastModifiedDate or self.lastModified)}.${self.shortRev or "dirty"}";
      |                                                                       ^
    49|                   system.nixos.revision = final.mkIf (self ? rev) self.rev;

attribute 'lastModified' missing

This patch adds the fallback-value "19700101" to versionSuffix if none
of lastModified{,Date} are set in the lockfile.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Ported to stable as f7fe019d53e63b7f29d415496d696c003dfcfffb.


Address API review feedback related to APIs in az_span.h.

Addressing API review feedback related to az_core from https://apiview.dev/Assemblies/Review/e29157a38d8e43949fae98a53381f5a5

API breaking changes:

az_span.h:

az_span_init(...) is renamed to az_span_create(...)
az_span_from_str(...) is renamed to az_span_create_from_str(...)
az_allocator_context is renamed to az_span_allocator_context
Removed az_pair_from_str(...)

Also went and formatted (spacing/etc.) of the files I modified, in VS, using Ctrl+K, Ctrl+D.
Since IoT uses az_pair atm in their public surface area, leaving it for now:
https://github.com/Azure/azure-sdk-for-c/blob/cb8694901af8ff23c81e05444557e1940a239869/sdk/src/azure/iot/az_iot_hub_client.c#L268
We should consider revisiting that and then remove the az_pair type.

In the interest of time, merging this. Others who haven't looked at it, please feel free to review and leave other feedback if it comes up.


info command output now ordered

issue #3

Unfortunately this is not the fix of original problem.
Example:
d = {'32': {}, '1': {}, '12': {}, '3': {}} orderedret = OrderedDict() for version in sorted(d):     orderedret[version] = d[version] print(json.dums(orderedret))
Output is

{"1": {}, "12": {}, "3": {}, "32": {}}

Ok, i will cast it as int

Could you check if this patch solve original problem?

I've commited patch as  77d7df4. So I close this.


v3.3.0

clean split into _normalize and _format to provide more user-friendly normalization methods
automatic tests now included
isotherm benchmark comparison added as example
FoKL-to-Pyomo significant improvements
multiple GP models in Pyomo added as example
updated documentation to reflect current changes

Tests fail here because 'tox.ini' links to ESMS repo which is not yet updated. The tests passed in my fork (when 'tox.ini' linked there), but can be confirmed within the ESMS repo prior to PyPI release (after merging PR).


Incompatibility issues with @apollo/subgraph

Issue workflow progress
Progress of the issue based on the
Contributor Workflow

[x] 1. The issue provides a
minimal reproduction available on Stackblitz.

Please install the latest @envelop/* packages that you are using.
Please make sure the reproduction is as small as possible.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
I am not sure where to attribute this bug, as it requires a very specific setup to trigger. Unfortunately for us, this is exactly our setup in our most important subgraph.
Following the instructions on Yoga site how to integrate Apollo Subgraph https://the-guild.dev/graphql/yoga-server/docs/features/apollo-federation#example-user-service
This works, of course, but when you add a few properties things get interesting:

Add DateTime scalar
Import DateResolver from graphql-scalars as a resolver for DateTime scalar (yes, that is correct, NOT DateTimeResolver)
Add response cache plugin from @graphql-yoga/plugin-response-cache

To Reproduce
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/node-psuvdg
run node index.js and you will get Error: Schema must contain uniquely named types but contains multiple types named "Date".
run DISABLE_RESPONSE_CACHE=1 node index.js and realize that if we instead remove response cache this also fixes the issue. But now we dont have a response cache so that is not good.
run DISABLE_DATETIME_RESOLVER=1 node index.js and realize that removing the Date resolver fixes the issue (but creates bugs in how DateTime is serialized since we want it serialized as DateResolver, this is a fix that contains a breaking change! this is also the reason we cannot use DateTimeResolver (using that also boots everything correctly))

Expected behavior
I need this to boot using node index.js and

having DateTime scalar resolved as Date
loading a schema containing federated directives
having response cache plugin active

Environment:

OS: All
NodeJS: 18.18
"@graphql-yoga/plugin-response-cache": "2.2.0",
"graphql": "16.8.1",
"graphql-yoga": "4.0.5"

Additional context
Since so many components are involved, the issue can be anywhere

response cache plugin transforming schema using mapSchema
buildSubgraphSchema doing something worse that makeExecutableSchema would do better  (but I need the federation directives, right?)
using DateTimeResolver instead of DateTime but our requirements/expectations are already years on this path so I cant do a breaking change easily. Unfortunately we need DateTime to be resolved as DateResolver.

Updated stackblitz to latest major versions of everything, the issue persists.

After investigation, The problem comes from @apollo/subgraph only relying on the config object keys for the scalar names, while @graphql-tools/schema is relying on the name defined on the type.
This is important because mapSchema allows to rename types. For this to work, it needs to update all references to the renamed type. But if the name defined in the type is not correct, it can't remap the old name to the new one.
So you can't really use a scalar resolver with a name Date and use it to resolve a scalar named DateTime.
An easy workaround that should work is to set the graphql-scalars resolver's name to DateTime:
const resolvers = {
  // Create a new Scalar by extending `graphql-scalars` Date config and just change the name.
  DateTime: new GraphQLScalarType({
    ...DateResolver.toConfig(),
    name: 'DateTime',
  }),,
};

WOw, awesome! Thank you so much ❤ ❤ ❤