Add correct diagnostic for missing parentheses

Resolves #1925

@swift-ci please test

@swift-ci please test

@swift-ci please test macOS

@ahoppen the job fails here: https://ci.swift.org/job/swift-syntax-PR-macOS/3177/console
Not sure why.
When I add the tests in the test suite they succeeds correctly.

@swift-ci please test

@swift-ci please test

It’s failing at the following test case. The in in the for statement is critical to reproduce the failure. Could you look into why it’s failing? If you can’t find it, I can try debugging as well.
state.withCriticalRegion { (1 + 2) }
for action in tracking {
  action()
}

@ahoppen found it.
We had inside canParseClosureSignature this !lookahead.at(.endOfFile, .rightParen, .keyword(.in)) and it didn't stop before hitting the in keyword.
Changed rightParen -> rightBrace

@swift-ci please test

@swift-ci Please test Windows


Add new page

I want to add new par for forget the password , add or edit Fiscal Years etc..
what is the easy and best procedure to that .
thanks

Irfan, for add/edit "Fiscal Year" you can start this by adding a new menu
under "Organization Settings" (i.e. "Fiscal Years). Once the user click the
menu, a list page of fiscal years is displayed in grid. The you have "New"
and "View" link button above. Check on others page implementation.
For the change password, you can suggest your idea. I kinda see this to be
on a "Profile" page. But we don't have that yet.
On Thu, Nov 3, 2016 at 2:09 PM, Irfan Samad<EMAIL_ADDRESS>wrote:

I want to add new par for forget the password , add or edit Fiscal Years
etc..
what is the easy and best procedure to that .
thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AccountGo/accountgo/issues/80, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKMkY1Jsx0N3EnXBcsIUhHHb0x4af4Vnks5q6XqzgaJpZM4KoB98
.

Thanks for the answer, i add the new line in _Layout.chtml

Fiscal Years

now how can i create FiscalYear method in controller .. because you are using some other tolls got generate view and models?
one i complete this i will give you forget pass process
thanks

You may need to create new view and models. The usual mvc way.
On Thu, Nov 3, 2016 at 7:10 PM, New Xp<EMAIL_ADDRESS>wrote:

Thanks for the answer, i add the new line in _Layout.chtml
Fiscal Years http://~/administration/FiscalYears
now how can i create FiscalYear method in controller .. because you are
using some other tolls got generate view and models?
one i complete this i will give you forget pass process
thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/AccountGo/accountgo/issues/80#issuecomment-258115430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKMkY_q1E1aOTOqps1CDAd1NlgI5bB5Rks5q6cEtgaJpZM4KoB98
.

yes! i am trying.. currently working on controller classes in api and AccountGoweb

Done

Great! you can do pull request.


Failed to parse SAS ISO8601 dates

Recently, we have received another dataset, the data contains date related information in SAS IS8601 format, more specifically, the dates are in IS8601DA and IS8601DT formats.
Here is a link explains different formats: http://www.lexjansen.com/nesug/nesug10/ph/ph11.pdf
My tests show that Parso does not handle these formats yet, any chances of getting them supported soon?

Added IS8601DA and IS8601DT formats.


v0.18.4

Fixes AppVeyor binary deployment HTTPS issue caused by NPM config

@defunctzombie Ready to NPM publish.


Task 1.3.1 does not carry corresponding when switch

Describe the Issue
task for rule 1.3.1 should have when: ubtu22cis_rule_1_3_1 but doesn't
Expected Behavior
when: ubtu22cis_rule_1_3_1 must be present
Actual Behavior
when: ubtu22cis_rule_1_3_1 is not present
Control(s) Affected
1.3.1
Environment (please complete the following information):
n/a
Additional Notes
Anything additional goes here
Possible Solution
Enter a suggested fix here

Thanks for raising this bug. We will add to a upcoming release to devel.

This was added to PR [#22] and is now in devel feel free to test out and let us know.


Daytime Charging?

Hi Nathan,
Many thanks for this great programme.  I installed HA and FoxESS EM in the middle of Feb this year and I have stayed up late each night to see how the Energy management system works. After 6 weeks of patient viewing, we finally hit pay dirt this morning when it was fairly clear that the forecast sunshine would not be sufficient to keep my batteries topped off.
According to my statistical guesstimates, my solar system should become pretty much self-sustaining from 20 March onwards so I was expecting to shut the system down until next November…..
So, at the allotted time this morning/last night, the batteries started charging all by themselves for the very first time and I was mightily impressed.
I’m a retired mechanical engineer who is loath to make too many tweaks on parameters without fully understanding the implications however I noticed that throughout the day, the battery has been charging in small doses to keep the battery topped up on daytime electricity (I’m in the UK).
I’ve attached two screen shots to show the grid consumption during the day and another to show the switch settings on the Energy Management system.  These settings are unchanged since February so I can only assume that the first use of the auto top up facility has somehow triggered this.  Note that I have the custom charge profile ‘set’ as this is the only way of keeping the system ‘on’ without doing a full charge, a top-up or disabling auto charge.
Are there some extra tweaks I need to make to prevent this from happening?
Thank you in anticipation,
Alan

Thanks @oilybits
A little concerned your next dawn time is showing as yesterday.
Can you include screenshots of 48h of load power, grid consumption and SoC please?

Hi Nathan,
Thank you for the speedy come-back.
I've attached 48 hr trends as requested.  I managed to turn off the
trickle charge by overriding the HA settings in Foxesscloud.
As I write this, 'capacity: next dawn time' is showing 9 hours which
seems about right.  For reference, the battery stack is 12.3 kWh.
Best regards,
Alan
On 30/03/2023 07:06, Nathan Marlor wrote:

Thanks @oilybits https://github.com/oilybits
A little concerned your next dawn time is showing as yesterday.
Can you include screenshots of 48h of load power, grid consumption and
SoC please?
—
Reply to this email directly, view it on GitHub
https://github.com/nathanmarlor/foxess_em/issues/249#issuecomment-1489747664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT62YQPZCUZDJXOGLAQPUQ3W6UPFRANCNFSM6AAAAAAWMQDMOU.
You are receiving this because you were mentioned.Message ID:
@.***>

@oilybits where have you attached them? :)

Hi Nathan,
I responded directly to the mail from my inbox with the files appended to that.  I can see that they don’t appear on GitHub so I just uploaded them……
Cheers,
Alan
Sent from my iPhone

On 31 Mar 2023, at 06:44, Nathan Marlor @.***> wrote:

@oilybits where have you attached them? :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Hi Nathan,
Any thoughts on this?
Cheers,
Alan

Hi Alan, ah I think I can see what's happening here. At 00:30 the integration started a force charge and set the min SoC (on grid) to ~50% (to prevent further discharge if load shedding etc.), however it appears to have failed to have set it back at the end of the off-peak period. This has resulted in grid usage until PV kicked in, and grid usage after sunset and any additional charge had been used.
Are you using the Cloud or Modbus integration?

Hi Nathan,
I believe I'm using the Cloud integration.  (I gave HA my Foxcloud login details).
Cheers,
Alan

Hi Nathan,
Have you eaten all your Easter eggs, yet?
I upgraded to v 1.7.4 yesterday.  Overnight, the batteries fully recharged and maintained 100% until end of cheap rate.  Is this related to the upgrade or my previous issue?
Thanks,
Alan

@oilybits are you still experiencing issues or can this be closed?

Hi Nathan,
I suspect the issue is still open.  My inverter (4 months old) chucked its hand in at the end of June.  The replacement is a later model which has a different wifi interface and I'm unable to make my rasp Pi talk to it......
I hope to re-install HA and have another bash at it shortly.  I'll let you know once I'm fully up and running again.
Thanks,
Alan

Hi Nathan,
Following premature failure of my inverter, I finally got my new inverter to talk to HomeAssistant (the process was somewhat beleageured by my low frequency wifi connection).
The 'updated' replacement inverter has no LAN connection and the usb port is taken up by an Anatel wifi stick.  For the moment, I see no possibility to communicate directly with the inverter through the cloud integration so I won't be able to explore this issue further.
Please close the query and I'll see if I can glean some more information from Will Eccle's Foxess webpage.
Thank you,
Alan


[Hold] $125 - Fix the Join button on the sign up screen

The Join button looks off in this screen when signing up on mobile web:

Go to Expensify.com on mobile web
Input your email > Click Get Started
You see the above which looks off.

I've attached a screenshot and video. @dubielzyk-expensify will suggest a fix, and then we'll ask a contributor to fix it. Jon, can you put in the updated mock and details?

https://github.com/user-attachments/assets/710e20ec-ea02-42db-870a-4041291a9fa3

The design fix should be as follows:

Make sure the button is 20px from the edge (which it is in most cases of the flow, but not when the button says "Join")
Have the button have a min-width of 88px

This should give us something like this:

Also be aware that this should only apply to this sign in flow and not all buttons. cc @Expensify/design

@Expensify/design would love your thoughts on this one

Sure thing, I'll assign myself this one.

https://github.com/Expensify/Web-Expensify/pull/44040 is on production. Do we just need to manually close this as done? @carlosmiceli


a11y linting

This project currently includes eslint-plugin-jsx-a11y for (mostly) web-oriented accessibility linting. Would the project also consider an eslint plugin for (specifically) react native accessibility? This project: https://github.com/FormidableLabs/eslint-plugin-react-native-a11y has just launched and I think it would be a great addition.
Would be interested to hear what you think!

Hi @alex-saunders, I'm definitely open to this. The plugin you mentioned was only included due to the fact that it was a peer dependency of eslint-plugin-airbnb I believe, so its only a dev dependency. How would you like to include the a11y plugin? I'm also open to merging the two plugins, to save React Native developers from an extra dependency.

Hey @Intellicode , sorry about the slow reply on this, I got super busy this past month.
I was thinking that react-native-a11y would fit as an extension of the all config that this plugin provides.
I.e:
all: {
      plugins: ['react-native', 'react-native-a11y'],
      parserOptions: {
        ecmaFeatures: {
          jsx: true,
        },
      },
      rules: allRulesConfig,
      extends: ['plugin:react-native-a11y/recommended'],
    },

If you can think of a more appropriate way to include the plugin, I'm open to suggestions!

Hey @Intellicode ,
I totally understand why you'd be concerned over extending the config. My thinking was that if a developer has spefically defined that they want to use the all config from eslint-plugin-react-native then they would want every eslint rule that may be applicable to react native development, including accessibility.
How would this work if eslint-plugin-react-native-a11y was a peer dependency? Would eslint error if a developer used the all config without also downloading eslint-plugin-react-native-a11y?

@alex-saunders That's true, it's not an out-of-the-box experience that way. Without the plugin, it will crash. However, that makes me think the config should live somewhere else, for example in a config like eslint-config-airbnb or similar that provide recommended configs accross plugins. I guess we should do that or merge code bases.

@Intellicode Good point, perhaps as eslint config project would be better suited to adopt eslint-plugin-react-native-a11y, I'll look into which projects might benefit from it!


rsz: Fix int overflow SIGILL

The result of Resizer::findMaxWireLength1() will eventually be
passed to this function, and can return (double)sta::INF. Which
when converted to an int in metersToDbu blows up.
double Resizer::findMaxWireLength1()
{
  double max_length = INF;
  for (const Corner* corner : *sta_->corners()) {
    if (wireSignalResistance(corner) > 0.0) {
      for (LibertyCell* buffer_cell : buffer_cells_) {
        double buffer_length = findMaxWireLength(buffer_cell, corner);
        max_length = min(max_length, buffer_length);
      }
    }
  }
  return max_length;
}

I added the code in this PR to protect against this.

@maliberty Base commit is formatting
Important changes are here https://github.com/The-OpenROAD-Project/OpenROAD/pull/2942/commits/07f45507898cc5c6f146cbec6a3c52c451b15381

I think you are addressing the problem at to low a level.  Either you have no resistance or no buffers both of which should be caught earlier and are more actionable by the user than this error.

Agreed, but I'll caveat that I do think we should still add this change. casting a double to an int is an inherently unsafe thing to do.
Should we error instead of returning sta::INF? @maliberty

Agreed, but I'll caveat that I do think we should still add this change. casting a double to an int is an inherently unsafe thing to do.
Should we error instead of returning sta::INF? @maliberty

I think so as the function has failed.  Also you can say something more useful at that point in the code about the problem.

@maliberty Separate, but related question.
I haven't dug into it, but I'm assuming this resistance value either comes from the LEF or the set_rc commands, but let's assume it does for the sake of the question.
Why do we rely on these values instead of going to RCX to estimate RC values from nets. Wouldn't it have a more accurate estimation?

Resistance is pretty simple so there isn't much benefit to using rcx.  If you haven't done routing it is even less useful as you don't have real layers.


ECS list_services ClusterNotFoundException Not raised

Reporting Bugs
Resource/Client:
ECS
Method:
list_services
When calling the ecs list_services with a cluster that does not exist (in moto) the API returns an empty list response.
Expected behavior:
It should raise ClusterNotFoundException if cluster_name is invalid.
Moto version: 4.1.0
Python: 3.9

Thanks for raising this @Bharat23 - marking it as an enhancement to implement this validation.
Just as an implementation note: the docs state that If you do not specify a cluster, the default cluster is assumed. Which means that the same error may be raised if you do not specify a cluster, and the default cluster does not exist.


[Enhancement] Eventmesh server go Makefile

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
Write makefile for eventmesh server go
Describe the solution you'd like
Include

build
test
coverage
build docker images

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

This task is finished.


Could you please add unicode sopport ?

This is an awesome library, however, I can not use it in my win32 project. Do you have a plan on it ?
Thank you!

What errors are you getting in your project?


Error: Unable to start Intel PT Service (err=50)

Windows 10 Pro build 17704
SDK version 17134
Command used: ipttool.exe --start 3332 100000 0 (pid of notepad.exe)
Output: Same in both admin and normal CMD

[*] Size will be aligned to a power of 2
[+] Using size: 65536 bytes
[+] Tracing Options:
           Match by: Any process
         Trace mode: User-mode only
    Timing patckets: None
[-] Unable to start Intel PT Service (err=50)
[-] Intel PT Service could not be started!

Please paste output of
sc query ipt
Also, is your build x64?

sc query ipt shows:

SERVICE_NAME: ipt
        TYPE               : 1  KERNEL_DRIVER
        STATE              : 1  STOPPED
        WIN32_EXIT_CODE    : 50  (0x32)
        SERVICE_EXIT_CODE  : 0  (0x0)
        CHECKPOINT         : 0x0
        WAIT_HINT          : 0x0

net start ipt shows:

System error 50 has occurred.
The request is not supported.

Yes, build is x64. Does this service depend on CPU like Intel Pentium or i3 or i5 etc.?

Intel PT is only available on Skylake processors and later, sorry!
On Wed, Jul 11, 2018 at 10:44 AM Biswapriyo Nath<EMAIL_ADDRESS>wrote:

sc query ipt shows:

SERVICE_NAME: ipt
TYPE               : 1  KERNEL_DRIVER
STATE              : 1  STOPPED
WIN32_EXIT_CODE    : 50  (0x32)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x0

net start ipt shows:

System error 50 has occurred.
The request is not supported.

Yes, build is x64. Does this service depend on CPU like Intel
Pentium or i3 or i5 etc.?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ionescu007/winipt/issues/1#issuecomment-404110143,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxIeEKK1SR1HU9ZIt76C3RoNFZNbN55ks5uFckPgaJpZM4VKxFt
.

--
Best regards,
Alex Ionescu

Will it work if I change the processor model in VirtualBox?

VBOX as far as I know does not support Intel PT pass through. Only hyperV
for now, and I think VMware latest versions (maybe ESXi only?)
On Wed, Jul 11, 2018 at 11:35 AM Biswapriyo Nath<EMAIL_ADDRESS>wrote:

Will it work if I change the processor model in VirtualBox?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ionescu007/winipt/issues/1#issuecomment-404123729,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxIeI-5F2KUuu8RZUvK7Ah2OAg4SoWCks5uFdULgaJpZM4VKxFt
.

--
Best regards,
Alex Ionescu

@ionescu007 I found two Event Trace Provider GUID, thought to be useful for your project:

IptGuid
{FF1FD2FD-6008-42BB-9E75-00A20051F3BE}

WnfGuid
{42695762-EA50-497A-9068-5CBBB35E0B95}

More in my repo TraceEvent.


pre-commit: PR117746

Link: https://github.com/llvm/llvm-project/pull/117746
Requested by: @dtcxzyw

/add-label reviewed
/close


some problem

Hello, I have a bug while trying to use your code,326 lines in utils/data_utilities.py file，ValueError: Not all axis codes [16, 42.0, 78.0, 57.0] in label set ['L', 'R', 'P', 'A', 'I', 'S', None].How do I fix this? Looking forward to your reply，thank you very much

Hy!! Thanks for opening this issue.
I'll go through it asap.
Nonetheless, check if your nibabel is correct and if it solves your issue, it has to be the version 3.2.1 according to the 'requirements.txt'
I'll update you as soon as I check this issue.

Hello! Everything working fine here.
I added 1 validation and 1 training volume of each dataset (VerSe19 and VerSe20) to be used as test. My suggestion is to test everything using this data (I mean, update your project with this data and skip the step of running the get_verse_data.py script).
Once everything is going well, you can then download the all data, by following the steps describe here.
I'm pretty sure your problem is related to the nibabel version! I'll keep this issue open until we manage to fix it for you.
Looking forward to hear from you.


Spawning Julia process fails

Hello,
I set up the CompileBot as described for a local (non-GitHub) package on Windows 10:
using CompileBot

snoop_bot(BotConfig("MyPackage"))

However, when running this file, I get the following error:
LoadError: IOError: could not spawn `julia --code-coverage=none<EMAIL_ADDRESS>-e "

Any idea on what goes wrong here?

Thanks for the fast reply. You identified the error: Julia was not on my systems path. Once it was addet, the CompileBot works perfectly :-)
For anyone who finds this thread: The solution is to add Julia to the system PATH as described here:
https://julialang.org/downloads/platform/
@aminya: I think it would be a good idea to add a hint to the documentation. When I find some time during the weekend, I'll issue a corresponding PR.

Feel free to open a pull request for adding this to the docs


Move Stats Generation to Individual repository

Moving stats generation and counting to an independent repository.

This is done as part of Bionorm


typescript unit tests fail in docker

    $ yarn run test && webpack --config webpack.dev.js && yarn run build-lab-extension
    $ TS_NODE_PROJECT=test mocha

      BeakerxDataGrid

      BeakerxCellRenderer
--- Logging error ---
Traceback (most recent call last):
  File "/opt/conda/envs/beakerx/lib/python3.6/logging/__init__.py", line 994, in emit
    stream.write(msg)
UnicodeEncodeError: 'ascii' codec can't encode character '\u2713' in position 8: ordinal not in range(128)
Call stack:
  File "/opt/conda/envs/beakerx/bin/pip", line 11, in <module>
    sys.exit(main())
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/__init__.py", line 233, in main
    return command.main(cmd_args)
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/commands/install.py", line 342, in run
    prefix=options.prefix_path,
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/req/req_install.py", line 845, in install
    install_options, global_options, prefix=prefix)
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/req/req_install.py", line 1007, in install_editable
    show_stdout=False)
  File "/opt/conda/envs/beakerx/lib/python3.6/site-packages/pip/utils/__init__.py", line 683, in call_subprocess
    logger.debug(line)
Message: '    \u2713 should create the columnManager'
Arguments: ()

the tests have been disabled until they work in docker.


CareKit

CareKit - CareKit is an open source software framework for creating apps that help people better understand and manage their health. By Apple :large_orange_diamond:

Official ?

@dkhamsing definitely!


This issue is for storing pictures in IoTDB website or other MD files

You can drag a picture here to get an URL on Github.

sync-1

sync-2

sync-3

sync-4

性能监控模块-1

性能监控模块-2

update-1

update-2

update-3

update-4

通过MXBean查看cache命中统计：/Users/suyue/Downloads/WechatIMG2.jpeg

HDFS storage Tsfile architecture

Architecture

Cluster Architecture
1.

2.

pictures for counting nodes and counting timeseries

System Design
chap 0

chap 1

chap2

chap3

chap4

An example of merging filters with same path.

An example of optimizing logical filters in IoTDB.

merge

OtherMaterial-Sample Data.txt

JMX

TsFile MetadataIndex

show storage group XXX

Monitor and Log tool

WAL test result

INPLACE selector

INPLACE merge

SQUEEZE selector

REGULARIZATION selector

REGULARIZATION merge

query-engine.pdf

【latest concept figure of IoTDB and TsFile】

Hi all,
better to store the picture in https://github.com/apache/iotdb-bin-resources
read the readme to see how to name the files.
Zesong Sun @.***> 于2021年6月20日周日 下午5:27写道：

【latest concept figure of IoTDB and TsFile】
[image: 1 1]
https://user-images.githubusercontent.com/19167280/122668849-b1c69280-d1ec-11eb-83cb-3b73c40bdf72.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thulab/iotdb/issues/543#issuecomment-864525314, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHZOVQUL6NY3SFXCUQRWMTTTWX6ZANCNFSM4GMDQYQA
.


Feature/edit shopping item css

Description
edit ShoppingItem component style.
Fixes # (issue)
How to test?
Please provide a short summary how your changes can be tested?
Checklist

[X] I have performed a self-review of my own code
[X] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[X] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[X] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[X] This PR is ready to be merged and not breaking any other functionality

but this issue is not related to this PR.


use File::Spec to remove double backslashes from grub conf file

This pull request fixes the double backslash issue without turning relative paths into absolute ones. My previous pull request used Cwd::abs_path() which mangled relative paths in the boot.cfg file.
Here is some example usage of File::Spec from the command line to give some intuition about how it works:
$ perl -MFile::Spec -le 'print File::Spec->canonpath("//")'
/
$ perl -MFile::Spec -le 'print File::Spec->canonpath("./test")'
test
$ perl -MFile::Spec -le 'print File::Spec->canonpath("a/b/c")'
a/b/c
$ perl -MFile::Spec -le 'print File::Spec->canonpath("//test")'
/test

Thanks!

Here is some documentation on File::Spec->canonpath from the documentation:

canonpath
No physical check on the filesystem, but a logical cleanup of a path.
$cpath = File::Spec->canonpath( $path ) ;
Note that this does not collapse x/../y sections into y. This is by design. If /foo on your system is a >symlink to /bar/baz, then /foo/../quux is actually /bar/quux, not /quux as a naive ../-removal would give >you. If you want to do this kind of processing, you probably want Cwd 's realpath() function to actually >traverse the filesystem cleaning up paths like this.

I am sorry, @benwbooth, but you are completely incorrect in your assertion regarding the nature of the problem---it was exactly a consequence of no value being returned.
  What you will then see is:
    Use of uninitialized value $kernel in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 356.
    Use of uninitialized value $initrd in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 357.
    Use of uninitialized value $kernel in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 356.
    Use of uninitialized value $initrd in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 357.
    Use of uninitialized value $kernel in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 356.
    Use of uninitialized value $initrd in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 357.
    Use of uninitialized value $kernel in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 356.
    Use of uninitialized value $initrd in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 357.
    Use of uninitialized value $kernel in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 356.
    Use of uninitialized value $initrd in concatenation (.) or string at /nix/store/sg7grlwi2xsx3f1gx081gh0vcw6y43x3-install-grub.pl line 357.

repeated many, many times.  And the resulting grub.cfg has entries that look like:
    menuentry "NixOS - Default" {
    search --set=drive1 --fs-uuid bbf97785-583e-42f5-a03b-15b3e183191a
    search --set=drive2 --fs-uuid bbf97785-583e-42f5-a03b-15b3e183191a
      linux  systemConfig=/nix/store/5hqspna6lybajh2bb8an0hy4mnr2xvvb-nixos-system-aching-16.03.git.b861bf8 init=/nix/store/5hqspna6lybajh2bb8an0hy4mnr2xvvb-nixos-system-aching-16.03.git.b861bf8/init loglevel=4
      initrd
    }

Which is missing paths to both the kernel and the initrd, which are the two things your changes were touching.
So I am, in fact, quite certain that the problem with your prior change was that you ended up returning no data, and thus I am skeptical of any change to this code that doesn't take the absolute minimum precaution of making sure that some value is returned before it makes changes that can so easily render someone's system unbootable.

I think it would be safer and cleaner to prevent the extra slash getting in the relevant variables ($grubBoot->path I think) in the first place, instead of trying to fix the damage everywhere it's used. I think https://github.com/benwbooth/nixpkgs/blob/master/nixos/modules/system/boot/loader/grub/install-grub.pl#L140 is the source of the problem.

@mdorman, sorry, I somehow misread that part of your issue description.
Maybe the problem with my original pull request was that Cwd::abs_path() returns undef if it fails to construct an absolute path from a given relative path. Based on a reading of the code I don't think File::Spec->canonpath has this issue, but in any case, I would be happy with any solution to this problem that both fixes the double backslash issue and doesn't break grub! @dezgeg's idea may be the more sensible one if it prevents any further issues like the one @mdorman experienced.
Thanks again for everyone's input on this issue.

If anyone else stumbles upon the same issue I've had with pygrub and nix, I found a workaround:
Apparently, this is only an issue if your /boot partition is formatted with ext2/3/4. If you format the /boot partition using fat32, reiserfs, or some other format, the double-backslash is ignored by pygrub.

It seems to me this could be closed?


Default lifetime for a type or tag

There's already a way to control lifetime for provided bindings: https://github.com/DevTeam/Pure.DI/blob/master/readme/default-lifetime.md.
It would be nice if there was a way to define lifetime for auto-binding for all classes implementing the interface or having an attribute.

Examples:

Default lifetime for a type
Default lifetime for a type and a tag


Vehicles Stuck in Air Floating

The staff on my Rocket server have run into the issue that whenever they spawn a vehicle, it gets stuck where it would spawn to fall above their head. This used to rarely happen, but now it happens no matter what. Anything that could help would be greatly appreciated. Already tried deleting Vehicles.dat in my server folder.

Have you tried it without Rocket?

happened to my server too. I just updated the game and rocket again, and it was fix.


rename DAGs.jl to DirectedAcyclicGraphs.jl

Codecov Report

Merging #1 (e1b09b3) into main (569b8dc) will increase coverage by 0.48%.
The diff coverage is 62.16%.

@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
+ Coverage   46.09%   46.57%   +0.48%
==========================================
  Files           7        7
  Lines         256      292      +36
==========================================
+ Hits          118      136      +18
- Misses        138      156      +18

Impacted Files
Coverage Δ

src/backend/ir.jl
56.36% <60.00%> (+1.36%)
:arrow_up:

src/dsl.jl
84.84% <100.00%> (+0.47%)
:arrow_up:

src/core.jl
70.00% <0.00%> (-10.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Change Transaction Fee from BigInt! -> String!

This fixes a backwards compatibility issue where Transaction Fee used to be a string but in GraphQL was set to be a BigInt!. Changing this to String means it passes our data parity tests.

Quick question about transactionsCount field @rhyslbw , should it be of type String or Integer ?
In Explorer it's a number. Details below.
Current Explorer:
Example query: https://explorer.cardano.org/api-new/epochs/0/3313
Field output: "cbeTxNum":1
VS
graphQL:
Example query: https://cardano-graphql-mainnet.daedalus-operations.com/
query EpochDetails{
epochs(where: {number:{_eq:0}}){
  number
     blocks (where: {slotWithinEpoch:{_eq:3313}}){
      id
      transactionsCount
      size
      fees
    }
}
}

Field output:"transactionsCount": "1"

@ArturWieczorek It should be a number for backwards compatibility - I checked with Alex.

The model of being able to aggregate across the whole chain can return some very large numbers, and I've tried to be consistent and present these values in the same way. In this case it's already a write-time calculated value, not a currency field like ADA, but is possible to be used in aggregation query across the whole chain https://github.com/input-output-hk/cardano-graphql/blob/master/src/schema.graphql#L282

What? What.. wait.. no !
If your test is expecting this to be a string @oneEdoubleD , I will definitely argue that your test is wrong.  Expecting data-type parity between graphQL and the legacy "REST" monster API is really not something we want. We do actually want the opposite in most cases!
This API was an abomination and, if we are forced to keep it as-is for backward compatibility reasons, we should not propagate those atrocities to the new components that are following a design that is actually sound.
fee here ought to be a number, not a string. Can we revert that please?

To summarise a conversation between @KtorZ, Alex, and myself:

So to summarise, for GraphQL we are okay with breaking changes but all the data available through the RestAPI should be obtainable through GQL too?

Therefore I will submit a PR to revert this change and update the CI tests accordingly.


Add config to preserve IDs based on prefix matching

There is currently an option to opt out of the ID cleanup by specifying IDs that we want left alone. This would also be useful to do on prefixes. That way, we could say, preserve rect_*, without having to enumerate all the possibilities of *. And, having both prefix and full string matching is useful, so you ca, for example, preserve rect specifically but not rect_blue.
This change adds another config array that takes in a list of prefix strings, and preserves any IDs that start with one of those prefixes. Also adds tests for this behaviour standalone, and in conjunction with force and the full-string ID preserving config.
PTAL @GreLI

@GreLI can you take a look when you get a chance please?

Thanks @GreLI. Any plans on an npm release soon?

Yep, I've released it just now.


fixing misalignment for right justification

same as #1 but for right justification

Hmm, have you seen this behavior under your fix? This is on my example project with the ERCollectionViewCell width set to 200 in the xib.  This is somewhat similar to the issue I was encountering in Issue #4, although it's happening with more frequency under FlowLayoutHorizontalJustificationRight.


[WIP] added an unleash client factory

Description
This change adds a client factory supports the ability to create clients synchronously.
Fixes #50
Problem solved
It is often better to fail to start up an app than to start it with incorrect feature toggle values. Before the changes in this pr there was no feedback from the unleash client if the unleash server was unreachable. This resulted in hacky, race prone workarounds to detect if the unleash client was "ready".
Considerations
Change an existing method?
Adding a .Wait() or .Result() within an existing synchronous block (the https://github.com/Unleash/unleash-client-core/blob/master/src/Unleash/DefaultUnleash.cs constructor, for instance) could result in deadlocks during usage. Adding a new factory with an explicitly async signature makes it clear that the call needs to be awaited.
ExecuteDuringStartup?!
Two workarounds for just this issue were already in the codebase. Unfortunately, they didn't resolve the issue. They have both been removed.

the IUnleashScheduledTask.ExecuteDuringStartup field is intended to signal to IUnleashScheduledTaskManager implementations that the task should run at startup. Unfortunately the startup process is synchronous; the default IUnleashScheduledTaskManager implementation, SystemTimerScheduledTaskManager, simply schedules the task for immediate execution, it does not wait for the task to complete before passing execution back.
UnleashServices contains a test shim that populates the feature toggle collection if it detects an api client has been provided. This works well for https://github.com/Unleash/unleash-client-core/blob/master/tests/Unleash.Tests/ExampleTests.cs where the test requires that the toggles be loaded by the time the unleash client is constructed. With the new client factory, that shim can be removed.

Type of change

[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

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

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

todo:

[ ] update readme
[ ] document public methods

I think this is ready for review.
attn @nmiddendorff and @njbatt

I found a bug that should be addressed before merging this.
in the api client, we swallow non-200 responses rather than throwing an exception. I want those exceptions to bubble up if SynchronousInitialization is true so that apps can cancel startup if unleash is misconfigured.
I'll take a swing at addressing that tonight


Expose VM/VMI readiness via a dedicated CLI column

What this PR does / why we need it:
Add a READY column to the tabular output of kubectl get vm and kubectl get vmi, indicating the readiness condition of the VMI.
Here's how the output looks now:
zvic@ZVICAHANA-D2LK:~$ kubectl get vm
NAME     AGE   STATUS    READY
cirros   30s   Running   True

zvic@ZVICAHANA-D2LK:~$ kubectl get vmi
NAME     AGE   PHASE     IP            NODENAME   READY
cirros   37s   Running   <IP_ADDRESS>   minikube   True

zvic@ZVICAHANA-D2LK:~$ kubectl get vmi -o wide
NAME     AGE   PHASE     IP            NODENAME   READY   LIVE-MIGRATABLE   PAUSED
cirros   39s   Running   <IP_ADDRESS>   minikube   True    True

Special notes for your reviewer:

~Note the slight casing difference between the VM and VMI outputs. The VM column feeds from the .status.ready field which is in lowercase, whereas the VMI column feeds from the status field of the VMI's "Ready" condition~ (no longer relevant as the VM column now too feeds from the VM's "Ready" condition).
I've used this chance to remove the obsolete VOLUME column from the output of kubectl get vm.
This column, added way back in #1654, was pointing to a non-existing VM field, vm.spec.volumes[0], and anyway incorrectly
assumed that the first listed volume for the VM is the "primary" one (although a provided boot order may indicate otherwise).
Also removed the "CREATED" column from kubectl get vm. With the recent introduction of the "STATUS" column, it has became quite redundant.

Release note:
Added a READY column to the tabular output of "kubectl get vm/vmi"

/cc @kwiesmueller

Thanks for adding this!
I wonder if a ready indicator should also be added to ReplicaSets (if we don't have them there already).
Also I'm not that excited about the casing differences you mentioned.
Does the VM not have a respective condition we can use as well?
/assign

Hm. Can we set a default value for the column which would be false instead of empty?
I'm fine with ditching CREATED in favor of this, but would like to hear other people on this.

The column just points to the .status.conditions["Ready"].status, which only gets populated at some later stage. We could modify the VM controller to set this value to false when the VMI doesn't exist or doesn't carry the condition, and modify the VMI controller to set this value to false before the virt-launcher pod is created.

/cc @rmohr @davidvossel since you were both involved in the original implementation of the Ready condition, I'd appreciate your feedback here, and particularly about the changes to when the Ready condition is present (i.e., is there a good reason to remove this condition instead of just setting it to status: False?).

Downside is, we'll have the READY column specifying True, while the CREATED column specifies true - at least until we have a corresponding Created condition. We can also consider removing the CREATED column, as it's quite redundant with the new STATUS column.

...and also regarding this matter :-)
/hold
There's additional work to be done on the VMI's unit-tests (mostly - aligning the current tests to not fail when the Ready condition is updated). I'm holding off to this work until we get an ack that these changes are acceptable.

Hey @zcahana, I see this PR is still in progress.
Will review then it goes "ready to review" (stops being draft).

I see this PR is still in progress.
Will review then it goes "ready to review" (stops being draft).

It's mostly ready to review, except for alignment of some existing unit tests that now fail due to a change in how we set the Ready condition (see this comment). I've been waiting with this in order to get some tentative ack about this change, and marked as WIP to avoid running the CI lanes in vain.
 Will do that in the upcoming days.

is this PR ready? (it looks like you did what you intended to do)

It looks like at least there's a make generate missing here somehow. However, otherwise besides small readability nits this looks good to me!
Unfortunately not yet. There are still quite some tests which are failing and needs to be aligned to this slightly revised conditions logic. Apparently, many tests (dozens of them) have/had fragile dependencies on VM/VMI conditions (e.g., they expect something on .Conditions[0], or expect no conditions to exist at all) and now fail when the Ready condition is being set all the time). I'm working through these errors one-by-one, but need some more time to complete.

@davidvossel @dhiller this PR is now ready for re-review. The main changes since the previous approval are:

Ensure we PATCH the VMI when it's handed over to virt-handler (the initial implementation used UPDATE in some cases). See bd2d87b.
Various fixes to unit and functional tests which had fragile dependencies on .status.conditions.
Some additional minor changes.

Thanks.

/retest

/retest

/unhold
The hold label was a leftover from a while back, forgot to remove it.
Thanks to all reviewers!


msg: chown failed on file ownership change of /vagrant/repos

I'm getting the following error message from on the final line (- file: dest=/vagrant/repos owner=vagrant group=vagrant state=directory) of playbook.yml. I'm thinking that with the change to NFS mount this line is extraneous. Thanks for your thoughts!
TASK: [file dest=/vagrant/repos owner=vagrant group=vagrant state=directory] ***
<<IP_ADDRESS>> REMOTE_MODULE file dest=/vagrant/repos owner=vagrant group=vagrant state=directory
failed: [default] => {"failed": true, "gid": 20, "group": "games", "mode": "0755", "owner": "501", "path": "/vagrant/repos", "secontext": "system_u:object_r:nfs_t:s0", "size": 68, "state": "directory", "uid": 501}
msg: chown failed

No worries brothers - zero urgency. Have a great vacation!


Docker No module named 'fritzconnection'

Home Assistant release with the issue:
0.84.6, 0.84.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:

FritzBox Device Tracker
Description of problem:
The module fritzconnection is missing in the docker image.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: fritz
  host: <IP_ADDRESS>
  consider_home: 120
  interval_seconds: 30

Traceback (if applicable):
  File "/usr/src/app/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/local/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/sensor/fritzbox_netmonitor.py", line 54, in setup_platform
    import fritzconnection as fc
ModuleNotFoundError: No module named 'fritzconnection'

Additional information:

Issue persists with 0.86.4.

Issue persists with 0.87.0.

Resolved for me in 0.88.0 – don't see anything in the changelogs about it, though.

I still see it on 0.89.1

same issue here. solved by reinstalling fritzconnection as described above, but this should be already in native image.

Seems the fix is reinstalling fritzconnection, closing as such, let me know if this assessment was incorrect and i'll reopen.

Reinstallaing is more a workarround. It should be native in the docker image. Funny thing, In 0.90 did not had the issue

@robbiet480 I would not consider it a fix, as we need to it after every single update. If we forget, stuff will just silently fail until we notice. As me and others have noted, it doesn't seem to hit everyone with every release, so it seems there is something weird about the build process for the docker images going on.

Just ran into this as well using image 0.92.1 :/

For whatever reason the requirement has been commented out
https://github.com/home-assistant/home-assistant/blob/b2a1204bc5fe8006a5343248e769f2e5194798ea/requirements_all.txt#L462

Facing this missing dependency with 0.94.0, possibly related to the "deps" no longer being used. The package cannot be installed on the fly, if Docker container is run by a non-root-user.

   homeassistant | 2019-06-05 22:22:10 ERROR (SyncWorker_5) [homeassistant.util.package] Unable to install package fritzconnection==0.6.5: ERROR: Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/usr/local/lib/python3.7/site-packages/fritzconnection-0.6.5.dist-info'
    homeassistant | Consider using the `--user` option or check the permissions.
    homeassistant | 2019-06-05 22:22:10 ERROR (MainThread) [homeassistant.requirements] Not initializing fritz because could not install requirement fritzconnection==0.6.5
    homeassistant | 2019-06-05 22:22:10 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform fritz.device_tracker: Could not install all requirements.

I can confirm changeing the requirements_all.txt to include fritzconnection again and build your own image solves this issue

Facing this missing dependency with 0.94.0, possibly related to the "deps" no longer being used. The package cannot be installed on the fly, if Docker container is run by a non-root-user.

Home-Assistant need run: as root and on the host network.
Why? A lot of internal function (like discovery/zeroconfig/ssdp) or library work not correct.
Is there no way to run Home-Assistant without root? Sure, but need a high Linux knowledge and maybe is better solved with a modern access control variant like AppArmor as a User system from Posix created around 1985.

Home-Assistant need run: as root and on the host network.
Why? A lot of internal function (like discovery/zeroconfig/ssdp) or library work not correct.

While it is correct to use host-network for discovery and other items to work, it is not necessary to run the container as root (or: it has not been required to do so until the fritzconnection-dependency was missed for 0.94.0).
Running the container as non-root-user still makes sense as the /config folder is mounted into the container from the host filesystem and using a dedicated host user id there helps in increasing system security.
As suggestion

For whatever reason the requirement has been commented out
https://github.com/home-assistant/home-assistant/blob/b2a1204bc5fe8006a5343248e769f2e5194798ea/requirements_all.txt#L462

How about including fritzconnection dependency in requirement_all.txt again and I check if the container is runnable by a non-root-user again? Would this be possible?

I beleive i am running as root. I didn’t face the issue for quite some time, but is i read others do face the issue.

For my installation i could solve this error by manually installing this package.
First, access your home-assistant container by the following command:
docker exec -it --user=root home-assistant bash
Second, install fritzconnection via pip3
pip3 install fritzconnection==0.6.5
Now you can restart home-assistant

any news here?

I did not had the issue occurred again for A long time

The issue is still present.
This is the output of the latest home assistant container using version 0.103.5:
bash-5.0# pip3 list | grep fritz
fritzhome                        1.0.4
pyfritzhome                      0.4.0

As you can see the fritz connection is still missing.
@balloob what would speak against putting fritzconnection into the requirements_all.txt?


SVD changes

I used resetValue in several places. This should be changed for reset masks that take into consideration the reserved fields in different registers.
The access property should be moved to fields too, for now I assumed some reserved registers to be read-write, but this might be incorrect. For the reserved fields it would be sensible to have them as read-only.
I will get this done after finishing the PMU register yaml

svd2rust does not take into account the reset mask, which means that at least for now, this will have no effect on pac generation, but that might changed in the future.


Push newly build images to local registry instead of just leaving in cache

The local registry is located at localhost:5000, but the pipeline doesn't need to know this as the website will add it to the image name in the deployment config. All that is needed is an additional line implementing the docker push <image-name> command.

 GHP: Push image to local registry

Waiting on #5 to be merged first so less rebasing


Suggestion: parameters order in EntityAdapter methods

As we can see in Fluxor reducer methods, the first parameter is 'state' and the second parameter is 'action'.
[ReducerMethod]
public static ProductState OnProductsLoaded(
    ProductState state,
    ProductsLoadedAction action
)
    => Adapter.SetMany(state, action.Entities);

So why not to sort parameters of all EntityAdapter methods with the same order?
public TState SetMany<TState>(TState state, ICollection<TEntity> entities)
    where TState : EntityState<TEntity, TKey>
{
    var items = entities.Select(x => KeyValuePair.Create(_selectIdCallback(x), x));

    return state with
    {
        Entities = state.Entities.SetItems(items)
    };
}

public TState Map<TState>(TState state, TKey key, Func<TEntity, TEntity> mapFunc)
    where TState : EntityState<TEntity, TKey>
{
    return state with
    {
        Entities = state.Entities.SetItem(key, mapFunc(state.Entities[key]))
    };
}

Check this:
// Before
Adapter.Map(key, entity => entity with
{
    Name = action.Name
}, state);

// After
Adapter.Map(state, key, entity => entity with
{
    Name = action.Name
});

I agree, it looks more convenient. I'll fix it on the weekend. By the way, do you have any experience in creating an application state? Right now I'm looking for this for a work project where we use Fluxor, but using it causes some performance issues.

By the way, do you have any experience in creating an application state? Right now I'm looking for this for a work project where we use Fluxor, but using it causes some performance issues.

Depends on what is need to do. My application state includes user profile, for example.

Fixed in v0.0.4

Depends on what is need to do.

I am interested in any general state building practices


Wrap unstable errors

Addresses issue #125

Wrap hyper::Error and serde_urlencoded::ser::Error
Update downcast_ref test
Remove re-export of hyper::Error (breaking change)

I'm sorry I wasn't clearer in the issue, but I wasn't sure if this is the direction that should be taken, just suggesting it's a possibility to implement. I'd rather close this for now, and try to get a proper solution into the guidelines first.
Thanks for being ready to jump on it so quickly, though!


Better workflow

Hi @bomsy!
Thanks for this extension :) I have some improvement ideas:

At the moment the first file must be in the current project file so it isn't possible to add a file with drag & drop to the working files and compare them because the context option 'Compare with file..' isn't available.
When the user choose the compare with file option a open file dialog appears and I need to choose the same directory to open the second file :/ An idea would be to check whether the user is a split mode and then the user can choose to compare these two files via the View menu.

What do you think about it?

Hey @Wikunia
Thanks for your ideas! Great suggestions!

I'll look into fixing this.
I can add an option to the view menu to compare the two files, when split mode is on.
Will that work?

Hey @Wikunia
Thanks for your ideas! Great suggestions!

I'll look into fixing this.
I can add an option to the view menu to compare the two files, when split mode is on.
Will that work?

I think that it should work with split view. I made some extensions but nothing with split view but you can send me a mail if you need some help :)

You're welcome!
I think that it should work with split view. I made some extensions but nothing with split view but you can send me a mail if you need some help :)

hey i've released v0.2.0. Please try it out and let me know.


Error: Dropdown element is in div

It is difficult to guess what issue you are trying to submit, could you provide more information?


Wiki Download page / update latest snapshot versions + dependency inconsistencies in Sonatype

Hello! Please update latest snapshot versions that exist on Sonatype (since the listed ones are not there anymore https://github.com/orientechnologies/orientdb/wiki/Download).
First of all blueprints-orient-graph shouldn't be here ...since it is only for versions <1.7 (as stated here: https://github.com/orientechnologies/orientdb/wiki/Graph-Database-Tinkerpop). Secondly I had to remove orient-commons. I am not sure about that. I also had to add another repository to get blueprint-core version 2.6 that is used from the new orientdb-graphdb 2.1.SNAPHSHOT. This version is not available in SonaType obviously since it is a release https://oss.sonatype.org/content/repositories/snapshots/com/tinkerpop/blueprints/blueprints-core/
This is my final dependencies list that made it work:

 	<repositories>
	    <repository>
	        <id>sonatype-nexus-snapshots</id>
	        <name>Sonatype Nexus Snapshots</name>
	        <url>https://oss.sonatype.org/content/repositories/snapshots</url>
	    </repository>
	    <repository>
	    	<id>mvn-repository-com-artifacts</id>
	    	<name>mvnrepository.com</name>
	    	<url>http://mvnrepository.com/artifact/</url>
	    </repository>
	</repositories>
	<dependencies>
        <dependency>
		  <groupId>com.tinkerpop.blueprints</groupId>
		  <artifactId>blueprints-core</artifactId>
		  <version>2.6.0</version>
		  <type>bundle</type>
		</dependency>
		<dependency>
		  <groupId>com.orientechnologies</groupId>
		  <artifactId>orientdb-core</artifactId>
		  <version>2.1-SNAPSHOT</version>
		  <type>bundle</type>
		</dependency>
		<dependency>
		  <groupId>com.orientechnologies</groupId>
		  <artifactId>orientdb-client</artifactId>
		  <version>2.1-SNAPSHOT</version>
		  <type>bundle</type>
		</dependency>
		<dependency>
		  <groupId>com.orientechnologies</groupId>
		  <artifactId>orientdb-enterprise</artifactId>
		  <version>2.1-SNAPSHOT</version>
		  <type>bundle</type>
		</dependency>
		<dependency>
		  <groupId>com.orientechnologies</groupId>
		  <artifactId>orientdb-graphdb</artifactId>
		  <version>2.1-SNAPSHOT</version>
		  <type>bundle</type>
		</dependency>
	</dependencies>

Thank you,
Michail

How did you arrive to that page?

https://github.com/orientechnologies/orientdb/wiki --> there is a link Download at the bottom. Google got me to wiki

I can't find it. Could you attach a screenshot please?

Nevermind: found it and fixed. Thanks.


Update index.js

My build uses Webpack and fails with TypeError: fetch is not a function.
Webpack and node-fetch 2.1.2 (in use here) will not use the CommonJS solution by default.
Explanations in this thread:
https://github.com/bitinn/node-fetch/issues/450#issuecomment-387045223

Hey @obax, yep makes sense - thanks for PR.


feat: Partial migration of test cases to new RBAC framework

/ok-to-test sha=b2e6cf2

Server workflow : https://github.com/appsmithorg/appsmith/actions/runs/2770796883

Server workflow : https://github.com/appsmithorg/appsmith/actions/runs/2773886416


add ScaledJob's label to its job

Signed-off-by: mosesyou<EMAIL_ADDRESS>
This PR is about to add ScaledJob's label to its job.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)
[ ] Tests have been added
[ ] A PR is opened to update the documentation on https://github.com/kedacore/keda-docs
[x] Changelog has been updated

Fixes #1311

@mosesyou could you please add this feature to the Changelog? to the Improvements section in Unreleased, thanks!

@mosesyou could you please add this feature to the Changelog? to the Improvements section in Unreleased, thanks!

Sure

@zroubalik I have big interest in this PR and tested it, here are my findings:

With this PR labels can be added to the "jobs" but not to the "pods" created by the job (see pictures). Was this intended?
Annotations are still not working :( This would be needed to disable istio, see here: https://github.com/istio/istio/issues/11659#issuecomment-625263887 (this annotation is needed in the pod spec but not in the job spec)

@ericbottard sad to hear that. Thanks for the confirmation! We should fix this. Could you please open an issue with the details. And are you willing to contribute this? Or @mosesyou do you want to take a look?

@zroubalik Can you maybe just reopen https://github.com/kedacore/keda/issues/1311 since the issue described there still exists?

Hi, I would like to help if I can.
I took a look at the code and it seems to do a deep copy of the scaledjob spec.template to job template.spec on creation.
If we want a label or annotation to appear in the pod, wouldn't we would add it in the scaledjob spec.template metadata?
@erichorwath would you provide a snip of your scaledjob? To help me better understand the issue.

@ycabrer sure, the interesting part is here spec.template.metadata part. It would be great to be able to set this in an scaledJob yaml (->  spec.jobTargetRef.template.spec.metadata).
apiVersion: batch/v1
kind: Job
metadata:
  name: redis-job
spec:
  activeDeadlineSeconds: 96000
  backoffLimit: 0
  completions: 1
  parallelism: 1
  template:
    metadata:
      labels:
        my-workload: app
      annotations:
        sidecar.istio.io/inject: "false"
    spec:
      containers:
      - image: busybox
        imagePullPolicy: IfNotPresent
        name: redis-worker
        env: []
        ports:
        - containerPort: 8080
          name: http
          protocol: TCP
        - containerPort: 5747
          name: status
          protocol: TCP
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /data
          name: worker-data
      dnsPolicy: ClusterFirst
      imagePullSecrets:
      - name: regcred
      restartPolicy: Never
      schedulerName: default-scheduler
      terminationGracePeriodSeconds: 259200
      volumes:
      - emptyDir: {}
        name: worker-data

https://github.com/kedacore/keda/blob/af5254dd2bf8f2b70eb73ddc6428b4fb01dc2fb2/pkg/scaling/executor/scale_jobs.go#L83-L90
I'm not sure,  Spec: *scaledJob.Spec.JobTargetRef.DeepCopy(),  Doesn't this line of code mean copy ScaledJob's Job Spec including PodTemplate to Job? So why we can't pass labels or annotations from ScaledJob's JobSpec{PodTemplate{ObjectMeta}} to Job{PodTemplate{ObjectMeta}}?

Hi there
I know this has been merged in and apparently works for labels but we still cannot add annotations on the PodSpec (to disable istio injection for example). Do we have a rough idea of when this feature will be brought back in Keda ?
Cheers

@julienperignon I've opened https://github.com/kedacore/keda/issues/1627 for it.


Gitcoin Round 10 Hackathon

Hello {{hacker.firstname}}!
Thanks for your interest in building your hackathon project on top of Uniswap! This is a journey we'll embark on together as it's our goal to ensure that we can empower you, the hackers, to easily participate in a growing ecosystem of DeFi primitives with all the tools and libraries you might need to get started.
If you are new to Uniswap, do not worry! We curated a list of resources that will be handy all throughout your journey during this hackathon. If you have any questions not addressed here, or if you just want to chat on where or even how to begin, please do not hesitate to reach out to any UGP mentors in the UNI-Grants discord channel!
Bounties & Prizes
Remember, the goal of this hackathon is to have fun! There are no winners and losers, just builders and multipliers. While we encourage you to just explore your passions and build to your heart's desire, there is no pressure. Just go get a lay of the land and see what the whole buzz is about for Uniswap! Whatever your project may be, we are hosting $10,000 worth of bounties for the 🌈 2 most innovative 🌈 and the 🦄 2 most impactful 🦄 hacks.
We are open to any and all new ideas and will never try to impose any limits where they aren't needed! However, if you are looking for some inspiration, check out our list of
👉🏼 **RFPs & Challenges** 👈🏼
Additionally, don't be afraid to mix and match our bounties with other projects! The idea is to produce composable hacks on top of everything! For example:

Maybe you want to see Uniswap V3 Gnosis Safe App!
Maybe you want to see Uniswap collabs (credit to Flashbots for the following)!

Uniswap with 0 gas private transactions with Flashbots bundles
Uniswap MEV inspectors (in python or on L2)
Open source trading bots on Uniswap!

naive v3<-> v2 arb bots
bots that take the reserves of v3 and v2 pairs and find the optimal position sizing for trading across them
tool for direct to miner liquidity addition that prevent token sniping
offchain bot that make all mempool txs for a pair and find the smartest permutation that minimizes slippage

Prizes

Award
Prize Amount

Most Innovative
$3,000

Innovative 2
$2,000

Most Impactful
$3,000

Impact 2
$2,000

Submission Requirements
Judging will happen once all submissions have been received. Please include the following:

Live demo (video is fine!)
Working code in a public github repo

The requirements for your submission must:

Be free and open source
Be impactful for Uniswap users
Be built during the hackathon
Be original
Have been fun to build 🤗

If you have any questions feel free to ping us on the discord or find us on twitter!

When is the winner announcement date?

I can see you are asking this question in every bounty mail, I request you to please be patient for 1 week atleat.If you will be the winner then you will be getting prize money for sure!!Don’t worry! Sent from Mail for Windows 10 From: ayushkumar63123Sent: 09 July 2021 17:45To: unigrants/ugp-hacksCc: PIYUSH CHOUDHARY; MentionSubject: Re: [unigrants/ugp-hacks] Gitcoin Round 10 Hackathon (#2) When is the winner announcement date?—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


fix(oci-registry: correctly set repo value for plugin artifacts

Signed-off-by: Aldo Lacuku<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
Correctly set the repo values for plugin artifacts when generating the index file.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval by writing /assign @jasondellaluce in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found 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

LGTM label has been added.  Git tree hash: 947c5a42a1901cc4bf9f4b6888784264e0b6fe8f

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, leogr
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 [leogr]

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


[beta] Copying solution copies the lines numbers in the code editor

Issue Description

I'm able to copy my solutions, but it also selects and copy the lines numbers too on the code editor for a challange.
Browser Information

Browser Name, Version: Firefox 50.1.0
Operating System: OSX 10.9.5
Mobile, Desktop, or Tablet: Desktop

Your Code

2

var myStr = ""; // Change this line

3

4

5

Screenshot

Hi @marvellousdesign! Thank you for reporting this issue.
I can not reproduce this on the same version of Firefox on Windows 10. Now matter how hard I try (Ctrl+a, select the text, select the numbers), I can't select the numbers.
Can you consistently do this? Does it happen on all challenges? Does it also happen in other browsers?

I updated my browser to Firefox 51.0 and I don't see this issue. Doesn't happen on Chrome. I've been using Freeter, it has a webpage wiget and it does happen there sometimes, not always.

That's really weird 😕
@freeCodeCamp/moderators Can anybody reproduce?

I can't reproduce with Firefox 50.1 (51 isn't published to the official Ubuntu repo yet).
To me it sounds like the issue is about the Freeter webpage widget. I wonder what browser it's based on - if we are lucky, some upstream fix might fix this issue.

Cant reproduce on Chrome, Firefox on macOS. Closing. Please comment if issue still persists to reopen.


Revoke method doesn't work

Hi,
When I call the revoke method I get an error message saying: Empty or no refresh token passed.
The code is exactly like in the docs:
 public async logout({ auth }) {
    await auth.use('jwt').revoke();

    return {
      revoked: true
    };
  }

Is there an issue with it or am I doing something wrong?

You need to be logged in to revoke, otherwise adonis-jwt doesn't know what to revoke. Make sure to pass the JWT token in header or cookie to be able to revoke it

reopen if you want to follow up

hello @maxgalbu , I'm testing the library and it's great, thank you very much for your contribution.
I have the same problem as @radudalbea, I already verified that when the request arrives the token is sent (bearer token authorization), it even shows the information that the token payload has in the console. But then when calling the logout function I get the same error
async logout({ auth }) {
        // Validating that the token has arrived
        await auth.use("jwt").authenticate();
        const the_user: User = auth.use("jwt").user!;
        //in the console I actually get the information that comes in the token
        console.log("Info user ", JSON.stringify(the_user))
        //revoke() --> i get the problem "Empty or no refresh token passed"
        await auth.use('jwt').revoke();
        return {
            revoked: true
        };
    }

Thank you

Vuelve a abrir si quieres hacer un seguimiento.

This is still happening.

@rodcrespo you are right. It's a problem with the specs. You cannot revoke a token that is not persisted somewhere(database or Redis).


feat: convert css tokens to sd-transforms

A number of things were done in this PR.

Converted to use sd-transforms for all css / less tokens.
sd-transforms is not available for IOS / Android yet so we are still using token transformer for those. These will likely be available for sd-transforms in the future.
split tokens doc.json into themes, previously this was just showing only dark mode tokens and showed up in our tokens doc as such. The doc.json is also combined from both style dictionary runs for sd and token-transformer.
separated sd-transforms and token-transformer config into two separate files.
separated custom dialtone style dictionary transforms into their own file.

@braddialpad,
I made a tiny adjustment at https://github.com/dialpad/dialtone-tokens/commit/089c2cefa5c770d104297e12a53d28fc46d43664
Softened the Search Field's far-too-stark hover color in dark mode. Screencast shows current undesired hover color. Screenshot shows update.
https://github.com/dialpad/dialtone-tokens/assets/1165933/3c77c0f8-c9d5-41be-b652-ee0b70a594ec


OPAL root path

Is it possible to configure OPAL Server and Client so that all API requests must start with a given path, i.e. /opal-server/healthcheck ?

I don't think there's an easy config for that; but you can use a reverse-proxy like NGNIX or Caddy to apply this manipulation

Would PR configuring FastAPI root_path be acceptable?


dev merge with main

Hi, I would merge dev with main now since we have everything ready for Übungbsblatt 4.

It's okay for me


Initial text for science page

I'm about to tidy up the text for the science page and push it, but unfortunately everything I have written so far was on a clone of the old repo on BitBucket. Now that everything has moved to Jekyll what I did before needs doing again. Can someone please tell me where assets go and how I use them? On the old site I was putting images in .../science/static/img/. I notice the Education page on the new Jekyll site doesn't have any images now, so I can't just copy what @ntoll has done :(

Usually there's an asset directory in the root somewhere that you can get to with:
{{site.baseurl}}/assets/picture.png

I see that /static/assets exists so perhaps that's where things should go:
{{site.baseurl}}/static/assets/science/picture.png

(Sorry if I'm in the way and shouldn't be throwing in my two cents: please ignore me :+1: )

Thanks @drvinceknight - so does Jekyll infer that something with a .png extension will live in the img directory?

Eeeeehhhh no not at all (saying that with 75% certainty :)), you can link to any file type just using the {{site.baseurl}}/path thing... Might be worth having a look at how other images have been put in for consistency... Running of to a class now, will check back in if I can be helpful :)

Hi @snim2 -- do you want to have a quick hangout to look at this together?  Might be easier to do in realtime...

This sounds great, many thanks for the offer @inglesp . I have classes all day today until 5, I should be home some time just after 7 today, if that suits you?

Yep -- let me know when's best.

Closed in PR #33


Vue 3.x Composition API use standard re-write

What kind of change does this PR introduce?
Ground up re-write of the source from index.js to index.ts such that it now works / is aligned with the Vue 3.x useSomeFunctionName standard.
What is the new behavior?
Behaviour is identical as before. Except now there is included types for the creation function options.
Additional context
While this re-write may not be entirely "necessary", it may be of some benefit to eventually re-write this module to follow the new standard. Following from this, re-writing it in TypeScript offers a few benefits, including the plugin option types being included.
I have currently marked the package as a 2.x.x release as I feel that this re-write is primarily targeted towards Vue 3.x users, it may be best to keep it as a new major release? Though this can be changed back and instead changed to 1.0.5
If any other changes I have made are not to your liking, let me know and I will change them 🙂

Hey @NuroDev
Thanks for this. Just a couple of initial things:

Can you switch this back to spaces, instead of tabs?
Should we increase the vue peerDependencies to >=3?
How can we maintain support for Vue 2.0? Or do you think we just leave that for the 1.x release, and backport any fixes if we need to?

Hey @scottrobertson
No problem, I'll get that changed over to spaces instead of tabs.
I think if we are going to have a separate major release for this it would be better to upgrade the vue peer dependency to >=3, yes.
As for maintenance I believe the best course of action would be the following:

Leave the 1.x / Vue 2.x release where it stands right now (Still readily available & ready to be patched if need be)
Release this new 2.x / Vue 3.x version under the 2.x or next version (Similar to Vue releases)
At a later date, when Vue 3.x is the default / primary Vue version, keep a 1.x branch ready for any maintenance / patches that need to be made to the 1.x release & then make the 2.x release the default / latest release.

This maintains the supabase/nuxt-supabase 2.x support but offers Vue 3.x support to those who want it.

Thanks @NuroDev.
Still some tabs in tsconfig, and the README, if you wouldn't mind sorting them?

Fixed. Let me know if there are any other issues that need resolving 🙂

Closing in favour of #2. Not really sure what is going on here, but whatever was done to the files in this PR was messing with diffs entirely. For example, it was saying the whole of package.json had changed, but it had no. I suspect it's some encoding issue.


IOS build failed 'Undefined symbols for architecture x86_64'

Updated React-Navigation to 3.0.8 and followed the instruction to install the react-native-gesture-handler.
after react-native link react-native-gesture-handler
i got error when building with react-native run-ios or building in the xCode.
Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_RNForceTouchHandler", referenced from:

      objc-class-ref in libRNGestureHandler.a(RNGestureHandlerManager.o)

ld: symbol(s) not found for architecture x86_64

clang: error: linker command failed with exit code 1 (use -v to see invocation)

i am using following packages:
  "react-native": "0.57.3",
    "react-native-gesture-handler": "1.0.12",
    "react-navigation": "3.0.8"

How do i fix this?  Please help.
Appreciate :)

Hi, @sliultron
Try changing 'Build System' to 'Legacy Build System'. I've just made fresh app with react-native init and then linked RNGH and it appears to work fine.

Downgrading to "react-native-gesture-handler": "1.0.10" works for us.

@osdnk
finally got project compiled in xcode with a clean build.
I have to run this first.
rm -rf ~/Library/Developer/Xcode/DerivedData/*

Then building is ok.
However, running react-native run-ios still give me that error. Wondering if that is caused by npm cache?

Hm, I wouldn't guess it's npm cache.

react-native-gesture-handler": "1.0.10

I uset it solve

I strongly advise you y to use the newest version since I suppose it's only matter of cache.

I strongly advise you to use the newest version since I suppose it's only matter of cache.

RN build  cache and xcode build cache is different  directory, so xcode build success, RN build fail

I changed iOS Deployment Target from 8.0 to 9.0 in RNGestureHandler.xcodeproj and solved for me

Cleaning the project and removing derived data doesn't do it for me. Seems to be more than a cache issue. Downgrading for now until the regression is fixed. I don't think using the Legacy Build System is a viable long-term solution. 1.0.10 seems to be fine. My guess is it has to do with https://github.com/kmagiera/react-native-gesture-handler/commit/8f0b33f5e93fe71b968da3a51c2f6d91a0f91996

In my case, after cleaning eveything and install dependencies, I changed target to 10.0
Like https://github.com/kmagiera/react-native-gesture-handler/issues/387#issuecomment-449424335

@osdnk
finally got project compiled in xcode with a clean build.
I have to run this first.
rm -rf ~/Library/Developer/Xcode/DerivedData/*

Then building is ok.
However, running react-native run-ios still give me that error. Wondering if that is caused by npm cache?

thanks

I got the same error, i update to latest version "react-native-gesture-handler": "1.10.1" and everything work fine :D


Documentation grammar and typo fixes

Many small fixes for typos and minor inconsistencies in the project documentation.

 It was such a good read (I will be recommending this document to students in future) that I wanted to help improve it however possible.

Apparently I typo'd identity as itentity in the commit title (the actual diff is correct, though). How embarrasing!


Add flake8 to tox.ini

Short description:
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

!verified


UI should only allow correct BFT numbers

Hit a bug if you try to do non BFT numbers. Should limit it to 4/7/10/13/16/19, can't go over that currently becuase of non-taproot OP_CHECKMULTISIG limitations

Closed by https://github.com/fedimint/ui/pull/464 afaik


Play with engine itself - LegalPlayer -> Winboard -> UI -> WinBoard -> LegalPlayer externally

Deprecated commands but maybe need taking into account:
white
go
Sent when the engine is in force mode or playing Black but should switch to playing White. This sequence is sent only when White is already on move. If you set the feature colors=0, "white" is not sent.
black
go
Sent when the engine is in force mode or playing White but should switch to playing Black. This sequence is sent only when Black is already on move. If you set the feature colors=0, "black" is not sent.

It looks like correct handling of
force
and
go commands should solve the issue completely.


Error: Unable to find git in your PATH.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
flutter upgrade
Error: Unable to find git in your PATH.
(base) PS C:\Users\god> git
Actual results
I have git

Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

Unable to run any flutte rcomand.

Please provide the logs and the flutter doctor output.

Please provide the output of a flutter command that fails with the --verbose flag

@jinfagang
As mentioned above, please provide verbose log. Does this error started occuring after flutter upgrade only or before it too ?
Also check if you have properly set the flutter root variable.


function page breadcrumb focus seems off

function page breadcrumb focus also seems off with keyboard navigation

Originally posted by @jnm2377 in https://github.com/carbon-design-system/carbon-platform/discussions/1268#discussioncomment-3684230

Hey team! Please add your planning poker estimate with Zenhub @andreancardona @francinelucca @jdharvey-ibm


One can open infinite number of Code Viewer Windows

Probably would be better to let user reuse an already opend generic Code Viewer Window.

But, if we need many Code Viewer Windows, then we should at least identify each Window (for instance with Session ID) and tag and bound them to each Column. Something like the image below:

It should be possible to configure the code viewer to display a single instance while still allowing the parent form to be usable.  It's probably not going to be worth the effort of keeping track of multiple code windows and working out if a new one should be launched.  It could either work as a single instance (still allowing access to the main form) or allow unlimited instances to be launched.

Fix included in 2.22.6 🚀


Feature/profile support

Codecov Report

Merging #1054 into develop will decrease coverage by 0.06%.
The diff coverage is 82.05%.

@@             Coverage Diff             @@
##           develop    #1054      +/-   ##
===========================================
- Coverage     84.3%   84.24%   -0.07%
===========================================
  Files           39       40       +1
  Lines         3390     3421      +31
  Branches       605      608       +3
===========================================
+ Hits          2858     2882      +24
- Misses         401      407       +6
- Partials       131      132       +1


Internal server error trying to open Trakt list (with and without custom APIs)

Expected Behavior
_Elementum >> Shows >> Trakt >> My Lists should display my custom TV Show lists but instead I receive:
Current Behavior
Elemental Internal Server Error
Note that Settings >> Trakt does display my username and the name of my userlists if I want to configure them for adding or removing.
Steps to Reproduce (for bugs)

Input custom Trakt API client and secret, recheck and save
Authorise Trakt and save
Try to open a custom Trakt list through Elemental
Uninstall and repeat without custom Trakt API client and secret.

Context
I am unable to browse my preferred content.
Your Environment

Version used: 0.1.58
Environment name and version: Kodi 18.7
Operating System and version: Ubuntu 18.04
Link to Kodi debug https://paste.kodi.tv/uvusenujiq

@antonsoroko pointed out the following error which looks like a bug:
runtime error: invalid memory address or nil pointer dereference

have same problem today .. any help or fix

I believe that the API was revoked.
got mine fixed using a private Trakt API.

@dairefagan is there still a bug with 0.1.61?

Closing without getting response.


CMD error with Plex when searching for query with special characters

To resolve, parse the input string to remove certain characters, in particular & which doesn't bode well with web GET/POST requests when appending to the URL for obvious standards reasons.

Resolved with build v1.5 (https://github.com/ThatGuyJacobee/Elite-Music/commit/c14f8e24edf477db10f3090561afcbb52d0f7051).


Test added for Number

In file spec/std/number_spec.cr, tests have been added for the methods abs, abs2, sign divmod and <=>

@HeleneMyr Thank you! Please run crystal tool format to fix indentation issues.

Done

Thanks!!


Twice render on update useQueryStates's field

Context
What's your version of nuqs?
"nuqs": "^1.17.1"

Next.js information (obtained by running next info):
Operating System:
  Platform: linux
  Arch: x64
  Version: #107~20.04.1-Ubuntu SMP Fri Feb 9 14:20:11 UTC 2024
Binaries:
  Node: 18.19.1
  npm: 10.2.4
  Yarn: 1.22.19
  pnpm: 8.9.2
Relevant Packages:
  next: 14.1.1
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.3.3
Next.js Config:
  output: N/A

Are you using:

✅ The app router
❌ The pages router
❌ The basePath option in your Next.js config
✅ The experimental windowHistorySupport flag in your Next.js config

Description
I'm using useQueryStates with multiple states, when i update any state using this
setFilter(previousState => {
    return {
        query: query
    }
}),

then it's render twice, i've added one trace hook on home page to debug how many times render and which prevent the multiple render.
Reproduction
you can find the minimal code here:- https://github.com/Kavan72/nuqs-test

I can confirm that. It re-renders twice on Next.js 14.1.0 and only once on Next.js 14.0.4. Is this possibly related to Next.js starting to support native window.history.pushState methods? https://github.com/47ng/nuqs/issues/516

I understand. Is useSearchParams used internally in nuqs? I don't use it anywhere in my code.

Yes, it's used internally to obtain the search params during SSR for dynamic pages, and at mount time when rendering client-side.

Got it. I just tested using useSearchParams as single "source of truth" for the state and the UI feels less "snappy" because it re-loads the full page before applying the state change in the query params.
Do you have a plan/idea for nuqs to support this new behavior of Next.js with a single re-render like for Next.js <=14.04?

it re-loads the full page before applying the state change in the query params.

That sounds like a bug, you should try using the latest<EMAIL_ADDRESS>which was published yesterday, it includes a fix for full-page reloads when combined with fragment anchor tags (see #498).
The whole "mental model" as @tordans mentioned in #408 needs to be laid out in the docs, with possible escape hatches to accommodate for various rendering patterns. Having an option to control what state is updated when would be a nice feature.
The issue becomes retro-compatibility with older versions of Next.js and the app/pages router behaviour differences, there are already a lot of hacks in place to make sure it all works fine. I've been considering dropping support for older Next.js versions in nuqs@^2, to remove those hacks and focus on making it future proof. Compatibility for the version range where shallow routing was in flux (14.0.2 to 14.1.1 included) will definitely be dropped, as there are contradictory behaviours which nuqs only supports through version detection. It's a matter of knowing what to do for 13.1.x to 14.0.1.

That sounds like a bug, you should try using the latest<EMAIL_ADDRESS>which was published yesterday

I mis-explained. It is not doing a hard reload but rather re-renders the page on the client-side (shows in XHR network requests in browser). Not a bug but expected for router.push from next/navigation, right?
Thanks for the explanation. A clarification of this would indeed be nice because it might confuse users (like me) why multiple re-renders occur when updating query state.

@franky47 @phipag i just pushed the updated version of next.js to 14.1.2, seems like issue is there, also update on my repo https://github.com/Kavan72/nuqs-test

@Kavan72 I don't think this behavior is changed in Next.js 4.1.0 vs. 4.1.2. As @franky47 explained, two re-renders are expected for all Next.js versions which support the native window.history.push operation. The first re-render will update the internal state of nuqs and the second re-render is caused by the Next.js useSearchParams hook picking up the change of the browser URL.
As I understand it, updating the internal state first will make the UI feel quicker because it does not block until the browser URL is updated.

@phipag okay thanks for clarification, btw i've found one more interesting bug, suppose i've 3 states
export const FilterType = {
    query: parseAsString.withDefault(''),
    leadStatus: customParseAsArrayOf(parseAsString).withDefault(['CONFIRMED', 'QUALIFIED', 'PENDING']),
    withRecognizedTerms: parseAsBoolean.withDefault(true)
}

export default function Home() {

    const [filters, setFilter] = useQueryStates(FilterType, {
        clearOnDefault: true,
    })

    useEffect(() => {
        console.log(filters)
    }, [filters.query, filters.leadStatus]);

    return (
        <Test filters={filters} setFilter={setFilter}/>
    );
}

if i update query state then useEffect will call once but if i update leadStatus which is array state then useEffect will call twice

@franky47 can we create some kind of unique hash to useEffect and have some parser for state and url 🤔

You don't need a hash:
useEffect(() => {},  [JSON.stringify(state)])

You don't need a hash:
useEffect(() => {},  [JSON.stringify(state)])

@franky47, I understand your point, but my question is: can't we use the same reference to update non-primitive data types to avoid this problem? I'm fine with using JSON.stringify for non-primitive data types; I just want to improve this library as much as we can 😊.

I did some experiments with a client-side cache of state references (keyed by the serialized search param string), which works fine to ensure referential stability for updates, but it breaks upon reloading the page when that cache gets rebuilt, so it's not super reliable.

PSA<EMAIL_ADDRESS>was released with a fix for this issue. Non-primitive states (arrays, objects etc) are now referentially stable when their values don't change. This will help reducing the amount of unnecessary useEffect re-runs.
Note that this does not prevent re-renders, unfortunately one side-effect of depending on Next.js' useSearchParams is that it re-renders anytime any search param changes.

Thanks, I will test this soon.


[website] Add new gold sponsor

[x] I have followed (at least) the PR section of the contributing guide.

@mnajdova
My bad. Fixed the url!


Finish user stories

As a project manager
I need to complete all user stories for the GiftLink project
So that the development team has a clear roadmap of features to implement
Details and Assumptions

User stories should cover all major features of the GiftLink application
User stories should follow the provided template

Acceptance Criteria
Given the project requirements and template
When I review the completed set of user stories
Then all major features of GiftLink should be represented
And each user story should follow the provided template format

All stories have been created


Signature V2, with ASCII-string prefixes

change out SignatureInfo to match KBFS's

ship KIDs not raw Ed25519 keys over the wire
use Go-based typing to make sure we got it
introduce new keybase1.BinaryKID datatype

add V2 prefixed signature options
upgrade to V2 prefixed signatures in chat
Add tests for a various verification failures

r? @oconnor663
Also, FYI: @akalin-keybase  and @aalness

Also, CC: @patrickxb, this changes the way chat signatures work a bit.

:+1:

LGTM


Ubuntu: depth socket not started

Hey there!
I have the very same code running on windows, but am encountering a problem when migrating over linux.
In short, in this line: start_depth_socket(exchange, binance_order_book_message_listener), binance_order_book_message_listener gets never called.

Hey,
This might just be variable naming but depth socket expects a market symbol not the exchange name. I can confirm it's working for me on ubuntu passing "LTCBTC"

yes, this is actually what i'm passing.
As i said, code is working on windows :)

Does this do anything?
https://gist.github.com/michja/c082c8bc65fe2471299264222b26ab17

damn... it does.

haha damn! I guess you got the good news... and the bad news

I don't get it... the only difference is that the access to bm is done through another thread, using a global var :/

Found my problem!
If bm.start() is called from the same thread that starts the depth socket, then there's no problem.
However, if a thread calls bm.start then a thread starts the depth socket... nothing.
any clue?

it seems to be a problem related to the socket library used.
I'm afraid I need to reformat my code to accomodate that.

wow, it is even stranger than that.... if the bm.start is not called from the same function then the depth socket doesn't start.

I guess I can close that issue given it's more of a socket issue :)


DLMBL 2024 notebook

This PR adds the exercise for the DL@MBL 2024 course (Part 1) for the image translation.

this PR should be merged before #119.

I am also updating cellpose to 3.0.10 in this PR. We were already using 3.0.1 so this shouldn't cause issues.

@ziw-liu merging this.


Generic trait bounds and the "?" operator

This is a followup to this stackoverflow question: https://stackoverflow.com/questions/67352894/rust-error-conversion-for-generic-fromstr-unwrap-errors-out-with-unsatisfied-tr/67354940#67354940
I have the following error defined
use thiserror::Error;

/// ParseTreeError enumerates all possible errors returned by this library.
#[derive(Error, Debug)]
pub enum ParseTreeError {
    /// Represents an empty source. For example, an empty text file being given
    /// as input to `count_words()`.
    #[error("Source contains no data")]
    EmptySource,

    /// Represents a failure to read from input.
    #[error("Read error")]
    ReadError,
}

Now consider the following code from the SO Answer (slightly changed)
fn test_parse<T>() -> Result<(), ParseTreeError>
where
    T: FromStr,
    <T as FromStr>::Err: fmt::Debug,             // <<<<
{
    let mut val = T::from_str("5").unwrap();
    Ok(())
}

As suggested by the answer it seems like to allow the usage of ? operator I would need to implement From<<T as FromStr>::Err>
What would be the correct way of doing this with thiserror? Changing ParseTreeError to generic and using #[from] seems to be causing issues with conflicting trait implementations.

Hi @vikigenius, sorry that no one was able to provide guidance here. If this is still an issue, you could try taking this question to any of the resources shown in https://www.rust-lang.org/community. This crate is fairly widely used so lots of people will have experience to help you out.


Problem on specify data-base-path on Docker image

Hello...
I am trying to specify the data-base-path , but it seem there is some permission issue while I mount the data-base-path to the file system, can anyone help?

Thanks for raising.
We recently updated the default log path to be under the data path if it's been specified, but it looks like the log-file-name-pattern path hasn't been updated.
As a work-around, you could specify
--log-file-name-pattern="/chaindata/logs/teku_%d{yyyy-MM-dd}.log"

on the command line and it should get you running again straight away.

Thanks, I also checked some permission issue can be fixed with chown 1000:1000 -R /xxxx which on the host system volume, Thanks.


♻ Use samples instead of hard-coded value

You defined the samples variable but don't reuse it. This should then also allow for setting samples from caller functions in the future

@janselv Heya, any plan on reviewing this ?

@janselv Heya, any plan on reviewing this ?

Thanks for contributing @MrSkwiggs, merged!


Alerts on pod failure

How can we the alerts for the pod in lens?

Please could you elaborate a bit what kind of alerts do you mean?

in lens we can see the pod status if its running or not. and lets say after new deployment pod went to failed state or shows some error, can I trigger alert that this pod is not running?

I believe that this would be best suited as an extension that can react to the status changes of kube objects.


Process incoming Facebook images in threads, to speed up save/retrieve queries for Facebook data

Enhanced delete_all_voter_information_permanently to also delete facebook_users and facebook_link_to_voter so that processing Facebook images in threads can be fully tested for new users.
Now delete_all_voter_information_permanently logs what voter was deleted, and who deleted them.

👍


Vet clinic database: query multiple tables

Create a table named owners with the following columns:

id: integer (set it as autoincremented PRIMARY KEY)
full_name: string
age: integer

Create a table named species with the following columns:

id: integer (set it as autoincremented PRIMARY KEY)
name: string

Modify animals table:

Make sure that id is set as autoincremented PRIMARY KEY
Remove column species
Add column species_id which is a foreign key referencing species table
Add column owner_id which is a foreign key referencing the owners table

Insert the following data into the owners table:

Sam Smith 34 years old.
Jennifer Orwell 19 years old.
Bob 45 years old.
Melody Pond 77 years old.
Dean Winchester 14 years old.
Jodie Whittaker 38 years old.

Insert the following data into the species table:

Pokemon
Digimon

Modify inserted animals so it includes the species_id value:

If the name ends in "mon" it will be Digimon
All other animals are Pokemon

Modify inserted animals to include owner information (owner_id):

Sam Smith owns Agumon.
Jennifer Orwell owns Gabumon and Pikachu.
Bob owns Devimon and Plantmon.
Melody Pond owns Charmander, Squirtle, and Blossom.
Dean Winchester owns Angemon and Boarmon.

Write queries (using JOIN) to answer the following questions:

What animals belong to Melody Pond?
List of all animals that are pokemon (their type is Pokemon).
List all owners and their animals, remember to include those that don't own any animal.
How many animals are there per species?
List all Digimon owned by Jennifer Orwell.
List all animals owned by Dean Winchester that haven't tried to escape.
Who owns the most animals?

Thanks for the review  @DeliceLydia !


Can't get the shortcuts, to move to specific desktop, to work

Hi.
Currently I use this script:
https://github.com/pmb6tz/windows-desktop-switcher
But wanted to try yours. But it does not want to switch to desktop by pressing e.g.:ALT+2
I am using Windows 10 PRO build 1607

Hi. It should do that with the default configuration. Do you have at least two virtual desktops (since ALT+2 switches to Desktop 2)?

I have 3 desktops
Sent from Nexus 6P
On 30 Oct 2016 21:53, "Sergio Dias"<EMAIL_ADDRESS>wrote:

Hi. It should do that with the default configuration. Do you have at least
two virtual desktops (since ALT+2 switches to Desktop 2)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sdias/win-10-virtual-desktop-enhancer/issues/7#issuecomment-257178311,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE7NQooMhRFaZ-mnEsFtdVR2HnCMzG19ks5q5QPQgaJpZM4KkbcD
.

@tomiu @phodge25
I am currently revising all the old issues and bugs: do you still experience this probem on the last version?


Add support to show settings panel when logged out

Right now, when logged out, none of the settings are accessible. I think this shouldn't be like that in my opinion.
At the moment, no pages will have to be hidden, but I'd like to get implementing "Login-required settings pages" out of the way within a PR to add the settings button to the login dropdown.

will do soon

if someone feels like tackling this first feel free since i have a plethora of fixes to tackle first lol

fixed it by directly committing, sorry pedrito, will do prs from now on
it hasnt implemented anything it just adds the button