Architecture decision - unlocking pending transfers on-chain

Also related to the channel lifecycle issue here https://github.com/raiden-network/raiden-contracts/issues/193
I wanted to describe the unlock issue in more detail.
Current channel lifecycle start & end:

openChannel
a) closeChannel & settleChannel

remove channel data
we might have unlock data remaining in the contract in a separate mapping
new channel can be opened between the same participants

b) cooperativeSettle

remove channel data
no unlock data remains in the contract
new channel can be opened between the same participants

The issue appears for 2.a), when we can have a totally different channel open and possibly multiple unlock data for multiple channels.
PROs:

pending transfers can be unlocked at a later time or not at all, depending on the value of tokens vs. gas needed for the transaction
transaction gas cost is somewhat fair: each participant can pay for the unlock that favors him, settleChannel (that potentially transfers the biggest amount of tokens) costs less gas due to clearing up storage.

CONs:

channel lifecycle is not simple anymore -> adds complexity in the Raiden client or any client that might be implemented

How we ended up here:

batch unlock discussions (why we wanted it): https://github.com/raiden-network/spec/issues/35, https://github.com/raiden-network/spec/issues/31,
batch unlock implementation: https://github.com/raiden-network/raiden-contracts/pull/39, with this thread also in mind: https://github.com/raiden-network/raiden-contracts/pull/39#discussion_r183750230 about the lifecycle issue and proposal of solution 2.

Possible solutions
1. Do nothing
and we will keep having channel lifecycle issues. Also keep in mind my two comments from https://github.com/raiden-network/raiden-contracts/pull/198#discussion_r206465782 & https://github.com/raiden-network/raiden-contracts/pull/198#discussion_r206465857
2. Remove channel data after both unlocks
, forcing participants to do the two on-chain transactions IF they want to open a new channel

suggested by @hackaugusto also in https://github.com/raiden-network/raiden-contracts/pull/39#discussion_r183750230
settleChannel transaction will become more costly
last unlock will become less costly (gas returned for removing more storage - channel data)
unlock data can be kept in the channel struct
channel lifecycle end is clear
Raiden client will have less complexity by not being forced to keep additional unlock data for (maybe) multiple channels
needs refactoring in TokenNetwork contract (more than 3.)
needs some Raiden client refactoring (not sure how much compared to 3.)

3. Make it clear that for a specific channel_identifier, we have unlock data in storage.

keep current way being able to unlock at any point in time or not at all
transaction gas cost is somewhat fair (see PROs)
channel lifecycle end is not clear, but it might be enough to clarify a channel's state if we link the unlock data with the channel_identifier
Raiden client needs to either keep unlock data separately (probably easier) or support multiple channels opened at the same time (something that we discussed before and wanted, but we changed our minds at some point)
needs some refactoring in the TokenNetwork contract, as described below:

https://github.com/raiden-network/raiden-contracts/pull/198 is not enough;
We also discussed https://github.com/raiden-network/raiden-contracts/issues/193#issuecomment-407689046 but that is not totally correct.
After thinking about this, we can have:
    // The key is `keccak256(uint256 channel_identifier, address participant, address partner)`
    // Where `participant` is the participant that sent the pending transfers
    // We need `partner` for knowing where to send the claimable tokens
    mapping(bytes32 => UnlockData) unlock_identifier_to_unlock_data;

    struct UnlockData {
        // Merkle root of the pending transfers tree from the Raiden client
        bytes32 locksroot;
        // Total amount of tokens locked in the pending transfers corresponding to the `locksroot`
        uint256 locked_amount;
    }

    function unlock(
        uint256 channel_identifier,
        address participant,
        address partner,
        bytes merkle_tree_leaves
    )

The next question is about our state getters.
After https://github.com/raiden-network/raiden-contracts/pull/198 we have:
// ! returns (settle_block_number, state)
function getChannelInfo(uint256 channel_identifier) view external returns (uint256, uint8)

// returns: deposit, withdrawn_amount, is_the_closer, balance_hash, nonce
function getChannelParticipantInfo(uint256 channel_identifier, address participant)

If we want to include the unlock data somewhere, we can include it in getChannelParticipantInfo for example. Then, this will look like:
// returns participant's: deposit, withdrawn_amount, is_the_closer, balance_hash, nonce, locksroot, locked_amount
// `participant` - sends the pending transfers
// `partner` - receives claimable locked amount from `participant`'s transfers
function getChannelParticipantInfo(uint256 channel_identifier, address participant, address partner)

Questions that may influence the decision through economic incentives:
a) who does the two unlocks on-chain?

each participant unlocks his own pending transfers that he has sent
each participant unlocks his partner's pending transfers that he might receive
Raiden client checks what pending transfers are most valuable to the participant based on the claimable and unclaimable amount of tokens and does the unlock automatically.

@hackaugusto , @LefterisJP,  @rakanalh ?

We have not actually considered solution 2 that much until now. I think we should, otherwise we might end up discussing the same problem after we consider the contracts finalized.
In my opinion, better discuss & decide now rather than after an audit.

Hey Loredana.
Discussion of approach 2
Questions
So you want to have logic in unlock() that deletes the channel data for the participants from the channels mapping once both sides have deleted the pending transfers, is that right?

forcing participants to do the two on-chain transactions IF they want to open a new channel

So by this you mean that if the people want to open a new channel they will be forced to call unlock on their pending transfers even if that makes no sense due to a dust amount, right?
Comments

Raiden client will have less complexity by not being forced to keep additional unlock data for (maybe) multiple channels

We already have an issue for this in the client as a direct result of our conversation about this last week. So this is actually being worked on. I don't want to have to undo this work again.

needs some Raiden client refactoring (not sure how much compared to 3.)

Yes it will need quite a bit of work, and can't answer with certainty until I (or augusto) sit down look at the code and think of what needs to be changed.
Opinion on approach 2
Keep in mind we are going to make more changes on the contracts after red eyes mainnet release so this can be scheduled for then if we deem it necessary. We also have set ourselves a rough date of summer 2018 which already seems rather hard to hit as it is. Adding more tasks at this point is going to make it near impossible to finish on-time.
I also don't like the idea of having to force users to unlock pending transfers if they want to open a new channel with the same participant. I find it a useful feature of the current approach that they have a choice.
At the moment my feeling is to not go with this at this time.
But I note your concern here:

I think we should (I also think it is a good option), otherwise we might end up discussing the same problem after we consider the contracts finalized.

and I would like to ask in what ways do you think the current way would manifest in a problem later if we implement the changes discussed in #193 for the smart contracts and for the client the issues described here.
Questions on approach 3

transaction gas cost is somewhat fair (see PROs)

As per my comment on approach 2, I like the current way of allowing to unlock or not unlock at all so I like this pro.

but it might be enough to clarify a channel's state if we link the unlock data with the channel_identifier

So this is what you are describing at the end of your comment with the source code snippet, right? To include the unlock data here:
function getChannelParticipantInfo(uint256 channel_identifier, address participant, address partner)?
Opinion on approach 3
So the only proposed change here is to link unlock data with channel identifier so that it's clearer to query a channel state? That sounds doable and useful.
Question on who does unlocks on chain

each participant unlocks his partner's pending transfers that he might receive

^ This is the way that we currently do it in the raiden client.
It's far from optimal.

Raiden client checks what pending transfers are most valuable to the participant based on the claimable and unclaimable amount of tokens and does the unlock automatically.

I think either asking the user for input at this point or the user having some kind of setting she can customize about the amount above which pending transfers should be unlocked would make sense

Discussion of approach 2 - Questions

Correct.

and I would like to ask in what ways do you think the current way would manifest in a problem later if we implement the changes discussed in #193 for the smart contracts and for the client the issues described here.

Not sure, because I don't have good enough view on the Raiden client as you or @hackaugusto . But roughly, it would be related to Raiden client [..] being forced to keep additional unlock data for (maybe) multiple channels

Questions on approach 3 & Opinion on approach 3

Correct.

First a clarification, this issue is about the definition of the life cycle of a channel, correct?
For the channel life cycle there are a few things to take into account:

Are two participants allowed to have multiple channels open at the same time?
How do we detect if an operation the node wants to perform is still valid?
2.1 And how do we identify the on-chain channel

It seems the discussion here is about 1. and not 2.
With that said, IMO the channel life cycle discussion started with issue https://github.com/raiden-network/raiden-contracts/issues/40 "Disallow multiple channels between two nodes" . It was decide to forbid multiple channels because having it would "[...] require changes in the Raiden API and the PFS". If the goal was to do the above then the life cycle of channel should have been as the following diagram:

And a new channel should only be available after both unlocks or coop settle. Having a pending operation for a channel (unlock) while another channel is opened means there are multiple channels open at the same time, which effectively means the goal of issue #40 to reduce the impact on the Raiden code base was not achieved (https://github.com/raiden-network/raiden/issues/1941).
The goal of issue #40 was not technical, but to save development time, so I don't have much more to say about it.

So, then @hackaugusto , what is your position on:

Are two participants allowed to have multiple channels open at the same time?

regardless of what decisions we already made?

Are two participants allowed to have multiple channels open at the same time?

I believe we already discussed this and I really don't see any compelling upside to allowing this for the near term release.
If some external feedback or input comes and proves otherwise then we can reconsider but with the current limited data we have I don't see this worth the hassle.
The only upside of the approach of multiple channels is that you don't have to wait for the close/settle in order to open a new channel with the same peer. Are there any other benefits?

Fixed through https://github.com/raiden-network/raiden-contracts/pull/203


Add Travis-CI / Fix travis.yml

Tests pass locally but are failing on Travis-CI. Possibly an issue around using the iOS keychain, needs more investigation

Simulator had problems with iOS keychain around this time. Closing.


Global tool was changed in the latest vscode update

Environment data

VS Code version: 1.59.0-insider
Commit: 9520f0a515e7e24802c92f885a0efb242fca3831
Jupyter Extension version: 2021.8.1025925145
Python Extension version: 2021.6.944021595
OS and version: Windows_NT x64 10.0.19041
Python version: 3.8.3 64-bit
Type of virtual environment used: virtualenv
Jupyter server running: Local

Description
The latest vscode update changes the global tool of notebook.
Comparison between the versions:

latest update
old update

Version
1.59.0-insider (user setup)
1.59.0-insider (user setup)

Commit
9520f0a515e7e24802c92f885a0efb242fca3831
a9070718ebb793eef4e94e34307eee15c09a0cd4

Electron
13.1.6
12.0.13

Chrome
91.0.4472.124
89.0.4389.128

Node.js
14.16.0
14.16.0

V8
9.1.269.36-electron.0
<IP_ADDRESS>-electron.0

OS
Windows_NT x64 10.0.19041
Windows_NT x64 10.0.19041

Expected behaviour

Actual behaviour

