Transform output file poishadow\build\libs\poishadow-all.jar does not exist.

Hi @centic9 ,
I am trying to build the sample project but I am getting the below error while running the application on my device.
transfer output file poishadow\build\libs\poishadow-all.jar does not exist.
Thanks in advance.
Rohit

Hi,
i think that the poishadow.jar does not exist, that's why it is throwing this kind of error.

This is because the "poishadow" module is not referenced in the project "poitest", but "../poishadow/build/libs/poishadow-all.jar" is referenced. When you build "poitest", it will look for "poishadow-all.jar", but will not let the initiative build the "poishadow" module. Therefore, you need to manually implementation the module "poishadow" in the "poitest" in the "project structure", and then rebuild the project, then the module "poishadow" will generate the jar.

implementation project(':poishadow')
Duplicate class org.apache.xmlbeans.xml.stream.Location found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.ReferenceResolver found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.XMLEvent found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.XMLInputStream found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.XMLName found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.XMLStreamException found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.utils.NestedThrowable found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)
Duplicate class org.apache.xmlbeans.xml.stream.utils.NestedThrowable$Util found in modules xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0) and xmlbeans-2.6.0.jar (org.apache.xmlbeans:xmlbeans:2.6.0)

It's been a long time. At that time, God and I knew. Now only God knows.
But I can offer you some ideas:
Premise, I want to use POI to decrypt office documents on Android, and I succeed.
To solve duplicate class, I changed the package name of POI.
You can refer to the code that I left at that time. I have transferred it to githun at https://github.com/zchworld/poi-on-android.git

I believe we added some missing build-dependency for poitest -> poishadow in the meantime and thus this should not occur any more.
Please reopen if you still encounter this problem with the latest version of poi-on-android


[v5] Universal/Server rendering error with boolean fields

On<EMAIL_ADDRESS>React is throwing an error related to boolean fields where the props passed from redux-form are different on the server than the client.
warning.js:44 Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
 (client) t name="newsletter" checked="" value="tr
 (server) t name="newsletter" value="" data-reacti

// server props:
{"name":"newsletter","value":"","initialValue":true,"valid":true,"invalid":false,"dirty":false,"pristine":true,"active":false,"touched":false,"visited":false}

// client props:
{"name":"newsletter","checked":true,"value":true,"initialValue":true,"valid":true,"invalid":false,"dirty":false,"pristine":true,"active":false,"touched":false,"visited":false}

Client adds checked: true and value:true

I had related problem on the client side with 'radio' and 'checkbox' inputs, quick solution was to create custom Checkbox and Radio components
function Checkbox({value, onChange, onBlur, ...restProps}) {
    return (
        <input
            type="checkbox"
            {...restProps}

            // set `checked` prop instead of `value` and cast it to boolean (in pre 5.x `undefined` to `false`, in 5.x `''` to `false`
            checked={Boolean(value)}
            // assure that `onBlur` ain't gonna update value (return `undefined`)
            onBlur={() => onBlur()}
            // assure that `checked` (not `value`) property of element will be returned
            onChange={({target: {checked}}) => onChange(checked)}
        />
    );
}

I have this problem with disabled.
valid prop return like true on the server and false on the client.

@danilvalov same issue

Closing with bulk close of v5 issues. If this issue still requires action and/or also affects v6/7, please request a reopen with details or raise a new issue. v5 PR's are still welcomed.


Add new features to BLE_EddystoneService

The following features were added:

It is now possible to stop any BLE activity from the Eddystone library. See EddystoneService::stopCurrentService().
Added yotta config options to easily modify the default values for the following:

URL
URL frame interval
UID frame interval
TLM frame interval
BLE Device Name
Eddystone-URL configuration service advertising interval

Refactor the PersistantStorageHelper to easily allow other platform-specific implementations to be added.

NOTE: Some of these changes still need to be tested!
@pan- @jaustin @janjongboom

See #57, so we don't have to push to @andresag01's remote.


import: --skip should probably be the default behavior?

Hi @ErickKramer,
it seems that the current default behavior of the import command when a repository / working dir of the same name already exists is to delete the existing directory before cloning anew.
This turned out to be somewhat of a foot gun in my case. I was using rv import as part of shell script that I run to make sure all my dependencies are up to date with what ever is specified in the repos files.
Happy to report, I am up to date now. On the flip side, though, I lost all my local branches, and AFAICS there is no way to recover from that since the repository including the object store and the reflog are all gone.
Ok, yes, I should have RTFM before automating this step. :facepalm:
Luckily for me, it did not take long until I realized my mistake and I sort of remember the changes I did there - so not too much work is lost.
Still... I think this should not be the default behavior. Maybe just output an error and exit. Maybe --skip. Or maybe something like rv sync. Just do something safe by default.
Cheers,
Lukas

Thanks for raising this issue. I see what you mean and understand how the consequences of such default behavior are quite destructive.
I was just following the default behavior provided by vcstool, but I do agree that what you are suggesting is the safer approach.
I will replace the --skip flag to something like an --overwrite (some name will pop-up) to force overwriting the existing repositories.


add: regex support

Support to add regexp rules from json file.

@lilinghai PTAL

@shuke987 pls resolve conflicts

@shuke987 pls resolve conflicts

done

@shuke987 pls resolve conflicts

done


Some help with unique_ptr, etc.

This time I'm really stumped (sorry for the previous false alarm).  I tried the 'members' trick that helped with the previous problem, but I can't get this one to work.  I put the example on godbolt, here:
https://godbolt.org/z/Ecbaq53hG
If you could let me know what I'm doing wrong, please.  Thanks in advance.

Ok, after much gnashing of teeth, I found the section in the doc about recursive data structures.  Used that and removed the members<> hint, and it seems to work.


Option for multiline strings

Not really a bug, more like a feature request
Some manifests contain multiline strings, like json/yaml embedded files in a configmap or whatever. When outputting from the cluster -o yaml, the multiline values are (usually, this might depend on their creator) using multiline yaml value, an example in a pgo-config configmap:
  pgo-pg-sa.json: |-
    {
       "apiVersion":"v1",
       "kind":"ServiceAccount",
       "metadata":{
          "name":"pgo-pg",
          "namespace":"{{.TargetNamespace}}",
          "labels":{
             "vendor":"crunchydata"
          }
       }
    }

Importing that yaml into terraform through tfk8s,the manifests seem okay but all the multiline values are "...\n...\n...\n"
This is working fine AFAICS, but for editabitility of the generated .tf, that'd be nice to use the multiline facilities in hcl, like the heredocs:
"pgo-pg-sa.json"= <<JSON
    {
       "apiVersion":"v1",
       "kind":"ServiceAccount",
       "metadata":{
          "name":"pgo-pg",
          "namespace":"{{.TargetNamespace}}",
          "labels":{
             "vendor":"crunchydata"
          }
       }
    }
JSON

Thanks for opening this @rienafairefr – we are using Terraform's repl package to do the formatting to HCL and it looks they recently added support for this so we should be able to bring this in.
I had a quick look and there's some additional munging we will have to do because the interface has changed to use cty.Value rather than map[string]interface{}.


Community Card Component #23

What
This new pull request introduces a customizable "Community Card" component in Astro, designed to showcase key community details. It includes an avatar, a brief description, relevant tags, the community name, and direct links to the community's website, Instagram, and GitHub profiles.
This new component provides a flexible way to present community information in a consistent, visually appealing format.
The only remaining task is to fix the link functionality, which is currently pending. This update offers a great opportunity to enhance how communities are represented on the platform.
Reference:

Read more
Discusion about UI

@nsandoya Would you mind helping me with the suggested changes?

@nsandoya Would you mind helping me with the suggested changes?

Sure 😊 I've just implemented suggestions in your branch


v2.2.6

Rate limit by middleware for future auth
Rename slp/address/convert to slp/convert

Pull Request Test Coverage Report for Build 1672

0 of 0   changed or added relevant lines in 0 files are covered.
300 unchanged lines in 9 files lost coverage.
Overall coverage increased (+0.02%) to 75.828%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/control.js
2
76.69%

dist/routes/v2/util.js
6
83.5%

dist/routes/v2/mining.js
8
77.19%

dist/routes/v2/block.js
8
85.92%

dist/routes/v2/transaction.js
8
81.25%

dist/routes/v2/rawtransactions.js
29
83.7%

dist/routes/v2/address.js
69
79.38%

dist/routes/v2/blockchain.js
74
77.28%

dist/routes/v2/slp.js
96
54.25%

Totals

Change from base Build 1654:
0.02%

Covered Lines:
1707

Relevant Lines:
2102

💛  - Coveralls

:tada: This PR is included in version 1.14.7 :tada:
Your semantic-release bot :package::rocket:


Feature/increase scene grid resolution

Overview
This commit replaces the current scene grid endpoint/implementation with
one that uses Leaflet's grid layer to request summary aggregate
information for each tile on the map. This speeds up the endpoint by
reducing the amount of information required to power it.
The grid layer itself uses Konva to draw rectangles and style
them. Canvas events are used to display the count when the canvas
rectangle is hovered over. Additional styling work is necessary, but
this gives us hopefully a start.
Checklist
~- [ ] Styleguide updated, if necessary~

[x] Swagger specification updated, if necessary

Demo

Notes
Styling will need to be updated, but I think this first implementation is passable for now.
Testing Instructions

Switch to branch, run migrations via ./scripts/setup
Start server and application front end
Go to browse page, ensure scene grid shows up

Closes #812

Just realized I should have added a bunch of docstrings, going to work on that now


UnparsedFlagAccessError when calling env.reset()

Hi,
I keep getting this weird error when calling env.reset(). I only get this error when running a Jupyter notebook
~/miniconda3/lib/python3.9/site-packages/dm_robotics/agentflow/spec_utils.py in debugging_flag()      45       46 def debugging_flag() -> bool: ---> 47   return FLAGS.debug_specs      48       49
Error Message:
UnparsedFlagAccessError: Trying to access flag --debug_specs before flags were parsed.
Defining the absl flag by myself does not help:
DuplicateFlagError: The flag 'debug_specs' is defined twice. First from dm_robotics.agentflow.spec_utils, Second from /home/ztan/miniconda3/lib/python3.9/site-packages/ipykernel_launcher.py.  Description from first occurrence: Debugging switch for checking values match specs.
Any ideas why this keeps happening?
Thanks a lot

