Icons alignment wrong.

Using:
@ng-bootstrap/ng-bootstrap": "^17.0.0",
@ng-icons/bootstrap-icons": "^28.0.0",
@ng-icons/core": "^28.0.0",

No custom SASS.
Icons, by default, are only showing their top half.
If I add:
ng-icon {
    @extend .d-inline;
}

They are closer to properly aligned, but not perfect.

Thanks for the report, the issue is due to how bootstrap sets the vertical align property globally on all svg elements. I'll get a fix implemented later today. Thanks!
https://stackblitz.com/edit/stackblitz-starters-nr3x6k?file=src%2Fmain.ts

The latest release should now resolve this issue. Any issues let me know.

Had to take a break, came back and noticed that vertical-align seemed to be the culprit, then I remembered this issue! Updated and it is significantly better, but still seems to be cut off by 1 or 2 pixels.
https://stackblitz.com/edit/angular-uejkh6?file=src%2Fapp%2Fngbicontest.html

Is this a Firefox thing?

Its a strange one, no amount of tweaking the vertical align, line-height etc.. makes the icon and text align perfectly as far as I can see. The only thing that appears to work in all scenarios is to set display: inline-flex and align-items: center on the .btn class.

Closing as I don't seem to find anyway to resolve this within the icon component itself. If you have any further suggestions we can re-open. Thanks.


[Enhancement]: allow users to specify a priority in register_reply

Is your feature request related to a problem? Please describe.
In ConversableAgent, reply handlers are registered via the register_reply function, which allows users to specify an index where the handler should be inserted to the _reply_func_list. The location of insertion is very important, as it determines when the handler will be invoked.
Unfortunately, _reply_func_list is a completely private / internal attribute and cannot be inspected by external code. As such, we really have no idea what index to use other than 0 (the default), and perhaps (-1, to insert at the end) -- though I've not actually tested if that can work. Moreover, the list of reply handlers keeps growing (it's at 9 already), so specifying an index other than 0 or -1 is guesswork, and prone to breaking over time.
This leaves us in an awkward spot. If I want to, for example, only change how llm replies are generated, and leave the handlers for human input, function handling, and code interpretation in place, I have to write code like this:
        self.register_reply([Agent, None], SocietyOfMindAgent.generate_inner_monologue_reply)
        self.register_reply([Agent, None], ConversableAgent.generate_code_execution_reply)
        self.register_reply([Agent, None], ConversableAgent.generate_function_call_reply)
        self.register_reply([Agent, None], ConversableAgent.check_termination_and_human_reply)

Which needlessly grows the handler list from 9 to 14, and actually duplicates 3 of the handlers.
Describe the solution you'd like
After discussions with @rickyloynd-microsoft , @qingyun-wu  and @sonichi , one proposal has been presented:
Rather than specifying an insertion location, register_reply should take a priority. We could then sort on priority (lower values being served first, like on unix). We could then have a convention like:

Human input handles are 1xx
Function / tool handlers are 2xx
Code interpretation handlers are 3xx
LLM handlers are `4xx'

With defaults for pre-registered handlers being 150, 250, 350, and 450
With such a convention, one can add a handler exactly where they want it relative to the defaults.  For example, preempting the LLM handler could be achieved by:
        self.register_reply([Agent, None], SocietyOfMindAgent.generate_inner_monologue_reply, 449)

Priority collisions would be handled as they are now: last registered is first to be executed. This can be done by using a stable sort on the handler list.
These numbers and conventions are for example purposes only. If we decide to act on this proposal, we should carefully consider these conventions and how to document them.
Additional context
No response

Nicely worded!

I am a little worried about introducing too many concepts. Can we just make it simple i.e., let user clear the reply functions and add them one-by-one?

That's exactly what I do in the society of mind agent (though I do so by accessing a private variable).
The trouble is that there are 9 to add once you've cleared it (again, if all you want to do is replace the LLM handler)

LinkedIn has expressed interest in a solution like this to avoid the guesswork of assigning a position to a new reply function. Do we have an alternative solution to the one proposed above by @afourney ?

Would love to understand their usage case. I still think the priority coding is not necessary and increases complexity. I suggest simply exposes a public clear_registered_reply_functions() method that clears all reply functions, and then in their subclass to ConversableAgent, register reply functions one by one according to their specific usage case.
It is more lines of code, but it is unambiguous.

They would like to only add the new reply function that their subclass needs, while leaving ConversableAgent's standard reply functions in place. I think that's the most common case for most new agent classes. They know where in the current list they would like their new reply function inserted, but they don't like having to count out a position because that's very brittle, since the insertion integer can change if a new reply function is ever added to ConversableAgent, for example. That brittleness is solved by the proposal in this issue, because if we even add a new reply function to ConversableAgent, we will give it a priority that leaves the rest of the list undisturbed. This new scheme is not absolutely worry free, but it's a lot more robust than the current scheme.

@shaileshj2803, could you add more about your use case?

One another we can avoid numbered priorities, which I think is adding unnecessary complexity, is to expose certain “registration points”, much like the hookable methods we have. We can expose the following registration points:

human input and termination: for all termination related reply functions
Code execution: customized code execution
Function and tool call
LLM reply.

We can then insert the custom reply functions in appropriate places based on that. Since user is now responsible for their reply functions, they are also responsible for registering them in proper order to make sure the correct precedence.

I see advantages to that, because it gives semantic meaning to the types of reply functions. But how would ordering within a registration point be controlled? It it's a fixed ordering like now, wouldn't that require users to deal with the same kind of brittleness as the current scheme? And wouldn't users have to define new registration points for types of reply functions we haven't thought of?

But how would ordering within a registration point be controlled?

Same as before, first-registered-last-triggered. Managed by the subclass.

And wouldn't users have to define new registration points for types of reply functions we haven't thought of?

I don't know how to handle this. In that case perhaps clearing the registered functions and registering them one-by-one is still easier to understand.
My proposal may be based on incorrect assumptions so please regard it as brainstorming. Would love to see the actual use case.

@ekzhu standardized priority codes (10x, 20x, 30x,  etc.) were meant to achieve the same thing as named registration points. But I agree that named locations are better than remembering how priorities map to the same 4 or 5 general reply classes. If you want to have named points, I would be happy to support that.

It seems like the benefits of these schemes can be combined by using comments (instead of registration points) to group the handlers into meaningful classes (where those are known) while resolving ambiguities with numeric priorities.
ConversableAgent registers 10 different handlers today:

Human input handlers
150,  a_check_termination_and_human_reply
160,  check_termination_and_human_reply

Function handlers
350,  a_generate_function_call_reply
360,  generate_function_call_reply

Tool handlers
550,  a_generate_tool_calls_reply
560,  generate_tool_calls_reply

Code execution handlers
750,  generate_code_execution_reply
760,  _generate_code_execution_reply_using_executor

LLM handlers
950,  a_generate_oai_reply
960,  generate_oai_reply

This leaves plenty of room for users to insert new types of handlers without the guesswork of counting list positions, or having to worry about handler ordering within a registration point. Even the priority can be an optional parameter to register_reply with a reasonable default value, and the whole thing can be made backwards compatible.

No activity since Feb. This remains a reasonable feature request for the 0.2 line, but most of my recent work has been elsewhere, and I've found workarounds. Reopen as needed.


Unit test to test for missing strings

Bit of a stretch but I was wondering if anybody has been able to find a nice way to unit test their front end code to locate strings / parts of the code that don't have translations?
I'd imagine that it'd be nigh impossible but worth a shot asking.

I would try to introduce a custom error handler for the test which make it to fail if no translation found:
https://angular-translate.github.io/docs/#/guide/17_custom-error-handler

Yeah, the approach of @dmitryevseev is a useful approach. For example, you could check for zero results of the error handler or something similar one.


[Minor] Improve logging and rename the health check endpoint name

Log the whole request instead of input text only
Rename /v1/health to /health_generate for the health check in #1154. The endpoint name "/v1" are for openai APIs so we do not want to use it for SRT endpoints.
Other minor style improvement

We found that when cuda-graph is turned on, the service will hunag after more than 200 consecutive /health_generate requests (1 req/s ). set --disable-cuda-graph can help solve this problem.  Can be reproduced in 0.3.0 with 4* A100.

May u try the latest main? Thanks!

Any ideas on this issue?


fix(GRO-882): prevents stretched thumbnail images

The type of this PR is: Bugfix
This PR solves GRO-882
Description
Prevents images from stretching by switching over to cropped. (We could also just use the image.width and image.height if we wanted to keep proportionally resized images, which would fix the stretching.)
Also just fixes the collapsing anchor tag. (I see this alot, if you're gonna wrap something in a link it and it's not a flex child should be display: block (or flex or whatever. Just not inline). It's less a problem for pointer device users and more a problem for when you're tabbing around (or trying to inspect the DOM).

Might as well bump the res on those headers while I'm in here. One moment.

Sharper now:


Add MIME type to Decker.desktop

This associates the application with the file type, so that if you double-click on a .deck file in a file manager, Decker will be used to open it. Without the MIME type in there, my file manager (Nemo) won't even suggest Decker at all.

Looks good. Thank you!


Add benchmark for double and float encoding/decoding, as well as NaN encoding

Was testing report that NaN encoding was slow. Couldn't reproduce it, but may as well have the benchmark.

I probably fixed this in https://github.com/apache/arrow/commit/250e97c70f497581bca412dfd2a654a1f9736064#diff-7fea663f71cf7f71bdda3fd6bd21727d

The original report was, I believe, against the old parquet-cpp repo, so I figured it might have been fixed here.

@fsaintjacques

@itamarst, generally we ask that a corresponding JIRA be opened and the PRs are title as PARQUET-XXX: [C++] description.

Will do later this week.

@emkornfield done!

I would have expected the performance problem to occur when writing whole files because of the statistics vectorization between 0.13 and 0.14.

Sadly the performance got worse from 0.13.0 to 0.14.1. I opened
https://issues.apache.org/jira/browse/ARROW-6051


Add workload-identity capabilities to live test

Example tests: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3795388&view=results

API change check
API changes are not detected in this pull request.

This PR was for testing initially but isn't needed any longer.


Authentication error with zone DNS API token

For minimal permissions I have created an API token locked down to zone DNS edits using the "Edit zone DNS" template:

When launching this service I get an Authentication error:
cloudflare-companion | [INFO] ** [traefik-cloudflare-companion] Starting Traefik Cloudflare Companion
cloudflare-companion | Traceback (most recent call last):
cloudflare-companion |   File "/usr/sbin/cloudflare-companion", line 276, in <module>
cloudflare-companion |     init(doms)
cloudflare-companion |   File "/usr/sbin/cloudflare-companion", line 211, in init
cloudflare-companion |     check_container_t2(c, doms)
cloudflare-companion |   File "/usr/sbin/cloudflare-companion", line 176, in check_container_t2
cloudflare-companion |     point_domain(extracted_domains[0], doms)
cloudflare-companion |   File "/usr/sbin/cloudflare-companion", line 91, in point_domain
cloudflare-companion |     records = cf.zones.dns_records.get(dom['zone_id'], params={u'name': name})
cloudflare-companion |   File "/usr/lib/python3.8/site-packages/CloudFlare/cloudflare.py", line 672, in get
cloudflare-companion |     return self._base.call_with_auth('GET', self._parts,
cloudflare-companion |   File "/usr/lib/python3.8/site-packages/CloudFlare/cloudflare.py", line 126, in call_with_auth
cloudflare-companion |     return self._call(method, headers, parts,
cloudflare-companion |   File "/usr/lib/python3.8/site-packages/CloudFlare/cloudflare.py", line 502, in _call
cloudflare-companion |     raise CloudFlareAPIError(code, message)
cloudflare-companion | CloudFlare.exceptions.CloudFlareAPIError: Authentication error

Am I missing other permissions?

By now I've switched to a wildcard certificate, thus I don't need specific DNS entries (and this image) anymore.
Still it would be interesting what the issue could be here.

Yeah I'm getting the same error myself even after generating a token with full permissions. Either the documentation is bad or something is broken for now..

Same issue here upon launching the service... no clue on how to debug unfortunately :-(

OK, figured it out. Please note the comment Leave Blank for Scoped API behind the CF_EMAIL environment variable. This is important! You need to remove the CF_EMAIL environment variable when using a domain specific API token!
The long explanation:
According to Cloudflare documentation here one of the common issues with API tokens is using the wrong authentication.

On occasion, customers will attempt to use an API Token with an API Key syntax. Ensure you are using the Bearer option, rather than the Email and API key pair.

According to the documentation of the Python wrapper for the Cloudflare v4 API, there are  multiple ways to make a call:
`# A minimal call - reading values from environment variables or configuration file
cf = CloudFlare.CloudFlare()
# A minimal call with debug enabled
cf = CloudFlare.CloudFlare(debug=True)