Thanks for the bug. That looks like the jupyter extension is not there or disabled.
Do you have the jupyter extension working in the new version?

Thanks for the bug. That looks like the jupyter extension is not there or disabled.
Do you have the jupyter extension working in the new version? (meaning can you run a cell)

Thank you. Yes, I can run a cell normally. I'm using the last version: 2021.8.1025925145


Reload bearer token from disk to work with kubernetes bound-service-a…

…ccount-tokens
What does this PR do?
fixes https://github.com/DataDog/datadog-agent/issues/10604
... still need to confirm this works
... also needs to be an opt-in flag since this check could be used for anything
Motivation
the token on disk is expired every 1h by kubernetes so we need to reload it
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

the change is to always set execute bearer_token = scraper_config['_bearer_token']
rest of the code is just moved/indented, https://github.com/DataDog/integrations-core/pull/11176/files?w=1 shows it better

added 3 more places that needed to be patched

was still missing:
file":"pkg/autodiscovery/config_poller.go","line":"128","func":"collect","msg":"Unable to collect configurations from provider kubernetes: couldn't fetch \"podlist\": unexpected status code 401

so resorting to manually doing what the token controller does + longer expiration:
      automountServiceAccountToken: false

          - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
            name: kube-api-access-stable
            readOnly: true

- name: kube-api-access-stable
          projected:
            defaultMode: 420
            sources:
            - serviceAccountToken:
                expirationSeconds: < whatever you deem safe >
                path: token
            - configMap:
                items:
                - key: ca.crt
                  path: ca.crt
                name: kube-root-ca.crt
            - downwardAPI:
                items:
                - fieldRef:
                    apiVersion: v1
                    fieldPath: metadata.namespace
                  path: namespace


how do you bind https to a net core?

Hi,
Been trying to figure this out, I'm trying to bind https on iis8 to my net core app and i'll get a "404 not found" when I type in the address https://mysite, but it's fine when I just use http://mysite
I'm hoping someone who ran into this can help.  Thanks so much!

There's nothing ASP.NET Core specific about IIS https binding.  https://www.digicert.com/util/ssl-certificate-configuration-iis-8.htm
If you turn on application logs do you see the request reaching your site?

Hi,
Thanks for the response,  it's showing it request didn't reach my site base on the logs.  here's a screenshot of the https setting in IIS.
The server I'm using has sites using asp 4.5 with ssl on a shared IP.  Trying to do the same for the net core app.  Does net core need a dedicated ip for ssl to work?

Thanks again!

Core does not have any special requirements for IIS bindings. If you're getting a 404 then the binding is likely not your issue, something in IIS is handling the request. What does the 404 page look like?

Hi,
Here's the site with https and just http
https 404

http (working)

Ok, that looks like the default IIS 404 page for remote users. If you can access the page from the same machine, or change the custom error settings to return detailed errors you'll be able to see who generated the 404.

Thanks so much Tratcher, I think i found the problem.  I guess I should never set up sites at 3am in the morning without coffee. the cause was the https ip is not the same used in my A record.
Apologies for the noob error. and thanks so much for helping on this. gonna go sit in the corner for a bit.


get_sessions(start, end)

Hi, get_user and get_session methods worked for me, but when I call get_sessions i got this error:
Call:
client.get_sessions(start=start_date, end=end_date)
Error:
Traceback (most recent call last):
File "", line 1, in
File "/home/pero/anaconda2/lib/python2.7/site-packages/beddit/client.py", line 15, in _is_logged_in
return func(self, *args, **kwargs)
File "/home/pero/anaconda2/lib/python2.7/site-packages/beddit/client.py", line 129, in get_sessions
return [Session(session) for session in r.json()]
File "/home/pero/anaconda2/lib/python2.7/site-packages/beddit/session.py", line 30, in init
self.sampled_tracks = [SampledTrack(obj) for obj in response_object.get('sampled_tracks', [])]
File "/home/pero/anaconda2/lib/python2.7/site-packages/beddit/session.py", line 14, in init
self.samples_per_frame = obj['samples_per_frame']
TypeError: string indices must be integers

Hi, thank you for reporting the issue.
I'll try to fix it.
Please moment.

Hi :) I would like to ask you how soon can I expect the fix.
Thank you for your response.

@peterrot
I fixed this issue and deployed as 0.1.1.
It is available on PyPI.
Sorry for waiting you.

Thank you very much. The method get_sessions now works fine, but the get_user() doesn't work anymore. The first one worked before. There are two calls below, one without parameters, and one with 1:

u = client.get_user()
Traceback (most recent call last):
File "", line 1, in
File "beddit/client.py", line 15, in _is_logged_in
return func(self, *args, **kwargs)
File "beddit/client.py", line 151, in get_user
return User(r.json())
File "beddit/user.py", line 18, in init
self.weight = float(response_object['weight'])
TypeError: float() argument must be a string or a number

u = client.get_user(1)
Traceback (most recent call last):
File "", line 1, in
File "beddit/client.py", line 15, in _is_logged_in
return func(self, *args, **kwargs)
File "beddit/client.py", line 153, in get_user
raise BedditClient.APIError(r.json()['description'])
beddit.client.APIError: Permission denied

@peterrot Thank you for reporting the issue.
I apologize to make you inconvenience.
I'll fix this ASAP.

Hi,
I would like to ask you if that usage of a function is supposed to be right?
u = client.get_user()
print u.id
print u.email
print u.name
print u.date_of_birth
print u.sex
print u.weight
print u.height
print u.sleep_time_goal
print u.tip_audiences
print u.created
print u.updated

Now I found the error - just add additional if statements to check if parameters do actually exist in User  init function, before you assign them (in my case those parameters like response_object["weight"] and response_object["height"] were not defined), so you can just add if like this:
if response_object['weight']:
self.weight = float(response_object['weight'])
if response_object['height']:
self.height = float(response_object['height'])
or something similar..
Thank you very much ;)

@peterrot
Sorry for my late reply.
I checked your situation.
It seems to work correctly.
beddit.client.APIError: Permission denied

You can only get the user information of yours or you permitted to access.
You seem not to have permission to access user 1.
See this documentation for detail.
https://github.com/beddit/beddit-api/blob/master/3_1-UserResources.md
If you'd like to get your own profile, you don't pass the user_id argument.
import os
from beddit.client import BedditClient

client = BedditClient(os.environ.get('BEDDIT_USERNAME'), os.environ.get('BEDDIT_PASSWORD'))

user = client.get_user()
print(user.name)
print(user.height)
print(user.weight)


Add Path#each_part iterator

This PR adds a new overload of Path#each_part that returns an iterator.

Nice demo for why the compiler should support generating such iterators automatically, but also why it's so difficult.

@crystal-lang/crystallers Would be great to get reviews on this because I have a follow-up waiting =)


Emitting usage events based on the terms consumption

The doc states that I should only event usage events when: "the consumption exceeds the included quantities set for the term selected by the customer, your service will emit usage events to Microsoft."
For a monthly metered subscription, the subscription api does not include the term start and end date though:
{ term: { termUnit: "P1M"} }
If I offer 25 widgets in my base monthly price, and then charge $1 per widget over 25 I need to track the excess during the term begin and end date, and then report the usage event when it is exceeded in that period.  But if the subscription from this SaaS api does not tell me the term date, what should the time boundary be?

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

ID: d5ffd9ad-9082-53ca-c9d8-ddfed292a048
Version Independent ID: 9de28f1d-9e27-11e9-652c-483d36deced0
Content: Marketplace metering service APIs - FAQ
Content Source: articles/marketplace/partner-center-portal/marketplace-metering-service-apis-faq.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @MaggiePucciEvans
Microsoft Alias: evansma

#reassign:dsindona

@brycepolly sorry for the delayed response. Use the get subscription API to get the term start and end dates for the SaaS subscription. The base amount and the overage will be calculated for this period.
Please let me know if you have additional questions. #please-close


files licensed under GPL

Dear Julian,
I noticed several files licensed under GPL, e.g.
https://github.com/anyoptimization/pymoo/blob/main/pymoo/vendor/hv.py
https://github.com/anyoptimization/pymoo/blob/main/pymoo/cython/vendor/defs.h
Under my understanding, these GPL files are not compatible with the Apache license of pymoo project. If these GPL files are included, the pymoo project's license has to be GPL instead of Apache. Please check https://www.apache.org/licenses/GPL-compatibility.html
Is there any way to get rid of these files?

Thank you for bringing this up. I was not aware that there is a license issue.
Let me contact the authors of the packages and figure something out. Otherwise, I will have to remove them in the next version.

vendor/hv.py: I have been in touch with Simon Wessing regarding the license problem. pymoo has now official permission to include his implementation.
vendor/defs.h The package the source code was from does not exist anymore and has been taken offline. Only two methods of the header files were necessary anyway, so the dependency could be resolved entirely.
Thank you for raising this issue!


Sprint 2 Issue

Link to page : https://shriya1401.github.io/shriyap_2025/

Assignment
Points
Grade
Evidence

Pull Request (Integration)
1
0.85
We had a pull request our project a few times but after incorporating the comments Mr.Mortenson left, we got our pull request to get pulled.

Relevancy Checklist (Peer)
2
2

Lesson Style (Period)
1
0.85
We initially had a good style but after the theme got changed for the whole website our theme got messed up, when we saw that we should've immediately fixed it. But later on after the comments, Mr.Mort left about our formatting we fixed it making it more visible, as for some of the java code we couldn't see it.

Homework, Popcorn Hacks
1 x 5
0.9
I spent a lot of time making the homework and popcorn hacks to make sure they were unique and had a different twist to it compared to others, such as additional uses in the code above what they asked me to do.

Individual Contribution
1
0.89
I contributed to making the 3.5 of the jogi

Personal Notebooks / Blogs
1
0.9

Total
11

Skill
Points
Grade
Evidence

Work Habits (Analytics)
1
0.85

Team Planning (Issue)
1
1

Presentation Memories
1
1

Grading and Feedback
1
1

Beyond Perfunctory
1

Lesson Review - Topic Relevancy (Student Review)

The use of the dropdowns is very nice, and can help you when studying college board topics. I also like your treasure hunt game which was original.

Hi Shriya, I liked the short descriptions you added for your lesson buttons, but I think you could make it a bit more readable by changing the positioning and size a bit. Great job on the game displaying your knowledge from the lessons.

Created Home Page buttons with hovers.  Created hw and descriptions.   3.6 trivia game and dictionary is favorite.  Really big NightHawk on screen ?   Discussed a lot of code logic.

I really like your homepage, I think that the website logo might be a little too big but the homepage format is really nice. I also like that you combined your hacks with a summary of the lesson, that seems helpful. I like how you combined lessons with conditionals to use multiple topics in one hack. Treasure hunt game seems fun, looks pretty complex!

