Minor Review Responses

From Reviewer 1, this includes:

[ ] The paper gives an overall good overview of the motivations for and the development of CYCLUS. Solution: no changes necessary
[ ] IA: would be good to mention that such simulators can be used both for questions addressing operational facets of nuclear energy systems, e.g. interim UNF storage optimisation,as well as 'strategic' questions allowing to address R&D-options assessment. Solution: mention this fact
[ ] IB.iii: The simulation codes with discrete materials tracking are: COSI, FAMILY21, GENIUS 1 and 2, DANESS, NFCSim and ORION. DESAE doesn't !! Not clear about VISION. DYMOND didn't neither. Solution: remove desae and add orion.

From Reviewer 2:

[ ] The manuscript has the following title: "Fundamental Concepts in the Cyclus Fuel Cycle Simulator Framework and Modeling Ecosystem". It is not clear to me what "and Modeling Ecosystem" refers to. Solution: Remove modeling ecosystem from the title, but add a mention of this notion in the abstract
[ ] page 3 (Background): "With few exceptions, these metrics are derived from mass balances calculated by a fuel cycle simulator. [4]." : It would be nice to illustrate what are the few exceptions. Please read "C. Poinssot, S. Bourg, N. Ouvrier, N. Combernoux, C. Rostaing, M. Vargas-Gonzalez et J. Bruno, Assessment of the environmental footprint of nuclear energy systems. Comparison between closed and open fuel cycles., Energy, n°1In Press, pp. 1-13, 2014. Solution: cite C. Poinssot.
[ ] page 18: "SWUs" please give a complete list of acronyms: here Separate Work Units Solution: expand SWU (this is why we have the glossary package, folks!!!)
[ ] page 20: bad reference for:  "Section ?? " Solution: fix ref.
[ ] page 20: What is a BatchReactor? Solution: Point out in the response that this term is defined immediately after its first mention on this page. I don't think this needs to be solved. However, we can point out that it's deprecated in 1.3 and we can add a parenthetical where it is mentioned on the third line.
[ ] page 23: typing error: "…and it cannot…" Solution: Fix typo.

on item IB.iii, maybe we just need to be more clear about what we mean by discrete material tracking - because I am fairly confident that what we mean by it in Cyclus is significantly different than what the reviewer is thinking.  And as far a my understanding goes for the other codes, they don't do discrete material tracking.  Some of the codes may have various pieces - i.e. knowing how long some material was in a reactor, remembering a "fuel type", or even remembering how long material has been at some location.  But they don't track and preserve material object identicy completely like we do in Cyclus.

@rwcarlsen are you suggesting we claim that Cyclus' "discrete material tracking" is significantly different than other codes that have that capability? While I don't doubt that there are different "flavors" and we might have the most feature-complete "flavor", this is likely not a battle we want to fight. I think the suggested fix is reasonable, but would of course be open to more discussion.

Regarding Poinssot et. al. (note they use 'indicator' rather than metric):
They are the green-house-gases emissions (GHG, gCO2eq/kWhe) [13] and [21], the atmospheric pollution (SOx and NOx, mg/kWhe) [21], the water pollution (mg/kWhe) [21], the land-use (m2/GWhe) [13], the water consumption and withdrawal (L/MWhe) [13] and [22], and the production of technological waste (g/MWhe) [23].

...

They are the radioactive gaseous and liquid releases (Bq/kWhe) and the solid radioactive waste production (g/MWhe or m3/MWhe) [21]. The footprint of the required repository is also considered.

... acidification, eutrophication and POCP (photochemical ozone creation potential) phenomena, ...,  eco-toxicity and human toxicity was used...

It seems like the reviewer would include only the middle group as actual fuel-related mass balances. If that is true and we cite this paper, I don't think we can say With few exceptions. Many of the others could be thought of as functions of mass flow (via power density) though...

On discrete material tracking... If we are going to declare that some codes have it and others don't, then we should probably define the criteria that we are using.  In so doing, we could find a way to acknowledge various implementations of all or part of what we there define.

On Poinssot...  I generally also include "facility operation history" with "mass balances" as the primary data that is necessary to derive the majority of indicators/metrics.  Would this help?

Regarding BatchReactor: If it doesn't produce substantive changes, I think it's fair game to update this section to be consistent with 1.3.

re: poinssot @gonuke I don't know if it does help. It risks turning one simple sentence into ten. That is.

creating an excuse to not cite it doesn't seem kind.
however, if we add operation history, then we don't have any need to include the citation.
So, if we add operation history, but also want to cite poinssot, so that no one is offended, a need arises to write a whole bunch of sentences about what kinds of metrics come from what kind of raw data.

I guess I missed the context of how we would cite Poinssot.  I was thinking option (3) but without as much connection between which raw data and which metrics.

@gonuke let me see what I can do on that front.

Doing a full cyclus/cycamore recycle simulation will require 1.3, so I
think all relevant changes should be included. However, I think the
reviewer was implying that it was unclear what a BatchReactor was, which I
agree with Katy: it is literally defined in a "term: definition" table.
On Tue, May 12, 2015 at 12:32 PM, Paul Wilson<EMAIL_ADDRESS>wrote:

Regarding BatchReactor: If it doesn't produce substantive changes, I think
it's fair game to update this section to be consistent with 1.3.
—
Reply to this email directly or view it on GitHub
https://github.com/cyclus/fundamentals-paper/issues/115#issuecomment-101359685
.

--
Matthew Gidden, Ph.D.
Postdoctoral Associate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph.<PHONE_NUMBER>

@gidden , I just went ahead and dropped batchreactor to make things 1.3 compatible. Should I add it back?

Nope I think that's fine.

closed with #131


Comment out build step from git_sync.sh

The build steps always fail anyways, so the only thing this script does it copy the generated code from running the program in XCode to the azure-sdk-for-ios-pr repo.

You change looks good.


Force apt cache update after installing Zabbix's gpg key

While I was testing zabbix-server role I encounter the following
problem:
...
WARNING: The following packages cannot be authenticated!
zabbix-server-mysql

msg: '/usr/bin/apt-get -y -o "Dpkg::Options::=--force-confdef" -o "Dpkg::Options::=--force-confold"   install 'zabbix-server-mysql'' failed: E: There are problems and -y was used without --force-yes

FATAL: all hosts have already failed -- aborting

This happens only when I call zabbix-server role quickly after other
role that updates apt cache.
I trace the problem to Debian.yml file "Installing zabbix-server-mysql"
task @ line 33.
According to apt module documentation [1] setting update_cashe to yes
should force cache update, but the next statment cache_valid_time=3600
will skip the update if the apt-cache has been ran in the last hour. But
if the cache is not updated after adding Zabbix's gpg key apt has no way
to vefify the integraty of the .deb file.
In order to force an cache update after adding Zabbix's key the statment
cache_valid_time=3600 has been removed.
Posible improvment would be to force apt cache update only if previus
task (adding the gpg key) have made real configuration change.
[1] http://docs.ansible.com/apt_module.html

Thanks!


Support for Tacx Neo Bike i.e. FE-C over Bluetooth support

Here are all my notes on this issue so far, the support for the bike I use is currently only partial, when I connect it via bluetooth, I see this warning message

Besides this, the following observations about dundring dashboard vs the built-in screen on the bike

Metric
Dundring
Bike Screen
Notes

Heart rate
138
138
✅ Dundring connected to the sensor via bluetooth, bike automatically picks ANT+ signals. Same sensor, same data, all good.

Wattage
4 watt
4 watt
✅ They are usually off a bit but mostly the same. I think dundring reads this from the device.

Speed
4.4 km/hr
17.8 km/hr
⚠ This is not a problem, the speeds are usually similar w/ dundring lagging a bit to reflect (so the big difference here is irrelevant). What I am curious about is why aren't they same? Can we just use the emitted speed from the device?

Cadence/rpm
0 rpm
0 rpm
❌ The image is misleading, I probably stopped paddling to take a picture. The bike screen shows RPM while dundring doesn't, it is probably not reading it from my bike.

So why doesn't it work? (snippets copied from code, might be outdated when you read this)

Dundring needs cycling_power (wattage) service and optionally fitness_machine (FTMS, controlling bike resistance) service

const device = await navigator.bluetooth
      .requestDevice({
        filters: [{ services: ['cycling_power'] }],
        optionalServices: ['fitness_machine'],
      })

Tacx Neo Bike doesn't do FTMS, quoting this article.)

Tacx actually does support FTMS on all their non-NEO series just fine, but doesn’t do FTMS on the NEO series due to FTMS’s lack of ability to transmit weight information to apps (which is needed for the virtual flywheel in this specific trainer series).

Tacx does Ant+ FE-C over bluetooth i.e. it uses bluetooth for transport but transfers FE-C protocol data (questions: how does bluetooth request this device, what service to use? (probably fec uuid directly))

Noting the screenshot above, I think we can use requesting cadence also https://www.bluetooth.com/specifications/specs/cycling-speed-and-cadence-service-1-0/

Next steps (for me), play around with web bluetooth a bit with a simpler device and eventually implement control support for Tacx Neo Bike 🎉
Resources

Bluetooth fitness machine service (FTMS) https://www.bluetooth.com/specifications/specs/fitness-machine-service-1-0/
CABLE FE-C to FTMS https://www.smartbiketrainers.com/cable-releases-a-new-firmware-update-and-support-for-fe-c-to-ftms-5046
https://www.dcrainmaker.com/2016/07/everything-you-ever-wanted-to-know-about-ant-fe-c-and-bike-trainers.html
https://www.thisisant.com/developer/ant-plus/device-profiles#525_tab
https://www.bluetooth.com/specifications/assigned-numbers/ (useful to find service UUIDs)
https://developer.chrome.com/articles/bluetooth/

Awesome work! 🙌 We obviously have a lot to fix, and your notes will be essential in that process!
Most of the core developers have been using Wahoo smart trainers, which would explain the lacking support for other brands at the moment 🙈 Very interesting that they use the Ant+ FE-C protocol over Bluetooth – I had no idea!
It appears that now is the time for a review and improvement of the bluetooth-parts of the code – I'll try to prioritize it during the holidays ❄
As always, feel free to reach out to me over at Slack if you want to discuss or even rubber duck 🦆