# An authenticated call using an API Token (note the missing email)
cf = CloudFlare.CloudFlare(token='00000000000000000000000000000000')

# An authenticated call using an API Key
cf =<EMAIL_ADDRESS>token='00000000000000000000000000000000')

# An authenticated call using an API Key and CA-Origin info
cf =<EMAIL_ADDRESS>token='00000000000000000000000000000000', certtoken='v1.0-...')

# An authenticated call using using a stored profile (see below)
cf = CloudFlare.CloudFlare(profile="CompanyX"))`

As we are using an API token, we need to use the 1st option of the authenticated call and omit the e-mail address.

Good catch, now it's working. Thanks!


UI issues when put in the right-side panel

Type: Bug Report

If you drag the serial monitor tab page from the bottom panel to the right side like this,

When you drop it, you will find that the UI will not displayed on the right-side panel, and there will be an invisible button on the top of the right-side panel, after you click it, the UI will display normally, but that button is still invisible.

@ZZH-Finalize Interesting, do you find that this happens with any other view? Either way, I'll mark it as a bug and put it on the backlog. Thanks

No, this is the only issue I have found so far.


EthStorage Devs Meeting #58 Agenda

Meeting Info

Nov 28, 2023, 10:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress

Devnet-1 progress track
https://github.com/ethstorage/es-node/pull/93

Devnet test
Review zk Fraud Proof Grant

EthStorage -- sync

Fix issues (investigate issue: https://github.com/ethstorage/es-node/issues/90, https://github.com/ethstorage/es-node/pull/77 (investigate "no NAT found" error))

Performance: benchmark: https://github.com/ethstorage/es-node/pull/2

Dashboard: https://github.com/ethstorage/es-node/pull/61, https://github.com/ethstorage/storage-contracts-v1/pull/35, https://github.com/ethstorage/es-node/pull/79, add contract report.

QuarkChain

API requirement: https://github.com/QuarkChain/quarkchain-dev/pull/469

zk Fraud Proof Grant
dive deep into zkWasm

mining performance tests and https://github.com/ethstorage/es-node/pull/98

https://github.com/ethstorage/es-node/pull/97

https://github.com/ethstorage/es-node/pull/94

guide update

pre-built es-node
pre-built docker img

upload to docker hub

directio

w3box supports eip-4844 devnet

zkWASM : memory continuation + eid expansion
Coq learning

Recording: https://drive.google.com/file/d/1nvfjs5ZrEbGFiSJ5Li5M38CVBKr2v7pU/view?usp=sharing


What's up with deploy_config?

The deploy_config function is defined as follows:
def deploy_completions():
    completions = {'complete.bash': Path('/etc/bash_completion.d/pew'),
                   'complete.zsh': Path('/usr/local/share/zsh/site-functions/_pew'),
                   'complete.fish': Path('/etc/fish/completions/pew.fish')}
    for comp, dest in completions.items():
        if not dest.parent.exists():
            dest.parent.mkdir(parents=True)
        shutil.copy(str(pew_site / 'shell_config' / comp), str(dest))

There is no chance this can work, unless it is run as root.
That function seems to only ever be called by pew/shell_config/complete_deploy, which itself doesn't seem to be used anywhere.
Shell completions work just fine when sourcing $(pew shell_config) in the shell startup file (e.g ~/.bashrc or the equivalent for other shells).
As a result, I wonder if complete_deploy and deploy_completions() aren't just forgotten historical relics which should be removed?
Or am I missing something? 😃

Good spot :slightly_smiling_face:
This is something that I've left slightly undocumented. Something similar to this should be done by the install script of a .deb/.rpm package (and I guess that complete_deploy) could be directly invoked by it, though I presume that packaging policies might prevent it
Anyhow, I didn't have an easy/obvious and platform agnostic way to deploy the completions (and also make them available and easily test them on different shells). Running that snippet from setup.py was also out of the question (it's better to avoid side effects, and as you know this would've required root permissions)
So, the idea was simply that I could've easily run that script while developing.
If you know about the git log -S'foo' command, you can easily inspect the history of this feature:
dario@feynman ~/P/pew (master$%)> git log -S'deploy_completions'
commit 920bb4a21dd5b4b707cbe285062bd23f9cf4955c
Author: Dario Bertini<EMAIL_ADDRESS>Date:   Tue Nov 26 04:37:41 2013 +0100

    Added completions deploy function/script
    Added completions scripts to the package

As you can see, it's from 2013
dario@feynman ~/P/pew (master$%)> git log -S'shell_config' --oneline
bfd1a1c (tag: 0.1.15) Update README and bump to 0.1.15
686c59e Add first_run_setup
9cb629a rename `pew completions` to `pew shell_config`. Now the shell files also update the prompt when sourced
dario@feynman ~/P/pew (master$%)> git log -S'completions_cmd'
commit 9cb629a7bc2bfcd7553029b96f563611f80f5f9a
Author: Dario Bertini<EMAIL_ADDRESS>Date:   Wed Sep 9 01:29:58 2015 +0100

    rename `pew completions` to `pew shell_config`. Now the shell files also update the prompt when sourced

commit c3f2dc24a3c25a035e5b1640925498418b596421
Author: Dario Bertini<EMAIL_ADDRESS>Date:   Fri Sep 4 14:33:56 2015 +0100

    Add completions command

While the completions command (now called shell_config) had been added only on 2015 :slightly_smiling_face:

One more thing: since complete_deploy is shipped in the tarball, it can be easily used not only while developing, but also at install time.
I don't feel like this should be removed, but indeed... if shell_config is enough for each and every user, I guess it's just old cruft that we could get rid of

Ok, I understand better now.

Something similar to this should be done by the install script of a .deb/.rpm package (and I guess that complete_deploy) could be directly invoked by it, though I presume that packaging policies might prevent it

Yeah, that doesn't jibe well with packaging policies. In general it would be better if python setup.py install installed everything, but everything outside of the Python module being installed (e.g files in config files /etc, data in /usr/share, ...) is always such a pain to handle. The joys of Python packaging... 😞
In addition, /etc/bash_completion.d/, being under /etc, is for the local admin to add stuff. Distro packages should install Bash completion files in /usr/share/bash-completion/completions/ instead. (I suspect the same is true for other shells, but I'll need to investigate what the right location would be for them)
At the very least, I think I'm going to do the following:

get the Fedora package to install the completion files (just running the /usr/bin/install command on those files;
remove the /usr/lib/pythonX.Y/site-packages/pew/shell_config/complete_deploy from the package, because it is useless once the completion files are in place;

I'm not sure what would be the best course of action for Pew upstream, though.

get the Fedora package to install the completion files (just running the /usr/bin/install command on those files;

Except that handles the various /usr/lib/pythonX.Y/site-packages/pew/shell_config/complete.$SHELL files, but not the /usr/lib/pythonX.Y/site-packages/pew/shell_config/init.$SHELL ones.
And in fact that's a problem shared by the complete_deploy command.

Yeah, never tried to source both of them though.
I think that even if the rpm doesn't install completion files, that would be ok.


TokenAuthorizerMixin error: "Assertion Failed: You're using the DataAdapterMixin without specifying an authorizer"

I get the following error message in my console at application load:

"Assertion Failed: You're using the DataAdapterMixin without specifying an authorizer. Please add authorizer: 'authorizer:application' to your adapter."

It occurs when my application adapter goes from this:
import DS from 'ember-data';

export default DS.JSONAPIAdapter.extend({
  namespace: '/api/v1',
})

to this:
import DS from 'ember-data';
import TokenAuthorizerMixin from 'ember-simple-auth-token/mixins/token-authorizer';

export default DS.JSONAPIAdapter.extend(TokenAuthorizerMixin, {
  namespace: '/api/v1',
})

What version of ember-simple-auth are you using?

1.5.1

Try upgrading to 1.6.1 and see if that helps: https://github.com/simplabs/ember-simple-auth#deprecation-of-authorizers

Maybe I should add ember-simple-auth as a dependency to prevent older versions from being used. @jpadilla thoughts?

Nope. I just also get a deprecation warning along with it. Which makes sense since they said that it would be removed in 2.0.

Mmm... I don't see this behavior in any of my apps nor does it happen in the demo. What adapter functions are getting called on app boot?

I went ahead and added it to https://github.com/jpadilla/ember-simple-auth-token/pull/235.


Update caServer to letsencrypt one in examples

What does this PR do?
Update caServer field in the examples to put the staging endpoint of letsencrypt instead of private IPs
Motivation
As we are talking about letsencrypt since the beginning of the doc, I have the feeling that this will help avoid confusion.
More

[ ] Added/updated tests
[x] Added/updated documentation

Additional Notes

Please do not keep merging master into this branch. Our merge bot will take care of that for us. Re-merging master just takes cycles out of our CI.
Thanks!

@dtomcej understood


Issue running with --eval

Describe the bug
Error getting model eval
To Reproduce
Steps to reproduce the behavior:
git clone https://github.com/Qerim-iseni09/Stock-Predictor-V4.git
cd Stock-Predictor-V4
python SPV4.py --install
python SPV4.py --prepare_data
python SPV4.py --train
python SPV4.py --eval

Expected behavior
In README.md is says it should print out the root mean squared error (RMSE), the MAPE and Total Rewards will be plotted
Desktop (please complete the following information):

OS: [e.g. iOS] linux
Browser [e.g. chrome, safari] Brave (don't see why this matters'
Version [e.g. 22] 3.11.4

Hello there!
The Evaluation feature has been taken out in this moment as it had some problems, you can skip the Evaluation Process as its in Work. I Forgot to write in the Readme that its in the works. Please wait until the Evaluation Logic comes back.
Thank you for your Support and Reading.


Create codemagic

This CI/CD workflow is designed to streamline Flutter iOS app development, testing, and deployment. It incorporates automatic triggering on main and prod branches, ensuring builds and tests run on every code push or pull request. Key steps include dependency installation, environment setup, and static code analysis with flutter analyze and flutter format. It performs integration and unit tests on an iOS simulator, followed by an iOS build without code signing. A separate deploy workflow manages code signing and app distribution through App Store Connect, with automated artifact generation and emails for build results. Optimized for quality, efficiency, and seamless deployment.

Automates iOS app CI/CD process, handling builds, testing, and deployment seamlessly with integration and error control features.

Automates iOS app CI/CD process, handling builds, testing, and deployment seamlessly with integration and error control features.


need to find out if az datafactory pipeline create-run  command finishes - need a show of run_id

[Enter feedback here]

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

ID: aa8bd249-9305-3c25-8bca-d911c5b86880
Version Independent ID: f50fd6b7-c141-982c-5362-f123b9e5bf04
Content: az datafactory pipeline
Content Source: latest/docs-ref-autogen/ext/datafactory/datafactory/pipeline.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

data factory create-run command doesn't tell you if it ran successfully.  get command or show command doesn't filter for run_id success - there is  no monitor command

@fengzhou-msft please take a look

@hansolo2218 There is the az datafactory pipeline-run show command. Can you try it and see if it meets your needs?

Please let us know if you still have questions.


Webpack configuration

It is posible to install this using webpack ?

Infinite Scroll v3 is out!. Take a look at Webpack usage for Infinite Scroll v3.


Prototype detekt watch service - run detekt on each file change

Heyho,
I just hacked together an idea I recently had: a watch service.
It helps to check for code smells while coding. The advantage is that no startup overhead is needed.

Is this ready to be reviewed?

Yes, please! The prototype in the title is just to make clear it is an experiment. If you guys find it useful, we can integrate as a module.
What is missing: using the same arguments as in cli-module and maybe printing just the code smell, not the rule set ids.

Good idea!
How does it affect RAM/CPU?
You might also add some tests?

That's a good question. I check every second for file changes which results in 0-2% cpu usage on my laptop. It would be awesome if you two can check it out locally and measure/guess. Maybe it is also ok to look all 2, 3 or 5 seconds.
RAM - I'm not sure, will check it tomorrow. My guess would be: not that much, as all data is immediately lost ...
Will write down tests on my list :)

Performing good on a 2,4 GHz Intel Quad Core i7.
This might be a good idea for an IntelliJ plugin.

Maybe the watch service should react to keystrokes instead of running periodically?

I think intellij has quite a similar api for changes as java7 has. Silently updating a table and showing the results when the user tries to commit something ^^.
Is it even possible to react to key strokes when the program runs in a terminal?

Uff I screwed the PR xD


Enable CORS for Northstar's API.

What's this PR do?
This pull request enables cross-origin resource sharing for Northstar (for example, so we can load a user's profile from https://vote.dosomething.org). We'd previously overridden this in Fastly in DoSomething/devops#416, but I'd like to clean this up and move it into the application now.
How should this be reviewed?
Here's an working example from my local (simulating an AJAX request from example.com):

One note – I had to change our PHP version constraint to accept any 7.2.* release since my Homestead is running PHP 7.2.7 and so failing with the ~7.2.9 constraint we'd added in #794.
Relevant Tickets
DoSomething/devops#439
Checklist

[ ] Documentation added for changed endpoints.
[ ] Tests added for new features/bug fixes.
[ ] Post a message in #api if this includes something that causes a rebuild!

And with the updated Wercker box we're back in business!


Debugger issue on Windows 11 ...

Hi, when I try to run my app in debugger mode, the app terminate with a debugger issue.
By removing the flag BDW_CUSTOM_FRAME  on main.cpp then I can run the app and my window resize correctly and appears in the selected position, of course, the title bar is then showed.
auto bdw = bitsdojo_window_configure(BDW_CUSTOM_FRAME | BDW_HIDE_ON_STARTUP); --> this does not work, crashes.
auto bdw = bitsdojo_window_configure(BDW_CUSTOM_FRAME); --> this does not work, crashes.
auto bdw = bitsdojo_window_configure(BDW_HIDE_ON_STARTUP); --> this works, but it does not hide title bar.
I'm running on Windows 11 (22H2) and the issue appearing when lauching the app is:
Launching lib\main.dart on Windows in debug mode...
lib\main.dart:1
Error waiting for a debug connection: The log reader stopped unexpectedly, or never started.
Error launching application on Windows.
Exited (sigterm)

Uploaded to version ^0.1.5 and issue solved. Thanks !


Add/improve docs for boolean Inputs

Works on IBM/carbon-components-angular#673
WIP since there's a ton of area to cover

this probably needs a fresh start


[uart/dv] Improve uart run time performance

spinwait consumes most of the sim time when uart freq is lowest and core
clk freq is highest.
Add spinwait_delay_ns to reduce the frequency of polling tx/rx_idle
It runs 4x faster with vcs in this case now
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

Is spinwait_delay_ns is the sampling period for checking over timeout_ns?

Is spinwait_delay_ns is the sampling period (for checking) over timeout_ns?

@tunghoang290780
No, spinwait_delay_ns is the delay before every csr_rd, and timeout_ns doesn't depend on it.
        fork
          while (!under_reset) begin
            if (spinwait_delay_ns) #(spinwait_delay_ns * 1ns);
            csr_rd(...);
           ...
          end
          begin
            wait_timeout(timeout_ns, msg_id, $sformatf(...));
          end
        join_any
        disable fork;


Implicit mass spring system

We provide a direct sparse solver and a CG solver.

LGTM. Thank you for your contribution!


Bug: Facing issue with chat completion streaming option using GPT4O and GPT4V model with Azure SDK V2

Describe the bug
Getting below error with GPT4O and GPT4V model in Azure SDK V2 using chat completion stream = true. Also, parallel function/tool call doesn't seem to work with GPT4O and GPT4_Turbo models with or without streaming option enabled:
HTTP 400 (invalid_request_error: unknown_parameter)
Parameter: stream_options
Unknown parameter: 'stream_options'.

Hi @AmanDogra1989 for Azure SDK v2 issues please create the issue in this repository https://github.com/Azure/azure-sdk-for-net/issues. This repository is just for the Semantic Kernel. Thanks!


Custom queries for projects?

Would that be possible to have custom queries that would yield the tasks for project?

Hey the problem is my workflow. I tend links on one level above my todos to not repeat mentioning #foo or [[foo]] on every todo. So my structure is usually:
* [[Foo]]
  * TODO one
  * TODO two
  * TODO three
*#Bar
   * TODO one one

so far from what I see that confuses your cool plugin and makes it all land under ~wrong project.

Sorry, this may not be supported, the plug-in plans support using custom property to set the owning project.

Agenda3 supports custom filters, you can give it a try.


Added Pascal triangle pattern java

Hey any problem with this pr?

Nice work, sorry for late review. Thanks for contributing


Add Missing Top 100 Currencies: (okb, aave, cro, sol, snx, ftt, luna, busd, cdai, egld, grt, sushi, ksm, avax, cusdc, cel, ceth, vgx, cake, ftm, xsushi, near, hbtc, rune, hbar, chsb, renbtc, husd, mdx, ust, btmx, flow, zks, ar, rsr)

Currency names: okb, aave, cro, sol, snx, ftt, luna, busd, cdai, egld, grt, sushi, ksm, avax, cusdc, cel, ceth, vgx, cake, ftm, xsushi, near, hbtc, rune, hbar, chsb, renbtc, husd, mdx, ust, btmx, flow, zks, ar, rsr
Symbol: Various
Website: Various
Theme color: Various
Logo (SVG format): Various

Have you considered scraping icons from an aggregator such as 1inch?:
https://docs.1inch.exchange/api/tokens

@jmynes hello.
The cryptocurrency ecosystem is using the icons from this repository to present assets without biases. We craft them carefully, they are custom made so there is a visual unity, are optically centered and we here aim for quality.
Please file one request per icon. Your request, coupled with the question about scraping, reflects at worse manifest condescension — at best a misunderstanding of the spirit and purpose of this repository.

@jmynes hello.
The cryptocurrency ecosystem is using the icons from this repository to present assets without biases. We craft them carefully, they are custom made so there is a visual unity, they are optically centered and we here aim for quality.
Please file one request per icon. Your request, coupled with the question about scraping, reflects at worse manifest condescension — at best a misunderstanding of the spirit and purpose of this repository.


restart unbound without starting and stoping the daemon

(default behaviour for..... e.g systemd), use the unbound-reload command also check the config before reload

this conflates two things into reload (check and reload), when they are better done when (before) writing the configuration (with validate_cmd)

ok split the validation and reload makes sense.
what do you mean with all platforms? unbound-control comes with unbound, so i assume it works on all platforms

ok i implemented the split

I'm curious what @xaque208 thinks about this

I'm generally +1 on this.  The validate_cmd needs to be worked out first.

Also, have you seen this one? https://github.com/xaque208/puppet-unbound/pull/70

yes, but its with an exec, i try to avoid execs....
2015-04-29 20:26 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
Also, have you seen this one? #70
https://github.com/xaque208/puppet-unbound/pull/70
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97532135
.

full qualified path is fixed
2015-04-29 20:28 GMT+02:00 Florian Koch<EMAIL_ADDRESS>
yes, but its with an exec, i try to avoid execs....
2015-04-29 20:26 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
Also, have you seen this one? #70
https://github.com/xaque208/puppet-unbound/pull/70
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97532135
.

I agree with avoiding execs, but having a $PATH to look in for the command should make some thing simpler.

yes thats true, but selectors are also an option, the default is /sbin and
each known other location needs to be defined
2015-04-29 20:34 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
I agree with avoiding execs, but having a $PATH to look in for the command
should make some thing simpler.
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97534569
.

Yep, and values that are missing for a given platform could just lack the validation functionality, perhaps.

hm yes but how to detect this? Wrapping the concat in an If block?
2015-04-29 20:53 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
Yep, and values that are missing for a given platform could just lack the
validation functionality, perhaps.
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97538956
.

I suspect that if validate_cmd => undef would solve it.  So as long as the variables looked up out of params had an undef if we don't know the path, that should handle it if we just passed that value to the concat resource.  I think.

ok do you man such a thing e.g
$unbound_validate_cmd = $osfamily ? {
/OpenBSD/ => '/usr/sbin/unbound-checkconf %',
/RedHat/  => '/sbin/unbound-checkconf %',
default   => undef,
}
concat { $config_file:
validate_cmd => $unbound_validate_cmd,
notify       => Service[$service_name],
}
2015-04-29 20:59 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
I suspect that if validate_cmd => undef would solve it. So as long as the
variables looked up out of params had an undef if we don't know the path,
that should handle it if we just passed that value to the concat resource.
I think.
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97541498
.

Yes, that kind of thing, though burying that in unbound::params.  That should make it easier, since the list of supported platforms is cased out in the params class.

ok i implement this tomorrow...
2015-04-29 21:14 GMT+02:00 Zach Leslie<EMAIL_ADDRESS>
Yes, that kind of thing, though burying that in unbound::params. That
should make it easier, since the list of supported platforms is cased out
in the params class.
—
Reply to this email directly or view it on GitHub
https://github.com/xaque208/puppet-unbound/pull/84#issuecomment-97547898
.

hm i have no idea where the errors come from, on my  local test (rhel7/puppet .7.x) it works without errors

I don't think that failure is you.  I've put in https://github.com/xaque208/puppet-unbound/pull/85 to address changes in the concat module.

ok i wait for #85 merged and then run the tests again

Merged.  Rebase and lets see how it goes.

ok fixes also the version in fixture, now the build works

Cheers, thanks for working it out.


can I set uniqueItems in json schema based on property of item

Question
Is it possible to base uniqueItems on CustomTypes property in my json schema on the property CustomType in the CustomTypes schema in definitions?
{
  "properties": {
    "CustomTypes": {
      "items": {
        "$ref": "#/definitions/CustomTypes"
      },
      "type": "array",
      "title": "Custom Types",
      "description": "The Custom type array.",
      "default": [],
      "widget": {
        "formlyConfig":{
          "templateOptions": {
            "uniqueItems": true
          }
        }
      }
    }
  },
  "type": "object",
  "required": [
    "CustomTypes"
  ],
  "title": "This is a Title",
  "default_values": [
    {
      "CustomTypes": []
    }
  ],
  "definitions": {
    "CustomTypes": {
      "properties": {
        "CustomType": {
          "allOf": [
            {
              "$ref": "#/definitions/CustomType"
            }
          ],
          "title": "Custom Type",
          "description": "The Custom type."
        },
        "TypeNames": {
          "items": {
            "type": "string"
          },
          "type": "array",
          "title": "Custom Type Names",
          "description": "The Custom name starts with.",
          "default": []
        }
      },
      "type": "object",
      "required": [
        "CustomType",
        "TypeNames"
      ],
      "title": "Custom Types"
    },
    "CustomType": {
      "type": "string",
      "enum": [
        "enum_1",
        "enum_2",
        "enum_3",
        "enum_4",
        "enum_5"
      ],
      "title": "CustomType"
    }
  }
}

Instead of putting unique items in the widget put it in the schema itself.
Unique items is a json schema validation key.
On Tue, May 7, 2024, 1:58 PM John Cutburth @.***> wrote:

Question
Is it possible to base uniqueItems on CustomTypes in my json schema on the
property CustomType of the item?
{
"properties": {
"CustomTypes": {
"items": {
"$ref": "#/definitions/CustomTypes"
},
"type": "array",
"title": "Custom Types",
"description": "The Custom type array.",
"default": [],
"widget": {
"formlyConfig":{
"templateOptions": {
"uniqueItems": true
}
}
}
}
},
"type": "object",
"required": [
"CustomTypes"
],
"title": "This is a Title",
"default_values": [
{
"CustomTypes": []
}
],
"definitions": {
"CustomTypes": {
"properties": {
"CustomType": {
"allOf": [
{
"$ref": "#/definitions/CustomType"
}
],
"title": "Custom Type",
"description": "The Custom type."
},
"TypeNames": {
"items": {
"type": "string"
},
"type": "array",
"title": "Custom Type Names",
"description": "The Custom name starts with.",
"default": []
}
},
"type": "object",
"required": [
"CustomType",
"TypeNames"
],
"title": "Custom Types"
},
"CustomType": {
"type": "string",
"enum": [
"enum_1",
"enum_2",
"enum_3",
"enum_4",
"enum_5"
],
"title": "CustomType"
}
}
}
—
Reply to this email directly, view it on GitHub
https://github.com/ngx-formly/ngx-formly/issues/3909, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADB4XNNMTFGQK25TRQXOXALZBEP4HAVCNFSM6AAAAABHLR7NG6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4DIMBRHE3TQOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Instead of putting unique items in the widget put it in the schema itself. Unique items is a json schema validation key.

so like this
     "CustomTypes": {
      "items": {
        "$ref": "#/definitions/CustomTypes"
      },
      "type": "array",
      "title": "Custom Types",
      "description": "The Custom type array.",
      "default": [],
      "uniqueItems": true
    }

So when dealing with an object vs a enum how do I tell it what property of that object to base uniqueness off of? I don't want more then one object with the same enum.

When dealing with objects unique items baseline from json schema is defined
to check if any one of the things in the entries is different.
If you want this to change you could write an extension or just a plain
validator that overwrites how unique items works by passing a unique items
key prop or something.
On Tue, May 7, 2024, 4:20 PM John Cutburth @.***> wrote:

Instead of putting unique items in the widget put it in the schema itself.
Unique items is a json schema validation key.
so like this
 "CustomTypes": {
  "items": {
    "$ref": "#/definitions/CustomTypes"
  },
  "type": "array",
  "title": "Custom Types",
  "description": "The Custom type array.",
  "default": [],
  "uniqueItems": true
}

So when dealing with an object vs a enum how do I tell it what property of
that object to base uniqueness off of? I don't want more then one object
with the same enum.
—
Reply to this email directly, view it on GitHub
https://github.com/ngx-formly/ngx-formly/issues/3909#issuecomment-2099328754,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADB4XNNXBT4UDSCEDWFRKILZBFAQHAVCNFSM6AAAAABHLR7NG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJZGMZDQNZVGQ
.
You are receiving this because you commented.Message ID:
@.***>


Analyzer: Functions that return Future accept void/null? #

This is in VM 2.0.0-dev.46.0+google3:
import 'dart:async';

Future<void> runFutureVoid(Future<void> Function() fn) => fn();
Future<Null> runFutureNull(Future<Null> Function() fn) => fn();
Future<bool> runFutureBool(Future<bool> Function() fn) => fn();

void main() {
  // OK
  runFutureVoid(() => new Future.value(null));
  runFutureNull(() => new Future.value(null));
  runFutureBool(() => new Future.value(true));

  // OK
  runFutureVoid(() async => null);
  runFutureNull(() async => null);
  runFutureBool(() async => true);

  // OK? Why?
  runFutureVoid(() => null);
  runFutureNull(() => null);

  // ERROR: 'The argument type '() => bool' can't be assigned to parameter type '() => Future<bool>'.
  runFutureBool(() => true);

  // OK? Why?
  runFutureVoid(() {

  });
  runFutureNull(() {

  });
  runFutureBool(() {

  });
}

There are bugs in the code labeled // OK? Why?. I promised to return a Future<void> and Future<Null>, but I did not - I returned null explicitly in the first callback, and I guess void (or null - I am not sure how the language works right now) in the second callback.
My guess this is because null can be anything (and I guess void too, now?), so even in:
  runFutureBool(() {

  });

... I returned, implicitly either null or void, which... could be a bool? I guess?
Seems really silly though. I did not verify what the CFE does here, unfortunately.

To be clear, I do understand nullable types mean => null is legal.
I just don't understand why no return type (i.e. void) is silently accepted too.

(() {}).runtimeType is () => Null in Dart 2 and () => dynamic in Dart 1. Everything else makes sense given that.
It would be really nice if (() {}).runtimeType was () => void which would make at least some of these cases work as expected. For the other cases we'd need NNBD.

@matanlurey – to triage, can you be explicit. Is this about runtime behavior or static analysis?

I think it's safe to assume that the main issue is static analysis here (also, as written, the program has a compile-time error, so it would have to be edited before we can talk about its run-time properties at all).
For the static analysis, I think it matters that we do not yet have a notion of 'voidness preservation' in the language and tools, which means that we do not make any attempt to track statically that "voidness" gets lost indirectly:
List<Object> xs = <void>['Do not use!']; // OK now, will be a voidness preservation violation.

main() {
  // OK, and this means that we have "forgotten" that we shouldn't use that value.
  print(xs[0]);
}

Without voidness preservation we only have subtyping, and in that domain void is just a top type, that is, Future<void> counts as the same thing as Future<Object> with respect to assignability etc.
import 'dart:async';

Future<void> runFutureVoid(Future<void> Function() fn) => fn();
Future<Null> runFutureNull(Future<Null> Function() fn) => fn();
Future<bool> runFutureBool(Future<bool> Function() fn) => fn();

void main() {
  ...

  // OK? Why?
  runFutureVoid(() => null); // 1.
  runFutureNull(() => null); // 2.

  // OK? Why?
  runFutureVoid(() {}); // 3.
  runFutureNull(() {}); // 4.
  runFutureBool(() {}); // 5.
}

We should note that it is possible to infer the type Null Function() for () {}, which is also sound because it has a well-defined return value of null:
Null function() f = () {}; // OK.

In particular, () {} can take the role of a T Function() for all T, which is probably what we see here.
Similarly, () => null can obviously have the type Null Function(), and also T Function() for any other T. So we have a lot of flexibility.
For each case above we have a type-from-context which is the declared type of the formal parameter ('tfc') and we have these function literals which have such a flexible type. If we choose the type Null Function() for the literals then it all adds up immediately:

The tfc is Future<void> Function(), returned value Null Function() is a subtype: OK.
Tfc is Future<Null> Function() and Null Function() is a subtype: OK.
Tfc is Future<void> Function(), returned value Null Function() is a subtype: OK.

and similarly for case 4 and 5.
So the real question is what type gets inferred for the function literal in each case. Here's a variant of the program which shows that (and I've added the printout after each line):
// Variant of n028.dart which shows the type of the function literal.

import 'dart:async';

void runFutureVoid(Future<void> Function() fn) => print(fn.runtimeType);
void runFutureNull(Future<Null> Function() fn) => print(fn.runtimeType);
void runFutureBool(Future<bool> Function() fn) => print(fn.runtimeType);

void main() {
  // OK
  runFutureVoid(() => new Future.value(null)); // () => Future<void>
  runFutureNull(() => new Future.value(null)); // () => Future<Null>
  runFutureBool(() => new Future.value(true)); // () => Future<bool>

  // OK
  runFutureVoid(() async => null); // () => Future<Null>
  runFutureNull(() async => null); // () => Future<Null>
  runFutureBool(() async => true); // () => Future<bool>

  // OK? Why?
  runFutureVoid(() => null); // () => null
  runFutureNull(() => null); // () => null

  // OK? Why?
  runFutureVoid(() {}); // () => null
  runFutureNull(() {}); // () => null
  runFutureBool(() {}); // () => null
}

So this explain why all those statements are accepted at compile time. You may think that the inference should produce some other results here and there (I don't have a strong opinion on that), but I think that this is what it boils down to.

I just saw the answer from @natebosch which gets to the core very concisely:

(() {}).runtimeType is () => Null in Dart 2
It would be really nice if (() {}).runtimeType was () => void

We actually discussed this recently. Issue #32843 requests just that, and in this comment I just added a test which will directly test at least some basic cases for this behavior.
So it should be coming.

One more comment, to @matanlurey:

I just don't understand why no return type (i.e. void) is silently accepted too.

void is not "no return type", it is a top type (like Object and dynamic) which, statically, has the associated flag "This is not valid". So the returned object could be absolutely anything, and there's nothing wrong with it per se, but the developer who wrote void (somewhere, maybe it was carried around due to inference) has chosen to say that the returned object does not convey any valid information, and hence you shouldn't use it.
For Dart 2 we have checks (under implementation) ensuring that a value of type void isn't used directly, and we expect to introduce rules soon after Dart 2 to check that we don't get to use "a void value" indirectly (that's voidness preservation). But for subtype tests, void counts as any other top type, so you'd have arguments of type void Function() in case 3, 4, and 5, and that's rejected at runtime because it's the same thing as Object Function() for the subtype test, and that's not a subtype of the declared type of the corresponding formal parameter.
So when the return type becomes void, you will have a run-time error in case 3-5, but given that it's caused by a failing downcast you won't get a compile-time error (without --no-implicit-downcast) unless we use exact types to determine that this downcast is statically guaranteed to fail (like
class A {}
class B extends A {}

main() {
  // Compile-time error.
  B b = new A();

  // But this is fine.
  final A a = new A();
  B b2 = a;
}

which fails as indicated with dartanalyzer --preview-dart-2). We have not specified any exact types, though, so the safe assumption is that it will not be a compile-time error.

Is there a bug here or is it working as intended?

It is actually working as intended: #32843 deals with inference of void as a return type, and I don't think any other bugs came up here.


Improve the output of pprint

The current implementation of pprint is pretty naive, in particular when it comes to printing nested data structures (lists and maps).
Concretely, there are several things to improve:

[ ] Sort the map keys.
[ ] Ensure that the map values are actually aligned (current implementation just inserts Tabs and hope they align).
[ ] Print lists and maps on one line when there is enough space to do so.

Citing an example from https://github.com/elves/elvish/issues/1494#issuecomment-1051282865:

This is the current output of pprint $unix:rlimits on my machine:
[
 &nofile=	[
   &cur=	(num 256)
  ]
 &fsize=	[&]
 &nproc=	[
   &max=	(num 2666)
   &cur=	(num 2666)
  ]
 &memlock=	[&]
 &cpu=	[&]
 &core=	[
   &cur=	(num 0)
  ]
 &stack=	[
   &max=	(num 67092480)
   &cur=	(num 8372224)
  ]
 &rss=	[&]
 &data=	[&]
]

But ideally it should look like this instead:
[
 &cpu=     [&]
 &core=    [&cur=(num 0)]
 &data=    [&]
 &fsize=   [&]
 &memlock= [&]
 &nofile=  [&cur=(num 256)]
 &nproc=   [&cur=(num 2666) &max=(num 2666)]
 &rss=     [&]
 &stack=   [&cur=(num 8372224) &max=(num 67092480)]
]

I'm a +1 for improving the output of pprint but I don't think that eliminates the need for dedicated commands for formatting the output of specific pieces of data such as UNIX resource limits or Elvish abbreviations. The pprint command is fundamentally targeted at Elvish programmers not shell users.

For the benefit of anyone stumbling upon this issue I'll note that I've published a module that provides a human friendly command for displaying and manipulating UNIX resource limits. See https://github.com/krader1961/elvish-lib/ (which has been added to the Awesome Elvish page. For example, on my primary macOS server:
> ulimit
resource      current    maximum
========== ========== ==========
core                0  unlimited
cpu         unlimited  unlimited
data        unlimited  unlimited
fsize       unlimited  unlimited
memlock     unlimited  unlimited
nofile            256  unlimited
nproc            5333  unlimited
rss         unlimited  unlimited
stack         8372224  unlimited

Having solved the first two goals (see the pull-requests above) it isn't clear the third goal, compact single-line output, is practical since any value can be a composite type such as a list or map. That makes deciding whether the values should be output on a single line or multiple lines difficult since a nested composite value may fit on a single line but not when embedded in another composite value. The easiest solution is to restrict the single line output to the case where the values are all primitive types; i.e., do not include any composite types such as lists or maps.


⚠ Bofrost.be has degraded performance

In f26df5b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2181 ms

Resolved: Bofrost.be performance has improved in 1ab8a7c.


Entire test bottom is not working....

Challenge Size your Images has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.109 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

If you need help please go to our help room. GitHub is for issues only :smile:


Write a specification for the definition.json files found in our open source PDKs

We should have a specification for the definition.json file found in our open source PDKs.
There is some details in https://bit.ly/open-source-pdks-naming
Example files;

https://github.com/google/skywater-pdk-libs-sky130_fd_sc_hd/blob/main/cells/a2111o/definition.json
https://github.com/google/skywater-pdk-libs-sky130_fd_sc_hd/blob/main/cells/sdfrtn/definition.json

We should also have a JSON schema for the format.

FYI - @proppy / @atorkmabrains

Example of a pull request adding these -> https://github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu9t5v0/pull/8

We should also have a JSON schema for the format.

+1, there are also support for describing using https://pydantic-docs.helpmanual.io/ which could give us typing in the scripts that process them (/cc @joamatab who was looking a something similar for @gdsfactory).


Appstudio update test-component-pac-qbhk

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-qbhk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
30 seconds
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
18 seconds
inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
39 seconds
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
11 seconds
optional-label-check

✅ Succeeded
11 seconds
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


Shell: Inconsistent escape characters highlighting

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.93.1 (Universal)
OS Version: macOS 14.6.1 (23G93)

Steps to Reproduce:

#!/bin/sh

usage() {
    echo "Usage: $0 [OPTIONS] <source directory>"
    echo ''
    echo "-p\t<destination directory prefix>\tDefault: \"$PREFIX\""
    echo "-n\t<batch directory capacity>\t\t\tDefault: (source directory entries)/2"
    echo "-v\tShow version"
    echo "-h\tShow usage menu"
}

VSCode does not highlight the \ts the same way that it highlights the \"s.

Only these characters support escaping $\n`"\