Your page looks different from all the other pages in our class. It had different colors for each button. One thing you need to fix are the numberings on the buttons, as there is no 3.11

I thought it was really cool how you added a part to your code so that a description would pop up over each module when you hovered over it.
You also discussed your favorite homework hacks and such, talking about a game within the dictionary model and how it worked.
I also liked your treasure finding game.

Hi Shriya,
I liked how you had button for each of the popcorn hacks and lessons. I also liked how you went over each hack that you felt like really resonated with you and a final project that helps you find treasure.

Hi Shriya, I like the buttons that have a summary when you hover over them.

I really like the theme of the page and the material that you have included. There is an issue with your code though because your Nighthawk logo is too big so you could fix that by reformatting your page but I really like the themes. I like the project done too.

She has a table that routes to the main blogs and adds a summary of what she learned from each lesson. Also made a cumulative project, that has game to find the treasure.

Shriya made a button that had floating summaries when you hovered over it. The summaries had code examples with notes here and there, which was cool. She talked about her favorite hacks, and how it helped her learn. She also had a final project.  In the future, I would recommend putting your homepage in the blogs, and formatting the submenu a little better.

I liked how your navigation bar with the drop-down allowed for definitions to show up. I also liked how you had little summaries for each part and fully used data abstraction (our lesson :) ).

Changed her buttons to be colorful and made the lessons. favorite popcorn hack she has done is in 3.6 creating a dictionary. also used if/else statements in her popcorn hack. overall she thinks it has been beneficial because with data extraction.

The dropdown menu is good. The formatting of the page and the application of your hacks in your treasure finding game was very detailed and nice.

made a float button and had various summaries of her code with notes, overall really good

I like how you have the buttons that take you to each blog. I also like the dropdown menu, however there is no lesson 3.11 so you probably want to fix that.

I like your buttons for popcorn hacks and homeworks and your design.


Add podspec

Removes the need to install/link 'Smooch' pod separately when using CocoaPods.

Thanks for the PR!

Cool, fixed the comments!

Hmm, another issue with this approach is that the react-native-smooch Pod target includes the ios/SmoochManager.h and .m files, so when you build you get an error: RCTBridgeModule.h file not found
Is there a simple way to fix that?

Oh, i'm not seeing that... In our Podfile we have the following:
pod 'React', :subspecs => [
      'Core',
      'RCTImage',
      'RCTNetwork',
      'RCTPushNotification',
      'RCTText',
      // ...
    ], :path => '../node_modules/react-native'

Having this is assumed, there are some references to this in the official docs:
http://facebook.github.io/react-native/releases/0.31/docs/troubleshooting.html#missing-libraries-for-react
http://facebook.github.io/react-native/releases/0.31/docs/integration-with-existing-apps.html#react-native-framework
but it doesn't hurt to add some note about it I guess.
I think we might want to keep something like this in the podspec too, which I saw in another podspec:
s.dependency   'React/Core'

Yeah the Podfile react requirements should be mentioned, but I can take care of that. Thanks again!


⚠ Bofrost.at has degraded performance

In 2994b4d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1982 ms

Resolved: Bofrost.at performance has improved in e9f68d9.


Add support for object input values

See 2.9.8 and the validation rules in 5.6.2-4.

Fixed in #41.

Fixed in #41.


extra missing packages (Ubuntu 16.04)

(before installing nautilus):

sudo apt-get install libffi-dev

(must be installed before graphene); if not installed before graphene, graphenelib must pip installed:

sudo apt-get install libssl-dev

awesome! thanks - i'll add those two packages to the docs

Also, the docker commands need to be run as root user

ah, that's an issue with docker in general - if you add your current user to the docker group then you should be good to go


Update .editorconfig with correct extensions

This is a holdout from when Rome was written with Flow and used the .js extension.
Thanks @RReverser for noticing! https://twitter.com/RReverser/status/1261841111498326021

Ah that’s right. Could you make a PR? Otherwise I can make the change when I can.
On Sat, May 16, 2020, at 10:24 PM, halfnelson wrote:

@halfnelson commented on this pull request.

In .editorconfig https://github.com/romejs/rome/pull/449#discussion_r426218800:

@@ -5,6 +5,6 @@ end_of_line = lf
trim_trailing_whitespace = true
insert_final_newline = true

-[.js]
+[{.ts,*.json}]

There are a few .cjs files under scripts that might gain from this setting too

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/romejs/rome/pull/449#pullrequestreview-413128137, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGQNUEWP3MVQSAVQJWDBW3RR5YITANCNFSM4NDFSCQQ.

Done, #454
On Sun, 17 May 2020, 3:26 pm Sebastian<EMAIL_ADDRESS>wrote:

Ah that’s right. Could you make a PR? Otherwise I can make the change when
I can.
On Sat, May 16, 2020, at 10:24 PM, halfnelson wrote:

@halfnelson commented on this pull request.

In .editorconfig <
https://github.com/romejs/rome/pull/449#discussion_r426218800>:

@@ -5,6 +5,6 @@ end_of_line = lf
trim_trailing_whitespace = true
insert_final_newline = true

-[.js]
+[{.ts,*.json}]

There are a few .cjs files under scripts that might gain from this
setting too

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub <
https://github.com/romejs/rome/pull/449#pullrequestreview-413128137>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/AAGQNUEWP3MVQSAVQJWDBW3RR5YITANCNFSM4NDFSCQQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/romejs/rome/pull/449#issuecomment-629745622, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEGS5FC6N46CBHPVGQGKRTRR5YQLANCNFSM4NDFSCQQ
.


DASHBUILDE-96: DB2 does not always support the LIMIT clause

The SQL provider test  suite has been executed successfully on DB2 9.7 and 10.5 so there is no need for extra tests in this case.

@psiroky jenkins is failing. It seems an infrastructure related issue though. Can you double check please?

@dgutierr I am not exactly sure what happened, looks like the Jenkins master lost connection to the slave. Always a good idea to re-trigger the job in cases like this (you can do that here by commenting "jenkins retest" or directly in the Jenkins UI by clicking on the failed build and then clicking on "Retry").

+1


Node local storage overrides all the nodes when swapping profiles

Related objects:
https://github.com/nemgrouplimited/symbol-desktop-wallet/blob/main/src/services/NodeService.ts#L40
https://github.com/nemgrouplimited/symbol-desktop-wallet/blob/main/src/core/database/storage/NodeModelStorage.ts

The list is re-saved resetting any manually added node in another profile (with different network type)
The list only filters by network type, if in the future the wallet handles multiple private network types (with different generation hashes), the list won't work

The solution would be to migrate (code and data) NodeModelStorage from VersionedObjectStorage to VersionedNetworkBasedObjectStorage, so only the nodes related to a network generation hash is loaded and stored leaving the other nodes untouched.

For point #1 mentioned in the previous comment by @fboucquez
User Story: “As a node owner, I want my edited files to remain untouched unless absolutely necessary when working with profiles or any other object that could force a re-write of the files, so I can continue to have my stored preferences or changes I have made on those files.”
Acceptance Criteria:

As a node owner creating a new profile, I don’t want to lose any configuration I have changed in my node
As a node owner, I don’t want th files NodeService.ts and NodeModelStorage to be automatically overriden unless absolutely necessary.

For Point #2
It will be tackled on task #752 .

Fixed in integration branch.


fix: quote accountid in configmap

Fixes #324

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 40.97%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3439

Relevant Lines:
8394

💛  - Coveralls

thank you


Correct str_replace using first argument an array

Since v0.7.0 (work with this one), it's not possible to use the function
str_replace(Array, Array|String, String)