I can probably explain the speed difference.
Regarding the picture, the reason the difference is so huge (17.8 km/h vs 4.4 km/h) is probably because the speed on your device is more realistic in terms of speed decrease over time.
Dundring's speed estimate is just based on the current power (4W) and therefore instantly decreases to a low speed.
The device probably decreases more gradually over time, like in the real world.
For the normal differences between the two, it is probably because the speed estimate from the device is just a bit different from the speed estimate used in dundring (which is based upon https://www.omnicalculator.com/sports/cycling-wattage).
The estimate implementation in dundring can be found here : https://github.com/sivertschou/dundring/pull/147
Regarding using the emitted speed from the device, I am not sure. I guess it is nice that they are in sync, but there is also some possible issues regarding "fairness" when comparing speeds across devices.
We will take it into consideration.
Hope that kinda made sense and thanks for the great and detailed feedback 🎉


Operator override in case when env is passed as map[string]interface{}

Hi @antonmedv ,
We have a use case where we want to support decimal in expr library. I was thinking to use ConstantNode for representing decimal values and overriding operators like +, -, *..etc. When I am adding my function for add in environment (which is a map of string to interface{}), it gives me error function add for + operator does not have a correct signature. I have a requirement to pass environment as map[string]interface{}, so I wanted to know if there is any workaround for this?  Please help!
Add Function Signature in environment
"add": func(a decimal.Decimal, b decimal.Decimal) (decimal.Decimal, error) {
			z := a.Add(b)
			return z, nil
		},

We are using github.com/shopspring/decimal library here.
Thanks

Check what operands are passed. Are they int? Try to use interface{} as parameter to check what kind of params are passed.

add function is not getting picked up only. The program fails at compile step. Sharing go dev playground link: https://go.dev/play/p/3GNKkZ0OezT where it shows this error.

The overloading of operators does not support the return of errors at the moment. If you adopt the example from the documentation, it works fine, i.e. use "add": func(a interface{}, b interface{}) interface{} rather than "add": func(a interface{}, b interface{}) (interface{}, error).
On the other hand, I agree: somehow it should be possible to throw errors even when overloading operators. At least it's not clear to me how, in the concrete example, you could handle an invalid Decimal. But that would be a feature request.

Panic will be okay in this case. All panics converted to errors.


Add Readme

Project needs a readme that instructs on how to build and run the application

Added Readme.md to master

Added Readme.md to master


use name as title for gravatar

Another tweak - and I didn't include it here as I'm not sure your opinion, but I'm trying to decide if it makes sense to have the Gravatar and/or text name be links (just to the homepage), and maybe hiding the top menu link if desired. LMK if you think it's worth having configuration for those sorts of things or if that's getting to be overkill, thanks!

New icons and/or buttons are cool, but I'm wary of adding too many configuration flags. At some point it's more straightforward to just provide an override for the sidebar partial with a tweaked layout. What do you think?

Not a bad idea - I'll give it some more thought and see if I can come up with a proposal. The main reason I think it's helpful to hide the author name is it may be redundant with the gravatar - that said, the change to put the name as title for the gravatar seems unobtrusive and helpful regardless. So I may tweak this pull request to just be that, and think more about configuration/overriding.

Pruned pull request, may revisit configurability/overriding later. Thanks!

Actually I think I'll redo this as well in whatever later changes. Thanks!

Or actually, I think this pull request fits. LMK, thanks!

Looks good.


Exercise indexes keep changing after each edit

After entering edit i/1 d/01/12/2019, the list is sorted? in some unintuitive manner.

Sorted by date


CU-86drpncat - Wrong tuple type hint is compiled on Neo3-boa

Summary or solution description
Changed how the compiler handles tuple annotations to have the expected Python behavior.
Before it was handled the same way as list annotations, making tuple[int], tuple[int, int], tuple[int, ...], tuple[int, int, int] and similar to be handled as the same.
Also added support to ellipsis (... annotation) to be able to handle tuples without specific length

Task linked: CU-86drpncat Wrong tuple type hint is compiled on Neo3-boa

coverage: 91.926% (-0.003%) from 91.929%
when pulling 63b77ec31dbe169bdd142e5ef2130e98a4db9d1d on CU-86drpncat
into 1a1f407ad7c9155479cc99adbbeab1d81e6155d1 on development.


net/rhp: add contracts tests

Adds tests that try to trigger all of the error states in ValidateContractSignatures, ValidateContractFormation, ValidateContractRenewal, ValidateContractFinalization, ValidateProgramRevision, ValidatePaymentRevision.  Brings contracts.go coverage to over 90%.

This should be good.


'ContainerInstanceManagementClient' object has no attribute 'container'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az container logs
Errors:
The command failed with an unexpected error. Here is the traceback:
'ContainerInstanceManagementClient' object has no attribute 'container'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 112, in handler
    client = self.client_factory(self.cli_ctx, command_args) if self.client_factory else None
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/container/_client_factory.py", line 18, in cf_container
    return _container_instance_client_factory(cli_ctx).container
AttributeError: 'ContainerInstanceManagementClient' object has no attribute 'container'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az container logs --resource-group {} --name {}

Expected Behavior
Environment Summary
Linux-5.4.0-88-generic-x86_64-with-debian-bullseye-sid, Ubuntu 20.04.3 LTS
Python 3.6.10
Installer: DEB

azure-cli 2.28.0

Extensions:
interactive 0.4.5

Additional Context

route to service team

@FatemeTardasti-TomTom Looks like this has since been fixed and I can confirm that there is no error now. Apologies for no updates on this issue sooner. Unless you are still seeing this error on your end with the latest version, I believe this could be closed.

Hi,
I admit that the issue isn't valid anymore. Please feel free to close the issue.
Thanks
Fateme Tardasti

From: msftbot[bot] @.>
Sent: Tuesday, February 15, 2022 3:01 PM
To: Azure/azure-cli @.>
Cc: Fateme Tardasti @.>; Mention @.>
Subject: Re: [Azure/azure-cli] 'ContainerInstanceManagementClient' object has no attribute 'container' (#19715)
Hi, we're sending this friendly reminder because we haven't heard back from you in a while. We need more information about this issue to help address it. Please be sure to give us your input within the next 7 days. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!
—
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli%2Fissues%2F19715%23issuecomment-1040309365&data=04|01|fateme.tardasti%40tomtom.com|633c5f9067d745e5467f08d9f08bac2f|374f80267b544a3ab87d328fa26ec10d|0|0|637805304955400714|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=mHiLHKLx3iXydNJWA5A6ohlVBfLxUN4AtDhm4GzUKlI%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVN6JCTJB4JK6LPUWAHXVT3U3JMDXANCNFSM5E5SSGVA&data=04|01|fateme.tardasti%40tomtom.com|633c5f9067d745e5467f08d9f08bac2f|374f80267b544a3ab87d328fa26ec10d|0|0|637805304955400714|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=lhXMbmvSEj6ZNoZCKLKXlksImCldgsrFQYDmkFbWc98%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|fateme.tardasti%40tomtom.com|633c5f9067d745e5467f08d9f08bac2f|374f80267b544a3ab87d328fa26ec10d|0|0|637805304955400714|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=b7domcmif6ho7H1LBB7TBnJwl0uWuDiEBDsS0c4qGiE%3D&reserved=0 or Androidhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|fateme.tardasti%40tomtom.com|633c5f9067d745e5467f08d9f08bac2f|374f80267b544a3ab87d328fa26ec10d|0|0|637805304955400714|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=qZAfVbHnrHP6Q8MoKZArDJcu%2FPv4X%2B4bA5VEkxyT3Ik%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


prompt form moved to a separate file

new gradio_ui directory was created with a first file prompt_form to which I extracted the prompt form logic

I believe this will improve code structure and readability in long run

Hi @fazpu , it's not what ChatGPT does or HuggingChat does, etc.  It feels more normal original way.  But happy to discuss.

Hi @fazpu , it's not what ChatGPT does or HuggingChat does, etc. It feels more normal original way. But happy to discuss.

@pseudotensor got it. Only the code refactoring changes remained.


xonsh: migrate to /etc/xonsh/xonshrc

Description of changes
xonsh no longer loads /etc/xonshrc by default. It now uses /etc/xonsh/xonshrc. See https://github.com/xonsh/xonsh/issues/4761

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

I wonder why this isn't mentioned in Xonsh's release notes.
From the code change in xonsh/xonsh@0957c2d, it looks to me like it was with v0.12.0 (particularly xonsh/xonsh#4635), but the release notes for that version only mention an additional directory, from which configs will be loaded if present:

Changed:

Both *.xsh and *.py files inside $XONSHRC_DIR will get loaded now.

(And the old path is still mentioned in https://xon.sh/xonshrc.html and https://xon.sh/platform-issues.html#path-helper .)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1529

Reproduced the issue.

(And the old path is still mentioned in https://xon.sh/xonshrc.html and https://xon.sh/platform-issues.html#path-helper .)

I guess it will be nice to ask (help) upstream to update the docs then :upside_down_face:


AttributeError when drawing circuit using Matplotlib

Information

Qiskit Terra version: 0.14.1
Python version: 3.7.6
Operating system: Ubuntu 16.04

What is the current behavior?
When trying to draw a QuantumCircuit object using .draw(output='mpl'), I'm getting AttributeError: 'Instruction' object has no attribute 'label' in qiskit/visualization/matplotlib.py. The circuit draws fine using the default text drawer.
Steps to reproduce the problem
The code is adapted from the linear_systems_of_equations HHL tutorial notebook. Here's a (probably non-minimal) snippet to reproduce:
from qiskit.aqua.algorithms import HHL
from qiskit.aqua.components.eigs import EigsQPE
from qiskit.circuit.library import QFT
from qiskit.aqua.components.reciprocals import LookupRotation
from qiskit.aqua.operators import MatrixOperator
from qiskit.aqua.components.initial_states import Custom

def create_eigs(matrix, num_ancillae, negative_evals):
    ne_qfts = [None, None]
    if negative_evals:
        num_ancillae += 1
        ne_qfts = [QFT(num_ancillae - 1), QFT(num_ancillae - 1, inverse=True)]

    return EigsQPE(MatrixOperator(matrix=matrix),
                   QFT(num_ancillae, inverse=True),
                   num_time_slices=50,
                   num_ancillae=num_ancillae,
                   expansion_mode='suzuki',
                   expansion_order=2,
                   evo_time=None,
                   negative_evals=negative_evals,
                   ne_qfts=ne_qfts)

matrix = [[1, 0], [0, 2]]
vector = [1, 4]

orig_size = len(vector)
matrix, vector, truncate_powerdim, truncate_hermitian = HHL.matrix_resize(matrix, vector)

# Initialize eigenvalue finding module
eigs = create_eigs(matrix, 3, False)
num_q, num_a = eigs.get_register_sizes()

# Initialize initial state module
init_state = Custom(num_q, state_vector=vector)

# Initialize reciprocal rotation module
reci = LookupRotation(negative_evals=eigs._negative_evals, evo_time=eigs._evo_time)

algo = HHL(matrix, vector, truncate_powerdim, truncate_hermitian, eigs,
           init_state, reci, num_q, num_a, orig_size)

circuit = algo.construct_circuit(measurement=True)
circuit.draw(output='mpl')

What is the expected behavior?
No errors, and the circuit is drawn correctly using Matplotlib.
Suggested solutions
Not sure - I don't know whether this could be a Terra visualization problem or an Aqua circuit design problem (or just a problem with the code above).

There is already a fix in progress for this issue: https://github.com/Qiskit/qiskit-terra/pull/4389 It's caused by certain gates (or circuit library elements that were converted to gates or instructions) missing a label attribute which the mpl drawer incorrectly assumes is always present.

Hi, is there an ETA on a fix for this?

This should be part of the 0.15 release which is scheduled for two weeks. #4389 is fixing this 🙂

Awesome, thanks 🎉


feature suggestion: banner supports the href link

as title, it will be better if the banner can support more format instead of only text

sounds good!  Im going to be pretty busy for the next week or 2.  If would like to take a hack at it and send a PR then I can check it out!
Thanks!


Callbacks in version 3

I hope this is the right place for comments in regard to version 3.
We noticed that callbacks such as afterFit on scales axis do no longer trigger when we switch between v2.93 and v3.
  options: {
    scales: {
      yAxes: [
        {
          stacked: true,
          afterFit: scaleInstance => {
            console.log('foo');
            scaleInstance.width = 50;
          }
        }
      ]
    }
  }

This looks as expected with v2.93

With v3 the stacking does not work and the afterFit callback does not log 'foo'. It seems the callback is never called.
Is there any documentation in regard to axis callbacks in v3?
This is the link for the codepen which is use to test the behaviour and toggle between versions.
https://codepen.io/dbauszus-glx/pen/gOpBLKG

Please take a look at the migration guide:
https://www.chartjs.org/docs/master/getting-started/v3-migration.html

Ace. I must have missed that guide.


Fix typo in README.md

Q
A

Bugfix?
✔

Breaks BC?
❌

New feature?
❌

Cheers :)

Summary by CodeRabbit

Documentation

Corrected a typo in the README.md to accurately describe the integration capabilities with existing applications.

Pretty sure we have a PR merge record time here :open_mouth:

Usual case for RoadRunners 😜 😆


make bcdb tests green [unstable_bcdb_tests]

Run all bcdb tests with no skips.

u can find all bcdb errors and fails there
https://zeroci.grid.tf/repos/threefoldtech%2FjumpscaleX_core/development_hotfix/5e6e42f5ca5626d08a17f28d#errored-1
and this issue effect 10_export test after u edit it with zdb client again
https://github.com/threefoldtech/jumpscaleX_core/issues/608

I have fixed first four tests on development_bcdb_tests
commits

3eee6a6df32f1110c4f716d83016cb22888dd20b
3dbfc9dae0ceabda526544b11c34daa44e7c8585

there was issue related to

secret length in zdb
soping sonic server
code refactor (some function names has been changed so i have updated them)

please work on unstable for all bcdb tests

I am blocked on this issue:
https://github.com/threefoldtech/jumpscaleX_core/issues/624

i have fixed five test and i was working on an issure related to desk I/O error

I got this issue and it was fixed and it was blocking me.
https://github.com/threefoldtech/jumpscaleX_core/issues/640

I am blocked on this issue https://github.com/threefoldtech/jumpscaleX_core/issues/645

fixed some issue in import to unblock me and now the export test(test 10) is working

Make _test_model_get function works with bcdb name as a parameter as it cause vfs test to crash
commit
b2b078678681b631b88f8d2a029209fc0c23e90f
fix vfs test
58f5e1195e655c04e6221fa53d81e77b9a80b529
Remove skip from tests:
commit
1841ec069373147f05bfc5517d61d40932bb7731

Fixing a sonic test in BCDB
update sonic factory to use testing client instead of using the development client by default
Adding testing_env flag in BCDB to differentiate between testing and development environment

commit
1c4c246738361ea1ec6f98971b15d42db94db4ad

Tests cleaning and removing skipping form it
commit
e5af817deb6927cbafa683c2cf1165c93f930a37
fixing query limit test in bcdb tests (test 21)
commit
8f0a1f9b375c9c167081e7b4714e21dadcbcdb2f

Every test work separated from other tests  (standalone)


Unable to install Windows 11 on Macbook pro using UTM

How do I resolve this issue? I am trying to install Windows 11 on my MacBook. This error below comes after selecting the spice-guest-tools-0.164.3 12.14.14 file and clicking on play
Error message:

qemu-aarch64-softmmu: -drive if=none,media=disk,id=drive0,file=/Users/mosvinbami/Library/Containers/com.utmapp.UTM/Data/Documents/Windows.utm/Images/Windows11_InsiderPreview_Client_ARM64_en-us_22598.qcow2,cache=writethrough: Could not open '/Users/mosvinbami/Library/Containers/com.utmapp.UTM/Data/Documents/Windows.utm/Images/Windows11_InsiderPreview_Client_ARM64_en-us_22598.qcow2': No such file or directory

Thank you for the feedback, the first drive is NVME see screenshot

I am experiencing a new issue, see screenshots below and help.


Update this Chart to track each Traefik release?

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Please make this Chart (e.g. appVersion) track the https://github.com/traefik/traefik releases.
Even thou we can set the image.tag chart value, that makes it harder to the user to known which chart versions are compatible with which traefik versions. It also means, we'll end up updating two versions, instead of one.

Hello,
The appVersion is mainly updated on a minor version (v1.X.y) of Traefik, the bug fix releases (v1.x.Y) of Traefik are always compatible with the minor version.

@ldez why? It much more practical to have a chart release for each Traefik release. This is common practice across helm community. Majority of chart users don't track traefik releases, they do track helm repo. Your approach breaks end user release cycle, where have to follow patch releases, but skip the minor and major.

everybody can create a PR

Problem with this, that community contributions remain stale for months without review. While PRs from traefik team members merged in seconds, like this https://github.com/traefik/traefik-helm-chart/pull/614.

we are working on a new maintenance mode.

I could help with the CICD of this... perhaps have something like dependapot update the docker tag for traefik and raise a PR that does some basic tests to ensure things are working? Even just getting the PR up might be nice.

Yes, sure. It would be awesome, @shadycuz !

@mloiseleur
Here are the rough steps that would need to happen.

The maintainers of this repo would need to enable dependabot or renovate to handle dependency updates. I personally prefer renovate and it would probably do a better job but I think dependabot might also work for this.
We would need to create a dummy Dockerfile with the line  from traefik:v2.9.6 inside the repo. This is the file that dependabot/renovate will watch and if a new version is released than a new PR will be opened.
This is the tricky part, we will need to use the CICD pipeline to grab the new version, say 2.9.7 for example and update the charts appVersion variable. This will involve committing this change back to the repo. Which is kinda tricky because you can't do it with the normal pipeline token you will need to create some kind of "bot" github user and generate an API token for that user and save it in your GitHub secrets. That way the pipeline can use the bot users API key to commit files back.

That would be a really good first step. If traefik maintainers are willing to help out on there end than I can submit the PR with these changes. We could start working on this now and release this feature sometime after the change to traefik 3.0. Because I'm pretty sure dependabot/renovate will try to give us that version very soon.
Once this work is done you could take this even further and add some functional testing. Once those tests have passed you could automatically merge and create a github release with release-drafter.

That seems quite complex.
In renovate doc, they provide an example to update appVersion, wdyt about it ?

@mloiseleur, oh yes. That is perfect. Here is an example  of it in use. So all Traefik maintainers would need to do is enable renovate on this repo. Than we could submite a PR where we leave a comment above the appVersion like with the name of the Docker repo/image.

Yes, sure.
Thanks for the example !
Currently, we have other priorities but we'll do it as soon as we can.

I'm reopening it because current PR opened by renovate increments only appVersion and not Chart version.


The width of the detail of a suggest can be inconsistent

Version: 1.51.0-insider
Commit: 7a3bdf4ee9588755d447aa1c3b5db4a123fc11a9
Date: 2020-10-23T07:32:22.213Z
Electron: 9.3.2
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 17.7.0
Steps to Reproduce:

Make the width of the detail of a suggest much larger.
Resize the window keeping the detail opened.
The detail flips.
Then, make the width of the detail much smaller.
Resize the window again.
The detail flips even when not necessary.

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

Yeah. It should only flip (be left aligned) when running out of space but there is still enough space...

Now, when making the width smaller, the details only move to the left if there's no more space.
When making the space larger, the details go back on the right side. I assume that's still the intended behavior.


Reimplemented problematic statement with hard-coded true to attempt fixing test

Determined with Michael that this hard-coded true is not going to resolve the issue and should be retracted.


Minor fix: Add code to the values.

Add code to the node page size i.e. 2048KiB and 1048576KiB mentioned under Manage HugePages, for better readability.

PTAL!

I don't think this is appropriate. Those numbers are not literals to use for huge pages.

Hi @tengqm

Those numbers are not literals to use for huge pages.

I agree with this, but here it uses as an example so can we bold these words? just for better readability.
WDYT?

If it does not seem good to you then I will close this PR.

I agree with this, but here it uses as an example so can we bold these words? just for better readability.
WDYT?

Hi, @tengqm your thoughts on this?

imo, the code example is already in inline code format, hugepages-2Mi and hugepages-1Gi.
I agree with @tengqm and closing this PR
Feel free to reopen with /reopen in a comment to further discuss
/close


These relative modules were not found

On npm run serve got this:
These relative modules were not found

../_library/apps/room/proto/room_pb in ./node_modules/ion-sdk-js/lib/connector/room.js
../_library/apps/room/proto/room_pb_service in ./node_modules/ion-sdk-js/lib/connector/room.js
../_library/proto/rtc/rtc_pb in ./node_modules/ion-sdk-js/lib/connector/rtc.js
../_library/proto/rtc/rtc_pb_service in ./node_modules/ion-sdk-js/lib/connector/rtc.js
No type errors found
Version: typescript 4.1.6

did you do a npm run install

I also have the same issue.
So, I download and use the file directly as shown below.
$ sudo apt install subversion
$ cd ~/some_package/node_modules/ion-sdk-js/lib/
$ svn export https://github.com/pion/ion-sdk-js.git/trunk/src/_library

Workaround to automate download missed files on npm install (only Linux, without subversion install, only git binary need):
add scripts/fix_deps.sh in your project folder with content:
#!/bin/sh

set -e

cd $INIT_CWD

if [ -d ./node_modules/ion-sdk-js/lib ] && [ ! -d ./node_modules/ion-sdk-js/lib/_library ]; then
	rm -rf ion-sdk-js
	git clone https://github.com/pion/ion-sdk-js
	mv ./ion-sdk-js/src/_library ./node_modules/ion-sdk-js/lib/
	rm -rf ion-sdk-js
fi

and then in your package.json add record in "scripts" section:
"scripts": {
    ......
    "postinstall": "scripts/fix_deps.sh"
  },

and thats all

error ./node_modules/ion-sdk-js/lib/signal/grpc-web-impl.js:7:0

Module not found: Can't resolve '../_library/proto/sfu/sfu_pb_service'

I have reviewed the source code and found that the _library directory is just a name, but there are some steps codes in Makefile compilation:
proto: mkdir - p lib===cp rf./src/_library lib

clean: rm - rf src/_library

Deleting the _library directory caused the ESModule not to find the corresponding module when referencing files
So I think it's a good idea to change the name of the _library directory to a unified lib in the source code~
cc: @ionorg


Nested .create() appears to use multiple transactions

Bug description
Use of .create() with a multi-layer nested graph of related insertions appears to be using multiple transactions, but it should not.
Using a query with 'READ COMMITTED' isolation level does see records being added, when it should not.
This suggests that there might be multiple transactions/commits happening within the usage of a single .create() request when nested requests are involved, or, that 'autocommit' is being used?
How to reproduce
In my case, I have a much more complex schema, but this simplified example should suffice:
Using this schema:
model User {
  id     Int          @default(autoincrement()) @id
  email  String
  name   String
  joined DateTime     @default(now())

  posts      Post[]

  @@map("users")
}

model Post {
  author_id Int
  id        Int      @default(autoincrement()) @id
  published Int?     @default(0)
  date      DateTime @default(now())
  title  String
  author User         @relation(fields: [author_id], references: [id])

  @@map("posts")
  @@index([author_id], name: "author_ix")
}

Create a single .create() which creates a user record, and perhaps 50K of nested post records.  The actual number will vary, depending on how fast your MySQL instance runs.
Arrange your environment so that while the .create() is running, you can also use something like MySQL Workbench, to observe the number of records that are in each table, using a query something like this, on a repeated basis:
SET TRANSACTION ISOLATION LEVEL READ COMMITTED;
select 'posts' as source, count(*) from `posts` where title is not null
union
select 'users' as source, count(*) from `users` where email is not null;
commit;

You should only see the numbers of posts and users jump up once.  They should not show a progressive increase before the .create() comes back.
But, what I see is that the numbers are progressively increasing over time during the .create() lifetime.
Note: the 'READ COMMITTED' means that the results should not include any data that is in the process of being written within a transaction that has not yet been committed.   If this were 'READ UNCOMMITTED' then I expect to see the counts include those records that are not yet committed.
I have verified that the READ COMMITTED works as expected by using the following (on a separate connection) while using the above query (repeatedly, during the below being executed) to observe the results:
note: The 'select sleep()` statements cause the query to run long enough to be able to make observations.
note2: if you are using MySQL Workbench, you will need to turn off the 'autocommit' so that each statement is not automatically committed.  I suspect it might detect that the transaction is being used, but, better to be safe than sorry.

The above shows what it looks like after autocommit has been disabled.  The 'tick' and 'cross' buttons just to the left of the highlighted button are for manual 'commit' and 'rollback'.
SQL that helps observe that 'READ COMMITTED' isolation level is working as advertised:
start transaction;

use `local`;

set @a = 1;
INSERT INTO `users` (`id`, `email`) VALUES (20000 + @a, concat('ralph', 2000 + @a, '@testing.com'));
select sleep(10);

set @a = @a + 1;
INSERT INTO `users` (`id`, `email`) VALUES (20000 + @a, concat('ralph', 2000 + @a, '@testing.com'));
select sleep(10);

set @a = @a + 1;
INSERT INTO `users` (`id`, `email`) VALUES (20000 + @a, concat('ralph', 2000 + @a, '@testing.com'));
select sleep(10);

set @a = @a + 1;
INSERT INTO `users` (`id`, `email`) VALUES (20000 + @a, concat('ralph', 2000 + @a, '@testing.com'));
select sleep(10);

set @a = @a + 1;
INSERT INTO `users` (`id`, `email`) VALUES (20000 + @a, concat('ralph', 2000 + @a, '@testing.com'));
select sleep(10);

commit;

Expected behavior
There should only be 1 commit during the lifetime of a single .create() usage.
Other database clients using 'READ COMMITTED' isolation should not be able to see records that are part of the single .create() statement progressively being added.
Prisma information
Environment & setup

OS: macOS 10.15.6
Database: MySQL
Node.js version: 14.8.0
Prisma version: 2.5.1

Hey @pbelbin
I am unable to reproduce this with 2.6.1.
I have put together a small video in which I go through this: https://www.youtube.com/watch?v=JiX_-7Hgs_c
I used the following script for a reproduction:
import {
  PrismaClient,
  Post,
  PostCreateWithoutAuthorInput,
} from "@prisma/client";
import fs from "fs";
import faker from "faker";

const prisma = new PrismaClient({
  log: [
    {
      emit: "event",
      level: "query",
    },
  ],
});
async function main() {
  const posts = Array.from({ length: 100000 }).map<
    PostCreateWithoutAuthorInput
  >(() => ({
    title: faker.lorem.sentence(),
    date: faker.date.past(),
    published: 1,
  }));
  const data = await prisma.user.create({
    data: {
      name: "test",
      email: "test",
      posts: {
        create: posts,
      },
    },
  });
  prisma.$disconnect();
}
prisma.$on("query", async (e) => {
  await fs.promises.appendFile(
    __dirname + "/test.sql",
    `${e.query} ${e.params} \n`
  );
});

main();

As you can see in the video, the count didn't increase until the transaction was finished.
Also, here are generated SQL statements which only has 1 BEGIN statement. https://gist.githubusercontent.com/pantharshit00/318cdf11f6b1563b5d768dcb6b179c98/raw/a93bbaf5cecc2acde90bf9077c7e54f11da9d6e5/test.sql
Can you please try this again with the latest version? If you can still reproduce this, can you please share the generated SQL by using the method which I used in the above script. Also, please tell me which MySQL version you are using.

Closing due to inactivity


Remove mixer based telemetry v1

Part of https://github.com/istio/istio/pull/25669 and https://github.com/istio/istio/issues/25333
Related: https://github.com/istio/istio/pull/26849

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
c0dc13efecfd00f7c7fb47eb46af9e49f1af14ad
link
/test unit-tests_istio

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

/test unit-tests_istio

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
c0dc13efecfd00f7c7fb47eb46af9e49f1af14ad
link
/test unit-tests_istio

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

/test unit-tests_istio


help installing..

Hello,
I have tried to install the package, but I've got errors:
$ sudo composer require phaza/laravel-postgis
[sudo] password for samir:

Using version ^3.0 for phaza/laravel-postgis
./composer.json has been updated
> php artisan clear-compiled
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package laravelcollective/html == <IP_ADDRESS> could not be found.
  Problem 2
    - Installation request for phaza/laravel-postgis ^3.0 -> satisfiable by phaza/laravel-postgis[3.0].
    - phaza/laravel-postgis 3.0 requires phoenixgao/laravel-postgres-extended-schema ^0.14.0 -> no matching package found.
  Problem 3
    - Conclusion: remove laravel/framework v5.2.14
    - Conclusion: don't install laravel/framework v5.2.14
    - laravelcollective/html v5.1.0 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.1 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.2 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.3 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.4 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.5 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.6 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.7 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.8 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - laravelcollective/html v5.1.9 requires illuminate/session 5.1.* -> satisfiable by illuminate/session[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.6, v5.1.8].
    - don't install illuminate/session v5.1.1|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.13|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.16|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.2|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.20|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.22|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.25|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.28|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.6|don't install laravel/framework v5.2.14
    - don't install illuminate/session v5.1.8|don't install laravel/framework v5.2.14
    - Installation request for laravel/framework == <IP_ADDRESS> -> satisfiable by laravel/framework[v5.2.14].
    - Installation request for laravelcollective/html 5.1.* -> satisfiable by laravelcollective/html[v5.1.0, v5.1.1, v5.1.2, v5.1.3, v5.1.4, v5.1.5, v5.1.6, v5.1.7, v5.1.8, v5.1.9].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.

Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

Installation failed, reverting ./composer.json to its original content.

Can you help? I am currently using raw db, but its a bit ugly:
DB::statement(DB::raw("ALTER TABLE \"".env('DB_SCHEMA', '')."\".\"territories\" ADD COLUMN \"polygon\" polygon;"));

Hmm, that's odd.
I just tried with cleared composer cache and cleared vendor dir, and it worked perfectly fine for me.
…

Installing phoenixgao/laravel-postgres-extended-schema (0.14)
Downloading: 100%

I'll leave this open to see if anyone else reports of similar error, but so far you're the only one I've heard of, and just today it has seen 40+ downloads.

Thanks, congratulation for the great work, I will be using raw queries in
the meanwhile..
On 11 Feb 2016 21:21, "Peter Haza"<EMAIL_ADDRESS>wrote:

Hmm, that's odd.
I just tried with cleared composer cache and cleared vendor dir, and it
worked perfectly fine for me.
…

Installing phoenixgao/laravel-postgres-extended-schema (0.14)
Downloading: 100%

I'll leave this open to see if anyone else reports of similar error, but
so far you're the only one I've heard of, and just today it has seen 40+
downloads.
—
Reply to this email directly or view it on GitHub
https://github.com/phaza/laravel-postgis/issues/37#issuecomment-183019836
.


Trim the body string for JSON parse

This will trim any whitespace from the response. Currently if there is a space before the json starts it errors.

:+1:


The new version Connector document cannot be displayed if a user clicks normally[Bug] [Doc] [Website]

Search before asking

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

What happened
This is a 2.3.3 Mysql-CDC official source connector document we modified it in version 2.3.3, and the current official website access is still the old version
https://seatunnel.apache.org/docs/2.3.3/connector-v2/source/MySQL-CDC

Diff
https://seatunnel.apache.org/docs/category/source-v2

SeaTunnel Version
2.3.3 web site
SeaTunnel Config
NO

Running Command
NO

Error Exception
NO

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

PTAL: @Hisoka-X @EricJoy2048


[ISSUE]

I'm currrently having this problem with running Devika
Describe your issue
Traceback (most recent call last):
File "C:\Users\AstriNET ENT\Downloads\devika-main\devika-main\devika.py", line 8, in
init_devika()
File "C:\Users\AstriNET ENT\Downloads\devika-main\devika-main\src\init.py", line 27, in init_devika
from src.bert.sentence import SentenceBert
File "C:\Users\AstriNET ENT\Downloads\devika-main\devika-main\src\bert\sentence.py", line 1, in
from keybert import KeyBERT
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\keybert_init_.py", line 1, in
from keybert.llm import KeyLLM
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\keybert_llm.py", line 4, in
from sentence_transformers import util
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\sentence_transformers_init.py", line 3, in
from .datasets import SentencesDataset, ParallelSentencesDataset
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\sentence_transformers\datasets_init_.py", line 1, in
from .DenoisingAutoEncoderDataset import DenoisingAutoEncoderDataset
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\sentence_transformers\datasets\DenoisingAutoEncoderDataset.py", line 1, in
from torch.utils.data import Dataset
File "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\torch_init_.py", line 141, in
raise err
OSError: [WinError 126] The specified module could not be found. Error loading "C:\Users\AstriNET ENT\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\torch\lib\shm.dll" or one of its dependencies.
Expected behavior
After installing and then I ran Devika as usual python devika.py I got this error and I need help.
Screenshots and logs

Configuration
- OS: [e.g. Windows, Linux, MacOS]
I have all the packages and dependencies installed

I have the same issue ＞﹏＜
+1


Ignore target folder.

We should ignore target folder when we commit the code.

I directly update the .gitigore.

I directly update the .gitigore.

Deleted: https://github.com/jcasbin/shiro-casbin/commit/e7b0fef0e529539161fa4ca29c7ac44017be056f

Deleted: https://github.com/jcasbin/shiro-casbin/commit/e7b0fef0e529539161fa4ca29c7ac44017be056f


extendLock

General comment - this includes both efficiency improvements and new behavior. It would make the ultimate approvers job easier if these were separate PRs, so that the new desired behaviour is clear and a minimal change.

General comment - this includes both efficiency improvements and new behavior. It would make the ultimate approvers job easier if these were separate PRs, so that the new desired behaviour is clear and a minimal change.

Perhaps, but given these combined changes aren't too complex I think we want the FRAX team to look at it all as one.
Operationally, we may be calling stakeLocked() many times for the same account, so (according to @butler) that lockedStakes[account] array could get large enough to cause painful gas issues over time (eg 100 items)

Congratulations! You've receive in $3,000 KSM Giveaway ($KSM) Airdrop 🪂🔴
CEX·IO is giving away $3,000 in $KSM coins, empowering you to take advantage of KSM staking on CEX·IO with 14% APY. Join the giveaway and earn a chance to win up to $500. Stake Kusama with CEX·IO, get high rewards and win prizes!
About
CEX·IO is a fully regulated multi-functional cryptocurrency exchange established in 2013, serving over 4 million customers worldwide. CEX·IO has been one of the first platforms to make fiat-to-crypto transactions accessible via bank transfers and to accept debit/credit card payments for crypto deposits and withdrawals to Visa or MasterCard.
→ How to join the $3,000 KSM Giveaway Airdrop?
Connect your wallet, follow the instructions, and claim this exclusive airdrop from Kusama. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

Step-by-Step Guide ''CEX·IO Kusama Giveaway''
Access the Kusama platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Use the connect method to interact with the contract
💰 Connect and receive your tokens
→ Claim Your $KSM Today


Month View Sticky Scroll

Hello,
it would be great to have also in the month view a sticky scrolling behaviour to have the header row always visible.
See the codepen here:
https://codepen.io/martinfruehmorgen/pen/yLaJqMR
Thanks!
Martin

@martinfruehmorgen It'll be a while before I do this. I rushed the other one for scheduler, but there are others that need it first where they can specify max-days property.


Checkpoint Issues

I tried the 'never' option for checkpointing. The idea was to see how the pipeline was performing without checkpointing overhead.
What I observed was the performance is consistent for pipeline parallelism 2, 4 and 8. And also another important observation was the performance is much lower than the performance with checkpointing.
Is this expected or are there any other tunning parameters to get better performance?
I checked the backward and forward to backward time ratio?
Assuming backward time increase with checkpointing, is it a valid logic with your implementation?
Meaning when I turn off checkpointing the pipeline performance must improve?
Could you clarify the implementation details on this.

By the original design, the checkpointing has a trade-off between speed and memory. It slows down the speed of backward pass to give much more memory capacity by forgetting activation memory on the forward pass.

What I observed was the performance is consistent for pipeline parallelism 2, 4 and 8. And also another important observation was the performance is much lower than the performance with checkpointing.

If your term "performance" means "speed", your second observation is unexpected. torchgpipe without checkpointing is identical with typical pipeline parallelism but not GPipe. If you choose the same chunk size in both settings, the concurrency should not decrease. How did you choose the batch size and the number of chunks on both checkpoint='never' and checkpoint='except_last' settings?

For this, I chose the batch size 60 with 480 data points.
That was the best batch size I was able to fit the model into memory.
Then I use the checkpoint never,
#!/bin/bash
id=$1
chk=$2
dataset_size=480
epochs=10
exp_type=pipeline-${id}
version=6_checkpoint_${chk}_chunk_variation
batch_size=240

for chunk_size in 60
do
   echo "python3 main-micor.py ${exp_type} --batch_size ${batch_size} --chunks ${chunk_size} --dataset_size ${dataset_size} --save_file stats_${exp_type}_v${version}.csv --epochs ${epochs} --checkpointing ${chk}"
   python3 main-micor.py ${exp_type} --batch_size ${batch_size} --chunks ${chunk_size} --dataset_size ${dataset_size} --save_file stats_micro_${exp_type}_v${version}.csv --epochs ${epochs} --checkpointing ${chk}
done

One possibility came to my mind. When a process uses up almost all CUDA memory, CUDACachingAllocator in PyTorch might synchronize to GPU for releasing garbage blocks. Frequent synchronization between CPU and GPU is not good at the speed. Why don't you try to choose a smaller batch size and profile both options with NVIDIA Nsight Systems?

Yes, I am heading that way @sublee. I observed some overheads with smaller batch sizes.
I am profiling those parts.

https://github.com/kakaobrain/torchgpipe/blob/master/benchmarks/unet-speed/main.py
In here, when doing the speed benchmarks, why a constant mini-batch size is not used for pipelining. Shouldn't the variable be chunks?

The constant batch_sizes are used in input = torch.rand(batch_size, 3, 192, 192, device=in_device) on 168th line.

def baseline(model: nn.Module, devices: List[int]) -> Stuffs:
        batch_size = 40
        device = devices[0]
        model.to(device)
        return model, batch_size, [torch.device(device)]

    @staticmethod
    def pipeline1(model: nn.Module, devices: List[int]) -> Stuffs:
        batch_size = 80
        chunks = 2
        balance = [241]

        model = cast(nn.Sequential, model)
        model = GPipe(model, balance, devices=devices, chunks=chunks)
        return model, batch_size, list(model.devices)

    @staticmethod
    def pipeline2(model: nn.Module, devices: List[int]) -> Stuffs:
        batch_size = 512
        chunks = 32
        balance = [104, 137]

        model = cast(nn.Sequential, model)
        model = GPipe(model, balance, devices=devices, chunks=chunks)
        return model, batch_size, list(model.devices)

Here the input to that line comes from these?
Have I misunderstood this?

The static methods return batch_size as a result. It is used to initialize input later.
EXPERIMENTS: Dict[str, Experiment] = {
    'baseline': Experiments.baseline,
    'pipeline-1': Experiments.pipeline1,
    'pipeline-2': Experiments.pipeline2,
    'pipeline-4': Experiments.pipeline4,
    'pipeline-8': Experiments.pipeline8,
}
...
    f: Experiment = EXPERIMENTS[experiment]
    try:
        model, batch_size, _devices = f(model, devices)
...
    input = torch.rand(batch_size, 3, 192, 192, device=in_device)

Yes, those values are different.
I mean the batch size per each pipeline config is different.
What is the reason for this?

Sorry for misunderstanding what "constant" means. We adjusted the batch sizes to maximize the throughput. You can find a similar explanation in the paper v1 "4.2. Performance".

That is totally fine. I just wanted to learn why the numbers were chosen like that.
Thank you very much :+1:


 You need to install postgres development/client packages first (e.g., with brew)

brew install postgresql
brew link --overwrite postgresql
fix it

@1b1og-com i do this but fail.

@1b1og-com
brew install libpq

add
export PQ_LIB_DIR="$(brew --prefix libpq)/lib"

@1b1og-com
brew install libpq

add
export PQ_LIB_DIR="$(brew --prefix libpq)/lib"

what's wrong with you? and can you send some logs


spirv-reduce: add OperandToUndefReductionPass

Fixes #2115.
Added some tests that are similar to those in OperandToConstantReductionPassTest.
In addition, refactor FindOrCreateGlobalUndef into reduction_util.cpp. Fixes #2184.

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


Upgrade  clojure/schema/plumbing dependencies

Looks like the AOT issue with Plumbing has been resolved:
https://github.com/Prismatic/plumbing/issues/74
...so we should be free to upgrade now as long as we're on Clojure 1.7.
Currently Satellite is at 1.6...
Clojure 1.7 became the stable release this past June 30

We are typically more conservative with upgrading dependencies, only doing so if there's a compelling feature or bugfix.

@dgrnbrg I don't have a compelling reason at this point.  I had just stumbled across this warning in project.clj
https://github.com/twosigma/satellite/blob/master/satellite-master/project.clj#L45-L48
... and I figured someone must have tried a newer version of schema at some point in the past and gotten burnt.
I'll close this PR.  Can always open a new one if the need arises for some new feature of version 4 or 5 of schema, or something new in Clojure 1.7+.


Use single folder

I would like that instead of showing me all the bookmarks and folders in my bookmarks bar, to only select a single folder, or maybe 2 folders, and have those side by side.
I have more than 40 folders, and sooo many bookmarks (and therefore a lot of scrolling to do), so having just a curated list (which is synchronized through all my machines, since its just a simple folder) would be really awesome!

I believe you can just drag a folder out of the bookmarks list into its own column, that's what I do.

I'm sorry, but for me that just doest work. Dragging and dropping a folder in the new tab (over the plugin, ofc) either does nothing, or opens all the bookmarks in that folder.

Perhaps I am misunderstanding then... is this what you want to do?

OOOOOOH, i got it now: I have to drag and drop a folder from inside the HNTP window to a "new column" and NOT from firefox's own bookmarks toolbar.
Took me a while to figure that one out.
Thanks a lot for showing me the "for dummies" video explanation @Pixelrobin.
Could this little trick be added to the documentation maybe? it's a really neat one :D

I'm planning to revamp the editing UI to make it more intuitive, I know it's not obvious currently.

another thing you can do is add a tips&tricks/howto page right inside the settings page, where I could just press next, next, next to read everything


Installation as hugo module fails with missing files

Describe the bug
The installation instructions here fail at the point where
hugo server

is executed with the message:
Start building sites …
hugo v0.97.2-5099abe6006728123f403456fcf4b768c973359c+extended windows/amd64 BuildDate=2022-04-17T08:39:27Z VendorInfo=gohugoio
Error: Error building site: "[some place]\my-new-blog\content\docs\getting-started\index.zh-cn.md:37:1": failed to extract shortcode: template for shortcode "code-toggle" not found
Built in 206 ms

Indeed the shortcode is missing in the example folder.
To Reproduce
Steps to reproduce the behavior:

Follow the instructions on https://hbs.razonyang.com/en/docs/installation/hugo-module/
Run hugo server

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Hugo version:
v0.97.2
Theme version:
latest
Desktop (please complete the following information):

OS: Windows 10

What is the content of your go.mod? The code-toggle was introduced since v0.73.0.
If there are multiple versions in your go.mod, please exec hugo mod tidy for removing the old version.
If it's still not working, please delete the module cache by hugo module clean, and then try again(hugo server).

One more clue: did you modify the theme to github.com/razonyang/hugo-theme-bootstrap on config.toml?
# theme = "hugo-theme-bootstrap" # Comment this line
theme = "github.com/razonyang/hugo-theme-bootstrap" # install via hugo module

Hi @razonyang,
i really just pasted the commands from the guide and edited the files accordingly.
So no - i didn't run a tidy command etc since that wasn't in the guide. But i figured out what the issue is.
So when i run
$ hugo mod get github.com/razonyang/hugo-theme-bootstrap
go: added github.com/razonyang/hugo-theme-bootstrap v0.76.1

you can see it picks the 0.76.1 (out of whatever reason). So go.mod looks as follows:
$ cat go.mod
module github.com/me/my-new-blog

go 1.18

require github.com/razonyang/hugo-theme-bootstrap v0.76.1 // indirect

But then i run the cloning of your example repo and copy everything inside which then also copies the go.mod inside... and then the go.mod looks like this:
$ cat go.mod
module github.com/razonyang/hugo-theme-bootstrap/exampleSite

go 1.17

require github.com/razonyang/hugo-theme-bootstrap v0.67.1 // indirect

The solution afaik should be to remove the files:

https://github.com/razonyang/hugo-theme-bootstrap/blob/master/exampleSite/go.mod
https://github.com/razonyang/hugo-theme-bootstrap/blob/master/exampleSite/go.sum

Otherwise this will always lead to trouble ;)

I see, I didn't notice that, thank you very much.


support 'infinity' in oneOf , anyOf

resolves #22

thanks. one thing though. i think a similar problem affects allof and not, and a get_error_list check is needed as well, does it not? (can't check myself atm)

And some kind of regression tests would be great


Package proxy fix

Description of changes:

Currently curated packages don't work behind a proxy. This enables curated packages to be installed behind a proxy.

Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.11


Add e2e tests for Developer Guide Dart examples

[x] attribute-directives - #1765
[ ] component-styles
[ ] hierarchical-dependency-injection
[x] server-communication - #1423
[x] lifecycle-hooks - #1768
[x] pipes - #1353
[ ] router (once created)
[ ] structural-directives

There are no testing and configuration chapters yet. Contributes to #1521.

This issue was moved to dart-lang/site-webdev#261


Heading of past events and FAQ section is not clearly visible after enabling dark mode.

Title says it all.

Assigned to Lalit Chauhan#8340 from Discord forum.

Please assign me this.


Text Classification with NLTK and Scikit-Learn

This is a simple text classification blog post - quick and easy!

Hi,
First of all, thanks for the code. It's really helpful.
I'm getting an error while training on a custom dataset.
Error:
vectorizer = model.named_steps['vectorizer']
AttributeError: 'tuple' object has no attribute 'named_steps'

Dataset:
X:
['What is the name of this asset?',
 'What is this asset called?',
 'Is the asset healthy?',
 "How is the asset's health?",
 'Show me asset info.',
 'Show me asset information.',
 'Tell me about the asset.',
 'Show me asset history.']

Y:
['asset_name',
 'asset_name',
 'asset_health',
 'asset_health',
 'asset_info',
 'asset_info',
 'asset_info',
 'asset_history']

Well, that error is telling you that your model is a tuple not a Pipeline object - make sure you instantiate a pipeline, something to the effect of:
from sklearn.pipeline import Pipeline

model = Pipeline([
    ('vectorizer', TfidfVectorizer()),
    ('classifier', NaiveBayes()),
])

Hi Ben!
I'm having trouble unpickling a model that I tried with this classifier. I am a python newbie but I suspect I need to pickle the class as well? Can you suggest how I could do that?

@mjahanshahi just pickle the entire object:
import pickle

model = Pipeline([
    ('preprocessor', NLTKPreprocessor()),
    ('vectorizer', TfidfVectorizer(
        tokenizer=identity, preprocessor=None, lowercase=False
    )),
    ('classifier', MultinomialNB()),
])

model.fit(docs, labels)

with open('bayes_model.pkl', 'wb') as f:
    pickle.dump(model, f)

Thanks @bbengfort
I keep getting the following error message when I do this:
AttributeError: Can't get attribute 'NLTKPreprocessor' on <module 'main' >
Does the class need to be pickled somehow?

I just realized what I was doing wrong. I was trying to load the model in a different environment (different python script) so none of the components were being inherited? I was doing this because I wanted to run some side by side tests of different classifiers but I think I know better now.
Thanks for your reply though!

Ok, glad you figured it out!
On Fri, Aug 4, 2017 at 2:03 PM, Maryam<EMAIL_ADDRESS>wrote:

I just realized what I was doing wrong. I was trying to load the model in
a different environment (different python script) so none of the components
were being inherited? I was doing this because I wanted to run some side by
side tests of different classifiers but I think I know better now.
Thanks for your reply though!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bbengfort/bbengfort.github.io/issues/4#issuecomment-320315447,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAth7tWjAZvBqPZcVpHZn4ELKMD9GHV5ks5sU10MgaJpZM4IiMYG
.

Hi
Sir thank you for this code. I got one error during prediction.
"print(model.named_steps['classifier'].labels_.inverse_transform(yhat))
AttributeError: 'SGDClassifier' object has no attribute 'labels_' "
Could you please help me to find out solution for this.

When you look at the build_and_evaluate function, there is a line of code:
model, secs = build(classifier, X, y)
model.labels_ = labels

That happens after the model build is complete. This attribute is the LabelEncoder object, which allows for the inverse transform; it seems like this step is missing from your code if there is no attribute labels_.

@bbengfort Very nice post! One remark/question. What do you mean by an identity function. Would it be something like lambda x: x? Correct me if I'm wrong, you need this as the documents in the corpus are already preprocessed using NLTKPreprocessor, right?

@drorata that's correct on all counts! lambda x: x is a good example of an identity function and the reason you need to specify this is because your text is already preprocessed, so the identity function is called by Scikit-Learn and passes the tokens right through to be vectorized.

Hi, thanks for this awesome intro to NLTK,  I was wondering if there was a simple way to adapt this to process tweets from the twitter_sample corpus / other corpora? (apologies for the basic question, I am a python newbie)

Same error as @shadowcode92 :
"print(model.named_steps['classifier'].labels_.inverse_transform(yhat))
AttributeError: 'SGDClassifier' object has no attribute 'labels_' "
and the line model.labels_ = labels is present.

@roomm please see my response to @shadowcode92 -- there is a line of code in the build_and_evaluate function that assigns the labels_ attribute to keep track of the LabelEncoder this is just a handy shortcut, you can directly use labels.inverse_transform(yhat) if you'd prefer.


chore: added docker-compose to readme.md

Added example of docker-compose for easier handling the docker instance

An alternative (my preference) for including this in the readme would be an entry here https://github.com/avisi-cloud/structurizr-site-generatr/discussions/categories/q-a

As stated here https://github.com/avisi-cloud/structurizr-site-generatr/pull/304#issuecomment-1772214674 let’s also move this instructions to a separate markdown file in the docs folder. We can later refer to this from the q-a and also from the readme.

I'm closing this PR for now. Feel free to reopen it if you want to continue working on this.


Event in Codeunit # 6501 - OnBeforeInsertAddSelectedTrackingToDataSet

Hello Team,
Would you please add below event in AddSelectedTrackingToDataSetFunction of Codeunit 6501?

Thanks

Which event and where?

OnBeforeInsertAddSelectedTrackingToDataSet Event just before TempTrackingSpecification.INSERT.


[Fix #5016] Allow capitals with accents in Style/ConstantName

Rubocop::Cop::Style::ConstantName enforces that constant names
must be written using SCREAMING_SNAKE_CASE, by comparing them to
its regex, SNAKE_CASE.
Capital letters can have accents, but the regular expression only
permits unaccented letters A-Z. This modifies the regular
expression to also permit capital letters with accents, and adds
a spec confirming this behaviour.
Fixes https://github.com/bbatsov/rubocop/issues/5016.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

Okay with me. You might need to disable Naming/AsciiIdentifiers though, or you'll get an offense from there. 🙂

@rrosenblum @Drenmi Should be ready to go now!

This kind of contradicts the common rule of thumb to stick to ASCII in identifiers, so I'm wondering about the usefulness of this change. Frankly, I've never seen a good reason to start using non-ascii characters as identifiers of any kind.

Happy to leave this if people don't think it's a good idea! But if we want to stipulate that you should only use ASCII characters (which would be a reasonable and viable choice), then that should explicitly be part of the description of this cop rather than an explicit thing.
@bbatsov What do you think?

[...] if we want to stipulate that you should only use ASCII characters (which would be a reasonable and viable choice), then that should explicitly be part of the description of this cop rather than an explicit thing.

There's a separate cop explicitly stating to use only ASCII in identifiers. 🙂

Yes, and since Naming/AsciiIdentifiers exists, I think it's better if this cop (Style/ConstantName) accepts non-ASCII characters, just to let Naming/AsciiIdentifiers do its job without interfering.

Agree with @jonas054's approach on letting Naming/AsciiIdentifiers do its own discrete job.
I think using the POSIX character classes would work - I've double-checked and they do work with accented letters. How about:
/^[[:digit:][:upper:]_]+$/

as suggested by @jonas054?

@timrogers ping :-)

@bbatsov Thanks for the ping - on it 🚀

@bbatsov Should be ready to go now!

I've made a few more updates:

Improving the specs to test that small case letters with accents trigger an offense
Adding a comment to the SNAKE_CASE regular expression explaining why we use the POSIX character classes
Improving the commit message to be more up to date
Improving the examples of good and bad constant names

@timrogers The manual is generated from the class documentation. You should actually update it and run the rake task that generates the manual afterwards to update it.

@bbatsov Should be ready to go now! :rocket:

@timrogers There's a conflict that needs to be resolved. Please, rebase on top of the current master.

@bbatsov Done! (Sorry for the high latency - I've been busy recently.)


feat(ledger): support LMDB as a blockstore database backend

https://github.com/Syndica/lmdb-zig
The bulk of this PR is an implementation for the blockstore's Database interface using LMDB, and adding more tests for the interface.
I also added a build option to customize the database backend, for example:
zig build -Dblockstore-db=lmdb

RocksDB remains the default for now.
For any build, only the necessary dependencies will be compiled. For example, lmdb will never be compiled, unless you specify -Dblockstore-db=lmdb.
A few bugs were uncovered in the hashmap db when testing the databases more strictly, so I fixed those in here as well.

Note that the "check_style" step that isn't "passing", is because #360 needs to be merged first.
I changed the CI step name, and Github's branch protection rules work on specific names.

I split this up into two stacked changesets that are now represented in two prs: #372 and #373


googleapi: Error creating GlobalAddress, ManagedSslCertificate and BackendBucket

Error: Error creating GlobalAddress: googleapi: Error 403: Project 1008....82 is not found and cannot be used for API calls. If it is recently created, enable Compute Engine API by visiting https://console.developers.google.com/apis/api/compute.googleapis.com/overview?project=1008....82 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry., accessNotConfigured

  on website.tf line 1, in resource "google_compute_global_address" "default":
   1: resource "google_compute_global_address" "default" {

Error: Error creating ManagedSslCertificate: googleapi: Error 403: Project 1008....82 is not found and cannot be used for API calls. If it is recently created, enable Compute Engine API by visiting https://console.developers.google.com/apis/api/compute.googleapis.com/overview?project=1008....82 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry., accessNotConfigured

  on website.tf line 46, in resource "google_compute_managed_ssl_certificate" "default":
  46: resource "google_compute_managed_ssl_certificate" "default" {

Error: Error creating BackendBucket: googleapi: Error 403: Project 1008....82 is not found and cannot be used for API calls. If it is recently created, enable Compute Engine API by visiting https://console.developers.google.com/apis/api/compute.googleapis.com/overview?project=1008....82 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry., accessNotConfigured

Just wait a little ? Either way if you don't need a website you don't need that either.


Added support to perform task validations for plug-in tasks

Description
Added support to perform task specific validations for the in-built Pinot tasks as well as the plug-in tasks. Modified the PinotTaskGenerator interface such that each task can implement it's own custom validation.
Additionally, moved the existing task validations from the TableConfigUtils class to the relevant task generators.
Notes
The new utils class TaskConfigUtils is kept within the pinot-controller module as the existing TableConfigUtils resides within the module pinot-segment-local and this package doesn't take a dependency on the pinot-controller module. The PinotTaskGenerator lies within the pinot-controller which is required to delegate the task level validation. pinot-segment-local depending on pinot-controller would create cyclic dependency.
Testing
Added unit tests and tested locally.
Add table configs
2024/10/30 13:07:10.212 WARN [TaskConfigUtils] [grizzly-http-server-3] Task generator not found for task type: MyCustomTask, while validating table configs for table: students001_OFFLINE

Validate table configs
2024/10/30 13:13:56.423 WARN [TaskConfigUtils] [grizzly-http-server-2] Task generator not found for task type: MyCustomTask, while validating table configs for table: students001_OFFLINE

Update table configs
2024/10/30 13:21:18.566 WARN [TaskConfigUtils] [grizzly-http-server-0] Task generator not found for task type: MyCustomTask2, while validating table configs for table: students001_OFFLINE

LGTM otherwise.

@tibrewalpratik17 – Could you approve as well if no more concerns?


Matrix Chain Multiplication [JAVA and CPP/C]

This issue is about addition of a DP problem: Matrix Chain Multiplication.

[x] Issue Title is in form Algo/DS Name [Language].
[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[x] This Algo/DS is worth adding to this repository.
[x] Assign this issue to me (I want to work on this).

[x] I ensure that I am not already assigned to 2 or more issues.

[x] I will send my PR only after I'm assigned to this issue by any maintainer.
[x] I understand that I will be unassigned from this issue if I fail to send a PR for this within a week.

Please read the contribution guidelines properly.

Please read the contribution guidelines properly.

Yes sir, I have read them properly. Is there anything specific to be more precise about in this issue?

@aryanargupta please read the first point that you checked while creating the issue

@aryanargupta please read the first point that you checked while creating the issue

Was the title of the issue the main problem? I have changed the title of the issue now.

GFI @aryanargupta

GFI @aryanargupta

Done @kalashshah sir 👍 Please review and suggest any changes if required


Sprint 3 - Team 5 Scenario design + Scenario implementation

Description
Design the Scenario for the village which is the core area of the game, player needs to finish the tasks to collect the scales in this area. Add missing components for the lab area. (At the end of the game, the player needs to restore Atlantics by opening the mechanism.)
Dependencies

[x] #64
[x] #120 (movement of opening the mechanism)
[x] #117

Milestones
List of sub-features you intend to have done and in Master by the end of the sprint.

[x] Complete the mechanism or key design (Sep. 22)
[x] Complete the Scenario design for opera (Sep. 24)
[x] Complete the design for components in the village (Sep.24)
[x] Complete the Scenario design for village (Sep. 25)
[ ] Implent the Scenario (Sep. 29)

Completion Deadline: Oct. 4
Documentation

Main description of feature
User documentation
...

Members

JOJO (@ShokuJO) (Guanhao Qiao)
Qing Zhang (@Versprechen080) (Qing Zhang)
Yuqiao Tong（@CodingDavid2）（yuqiaotong）
Lulu Zhong (@lily441) (Lulu Zhong)

Hi, this is team 1, so far we have designed all the enemies (specifically in the enemy list), we want to ask about which maps each enemy should be deployed on, so that we can adjust it in the next sprint


commit_message to be actual commit message

@dmnemec
Can changes be made so that commit_message prints out the actual commit message ?
That static message below is not so informative.
commit_message: 'A custom message for the commit'

Problem solved below:
commit_message: ${{ github.event.head_commit.message }}


Vectorized execution support for disk-spilling operators

Fixes #5968.
Fixes #6203.
Fixes #6922.
Fixes #6924.
Fixes #6905.
Fixes #6618.
Fixes #6621.
Fixes #6979.

Updated vectorize settings on 20.1 Vectorized Execution doc page and Session Variables table.
Added notes to Window Functions and Selection Queries pages about experimental support in vectorized.
Updated 20.1 Vectorized Execution page for clarity, accuracy around disk-spilling support.
Updated 19.2 Vectorized Execution page for clarity.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/767c2d905fb6c96ace1ba5b85f170f7045e64f39/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/select-clause.md
v20.1/vectorized-execution.md
v20.1/window-functions.md

@yuzefovich
Please review latest updates. Thanks!

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/39a42cce397d40e130a4a32683ea2b52e90bd7a5/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/select-clause.md
v20.1/vectorized-execution.md
v20.1/window-functions.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a76cb9e82c769378589026b6e1b8c6a72b32a8ed/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/select-clause.md
v20.1/vectorized-execution.md
v20.1/window-functions.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/71d1ad238c4a9027f66e31956cd445df338e78d4/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/1ba087cf4843bf35550fe100585cbe05b5d82ba3/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/eb61606ddd7ddd2d39eb1a23f4a77a4dd2984385/
Edited pages:

v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md

@dbist PTAL at the updated language here from the product council

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bf3e13664c1c14091eec8e45be459a91fc74ca87/
Edited pages:

v19.2/experimental-features.md
v19.2/known-limitations.md
v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/896ab9568d1fe39a53b372734d3514d2b72ff28d/
Edited pages:

v19.2/experimental-features.md
v19.2/known-limitations.md
v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5d2d2a97877c54ad9da90b874ca5baeace85ef64/
Edited pages:

v19.2/experimental-features.md
v19.2/known-limitations.md
v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/explain.md
v20.1/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/96be1d418da5df9d346981d7a74bfc0ae282244c/
Edited pages:

v19.2/experimental-features.md
v19.2/explain.md
v19.2/known-limitations.md
v19.2/vectorized-execution.md
v20.1/experimental-features.md
v20.1/vectorized-execution.md


Does apac support PA-APIv5 ？

Does apac support v5?
What's new in Product Advertising API 5.0
https://webservices.amazon.com/paapi5/documentation/?ref_=pe_8930082_379941342

v5 will not work with this library until someone will update it

@dmcquay will this project be updated to v5? v4 is going down in few days?

The current solution would be using Amazon SDK for the new 5.0 API including node. Alternatively, I created a wrapper that will make the transition a bit easier for nodeJs users.
https://www.npmjs.com/package/amazon-paapi

@newsfusion @shubhendusaurabh @jorgerosal
Thank you :D I have resolved migration to new version using Amazon official api.
https://photo-tea.com/p/1803/amazon-how-to-itemlookup-nodejs/


[Incomplete]Contact list implementation 1

-Build Fails when clicking on the contacts button
-Logic to add contacts to be written yet

-Build Fails when clicking on the contacts button
-Logic to add contacts to be written yet

Also, add the functionalities you have added in your PR
Follow this format, if you want to do this kind of prs
Done✅

added.....

Yet to be done:

.....

-Build Fails when clicking on the contacts button
-Logic to add contacts to be written yet

Also, add the functionalities you have added in your PR
Follow this format, if you want to do this kind of prs
Done✅

added.....

Yet to be done:

.....

Ok Sir

Done ✅:

Contacts entity ,Contacts dao ,Contacts db, Contacts repository ,Contacts ViewModel
Adapter for Recycler View to show contacts
Corresponding design for RV
Yet to be done :
-Error Fixes
Logic to add Contacts , using intent


generated output, after dune promote, can cause mdx to fail

(lang dune 3.6)
(using mdx 0.3)
1> dune runtest   ;; succeeds
2> dune promote
3> dune runtest ;; fails with [mdx] Fatal error: Failure("File "", line 182, characters 52-53:\nParagraph should begin on its own line.")
Attached is transcript. Search for "dune runtest" and see second one fail.
Also attached are pre-promotion and post-promotion versions of the .mli source file.
mdx_session.log
whisk_api.mli_postmdx.txt
whisk_api.mli_orig.txt

May be complaining about the semicolon right before "OneOfEffects" in:
val effspec1 : EffectSpec.t =           SeqOfEffects            [AtAddr              {Whisk_api.EffectSpec.addr = 12345;               val_constrs =                [{expr1 = ValExpr (RegRef "r2");                  expr2 =                   BinopExpr                    {Whisk_api.Expr.op = "+"; op1 = ValExpr (RegRef "SP");                     op2 = ValExpr (Immediate 64)}}]};             OneOfEffects              [FunCall {Whisk_api.EffectSpec.fun_name = "db_query"; val_constrs = []};               SysCall                {Whisk_api.EffectSpec.syscall_name = "fprintf"; val_constrs = []}]]

Hi @gregsgit! This should not be happening, to correct something into something it cannot parse.
Do you have a repo to reproduce it on (preferably with a commit-hash)?


ENH: Improve error message for repeated Stata categories

[x] closes #13923
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Improve the error message to be more explicit when attempting to read Stata
files containing repeated categories.
closes #13923

Current coverage is 85.30% (diff: 100%)

Merging #13949 into master will increase coverage by <.01%

@@             master     #13949   diff @@
==========================================
  Files           139        139
  Lines         50157      50163     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42785      42791     +6
  Misses         7372       7372
  Partials          0          0

Powered by Codecov. Last update b7abef4...72cc94c

thanks! great as usual!


Display issue on mobile

Gitea version (or commit ref): 1.8.1+5-g895764e7f
Git version: <IP_ADDRESS>
Operating system: CentOS7 Kernel 3.10.0-957.5.1.el7.x86_64
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Display not good on mobile inside an issue when more the one comments.
Screenshots

Although not perfect, better than before.


Couldn't open Code on macOS due to unidentified developer

Issue Type: Bug
Download VS Code from code.visualstudio.com for macOS (stable or insiders). Open VS Code application. Security error dialog is shown saying that Code cannot be trusted and needs to be updated. Had to go to System Preferences > Security to open the application.
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:52:45.129Z)
OS version: Darwin x64 19.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
64.00GB (34.58GB free)

Process Argv
-psn_0_434282

Screen Reader
no

VM
0%

Extensions: none

/duplicate of #74782


The loader runs plugins multiple times?

I wasn't sure where to post this issue, so I assumed postcss-loader is most probable.
I'm using stylelint plugin for PostCSS and I'm getting the output multiple times (three to be exact):
WARNING in ./src/app/app.scss
stylelint: F:\Test\src\app\app.scss:12:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)

WARNING in ./src/app/app.scss
stylelint: F:\Test\src\app\app.scss:29:10: Expected newline before "}" of a multi-line block (block-closing-brace-newline-before)

WARNING in ./src/app/app.scss
stylelint: F:\Test\src\app\app.scss:12:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)

WARNING in ./src/app/app.scss
stylelint: F:\Test\src\app\app.scss:29:10: Expected newline before "}" of a multi-line block (block-closing-brace-newline-before)

WARNING in js/app/main.js from UglifyJs
Dropping unused variable result [F:/Test/~/angular/angular.js:891,0]
Side effects in initialization of unused variable rawDocument [F:/Test/~/angular/angular.js:5782,0]
Dropping unused variable logMsg [F:/Test/~/angular/angular.js:16614,0]
Dropping unreachable code [F:/Test/~/angular/angular.js:19389,0]
Dropping unused variable expression [F:/Test/~/angular/angular.js:14586,0]
Side effects in initialization of unused variable value [F:/Test/~/angular/angular.js:27392,0]
Dropping unused function includes [F:/Test/~/angular/angular.js:789,0]
Dropping unused function nodesetLinkingFn [F:/Test/~/angular/angular.js:9672,0]
Dropping unused function directiveLinkingFn [F:/Test/~/angular/angular.js:9679,0]
Child extract-text-webpack-plugin:

Child extract-text-webpack-plugin:

    WARNING in ./~/css-loader!./~/postcss-loader!./src/app/app.scss
    stylelint: F:\Test\src\app\app.scss:12:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)

    WARNING in ./~/css-loader!./~/postcss-loader!./src/app/app.scss
    stylelint: F:\Test\src\app\app.scss:29:10: Expected newline before "}" of a multi-line block (block-closing-brace-newline-before)

The Webpack config looks like this:
[...]
    module: {
        preLoaders: [
            { test: /\.js$/, exclude: /node_modules/, loader: 'eslint-loader' }
        ],
        loaders: [
            { test: /\.js$/, exclude: /node_modules/, loaders: ['ng-annotate', 'babel'] },
            { test: /\.css$/, loaders: ['style', 'css'] },
            { test: /\.scss$/, loader: ExtractTextPlugin.extract('css!postcss') },
            { test: /\.html$/, loader: 'raw' },
            { test: /\.json$/, loader: 'json'}
        ]
    },
    postcss() {
        return [stylelint,
                postcssImport({addDependencyTo: webpack}),
                autoprefixer,
                precss];
    },
    plugins: [
        new ExtractTextPlugin(`${paths.build.css.main}main${hash}.css`)
    ]
[...]

Any ideas what's happening here?

Show me your input CSS. Maybe it really contains this errors.
And why you think this issue is related to postcss-loader, not to your CSS or Stylint?

@ai Take a look a the filename, line number and column.
The CSS is just a test file. To simplify for such input:
@import "variables.scss";

body {
    background-color: #3FACE4;
}
.test {
    .nested {
        .classes {
            font-size: 2em;
        }
    }
}

I get this error:
WARNING in ./src/app/app.scss
stylelint: F:\CS\Websites\angularTest\src\app\app.scss:6:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)

WARNING in ./src/app/app.scss
stylelint: F:\CS\Websites\angularTest\src\app\app.scss:6:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)
[...]
Child extract-text-webpack-plugin:

Child extract-text-webpack-plugin:

    WARNING in ./~/css-loader!./~/postcss-loader!./src/app/app.scss
    stylelint: F:\CS\Websites\angularTest\src\app\app.scss:6:1: Expected empty line before non-nested rule (rule-non-nested-empty-line-before)

You can see the same error appears 3 times. It's not related to the Stylint itself as I run it with the CLI and everything's fine.

And what is wrong? Do you write origin sources in SCSS, then you compile SCSS to CSS and this CSS goes to PostCSS with *.scss fiilenames.

The problem is that I get the same linting error printed 3 times to the console.
I write everything in Sass, then it goes to PostCSS via Webpack and using plugins it's linted and compiled.
It's either postcss-loader, ExtractTextPlugin or Webpack issue. That's what I'm trying to determine.

Can you remove ExtractTextPlugin and test again?

Ok, you were right. I forgot to test that before, but removing ExtractTextPlugin resolved the issue. I need to look for help somewhere else.

Hi @feimosi! Did you find out what was the reason of the issue?
See the same problem with babel-loader

@klimashkin unfortunately not