Throw errors when the response throws and error

Currently when a request fails it won't throw an error. This makes it hard to detect what went wrong (404, 500, etc...). I'd like to add some error handling decorators to the API: https://uplink.readthedocs.io/en/stable/user/quickstart.html#response-and-error-handling

I was wrong. Uplink does throw errors, ACA-Py doesn't throw an error when the schema is not found, but returns 200 with an empty schema object

See https://github.com/hyperledger/aries-cloudagent-python/issues/1487


Ability to add classes to the table wrapper

Let's say I need to add a 'table-responsive' class or another custom class.

@MikeSha sorry for lately reply, I will check this asap, thanks your contribution

@MikeSha would you please change your target branch to develop?  thanks!

@AllenFang Sure!
Done!

@AllenFang Sure. I'll do that a little bit later.

@AllenFang I created a new PR https://github.com/react-bootstrap-table/react-bootstrap-table2/pull/325. This one could be closed.


Support GitHub files

Is your feature request related to a problem? Please describe.
Scala-cli already supports running code from gists, so I find it a bit surprising that it doesn't work with normal GitHub links.
While Gists are nice for code snippets, the flat structure limits what can be done (e.g. to write an digital book where each folder contains a chapter). This could be addressed by supporting links to directories and files in a repo.
Describe the solution you'd like
I would like for GitHub links (directories and files) to work just as gists.
Describe alternatives you've considered
Pointing users to the relevant files in https://raw.githubusercontent.com, but this wouldn't support directories (so with multiple files it becomes a huge command).