Hi, what is in your function that runs when name == "main "?
It's important to have this kind of structure:
from absl import app

def main(argv):
  ...

if __name__ == '__main__':
  app.run(main)

See also the abseil/Flags docs

Is this issue resolved?


more errors updating

Hello, i am now getting an update message in domoticz about version (beta) 5276, current 3.5231
if i press the update button it doesnt update, how do i update to the current version ?
do i need to recreate the docker image ?

Hello,
I've pushed a new version on the repo. Can you try with it (docker pul...) ?

seems to be updated to latest version now, will keep an eye on it for the next update, should it update from the GUI now ?

I don't take release of domoticz but I retrieve the last git version. Normally with updates from Debian I publish regularly new versions so I don't really need to rebuild manually but there were few Debian new docker images these last days.
Being able to update inside a docker is a bit "breaking" docker philosophy.

Le 30 juin 2016 à 22:16, deejc-uk<EMAIL_ADDRESS>a écrit :
seems to be updated to latest version now, will keep an eye on it for the next update, should it update from the GUI now ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


TDR-2816 - Add file properties for the descriptive metadata

Add former_reference_department and creating_body properties
Add or update display properties for file_name_translation, former_reference_department and creating_body properties
Add file property values for the Language properties

:bell: Just a reminder to increment the script number as I've just merged a new script


[Incompatiblity] Extended clouds render on top of Distant Horizons terrain

Pretty simple incompatibility but I figured I'd document it - extended clouds and distant horizons are both meant to give an impression of a larger rendered world, but of course the depth information thats... somewhere, isn't able to be used by extended clouds without a tergeted patch (I imagine).

Here's an example without extended clouds:

It's notable that cloud distance looks even shorter using distant horizons than vanilla, because more terrain is rendered than clouds - so It'd have quite a visual impact to resolve the conflict.

Ah okay, turns out this one is invalid - DH's method explicitly sets depth like a skybox, meaning their only way out is to just draw new clouds themselves as far as I know.


Device Requirement for optimal performance?

Hi @justadudewhohacks
Many many thanks for this awesome project <3 i just wanted to ask what are the best specs for using this project? While running this on my Mac Air, 8 GB Ram it started taking some loaded and my laptop fan started. I just want to understand what requirements could be the best fit for this project? (I just want to tell me, users, to meet a minimum of these requirements. before using this)
I want to run this project in these conditions.

This project will run continuously for 2 to 3 hours
User will use only and only this program on a single chrome instance and nothing running in any tab or any software in the background

Fan running ...yeah, that's tough to avoid that with this project lol. Is that a problem for you if the fan kicks in?
Not sure of minimums, but I've let it run for ~9 hrs overnight, getting ~20fps on a MacBook Pro (2.4 Ghz 8-core i9 with 32 GB RAM) - and yeah, fans kicked in - but that's just because I'm limited to two cores on the MBP because the Ui takes one core and web worker takes one core - no real easy way to split work across more than 2 CPUs here in this case.
So yeah.. multiple cores? RAM is not as important as CPU ..still need healthy ram, but ymmv ... @rohitcoder - just my thoughts, hope it helps. Happy to answer questions, lmk


Update tgi cpu image version to 2.4.0-intel-cpu

Description
Update tgi cpu image version to 2.4.0-intel-cpu
Issues
n/a.
Type of change
List the type of change like below. Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.

https://github.com/opea-project/GenAIExamples/pull/1035
Wait untill the above PR verified/merged.

PR https://github.com/opea-project/GenAIExamples/pull/1035 is merged, we can go on with this PR


Replace cypress email.cy.js with PyTest integration tests.