in zephir (see bug [#1055] for the latest one). If the first argument is an array in the function str_replace, the return was always NULL. The cause is the commit [c763915fc619151ca5498c9377f8df0da51f5536], solving a memory leak. I think, this is a C pointer error. This work correctly for me, in master after doing this modification).

Can you push a test for it?

@lolo32

Fixed in development.


Enabling PAT authentication support

After reviewing a few others' attempts at enabling PAT authentication to Azure DevOps, I thought I would try my hand at it. There were some quirks that I've worked out in this version, and I hope it is useful for others.
The main difference I've made is that when a GIT_TFS_PAT environment variable is present, it bypasses any chances at user interactive authentication. This helps with debugging things like automated scripts, because you won't get false negatives as user prompts when the PAT env var is detected and it's possible bad, the authentication will simply just fail.
A note for folks wanting to use this build directly
So because this repo has fallen out of active support, you may be wondering if you can download this branch, build it, and use this version for your own pipelines/automation scripts. Well, you can! I didn't even bother with the build.ps1/cake scripts. All I did was download this repo, open the GitTfs.sln in Visual Studio 2022, switch to this branch, rebuild the solution, and voila! You should have the output under src/GitTfs/bin/...
Note: If you do change anything in the common code, which is shared code amongst all the other VS-versioned projects, you'll need to rebuild all to have those VS-versioned projects build and push their binaries to the src/GitTfs/bin/... folder (it's a post-build step on each of those projects). This was a major source of my troubles when testing this code out, as I didn't rebuild the right projects during testing. Just be safe and rebuild everything.
About migrating from TFVC(TFS) to Git
I ended up using this version of git-tfs to enable an Azure Pipeline to automate conversion/migration of TFVC(TFS) code into Git repositories. It works great!

Thanks.

Thank you! Happy to see the active support! 😄


BUG: Mistake in the signup page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hey there is the little issue there i saw while signing in there is semi colon and bracket which not correctly closed so appearing in the screen which is the mistake i want to fixt that.

Please assign this to me .
Thank You!
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok fix that


Multiple Imputation - Continuous Data MNAR	- Reference-Based Imputation/Sequential Methods

In-progress, possibly finish by end of March 2024

[ ] find dataset
[ ] identify packages and functions
[ ] produce an example
[ ] document
[ ] PR


Need to keep the Ansible log of first reconcile

Feature Request
Describe the problem you need a feature to resolve.
Ansible Operator keep reconciling and the log of first reconcile maybe overwritten, but for problem definition, the log of the first reconcile is very important and some error may only exist in first reconcile.
Describe the solution you'd like.
Keep the log of first reconcile, don't overwrite it.

@YanFenGuo do you have repro steps for this scenario?

@YanFenGuo do you have repro steps for this scenario?

Yes see if this description make it clear, I'm using Ansible operator-sdk 0.17.2, for watch.yaml, I set reconcilePeriod: 5m so Operator will continue reconciling.
When I check Ansible log under /tmp/ansible-operator/runner/<group>/<version>/<kind>/<namespace>/<name>., I found it only keep logs of latest 20 reconcile? What I'm expecting is we can keep the log of first reconcile, don't overwrite it.

/assign @asmacdo

We've implemented this in our own Operator code, so this is not urgent for us now.


findDOMNode

Page: https://id.react.dev/reference/react-dom/findDOMNode

Hi @resir014 & @zainfathoni,
I'd like to continue @afrianjunior's effort on translating this page, is it okay to do that?
Thank you!

Yes, sure, please go ahead. Thanks!


add data-type(go)

I am adding go data-types for the Hacktoberfest2022

@nirajpdn great to know!!
thank you so much; love to contribute!!


Add a new image label if it is docker schema 1

Resolve https://github.com/containerd/containerd/issues/9202
This PR adds a new image label if it is docker schema 1. With the new label, users can find the docker schema1 usage by running ctr images
Local Test
$ sudo /usr/local/bin/crictl pull docker.io/progrium/stress:latest
$ sudo ctr -n k8s.io images ls
REF                                                                     TYPE                                       DIGEST                                                                  SIZE     PLATFORMS   LABELS
docker.io/progrium/stress:latest                                        application/vnd.oci.image.manifest.v1+json sha256:fe33932960bfb41cab97a147e7c38e0c787424dd374deba976dd6e9803fb3666 86.5 MiB linux/amd64 io.containerd.image/converted-docker-schema1=sha256:e34d56d60f5caae79333cee395aae93b74791d50e3841986420d23c2ee4697bf,io.cri-containerd.image=managed
sha256:9d98b4a6b6d1fe7346174111650132fd189b9153fb48a94a7b898f55f9d15a17 application/vnd.oci.image.manifest.v1+json sha256:fe33932960bfb41cab97a147e7c38e0c787424dd374deba976dd6e9803fb3666 86.5 MiB linux/amd64 io.cri-containerd.image=managed

$ sudo bin/ctr image pull docker.io/progrium/stress:latest
$ sudo bin/ctr image ls
REF                              TYPE                                       DIGEST                                                                  SIZE     PLATFORMS   LABELS
docker.io/progrium/stress:latest application/vnd.oci.image.manifest.v1+json sha256:fe33932960bfb41cab97a147e7c38e0c787424dd374deba976dd6e9803fb3666 86.5 MiB linux/amd64 io.containerd.image/converted-docker-schema1=sha256:e34d56d60f5caae79333cee395aae93b74791d50e3841986420d23c2ee4697bf

It'd be nice to have a unit test for this, but since there are no existing tests for pull.go I don't consider that blocking.

If there is a "well-known" docker schema 1 for integration test, I probably can add a new test case in https://github.com/containerd/containerd/blob/main/integration/containerd_image_test.go

/retest

/ok-to-test

@qiutongs It looks like the Kubernetes project has a schema 1 image we can use: https://explore.ggcr.dev/?image=registry.k8s.io/busybox@sha256:4bdd623e848417d96127e16037743f0cd8b528c026e9175e22a84f639eca58ff&mt=application%2Fvnd.docker.distribution.manifest.v1%2Bprettyjws&size=3205
/cc @dims

@qiutongs It looks like the Kubernetes project has a schema 1 image we can use: https://explore.ggcr.dev/?image=registry.k8s.io/busybox@sha256:4bdd623e848417d96127e16037743f0cd8b528c026e9175e22a84f639eca58ff&mt=application%2Fvnd.docker.distribution.manifest.v1%2Bprettyjws&size=3205
/cc @dims

Great, let me add an integration test based on it

@qiutongs It looks like the Kubernetes project has a schema 1 image we can use: https://explore.ggcr.dev/?image=registry.k8s.io/busybox@sha256:4bdd623e848417d96127e16037743f0cd8b528c026e9175e22a84f639eca58ff&mt=application%2Fvnd.docker.distribution.manifest.v1%2Bprettyjws&size=3205
/cc @dims

Great, let me add an integration test based on it

Added a new test case in containerd_image_test.go

/retest

/retest

This only works for the prow jobs.  Most of containerd's CI is via GitHub Actions.  I'll trigger a retest for you.

/retest

This only works for the prow jobs. Most of containerd's CI is via GitHub Actions. I'll trigger a retest for you.

Thank you. And I don't think the failed windows integration test is caused by this PR.
mv: cannot remove 'C:/Windows/Temp/test-integration/containerd.log': Device or resource busy

=== RUN   TestContainerdImageWithDockerSchema1
    containerd_image_test.go:244: make sure the test image doesn't exist in the cri plugin
    containerd_image_test.go:251: pull the image into containerd
time="2023-10-13T07:57:01Z" level=info msg="apply failure, attempting cleanup" error="failed to extract layer sha256:2c84284818d186d88a16ac7fa731d4b71ba69ecfe11b4ce00413366833cb2403: hcsshim::ProcessBaseLayer \\\\?\\C:\\ProgramData\\containerd\\root-test\\io.containerd.snapshotter.v1.windows\\snapshots\\114: The system cannot find the path specified.: unknown" key="extract-724054400-oMXA sha256:2c84284818d186d88a16ac7fa731d4b71ba69ecfe11b4ce00413366833cb2403"
    containerd_image_test.go:254:
        	Error Trace:	D:/a/containerd/containerd/src/github.com/containerd/containerd/integration/containerd_image_test.go:254
        	Error:      	Received unexpected error:
        	            	failed to unpack image on snapshotter : failed to extract layer sha256:2c84284818d186d88a16ac7fa731d4b71ba69ecfe11b4ce00413366833cb2403: hcsshim::ProcessBaseLayer \\?\C:\ProgramData\containerd\root-test\io.containerd.snapshotter.v1.windows\snapshots\114: The system cannot find the path specified.: unknown
        	Test:       	TestContainerdImageWithDockerSchema1
--- FAIL: TestContainerdImageWithDockerSchema1 (1.07s)

This image is not a multi-platform image and won't work on Windows.  Can you skip the test on Windows?


DRILL-5664: Enable security for Drill HiveStoragePlugin based on a co…

…nfig parameter
     Change to enable/disable HiveStoragePlugin security configuration based on Drill's "security.storage_plugin.enabled" configuration. This will help to open secure channel between Drill's HiveClient and HiveMetastore

More details in JIRA DRILL-5664.
@parthchandra / @ppadma - Can you please help to review this change ?

I have a question.  The user can change the storage plugin configuration itself instead of enabling/disabling this option.  what is the use of this option ?

@ppadma -

For the ease of use. Once the security is being configured for Drill then it's better if all the config changes are done in single place (which is drill-override.conf) and then make sure that all communications are secure. With this option user doesn't have to explicitly go and edit the Storage Plugin configuration to enable security.
It will take care of enabling/disabling security both for active and inactive plugins.
This can be extended for handling security of all the storage plugin. Hence this config can be a single place to enable/disable security for all the plugins in one go rather than doing it manually for each and every plugin one by one.

What is the status of this PR? Appears that there was substantial discussion about whether the approach here is appropriate for Apache Drill. Had the PR been updated with an alternative? Or, should this PR be closed pending a revised solution?

closing this PR pending the revised version.


ActiveAdmin powered app performance

Disabling rails_footnotes decreased request execution time in AA powered app from 8-13 seconds to about 0,5
I have 5 registered resources
Run on rails 3.2.1; MRI 1.9.3-head;  ubuntu 11.10

It seems I'm experiencing this issue as well.

This is related to #102


add Machinist, newer Spire, re-add Breeze, run Scalikejdbc tests

breeze depends on spire, spire depends on machinist, so I did the three
together
the scalikejdbc thing is an unrelated bonus.

green run at https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/647/console
fyi @dlwh

fyi @non

fyi @non

Spire tests remain disabled. reported upstream at https://github.com/non/spire/issues/580

scalikejdbc's issue must be fixed by https://github.com/scalikejdbc/scalikejdbc/pull/566
pls check the latest revision.


HW-4: Add JS basics

Hello! I did the task, but the calculation result will be shown on the page (I find it uncomfortable to use alert() or console.log() for this purpose). Also I made the task for the star, hope that everything is great!
For the page i chose such design:

Also, as example, if I'll try to calculate something, I will receive such answers:

or

Thanks for the review! 🌷

Hey @rogerosha! Great design) Please fix two small issue


Adding Caribbean content, modern status

Addresses #149
Taking a break from US colonialism, adding Caribbean content. This handles the western island nations and territories, with no historical content yet, just from independence - or declared territorial status - to present. Will complete another PR for occupation history.
Signed-off-by: Scott Johnson<EMAIL_ADDRESS>

This is now complete, that is has all of the North American Caribbean islands and their modernity. The historical perspective will come in another pull request. Note that there are a few Caribbean islands that are considered part of South America (primarily Aruba, Bonaire, Curaçao and Trinidad and Tobago), and will be handled at the time of South American updates.

Sidenote: I have a strange design for a Caribbean vacation after this 🤣


Prevent encoding of IPv6 brackets in URL from route helper

Solving the issue of: IPv6 URLs with square brackets are not supported by Laravel's routing system
Adding test before change, and running the same test after changes:

The test failure is not related to this PR:

Illuminate\Tests\Support\SleepTest::testItSleepsForSecondsWithMilliseconds
Failed asserting that 1.5307338237762451 matches expected 1.5.

D:\a\framework\framework\tests\Support\SleepTest.php:39

This is a breaking change. Most people do want to encode braces. For example applications running behind aws cloudfront will break without the en coding.

I think it is pretty unusual to want to use ipv6 ips directly as hostnames (not the same as having an AAAA record).

This is a breaking change. Most people do want to encode braces. For example applications running behind aws cloudfront will break without the en coding.

Reading the RFC - Uniform Resource Identifier (URI): Generic Syntax, i can undertand you, the square brackets must be encoded for the URI part of a complete URL.
But in the same RFC3986 in section 3.2.2, he said with the new IPv6 identifier notation using "[" and "]" to identify the IP literal of host, we cannot encode these characters.

Maybe we can modify the method of encode the entire URL at rawurlencode for two encodes, not encoding square brackets for literal IPs and encoding for the URI as described on the RFC3986.

Only brackets as part of the hostname should not be encoded. The problem is that your changes applies across the whole URL. My point is that it's better to get this wrong for hostnames because this affects almost nobody (possibly you are the first ever person to have encountered this issue) vs the issues this fix will in turn cause. I don't object to a proper fix, but this isn't a proper fix, unless I'm missing something?

I think it is pretty unusual to want to use ipv6 ips directly as hostnames (not the same as having an AAAA record).

In my use case, its not just simple like that...
My application is self hosted by the customers and they cannot trust in the domain name resolutors for access their system, given that systems not be able have any time of downtime (like SLA of 99.99999% of acessibility a year). Them have a few problems with can occore to cause a inacessibility:

DNS server going down => the web server is up and running, but the name resolver is down.
The route for server in IPv4 is congested or down (maybe a DDoS attack via IPv4) and have only up the IPv6 network.
...