@JD557 Thanks for reporting.
ScalaCLI already has the feature to download and run single files from GitHub. For example, to execute my simple Scala app available at https://github.com/lwronski/ls-demo/blob/main/Ls.scala, you can run the following command:
scala-cli https://raw.githubusercontent.com/lwronski/ls-demo/main/Ls.scala

ScalaCLI currently doesn't support downloading directories from Github, only individual files. Therefore, I will keep open this issue.


Make EventSourceLogger public

I would propose to make EventSourceLogger public.
The reasoning is two-fold:

Consistency with the other ILogger implementations in this namespace
Users then would have the possibility either derive from it or compose inside their custom ILoggers.

Dup: https://github.com/aspnet/Logging/issues/846


Fork Cosmic Terminal and retheme

We need to get an ICED theme made for AcreetionOS for the cosmic terminal. Alacritty, the core of the Cosmic Terminal is great. We need to look at different approaches to making this happen @Oliveoilollie

Alrighty, I'll look into it and we will talk about it in a vc chat

I appreciate the quick response.  What are you thinking in regards of basing the terminal emulator? Cosmic or Alacritty? I feel that is the first question we need to answer. My vote is build on top of Cosmic, as most of the work is already done, and I trust them as our upstream more than a hectic open source project. But feel free to give your opinion, as I'm wanting to hear what you think. @Oliveoilollie