@danmash I've added new or adapted existing tests (as much was already implemented) for all the email cypress tests I think. But I'm not sure about a couple of things. First, there were existing pytest tests that were structured a little differently than in other parts of the app (e.g., multiple explicit tests of the same route inside the same function rather than use of pytest.mark.parametrize, and I didn't restructure. Do you think this is ok, or should they be restructured? Second, I'm not sure how much of the cypress stuff can now be safely purged from the repo, so I haven't deleted any cypress code. Is there still need for cypress code, in being used for stuff other than the e2e tests, such as github integration? As a result, I haven't made a PR yet.

@cyplas it's fine to have unit tests for functions and integration tests for the route. it's fine if these tests are overlaps.
Please proceed with removing all cypress/js files and configs and update CI configuration in .circleci folder
https://github.com/ClimateMind/climatemind-backend/issues/406


Smart Hide Rooftops On Mouseover

Is your feature request related to a problem? Please describe.
It is hard to play the game while showing rooftops due to reduced visibility, yet some people like me greatly prefer the look of the game with them enabled.
Describe the solution you'd like
The ability to hide building rooftops when the mouse is hovering over a roof. This would be a plugin toggle.
Describe alternatives you've considered
A few people have suggested a keybind which quickly toggles whether rooftops are on or off. (See https://github.com/runelite/runelite/issues/11376 and https://github.com/runelite/runelite/issues/8943) Besides this idea being disliked, I suspect it would probably have been disliked by users too as it's a very manual solution requiring constant button presses.
Additional context
I'm not extremely familiar with how scenery objects are assembled. I assume since there already is an on/off rooftop toggle that these would be independent objects, making it somewhat easy to detect a mouse hovering over them.
Similarly, I'm not sure if hiding only the single roof you mouse over would be possible or if it would have to hide all rooftops or show them all for each mouseover event.
Ideally, if possible, I think this would mean there should be an inner-toggle for mouse rooftops, selecting to either hide all roofs when hovering or to only hide the one roof, like so:
[✓] Enable smart-hide rooftops
   [✓] Hide all rooftops on mouseover

Added in https://github.com/runelite/runelite/commit/a6e3d4ca20de7d095020942f774a847be6f4a2b4
Big thanks to @Hydrox6 for your hard work on this


[#3637]improvement(IT):  Add ignore_failed flag and test report for TrinoQueryTestTool

What changes were proposed in this pull request?
1、Support ignore_failed on run TrinoQueryTestTool.
2、Summarize all the test cases results when test finished
Why are the changes needed?
Fix: #3637
Does this PR introduce any user-facing change?
NO
How was this patch tested?
Exist IT

LGTM


Exposed SyncService, NotificationService classes

- What I did

Exposed SyncService, NotificationService classes
Added minor docs for the classes.

- How I did it

Forked and edited.

- How to verify it

Generate the docs using dartdoc command and head over to the /at_client/doc/api/at_client/AtClientManager-class.html file and check if it has SyncService and NotificationService references.

- Description for the changelog

This is already done.


Is it possible to play a video from a stream?

I have a video in the following URL here. It's to be played from a CDN as a stream and doesn't end in the typical .mp4 extension. I tried passing this as the video uri, but it doesn't work on Android. When I console the error in the onError method, this is what is logged to the console
{"error": {"extra": -2147483648, "what": 1}}

Any idea what's wrong here?

Look here https://stackoverflow.com/questions/63234718/unable-to-play-drm-video-with-react-native-video-drm-fork-in-android


[10.x] Configuration value injection

Discussion
The idea from this pull request has been discussed here and here previously.
I have wrapped this functionality in a package which already achieves this here but received enough positive feedback to believe this way of injecting configurations might be a nice feature for the framework itself.
Functionality
This PR adds a way of injecting configuration values into class dependencies through the usage of a newly included php attribute, the InjectedConfig attribute. An example of this would be:
class Foo {
    public function __construct(
        #[InjectedConfig('app.name')]
        public string $myConfiguredAppName,
    ){}
}

in which case the app.name config key's value would be injected into the string property of $myConfiguredAppName.
A clear benefit of this, is that config values can now be supplied as class dependencies, as opposed to using the config() helper in code (which I often see being scattered all around the place). This makes it very clear which configuration values a class depends on, and provides (in my opinion) a cleaner way of using config values.
I don't see how this might break backwards compatibility, and don't really think it does, as it hooks in right before the part of the resolvePrimitive where a BindingResolutionException is thrown.
I am really interested if this is a feature that is desired, and am open to any and all feedback regarding it.
Kind regards,
Melchior Kokernoot

@MelchiorKokernoot I like the idea but unfortunately the implementation could use some improvement.
Specifically the related part where you do isset($this['config']) essentially we are now recursively looking for a class which implements the \Illuminate\Contracts\Config\Repository class but not the fqcn of this class what if the config alias to this class gets updated to a class that doesn't adhere to the interface?
Or what if the config component isn't installed i don't know if the Illuminate\Container should be dependent on the \Illuminate\Config component. I am mostly concerned about the persons using the container component outside laravel? I might be wrong but it seems to me that this would (in order to work probably put a soft dependency on the Config component)?

@morloderex I changed it to check for the existance of the \Illuminate\Contracts\Config\Repository::class interface, and added an instanceof check, do you think this prevents the situation you described?

Should be fine now :)

As far as I am concerned, this PR is ready for another review

https://gist.github.com/bert-w/78988fff5e280f353840c25fbcc5ea43 Try this out for the testing class. I also removed the $object->fooNull assertion because it throws a warning since it is not a property of the class.

I dont see how that failing test is related to the code in this PR. It has to do something with mt_srand() (seeding the RNG so tests are consistent)? Not sure what is happening there.

Should default values be supported?

Should default values be supported?

@taylorotwell haven't really had the need for it myself, but of course I can adjust it so default values are supported. Would you like that?

I have implemented default values through the InjectedConfig attribute's second parameter

Does the new method hasConfigRepository(): bool need to be public? Does it need to be a method at all? Adding a new method (especially a public method) adds to the long term maintenance burden, it will be hard to rename/move it later because of a potential BC-break.

Hi @taylorotwell, would you please elaborate a little as to why you are leaving it out? It could help me improve for the next time I want to PR something, and it would mean a lot to me :) I'd love to hear from you

@taylorotwell If you have any chance of taking a look at this, I'd still appreciate it a lot!

A little torn on this one because I do kind of like it, but going to leave it out for now. May revisit this in the future though.

Is 'the future' now @taylorotwell ?


Don't mine

I have a small problem, when I ask my bot to mine, it thinks it has mined the block when it has not. Look at the screen below. I'm in 1.20.4

I think this issue is fixed in code in github. But not published to npm...

Hum :/


Documentation for PyGCP APIs

I've spent some time on this already, and have run into a few issues which mean we may have to change how we structure our package. I'm using Sphinx, which is the documentation generator used by the Python core team for the Python SDK.
Issues I've hit so far:

Sphinx needs to actually import the code it is generating documentation for, as it uses introspection. It throws an exception on gcp.interactive due to the IPython attributes on cell magic methods. I expect there must be a workaround for this as the IPython API docs are generated with Sphinx too.

Sphinx doesn't include functions in __init__ files by default. There is a workaround for this; in the conf.py file for Sphinx we can override the way it determines what to skip as follows:
def skip(app, what, name, obj, skip, options):
  if name == "\_\_init\_\_":
    return False
  return skip

def setup(app):
  app.connect("autodoc-skip-member", skip)

that said, it seems that it is considered bad practice to define functions in __init__ files and we should probably move them out. Generally speaking, it seems that it is recommended that __init__ files consist just of imports and optionally setting the __all__ variable

the way we currently keep our classes in files with leading underscores means that they are not exposed and Sphinx will not generate documentation for them. The workaround for this is to specify them explicitly in __all__ in __init__.py, but this does mean we need to import them with their real names (i.e. instead of "from ._table import Table as _Table" we need to use "from ._table import Table", etc)

there is no way to prevent documentation for the class constructors, even though we want to enforce the use of the factory methods. This is because the constructor is not documented as a method __init__; it is documented as part of the class section header (i.e. instead of "__init__(stuff)" as a method in the section for class Foo, the section header for class Foo will say "class Foo(stuff)")

We have Sphinx-generatable docs now. This was spread over time and various changes but with PR #666 can be considered done.


[5.5] Add a firstWhere method to the Collection class

$firstActiveUser = $users->firstWhere('active', true);

Just needed this yesterday :smile:

Where's firstOrFailWhere?

@jordyvandomselaar right here:
$value = $c->firstWhere(...) ?? throw new Exception(...);

No no no, if we're adding methods for the heck of it that provide no value anyways we're gonna need this.


Failed: Update autorest version to prerelease 3.0.0-alpha.20240425.4.x

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4629

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

Close in favor of #43678


More of Question to all

When you have multiple controls on a form that all have async validation how does one fire all of these on form submit and then continue the form submit when complete if all async validations pass? I can't seem to find a way to group async validations whereas sync validation is easy on enter key/form submit button

I'd recommend coding your async validations as a Promise.all() inside the form's submit method instead.

@davidkpiano Awesome idea David! That actually solved a couple of problems for me. Thanks!!


ERROR! the playbook: Hat could not be found

I am struck at the below error while running "gcloud.sh" script, where there are no failures in the "PLAY RECAP" section, but got an error after that saying "ERROR! the playbook: Hat could not be found".
The intermediate steps that are executed are deprecated and given below:
[root@docker gce-cli]# ./gcloud.sh
~/openshift-ansible-contrib/reference-architecture/gce-cli/ansible ~/openshift-ansible-contrib/reference-architecture/gce-cli
PLAY [configure ansible connection to the gcp and basic stuff] ****************************************************************************************************
TASK [ansible-gcp : check for service account] ********************************************************************************************************************
ok: [localhost]
.
.
.
TASK [deployment-create : create deployment ocp-network config file] **********************************************************************************************
ok: [localhost]
TASK [deployment-create : check if deployment ocp-network exists] *************************************************************************************************
ok: [localhost]
TASK [deployment-create : create deployment ocp-network] **********************************************************************************************************
skipping: [localhost]
PLAY RECAP ********************************************************************************************************************************************************
localhost                  : ok=10   changed=0    unreachable=0    failed=0
ERROR! the playbook: Hat could not be found
[root@docker gce-cli]#

@pschiffe  Please help  me resolving this..

Hello @achyuthk , could please try to run bash -x gcloud.sh and then paste the output here? But don't forget to strip the passwords from there.

Sorry I am totally new to google cloud engine...
I didn't get what does stripping passwords mean...
I got a new error now for creating dns zone. I was not getting this error before..
[root@docker gce-cli]# ./gcloud.sh
~/openshift-ansible-contrib/reference-architecture/gce-cli/ansible ~/openshift-ansible-contrib/reference-architecture/gce-cli
PLAY [configure ansible connection to the gcp and basic stuff] ****************************************************************************************************
TASK [ansible-gcp : check for service account] ********************************************************************************************************************
ok: [localhost]
TASK [ansible-gcp : create new service account] *******************************************************************************************************************
skipping: [localhost]
TASK [ansible-gcp : create service account key] *******************************************************************************************************************
ok: [localhost]
TASK [ansible-gcp : grant service account access to the gcp project] **********************************************************************************************
skipping: [localhost]
TASK [ansible-gcp : create secrets file] **************************************************************************************************************************
ok: [localhost]
TASK [dns-zone : create dns zone] *********************************************************************************************************************************
fatal: [localhost]: FAILED! => {
"changed": false,
"failed": true,
"module_stderr": "[DEPRECATION WARNING]: secrets file found at '/root/openshift-ansible-contrib\n/reference-architecture/gce-cli/ansible/inventory/gce/hosts/secrets.pyc'.  This\n method of specifying credentials is deprecated.  Please use env vars or \nAnsible YAML files instead.\nThis feature will be removed in version 2.5. \nDeprecation warnings can be disabled by setting deprecation_warnings=False in \nansible.cfg.\nTraceback (most recent call last):\n  File "/tmp/ansible_WzzS8G/ansible_module_gcdns_zone.py", line 386, in \n    main()\n  File "/tmp/ansible_WzzS8G/ansible_module_gcdns_zone.py", line 348, in main\n    zone = _get_zone(gcdns, zone_name)\n  File "/tmp/ansible_WzzS8G/ansible_module_gcdns_zone.py", line 277, in _get_zone\n    for zone in available_zones:\n  File "/usr/lib/python2.7/site-packages/libcloud/dns/drivers/google.py", line 311, in _get_more\n    **kwargs)\n  File "/usr/lib/python2.7/site-packages/libcloud/dns/drivers/google.py", line 332, in _get_data\n    params=params,)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 806, in request\n    *args, **kwargs)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/base.py", line 571, in request\n    params, headers = self.pre_connect_hook(params, headers)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 788, in pre_connect_hook\n    self.oauth2_credential.access_token)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 666, in access_token\n    self._refresh_token()\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 674, in _refresh_token\n    self.token = self.oauth2_conn.refresh_token(self.token)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 393, in refresh_token\n    return self.get_new_token()\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 537, in get_new_token\n    return self._token_request(request)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 370, in _token_request\n    raise GoogleAuthError('Invalid authorization response, please '\nlibcloud.common.google.GoogleAuthError: 'Invalid authorization response, please check your credentials and time drift.'\n",
"module_stdout": "",
"rc": 0
}
MSG:
MODULE FAILURE
PLAY RECAP ********************************************************************************************************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1
[root@docker gce-cli]#

I am using the same "config.sh" file which I used to.. Is it a problem with Client credentials?

What version of ansible are you using? Unfortunately, currently only 2.2.x is supported, 2.3 and newer won't work, so you might need to downgrade the ansible package.

Am using "<IP_ADDRESS>" version of ansible.
[root@docker gce-cli]# ansible --version
ansible <IP_ADDRESS>
config file = /etc/ansible/ansible.cfg
configured module search path = Default w/o overrides
python version = 2.7.5 (default, Nov  6 2016, 00:28:07) [GCC 4.8.5 20150623 (Red Hat 4.8.5-11)]
[root@docker gce-cli]#

I was not facing this issue before, where the version was the same as now... Wierd.. Let me try by downgrading ansible.

I am still getting the same error even with <IP_ADDRESS>...
Commands used:
yum downgrade ansible
#cd openshift-ansible-contrib/reference-architecture/gce-cli/
#bash -x gcloud.sh
[root@docker gce-cli]# ansible --version
ansible <IP_ADDRESS>
config file = /etc/ansible/ansible.cfg
configured module search path = Default w/o overrides
[root@docker gce-cli]#

So what's the output of bash -x gcloud.sh? BTW by stripping passwords I just meant to delete them from the output before posting it here.

The same error as before.
[root@docker gce-cli]# bash -x gcloud.sh
. . .
TASK [dns-zone : create dns zone] **********************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: libcloud.common.google.GoogleAuthError: 'Invalid authorization response, please check your credentials and time drift.'
fatal: [localhost]: FAILED! => {
"changed": false,
"failed": true,
"module_stderr": "Traceback (most recent call last):\n  File "/tmp/ansible_gKjHp5/ansible_module_gcdns_zone.py", line 381, in \n    main()\n  File "/tmp/ansible_gKjHp5/ansible_module_gcdns_zone.py", line 343, in main\n    zone = _get_zone(gcdns, zone_name)\n  File "/tmp/ansible_gKjHp5/ansible_module_gcdns_zone.py", line 272, in _get_zone\n    for zone in available_zones:\n  File "/usr/lib/python2.7/site-packages/libcloud/dns/drivers/google.py", line 311, in _get_more\n    **kwargs)\n  File "/usr/lib/python2.7/site-packages/libcloud/dns/drivers/google.py", line 332, in _get_data\n    params=params,)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 806, in request\n    *args, **kwargs)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/base.py", line 571, in request\n    params, headers = self.pre_connect_hook(params, headers)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 788, in pre_connect_hook\n    self.oauth2_credential.access_token)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 666, in access_token\n    self._refresh_token()\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 674, in _refresh_token\n    self.token = self.oauth2_conn.refresh_token(self.token)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 393, in refresh_token\n    return self.get_new_token()\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 537, in get_new_token\n    return self._token_request(request)\n  File "/usr/lib/python2.7/site-packages/libcloud/common/google.py", line 370, in _token_request\n    raise GoogleAuthError('Invalid authorization response, please '\nlibcloud.common.google.GoogleAuthError: 'Invalid authorization response, please check your credentials and time drift.'\n",
"module_stdout": ""
}
MSG:
MODULE FAILURE
PLAY RECAP *********************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1

The problem was with the credentials in the config.sh. Now, we are back at the same error, for which issue was reported...
[root@docker gce-cli]# bash -x gcloud.sh
. . .
TASK [deployment-create : create deployment ocp-network config file] ***********
ok: [localhost]
TASK [deployment-create : check if deployment ocp-network exists] **************
ok: [localhost]
TASK [deployment-create : create deployment ocp-network] ***********************
skipping: [localhost]
PLAY RECAP *********************************************************************
localhost                  : ok=11   changed=0    unreachable=0    failed=0

ansible-playbook -e<EMAIL_ADDRESS>-e rhsm_password=cloudgen@123 -e rhsm_pool=Red Hat OpenShift Container Platform, Standard, 2-Core playbooks/main.yaml
ERROR! the playbook: Hat could not be found
[root@docker gce-cli]#

@pschiffe please assist.

OK, I think I see the problem, please try to use the pool ID for the RH_POOL_ID variable, it looks like hash, e.g. 3de403e3ce9... and not the pool name. I'll try to fix this issue in a PR later.

Do you mean the pool id that we get for openshift with the command "subscription-manager list --available --matches 'OpenShift'" ??

Yes, it's the Pool ID:             8a815bef7... field.

I have a connectivity issue to the redhat server. I will try with pool id and update.

@pschiffe Apologies for the delay..
I have updated the RH_POOL_ID in config.sh with the actual pool-id and the previous error is resolved.
But, got a new hurdle as below:

Please suggest.

Thanks for reporting and helping out with the issue! :-)