Only brackets as part of the hostname should not be encoded. The problem is that your changes applies across the whole URL. My point is that it's better to get this wrong for hostnames because this affects almost nobody (possibly you are the first ever person to have encountered this issue) vs the issues this fix will in turn cause. I don't object to a proper fix, but this isn't a proper fix, unless I'm missing something?

I wasnt the first person hahaha, @kronos1345 on IPv6 URLs with square brackets are not supported by Laravel's routing system that was.
And yes, now reading the RFC, I realize the correction is wrong and could cause problems, I'll try to modify it to cover the other use cases, and to be right with the RFC.

The route for server in IPv4 is congested or down (maybe a DDoS attack via IPv4) and have only up the IPv6 network.

That would not cause a problem, because the hostname would just be a string.

The route for server in IPv4 is congested or down (maybe a DDoS attack via IPv4) and have only up the IPv6 network.

That would not cause a problem, because the hostname would just be a string.

Not all of my customers use domain names for their internal systems, or havent the two A and AAAA records, so they directly use IPv4 and IPv6 addresses to access the systems in cause of problems, and if the route for the system using IPv4 address is unavailable, the last option to use is IPv6 address directly using the syntax of "http://[IPv6Address]".
Idk if it would be a framework guideline to just use it with domain names.


Only parse REQUIRE file in binder (if exists)

If the binder folder is in a package directory, then the existing setup will call using on the package requirements instead of binder/REQUIRE.  This can run into some issues if the package requires a specific version of Julia:
ERROR: ArgumentError: Module julia not found in current path.
Run `Pkg.add("julia")` to install the julia package.

This is awesome! ty :)

Congratulations on your first patch getting merged! Hopefully one of many more to come to help us solidify our Julia support (and elsewhere too :D)


Add cameraconnect

Cameraconnect takes pictures from Industrial Cameras using Cognex and GenICam protocol and sends them to MQTT.
Requirements

connects to Cognex and GenICam cameras and retrieves images
sends MQTT messages (see also #261 and #262)
can be written in Python, prefered language is golang (for higher runtime stability)
will be moved into a Docker container.
logs to the terminal

Definition of done

The service can handle up to 10 images per second, preferably 100
It runs across multiple device restarts and at least for 48h continiously
When it encounters a fatal error it will crash and will then be automatically restarted by Kubernetes
The folder with all the source code will be called /cameraconnect and the Dockerfiles will be in /deployment/cameraconnect
An automated build pipeline using Docker will be setup
There is a documentation in the docs similar to https://docs.umh.app/docs/developers/factorycube-edge/sensorconnect/
It must be entirely configurable via environment variables
There are Kubernetes files available for factorycube-edge and it is included in the Helm chart

This issue needs to be split up into multiple smaller issues during the development sprint.

Hi, thanks for the great summary. I added some minor changes. Looking forward to our developing session ;)
Cameraconnect takes pictures from Industrial Cameras using  GenICam protocol and sends them to MQTT.
Requirements

connects to cameras manufactured by Basler and AlliedVision which are using the GenICam standard and retrieves images
sends MQTT messages (see also Rework data model for image and image recognition #261 and Rework data model for image and image recognition #262)
acquiring and sending images can be triggered either via MQTT message or automatically by a given frequency
can handle rgb and mono8 cameras
can handle resolutions up to 4000x4000
can be written in Python, prefered language is golang (for higher runtime stability)
will be moved into a Docker container.
logs to the terminal

Definition of done

The service can handle up to 10 images per second, preferably 100
It runs across multiple device restarts and at least for 48h continuously
When it encounters a fatal error it will crash and will then be automatically restarted by Kubernetes
The folder with all the source code will be called /cameraconnect and the Dockerfiles will be in /deployment/cameraconnect
An automated build pipeline using Docker will be setup
There is a documentation in the docs similar to https://docs.umh.app/docs/developers/factorycube-edge/sensorconnect/
It must be entirely configurable via environment variables
There are Kubernetes files available for factorycube-edge and it is included in the Helm chart
This issue needs to be split up into multiple smaller issues during the development sprint.

Variables provided by the module user

Serial number of camera (to decide which camera gets picked if multiple are available)
resolution of image (can be used to crop the image)
offset of image in x and y axis (in case the camera should only send a specific section of the image)
exposure time
trigger frequency (for self-timer)
color mode (mono8 or rgb)


"green_goal, yellow_goal, red_goal and blue_goal" interaction dont work.

Impossible place a ball on goal.

Can send Screenies?

The bug is:
Impossible place the ball on goal.

They are with right interaction at catalogue?

Yes, i am no noob lool

@Anthologique i know that. More respect on issues. They're public ;)
I only will check any Yupi bug in June with @TheDoct0r11, since we're under faculty semester.

Remembering @Anthologique that these errors are from NIO branch, and aren't being supported. Since the current development branch is LINUX, also the LINUX branch isn't ready for usage.
We will update LINUX branch starting at June.

No, i use only "Yupi Emulator 1.1.0 stable release version".
When You update this version ?
(https://github.com/sant0ro/Yupi/releases/tag/1.1.0-stable)

As i said The 1.1.0 stable release version is from NIO branch.
We're developing the new release in LINUX branch, that is far to be done.


Will not build on Heroku with Node 0.12.2

I'll give it a stuv here, although I have successfully manage to build mmmagic on my local 0.12.2 node environment, it isn't the case when deploying to Heroku.
While installing the following compile errors occurred:
       ../deps/libmagic/src/cdf.c: In function 'cdf_read':
       ../deps/libmagic/src/cdf.c:303:2: warning: implicit declaration of function 'pread' [-Wimplicit-function-declaration]
         if (pread(info->i_fd, buf, len, off) != (ssize_t)len)

         CC(target) Release/obj.target/libpcre/deps/libmagic/pcre/pcre_compile.o
       ../deps/libmagic/pcre/pcre_compile.c: In function 'add_to_class':
       ../deps/libmagic/pcre/pcre_compile.c:3487:30: warning: 'od' may be used uninitialized in this function [-Wmaybe-uninitialized]
              else n8 += add_to_class(classbits, uchardptr, options, cd, oc, od);

         COPY Release/pcre.a
         CXX(target) Release/obj.target/magic/src/magic.o
       ../src/magic.cc:18:5: error: 'uv_work_t' does not name a type
            uv_work_t request;
            ^
       ../src/magic.cc:39:33: error: expected class-name before '{' token
        class Magic : public ObjectWrap {
                                        ^

The full install logs can be found in this gist.

Can you try installing the master branch instead of mmmagic 0.3.12? If that works, I will publish a new version.

yes 0.3.13 worked! Thank you @mscdex


Transparency slider and interactive mode does not work for centroid-and-size-only annotations

In the screenshot below, the annotation transparency slider is set to zero, yet the centroid-and-size-only (CASO) elements are not affected and retain their "native" fill value.

A related problem occurs when I activate interactive mode and hover over an the element text -- it fades everything except the element itself (expected behavior) but fails to fade out CASO elements (the bug).
If the simplest fix for this is to not show transparency for CASO elements, that's totally OK. The issue of interactive mode for CASO elements remains withstanding even if this simple fix is adopted, though.

This is a more complex issue that it initially appeared.  We have some code that implies we intended to not highlight an element when there were too many of them to remain performant (default is 5000), but, when we load centroids the number of elements that AREN'T centroids is often below this threshold which leads to inconsistent behavior.  One solution would be to prevent highlighting annotations if part of the elements are shown as centroids.  The other option would be to see if we can improve the performance of toggling large numbers of opacities and then do it for centroid annotations, too.


Feature Request: Include custom .cfg menu entries

I have a custom.cfg for custom menu entries, which also calls other custom cfgs, which have their own menu entries.

For example, in custom.cfg I have the following, which calls liveiso.cfg and gpu-switcher.cfg:
menuentry 'Live ISOs' --class iso {
	insmod part_gpt
	insmod btrfs
	search --no-floppy --fs-uuid --set=root XXXXXX-XXXXXXXXX-XXXXXXXXXXXXXXX
	configfile<EMAIL_ADDRESS>}
menuentry 'GPU Switcher' --class nvidia {
	insmod part_gpt
	insmod btrfs
	search --no-floppy --fs-uuid --set=root XXXXXX-XXXXXXXXX-XXXXXXXXXXXXXXX
	configfile<EMAIL_ADDRESS>}

There is also one for btrfs snapshots that's generated on its own (although that's not custom.)
Anyway, if I may, I'd like to request other .cfg file menu entries show up in the grub-reboot-picker as well, please.

Here are the items in grub.cfg that make the magic happen for custom configs and snapshots:
### BEGIN /etc/grub.d/41_custom ###
if [ -f  ${config_directory}/custom.cfg ]; then
  source ${config_directory}/custom.cfg
elif [ -z "${config_directory}" -a -f  $prefix/custom.cfg ]; then
  source $prefix/custom.cfg
fi
### END /etc/grub.d/41_custom ###

### BEGIN /etc/grub.d/41_snapshots-btrfs ###
submenu 'Manjaro Linux snapshots' {
    configfile "${prefix}/grub-btrfs.cfg"
}
### END /etc/grub.d/41_snapshots-btrfs ###

I do get the Majnaro Linux snapshots sub-menuentry in grub-reboot-picker, however, I don't get whatever is located in grub-btrfs.cfg or any of the custom.cfg entries.


completed assignment-1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Completing Assignment-2
What did you learn from the changes you have made?
Working with neural networks
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
NA
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
NA
How were these changes tested?
NA
A reference to a related issue in your repository (if applicable)
NA
Checklist

[X] I can confirm that my changes are working as intended

Hello @kunzhi-yu I've run and completed the assignment, can you please re-check?


Intermittent unit test failure RevChainParameterParseTest.testValidReverseChainWithDateSearchParameter

Describe the bug
Build threw the following failure:
[ERROR]   RevChainParameterParseTest.testValidReverseChainWithDateSearchParameter:617 expected [2021-11-12T11:43:49.999999Z] but found [2021-11-12T11:43:49Z]

Environment
Which version of IBM FHIR Server? 4.10.0-SNAPSHOT
To Reproduce
Steps to reproduce the behavior:

git clone
mvn clean install -f fhir-parent

Expected behavior
Unit test should pass.
Additional context
Happened only once, but indicates a rounding issue in the code or unit test that should be investigated.

The issue occurs when the value of Instant.now() has a milliseconds value of exactly 0 (so roughly a 1 in 1000 chance).  In which case, the generated date string in the query will not include milliseconds. Therefore, the search upper-bound will be that second plus 999999 nano seconds.  As long as Instant.now() has a non-0 milliseconds value, the test as written will succeed.

Same issue can happen with:
ChainParameterParseTest.testValidChainWithDateSearchParameter()

Since this is a Unit test we can skip QA as it is hit on CICD.
@punktilious I'll leave you to close


Build FaceMesh tutorial scene to iOS not showing face landmarks

Plugin Version or Commit ID
v0.12.0
Unity Version
2022.1.10f1
Your Host OS
macOS Ventura 13.1 -> iOS 16.0
Target Platform
iOS
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
No response
[iOS Only] XCode Version
14.3.1
Command Sequences
Thank you for the amazing plugin!! The tutorial and sample scene perform really well on Unity!
Currently, I am trying to build the FaceMesh from tutorial to iOS. I looked into the previous issues and instructions, and changes the Config Asset to face_mesh_gpu for iOS building, and changed some of the code. The current FaceMesh class looks like this:
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

using Stopwatch = System.Diagnostics.Stopwatch;

namespace Mediapipe.Unity.Tutorial
{
  public class FaceMesh : MonoBehaviour
  {
    [SerializeField] private TextAsset _configAsset;
    [SerializeField] private RawImage _screen;
    [SerializeField] private int _width;
    [SerializeField] private int _height;
    [SerializeField] private int _fps;
    [SerializeField] private MultiFaceLandmarkListAnnotationController _multiFaceLandmarksAnnotationController;

    private CalculatorGraph _graph;
    private ResourceManager _resourceManager;

    private WebCamTexture _webCamTexture;
    private Texture2D _inputTexture;
    private Color32[] _inputPixelData;

    //gpu add-on
    //private NativeArray<byte> _buffer;
    private GpuResources _gpuResources;

    private IEnumerator Start()
    {
      if (WebCamTexture.devices.Length == 0)
      {
        throw new System.Exception("Web Camera devices are not found");
      }
      var webCamDevice = WebCamTexture.devices[0];
      _webCamTexture = new WebCamTexture(webCamDevice.name, _width, _height, _fps);
      _webCamTexture.Play();

      yield return new WaitUntil(() => _webCamTexture.width > 16);

      _screen.rectTransform.sizeDelta = new Vector2(_width, _height);

      _inputTexture = new Texture2D(_width, _height, TextureFormat.RGBA32, false);
      _inputPixelData = new Color32[_width * _height];

      _screen.texture = _webCamTexture;

      _resourceManager = new StreamingAssetsResourceManager();
      yield return _resourceManager.PrepareAssetAsync("face_detection_short_range.bytes");
      yield return _resourceManager.PrepareAssetAsync("face_landmark_with_attention.bytes");

      var stopwatch = new Stopwatch();

      _graph = new CalculatorGraph(_configAsset.text);
      //gpu
      _gpuResources = GpuResources.Create().Value();
      _graph.SetGpuResources(_gpuResources).AssertOk();
      //output stream
      var multiFaceLandmarksStream = new OutputStream<NormalizedLandmarkListVectorPacket, List<NormalizedLandmarkList>>(_graph, "multi_face_landmarks");
      multiFaceLandmarksStream.StartPolling().AssertOk();

      var sidePacket = new SidePacket();
      sidePacket.Emplace("num_faces", new IntPacket(1));
      sidePacket.Emplace("with_attention", new BoolPacket(true));
      sidePacket.Emplace("input_rotation", new IntPacket(0));
      sidePacket.Emplace("input_horizontally_flipped", new BoolPacket(false));
      sidePacket.Emplace("input_vertically_flipped", new BoolPacket(true));
      sidePacket.Emplace("model_type", new IntPacket(0));

      _graph.StartRun(sidePacket).AssertOk();
      stopwatch.Start();

      var screenRect = _screen.GetComponent<RectTransform>().rect;

      while (true)
      {
        _inputTexture.SetPixels32(_webCamTexture.GetPixels32(_inputPixelData));
        var imageFrame = new ImageFrame(ImageFormat.Types.Format.Srgba, _width, _height, _width * 4, _inputTexture.GetRawTextureData<byte>());
        var currentTimestamp = stopwatch.ElapsedTicks / (System.TimeSpan.TicksPerMillisecond / 1000);
        _graph.AddPacketToInputStream("input_video", new ImageFramePacket(imageFrame, new Timestamp(currentTimestamp))).AssertOk();

        yield return new WaitForEndOfFrame();

        if (multiFaceLandmarksStream.TryGetNext(out var multiFaceLandmarks))
        {
          _multiFaceLandmarksAnnotationController.DrawNow(multiFaceLandmarks);
        }
        else
        {
          _multiFaceLandmarksAnnotationController.DrawNow(null);
        }
      }
    }

    private void OnDestroy()
    {
      if (_webCamTexture != null)
      {
        _webCamTexture.Stop();
      }

      if (_graph != null)
      {
        try
        {
          _graph.CloseInputStream("input_video").AssertOk();
          _graph.WaitUntilDone().AssertOk();
        }
        finally
        {

          _graph.Dispose();
        }
      }
    }
  }
}

And the Unity scene looks like this:

I successfully build the project to iOS through XCode and can run it on my phone. However, the landmarks are not showing although the camera is running normal.
I am very very new to coding, and also this is the first time I'm using XCode to build things on phone, so I am not so familiar with the output in XCode...not sure what to paste. Apologize if there are similar questions being asked before, but I tried my best to go through the issues and docs you provided for week T_T
Thank you so much for your help!!!!
Log
Unity console when running the app on phone:
MediaPipeException: MediaPipe Aborted, refer glog files for more details
Mediapipe.MpReturnCodeExtension.Assert (Mediapipe.MpReturnCode code) (at Packages/com.github.homuler.mediapipe/Runtime/Scripts/PInvoke/MpReturnCode.cs:48)
Mediapipe.CalculatorGraph..ctor (System.Byte[] serializedConfig) (at Packages/com.github.homuler.mediapipe/Runtime/Scripts/Framework/CalculatorGraph.cs:27)
Mediapipe.CalculatorGraph..ctor (Mediapipe.CalculatorGraphConfig config) (at Packages/com.github.homuler.mediapipe/Runtime/Scripts/Framework/CalculatorGraph.cs:31)
Mediapipe.CalculatorGraph..ctor (System.String textFormatConfig) (at Packages/com.github.homuler.mediapipe/Runtime/Scripts/Framework/CalculatorGraph.cs:33)
Mediapipe.Unity.Tutorial.FaceMesh+<Start>d__12.MoveNext () (at Assets/MediaPipeUnity/Tutorial/Official Solution/FaceMesh.cs:55)
UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) (at /Users/bokken/buildslave/unity/build/Runtime/Export/Scripting/Coroutines.cs:17)

Some output from XCode window:
Thread Performance Checker: Thread running at QOS_CLASS_USER_INTERACTIVE waiting on a thread without a QoS class specified. Investigate ways to avoid priority inversions
PID: 10990, TID: 1419922
Backtrace

WARNING: Logging before InitGoogleLogging() is written to STDERR
I20230811 16:14:09.857651<PHONE_NUMBER> gl_context.cc:342] GL version: 3.0 (OpenGL ES 3.0 Metal - 99)
I20230811 16:14:09.861171<PHONE_NUMBER> gl_context.cc:342] GL version: 3.0 (OpenGL ES 3.0 Metal - 99)
2023-08-11 16:14:09.870323+0800 MediaPipeUnityAll[10990:1420241] Created TensorFlow Lite delegate for Metal.
INFO: Created TensorFlow Lite delegate for Metal.
2023-08-11 16:14:09.870457+0800 MediaPipeUnityAll[10990:1420241] Initialized TensorFlow Lite runtime.
INFO: Initialized TensorFlow Lite runtime.
-> applicationWillResignActive()
-> applicationDidEnterBackground()