i feel cosmic is gonna be better in the long run. our focus is to make a user friendly distribution. and i think cosmics terminal will fit that bill. yet its still gonna be almost as performant as Alacritty

I agree with this entirely. Cosmic is the base for sure then. I am going to look at the color themeing tonight, as it doesn't seem that complicated from what I'm seeing.


Add support for curator jobs in hiera

This looks good to me.

Nice addition - thanks!

Nice addition - thanks!


Merge is not recorded in TFS if a file was added in merged branch

The following scenario fails for me:

Have two branches in TFS: $/MergeTest/Master and $/MergeTest/Branch1 (which is branched directly from $/MergeTest/Master). Corresponding branches in git repo are master and Branch1; remote branches are tfs/default and tfs/Branch1.

Commit a new file to Branch1:
$ git checkout Branch1
$ echo foo > new-file.txt
$ git add new-file.txt && git commit -m "Some commit"
$ git tfs rcheckin -i Branch1

Merge Branch1 into Master:
$ git checkout master
$ git merge Branch1
$ git tfs rcheckin -i default

The problem: git commit created by rcheckin is not a merge commit; in TFS this commit lists changes to $/MergeTest/Master/new-file.txt as [add] (instead of [merge, branch]).
I'm using VS 2012 and TFS on visualstudio.com and TFS 2013 (installed on company server).
$ git tfs --version
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (32-bit)

