ValueError from the regions module with the updated deps

ValueError                                Traceback (most recent call last)
Cell In[6], line 7
4 r500 = Quantity(samp['MCXC_R500'].values, 'Mpc')
5 name = samp['LoVoCCSID'].apply(lambda x: 'LoVoCCS-' + str(x)).values
----> 7 srcs = ClusterSample(ra, dec, z, name, r500=r500, clean_obs=True, clean_obs_reg='r500', clean_obs_threshold=0.9,
8                      use_peak=False, load_fits=True)
File /mnt/ufs18/home-218/turne540/code/XGA/xga/samples/extended.py:110, in ClusterSample.init(self, ra, dec, redshift, name, r200, r500, r2500, richness, richness_err, wl_mass, wl_mass_err, custom_region_radius, use_peak, peak_lo_en, peak_hi_en, back_inn_rad_factor, back_out_rad_factor, cosmology, load_fits, clean_obs, clean_obs_reg, clean_obs_threshold, no_prog_bar, psf_corr, peak_find_method)
107 from xga.sas import evselect_image, eexpmap, emosaic
109 # Using the super defines BaseSources and stores them in the self._sources dictionary
--> 110 super().init(ra, dec, redshift, name, cosmology, load_products=True, load_fits=False,
111                  no_prog_bar=no_prog_bar)
113 # This part is super useful - it is much quicker to use the base sources to generate all
114 #  necessary ratemaps, as we can do it in parallel for the entire sample, rather than one at a time as
115 #  might be necessary for peak finding in the cluster init.
116 evselect_image(self, peak_lo_en, peak_hi_en)
File /mnt/ufs18/home-218/turne540/code/XGA/xga/samples/base.py:80, in BaseSample.init(self, ra, dec, redshift, name, cosmology, load_products, load_fits, no_prog_bar)
75     z = None
77 try:
78     # We declare the source object, making sure to tell it that its part of a sample
79     #  using in_sample=True
---> 80     temp = BaseSource(r, d, z, n, cosmology, load_products, load_fits, True)
81     n = temp.name
82     self._sources[n] = temp
File /mnt/ufs18/home-218/turne540/code/XGA/xga/sources/base.py:238, in BaseSource.init(self, ra, dec, redshift, name, cosmology, load_products, load_fits, in_sample)
236     self._lum_dist = None
237     self._ang_diam_dist = None
--> 238 self._initial_regions, self._initial_region_matches = self._load_regions(region_dict)
240 # This is a queue for products to be generated for this source, will be a numpy array in practise.
241 # Items in the same row will all be generated in parallel, whereas items in the same column will
242 # be combined into a command stack and run in order.
243 self.queue = None
File /mnt/ufs18/home-218/turne540/code/XGA/xga/sources/base.py:1316, in BaseSource._load_regions(self, reg_paths)
1314 for obs_id in reg_paths:
1315     if reg_paths[obs_id] is not None:
-> 1316         ds9_regs = Regions.read(reg_paths[obs_id], format='ds9').regions
1317         # Grab all images for the ObsID, instruments across an ObsID have the same WCS (other than in cases
1318         #  where they were generated with different resolutions).
1319         #  TODO see issue #908, figure out how to support different resolutions of image
1320         try:
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/core/regions.py:175, in Regions.read(cls, filename, format, cache, **kwargs)
136 @classmethod
137 def read(cls, filename, format=None, cache=False, **kwargs):
138     """
139     Read and parse a region file and return as a Regions object.
140
(...)
173         A ~regions.Regions object containing the file contents.
174     """
--> 175     return RegionsRegistry.read(filename, cls, format=format,
176                                 cache=cache, **kwargs)
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/core/registry.py:78, in RegionsRegistry.read(cls, filename, classobj, format, **kwargs)
73     msg = (f'No reader defined for format "{format}" and class '
74            f'"{classobj.name}".\n'
75            f'{cls._get_format_table_str(classobj)}')
76     raise IORegistryError(msg) from None
---> 78 return reader(filename, **kwargs)
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/io/ds9/read.py:46, in _read_ds9(filename, cache)
44 with get_readable_fileobj(filename, cache=cache) as fh:
45     region_string = fh.read()
---> 46     return _parse_ds9(region_string)
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/io/ds9/read.py:71, in parse_ds9(region_str)
69 regions = []
70 for region_data in region_data:
---> 71     region = make_region(region_data)
72     if region is not None:  # skip region if error during parsing
73         regions.extend(region)
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/io/ds9/read.py:667, in _make_region(region_data)
664     shape_params.append(region_data.raw_meta.get('text', ''))
665     meta.pop('text', None)
--> 667 region = ds9_shape_to_region[region_type]shape
669 region.meta = RegionMeta(meta)
670 region.visual = RegionVisual(visual)
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/shapes/ellipse.py:368, in EllipseSkyRegion.init(self, center, width, height, angle, meta, visual)
366 self.center = center
367 self.width = width
--> 368 self.height = height
369 self.angle = angle
370 self.meta = meta or RegionMeta()
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/core/attributes.py:41, in RegionAttribute.set(self, instance, value)
40 def set(self, instance, value):
---> 41     self._validate(value)
42     instance.dict[self.name] = value
File ~/software/anaconda3/envs/xga_dev/lib/python3.12/site-packages/regions/core/attributes.py:150, in PositiveScalarAngle._validate(self, value)
147         raise ValueError(f'{self.name!r} must have angular units')
149     if not value > 0:
--> 150         raise ValueError(f'{self.name!r} must be strictly positive')
151 else:
152     raise ValueError(f'{self.name!r} must be a strictly positive '
153                      'scalar angle')
ValueError: 'height' must be strictly positive
This may be what was happening with all the divide by zero warnings I was getting with some of the LoVoCCS clusters' regions, except no regions is doing an explicit check. I don't expect that this is an XGA or regions problem, rather that there is something funky in one of the XCS region files - though I may add a cleaning step in XGA to avoid this in the future.

Have identified the ObsIDs for which this is an issue for LoVoCCS:
'height' must be strictly positive
0674560201
'height' must be strictly positive
0843890301
'height' must be strictly positive
0401520801

Right, having just read in one of these region files<PHONE_NUMBER>) imagine my surprise when there are no negative width or height values - so something more complicated is going on unfortunately. I will read in the region file into regions line by line to see which one is causing the problem

Hang on nope, I was hasty - there is a region with zero height.
I could add a check for negative or zero valued width/heights in region files - but that would involve more file reading. I think instead I'll add a try-except around the region-read-in so I can at least tell the user which ObsID's region files are causing the problem.


Add university of Quilmes

@AgustLuj I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host unq.com.ar not found: 2(SERVFAIL) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


[Refactor] Add verbose log for stream load request

What type of PR is this：

[ ] bugfix
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Add verbose log for stream load, easy to debug
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

@mergify backport branch-2.1

run starrocks_admit_test


Bug Fix: Allow deferring file inclusion to tsconfig.json

Currently you must specify a path to your typescript project at the end of the typedoc command. Many projects already define include in their tsconfig.json files, describing which files they wish included on TS compilation. Fixing this bug allows those configs to be used rather then specifying them again from the command-line.
Example include in tsconfig.json taken from 0x.js:
"include": [
        "./src/**/*",
        "./test/**/*",
        "../../node_modules/types-bn/index.d.ts",
        "../../node_modules/types-ethereumjs-util/index.d.ts",
        "../../node_modules/web3-typescript-typings/index.d.ts",
        "../../node_modules/chai-typescript-typings/index.d.ts",
        "../../node_modules/chai-as-promised-typescript-typings/index.d.ts"
    ]

I believe this is just a bug fix since the relevant conditional could never be false given that files is initialized to an empty array.

Hello @fabioberger. I'm likely going to merge #610 which should fix this issue (see the change to arguments.ts. Thanks for your work though. Feel free to create an issue or reach out on gitter so that we can make sure we aren't duplicating effort.

Awesome, #610 should fix my issue as well. Np, hope to contribute more soon!


Error when restoring Database for a new container

Hi,
sometimes we got an error when creating our docker container with a backup. The following error is thrown, but not always. We have 4 agents on one virtual machine and nearly every agent throw this error once since they exist.
Using navcontainerhelper: <IP_ADDRESS>

C:\ProgramData\NavContainerHelper\bakFolder\Agent-03\sandbox1523955839825de\database.bak
Removing Database mydatabase from localhost\SQLEXPRESS
Warning, exception when running: ALTER DATABASE [mydatabase] SET OFFLINE WITH ROLLBACK IMMEDIATE
Waiting... - retrying
Warning, exception when running: ALTER DATABASE [mydatabase] SET OFFLINE WITH ROLLBACK IMMEDIATE
Waiting...... - retrying
Error when running: ALTER DATABASE [mydatabase] SET OFFLINE WITH ROLLBACK IMMEDIATE
ALTER DATABASE is not permitted while a database is in the Restoring state.
ALTER DATABASE statement failed.
at Invoke-SqlCmdWithRetry, C:\Run\HelperFunctions.ps1: line 424
at Remove-NavDatabase, C:\Run\HelperFunctions.ps1: line 311
at Restore, : line 19
at , : line 57

Executed by:

try {
    Restore-DatabasesInBCContainer -containerName $containerName -bakFolder $bakName
}
catch {
    $restoreDb = $false
}

Could you share the command used to create the container (before restoring databases)?
and the output of that?