Additional Context
Thank you so much in advance!!!

Some output from XCode window:

Please share the complete log (not the excerpt).
I think some error logs are output.

By the way, in the near future, the MediaPipe Task API will be ported, making it much easier to perform the same processes.


Introduce CODEOWNERS

As explained in https://github.com/blog/2392-introducing-code-owners

[X] Have you followed the guidelines in our Contributing document?
[X] Have you created a JIRA and used it in the commit message?
[X] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[X] Have you built the project locally prior to submission with mvn clean install?

Probably needs to be enhanced to add the correct owners to the directories (eg. camel)


Code Difference in same version

both of this file is the same version (1.8.0).
but why there is so much difference between them ?
https://www.diffchecker.com/6NkYrNeV

What is the source of both?


Add vendor

@frjo, can we deploy this to test?

I think we also need to set the PROJECTS_AUTO_CREATE env variable for projects to be auto created on determination approve.

@sks444 Deploying now, will take a few minuts. Might fail as I see "test-be" above. I set PROJECTS_AUTO_CREATE to True on test.

@frjo, can we redeploy, I added the second address field.

Deploying to test now.

Thanks for the review @frjo,
Have fixed the currency list and other requested changes.

Deployed the latest updates to test now.

@frjo, can we redeploy, have added vendor detail page and other changes.

The tests has been failing on this pr with error:
psycopg2.errors.NotNullViolation: null value in column "locale_id" violates not-null constraint

Which doesn't seem related to these changes.
Possibly that is happening because of the migrations file.
I will rebase the migrations and make it just one file once everything has been approved.
Doing it now would mean un-applying migration on test as well, which deletes the data that we have on test for these changes.

@frjo, have pushed the recent changes discussed at https://github.com/HyphaApp/hypha/issues/2379#issuecomment-872932911

@frjo, added a few more changes..i.e. not letting user to update info unless a lead is assigned. And sending an email to lead on update info.

@frjo, should I go-ahead and try to fix the tests on this?

@sks444 Yes, maybe better to do that before we merge it it to main. I can fix the migrations on test afterwards.

@sks444  This is now merged in to main.


Add option to swap the sound to a custom file

Pretty self explanatory, and I think adding this feature would be fairly simple.

Hi @frarg, you can do this yourself by replacing any of the files in C:\Users<user>.runelite\casket-sounds. The plugin picks one of them at random so if you have just 1 sound you want to play, you can replace all of them with the same file. Hope this helps.


extract heated floor area data

This PR adds functionality to extract the above and below grade heated floor area data.
https://trello.com/c/q0fW8AUX/112-extract-load-heated-floor-data

 Extract/Load Heated Floor data


[Bug]: can't load external jars when running hop pipeline in docker

Apache Hop version?
2.7.0
Java version?
11
Operating system
Linux
What happened?
My project structure is as follows:
/opt/workspace/hop-workspace/docker/example001
    - metadata
    - project-config.json
    - shared_jars
      - ojdbc8-<IP_ADDRESS>.jar
    - transformers
      - example001.hpl

example001.hpl
TableInput(Oracle Table Input) -----> Select Values -----> JSON output(Output value) ------> REST Client