Think I may have just encountered this, just pushed a merge to a large repo and it wasn't recorded as a merge by TFS. It was a big change with just about every variation you could think of so could well have been this.
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (64-bit)

We are trying to clean up old/stale issues. As this issue hasn't seen any updates (or progress with pull requests) in quite a while, the issue will be closed. In case further information is provided, we can revisit that and reopen the issue again.


Release v3.5.0

Pull Requests
Please note that we accept pull requests from anyone, but that does not mean it will be merged.
What kind of change does this PR introduce?

Other

Did this PR introduce a breaking change?
A breaking change includes anything that breaks backwards compatibility either at compile or run time.

No

v3.5.0 - New VisionPortal and VisionProcessor API
This is the 18th release for EOCV-Sim
Changelog:

Addresses the changes made in the FTC SDK 8.2 to prepare for the 2023-2024 season:

EOCV-Sim's Viewport implementation has been changed to one using Skiko (Skia) rendering - to address new features implemented in EasyOpenCV v1.7.0
The VisionPortal & VisionProcessor interfaces have been implemented onto EOCV-Sim - VisionProcessors are treated just like OpenCvPipelines and are automatically detected by the sim to be executed from the user interface.
In order to use the VisionPortal API, OpModes have been added onto the simulator - a new "OpMode" tab on the user interface has been added to address this addition. NOTE: OpModes are only limited to use VisionPortal APIs, other FTC SDK apis such as hardware DcMotor have not been implemented.
A new public API for android.graphics has been adding onto the simulator, translating android.graphics API called by the user into Skiko calls, adding compatibility to the new features in EasyOpenCV v1.7.0 related to canvas drawing.
AprilTagProcessor has also been implemented straight from the SDK, allowing its full API to be used and attached to a VisionProcessor - see this example OpMode.
AprilTagDesktop plugin has been updated to match EOCV-AprilTag-Plugin v2.0.0
Support for Apple Silicon Macs has been added to AprilTagDesktop
Several quality of life upgrades to the UI

Bug fixes:

Fixes issues related to pipeline and input source selection - UI components now exclusively react to user interactions as opposed to past versions where changes triggered by EOCV-Sim were picked up as user-made and caused several issues


to be honest the front page is very confusing

After reading the front page I still don't know how to make a cli.
Seems to have lots of details that's distracting like ➜  Comonicon git:(master) What's that for? It's master branch of your package. But it doesn't help me understand how to make a cli?
A simple step by step guide on how to make a CLI would be nice.

I think as for the front page, it's more for showing off the features first, thus I put some benchmarks using hyperfine.
have you read https://rogerluo.me/Comonicon.jl/dev/#Quick-Start ? this is the document, does this help?

I hope this is resolved by https://github.com/Roger-luo/Comonicon.jl/commit/512a4de2ad577849f1701fc6609b9a43fcc50c80 there really any more steps you need to create your first CLI. Just copy what's in the code block of getting started, and run julia main.jl -h, it will print help msg and that's it.

I'm closing this for now, but feel free to ask other questions.


make it a jquery plugin

jQuery is a tool developed by john resig to make development easier. Why don't you take adventage of it?

There's no deal with uselessly wrapping this into a jQuery plugin. Also the code you submitted wouldn't even work.

jQuery seems to have no longer a bright future. jQuery was important to allow developers to create JavaScript code without the necessity to care about browser differences of the JavaSript implementation. But modern browsers are getting more and more homogeneous reg. JavaScript (ECMA 5.1 made this happen and the upcoming version 6 might cause difficulties for additional layers like jQuery to utilize all language features. So it is time to dig deeper into native JavaScript soon).
For many scenarios vanilla JS (in its modern forms) is perfectly fine:
http://toddmotto.com/is-it-time-to-drop-jquery-essentials-to-learning-javascript-from-a-jquery-background/
If you need bulky stuff then the future will more likely move towards JS frameworks (like AngularJS):
https://www.airpair.com/angularjs/posts/jquery-angularjs-comparison-migration-walkthrough
So there is zero need for a jQuery plugin of screenfull.js.


Edit icon should only show up on one's own profile

See https://github.com/participedia/usersnaps/issues/51

This has already been covered in this commit.
https://github.com/participedia/frontend/commit/852ba63d1215400dcf35d59cbfa887d15cc076ae


Region FromRhino() has a different normal vector than the normal of the RH circle.

For circles the FromRhino() method gives back the inverse of the normal vector of Rhino circle.
https://github.com/strusoft/femdesign-api/blob/d86bc1b77743c2d2b0d8ce9ba559a8f542581968/FemDesign.Grasshopper/Geometry/Convert.cs#L566-L734

in which case does this happen? do you have an example such that the bug can be reproduced?

https://github.com/strusoft/femdesign-api/blob/d86bc1b77743c2d2b0d8ce9ba559a8f542581968/FemDesign.Grasshopper/Geometry/Convert.cs#L607-L608

It is not obvious why contours made of circles have normal in opposite direction. users exploring this might be confused in certain situations. this should be obscured or commented better.

@andosca
we have spotted the issue while testing "Surface.Temperature load".
A gh user has to specify the TopValue and, in order to understand what Top is, it would display the Normal Vector of a Circle.
The issue seems to occur for circle only and it should be an issue for only grasshopper users.

@Marco-Pellegrino
This is an issue with the .struxml-schema. Regions made up of a circle has a normal vector in the opposite direction than you would expect - for some reason. If we change this the region will be corrupt in FEM-Design instead. So we need to obfuscate this from the user and give them the "correct" information.


Off-Peak Next Start/End fail to populate and in turn breaks intelligent dispatching

Describe the bug
I had a strange scenario where my car failed to charge and in turn when asking for 100% battery from octopus I essentially got a full day of intelligent rate. This causes the funtion get_off_peak_cost in __init__.py to fail to get a rate because it expects the len(rate_charges) to be 2 or 3, but in this case it landed on 1 (all day cheap rate). The result is that the rates are off, then off peak measures are off, dispatching doesn't think it's on (when it is) and finally all my automations fail.
Potential Fix - I tested a simple fix to just increase the today_end to be an extra day out (2 days) and get as many rates as possible in case between now and the end of the day there are only cheap rates to see. However, you may know a better fix.
Reproduction steps
The planned dispatches I had were:
start: '2024-09-30T12:01:00+01:00'
end: '2024-10-01T04:00:00+01:00'
charge_in_kwh: -37.03
source: null
location: null

See the attached log files for debug messages I added to track this down
Expected behaviour
There is always an off peak next start/end and in turn the dispatching always works. I didn't get to why dispatching thought it was off but fixing the off peak also fixed the dispatching state, my automation looks for both.
Tariff Code
E-1R-INTELLI-VAR-22-10-14-J
Integration Version
v12.2.1
Home Assistant Version
2024.09.3
Fresh Install?
Not specified
Home Assistant Logs
See attachment
OffPeakLog.md
Confirmation

[X] I confirm that I cannot find my solution within the documentation
[X] I confirm that I cannot find my solution within the FAQ

Hello. Sorry youve encountered this issue. I think I know where it's gone wrong, but will take a further look (and hopefully a fix) when I get some more time.


Verify that the expected groups are created in the init container

systemd-network is not in the Tumbleweed init container

well, the question if thats a testsuite problem or a container problem

I've removed it for now

it's not clear this serves any value. lets not bother with it.


kong route path contains request query params,when invoke kong return 'no Route matched with those values'

Is there an existing issue for this?

[x] I have searched the existing issues

Kong version ($ kong version)
2.2.0
Current Behavior
I created a route, this route path is as '/geo-server/s?service=WFS&version=1.0.0&typeName=wk_cube:11f0fb2e4bd09cbf841d46&request=GetFeature&outputFormat=application/json';
as i say, i try take request query params in path;
but when i use gateway type to invoke this api, the gateway return 'no Route matched with those values'
Expected Behavior
use gateway type to invoke api(/geo-server/s?service=WFS&version=1.0.0&typeName=wk_cube:11f0fb2e4bd09cbf841d46&request=GetFeature&outputFormat=application/json) Can be called normally
Steps To Reproduce
The issue happens common.

Anything else?
hi, kong gateway (v2.2.0) if support this case?
eg:
route path contains request query params, '/geo-server/s?service=WFS&version=1.0.0&typeName=wk_cube:11f0fb2e4bd09cbf841d46&request=GetFeature&outputFormat=application/json'
thanks !