Actually, we just test if the container still exist. If it exists, restore the database, if not set up a new container with a backup.
write-host $testPath
$restoreDb = Test-BCContainer -containerName $containerName
if ($restoreDb -and $bakFile -and -not $NoBackUp) {
    try {
        Restore-DatabasesInBCContainer -containerName $containerName -bakFolder $bakName
    }
    catch {
        $restoreDb = $false
    }
} else {
// Some more source code, but in general:
if(-not $NotSQLBak){
    New-BCContainer -accept_eula `
                    -containerName $containerName `
                    -auth NAVUserPassword `
                    -Credential $databaseCred `
                    -imageName $imageName `
                    -additionalParameters $additionalParameters `
                    -licenseFile $licensefile `
                    -shortcuts $shortcuts `
                    -doNotUseRuntimePackages `
                    -assignPremiumPlan `
                    -updateHosts `
                    -useTraefik:$false `
                    -EnableTaskScheduler:$false `
                    -usebestcontaineros `
                    -bakFile $bakFile
  } else {
    New-BCContainer -accept_eula `
                    -containerName $containerName `
                    -auth NAVUserPassword `
                    -Credential $databaseCred `
                    -imageName $imageName `
                    -additionalParameters $additionalParameters `
                    -licenseFile $licensefile `
                    -shortcuts $shortcuts `
                    -assignPremiumPlan `
		    -includeTestToolkit `
                    -updateHosts `
                    -EnableTaskScheduler:$false `
                    -useTraefik:$false
  }
}

Btw. I just turned off 2 of our 4 agents and so far the error didn't occur.

Does the containername depend on the agent name?

It looks like one agent is trying to restore a database in a container where another agent is restoring...

Does the containername depend on the agent name?

Yes, the container's names are the same as the agent names. All are different:
Agent: Agent-01, Docker: Agent-01
Agent: Agent-02, Docker: Agent-02
Agent: Agent-03, Docker: Agent-03
Agent: Agent-04, Docker: Agent-04
When starting the pipeline a variable containerName: $(Agent.Name) is set. This variable is used during the docker creation with $containerName = $ENV:CONTAINERNAME.
Just checked the "Name" in the agent's capabilities and all are different.
Why do you think so and any ideas how to figure that out?

Because of this:
Error when running: ALTER DATABASE [mydatabase] SET OFFLINE WITH ROLLBACK IMMEDIATE
ALTER DATABASE is not permitted while a database is in the Restoring state.

BTW - I would change the script so that if restoring the DB fails - then it should revert to recreate the container. The container can be contaminated or be of a different version or whatever.

Because of this:
Error when running: ALTER DATABASE [mydatabase] SET OFFLINE WITH ROLLBACK IMMEDIATE
ALTER DATABASE is not permitted while a database is in the Restoring state.

Well, obvious. Still don't know how, how this situation can occur.

BTW - I would change the script so that if restoring the DB fails - then it should revert to recreate the container. The container can be contaminated or be of a different version or whatever.

At the moment, all containers have the same BC version, so that wouldn't be a problem right now, but maybe in the future. I will give it a try in the next days and will let you know.

Hey Freddy,
you were right. I changed the script, so that it only restores if the version matches. If not, create a new container.
We are restoring the container because it saves us time.
Nevertheless, thanks for your help!


failureRedirect not working with multiple forms

Describe the bug
I'm having an issue receiving authentication errors while using multiple form elements. Here's a demo of my login page, the component has a login form and create an account form.
https://user-images.githubusercontent.com/83249002/178132898-56c3c88f-b63e-4493-8c37-0b2ad792f06c.mov
FORM 1
If I login with a random email and password, the correct error is presented:

If I login with correct email and password, a successful redirect happens.
====
FORM 2
If I login with correct email, confirmation email and valid password, a successful redirect happens.
If I login with mismatching email and confirmation email (this throws exception inside strategy), nothing happens, but an error should be returned. If I refresh the page, the error appears. After checking the networking tab, it seems after the POST there's no subsequent GET request to fetch the error from the Loader.
https://user-images.githubusercontent.com/83249002/178133000-1f21d660-80fe-46c3-9d22-3ad9fbb45a38.mov
Any idea what's happening here?
Your Example Website or App
https://github.com/sergiodxa/remix-auth
Steps to Reproduce the Bug or Issue
See > Describe The Bug
Platform

OS: macOS
Browser: Brave
Version: OS 12.4

Additional context
LoaderFunction of /login.tsx

Login Page component of /login.tsx

ActionFunction of /login.tsx

As you can see I even made multiple strategies but that didn't fix the issue.

The issue was the  href values in each component. Setting the href to "#" instead of "signin" resolved the issue.


At modifier performance issue

From the benchmark from https://github.com/thanos-community/promql-engine/issues/105#issuecomment-1299517569 we could see that at modifier test case is even > 50% slower than the old engine.
We should investigate and see how we could improve the performance.

Closed by #132


samv7: add support for ADC conversion triggering with PWM

Summary
This commit enhances ADC and PWM drivers with option to trigger ADC conversion with events generated by PWM comparison units.
The comparison value is currently set from the configuration, future enhcement could be an additoon of IOCTL command that would handle this kind of stuff.
Impact
SAMv7 MCU only.
Testing
Tested on a custom board with SAMv7 MCU.

Thanks Alan @acassis, good catch.


pen v7 breaks my project

project designed with pen v6.
The v7 update causes the triangles to not be drawn.
with v6

with v7

@David-Orangemoon

Ill look into it

@penta-quark-neutro could you send the project file?

here the project.
Several others rely on this to function so it is important.
Motor 3D.zip


[PDI-19869] - TextFileOutput throws transformation runtime error if output field columns are changed

:x: Build failed in 1h 40m 12s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3633, Failures: 0, Skipped: 12    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 52m 48s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 4093, Failures: 0, Skipped: 13    Test Results

:information_source:  This is an automatic message


Avro-C 1.7.6

Type: Missing
Summary:
Avro-C 1.7.6
Details:
ClearlyDefined license is Apache-2.0 AND BSD-3-Clause AND MIT
Resolution:
Apache-2.0 AND BSD-3-Clause AND MIT
Affected definitions:

Avro-C 1.7.6

@capfei I think the Declared License is Apache-2.0. Other licenses as discovered. Thoughts?
https://github.com/apache/avro/blob/release-1.7.6/LICENSE.txt


Cannot change select element value with enzyme

React: 16.0.0
Enzyme: 3.2.0
Jest: 21.2.1
The following test is not working. Also I wrote down the list (in comments) of not working code that I already tried.
    import React from 'react';
    import { configure, mount, shallow, render } from 'enzyme';
    import Adapter from 'enzyme-adapter-react-16';

    configure({ adapter: new Adapter() });

    /* ... */

    it('enter valid data for connection; enable proxy; check if button is disabled', () => {
        const component = getComponent(mount); // mount from enzyme
        enterCredentials(component);
        const proxySelect = component.find('select');

        // currently tried these variants:
        proxySelect.simulate('change', { target: { value: true } } );
        // component.find('select').simulate('select', true);
        // component.find('select').simulate('select', 'Yes');
        // component.find('select').simulate('change', { target: { value: true } } );

        // /* Mutating: */
        // proxySelect.node.selectedIndex = 1;
        //
        // Producing this error:
        //
        //      Attempted to access ReactWrapper::node, which was previously a private property on
        //      Enzyme ReactWrapper instances, but is no longer and should not be relied upon.
        //      Consider using the getElement() method instead.
        //
        //  at ReactWrapper.get (node_modules/enzyme/build/ReactWrapper.js:1689:15)

        expect(proxySelect.prop('value')).toBe(true);
        expect(component.find('button').prop('disabled')).toBe(false);
    });

Markup for select element:
    <select>
        <option value="false">No</option>
        <option value="true">Yes</option>
    </select>

Found an issue here: https://github.com/airbnb/enzyme/issues/389
But solutions from that issue isn't working too.
How do you change select's value in tests?

simulate doesn't actually simulate anything; it certainly won't cause any changes in the DOM. If you want to change the value, you have to manually change the value.

Still no solution that works for changing select value ?


Change 'X drains adopted in GRB' to 'X drains adopted in LGRW'

Review Instructions
Task: Substitute "LGROW" for "GRB"

goto home page, look for "N Drains adopted in LGROW".

https://lgrow-staging.herokuapp.com

if found and satisfactory then move issue #98 to Done in waffle.io


Display all assets

I dragged the project files into my session and been playing around with the loadPhotos function. But can't seem to make it load up all the photos from a persons photos library. any help appreciated! whenever I change the "ALAssetsGroupSavedPhotos" to anything else I get no photos!

May be it has no photos in your selected group type?

Did some research... IOS8 uses a new photo framework. PHPhotoLibrary etc... ALAssetsLibrary only works with IOS7 and below. And chance you could update it? I'm struggling to any help appreciated.

Really？ALAssetsLibrary works with iOS8 very fine!
I will consider PHPhotoLibrary later.


eslint 报错