docker run -it -u root --rm \
  --env HOP_LOG_LEVEL=Basic \
  --env HOP_FILE_PATH='${PROJECT_HOME}/transforms/example001.hpl' \
  --env HOP_PROJECT_FOLDER=/files \
  --env HOP_PROJECT_NAME=hop_example_001 \
  --env HOP_RUN_CONFIG=local \
  --env HOP_SHARED_JDBC_FOLDERS=/lib/jdbc,/opt/workspace/hop-workspace/docker/example001/shared_jars \
  --name hop-example001-container \
  -v /opt/workspace/hop-workspace/docker/example001:/files \
  apache/hop:latest

When I run the above command, the exception error message is as follows:
2024/03/04 07:27:42 - Table input.0 - ERROR: An error occurred, processing will be stopped:
2024/03/04 07:27:42 - Table input.0 - Error occurred while trying to connect to the database
2024/03/04 07:27:42 - Table input.0 -
2024/03/04 07:27:42 - Table input.0 - Driver class 'oracle.jdbc.driver.OracleDriver' could not be found, make sure the 'Oracle' driver (jar file) is installed.
2024/03/04 07:27:42 - Table input.0 - Unable to load class 'oracle.jdbc.driver.OracleDriver' in this classloader or in the parent
2024/03/04 07:27:42 - Table input.0 - ERROR: Error initializing transform [Table input]
2024/03/04 07:27:42 - example001 - ERROR: Transform [Table input.0] failed to initialize!
2024/03/04 07:27:42 - Table input.0 - Finished reading query, closing connection.

Am I setting the HOP_SHARED_JDBC_FOLDERS environment variable incorrectly?
How should this parameter HOP_SHARED_JDBC_FOLDERS be set? Can you give a specific example?
Thanks a lot!
Issue Priority
Priority: 3
Issue Component
Component: Containers, Component: Database

you created a volume that maps /opt/workspace/hop-workspace/docker/example001/shared_jars (local) to /files in the container, and then you use the local path to set your HOP_SHARED_JDBC_FOLDERS.
try with --env HOP_SHARED_JDBC_FOLDERS=/lib/jdbc,/files \ and see if that makes a difference

you created a volume that maps /opt/workspace/hop-workspace/docker/example001/shared_jars (local) to /files in the container, and then you use the local path to set your HOP_SHARED_JDBC_FOLDERS.
try with --env HOP_SHARED_JDBC_FOLDERS=/lib/jdbc,/files \ and see if that makes a difference

@bamaer , yes, it works.


Ingested data along with airflow docs with tokenization as lowercase to new weaviate class

Ingested data along with airflow docs with tokenization as lowercase to new weaviate class DocsDevLowercase
Create a new Slack deployment so that this can be tested.

Deferring this for next release as OSS provider is of higher priority.

I have ingested this to DocsDevAnkit class in Weaviate. This change is deployed on dev slackbot

@vatsrahul1001 I have deployed the changes in the dev slackbot. You can test this now

@mpgreg test results can be found here This doesn't improve the answers


Deciding the categories for the bogs || Discussion issue

I wanted everyone's opinion on what are the different categories that we should divide/ separate the blogs under.
This discussion is in correspondence to the issue #224 .

Hey There!
I would Like to suggests some categories.
Here they are:

IT & Software
Business
Personality Development
Design
Marketing
Lifestyle
Photography
Health & Fitness
Music
Academics
Language
sports
social media
History
space and research

@ankitbiswal547  Thank you for all the suggestions, these will be really useful.

can you please assign this issue to me?

can you please assign this issue to me?

This is just a discussion, issue to decide categories for #224 , you can work on some other issue..


Fix the error documented to report an unvetted package

This has been true since 2.4.0 but apparently never fixed in the documentation.

Error on CI which seems very unlikely to have been caused by this PR:
Running Sphinx v4.3.1
loading translations [en]... done

Sphinx version error:
The sphinxcontrib.applehelp extension used by this project needs at least Sphinx v5.0; it therefore cannot be built with this version.

https://github.com/digital-asset/docs.daml.com/pull/614 should fix that.

I'll wait for that to be merged then, thank you!


MoveValue without copy constructor

I have a class with only a move constructor but without a copy constructor. When I use it as a an output from a component, I get an error. Below I give a minimal example where this happens together with the compilation error. My platform is macOS Sonoma 14.2.1 on a M2 processor, and compilation is done using Apple clang version 15.0.0 (clang-15<IP_ADDRESS>.5)
Code:
{
class movable_class {
public:
movable_class() {}
movable_class(movable_class && other) {};
};
class test_component final : public DSPatch::Component {
	public:
		test_component()
		{
			SetInputCount_(0);
			SetOutputCount_(1);
		}
		void Process_(DSPatch::SignalBus& inputs, DSPatch::SignalBus& outputs) override
		{
			movable_class x;
			outputs.MoveValue(0, std::move(x));
		}
};
auto c = std::make_shared<test_component>();

auto circuit = std::make_shared<Circuit>();
circuit->AddComponent(c);
circuit->Tick();

}
Compilation error:
In file included from /Users/pswoboda/repo/dspatch/include/DSPatch.h:31:
In file included from /Users/pswoboda/repo/dspatch/include/dspatch/Circuit.h:31:
In file included from /Users/pswoboda/repo/dspatch/include/dspatch/Component.h:31:
In file included from /Users/pswoboda/repo/dspatch/include/dspatch/SignalBus.h:33:
/Users/pswoboda/repo/dspatch/include/fast_any/any.h:279:58: error: object of type 'movable_class' cannot be assigned because its copy assignment operator is implicitly deleted
static_cast<value_t*>( _value_holder )->value = std::forward( value );
^
/Users/pswoboda/repo/dspatch/include/dspatch/SignalBus.h:172:31: note: in instantiation of function template specialization 'fast_any::any::emplace<movable_class>' requested here
_signals[signalIndex].emplace( std::forward( newValue ) );
^
/Users/pswoboda/repo/dspatch/tests/main.cpp:982:13: note: in instantiation of function template specialization 'DSPatch::SignalBus::MoveValue<movable_class>' requested here
outputs.MoveValue(0, std::move(x));
^
/Users/pswoboda/repo/dspatch/tests/main.cpp:959:4: note: copy assignment operator is implicitly deleted because 'movable_class' has a user-declared move constructor
movable_class(movable_class && other)
^
Thank you for any help!

Hi @pawelswoboda, thanks for the interest in DSPatch.
Signal types have a strict requirement that they must be both copy constructible and assignable. Sorry that isn't made clear.
This requirement comes from the fast_any type, and is subsequently the reason it outperforms boost/std::any.

In other words, for your type to be used as a signal it must at least define a copy constructor and a copy assignment operator.
For it to be most effective it'd also define a move constructor and a move assignment operator:
class movable_class
{
public:
    movable_class()
    {
        // ...
    }
    movable_class( const movable_class& other )
    {
        // ...
    };
    movable_class( movable_class&& other )
    {
        // ...
    };
    movable_class& operator=( const movable_class& other )
    {
        // ...
        return *this;
    };
    movable_class& operator=( movable_class&& other )
    {
        // ...
        return *this;
    };
};


Documentation should reflect the OS versions that the command lets are compatible with

Like many PowerShell Core users I’ve spent many a fruitless hour trying to understand why Microsoft.SharePoint.Online.PowerShell module does not work on non-Windows OSes. One of your colleagues pointed out that this PowerShell module has a dependency on .NET Framework 4.x which is only compatible with Windows.
I think that the documentation for PowerShell modules should reflect which OSes a module is compatible with.  It would save the module user a lot of time if this information was available to them within each module’s documentation.
I hope you can consider this enhancement request to your product documentation.
kind regards,
Pete

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