Hello @zhangqingzhou1024,
thank you for reaching out.  If I understand correctly, you are trying to route based on the query parameters of the request.  Unfortunately, Kong does not currently provide this feature - The query parameters are stripped from the path before the router processes it.  Being able to route based on query parameters is not a completely new requirements and you may find some suggestions how to work around this limitation here: https://github.com/Kong/kong/issues/4919
Kind regards,
Hans

Hello @zhangqingzhou1024,

thank you for reaching out. If I understand correctly, you are trying to route based on the query parameters of the request. Unfortunately, Kong does not currently provide this feature - The query parameters are stripped from the path before the router processes it. Being able to route based on query parameters is not a completely new requirements and you may find some suggestions how to work around this limitation here: #4919
Kind regards, Hans

Thanks for your reply， I have a general overview of the present situation，Try to handle it in a different way。

Hello @zhangqingzhou1024,
thank you for reaching out. If I understand correctly, you are trying to route based on the query parameters of the request. Unfortunately, Kong does not currently provide this feature - The query parameters are stripped from the path before the router processes it. Being able to route based on query parameters is not a completely new requirements and you may find some suggestions how to work around this limitation here: #4919
Kind regards, Hans

Hello @zhangqingzhou1024,
thank you for reaching out. If I understand correctly, you are trying to route based on the query parameters of the request. Unfortunately, Kong does not currently provide this feature - The query parameters are stripped from the path before the router processes it. Being able to route based on query parameters is not a completely new requirements and you may find some suggestions how to work around this limitation here: #4919
Kind regards, Hans

Thanks for your reply，my english is very poor, I have a general overview of the present situation,Try to handle it in a different way。

hi @hanshuebner
Thanks for your reply!
My english is very poor, I have a general overview of the present situation,Try to handle it in a different way。


[些細]HeatmapのAll Submissionsのポップアップ

"1 submissions" → "1 submission"

"0 submission" → "0 submissions"

:ok:


[feat] Redis cluster load unbalanced across nodes

Is your feature request related to a problem? Please describe.
When I load ttest aedes with redis persistence I found that only one of the six nodes gets very high CPU (100%) and the others are under 5% during the load test. The reason being retained records are all stored on that node. Digging into the code I see that all retained records are stored with 'retained' key and not something like 'retained:344545' . This can explain why the load is not equaly distributed.
Describe the solution you'd like
We could store retained records in the format retained:344545
Describe alternatives you've considered
Additional context

Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Would you like to send a Pull Request to address this issue? Remember to add unit tests.


Specify Requirements for basic differencing analysis

Write and circulate a document saying what might be possible for discussion
Target Audience:  TREs , WOP1

Since we have the commands and the table it should not be too hard to capture:
Each row/column name in a table:
and what the set of unique values for them is
and what the total count of records in a table is.
That would allow for some simple checks against differencing
at minimum we could want which outputs report on similar attributes
and if two tables share the same attributes but have different counts, we would know that an exclusion operation had ben applied which might leave the excluded people vulnerable
e.. if table cell for some combination of attributes has count 11, and in a second table with smaller total combat, the cell has value 10, then we can infer that a table made from the excluded set would have 1 in that cell.


re.match for select_branch_options rather than re.fullmatch

Hello 👋
Is the use of re.match rather than re.fullmatch a bug or a feature ?
Here
Using re.match will return something True if a substring of the branch name match the python regex.
So for example, '([0-9]*)' will match "1-correct-issue" as well as "my-br4nch". Using fullmatch should force the branch name to match the python regex pattern, as documented in the Multibranch Releases documentation.

Hey @zckv - it's a feature, because if you want an exact match you can use the ^$ anchors for the start/end of the string. But the docs don't call this out explicitly so agreed it might be helpful to state that the match is done with re.match


websocket does not close when wlan/lan is disconnected.

Hi. We are trying out websockets on a raspi and udoo boards. Most things work fine. But:
Giving the following setup
[Tablet] <=> WLAN <=> [Udoo-as-WLAN-Accesspoint] running a ws-server using server_ev.lua. Our server produces a data stream, which is send over the socket in a regular schedule.
Now if the browser on the tablet opens a webpage with js accessing the websocket => Data is flowing.
If I close the browser, then ws:on_close() is called and everything is fine.
But, (assuming the webpage is opened and data is flowing) when I disconnect the tablet from the WLAN, then the websocket lua script begins to eat more and more memory and cpu-time (uptil 100%). The same is true, if I use the LAN-Port and change the ip-address of the udoo, while a connection is active.
I have added some debug code:
:::
local ok=ws:send(state)
if type(ok)~="number" or ok~=0 then
    log("ws:send()=%s\n",vis(ok))
end
:::

and I get something like this (first number is timestamp HHMMSS)
051540:ws:send()=388
051540:ws:send()=832
051541:ws:send()=1246
051541:ws:send()=1660
051541:ws:send()=2074
051542:ws:send()=2488
051542:ws:send()=2902
:::

number is increasing until memory is completely used up
How do I catch this "connection lost" condition? Is it possible to set a timeout for the send? Or to set a limit of send data in queue?

could you make a small test using basic tcp? using luasocket...
i don't know if luasocket forwards loss of network interfaces in any way...

Did as requested.
Server script simple-server.lua:
local socket=require"socket"
local server=assert(socket.bind("*", 23456))
print("waiting...",server:getsockname())io.flush()
for client in function() return server:accept() end do
	print("conn:",client:getsockname())io.flush()
	client:settimeout(10)
	while true do
		local msg, err = client:receive()
		if err then
			print("err:"..err)io.flush()
			break
		end
		print("rx:"..msg)io.flush()
		msg=msg+1
		print("tx:"..msg)io.flush()
		client:send(msg.."\n")
	end
	client:close()
	print("waiting...",server:getsockname())
end

Client script simple-client.lua
local socket=require"socket"
local client=assert(socket.connect("<IP_ADDRESS>", 23456))
client:settimeout(10)
local msg=100000
while true do
	print("tx:"..msg)io.flush()
	client:send(msg.."\n")
	local line, err = client:receive()
	if err then
		print("err:"..err)io.flush()
		break
	end
	print("rx:"..line)io.flush()
	msg=msg+10
	socket.sleep(1)
end
client:close()

Starting server script on the udoo-board. Output.
waiting...      <IP_ADDRESS> 23456

Starting client script on a acer windows 10 tablet:
tx:100000
rx:100001
tx:100010
rx:100011
tx:100020
rx:100021
tx:100030
rx:100031
tx:100040
rx:100041
tx:100050
rx:100051
::::

Server says:
conn:   <IP_ADDRESS>    23456
rx:100000
tx:100001
rx:100010
tx:100011
rx:100020
tx:100021
rx:100030
tx:100031
rx:100040
tx:100041
rx:100050
tx:100051
rx:100060

Now, when I disconnect the WLAN connection (at the acer tablet),
client immediately stops with message
:::
rx:100111
tx:100120
rx:100121
tx:100130
err:closed

The server stalls for the timeout (10 seconds), an then says:
err:timeout
waiting...      <IP_ADDRESS> 23456

So, the only thing, the server lua script gets, is a timeout.
Then I change the server script to
    ::::
    client:settimeout()
    ::::

same test procedure, at disconnect the server stalls forever, no error message.
Then I change the server code to
    ::::
    client:settimeout()
    client:setoption("keepalive",true)
    ::::

same test procedure. After 2 hours (to be precise, after 7875 seconds) master gets the timeout.
err:timeout

This can be controlled via standard unix files
/proc/sys/net/ipv4/tcp_keepalive_intvl:75
/proc/sys/net/ipv4/tcp_keepalive_probes:9
/proc/sys/net/ipv4/tcp_keepalive_time:7200

eg, writing 600 to /proc/sys/net/ipv4/tcp_keepalive_time reduces the timeout to 10 minutes.
After all this isn't luasocket specific. Writing a client in C using the socket interface would reveal the same results.
I don't know, how this would be done on windows systems ... and I don't even care :grin:

After all I did a workaround (I wouldn't call it solution, but it's good enough for the moment).
I changed my code in the ws-server to
local ok=ws:send(state)
if type(ok)~="number" or ok~=0 then
    ws:close()
end

With this the connection gets closed (after some minor delay), when the send is incomplete.
It's not perfect, but it keeps memory and cpu usage clean.

@jjvbsag thanks for the detailed results. i think your solution is good. dealing with unstable tcp connectins / network interfaces always is tough. if i understand correctly, you are handling this now in your "application logic" (and not by patching lua-websockets), is this correct?

@lipp Yes, I only changed the code in my ws-server script, the websocket library scripts themself are unchanged.

:100:


SFTP issue

after running the script and doing a full system update, and applying the bash prompt setup, I can no longer sftp into my system.
SFTP session can not be started. Reason: Timeout waiting for response from SFTP subsystem

I think I fixed the issue. I added this
if [[ $iatest -gt 0 ]]; then
to check if the shell is interactive.

I think the issue should be created in https://github.com/ChrisTitusTech/mybash


The app should prioritize exposureSummary by daysSinceLastExposure rather than the most recent check

What?
If the app is in exposed state, it uses daysSinceLastExposure to clean up the state back to monitoring when daysSinceLastExposure is more than 14 days. However, the app should prioritize exposureSummary by daysSinceLastExposure rather than the most recent check.
For instance:

Phone A was in close contact with phone B 2 days ago who submitted their keys to server yesterday.
Phone A was in close contact with phone C 10 days ago who submitted their keys to server today.

=> In this case, the app should keep exposureSummary from phone B and ignore phone C because phone B is most recent contact.
Related issue:
daysSinceLastExposure cannot be used for calculating time to reset back to monitoring state because daysSinceLastExposure is returned from EN framework as part of exposure summary and never changes. We need to find a way to retrieve lastExposureTimestamp instead. There are two options:

Option 1: do in js layer. This bases on assumption that EN framework will return exposure summary within the same day of the request. If so, lastExposureTimestamp = today() - daysSinceLastExposure.
Option 2: do in native layer. Before feeding TEKs to the framework, the native layer should store current timestamp somewhere and retrieve it later. The timestamp could be combined with token on Android (iOS TBD).

Action items:

[x] Add lastExposureTimestamp via js layer as it's an okay approach.
[ ] Change lastExposureTimestamp to use native layer as a follow up PR.

Reopen for a follow up PR

@henrytao-me anything further one this?

Closed - will reopen as needed.


fix: support deserialization of all known examples

in https://github.com/CycloneDX/specification/tree/master/tools/src/test/resources
there are some example data, that do not work wit hthe deserialization of this library.
lets fix this