Oops! Something went wrong! :(
ESLint: 8.15.0
ESLint couldn't find the plugin "eslint-plugin-jest".

已解决


Register and Login

Hi,
What route do you get in the email following the registration?
I want to test the login and register but my server is on a Windows so I don't have the SendMail. I figured that you might have to put that "verkey" in a url to get prompted to insert a password but I don't know what route would that be.
Thank you!

It's a little more complicated, but you don't need to worry about that. If you pull the commit I just pushed and register a user, the verification URL will be printed directly to the console/log, prefixed by "VERIFICATION URL:"
The mailing system is definitely the project's weakest link, and the main thing holding back the Dockerfile. Maybe it would be better to use a plain SMTP Haskell-native library? Although that would presumably require storing credentials in environment variables. In the meantime, the latest commit should work as a stopgap for you.

Hi,
Thank you very much for your help! I know it was a basic thing to do, but Haskell is giving me a hard time, not being used with functional programming so sorry for the noobie request.  There is this service called SendGrid which has an API which you can call and they send your mail. Problem is, for the free package you can only send 100 emails per month, but they have a good reputation and rarely end in Spam.
I don't really know at the moment if Haskell has the Service concept, but if it does have it or something similar you could create a Mailer Service with some constants that handle the mail sending, so you don't share your credentials between all files.
Thank you again for your help!

Interesting. It looks like SendGrid even has a Haskell library. I will definitely look into this!
Haskell is weird and so is this project, so feel free to email me if you have any questions about how the code fits together. Things have stalled because I have neither the time nor the funds to commit to an actual site launch, but I'm still around.


Adopt BNF (via llamacpp) and/or Dspy

Feature/Improvement Description
I've been using Localai (as an openai replacement) to serve multiple local models for inferencing and came across the idea of BNF grammars in llamacpp. I recall some talk here about the challenges of getting consistent responses from opensource models.
Adopting BNF grammars to constrain responses (See BNF grammars in llamacpp and other projects), or the new DSPy work from Stanford (You might be able to provide a good working conversation flow/examples from say GPT4, to give the DSPy compiler targets to learn how to use any given open source model the same or a similar type response) might be the solution to reliable responses.
One or both of these approaches would push AGIxt to the next level of usability and reliability, in my opinion.
Proposed Solution
See the BNF conversation here:
https://www.reddit.com/r/LocalLLaMA/comments/16wf3w5/ggml_bnf_grammar_the_key_to_autonomous_agents/
Or:
https://github.com/stanfordnlp/dspy
Acknowledgements

[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for everyone to understand why this feature request is needed in AGiXT.

So I'm gonna try to take a crack at this myself It would seem. I'm adding "grammar_json_functions" and "grammar" attribute to the "ChatCompletion" class in agixt/Models.py. Next I will need to check the custom provider options to see how ChatCompletions are used and providing the grammar attributes and make some if they done.. Ultimately since the grammar would impact the responses coming back, I think I will add custom prompts for my custom provider so that I can have some sort of embedded grammar instruction in the prompts.

  class ChatCompletions(BaseModel):
      model: str = "gpt-3.5-turbo"
      messages: List[dict] = None
      temperature: Optional[float] = 0.9
      top_p: Optional[float] = 1.0
      functions: Optional[List[dict]] = None
      function_call: Optional[str] = None
      n: Optional[int] = 1
      stream: Optional[bool] = False
      stop: Optional[List[str]] = None
      max_tokens: Optional[int] = 8192
      presence_penalty: Optional[float] = 0.0
      frequency_penalty: Optional[float] = 0.0
      logit_bias: Optional[Dict[str, float]] = None
      user: Optional[str] = None
      system_message: Optional[str] = ""
      grammar_json_functions: Optional[List[dict]] = None
      grammar: Optional[str] = None
Example BNF grammar call to llama.cpp api:
curl http://localhost:8080/v1/chat/completions -H "Content-Type: application/json" -d '{
"model": "gpt-4",
"messages": [{"role": "user", "content": "Do you like apples?"}],
"grammar": "root ::= ("yes" | "no")"
}'
So i'm thinking a prompt say like "Break into steps.txt" might look like:
{context}
Primary objective: {user_input}
{introduction}
Instructions:

Instead of solving the problem, we want to break it down into smaller problems that are easier to solve.
Be very descriptive for every step that will be required and give those steps in a numbered list.
Consider that each step needs to be worded as an instruction for another AI to follow that is not as smart as you.

{"grammar": "numbered_list ::= (task_number task_description)+; task_number ::= INTEGER; task_description ::= /[^"]+/;"}
Any thoughts/advice?

Hey @kratosok ! I actually haven't used the grammars. I found an approach the worked before those started being implemented and stuck with it. I am open to moving to grammars since they're trending towards being the standard for that, but I don't currently have any data formatting issues with AGiXT that I am aware of.


fix: schedule events marked as reside-camcho [PLAT-4954]

:tickets: TICKET-000

Upstream PRs

Downstream PRs

Changes

Notes for Reviewers

Recordings/Screenshots

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 87.805%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
87

Relevant Lines:
96

💛  - Coveralls


Test 130 Failure for 2017_09_01 on i386 arch

Description:
Test 130 fails on i386 arch like this (x86_64 passed it)
130: ======================================================================
130: FAIL: testEnumerateStereoisomersRandomSamplingShouldBeDeterministicAndPortable (main.TestCase)
130: ----------------------------------------------------------------------
130: Traceback (most recent call last):
130:   File "UnitTestMol3D.py", line 218, in testEnumerateStereoisomersRandomSamplingShouldBeDeterministicAndPortable
130:     self.assertEqual(smiles, expected)
130: AssertionError: Items in the first set but not the second:
130: 'C/C(F)=C/C@@H/C=C(\Br)C@@HI'
130: 'C/C(F)=C\C@H/C=C(\Br)C@@HI'
130: Items in the second set but not the first:
130: 'C/C(F)=C/C@@H/C=C(/Br)C@@HI'
130: 'C/C(F)=C/C@H/C=C(\Br)C@HI'

Ah the fun of still needing to support 32bit systems.
I suspect that both this and #1662 are caused by the same problem. Should be an easy fix.

And I didn't even start filing the test failures for ARM...

Yeah, I can imagine that those will be numerous and painful. Is there a way I could get access to those test machines so that I can help deal with that?

Package maintainers have SSH access to some resources that could help: https://fedoraproject.org/wiki/Test_Machine_Resources_For_Package_Maintainers
but I need to ask whether I am allowed to put a (trusted) upstream in the ssh allowed keys. If not, we can still work it out something different.
However, I think most issues are related the the ones I opened; in fact, until the 2016.09.04  tests passed on all architectures (and boy, we have quite some of them... https://koji.fedoraproject.org/koji/buildinfo?buildID=861476).
BTW, if you think I can collect more/different info just let me know where to look and I will do.
Thanks for you help


scoop couldn't find manifest for 'hugo'.

At first, I have executed 'scoop update' successfully!
However, when I execute 'scoop install hugo', I get the following error:
scoop couldn't find manifest for 'hugo'

I also found this problem.
PS C:\Users\Ilooker> scoop search python36
Results from other known buckets...
(add them using 'scoop bucket add ')
'versions' bucket:
bucket/python36
PS C:\Users\Ilooker> scoop install python36
Couldn't find manifest for 'python36'.

What is result of scoop bucket list command?

What is result of scoop bucket list command?

main

Then you should see hugo manifest.
Try scoop info hugo and you should see something like this:

Then you should see hugo manifest.
Try scoop info hugo and you should see something like this:

If no, then i would suggest to do scoop bucket rm main; scoop bucket add main

What should I do next? Just waiting？

The issue was resolved, thanks for your advice.

I suspect there was some network problem before so git clone cannot be done on previous update.

Yep, it must be network problem. Since I had interrupted the first command "scoop update"


Dropdown stops working if rendered in form and tab is used.

I'm using some dropdown components in a form, without using a custom template. I have realized that if I change from input to input using tab, the dropdown components break throwing the following error:
Uncaught TypeError: Cannot read property 'label' of undefined.
I have notice that this.getSelectedItem() in the dropdown component is returning undefined, and I guess it's related with the code happening onMouseDown not getting executed because the input doesn't get the focus by clicking on it.

This is very interesting, looking at it! Thanks @YoYuUm

This should be fixed already!

Hi,
The problem with this was the change to only render the active tab. If you have a component with it's own state, it gets unmounted and destroyed.
My solution is to pass a prop to the Tab component onlyRenderActive that defaults to false (to keep with the same way as 0.14) that controls if it will render all the tabs or only the active.
What do you guys think? I can send this PR

Yeah ... I misunderstood this bug report, perhaps. I interpreted "and tab is used" to mean "tabbing through the form fields with the keyboard". There is a bug, as I described above, but it might be a different / new issue. LMK if you'd like me to file it as a new issue.

Maybe it was me. I read the issue and immediately thought of my case... I'll send a repro in a new issue and fix it.

Thanks @felipeleusin ! It makes a lot of sense to have that prop. In fact, for this kind of component, probably even it should default to false

I also reproduced the bug described by @jeffreywescott
When you have a form with dropdown and you're tabbing through it the value of dropdown gets changed and returns a label-attribute, but not a value-attribute of source item.

Yes please!

I think the issue was that we were spreading the onChange callback to the Input element. Then it was called with the current value. As we want to display the label as selected value, that's the passed value to the input. This way, Dropdown onChange was called with the Input value which was actually the label. It has been solved by adding it's own tab level, removing the tabIndex from Input and finally, extracting the onChange prop instead of passing it down to the Input.


Better handling of invalid local_directory

When copying jobqueue.yaml from one system to another, I missed updating local_directory to point to a valid path. When I tried to run a job, I got this output:
distributed.core - ERROR - addresses should be strings or tuples, got None
Traceback (most recent call last):
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/core.py", line 407, in handle_comm
    result = handler(comm, **msg)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/scheduler.py", line 2146, in remove_worker
    address = self.coerce_address(address)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/scheduler.py", line 4855, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))
TypeError: addresses should be strings or tuples, got None
distributed.core - ERROR - addresses should be strings or tuples, got None
Traceback (most recent call last):
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/core.py", line 407, in handle_comm
    result = handler(comm, **msg)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/scheduler.py", line 2146, in remove_worker
    address = self.coerce_address(address)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/scheduler.py", line 4855, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))
TypeError: addresses should be strings or tuples, got None

and then a hang. For a comparison, when using distributed.LocalCluster with an invalid local_directory, I get an infinite stream of this message:
Process Dask Worker process (from Nanny):
Traceback (most recent call last):
  File "/data/conda/envs/test/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/data/conda/envs/test/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/process.py", line 191, in _run
    target(*args, **kwargs)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/nanny.py", line 675, in _run
    worker = Worker(**worker_kwargs)
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/worker.py", line 503, in __init__
    self._workspace = WorkSpace(os.path.abspath(local_directory))
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/diskutils.py", line 118, in __init__
    self._init_workspace()
  File "/data/conda/envs/test/lib/python3.8/site-packages/distributed/diskutils.py", line 124, in _init_workspace
    os.mkdir(self.base_dir)
FileNotFoundError: [Errno 2] No such file or directory: '/u/test/scratch'

I don't know dask, distributed, and dask_jobqueue well enough to say for sure, but I wonder if there is a way for more information about errors to be propagated back to dask_jobqueue from the remote workers to make problems like this easier to figure out?

Hi,
There is certainly room for improvement in the handling of this error and others! I've not looked into the code, but we can try to use your example to see what goes wrong.
Did you have the correct message in the worker logs, so the logs from the job where the worker was started?

Where could I access the worker logs? Are they just printed to stdout/stderr of the worker and I need to catch them from the job queuing system?

Are they just printed to stdout/stderr of the worker and I need to catch them from the job queuing system?

Exactly. There were some discussion to retrieve them from the client at one point IIRC, but I don't know if this was pursued.
You can look at https://jobqueue.dask.org/en/latest/debug.html for other advices.

Just tried to reproduce this one, I think this is a distributed problem rather than dask-jobqueue.
Opposite to @willsALMANJ, I'm able to reproduce it with LocalCluster. Just doing:
from distributed import LocalCluster
cluster = LocalCluster(local_directory='/some/inexistent/path')

Trigger a lot of errors:
distributed.utils - ERROR - addresses should be strings or tuples, got None
Traceback (most recent call last):
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/utils.py", line 656, in log_errors
    yield
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 2214, in remove_worker
    address = self.coerce_address(address)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 4955, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))