fix: removed Object.values usage in math

Object.values is not supported by IE11, causing this feature to break in
IE11. Converted Object.values to Object.keys.map to simulate the same
behavior.
This will resolve #407

Closing in favor of #411. Thanks @goldsziggy


Move "installation without scheduler" from README.md to installation docs

Summary
The README.md file includes information about how to install Keptn without the Keptn scheduler, which is possible when installing Keptn on K8s v1.26 and later.  The "Components" section of the documentation discusses this but the "Installation" section does not include that information
Related to https://github.com/keptn/lifecycle-toolkit/issues/3200
DoD

Add "Installation without scheduler" (better title?) to Installation docs
Delete this information from the README.md file

the information about how to install w and w/o scheduler is already on the installation page here.
I think that is enough information. It's not really important for users since it only changes compatibility options under the hood but is transparent for the user. So I think the info on the installation page is already enough here.
I would just go for removing the info from the readme.


CASMTRIAGE-3637: fix use of undefined variable

Summary and Scope
Instead of returning the actual response from bss, this
code was returning an undefined variable.
Issues and Related PRs

Resolves CASMTRIAGE-3637
Change will also be needed in release/1.2

Testing
Tested on:

rocket

Test description:
Fixed the code, ran the script, success.
Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Did you use the PR template from somewhere else or did I forget to update this repo? Just curious.

Since I had to do like 8 PRs for this change, I just copy-pasta'd the PR words from the other repos.  This is from the template in the CSM repo, more or less.


Use UUID instead of getTime()

If you fire multiple instances of this within a second the old getTime method could be unsafe :)

You're right, I missed that. Merged.


3.8 database.registerUnknown does not map device to user

Hello,
database.registerUnknown in 3.8 registers the device into device table, however does not map the device to any user. in user_device table.
This makes it difficult even for admin to manage the device. other than manually making entries into db.
Am i missing out on something, or is this a bug?

It behaves as should.
Admin can link any device to any user, even device linked nobody, just via web-interface/API.
It is not necessary make entries into db manually.

@Abyss777 I think, maybe i should have logged out and re-loged into web-interface? then it would show up maybe? because it wasnt showing up then.. I will try with another device :) Thanks for the info

Works as expected.


3.10.0: unexpected EOF

Summary

Backup is not working since upgrading to 3.10.0
Steps to reproduce

Upgrade to 3.10.0

Relevant logs and/or screenshots
** Performing Manual Backup
/assets/functions/10-db-backup: line 920: syntax error: unexpected end of file
/etc/services.available/10-db-backup/run: line 52: check_availability: command not found
/etc/services.available/10-db-backup/run: line 53: backup_mysql: command not found
2023-09-14.05:23:17 [INFO] ** [db-backup] Backup routines finish time: 2023-09-14 05:23:17 GMT with overall exit code 0
2023-09-14.05:23:17 [NOTICE] ** [db-backup] Backup routines time taken: Hours: 0 Minutes: 00 Seconds: 00
/etc/services.available/10-db-backup/run: line 79: cleanup_old_data: command not found

Hi thanks for the report. I can reproduce and 3.10.1 is being built as we speak.

Thank you for your instant response and help :)


logging: Do not set HttpRequest.Latency if Entry.HTTPRequest.Latency is 0

A latency of 0 ns doesn't make any sense, but the current logging package adds the field unconditionally if the HTTPRequest is present. Other fields where a 0 value doesn't make sense are omitted from the stackdriver log.

Closed via https://github.com/GoogleCloudPlatform/google-cloud-go/commit/1a6ade992dec11be6d95ecb707336154711c4723.

Why was this ticket closed? It has nothing to do with the referenced commit.

Closed via 78759a61330946eb0d8f57eb6f88a7c7d6574912.

Thanks!

(Edited the wrong hash.)


Remove cockroach demo telemetry message

Fixes:

DOC-11646
DOC-11657
DOC-11658
DOC-11659
DOC-11660
DOC-11661

Summary:

Update cockroach demo docs to remove the banner output about disabling telemetry

This output was removed in https://github.com/cockroachdb/cockroach/pull/133284 and then backported

@angles-n-daemons this docs PR is pretty basic (just deleting some lines) but wanted to keep you in the loop on it, PTAL
Note: the text being removed from docs was also outdated vs what was actually removed from your PR so that's why it's not the same exact text


Replace Logo component with a more idiomatic example

Replaces the Logo component with a simple grommet Icon to prevent users from assuming there is a built in logo component with appropriate styling.
This addresses grommet/grommet/issues/1073

Build appears to be failing due to not being able to execute eval yarn

Hey @jaitaiwan. Sorry for the delayed feedback on this PR. Would you mind updating the pull request to remove the conflicts so we can successfully merge? Thanks for your contribution!

I believe that with the requested changes my PR becomes pointless perhaps?

I don't think so @jaitaiwan
how about
A react node. Best suited to an svg element or a custom component containing an svg element.

Fixed in https://github.com/grommet/grommet-docs/commit/3c7be98435a3670d61c09db5954112643edfa202


Help - Nx --affected with targets by modules -  Problem to find package.json file in the root

Hi;
i'm trying to use Knip actually with NX;
In my project I have more than 20 different modules, so... to have an efficiency running when with Knip I decided to use the nx --affected (with this only the modified module will run the knip) and with this I created one target; but I havent sucess,
If I use the command nx run someModule:lint:knip --skip-nx-cache for example, I'll have this following message:

Unable to find package.json

Project Structure - Base:
project/
 apps/
 modules/
  someModule/
   project.json
knip.ts
package.json

So, in this case, I havent package.json inside of someModule, only I have in the root;

But when I created an file named package.json only with { } inside of someModule folder I have success; the Knip run perfectly;

So, because this, I understood the problem are with "my Knip cant find the package.json on the root of project and I dont know how to fix that";
Can anyone help with this?