ID: 7d3c1e49-2acd-707a-33b2-2abe705fa123
Version Independent ID: a6afec72-10c8-84c9-9435-74602c30a3c0
Content: Connect-SPOService (Microsoft.Online.SharePoint.PowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/Connect-SPOService.md
Product: sharepoint-server-itpro
GitHub Login: @trent-green
Microsoft Alias: trgreen

Hello @loaderladdy
Many thanks for your feedback!
That is documented in the index file for the module description here
https://learn.microsoft.com/en-us/powershell/sharepoint/sharepoint-online/introduction-sharepoint-online-management-shell?view=sharepoint-ps
where it says it is a Windows PowerShell (not core/7) module.
Hope this information helps you,

Hi @get-itips I could not find the reference on that link, but I might not be understanding what the "index file" is.  I did find it via the link in the first para of the page here https://learn.microsoft.com/en-us/microsoft-365/enterprise/why-you-need-to-use-microsoft-365-powershell?view=o365-worldwide where it states

Am I looking in the wrong place Andreas?

Here in the article I linked

The SharePoint Online Management Shell is a Windows PowerShell module that you can use to manage SharePoint settings at the organization level and site collection level. Command-line operations in Windows PowerShell are composed of a series of commands. Each command uses a cmdlet and a set of settings known as parameters. For example, you use the New-SPOSite cmdlet to create a new site collection with a specified title, URL, owner, storage quota, and template (the parameters).

I was being thick! I was searching for text that said "core" and couldn't find it. I took your explanation literally.  Of course what you meant was Windows PowerShell means precisely that and by inference excludes the other PowerShell products PowerShell Core and PowerShell 7.  And that was the point I was making in the case that a casual PowerShell user does not make the distinction that Windows PowerShell excludes Core and 7. I know it probably seems daft because the article explicitly says Windows PowerShell and the author expects the reader to understand that implicitly but on occasions, especially in technical support documentation it might be more prudent to assume the reader has limited understanding and spell out the constraints so there is no room left for doubt because the author has removed all doubt. Is that fair?

I'll ask the Docs team what they think about this addition and get back, however, I think we don't tend to add those kinds of notes stating what is not supported but what is supported.

I'd suggest that it included as a system requirement for using PowerShell modules that have Windows dependencies because this explicitly excludes PowerShell version that work on Linux and MacOS, and if it is well worded you'd not have to mention the other OSes.
Mind you, does PowerShell Core and 7 work with SharePoint.Online.PowerShell? Because that would serve to confuse everyone if those PowerShell versions do on Windows but not on Linux or MacOS.
Hopefully Microsoft are working to retire Windows PowerShell in the near future so there isn't any confusion and Microsoft's SaaS products can be managed from any OS platform
Thanks again

@get-itips

@get-itips Kindly advise.

@yogkumgit Asked internally and I received the feedback to leave it as-is, only adding a link to Windows PowerShell requirements.

Thank you for taking this forward Andres 👍

Consider the matter closed

Thank you for taking this forward Andres 👍

thanks for returning and sharing feedback, that improves articles.


Update sources.py

solves bug on os x

how this is related to os x? I'm using os x and there is no problems with lxml package


Update installation instructions

RF 3.0 contains some changes to installation that need to be documented in the installation instructions. These new docs cannot be fully finalized until the final release, though. This issue tracks such changes so that we don't forget about them:

[x] Python 3 support (#1506).
[x] New generic robot and rebot start-up scripts (#2216)
[x] python -m robot (#2223)
[x] No more binary Windows installers (#2218)

It's not really about documentation, but we should also remove all artifacts related to generating Windows installers. We could do it right away, but there's always a risk that we need to revert the decision and create them anyway. Probably we should wait at least until RF 3.0 alpha 1 when the plan to not generate Windows installers is made more public.
Assuming we actually can stop generating Windows installers, the artifacts to remove include at least these:

robot.bmp
robot_postinstall.py
Code related to robot_postinstall.py in setup.py.
wininst task in tasks.py

added missing mention of python -m robot in 959169aada0a2f4e6382e9a8f0d1ecdd9aa53859

Unfortunately I was already working with larger changes to INSTALL.rst when @jussimalinen committed his enhancements. Merging the changes to docs wasn't easy so b9842cf reverts all Jussi's changes. Review for my changes, with those reverted changes in mind, still needed.

The new INSTALL.rst is much better than the slighly updated version I did. I updated a few links, but now it looks good!

Cool, I guess this issue can be considered done. RF 3.0 will be released tomorrow.


bson.InvalidDocument while using MongoDBObserver

This occurs while adding a new experiment entry into a MongoDB with command-line interface. Both '--pdb' and '--pdb.' are inserted in meta info at the same time. Perhaps the problem may caused by some little bugs in newly updated command-line classes? I get this error using current master branch code. It does not exist using stable version 0.7.5 on PyPI.

Ok I understand what is going on.
To get the list of default command line arguments here:
https://github.com/IDSIA/sacred/blob/0a62a091709c5408e337fee8eb81840a0226fbbf/sacred/experiment.py#L283
we first generate the help string (usage) and then use docopt to parse it and get the arguments again. Needless to say, this is not a good pattern to parse something that sacred generated itself, quite inefficient and error prone. In our case, here is the help message:
Usage:
  dummy [(with UPDATE...)] [options]
  dummy help [COMMAND]
  dummy (-h | --help)
  dummy COMMAND [(with UPDATE...)] [options]

Options:
  -b BEAT_INTERVAL --beat_interval=BEAT_INTERVAL  Control the rate of heartbeat events.
  -C CAPTURE_MODE --capture=CAPTURE_MODE  Control the way stdout and stderr are captured.
  -c COMMENT --comment=COMMENT  Adds a message to the run.
  -e --enforce_clean            Fail if any version control repository is
                                dirty.
  -F BASEDIR --file_storage=BASEDIR  Add a file-storage observer to the experiment.
  -f --force                    Disable warnings about suspicious changes for
                                this run.
  -h --help                     Print this help message and exit.
  -m DB --mongo_db=DB           Add a MongoDB Observer to the experiment.
  -n NAME --name=NAME           Set the name for this run.
  -D --pdb                      Automatically enter post-mortem debugging with
                                pdb on failure.
  -p --print_config             Always print the configuration first.
  -P PRIORITY --priority=PRIORITY  Sets the priority for a queued up experiment.
  -q --queue                    Only queue this run, do not start it.
  -S3 BUCKET_PATH --s3=BUCKET_PATH  Add a S3 File observer to the experiment.
  -s DB_URL --sql=DB_URL        Add a SQL Observer to the experiment.
  -t BASEDIR --tiny_db=BASEDIR  Add a TinyDB Observer to the experiment.
  -u --unobserved               Ignore all observers for this run.
  -d --debug                    Set this run to debug mode.  Suppress warnings
                                about missing observers and don't filter the
                                stacktrace. Also enables usage with ipython
                                --pdb.
  -l VALUE --loglevel=VALUE     Set the LogLevel.  Loglevel either as 0 - 50 or
                                as string: DEBUG(10), INFO(20), WARNING(30),
                                ERROR(40), CRITICAL(50)

Arguments:
  COMMAND   Name of command to run (see below for list of commands)
  UPDATE    Configuration assignments of the form foo.bar=17
  BEAT_INTERVAL  Time between two heartbeat events measured in seconds.
  CAPTURE_MODE  stdout/stderr capture mode. One of [no, sys, fd]
  COMMENT   A comment that should be stored along with the run.
  BASEDIR   Base-directory to write the runs to
  DB        Database specification. Can be
            [host:port:]db_name[.collection[:id]][!priority]
  NAME      Name for this run.
  PRIORITY  The (numeric) priority for this run.
  BUCKET_PATH  s3://<bucket>/path/to/exp
  DB_URL    The typical form is: dialect://username:password@host:port/database

Commands:
  print_config         Print the updated configuration and exit.
  print_dependencies   Print the detected source-files and dependencies.
  save_config          Store the updated configuration in a file.
  print_named_configs  Print the available named configs and exit.
  main

As you can see, one of the descriptions has --pdb. on a single line. The parsing went wrong here and saw this description as an option. The easy fix is to change the docstring. The true fix is to avoid parsing something that was generated by sacred and get the arguments directly. I'll work on it.


Protocol and client options - RDSTLS vs PKU2U

The "Protocol and client options" is somewhat confusing  and suggest breaking into two sections
First section RDSTLS protocol, as this handles the wider number of clients and does not require local computer to domain joined to the same tenant as the AVD session host, which I think norm. User on personal or organisation 'A' device connecting to organisation 'B' AVD infrastructure.
Second section PKU2U protocol, as this will only work with Windows Azure Virtual Desktop client and requires both local computer and session host to be domain joined to the same tenant.

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

ID: 96947883-d52b-1ed8-7db2-915f025060af
Version Independent ID: 96947883-d52b-1ed8-7db2-915f025060af
Content: Azure AD join for Azure Virtual Desktop - Azure Architecture Center
Content Source: docs/example-scenario/wvd/azure-virtual-desktop-azure-active-directory-join.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @TomHickling
Microsoft Alias: thhickli

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 518821

Thank you for the feedback.

We have this captured as an article to improve. Thank you for your feedback!


On project detail pages make project title page title

Need
Google shows the same title for all projects. We need to  let it know the project title.
Approach
Use the project title in the pages HTML  tag.

closing dupe of #1542


⚠ Processors has degraded performance

In e6860f5, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 61 ms

Resolved: Processors performance has improved in 47ff277.


Fix Twitter and WhatsApp share.

fix(share): fix share intent for both Twitter and WhatsApp

LGTM


Focus on new folder by default when using ctrl + shift + n

What's the Problem?
Long winded UX for creating new folders
Solution/Idea
A section in preferences to set a folder as the new item type when using ctrl+shift+n.  Creating a lot of folders is very cumbersome.
Additionally, when executing the shortcut, the new item type has folder highlighted as default, so hitting enter would then create the folder, or being able to shift up and down the list using the arrow keys and confirm the new selection using enter to execute would be ideal.
Alternatives
n/a
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
Win 11 22H2 22622.436
Comments
No response

Hi there, I think I found a solution to the problem, by adding an option on PreferenceSettingService and changing both ModernShellPage and ColumnShellPage to check if the option is enabled and redirect the shortcut "Ctrl + Shift + N" directly to CreateFileFromDialogResultType from UIFilesystemHelpers class, for the other part I've managed to focus the first item of AddItem ListView to be focused so it should be possible to travel the List via arrow keys.
The only problem I found so far is adding the Setting/Preference text to all the Resources.resw files, and if possible I would like to know what steps to follow or if it's not necessary on my part to complete that part.
Thanks on advance
@yaichenbaum

@z3r0tru5tk4l1 we shouldn't need a setting for this as it should be the default behavior 🙂
For the future, when adding a new string, adding the string to the en-us resource file is enough, the other strings are generated on our end.

@yaichenbaum Thanks for the quick answer, final question on how to continue, should I revert the changes made to the Setting/Preference and both ShellTypes as well as the Resource.resw change I made in the en-us folder? And only leave the autofocus on the firs item of the AddItem ListView or do I just create a PR for the changes made?
Thanks in advanced

should I revert the changes made to the Setting/Preference and both ShellTypes as well as the Resource.resw change I made in the en-us folder? And only leave the autofocus on the firs item of the AddItem ListView

That would be ideal 👍


Add example config file

Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>
Add example.yml file for https://github.com/product-os/flowzonify to reference

@joshbwlng this would need to be synced up with the README.md (which already has this information). Do we add a lot of value by placing another file in there to keep in sync with the README manually? cc @klutchell

I agree with Anton that this is duplicating the information we would have to keep up-to-date. If we want an example file we should link to it from the README to avoid having the info in two places.
However I think what we have on the README is good enough?

@joshbwlng I'll close this for now, if there is a strong reason for a separate file in the future, let's re-open then.

@klutchell @ab77 I had the same thought about it duplicating data before opening the PR (why I didn't merge it myself btw). The idea here is to enable flowzonify with https://github.com/product-os/flowzonify/pull/9 to be able to automatically open PRs on repos to update their flowzone config when necessary (think renovate/dependabot but for flowzone configs). For example to automatically open PRs to enable external contributions for our public repos, instead of having to do it manually. After thinking about it a bit more, I think it could also make sense to keep the reference in the flowzonify repo. This means we would need to manually update the reference in that repo, but it could then open all other PRs for us, much better than what we have now at least.
I had thought about moving the examples out of the flowzone readme into yaml files and then referencing them from the readme, but as an end-user I much prefer making examples like that immediately visible and available in the main readme. :thinking:


Fix multiplayer concurrency

Hotfix for https://github.com/excalidraw/excalidraw/issues/906
No satisfied. Definitely missed major cases, and this may introduce some subtle (or not so subtle) race conditions. As a hotfix it should probably work.
What I'm doing is

Not syncing elements being edited (e.g. arrows until they're finalized, or elements during drag etc.)
When syncing from remote, prefer local version if the element is currently being edited.

(Note: for reference, some discussion on this was done in our group chat.)


Testing Dropdown Selections

Similar to #50, issues with selecting elements react-native-element-dropdown in jest.

This will be resolved similarly to how #50 was resolved. In testing mode, we'll render radio buttons for each item in the dropdown menu.

#70 is a duplicate of this task, but it better describes the changes, so this task will be closed.


Blog app - Add forms

Project requirements

Create a method called current_user in your ApplicationController to make current user data available to all controllers.

It will return the first user from the database.
Don't worry about proper authentication at this time. You will add it in the upcoming projects.

Create forms to perform the following functions:
Creates a Post on behalf of the current_user (use the method that you created in your ApplicationController).
Create a comment on behalf of the current_user (use the method that you created in your ApplicationController).
Allow Users to add likes to Posts.

@prolajumokeoni ,Thank you for the review.
Can you please mark my project as approved in the Dashboard?
It is marked as changes required and i can not submit my project.

@prolajumokeoni ,Thank you for the review.
Can you please mark my project as approved in the Dashboard? It is marked as changes required and i can not submit my project.

Hi @TamaraDimov, apologize for the mixup this will be updated soon, thank you.