TypeError: addresses should be strings or tuples, got None
distributed.core - ERROR - addresses should be strings or tuples, got None
Traceback (most recent call last):
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/core.py", line 513, in handle_comm
    result = await result
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 2214, in remove_worker
    address = self.coerce_address(address)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 4955, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))
TypeError: addresses should be strings or tuples, got None
tornado.application - ERROR - Exception in callback functools.partial(<bound method IOLoop._discard_future_result of <zmq.eventloop.ioloop.ZMQIOLoop object at 0x2b139447a210>>, <Task finished coro=<Nanny._on_exit() done, defined at /softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/nanny.py:440> exception=TypeError('addresses should be strings or tuples, got None')>)
Traceback (most recent call last):
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/tornado/ioloop.py", line 743, in _run_callback
    ret = callback()
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/tornado/ioloop.py", line 767, in _discard_future_result
    future.result()
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/nanny.py", line 443, in _on_exit
    await self.scheduler.unregister(address=self.worker_address)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/core.py", line 861, in send_recv_from_rpc
    result = await send_recv(comm=comm, op=key, **kwargs)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/core.py", line 660, in send_recv
    raise exc.with_traceback(tb)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/core.py", line 513, in handle_comm
    result = await result
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 2214, in remove_worker
    address = self.coerce_address(address)
  File "/softs/rh7/conda-envs/pangeo_202008/lib/python3.7/site-packages/distributed/scheduler.py", line 4955, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))

I'm on Dask and distributed 2.2.0.
cc @TomAugspurger @jrbourbeau you may want to move this one to distributed repo.

Thanks for the ping @guillaumeeb, transferring over to distributed


Ignore bank account for which operations can't be retrieved

Hello,
Kresus worked one time, just after installation, but since then, when I try to sync the data, I obtain this error :
xhr error: Not Found
bank access not found
What does it mean? How can I solve it?
Thanks,

Hi! I think that when you setup an account for the first time but things don't get well (for instance, operations or accounts import get aborted for some reason), data is in an unstable state and then you can't fetch new transactions.
Can you try deleting your bank in the Settings section, and restart the process entirely, please?

I tried to do so but when I start to import the account, I have this error :
"
xhr error: Internal Server Error
when retrieving operations: Error when parsing weboob json: 'content'
"

Ha, I have never seen this before. Any chance you could remove Kresus and all its data (through the DataBrowser app) and try again? If you know how to use weboob/boobank, I'd be curious to know whether your issue reproduces with these too, or if it is specific to kresus.

Ok, I can try to remove Kresus and all it's data. What data should I delete in DataBrowser? (I don't want to delete data that are shared with other app. I don't know weboob/boobank, sorry.

Thanks! Removing all BankAccess / BankAccount / BankOperation / KresusConfig should be enough.