@jkowalleck - if I'm reading the current code in main correctly, this is already covered by the work you did to add schemaTestData used in test_validation_json.py and test_validation_xml.py?

[...] this is already covered by the work you did to add schemaTestData used in test_validation_json.py and test_validation_xml.py?

Nope. this issue is not about validating test data, but about deserializing the test data.
see how https://github.com/CycloneDX/cyclonedx-python-lib/pull/538 failed?

Gotcha! Will work on #581 and #422

related: #600

:mega: this issue is blocked by

#422
#561
#581


Claiming multiple claimpays at once doesn't seem to work.

Check out https://eosq.app/tx/7f0271099e22efcdbf4d9d272a972bcf5a23aadd74abe21c3c5d8b6107d0e7a3

This one worked: https://eosq.app/tx/3711de77c6dbe78003c75d8db8eb6ed3c366b5269f235dd6a360eb5c0f41a9f7
These two did not:
https://eosq.app/tx/422b1729275e32871917a9becf45466e943a16de9f010d9715092c27e5b0a813
https://eosq.app/tx/b9ab0c6d7bc03986d1d6fc10391055f85536bb9849cd9c96138019366d4de90e
This is the error:

deferred transaction with the same sender_id and payer already exists

Seems it's not taking into account the different payid's?

What's the latest on this? I have two payments queued up to test, if we want to (currently the button is disabled in the UI)

@lukestokes it is fixed in the contract. however the fix doesn't work for pending pays before the fix was applied. at least that is my understanding. so I have disabled the button until all if not most "old" pays are claimed.

I believe this is done but needs to be tested for a while to ensure it is fixed since the cause was not 100% understood. Will leave the issue open for now.

The new way of claiming pay means that even if the payments fail, the pending pay row is not deleted.  This means it is a lot safer now to try to claim all so i think we should enable the button in the mainnet member client again and ask someone to test @lukestokes


latest updates to dev tile

Description

Adding documentation updates to the catalog json (https://github.com/terraform-ibm-modules/dev-rag/issues/44)
ES update to v1.20.2
Reverse code engine / roks and alm in the stack (https://github.com/terraform-ibm-modules/dev-rag/issues/49)

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Waiting on this fix: https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/pull/301

Need a fix for https://github.com/terraform-ibm-modules/terraform-ibm-rag-sample-da/issues/184 but going to run pipeline to see how everything else is

/run pipeline

/run pipeline

/run pipeline


DOC: Fix groupby nth

[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff

Current API doc looks incorrect. Fixed it to meet #11039.

http://pandas-docs.github.io/pandas-docs-travis/generated/pandas.core.groupby.GroupBy.nth.html

cc @behzadnouri

[Current coverage][cc-pull] is 85.23% (diff: 100%)

Merging [#13810][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master     #13810   diff @@
==========================================
  Files           140        140
  Lines         50415      50415
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          42971      42970     -1
- Misses         7444       7445     +1
  Partials          0          0

Powered by Codecov. Last update [a3cddfa...5d96464][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/a3cddfa9c4302c13747509caef7a056159010adf...5d96464e33d30d717bbe30f45cf029942cf8870a?src=pr
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13810?src=pr

Can you show an example with as_index=False as well?

@jorisvandenbossche good idea. added.

thanks!


pool.go doesn't have any explanation of what it's for

I'm trying to decide whether to use a PoolClient, but pool.go doesn't say what benefit it has. I guess it's supposed to be for performance but I don't see any benchmarks that would give evidence for that.
I hope this isn't taken as too critical. I'm glad you wrote this library and I'm getting nice results from it.

Okay, I see there's a section in the README, but I still think it would be more convincing with a benchmark.

Thanks for the advice. I'll try to come up with one.
Or, you can send me a PR for it :-)


[JSONCPP] Relocation R_X86_64_32S against `_ZTVN4Json17CharReaderBuilderE' can not be used when making a shared object

Hello,
I'm trying to build the OSVR-Core on an Ubuntu 14.04 but I get this error when executing make after successfully running cmake .. -DCMAKE_INSTALL_PREFIX=~/osvr.
Here is the error:
Linking CXX shared library ../../../lib/libosvrCommon.so
/usr/bin/ld: /root/jsoncpp/build/release/x86_64-linux-gnu/libjsoncpp.a(json_reader.cpp.o): relocation R_X86_64_32S against `_ZTVN4Json17CharReaderBuilderE' can not be used when making a shared object; recompile with -fPIC
/root/jsoncpp/build/release/x86_64-linux-gnu/libjsoncpp.a: error adding symbols: Bad value
collect2: error: ld returned 1 exit status
make[2]: *** [lib/libosvrCommon.so.0.6] Error 1
make[1]: *** [src/osvr/Common/CMakeFiles/osvrCommon.dir/all] Error 2
make: *** [all] Error 2

And here is the Dockerfile with which you can reproduce the error:
FROM ubuntu:14.04

# Set workdir
WORKDIR /root/

# Add repo for CMake 3
RUN apt update
RUN apt install -y software-properties-common
RUN add-apt-repository -y ppa:george-edison55/cmake-3.x

# Install tools
RUN apt update
RUN apt -y upgrade
RUN apt install -y git cmake build-essential libboost-all-dev libusb-1.0-0-dev libopencv-dev

# Install libfunctionality
RUN git clone https://github.com/OSVR/libfunctionality.git
RUN mkdir libfunctionality/build
WORKDIR /root/libfunctionality/build
RUN cmake ..
RUN make
RUN make install
WORKDIR /root/

# Install jsoncpp
RUN git clone https://github.com/vrpn/jsoncpp.git
RUN mkdir -p jsoncpp/build/release
WORKDIR /root/jsoncpp/build/release
RUN cmake -DCMAKE_BUILD_TYPE=release -DBUILD_STATIC_LIBS=ON -DBUILD_SHARED_LIBS=OFF -DARCHIVE_INSTALL_\
DIR=. -G "Unix Makefiles" ../..
RUN make
RUN make install
WORKDIR /root/

# Install OSVR
RUN git clone --recurse-submodules https://github.com/OSVR/OSVR-Core.git
RUN mkdir OSVR-Core/build
WORKDIR /root/OSVR-Core/build
RUN cmake .. -DCMAKE_INSTALL_PREFIX=~/osvr
RUN make

Thank you very much for your help :)

I found my mistake, it is needed to build jsoncpp like this:
RUN cmake -DCMAKE_BUILD_TYPE=release -DBUILD_STATIC_LIBS=OFF -DBUILD_SHARED_LIBS=ON -DARCHIVE_INSTALL_\

It has to be a shared lib for -fPIC to be applied (I think)


Add deletionDate to LocalAccount

Readiness checklist

[x] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.
[x] I self-reviewed the PR.

@Siolto Please note that the DatawalletSynchronizedEvent is now published in transport instead of the app-runtime, which is why its namespace changes from "app.datawalletSynchronized" to "transport.datawalletSynchronized".

@Magnus-Kuhn please re-check your comments and resolve if you are ok with the answers. If not we have to discuss the things.


feat(tools): knobs

What I did

add elements to tools
add pft-element-knobs
add knobs demos to dev server

Testing Instructions

run the dev server
http://localhost:8000/
click on knobs

Notes to Reviewers

@bennypowers this is great!  Thinking about picking up pf-drawer to support this.  My thinking is maybe the knobs could be out of the way on initial load then toggleable via a drawer.  Thoughts?

There can be quite a lot of content, more than a side drawer
See text-input for example
Maybe a bottom panel

Maybe a bottom panel

Yeah I also noted the resizable drawer too which might be nice on certain viewport sizes.


fix: Fix broken configuration link

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
Fixes broken link to Falco configuration page
Which issue(s) this PR fixes:
No issue
Special notes for your reviewer:

Welcome @theoretick! It looks like this is your first PR to falcosecurity/falco-website 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

eeb7e8a fix: Fix broken configuration link

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kris-nova
If they are not already assigned, you can assign the PR to them by writing /assign @kris-nova in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 3d5c834c74a3a30175b299757a977f4c3db53ca1


Add DeutscheWelle FeedExpander bridge.

Adds a FeedExpander bridge for the Deutsche Welle RSS feeds from

https://corporate.dw.com/en/rss/s-31500
https://corporate.dw.com/de/rss/s-9773

This is my first time working with PHP, so please let me know any mistakes I may have made. I based this approach mostly on the ZeitBridge.

looks kinda slow but if it works it works


Won't upgrade on ubuntu 21.10 (deb package)

I was running 5.6.2, went to settings/upgrade but no upgrades were available nor searched.
I had to download and upgrade manually.
Please solve.

@jasmines - please search for other tickets that have the same issue.
Closing as a duplicate of #329


DOCS-#2941: document modin/data_management module

What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #2941
[x] tests added and passing

While reading documentation for data_management/functions/default, you may find that some classes or methods from that module are useless or duplicates logic of each other. I've fixed much of this inconsistency by refactoring that module in a separate branch (dc36e77) on top of the commits from this branch, we can apply this refactoring after merging this PR.
Built docs for this PR: https://modin--2956.org.readthedocs.build/en/2956/index.html
Affected sections:

https://modin--2956.org.readthedocs.build/en/2956/flow/modin/data_management/factories.html
https://modin--2956.org.readthedocs.build/en/2956/flow/modin/data_management/functions.html

@vnlitvinov ready for review

@vnlitvinov

What's left for this to move from "draft" stage into "ready"?

Your review for factories.rst that I've just added
Final version of Anatoly's script from #2968, to check and fix possible doc-style mismatches

@YarShev @vnlitvinov any other comments?

LGTM except this.

Built docs for this PR: https://modin--2956.org.readthedocs.build/en/2956/index.html

@dchigarev Looks good, please rebase and it can be merged.

@devin-petersohn rebased and CI is green

rebased on master to fix merge conflicts

@devin-petersohn CI is green


fix(Dribbblish): invisible playlist play buttons

Added some JS to change the color of the playlist play button icons if the background color and color are the same which happens in the case of certain color schemes like base, dark and purple. Resolves #852.
Before:
Purple scheme

After:
Purple scheme

Schemes not affected by the bug have no change in button colors.
I fixed the bug by changing the color properties of two style classes in head fpCtYO [for the small play buttons next to playlist thumbnails] and gHYQaG [for the large play button in a playlist or artist page] to --spice-text which would make sure that the color integrates with the theme. The style is added to head to make the CSS changes permanent for a session.
Please tell me if I'm doing something wrong or if something could be made better.

If you still want to fix this up so that it works go ahead and ill merge it.

Superseded by #874