Files:
package.json
scripts: {
  [...]
     "lint:knip": "nx affected --target=lint:knip --base=$NX_BASE --head=$NX_HEAD",
  [...]
devDependencies: {
  [...]
     "@nrwl/react": "15.8.6",
     "knip": "^1.17.0",
  [...]
}

knip.ts
import type { KnipConfig } from 'knip';

const config: KnipConfig = {
  nx: {
    config: ["{apps,modules}/**/project.json"]
  }
};

export default config;

someModule/project.json
"lint:knip": {
    "executor": "nx:run-commands",
    "options": {
      "cwd": "modules/someModule",
      "commands": ["knip"]
    }
  },

I think I understand what you're trying to do: you are only interested in unused files and exports (and not dependencies). You would also need to add --include files --exports otherwise the reported unused (dev) dependencies would be false positives. You want to see unused files and exports from basically any directory, right? I say "any", because Knip does not know about Nx project structure or project.json files.
To be honest, what you are trying to do was actually supported in an early version. It is not anymore (and I renamed the --dir argument to --workspace), because it's confusing for most use cases.
I do think you have a valid use case here, perhaps it's interesting to look into a --dir argument that basically ignores package.json and (unused) dependencies, but has --include files --exports.
(By the way, I would recommend to move to Knip v2 as it is faster and should give more accurate results.)

Hi, thanks for the reply;
"You want to see unused files and exports from basically any directory, right?"

Yes, by directory;

"To be honest, what you are trying to do was actually supported in an early version."

You remenber the version number?

Ive done severals tests with the Knip v2, but I run into this same issue; I set --include files --exportsand get the errorUnable to find package.json`;
Do you know any options I apply to tell what is the package.json path in the root (because I only have this one)? I just think in something similar with -t, --tsConfig;
Thanks again

You remenber the version number?

The last version before v1: v0.13.3

I've done severals tests with the Knip v2, but I run into this same issue; I set --include files --exports and get the error Unable to find package.json;

Sorry if that wasn't clear. What you want is not supported in either v1 or v2. It requires to be ran from the root of the repository with a package.json.

Do you know any options I apply to tell what is the package.json path in the root (because I only have this one)? I just think in something similar with -t, --tsConfig;

All options are visible with knip --help and I knip does not have this option to provide a different path for package.json.

I created a small project (repro) with this problem, you can try to run?
Project: https://github.com/ilucasanderson/nx-knip
Main commit: https://github.com/ilucasanderson/nx-knip/commit/ce46c2652b6984c5d54ba4714759ab7b4984f8d8;
Command: yarn knip;
You can see if I forgot to insert some information/code?

But I said it is not supported :)
Note that this "integrated repo" setup in your repo with Nx projects is not the same as workspaces ("package based").
I'm going to stay here in this issue to keep things together (to be honest I don't want to get lost in other tooling like Nx). Let me re-iterate on your options:

You can go back to v0.13.3 and try using --dir with that version.

From the root of the project:
{
  "entryFiles": ["src/index.ts"],
  "projectFiles": ["src/**/*.{ts,tsx}"]
}

And run something like this from the root (or with Yarn or Nx or whatever):
knip --dir modules/ui/input --include files,exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates

This works, but it's a bit slow. Of course it depends on the size of the actual project. But I strongly recommend to go with Knip v2 and analyze the project as a whole. If you want to use Knip you will need to migrate to v2 sooner or later anyway as you may understand I'm not keen on supporting such an early version.

Use the latest v2 and analyze the project as a whole.

{
  "entry": ["apps/*/src/index.tsx", "modules/*/*/src/index.ts"],
  "project": ["{apps,modules}/**/*.{ts,tsx}"],
  "nx": ["{apps,modules}/**/project.json"],
  "babel": ["babel.config.json", "{apps,modules}/**/.babelrc"]
}

As this example knip.json shows, you will need to override the config files for all plugins that are scattered around the project in this way.
And run Knip from the root:
knip --tsConfig tsconfig.base.json

From a Knip (v2) perspective, if you still have the option, a package-based workspaces setup is better. Then you can also use the --workspace option with Nx affected.
Hope this helps. Feel free to ask questions :)

Ah, ok... I understood the situation;
Sorry if I got confused haha
My team and I really appreciate all the support you give us;
We changed the proposal on the feature, decided to use Knip running throughout the project;
I need to say thank you again and congratulations for the project :)

Thank you! Good luck with the project 🚀


Cannot select player to play music on

What version of Music Assistant has the issue?
2.0.0
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Not sure if I'm doing anything wrong. I am getting started with Music Assistant since the announcement, but I simply cannot select player to play music on.
https://github.com/music-assistant/hass-music-assistant/assets/29582865/7bd947e1-73fe-49bf-8e82-78e141053652
How to reproduce
See video above.
Music Providers
Youtube Music
Player Providers
Tried Chromecast, DLNA, and HA.
Full log output
2024-05-09 19:29:46.112 INFO (MainThread) [music_assistant] Starting Music Assistant Server (0772b0674c674da8bc848753922662b4) version 2.0.0 - HA add-on: True - Safe mode: False
2024-05-09 19:29:46.152 INFO (MainThread) [music_assistant.cache] Initializing cache controller...
2024-05-09 19:29:47.085 INFO (MainThread) [music_assistant.music] Using a sync interval of 180 minutes.
2024-05-09 19:29:47.885 INFO (MainThread) [music_assistant.streams] Detected ffmpeg version 6.0.1 with libsoxr support
2024-05-09 19:29:47.886 INFO (MainThread) [music_assistant.streams] Starting server on  <IP_ADDRESS>:8097 - base url: http://<IP_ADDRESS>:8097
2024-05-09 19:29:47.904 INFO (MainThread) [music_assistant.webserver] Starting server on  <IP_ADDRESS>:8095 - base url: http://<IP_ADDRESS>:8095
2024-05-09 19:29:48.245 INFO (MainThread) [music_assistant] Loaded metadata provider MusicBrainz Metadata provider
2024-05-09 19:29:48.247 INFO (MainThread) [music_assistant] Loaded music provider Music Assistant
2024-05-09 19:29:48.249 INFO (MainThread) [music_assistant] Loaded metadata provider fanart.tv Metadata provider
2024-05-09 19:29:48.251 INFO (MainThread) [music_assistant] Loaded metadata provider TheAudioDB Metadata provider
2024-05-09 19:29:48.615 INFO (MainThread) [music_assistant.music] Sync task for Music Assistant completed
2024-05-09 19:34:25.897 INFO (MainThread) [music_assistant] Loaded music provider YouTube Music
2024-05-09 19:34:41.404 WARNING (MainThread) [music_assistant.music] Sync task for YouTube Music completed with errors
2024-05-09 19:35:18.590 INFO (MainThread) [music_assistant] Loaded plugin provider Home Assistant
2024-05-09 19:35:35.612 INFO (MainThread) [music_assistant] Loaded player provider Home Assistant MediaPlayers
2024-05-09 19:35:35.797 INFO (MainThread) [music_assistant.players] Player registered: media_player.home_theater_do_escritorio/Home Theater do escritÃ³rio
2024-05-09 19:35:35.799 INFO (MainThread) [music_assistant.players] Player registered: media_player.assistente_do_escritorio/Assistente do escritÃ³rio
2024-05-09 19:38:56.386 INFO (MainThread) [music_assistant.players] Player removed: Home Theater do escritÃ³rio
2024-05-09 19:38:56.390 INFO (MainThread) [music_assistant.players] Player removed: Assistente do escritÃ³rio
2024-05-09 19:39:00.025 INFO (MainThread) [music_assistant] Loaded player provider Chromecast
2024-05-09 19:39:00.777 INFO (MainThread) [music_assistant.players] Player registered: 36b057b7-a6d4-a85f-9063-7afa9668e696/Assistente da cozinha
2024-05-09 19:39:00.906 INFO (MainThread) [music_assistant.players] Player registered: a7c034e7-b4c2-44a4-9118-19fc724852ad/Assistente do escritÃ³rio
2024-05-09 19:39:01.015 INFO (MainThread) [music_assistant.players] Player registered: e9d43003-03f1-a22b-6247-7a260e01b168/Quarto do Felipe
2024-05-09 19:39:32.701 ERROR (MainThread) [music_assistant.webserver] Error handling message: player_queues/play: Resume queue requested but queue a7c034e7-b4c2-44a4-9118-19fc724852ad is empty
2024-05-09 19:41:12.299 INFO (MainThread) [music_assistant] Loaded player provider UPnP/DLNA Player provider
2024-05-09 19:41:12.517 INFO (MainThread) [music_assistant.players] Player registered: uuid:910355b0-6785-95e2-408f-d126df8160ef/[LG] webOS TV LM6300PSB
2024-05-09 19:41:13.687 INFO (MainThread) [music_assistant.players] Player registered: uuid:0d1cef02-00dc-1000-8d3a-90f1aa5bce3d/[HTS] Home Theater
2024-05-09 19:42:10.857 ERROR (MainThread) [music_assistant.webserver] Error handling message: player_queues/play: Resume queue requested but queue uuid:0d1cef02-00dc-1000-8d3a-90f1aa5bce3d is empty
2024-05-09 19:45:09.380 INFO (MainThread) [music_assistant] Loaded player provider Home Assistant MediaPlayers
2024-05-09 19:45:09.673 INFO (MainThread) [music_assistant.players] Player registered: media_player.home_theater_do_escritorio/Home Theater do escritÃ³rio

Additional information
No response
What version of Home Assistant Core are your running
2024.5.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Looking at your video you are selecting it. When you click on a player that selects it. Back out of the player select and you should see the player you have selected indicated on the player batter at the bottom

Also don’t use the HA players if there is a native MA player available.

Looking at your video you are selecting it. When you click on a player that selects it. Back out of the player select and you should see the player you have selected indicated on the player bar at the bottom

Oh wow! That was it. I can't believe. A little suggestion is to close the player selection screen once a player is selected. ;)

Also don’t use the HA players if there is a native MA player available.

Noted. Thanks!
Feel free to close this issue if no plans to implement anything in this regard.

We won’t plan on closing that view on selection  as you might be adjusting the volumes of multiple players or linking players.
Hope you enjoy using MA


Hi,

안녕하세요. 한국말이서툴러도 이해해주세요.
NA서버가열렷는데 이거랑연결이안되니까요 이걸연결하고싶은데 코드를써도될가여?;

I am planning global region support when Amazon makes it possible. What features are you interested in?

the engraving optimization. for now, it only supports the korean server  i believe.
if u can actually create the api connects the NA Market, then that would be perfect.


attestation: handle bad configurations better

After #17760, we seem to now be forcing installing gh for everyone which I didn't initially realise based on the PR description. This can be slow for unsupported configurations as they would need to build Go from source so disable attestations for the OS.unsupported_configuration? case. (Though IMO maybe worth thinking whether we actually want to be forcing installs to 15% of users.)
Set GH_HOST explicitly which should avoid the need for gh auth switch
Silently handle 401 bad auth for now. Removing auth is a hard blocker for wider roll out so I don't think we should pain users in the meantime. We've been getting multiple reports a day on this issue and that number doesn't seem to be going down.

I wonder if we should just skip HOMEBREW_GITHUB_API_TOKEN for attestations and rely solely on gh auth token instead. (And error only if HOMEBREW_VERIFY_ATTESTATIONS is specifically requested.)

Potentially, though it's all silenced now anyway and in reality we need the auth requirement to be dropped entirely for the beta to progress any further.

Though IMO maybe worth thinking whether we actually want to be forcing installs to 15% of users.

Yes, we should I think.


Closes #122 Remove explicit dependency on the Kotlin stdlib

Hi again @DriblingTrex! Thanks for filing this PR. However, there is no point in merging this until the commit messages are linked to your GitHub account like I mentioned in your previous PRs;

https://github.com/vestrel00/contacts-android/pull/127#issuecomment-986270221
https://github.com/vestrel00/contacts-android/pull/133#issuecomment-988072393


[SPARK-41742] Support df.groupBy().agg({"*":"count"})

What changes were proposed in this pull request?
Compatibility changes to support count(*) for DF operations that are rewritten into count(1).
Why are the changes needed?
Compatibility.
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

thanks, merged into master


Dark mode - please fix checkmarks on headless CMS page

Let's make these blue in dark mode to match the other pages. Thank you!


.Net Agents: Support ablity to end / delete an AgentChat

Based on customer feedback, we might want to consider how a chat might be terminated.  For agents based on ChatHistory this might be trivial...but not-allowing assistant based agents to delete a thread impacts cost considerations ($$$).
In addition, certain agent customization may also have non-trival clean-up requirements.
This might be exposed on the AgentChat abstraction, and then each active channel would proceed to perform the appropriate clean-up / terminal action

@shreyas-pandey - Appreciate any feedback you have at this juncture!


Stop forking neauoire and use a template instead

This pull request essentially moves rotonde-portal into this repository and uses it as a template when create new portals with the onboarding site. This avoids having to fork and clean up @neauoire's portal. I've also added a "loading" state while checking to make sure all files are present before moving forward.

@neauoire I will work on the "enter" button and other form improvements tomorrow. Btw, I noticed you removed most of the :focus styles when you redesigned things. Was that intentional? If not did you have a preference how you wanted that to look? I was just going to add the outlines back on focus but figured since you removed them you might not want to go that route.


Feature schema cannot have more than one key name

Feature schema is invalid JSON, because multiple extractors have same key name. Instead, need to use an array, with objects, with a "name" attribute so that they dont clash....
For example, ClarifaiAPI

Actually, just using the object's as an array of objects might work...


Update our EsLint plugins to their latest versions

Issue Description
Our EsLint plugins our outdated. This is causing issues like this (https://github.com/evcohen/eslint-plugin-jsx-a11y/issues/214)  to remain hard to detect in the code base. To ensure we are following proper accessibility while coding, we should update our plugins.
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Once https://github.com/cerner/eslint-config-terra is released, we can remove our eslint dependencies and add the estlint-config-terra as a dev-dependency and then add to the package.json to use it.
  "eslintConfig": {
    "extends": "terra"
  },

I'm starting work on this.

Resolved in https://github.com/cerner/terra-core/pull/1775
Logged:

https://github.com/cerner/terra-ui/issues/114
https://github.com/cerner/terra-toolkit/issues/163
https://github.com/cerner/terra-dev-site/issues/107
https://github.com/cerner/generator-terra-module/issues/111


feat(dojo-lang): support components with primitive types

Resolves #87.
We can now build Components with all primitive types (felts as well as u8, u16, etc.).
Had to add a bunch of use clauses to the generated Component file. They aren't necessary if the component is only made out of felts, but we need them for the rest. Not sure if it's ok like this.
There's also one part of the code I'm not very fond of - the one where it checks if a member is a felt type (is_felt). I'm converting the syntax node into String, trimming it and comparing to "felt". I didn't find any other way how to do this. Is there a better alternative?

brb, resolving conflicts

Ok, haven't noticed the changes in #92, will take me a while to update the PR.
WIP for now 😅

@tarrencev ready for review


Require license to have scope property

Implements https://github.com/tdwg/camtrap-dp/issues/189 by requiring a scope = data or scope = media property for licenses. Notes:

licenses is not a required property (cf. Data Package and behaviour before), so it can be excluded. This is useful for e.g. exports from management systems where no license is specified.
When added, at minimum two licenses are required, because we want people to always indicate one for data AND media. @ben-norton @kbubnicki is that OK? Note that one could indicate even more licenses, e.g. if multiple licenses apply to the media files. It has the drawback that users don't know which one applies to which media files however, a drawback already discussed in #189.
When added, the license needs a scope property, which should either be data or media. Note that I don't know how to enforce both to be different. @ben-norton @kbubnicki is data a good name? Alternatives I consider less good are metadata and package.

~Note to self: only merge PR after the example package has licenses (currently it has none, which also passes validation).~ done

This PR updates https://tdwg.github.io/camtrap-dp/metadata/ as follows:

This to meet https://github.com/tdwg/camtrap-dp/issues/189#issuecomment-1015688694 suggested by @augusto-herrmann

@peterdesmet

When added, at minimum two licenses are required, because we want people to always indicate one for data AND media.
Yes

is data a good name?

Yes

@ben-norton can you accept this PR then? Review > Approve?

I see that this proposal is very specific to camtrap-dp and don't generalize well to any data package, right @peterdesmet ?
For instance, the specs assume that you necessarily have a media.csv file there as a resource.

@augusto-herrmann, that is correct: this proposal is specific to Camtrap DP. We need this change here already, but I agree that scope is general enough that I will also submit it as an idea to the frictionless specs. If adopted, then Camtrap DP will only have the additional requirement of having an enum for that field.


[lab] Fix build

Closes #23559

it doesn't seem to be enough but I don't understand why: https://codesandbox.io/s/basicdatepicker-material-demo-forked-l72my?file=/demo.tsx.


Getting Error "NameError - uninitialized constant Socket::MSG_DONTWAIT Did you mean? Socket::MSG_DONTROUTE"

Getting error while running get method for snmp v3 credential
"netsnmp/session.rb:94:in `block in write': uninitialized constant Socket::MSG_DONTWAIT (NameError)

        @socket.sendmsg(payload, Socket::MSG_DONTWAIT, @destaddr)
                                        ^^^^^^^^^^^^^^
Did you mean? Socket::MSG_DONTROUTE
     from C:/lib/netsnmp/session.rb:110:in `block in perform_io'
     from C:/ib/netsnmp/session.rb:108:in `loop'
     from C:/lib/netsnmp/session.rb:108:in `perform_io'
     from C:/lib/netsnmp/session.rb:93:in `write'
     from C:/lib/netsnmp/session.rb:88:in `send'
     from C:/lib/netsnmp/session.rb:53:in `send'
     from C:/lib/netsnmp/client.rb:57:in `block in get'
     from C:/lib/netsnmp/client.rb:157:in `handle_retries'
     from C:/lib/netsnmp/client.rb:57:in `get'"

     Version used for netsnmp : 0.7.1
     System Specification : Windows Server 2019 Datacentre

fixed with 83c3f3a1a3db976ae213638052ab71b87a89125e


Landing Page feature

It would be great if there is a function to generate Landingpages. What I mean when writing about landing pages is the following functionality:

you can create a teaser for a page
you can generate a page where you can include multiple teasers as sections of the page in an customizable order
you can include full pages as section of another page

A Teaser can have different layouts (according to the theme chosen by the user) like image on the left and text on the right or three columns of text with icons above. Each teaser provides settings like color (background, font-color, etc.). Each teaser can have a call to action text and as so can provide a link to another page or an external URL. The call to action is rendered as button according to the teaser theme
Could you tell me if one should implement this as a module/package or you can imagine to take it on your roadmap? Another option could be to create a pull-request so this is getting into the core. What do you prefer?
Thank you in advance and congratulations to the great product you released with version 1.0

+1

@LrMiLoef What do you think is the correct way to implement this? Module/Package or as Core Functionality and open a Pull-Request?

Don't know if I understood you correctly. But aren't widgets supposed to do this kind of job?

I think this should be an extension, not part of the core.

@benedu But you can just put widgets in predefined positions. What I would like to do is to allow a user to put as much widgets / teasers on a page as he wants to without having technical knowledge.
The second problem is that afaik you cannot put the same widget to different positions on different pages? Okay you could duplicate it but then your changes would be necessary for each duplicated widget again.
I am not sure how to solve this - thank you very much for your suggestion but I think it's still not exactly what fits for my needs :(


Fix - Homepage semantic headings

What
Add sections around different areas of homepage so we can have multiple semantically correct h1 headings

Cookie banner
Covid banner
In focus

How to review

Load the home page
See that the above areas have no h1
Switch to this branch
See the design is unchanged but are now using h1

Who can review
Anyone but me

Should the 'Around the ONS' also be an H1 of a new section?
And should the footer links also have their own H1 so they aren't part of the previous sections information/heading hierarchy?


Authors not visually shown

Users Page

It clearly shows who are Owners, Admins and Editors but not Authors
User Profile Page

It shows Authors as Administrators.
When you press save it doesn't make them admins which is good however feel this should be filled with author from the start.

Hi @JoeeGrigg, thanks for the report.
To summarize your first issue, you're referring to the fact that the Author users don't have a badge saying AUTHOR next to their name, while every other role has a badge, right?  We had discussions around that when it was originally implemented, and made the conscious decision to go with the current implementation.
As for your second issue, I can't reproduce it. My Author users show the correct role.  Can you check out our contributing bug reports guide and add some information on how I can get my page to show what you're seeing, as well as what version of ghost etc you're running.
Cheers

Just to follow up, that second issue was fixed in 0.5.9. @JoeeGrigg if you upgrade you should find the problem gone. :)


Allow subsets of ring systems to match templates

Reference Issue
CRDGEN-360
This makes the rings 2d coordinates templating search is run twice, first on the whole set of fused rings and then on a subset of "core" rings. Core rings are what remains after iteratively stripping away rings that share only one atom (spiro) or one bond (two consecutive atoms) with other rings.
This makes the same template usable for all the structures below

I originally removed the templating on the whole set of rings, but structures like C1CCC23CCCCC2CC2CCCC(C1)C23
(template #15) would be hard to template that way, so I opted for running the search twice

Any other comments?
This is my first PR to RDKit, I need some guidance with adding tests

I think this looks good and is ready for @greglandrum to take a look -- these changes shouldn't impact coordinate generation at all if the useRingTemplates option is off.
@ZontaNicola have you tried these changes on some other examples? Is there any performance impact?

My two cents on testing this:
It's easiest to add C++ tests in the catch_tests.cpp file. You can use the tests that are there as examples and just ask if you have questions.
The majority of the tests for the current ring-system template code are in Python, starting around here: https://github.com/rdkit/rdkit/blob/master/Code/GraphMol/Depictor/Wrap/testDepictor.py#L606
It can be easier to add Python tests, and there's definitely nothing wrong with them, but C++ tests are easier to run and iterate on when developing and fixing bugs.
It would definitely be good to include some tests verifying that the new code is actually being used and is doing the right thing. This is probably most easily done by generating coords for molecules where the pre-PR code does not use a template and then testing that a particular bond angle or torsion is what you'd expect it to be based on the template. There's an example of one of these here:
https://github.com/rdkit/rdkit/blob/master/Code/GraphMol/Depictor/catch_tests.cpp#L729
Here's the only test in C++:
https://github.com/rdkit/rdkit/blob/master/Code/GraphMol/Depictor/catch_tests.cpp#L214
I also think that it would be useful to include some unit testing for findCoreRings(). That's the central piece of the PR and is exposed in the API, so direct tests could be added.

A data point on the performance impact of this. I generated 2D coords with and without templates for the first 5K molecules from this set: https://github.com/rdkit/rdkit/blob/master/Regress/Data/znp.50k.smi.gz
Timing with master:
In [6]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
632 ms ± 682 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [5]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
725 ms ± 995 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

And with this PR:
In [10]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
611 ms ± 525 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [12]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
898 ms ± 1.32 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

So it takes the additional time for using templates from !00ms to 250-300ms. That's not insignificant.

A data point on the performance impact of this. I generated 2D coords with and without templates for the first 5K molecules from this set: https://github.com/rdkit/rdkit/blob/master/Regress/Data/znp.50k.smi.gz
Timing with master:
In [6]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
632 ms ± 682 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [5]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
725 ms ± 995 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

And with this PR:
In [10]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
611 ms ± 525 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [12]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
898 ms ± 1.32 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

So it takes the additional time for using templates from !00ms to 250-300ms. That's not insignificant.

Thanks Greg. Let me play with it a bit. I am assuming the substructure match is the bottleneck here, I think we can do a lot better in terms of avoiding unnecessary ones

on our testset (~4k molecules) these changes take the number of calls to substructure search from 436 to 103.

Also., if we decide to keep template search for the whole structure, for edge cases like the one I show in description, we should isolate those molecules into a different template database, but that sounds like a different PR

Thanks @ZontaNicola , I will take a look at it again in the next couple of days!

@ZontaNicola could what are the runtimes before and after this updated PR when running on our test set?

on our testset (~4k molecules) these changes take the number of calls to substructure search from 436 to 103.

FWIW I don't see any improvement on my test set.
Here's a re-run of my test with 5K molecules on master:
In [7]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
624 ms ± 5.39 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [8]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
713 ms ± 593 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

and then with your branch:
In [5]: %timeit [rdDepictor.Compute2DCoords(x) for x in ms]
622 ms ± 529 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [6]: %timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]
897 ms ± 7.65 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

@ZontaNicola : I'm about to make a PR against your branch with some suggested optimizations.
With these changes the timings on my office machine (the previous timings were from my home machine, which is faster) are:
# without using ring templates:
894 ms ± 6.04 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
# with ring templates:
938 ms ± 1.36 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

The timing for using ring templates on master on this machine is:
1.04 s ± 3.16 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

So with this change made, the new code is actually faster than what's on master. :-)
Here's the little benchmark (I run this in ipython so that the timeit magic works):
from rdkit import Chem
from rdkit.Chem import rdDepictor
suppl = Chem.SmilesMolSupplier('../Regress/Data/znp.50k.smi')
ms = [next(suppl) for i in range(5000)]
%timeit [rdDepictor.Compute2DCoords(x) for x in ms]
%timeit [rdDepictor.Compute2DCoords(x,useRingTemplates=True) for x in ms]

@ZontaNicola there's a merge conflict here because of changes in catch_tests.cpp.
I will do a PR against your branch with a resolution

@greglandrum sounds good! I tried to merge myself yesterday but I don't think I have write access

@greglandrum sounds good! I tried to merge myself yesterday but I don't think I have write access

@ZontaNicola it's there now. The PR is giant since your branch is far from master, but the only significant change is a modification to resolve the conflict in catch_tests.cpp

Argh...  I will try to fix this

@ZontaNicola I just made a PR against your fork that includes a fix for the problem causing the windows CI builds to fail

Thanks for seeing this one through to the end @ZontaNicola


[Enhancement] Width of TabViewItemDataTemplate

Summary
Currently it is not possible to specify the Width of TabView item when it is defined by TabViewItemDataTemplate.
API Changes
Add double TabWidth property on TabView or support Width=GridLength.Auto when TabViewItemDataTemplate is defined.
Intended Use Case
Specify Width of TabView item when defined by DataTemplate.
Currently the Width is GridLength.Star when the TabViewItemDataTemplate is defined.
Who Will Do The Work?

[ ] I am willing to take this on myself
[x] Just putting this out there for someone to pick up

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


Why can't header and Layout be functional components?

Wondering, why header and layout components are extended from React.Component?  These components are not using state or lifecycle hooks.  Why can't they be functional components?  Correct me if I am wrong.

Thanks for this comment!
You're right, I think a lot of the components could be refactored to be functional components.
I'll do this in a future update to the bootstrap branch, which I'm currently working on.
Example for reference for folks wondering what this is about:
https://hackernoon.com/react-stateless-functional-components-nine-wins-you-might-have-overlooked-997b0d933dbc

Going back on what I've said above, as things have moved on I've decided against this - the layout and header components have been refactored and I don't think it makes as much sense now (it's still possible, but then would need to explain in comments what's going on and wy).
Overall things are simpler, but there is more logic in the layout - a tradeoff for readability in an example project where the primary goal is ease of comprehension - and it's seems reasonable that people playing around with the project might want to expand it in their own projects, so leaving as a React.Component makes the most sense I think.
It's a good point though and might be good to apply elsewhere. It's certainly something that would be good to include as an example in the 'performance' documentation section, even if it's if only by way of explaining why some components are sometimes written this way now.


Update ti-indicator.md

Updated properties

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/5887

@JohanFreelancer9 Please copy review


Feat: Throttling of listener calls from solver manager

Is your feature request related to a problem? Please describe.
note problem as such but can cause too many updates when solver manager invokes listeners that must store data in 3rd party storage
Describe the solution you'd like
Configurable throttling mechanism on solver manager that allows to control the rate at which listeners are invoked with best/final solution
Describe alternatives you've considered
Handling throttling within the listener but that might not be good enough as solver might have better knowledge on when and what to send to listeners.
Additional context

Requirements given a throttling of 1 event per 5 seconds:

Never 2 events within the timespan of 5 seconds.
Skip ahead. Once a new event has been received, there is no point sending out an older event.
Given event A at 0.1s, B at 0.2s, C at 0.3s, D at 5.1s, it would typically output A and D. This means B and C never make it the client.
Delay but not eaten An event that happens too early, must still be forwarded (unless it got skip ahead).
Given event A at 0.1s, B at 0.2s, C at 0.3s and D at 999s, it should output A at 0.1s, C at 5.1s and D at 999s.

Doesn't that last case (4s termination) violate the first requirement?
Probably, we 'll need to make a decision there.

I think second approach will be better. Let's assume that we have event A at 0.1s, B at 4.9s and  C at 999s. With skip ahead we won't see any improvement in solution till 999s as B will be skipped. If you use stored data to display intermediate solution to end-user and solution B is the first one feasable (hard score == 0) missing it will be bad for UX perspective.


Update ImageSet docstring

The ImageSet documentation can be a little confusing when having custom image prefixes.
We'll clarify the expected value of the "image" field, underlining the fact that the original image names should be used, without the custom prefixes that may be applied through the Installation spec.
Description

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

/sem-approve

Can you run make manifests, that should update the file pkg/crds/operator/operator.tigera.io_imagesets.yaml and then include that update? That should fix the CI failure.

Done, thanks!

@tmjd Please retrigger the CI when you get the chance.

/sem-approve


panic: runtime error: index out of range [0] with length 0

What happened:
panic: runtime error: index out of range [0] with length 0 when dealing with files without extension.
What you expected to happen:
Support when file has no extension?
How to reproduce it (as minimally and precisely as possible):
On Windows, execute it , guaranteeing that inside a folder, there is a file without an extension.
Anything else we need to know?:
As mentioned in https://github.com/anchore/syft/issues/1024, I tried to use the exclude as a workaround but didn't work.
Environment:
Application:          grype
Version:              0.41.0
Syft Version:         v0.50.0
BuildDate:            2022-07-06T15:20:18Z
GitCommit:            0e0a9d9e7a28592db489499db0294608e5fe69b8
GitDescription:       v0.41.0
Platform:             windows/amd64
GoVersion:            go1.18.3
Compiler:             gc
Supported DB Schema:  4

Sorry for the delay here @nba1992 -- could you try the latest version of Syft and provide the stack trace output if still seeing this issue?

I had to run a vulnerability check again today.
Built:     2022-11-17 08:20:14 +0000 UTC
Schema:    5
Checksum:  sha256:cc0cfd203f5f818f23f5374defc30f7b4efc48d42b14ff0b860af16724d44730
Status:    valid
Application:          grype
Version:              0.52.0
Syft Version:         v0.60.3
BuildDate:            2022-11-03T17:15:32Z
GitCommit:            c8ddd7e218f63eb3adac1ec98ba9d8db9f3f3fec
GitDescription:       v0.52.0
Platform:             windows/amd64
GoVersion:            go1.18.7
Compiler:             gc
Supported DB Schema:  5
After changing the $env:SYFT_LOG_LEVEL = "debug", i found what was happening, and it seems that for syft for windows does not support files without extension.
I deleted the problematic files/folder, and it finish with success.
Stacktrace:
[0000]  INFO grype version: 0.52.0
[0000] DEBUG   ├── buildDate: 2022-11-03T17:15:32Z
[0000] DEBUG   ├── compiler: gc
[0000] DEBUG   ├── gitCommit: c8ddd7e218f63eb3adac1ec98ba9d8db9f3f3fec
[0000] DEBUG   ├── gitDescription: v0.52.0
[0000] DEBUG   ├── goVersion: go1.18.7
[0000] DEBUG   ├── platform: windows/amd64
[0000] DEBUG   ├── syftVersion: v0.60.3
[0000] DEBUG   └── version: 0.52.0
[0000] DEBUG gathering packages
[0000] DEBUG loading DB
[0000] DEBUG indexing filesystem path="D:\"
[0000]  WARN unable to access path="D:\System Volume Information": open D:\System Volume Information: Access is denied.[0008]  WARN unable to access path="/d/VB/crate/lib": unable to readlink for path="/d/VB/crate/lib": readlink /d/VB/crate/lib: The system cannot find the path specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/os-release" : CreateFile D:\etc: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/usr/lib/os-release" : CreateFile D:\usr: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/system-release-cpe" : CreateFile D:\etc: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/redhat-release" : CreateFile D:\etc: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/bin/busybox" : CreateFile D:\bin: The system cannot find the file specified.
[0010]  INFO could not identify distro
[0010]  INFO cataloging directory
[0010] DEBUG cataloging with "alpmdb-cataloger"
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/os-release" : CreateFile D:\etc: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/usr/lib/os-release" : CreateFile D:\usr: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/system-release-cpe" : CreateFile D:\etc: The system cannot find the file specified.
[0010] DEBUG directory resolver unable to evaluate symlink for path="/etc/redhat-release" : CreateFile D:\etc: The system cannot find the file specified.
[0011] DEBUG directory resolver unable to evaluate symlink for path="/bin/busybox" : CreateFile D:\bin: The system cannot find the file specified.
[0013] DEBUG directory resolver unable to evaluate symlink for path="D:\VB\chocolatey-tools\msys64\var\lib\pacman\local\gmp-6.2.1-1\mtree" : CreateFile D:\D:: The filename, directory name, or volume label syntax is incorrect.
panic: runtime error: index out of range [0] with length 0
goroutine 36 [running]:
github.com/anchore/syft/syft/pkg/cataloger/alpm.getFileReader({0xc0081270e0, 0x44}, {0x29cb570, 0xc00028ad20})
/Users/runner/go/pkg/mod/github.com/anchore/syft@v0.60.3/syft/pkg/cataloger/alpm/parse_alpm_db.go:110 +0x1a5
github.com/anchore/syft/syft/pkg/cataloger/alpm.parseAlpmDB({0x29cb570, 0xc00028ad20}, 0xc00d28ea28, {{{{0xc0080d2b90, 0x43}, {0x0, 0x0}}, {0x0, 0x0}, {0x18fdb, ...}}, ...})
/Users/runner/go/pkg/mod/github.com/anchore/syft@v0.60.3/syft/pkg/cataloger/alpm/parse_alpm_db.go:41 +0x13d
github.com/anchore/syft/syft/pkg/cataloger/generic.(*Cataloger).Catalog(0xc007c22810, {0x29cb570, 0xc00028ad20})
/Users/runner/go/pkg/mod/github.com/anchore/syft@v0.60.3/syft/pkg/cataloger/generic/cataloger.go:127 +0x70e
github.com/anchore/syft/syft/pkg/cataloger.Catalog({0x29cb570?, 0xc00028ad20}, 0x8?, {0xc00037ef20, 0x16, 0x0?})
/Users/runner/go/pkg/mod/github.com/anchore/syft@v0.60.3/syft/pkg/cataloger/catalog.go:54 +0x3d8
github.com/anchore/syft/syft.CatalogPackages(0xc0004fca80, {{0x1, 0x0, {0x2423d77, 0x8}}, {0x0, 0x0, 0x0}})
/Users/runner/go/pkg/mod/github.com/anchore/syft@v0.60.3/syft/lib.go:72 +0x5a5
github.com/anchore/grype/grype/pkg.syftProvider({0xc0000460b0?, 0x0?}, {0xc0006a71d0, {0x3e47a58, 0x0, 0x0}, {{0x1, 0x0, {0x2423d77, 0x8}}, ...}, ...})
/Users/runner/work/grype/grype/grype/pkg/syft_provider.go:24 +0x17e
github.com/anchore/grype/grype/pkg.Provide({0xc0000460b0, 0x8}, {0xc0006a71d0, {0x3e47a58, 0x0, 0x0}, {{0x1, 0x0, {0x2423d77, 0x8}}, ...}, ...})
/Users/runner/work/grype/grype/grype/pkg/provider.go:32 +0x20c
github.com/anchore/grype/cmd.startWorker.func1.2()
/Users/runner/work/grype/grype/cmd/root.go:342 +0x16f
created by github.com/anchore/grype/cmd.startWorker.func1
/Users/runner/work/grype/grype/cmd/root.go:339 +0x83d

Hello @spiffcs, @kzantow
We got the same error for both latest and previous releases.
Error:
[xxxx] DEBUG directory resolver unable to evaluate symlink for path="C:\msys64\var\lib\pacman\local\libp11-kit-0.24.1-4\mtree" : CreateFile C:\C:: The filename, directory name, or volume label syntax is incorrect.
panic: runtime error: index out of range [0] with length 0
Repro steps (with choco, prev. version): run workflow with job:
  Windows:
    runs-on: windows-2019
    steps:
      - name: 'Install syft'
        run: choco install syft -y

      - name: 'Creating SBOM'
        run: syft dir:C:/ -vv -o spdx-json=sbom.json

Logs: logs_74.zip
Repro steps (installed from sources, latest) : run workflow with job:
  Windows:
    runs-on: windows-2019
    steps:
      - name: 'Install syft'
        run: curl -sSfL https://raw.githubusercontent.com/anchore/syft/main/install.sh | sh -s -- -b D:/syft

      - name: 'Creating SBOM'
        run: D:/syft/syft dir:C:/ -vv -o spdx-json=sbom.json

Logs: logs_77.zip

https://github.com/anchore/syft/pull/1366 removes the panic condition. We should start seeing sane errors again for this failure. Based on the debug log posted by @erik-bershel it looks like the directory resolver needs a look so that the symlinks are being handled correctly.

Closing this since the underlying panic has been solved. Kept the replication and steps for debugging purposes on the new #1373 issue that addresses the directory resolution error case


'headers' field does not allow multiple headers with the same name

A response returns multiple Set-Cookie headers. When I wanted to see these using response.headers, only a single Set-Cookie header is shown. I have expected all Set-Cookie headers to be shown.
Is it intentional to show only one header key-value pair per header key?

Requests does show multiples, but it joins them into a single header field. This allows users to get a slightly more intuitive behaviour than returning lists, though it does reduce fidelity slightly.


Echo hangs and loops infinitely

If I echo out to a file, the command runs, but it loops infinitely and recursively writes to the target file.
Here's my use-case as to why I'm even performing an echo:
I really enjoy the safeguard in password protecting my SSH key.  Not so much for the extra 'security', but it gives me a brief moment to decide if there's something I forgot and back out of a push.  I'm in the process of automating an upstream setup using Gulp where I need to use a password-less key to handle the subtree setup.  The point of the echo is to write to the SSH config to change the key to the p/w-less one for setting, pulling and merging the subtree, then restoring it once the process completes.
This is all you need to reproduce:
GULP.task('_homez', () => {     return GULP.src('./**/**')         .pipe( PLUGINS.exec('echo test > test.txt') )         .pipe( PLUGINS.exec.reporter()); } );

./**/** grabs everything including node_modules. It ran fine for me,
but took a long time. Switch to echo test >> test.txt to see how many
times it ran. Add a !./node_modules/**/** glob or otherwise limit the
input file list so it'll do less work.
On Nov 17, 2016 5:07 PM, "Theo P."<EMAIL_ADDRESS>wrote:

If I echo out to a file, the command runs, but it loops infinitely and
recursively writes to the target file.
Here's my use-case as to why I'm even performing an echo:
I really enjoy the safeguard in password protecting my SSH key. Not so
much for the extra 'security', but it gives me a brief moment to decide if
there's something I forgot and back out of a push. I'm in the process of
automating an upstream setup using Gulp where I need to use a password-less
key to handle the subtree setup. The point of the echo is to write to the
SSH config to change the key to the p/w-less one for setting, pulling and
merging the subtree, then restoring it once the process completes.
This is all you need to reproduce:
GULP.task('_homez', () => { return GULP.src('.//') .pipe(
PLUGINS.exec('echo test > test.txt') ) .pipe( PLUGINS.exec.reporter()); } );
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/robrich/gulp-exec/issues/30, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAolfG155be6k5YshXHodz_tJOJtUl1sks5q_OvZgaJpZM4K1-CB
.

Interesting.  When I switched the command to append the echoes, I got to about 8 tests until I killed the task.  For a second I thought maybe it was because the task was trying to also run through the IntelliJ project folder, but it would still hang even with !./.idea/**/**.  I got it working, however instead of the wildcard glob, I passed src the project root ./ and the task would successfully finish.

Glad you got it solved.  I wonder if the shell was waiting to read a locked file.


feat(html,asset): improve asset resolution

Description

Expand on the default set of tags to convert into imports:
Closes https://github.com/vitejs/vite/issues/7362
Closes https://github.com/vitejs/vite/issues/5098

Use Rollup to resolve import urls (edit by @patak-dev: To be done in a separate PR)
https://github.com/vitejs/vite/issues/3000

What is the purpose of this pull request?

[x] Bug fix
[x] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

[ ]  Ideally, include relevant tests that fail without this PR but pass with it.

I didn't look further into it but if needed/wanted I can implement tests to cover the tagged issues.

Is there any update on this? Do you need some support for this?

There's a merge conflict for this PR that would need to be resolved

This looks great. Are you still willing to carry this forward, @grievouz? We would love to ship this in Vite 4.

Looks like we had discussed this in a couple meetings ago and forgot to give an update. We think this feature is great for Vite 4, but should be shipped without an option first.

Closing this one for now in favour of https://github.com/vitejs/vite/pull/11138


SQ Login TImeout

Trying to search SQ (0.3.28) and getting timeouts on login, it never succeeds, is this just me or an issue?
andrew@fp:~$ flightplan search -w SQ -f MEL -t SIN -c first -s 2018-12-06 -e 2018-12-31 -q 2 -h --force
flightplan search 0.3.28
Searching 26 days of award inventory: 12/06/2018 - 12/31/2018
[SQ] DEPARTURE [SIN -> MEL] - 2018-12-06 (2 Passengers)
[SQ] info Logging in...
[SQ] error Unexpected error occurred while searching!
{ TimeoutError: waiting for selector "#kfLoginPopup #membership-1" failed: timeout 30000ms exceeded
at new WaitTask (/usr/local/share/.config/yarn/global/node_modules/puppeteer/lib/FrameManager.js:991:28)
at Frame._waitForSelectorOrXPath (/usr/local/share/.config/yarn/global/node_modules/puppeteer/lib/FrameManager.js:882:12)
at Frame.waitForSelector (/usr/local/share/.config/yarn/global/node_modules/puppeteer/lib/FrameManager.js:836:17)
at Frame.waitFor (/usr/local/share/.config/yarn/global/node_modules/puppeteer/lib/FrameManager.js:821:19)
at Page.waitFor (/usr/local/share/.config/yarn/global/node_modules/puppeteer/lib/Page.js:1022:29)
at module.exports.login (/usr/local/share/.config/yarn/global/node_modules/flightplan-tool/src/engines/sq/searcher.js:34:18)
at
at process._tickCallback (internal/process/next_tick.js:189:7) name: 'TimeoutError' }
[SQ] DEPARTURE [SIN -> MEL] - 2018-12-07 (2 Passengers)

Working for me, when I tried just now. Can you try without -h, and verify visually if the login page is loading or not?

This is on a headless linux box so i can't run it without -h ...

Ok, I'm seeing the same timeout error when I enable headless mode, for SQ only. It's likely that SQ has figured out a way to detect that you're using a headless browser (there's a few ways they can tell, though I've patched most of them), and is blocking you on the basis of that. I'll need to dig a bit deeper, to see what's going on.

Ok thanks for confirming :)
On Fri, 7 Dec 2018 at 13:45, Jason<EMAIL_ADDRESS>wrote:

Ok, I'm seeing the same timeout error when I enable headless mode, for SQ
only. It's likely that SQ has figured out a way to detect that you're using
a headless browser (there's a few ways they can tell, though I've patched
most of them), and is blocking you on the basis of that. I'll need to dig a
bit deeper, to see what's going on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flightplan-tool/flightplan/issues/29#issuecomment-445104291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOWZdSY2G2bLB8KRct_DYjWqbzc5JPG2ks5u2dZRgaJpZM4ZHlhO
.