I did it! Here is the error that occur at the first "import".
********* is normally a list of banking operation, similar to the first and last one.
I don't know weboob/boobank. Sorry about that
How can I help?
xhr error: Internal Server Error
when retrieving operations: Error when parsing weboob json: The account type has not been implemented by weboob.
The account type has not been implemented by weboob.
{"bred": [{"account": "0074364.000", "label": "GOOGLE *LiveQos", "raw": "GOOGLE *LiveQos Carte 00099   LE 01/06/15 CB.600633 ORIGINE GBR 10 EUR", "amount": "-1.6", "rdate": "2015-05-31T00:00:00Z", "date": "2015-05-31T00:00:00Z"}, ********* {"account": "0074364.000", "label": "Virement automatique 31012", "raw": "Virement automatique 3012 MR LETRE             ECH0115 REF

Thanks for the input. It seems that weboob can read an account but can't import the operations. In this case, I guess Kresus should just shut up and skip the account. I'll fix this soon.

ok.
Let me know when I should try again. I'll delete the data again.
Thank you for your investment for cozy-cloud and the futur users.
Thatoo

@Thatoo : I've submitted a patch in incoming. Can you uninstall Kresus, and install the incoming version? (that would be by entering https://github.com/bnjbvr/kresus.git@incoming in the installation text field on the config page)

@bnjbvr : done! The first import worked well. I'll let you know how it works for the next sync and see how it evolves.
Thank you,

It seems that it's working except a weird thing.
Synchronisation are working.
By aacount, "Reçu", "Payé and "Economisé" seems correct.
However, "Balance" is really weird. I can't see what it is link to... is it suppose to be "le solde du compte"? If it is, they are all wrong.

It is supposed to be solde du compte. But it is computed live. At the first import, an initial value is put inside the db (calculated based on  imported value of 'solde'+ operations) and the current sold is calculated taking into account all operations. If you have a wrong initial value or missing operations  (accounts are never deleted even when kresus is uninstalled) then the solde can ba wrong.

ok, thank you for this information, but what should I do?
I've just installed Kresus...

wait for @bnjbvr :)

@Thatoo, there are several possible causes for this issue:

there were already operations / accounts before you registered your first account, in which case deleting all Kresus data (again...) would solve it. If you've done this before installing the incoming version of Kresus, it probably is less likely to be the cause of your issue. In case all other solutions don't work, you can retry that!
there could be duplicate operations. You can check them in "duplicate operations" and merge them together. However, after a first import, there shouldn't be any.

So yeah, if 2 doesn't work, I would suggest retrying 1. However, if the difference stays around, I am afraid I won't be able to debug it remotely. I'll try to see whether I can reproduce locally on my account though. If you can find where the differences come from (that is, which operations), maybe I can help, but I understand that's a tedious task... Good luck!

I've done 2, but it doesn't solve the problem. There were few "duplicate" but at the same time, I was not sure they were real "duplicate" because they had different "titles" even though they had the same ammount at the same date...
So 'Ill delete again BankAccess / BankAccount / BankOperation / KresusConfig, uninstall and reinstall Kresus from https://github.com/bnjbvr/kresus.git@incoming and I'll try again.
I'll let you know the result.

I can't delete BankAccess neither BankOperation...

Weird, seems like an issue in the data browser (feel free to open an issue there, maybe?). An other option is to delete the entire tables ("bankoperations" and "bankaccess"), this should work as well.

done, I've deleted all the four tables and reinstall Kresus.
I'll make the first import later and will let you know if it finaly worked.

It's going to be more complicated because I can't install Kresus :
Kresus
Application cassée
Cette application est cassée. Vous pouvez essayer de la réinstaller: réinstallation.
Si l'erreur persiste, vous pouvez nous contacter par mail à contact@@cozycloud.cc ou sur IRC #cozycloud sur irc.freenode.net.
Application already exists : Error: Application already exists at /usr/lib/node_modules/cozy-controller/build/server/controllers/applications.js:74:13 at startApp (/usr/lib/node_modules/cozy-controller/build/server/lib/controller.js:45:12) at Object.module.exports.install (/usr/lib/node_modules/cozy-controller/build/server/lib/controller.js:198:12) at Object.module.exports.install [as handle] (/usr/lib/node_modules/cozy-controller/build/server/controllers/applications.js:71:21) at next_layer (/usr/lib/node_modules/cozy-controller/node_modules/americano/node_modules/express/lib/router/route.js:103:13) at Object.module.exports.check [as handle] (/usr/lib/node_modules/cozy-controller/build/server/middlewares/token.js:31:16) at next_layer (/usr/lib/node_modules/cozy-controller/node_modules/americano/node_modules/express/lib/router/route.js:103:13) at Route.dispatch (/usr/lib/node_modules/cozy-controller/node_modules/americano/node_modules/express/lib/router/route.js:107:5) at /usr/lib/node_modules/cozy-controller/node_modules/americano/node_modules/express/lib/router/index.js:213:24 at param (/usr/lib/node_modules/cozy-controller/node_modules/americano/node_modules/express/lib/router/index.js:303:14)
And if I go back to allmydata, Kresus appear three time in the list.
I'm trying to delete Kresus from "Etats", it says "en cours de suppression" but it never ends...

It's ok, after some update, some delete, uninstall and stuff, I'm back to normal, all Kresus database erased and the three Kresus uninstall.
So, should I try installing https://github.com/bnjbvr/kresus.git@incoming again?

From my side, I'm using incoming and it looks to fix BNP account issue.

I think an old bug is back, the Balance/Receive amount are not ok

HAve you checked "doublons"?

I'll wait for your answer @eravion before reinstalling incoming Kresus.

No doublon.
The difference between amount in BNP website and amount in Kresus is huge.

could this be due to a wrong initial amount?
Did you also remove the bankaccess before reimoprting?
Kresus works like that :

@eravion If the failure has been "long enough" (for Credit Mutuel, that would be 3 months though), I am afraid you may have lost any statement lines in the meanwhile, which may be the cause of the difference. Any chance that you can correlate the difference with any operations? Maybe you can see how many operations in total you have for this month in Kresus, and compare with the number of operations you have in your bank? (and if it's not this month, might it be last month, etc.)
Have you deleted any bank information before installing the incoming version? (shouldn't be necessary) If so, you would need to delete all the data indicated in one of the above comments.

I removed everything and add my account again, now it is clean.
Of course all data were lost, that's the risk when you are beta tester :)
This version look stable and I confirm that the graph are nice.
Thanks for your good job.

ok, so I deleted everything and I'm going to try again as it seems to work for @eravion

Kresus installed. with the incoming but I have again an error at the first import :
xhr error: Internal Server Error
Weboob error: /usr/local/cozy/apps/kresus/weboob/env/local/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
InsecurePlatformWarning
Traceback (most recent call last):
File "py/endpoint.py", line 109, in
content = BankHandler(id, password, website).post(bankuuid)
File "py/endpoint.py", line 66, in post
return self.load_from_connector(name)
File "py/endpoint.py", line 46, in load_from_connector
raise Exception("Unexpected Weboob error: %s" % unicode(e))
Exception: Unexpected Weboob error: 'LoginPage' object has no attribute 'iter_accounts'

I installed again Incoming Kresus and I did again the first import. It seems to work this time. "La Balance" is iqual to "Solde" on the bank website.


volume mounts in Docker for Desktop Kubernetes

Hello,
Does anybody know why i cant mount files from my compose file in Kubernetes but for Docker swarm it works?
i am using the new Kubernetes feature in Docker for Desktop. The mount im trying to make is:
Volumes:

C:\docker\appsettings.json:/app/appsettings.json

i am getting this error:
Stack.compose.docker.com "adapter" is invalid: adapter: Invalid value: "null": conversion to kube entities failed: C:\docker\appsettings.json: only absolute paths can be specified in mount source

@DdannyB this isn't related to a specific Docker docker samples app so I'm closing it. I suggest joining the Docker Community Slack and asking questions there: https://community.docker.com/registrations/groups/4316 or in the Docker Forums: https://forums.docker.com/.


Empty chunk choices

Some OpenAI "compatible" APIs (e.g. https://lepton.ai) return a chunk with an empty list of choices at the end of their responses.
Currently this causes an error with shell-gpt as this case is not handled, which this PR fixes.

This issue should be solved in the latest release by LiteLLM. Given that, I'll pass on this, but thanks for the effort! 🍰


Throw in same method as check may be 3 times slower (and used in hot paths, seems)

May be slow:
https://github.com/dadhi/DryIoc/blob/c2729e5e4d05be1143f95266d97efa986b032581/src/DryIoc/Container.cs#L10611
Seems right:
https://github.com/dotnet-fun/dotnet-system-except/blob/master/src/GenericClassExtensions.cs
https://github.com/dotnet-fun/dotnet-system-except/blob/master/benchmarks/NonNullVariants.cs
.NET Core 3 does not fixes it either. No seems it will be fixed..

Need to benchmark a different approach against the current Thow.It

Too small of thing to worry at the moment


Cannot create TKE in Rancher

Tencent Kubernetes Engine introduced a breaking change in V3 API.
And Tencent has removed V2 API from their service endpoints last month.
However, Rancher TKE kontainer engine is using V2 API.

Reproduced in version v2.4.3

Validated in version v2.4-head off 25th May
Steps:

Create cluster, choose Tencent TKE
Use valid Tencent credentials to proceed

Result:
TKE clusters can be provisioned successfully.

Validated in version master-head off 25th May
Steps:

Create cluster, choose Tencent TKE
Use valid Tencent credentials to proceed

Result:
TKE clusters can be provisioned successfully.


BREAKING CHANGE: @accounts/rest-client now exports the Client class

BREAKING CHANGE: @accounts/rest-client now exports the Client class that expects an option object to be supplied on the constructor.
Added flow to client
Added simple test for options object

@TimMikeladze Are you okay with moving the url config to the RestClient rather then the @accounts/client?
It seems like the @aacounts/client package has a couple more settings to move from it.

Yeah. I've already  done that in my PR in this repo.

Aside from the latest commit of moving the peer dependencies, my branch is ahead of this one. I think this PR can be closed in favor it.


Fix carousel on index.php

Carousel is a little broken. Seems like an issue with the JS, as the thing it's based off of seems to work like a dream...

The issue seems to be caused by the CSS not taking for the transformations. Learned via isolation that the CSS for the site comes from the minimized version in the bootstrap distribution folder, instead of from the css folder.

Fixed. CSS was originally compiled in Less, which Omeka couldn't read. Once compiled into normal CSS and fiddled with, it finally works. Should be copyable, if you change the IDs


bug: Module "xyz" has been externalized for browser compatibility

What version of daisyUI are you using?
latest
Which browsers are you seeing the problem on?
Chrome
Reproduction URL
not available
Describe your issue
I installed daisyui with bun i --dev daisyui@latest into my project running "vite": "^4.0.0".
In the browser logs I get:
 Module "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapConsumer" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapGenerator" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.dirname" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.relative" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.resolve" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.sep" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "url" has been externalized for browser compatibility. Cannot access "url.pathToFileURL" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapConsumer" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapGenerator" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "url" has been externalized for browser compatibility. Cannot access "url.fileURLToPath" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "url" has been externalized for browser compatibility. Cannot access "url.pathToFileURL" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.isAbsolute" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.resolve" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapConsumer" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "source-map-js" has been externalized for browser compatibility. Cannot access "source-map-js.SourceMapGenerator" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "fs" has been externalized for browser compatibility. Cannot access "fs.existsSync" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "fs" has been externalized for browser compatibility. Cannot access "fs.readFileSync" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.dirname" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Show lessModule "path" has been externalized for browser compatibility. Cannot access "path.join" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.

daisyUI is a dev dependency. It doesn't go to your bundle neither needs to loaded to the browser.
It only works at build time to add additional class names to Tailwind CSS.
Do you get the same error if you uninstall daisyUI?

Thank you for the hin. When run the build the warnings are gone but during development they are very annoying.

Do you get the same error if you uninstall daisyUI?
If yes, it's not related to daisyUI.

If I uninstall daisyUI, the error is gone

Can you share a reproduction repo?

I am using a custom composable in vue:
import resolveConfig from 'tailwindcss/resolveConfig'
import tailwindConfig from<EMAIL_ADDRESS>
export function useColors() {
    const fullConfig = resolveConfig(tailwindConfig)

    const tailwindColors = fullConfig.theme.colors

    const extendedColors = {
        ...tailwindColors,
        ...tailwindConfig.theme.extend.colors,
    }

    return extendedColors
}

When I import this, I get the errors. When I don't import it, I don't get it.

Feel free to re-open the repo after providing a reproduction repo.

I think because tailwind.config.js is a Node.js module, which relies on the Node.js environment and specific modules, which are not available in the browser environment, and it only used while building


update docs, template parameters

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

tested PR on local, looks good.


[textanalytics] add multi label classify bespoke method

Part of https://github.com/Azure/azure-sdk-for-python/issues/23528

/azp run python - textanalytics - tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-textanalytics

/azp run python - textanalytics - tests

/azp run python - textanalytics - tests


spelling fix and remove missing link to data

Removed missing link to data and fix spelling.

bump. Resolved conflicts.


ConfigurationNode - Typo issue

Hello,
Typo issues are located here and around:

https://github.com/SpongePowered/Configurate/blob/5695117e1a92dd1c3694053f453e6445d0d7c323/core/src/main/java/org/spongepowered/configurate/ConfigurationNode.java#L437

require()d -> required() (3 occurences of the issue in the file)
Best Regards,
Djaytan

That's not a typo - it's the past tense of the call to the require() method.


Update newsletter schedule

This adds an option for each day of the week to Newsletter Frequency. Let me know if you have any changes! :smile_cat:

Thanks! But we probably need a migration to port over the old settings, too?

Good point! I'll take a look soon. Thanks.
On Tue, Jun 23, 2015 at 11:03 PM, Sacha Greif<EMAIL_ADDRESS>wrote:

Thanks! But we probably need a migration to port over the old settings, too?
Reply to this email directly or view it on GitHub:
https://github.com/TelescopeJS/Telescope/pull/1041#issuecomment-114709621

Nice work @johnthepink. Thanks for implementing this!

@SachaG I added a migration for this. Hope I did it correctly :smile:. Please send any feedback. Thanks!

Looks good! I think I'll wait until the next release to merge this, just so I have some time to test it out. But it's coming :)

@SachaG sounds good. :smile_cat:

Does on() take an array as argument, too?
And have you been testing this in production by the way?

Yeah seems like it does. I'll merge this to devel and test it out on Meta.

@SachaG
Cool, let me know what you find. I haven't tested it in production. Just locally using the getNextJob method and others to make sure it was scheduling properly. Thanks!


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/sirjojo69/ioBroker.solaredge

[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E300] Not found on travis. Please setup travis
[ ] [E816] file tsconfig.json found in repository, but not found in .npmignore

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "solaredge" in latest repository

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Do you need help fixing the bugs?

Do you need help fixing the bugs?


Jenkins: test_no_std and remaining examples

Active the remaining items from #25 in Jenkins once they are fixed.

Jenkins currently doesn't build pull requests. I think we should enforce that only PR's which build successfully may be merged into master. But that needs jenking to build PR's automatically

Remaining examples are integrated into Jenkins and expected to timeout -> this needs fixing anyhow.
test_no_std also working and integrated in Jenkins (can fail without setting the status to failure).
Jenkins now also builds PRs (currently only tested for substrate-test-nodes and substrate-api-client)


Unable to use the Kubernetes deployment file

Hi, I'm running a kubernetes cluster on my local machine using virtual machines (virtualbox). The cluster is working fine and I have created some deployments on the nodes. I've also applied the calico network add-on.
Now I'm trying to deploy netshoot on my cluster. I'm using the following deployment file:
kubectl apply -f https://github.com/nicolaka/netshoot/blob/master/configs/netshoot-calico.yaml
However, I get the error on replica set that service account 'cni-plugin' not found, so I changed the service account to 'calico-node' in the deployment file and tried again. This time the replica was created successfully, however I'm getting an error on the pod that was created by the deployment. The following is the log for the netshoot pod
Events:
  Type     Reason       Age               From               Message
  ----     ------       ----              ----               -------
  Normal   Scheduled    40s               default-scheduler  Successfully assigned kube-system/netshoot-calico-deploy-778d8f88b4-cvgj6 to k8s-head
  Warning  FailedMount  8s (x7 over 40s)  kubelet, k8s-head  MountVolume.SetUp failed for volume "etcd-certs" : secret "calico-etcd-secrets" not found

How can I solve this problem? If I'm using the wrong service account, then which service account should I be using?
Any help will be greatly appreciated. Thanks!

Fixed after:
kubectl apply -f https://docs.projectcalico.org/v3.8/manifests/calico-etcd.yaml
Thanks!


Edit 3rd book

*can ever be

Got it.

*Its

Got it.

*Contract_Cases

That too.

+sets of inputs . This allows you to find bugs in both the code and its
Minor editorial remark: there's an extraneous whitespace after sets of inputs

And this one.  And I checked to see if there were any more (there aren't).
I updated the branch.


Configuration stuck in 'batch job preparing' status

Summary
I'm having trouble with one configuration that stopped running a week ago. This configuration worked for the past 6+ months and now it gets stuck in 'batching job preparing' status. I've tried manually running the configuration at different times, but I can't get it to successfully run anymore. The source report has 3,500 records and 20 filters, which I haven't had a problem with before.
I have not been having trouble with any other Mass Action Scheduler jobs. The only thing that's recently changed with this configuration is that the report's record count increased by ~1000 and we added additional filters (but no cross filters).
Version of Mass Action Scheduler
2.400

Steps To Reproduce

Create a new scheduled configuration
Source is a tabular report with no groupings or cross filters (3,500 records with 20 filters)
Connect to a flow
Schedule a daily run
Tried different batch sizes from 1-50

Expected Behavior
I would expect the configuration to run to completion as it has many times before 3/1/22. I've confirmed I have access to the source report and its folder. I've debugged the flow using the record id used in this configuration to confirm it works succesfully.
Actual Result
Every scheduled or on demand run of this configuration since 3/1/22 has been stuck in 'batch job preparing' status
Source Type

[X] Reports
[ ] List Views
[ ] SOQL Queries

Action Type

[X] Process Builder & Flows
[ ] Workflow Rules
[ ] Email Alerts
[ ] Quick Actions
[ ] Apex

Does your configuration use Named Credentials?
No

I tried creating a new copy of this existing configuration to see if that would change anything and I received the following error in my inbox after trying to run it:
Failed to process batch for class 'dca_mass_action.MA_ReportSourceBatchable' for job id '7073m0000GSweCY'
caused by: reports.InvalidFilterException: [A report can't have more than 20 filters.]
(System Code)
The source report had 20 filters, but I removed one of them and now the configuration is working!


Make IncludeWorkflow share resources with parent. Other cleanup and reorg.

Codecov Report

Merging #89 into master will increase coverage by 0.15%.
The diff coverage is 73.46%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
+ Coverage   79.11%   79.27%   +0.15%
==========================================
  Files          22       20       -2
  Lines        1566     1549      -17
==========================================
- Hits         1239     1228      -11
+ Misses        252      246       -6
  Partials       75       75

Impacted Files
Coverage Δ

daisy/workflow/step.go
91.35% <ø> (-0.6%)
:arrow_down:

daisy/workflow/sources.go
67.64% <100%> (ø)
:arrow_up:

daisy/workflow/resources.go
95% <100%> (+0.06%)
:arrow_up:

daisy/workflow/step_includeworkflow.go
56.75% <60%> (+56.75%)
:arrow_up:

daisy/workflow/step_sub_workflow.go
48.14% <68.42%> (+48.14%)
:arrow_up:

daisy/workflow/workflow.go
82.16% <84.21%> (+1.1%)
:arrow_up:

Continue to review full report at Codecov.

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


Integrating Windows Testing Material

Integrating the Windows specific material into the contributor guide
Carries and Closes #11296
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

yayyyyy! thanks so much!

:heart:
:heart_eyes:

Niiiice :-)

ping @docker/docs-owners

Looks like it needs a rebase.

ping @docker/docs-owners

LGTM

Author LGTM.


Tiered permissions - Managers can promote employees to managers, but not to admins.

Describe the feature you'd like
I would like to open the app up to be less managed by me and stakeholders, while also keeping empyrical permissions set in stone. This could be achieved by:

Adding "Tiered User Permissions" to the settings/features page as a checkbox. This would affect manage users and manage roles and permissions.

When Checked, tiered user permissions assignment would appear in /settings/roles below as a seperate card/body. Name of role and dropdown. 1-10. Looking like a separate set of settings.
E.x. Default administrator role would be assigned "10" and the default guest role would be hardcoded "0"

Using this feature, any role trusted with manage users and manage roles and permissions could affect up to thier tier.
e.x. A Manager could promote another employee(tier 1) to manager(tier 4) or developer(4), and adjust permissions for each, but cannot promote an employee to senior developer(5), Csuite(6), or admin(10). They could invite and onboard a contractor without me having to worry as much that a brain flub by the manager would accidentally give the contractor admin access. Talk about a nightmare.

The beauty of this is everything else could stay the same.  Managers could add another user group like asst. manager(3) if they want. Asset privileges are unchanged for the most sensitive information, as they are set to visible for only specific roles anyway and can ignore tiers.
Describe the benefits this would bring to existing BookStack users
More granular control of user permissions. Allowing us to give trusted individuals more freedom, while also providing admins more security.
Can the goal of this request already be achieved via other means?
Not easily. I could code it if I get desperate enough. I'm more of a Nodejs/Typscript guy though.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
1 to 5 years
Additional context
No response

Thanks for the request @sjacobflaherty. The fundamental need is very similar to  #2713, so I may dedupe this down to that issue, with this referenced as a potential implementation idea.
Looks like a good potential solution if that issue was to be addressed, although I would look to avoid the settings checkbox to enable/disable the feature, as I try to avoid option driven features in BookStack, and instead prefer a forward compatible migrated route of some kind, with instead an added role level control that has to be assigned before user roles can be changed via such a mechanism.


UITableViewController initialization(?)

I don't know if this is possible, but is there a way to show this on top of a UITableViewController? I have a TabBarController that holds around 3 UITableViewControllers, and 2 UICollectionViewControllers. Is there a way to initialize this on any one of them? This is awesome btw.

Nevermind, got it. Just attach the view to UIApplication.sharedApplication().
Thanks!!!

@HackShitUp Do you have an example of how you did this?


Breaking Texture (1.19.2)

Can you make it so the breaking texture follows the hitbox and is not in midair:

Here's how Mojang's Small Amethyst Bud breaks:

So I don't think this can be fixed easily.
If you have a solution to this that does not require too much effort - please let me know.


Team Editor Page

From the list of teams (which only contains my team) I clicked Team Katrina.
You need white space between the buttons at the top.
Invite Contacts makes sense, however, it doesn't work. I tapped one of my contacts to invite it and the checkbox didn't work.
Should we rename the Invite to Team button as "Invite by Email"? Because to me "Invite Contacts" is also inviting them to the team.
When I clicked Invite to Team and entered an email, first and last name, the Invite to Team button was still greyed out.

I am testing on Android, Samsung Galaxy S5

@jmargolisvt How would I get an updated app to test your fixes? I went into Expo and I'm not seeing any changes.

This all passes in the new version, Thanks @jmargolisvt!


Chore: Updated releaserc and travis.yml files to include default main branch

What is the purpose of this pull request? (Check any that's applicable)

[ ] Feature
[ ] Bug fix
[ ] Dependency updates
[ ] Documentation updates
[x] Build related changes
[ ] Changes an existing functionality
[ ] Other, please explain:

Does this introduce a breaking change?

[ ] Yes
[x] No

What changes did you make? (Give an overview)

Updated the travis.yml and releaserc files to specify the default branch name "main"

Removed the tags "chore", "build" and "docs" releaserc file as we don't want to release if we have commit tag message contains one of the tag

Add any screenshots

:tada: This PR is included in version 1.11.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


context traits behave differently to mobile Analytics SDKs

The context doesn't have a trait when a user is signed in. Is it intended or simply a missing feature?
The same issue is documented in the old repo: https://github.com/segmentio/analytics.js/issues/656

@pooyaj Thank you for the response! I'd look into the reference.

@pooyaj I'm sorry to reopen this issue, but I'm very confused about how traits work on the web now. From my testing, it appears that traits show up only when you supply them for each tracking call. But if that's the case, then what is the value of providing traits for the identify call? And what's the value of traits in general? I think I'm missing something but I'm not getting much clarity from the referenced article.


Fix Optional Amount Serialization

This patch fixes the Optional Amount serialization for Amount types and adds tests that ensure that round tripping actually works.
This issue comes up because e.g., {amt: null} and {} are theoretically equivalent types, but our code did not properly handle the {amt: null} serialization output by serde_json.

W.r.t. the changes suggested by @thomaseizinger to the newly introduced test; my preference is to not modify the test code as it is copy/pasted from the earlier test for the bitcoin representation with minimal changes. If someone wants to rewrite both tests that seems like a separate PR.

W.r.t. the changes suggested by @thomaseizinger to the newly introduced test; my preference is to not modify the test code as it is copy/pasted from the earlier test for the bitcoin representation with minimal changes. If someone wants to rewrite both tests that seems like a separate PR.

Summary:
@thomaseizinger utack da21294
@dr-orlovsky utack da21294
@sgeisler  tACK da21294
Probably ok to merge?

I just wanted to merge, but my spidey sense was tingling: we historically we didn't care as much about API breaks because most releases were breaking. But since we want to get away from that we need to pay more attention to API break. While I think this could be merged in a 0.27.0 staging branch it can't in 0.26.1 which we target to release next.
SerdeAmount is a public trait, so adding functions to it is a breaking change.
We need to pay more attention to this on the account that none of the more than three reviewers noticed that.

Personally I'm ok with having only major releases; what I'd like to have is a more releases per year - not once a quarter or twice a year, especially while we will need to get a lot of changes (Taproot, PSBT2, refactoring). And it seems that minor versions may help with that.

Some downstream users want fewer API breaks and I think we should try to respect that. The question is how to achieve it without making others depending on new APIs wait for too long. Personally I've been able to work around that by just replacing the dependency with a git version workspace-wide in the past. If that's not a good option for you (e.g. because you want to release to crates.io) you'll have to find a way to make either @apoelstra or @TheBlueMatt cut releases more often.
Since there doesn't seem to be a threshold-signed-release workflow, there's always a set of people with a key each that can make releases unilaterally, so that set has to be kept small for security reasons. Maybe we could build a CI tool that releases the crate if enough authorized people sign a tag (is that even possible to have multiple sigs on a tag?), but I doubt Andrew trusts CI enough for that :sweat_smile: (except maybe if it ran on his server).

Personally I think it's OK since the breaking API change:

Behind a feature flag (serde)
is fixing a bug that could cause serious issues for users of the library
A trait that a user has no reason to ever implement on their own types.

Much better to have a break cause compilation to fail than to have users compile code that is incorrect.
That said:

We should probably make more judicious use of trait seals so we can expose traits as pub without allowing users of the library to implement them
We could also not touch SerdeAmount and implement these methods in SerdeAmountExt to prevent the break here... not sure that is worth it though.

BTW I think it is also a breaking change to fix the bug in the first place...

@sgeisler  could you say more on "Personally I've been able to work around that by just replacing the dependency with a git version workspace-wide in the past." btw?
I'm currently using patches to do this and it breaks when a release comes out... are you setting the minor version to like 999?

@JeremyRubin I never got rid of the annoyances patching causes, they were always small enough to grudgingly accept them. I typically had branches with all the things merged that I had lined up as PRs for upstream. When the next release came I could typically remove the patch. It gets more annoying the deeper in the dependency tree the changes are though, I'm scared of needing changes to bitcoin_hashes for that reason :grimacing:

I've added a patch which I think strikes a decent compromise.
This leaves SerdeAmount and ser/de for SerdeAmount untouched. However, it introduces a new trait SerdeAmountForOpt which is a required bound just for the broken opt module.
Therefore it is a breaking API change which should only impact users who are already depending on the broken code, and even then, it is only impacting the subset of users who are both depending on the broken code and bothered to implement the SerdeAmount trait on their own type (which btw, they could likely just add a derive macro on if it is their type rather than using the SerdeAmount trait, which is also clearly documented that:
     /// This trait is used only to avoid code duplication and naming collisions
     /// of the different serde serialization crates.

so I think any user implementing that would surely have some pause...
).
I don't think it makes sense to wait for a breaking API update -- unless we adopt a paradigm to do breaking API updates per-bug -- as it will decrease the likelyhood that a user can easily get working code because it will make them likely take a number of other breaking changes they may or may not want to accept.

Agree that this obscure enough (and the existing code is broken) that we can put this in a non-breaking release.
Regarding the general "we should have releases more often", I agree. I think we can probably speed up a bit now that we have some more maintainers. But historically, only one of {me, steven, matt} have been available in any given month which results in a ton of single-ACK PRs building up.
I appreciate that downstream users do not like our pace of breaking changes. I don't think we'll be able to make them happy in the next year or two. The bitcoin space is moving too quickly. There are going to be breaking changes for PSBT2, for bech32m, and for Taproot. There is going to be a slew of breaking changes when we flatten the module hierarchy here and clean up our error types. Between Satoshi and Rust 1.29, there will be an ongoing cadence of breaking changes as we adjust how we compromise on impossible API decisions.

Also +1 for releasing this as a minor release as it's a bug fix IMO.

And yeah we should probably have a top-level private::Sealed trait that we can add to all internal traits so that we can discard changes that are in principle breaking, but not really.


[DO NOT MERGE] Update to Salt 2015.8.1

This update also removes usage of deprecated options, so these states will not work properly on 2015.5. (The run will attempt to succeed with an exit code of 0, but with a warning that reflects the broken state.) Hence, it is important that the the Salt installations in production are updated when these states are deployed.
Blocked on #165.

:umbrella: The latest upstream changes (presumably #229) made this pull request unmergeable. Please resolve the merge conflicts.

I'm working on a rework of this anyways (since Salt 16.03 is now in feature freeze!) to update to 2015.8.7, so I'm going to close this.


Remove Google APIs from dependency and fix tests

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I signed it!

@googlebot I fixed it.


feat: added samples for regional SM

Description
Fixes #
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[ ] I have followed Contributing Guidelines from CONTRIBUTING.MD
[ ] Tests pass:   go test -v ./.. (see Testing)
[ ] Code formatted:   gofmt (see Formatting)
[ ] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[ ] Please merge this PR for me once it is approved

Golang reviewers,
Thanks for reviewing this PR.
The samples are for DRZ compliant new APIs in Secret Manager. Though the samples look like it's a duplicate of the existing ones, there's a tweak in the API resource expectations.
Would be happy to clarify further, internally.

Hey, @grayside, Is there any update regarding the review?


fix bug of  "Failed to start when metadata-report.cycle-report=false is set"

What is the purpose of the change
Issue Number: close #11096
It should be a bug brought by pr #9820
Brief changelog
Add the judgment of the attribute cycle-report.
Currently this attribute is not used, but the documentation suggests that if this attribute is false, metadata will not be checked.  https://dubbo.apache.org/zh/docsv2.7/user/references/metadata/
Verifying this change
The demo of issue #11096 can be started normally when apply this pr.

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #11097 (736104f) into 3.0 (d192c7b) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

:exclamation: Current head 736104f differs from pull request most recent head 759ae0f. Consider uploading reports for the commit 759ae0f to get more accurate results

@@             Coverage Diff              @@
##                3.0   #11097      +/-   ##
============================================
- Coverage     66.01%   65.95%   -0.06%
- Complexity      317      318       +1
============================================
  Files          1240     1240
  Lines         54041    54043       +2
  Branches       8179     8180       +1
============================================
- Hits          35673    35646      -27
- Misses        14508    14534      +26
- Partials       3860     3863       +3

Impacted Files
Coverage Δ

...ache/dubbo/config/utils/ConfigValidationUtils.java
76.94% <0.00%> (-0.45%)
:arrow_down:

.../remoting/api/SingleProtocolConnectionManager.java
61.53% <0.00%> (-23.08%)
:arrow_down:

.../apache/dubbo/rpc/filter/ProfilerServerFilter.java
62.85% <0.00%> (-20.00%)
:arrow_down:

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
79.59% <0.00%> (-12.25%)
:arrow_down:

...bo/rpc/protocol/dubbo/DecodeableRpcInvocation.java
70.19% <0.00%> (-10.58%)
:arrow_down:

...e/dubbo/rpc/protocol/tri/transport/WriteQueue.java
73.80% <0.00%> (-4.77%)
:arrow_down:

.../apache/dubbo/rpc/protocol/injvm/InjvmInvoker.java
63.56% <0.00%> (-3.88%)
:arrow_down:

...he/dubbo/remoting/transport/netty/NettyServer.java
70.17% <0.00%> (-3.51%)
:arrow_down:

.../apache/dubbo/remoting/transport/CodecSupport.java
80.30% <0.00%> (-3.04%)
:arrow_down:

.../dubbo/registry/client/metadata/MetadataUtils.java
69.44% <0.00%> (-2.78%)
:arrow_down:

... and 12 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wxbty Please fix it on the 3.1 branch, 3.0 will only fix security issues.


Beacon stops advertising when reading Mi Flora data

Hi,
I am using a Raspberry Pi to read the data of the Mi Flora and it is working perfectly. I am now also trying to use the Raspberry Pi as an Beacon to monitor the presence of people in my house but I realised that whenever the Mi Flora data is read, my Raspberry Pi stops advertising and I have to run sudo hciconfig hci0 leadv 0 again to be able to see my beacon.
Would it be a behavior of the bluetooth not allowing Beacon advertising and connection to a bluetooth device or is it something that could be fixed?
Ben

Hi,
I don't think, both will work on the same BT device.
Daniel


Add a way to view the login details

Currently there is no way to view the login details other than using 'pick all' which isn't ideal since it prints the passwords to the terminal and it isn't very scalable.
I suggest adding something like:
pick alias -u
# outputs<EMAIL_ADDRESS>
Might even be worth adding a flag to view the entire entry?
Also to clarify, this isn't the same as export PICK_CONFIG='{"verbose":true}' because that prints the password and the entry to the screen every time.

I agree- a way to access a username would be useful and I think https://github.com/bndw/pick/pull/4 will suffice.
Looking ahead, I'm not super thrilled with the interface. I initially had a few more flags, like --token, which today lives in PICK_CONFIG. The current design of the cli/config feels confused and lacking cohesion for me. I'm going to open an issue to track some design changes and would welcome your feedback once I pin a few more things down.


Something went wrongHTTP/1.0 500 Internal Server Error

First this is great work so thank you.
I'm using Postgrest on AWS and I am encoutering the following message intermittently.

Something went wrongHTTP/1.0 500 Internal Server Error
Date: Sat, 24 Dec 2016 05:24:54 GMT
Server: postgrest/<IP_ADDRESS>
Content-Type: text/plain; charset=utf-8
Something went wrong

This happens when I put a wrong query at the end such as "count"
<EMAIL_ADDRESS>After I leave the server running overnight and try to login again, again I encounter the problem.

Current solution is to terminate the Postgrest process and start again.
What could be the issue ?
Thanks
P.S  I'm using the latest release.

Thanks for reporting, I haven't seen this bug before.
Some questions to help debug:

How often would you say the error happens? Is it most requests or pretty rare? Are certain requests more prone to it, or does it happen about equally for any kind of request?
Are you able to examine the CPU and memory usage on the machine?
Anything funny in the postgresql logs at around the times of the error?
Does it help if you send SIGHUP to the postgrest process? (For instance by running killall -HUP postgrest)

I will work on the logs and mem.
It happens if I do not query the server for few hours, I am suspecting
session timeouts.
This happens both in my local VM 2GB ram and AWS 1 GB ram with only
Postgres running.
Plus it happened when I put "count"  in from entity in URL
Like API/employees?count
On 24-Dec-2016 9:56 pm, "Joe Nelson"<EMAIL_ADDRESS>wrote:

Thanks for reporting, I haven't seen this bug before.
Some questions to help debug:

How often would you say the error happens? Is it most requests or
pretty rare? Are certain requests more prone to it, or does it happen about
equally for any kind of request?
Are you able to examine the CPU and memory usage on the machine?
Anything funny in the postgresql logs at around the times of the
error?
Does it help if you send SIGHUP to the postgrest process? (For
instance by running killall -HUP postgrest)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/begriffs/postgrest/issues/780#issuecomment-269090759,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvQ4p7cqs2QKgxipPNJKxgEpVePLpmnks5rLUe7gaJpZM4LVKew
.

Reproduced it again.
Left the server running for some time ( 4-6 hrs ) without making any requests, got the same error.
killall -HUP postgrest did not help.
Logs and mem/cpu on the server is fine, nothing out of ordinary.

When you say you're using the latest release do you mean version <IP_ADDRESS> or a custom build from the master branch?

No, it's not a custom build, it's <IP_ADDRESS>
On 26-Dec-2016 01:27, "Joe Nelson"<EMAIL_ADDRESS>wrote:
When you say you're using the latest release do you mean version <IP_ADDRESS> or
a custom build from the master branch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/begriffs/postgrest/issues/780#issuecomment-269135407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvQ4vz4NJWAeHVM2IZjbwxCi8tMGc5Tks5rLsqzgaJpZM4LVKew
.

This is recurring now consistently, is there a fix expected in coming release. Let me know
Thanks

I don't know whether it has been fixed in the new version because I never noticed it in the original. We'll be releasing v0.4 pretty soon and you can try that. If you want to try it early you can build the master branch of this repo from source. Here are the new docs that describe how to do that https://postgrest.readthedocs.io/en/v0.4/install.html#build-from-source
Also is there a way to modify the client to avoid sending a malformed query param? I recognize this is a postgrest bug but fixing the client would be a workaround while we work to reproduce, test, and fix the server bug.

Yeah, I see that no one has reported this earlier, so it's a bit perplexing,  will wait for the new release.
Thanks again.

Ah I figured it out , it was my ssh terminal session timeout causing the issue. Using "screen" command before postgrest did the trick. Closing this and wishing you happy new year.

Happy new year!
Should we mention the issue of terminal session in the docs? I wonder if piping the output of postgrest to the syslog and backgrounding the process would accomplish the same thing.

It would be better for the new community members as I was doing this
primarily on AWS and thought the issue was with postgrest. so yes please if
you can.
Thanks
On 31-Dec-2016 7:58 AM, "Joe Nelson"<EMAIL_ADDRESS>wrote:

Happy new year!
Should we mention the issue of terminal session in the docs? I wonder if
piping the output of postgrest to the syslog and backgrounding the process
would accomplish the same thing.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/begriffs/postgrest/issues/780#issuecomment-269844453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvQ4mmxrRCsA5EITBSI9aB4tzDvYy0hks5rNb3ogaJpZM4LVKew
.

I added a note to the docs about it.


Update installation.md

Added python version 3.8 to helm-venv installation command w/ virtualenv.
Without it, latest version of Python (3.11) will throw error when running pip install crfm-helm

Thanks for the feedback. We should try to support Python 3.11; I filed #1511 for this. I'll go ahead and merge this in the meantime.


Add kijiji

Issue: #1169

Tip, if you phrase the commit title, or the commit description or PR description with the "Closes #..." or "fix #..." keyword, then when it is merged the corresponding issue is automatically closed.
Example:
Close #1169: Add kijiji as hard


Need to access Github Secrets in Fastlane(Fastfile/env) files

Discussed in https://github.com/fastlane/fastlane/discussions/21030

Originally posted by nnvdp February  8, 2023
Hello Team,
We would like to access our mobile app git repository github encrypted secrets in Fastfile or env files of Fastlane while executing the lane to upload_to_playstore.
I have gone through the blog at https://www.runway.team/blog/ci-cd-pipeline-android-app-fastlane-github-actions for the reference. But not getting the exact way to get access encrypted secrets in fastlane.
Please note, we don't want to use Github Actions workflow in this case.
Please suggest if there is anyway to achieve the above one.
Thanks & Regards,
Durga Prasad Nallamothu.

I would also like to know this, as I want to access a repo secret in my Fastfile as opposed to my Github Actions file.


fix(FocusManager): Replaces implementation with focus-trap-react

Because
In order to fix the bug reported on issue 2
This Commit
Does a re-implementation of the FocusManager, by replacing the current functionality, with the functionality provided by the "focus-trap-react" package.
In order to keep the same API as before, it makes them still available as before, but changes the default values for autoFocus, restoreFocus and contain props. It also exposes the "focus-trap" configuration API on the options prop.
As such, the component itself also gains a whole set of new functionalities, made possible by the focus-trap package.
BREAKING CHANGE: autoFocus, restoreFocus and contain are now set to true by default
BREAKING CHANGE: the useFocusManager was reimplemented. Since we no longer use a React context state management solution to travel between elements, the hook was re-implemented as an optional way to facilitate the creation of a FocusTrap instance, but without using the provided element by the package.
Closes #27

I can’t approve here, but from my side, everything seems good to go!
One suggestion for improvement would be to include a small video or screenshot of the demo. While this may not be necessary for this particular case, I believe it can always help people understand the context a bit better. This is just a personal preference, but I think it would be beneficial for future PRs.
Nonetheless, great job!!


fix: readme repository links

Are these repositories private? As those links are currently dead.

Good catch @zhedar! These repositories will be the next to go public, but are currently still private.


How to remove all user roles in a group?

Hi!
I have several groups named for example: 'soccer', 'volleyball', 'waterpolo'...
In each group, you can have one of 3 different roles: 'admin', 'captain', 'player'.
I have a method to set a role to a user within a group, but I don't know the good method to remove all others roles before adding the new one. (I'm using the V2 package)
Removing one by one all roles (even if user do not have it)
Roles.removeUsersFromRoles( userID, 'admin', group );
Roles.removeUsersFromRoles( userID, 'captain', group );
Roles.removeUsersFromRoles( userID, 'player', group );
Roles.addUsersToRoles( userID, newRole, group );

Using setRole (working with groups?)
Roles.setUserRoles( userID,  newRole, group );

Thanks!

Roles.setUserRoles( userID,  newRole, group );
seems to do the trick :)

The below is the best page to visit remove roles from SharePoint group.
https://github.com/SharePoint/PnP-PowerShell/blob/master/Documentation/SetPnPGroupPermissions.md
But here I want to know if I want to remove all Roles. How to do .
Please help me


Sema: Fix crash if we try to look up constructors while validating a constructor

addImplicitConstructors() will crash if validateDecl() does not produce
a valid signature for one of the class's constructors because the
constructor is already being validated.
This was triggered during associated type inference in the test case
in the radar.
Fixes rdar://problem/30751491.

@swift-ci Please smoke test