Dynamo Customizer to Hydra

@alfarok is possible to link Dynamo Customizer to Hydra? We can add a new key to input.json where user can share the link and then we can embed the page in Hydra. That will be pretty cool IMO.

Yes we discussed this after the hackathon. It would be very cool
-matt

On Nov 23, 2015, at 9:02 PM, Mostapha Sadeghipour Roudsari<EMAIL_ADDRESS>wrote:
@alfarok is possible to link Dynamo Customizer to Hydra? We can add a new key to input.json where user can share the link and then we can embed the page in Hydra. That will be pretty cool IMO.
—
Reply to this email directly or view it on GitHub.

This is a great idea, I will start looking into it!  I also want to get a screenshot feature for the background 3D preview as soon as possible.  There is built in functionality for capturing the canvas but not geometry.  I have messed around with it a little but haven't quite figured out how to implement it into a custom UI node.  Even as a standalone feature it's something I think many people would be excited about.

@alfarok I agree and I remember some other users asked for it on Dynamo's github. Looking forward to it.


Read data as array

Is there option for getting records from mysql as flat array rather than object. Something like this { contractor_id: 199 , contractor_name: ... }  represented as  [199,"Andrew","Banks", 88.92, 55,15].
I want to speed up reading of 75k records, now it takes 35 seconds I am wandering is array representation would be transferred to node faster as 35 seconds.

This would be a great feature. Node-pg has this, it's called a config called rowMode='array'

PRs to implement are welcome and will speed up the time-to-release significantly :)!

Kinda like fetch_row vs fetch_assoc. The array is beneficial in some cases because it can be passed directly to grid views of certain frameworks. Now the result set needs postprocessing for that.
Subscribing because interested.

Agree that this is highly desirable and has to be more efficient the more data there is.
By calling connection.query(sql, function(err, rows, flds) we can have the fields meta data so outputting fieldname for every field for every record must be slower than just sending a simple js array.
pg & firebird + maybe others have this already.

Hi @tomcon, you're welcome to work on a pull request to implement this feature! We would love it!

+1
Would be great to have :-)
JSON file would be a lot smaler. (20-30 percent smaler in my app). That's a lot when you are on bad mobile connections most of the time. The option to parse result from object to array before sending data feels like a waste of extra CPU time. (and waiting time for user)

Hi @Stavanger75, you're welcome to work on a pull request to implement this feature! We would love it!

Hi @dougwilson, thanks for the invite.  It looks like @ifsnow have allready completed the task. Thats great :-)

Indeed, @ifsnow submitted it a few days after my invite  :)

is there any update on this option?

For anyone reading this and waiting, you can use the mostly compatible mysql2 library in the meantime: https://github.com/sidorares/node-mysql2/blob/master/documentation/Extras.md#receiving-rows-as-array-of-columns-instead-of-hash-with-column-name-as-key


Have X509_load_cert_crl_file() clear error queue

Checklist

Description of change
Handle the return value from X509_STORE_add_{cert,crl}() appropriately
so that no errors leak on the OpenSSL error queue on a successful
return.
A specific error "cert already in hash table" from these functions is
ignored as there's software expecting it to skip already-loaded
certificates.

This was fixed by #2830.


Add instructions for multi-tenant login. [Closes #667]

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/authenticate-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

docs-ref-conceptual/manage-azure-subscriptions-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Rangeslider

Hello,
I am working on some time series data and I'm using Dash to make a cool app. I want to my scatter plot has rangerslider like your plot. I could not figure it how you made it. Is the rangerslider comes with candlestick plot or I can have it with the scatter plot too?
I tried this but it didn't work:
traces =[{

		'x':df.index,

		'y':df.level,

		'type': 'scatter', 'mode': 'lines',
		'name': 'a_level'
	}]
	graphs.append(dcc.Graph(
		id='a_level',
		figure={
			'data': traces,
			'layout': {
					'type': 'date',
					'rangeslider': {'visible':True},
                                        'margin': {'b': 0, 'r': 10, 'l': 60, 't': 0}
            }
		}
	)

Thank you in advance for your help

@alrghods the correct syntax would be something like:
'data': [{
     'x': df.index,
     'y': df.level,
     'type': 'scatter',
 }],
'layout': {
     'xaxis': {
         'rangeslider': {}
      }
 }

Also, see here for additional examples https://plot.ly/python/range-slider/
For future reference, it would be better posting these types of questions on the community forum https://community.plot.ly/ You'd get better traction there.


[Site support request] kocowa.com

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.07.10. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.07.10

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.kocowa.com/episode/tempted-episode-1/842290']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.07.10
[debug] Python version 2.7.10 (CPython) - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.2, ffprobe 3.2, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 842290: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 842290: Downloading webpage
[generic] 842290: Extracting information
ERROR: Unsupported URL: https://www.kocowa.com/episode/tempted-episode-1/842290
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2298, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2542, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2531, in _XML
    parser.feed(text)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: mismatched tag: line 28, column 2
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 502, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3258, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: https://www.kocowa.com/episode/tempted-episode-1/842290

Single video: https://www.kocowa.com/episode/tempted-episode-1/842290

Description of your issue, suggested solution and other information
Would love Kocowa support. Thanks!

youtube-dl.exe https://www.kocowa.com/episode/episode-itemepisodenumber/1081443 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.kocowa.com/episode/episode-itemepisodenumber/1081443', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.09.08
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: ffmpeg N-91500-g3870ed7ab3
[debug] Proxy map: {}
[generic] 1081443: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 1081443: Downloading webpage
[generic] 1081443: Extracting information
ERROR: Unsupported URL: https://www.kocowa.com/episode/episode-itemepisodenumber/1081443
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpork090uw\build\youtube_dl\YoutubeDL.py", line 792, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpork090uw\build\youtube_dl\extractor\common.py", line 502, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpork090uw\build\youtube_dl\extractor\generic.py", line 3298, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.kocowa.com/episode/episode-itemepisodenumber/1081443

Single video: https://www.kocowa.com/episode/episode-itemepisodenumber/1081443
Playlist: https://www.kocowa.com/series/hi-school-love-on/1081437

Its is Geo-restricted to USA  (not sure about what other countries it operates in)
It does need a premium account for few series / episodes but rest are free to watch with ads

is it possible to add support for this site, with only using the manifest.mpd url?  This manifest url is not geo restricted, like single video or playlist url as mentioned in the Anvithlobo post above. Because if I embed mpd manifest url to html5 player on my blog, anyone can play this video without geo restricted.

Is there any progress about this request?
Kocowa is really great


fix(docs): enclose inline code in backticks

Fixes #169
Besides enclosing inline code in backticks, I also made two tiny documentation improvements - please let me know if you want them undone:

In the Report::msg docs, the user is referred to Report::new. I changed this from plain inline code into a link.
I expanded "Display impl" to "Display implementation", since impl is a Rust token, but

"Display impl" looked silly, and
"impl Display" disturbed the flow of reading.

The CI failures confuse me. They seem unrelated?

@LeoniePhiline I think we'll land 163 then you can go ahead and rebase on master

turns out I was able to do the rebase myself easily ^_^

Thank you! ☺


Bug 1924422: vendor: bump kube 1.20.1 and gogo proto

this bump resolves https://access.redhat.com/security/cve/CVE-2021-3121[1]
[1]https://github.com/gogo/protobuf/releases/tag/v1.3.2

/bugzilla refresh

cc @soltysh

CI build errors
/retest

/test all

/hold
working through this

@hexfusion it might be a golang/protoc issue, there's a container you can use make generate-with-container

As master has been rebased onto v1.21.0-rc.0 which includes this fix, I don't think this is still required. However, we don't have a cherry-pick for the release-4.7 branch as far as I can tell, is that planned to be worked on? The patch has been released in k/api v0.20.6


Image conditioned

Hi,
I have a question how to train model with image conditioned? (use image query)
thanks

You'd have to go to the original model for this. This implementation is strictly non-conditioned.


watch() won't rebuild StreamBuilder on update

I have an app that handles form data offline and sends them to a back-end when a connection is available, so I need to show locally if the data is uploaded or not.
For this, I'm using moor_flutter and the watch() function with a StreamBuilder. My watch function is the following:
  Stream<List<Enterprise>> watchEnterprises(String query) {
    return (select(enterprises)
          ..where((enterprise) => enterprise.name.like("%$query%")))
        .watch();
  }

and my StreamBuilder looks like this:
StreamBuilder<List<Enterprise>>(
  stream: dao.watchEnterprises(_query),
  builder: (context, snapshot) {
    List<Enterprise> enterprises = snapshot.data ?? List<Enterprise>();
    return SliverList(
      delegate: SliverChildBuilderDelegate((context, index) {
        if (index == 0 ||
            enterprises[index].name[0] !=
                enterprises[index - 1].name[0]) {
          return Column(
            crossAxisAlignment: CrossAxisAlignment.start,
            children: <Widget>[
              InitialDivider(enterprises[index].name[0]),
              EnterpriseCard(enterprises[index])
            ],
          );
        }
        return EnterpriseCard(enterprises[index]);
      }, childCount: enterprises.length),
    );
  },
)

my Enterprise class has a localId and an id (returned by the back-end), so I show an indicator based on if id is null or not. For updating this id, I use the following function:
  Future setExternalId({localId, id}) => (update(enterprises)
        ..where((enterprise) => enterprise.localId.equals(localId)))
      .write(EnterprisesCompanion(id: Value(id)));

but the Stream is not auto-refreshing when I do it, and the indicator stays the same. Am I doing something wrong?

Hm no, your code looks like it should work. Can you tell me which version of moor version you're using (this is written in the .packages file)?
If you enable logStatements: true on the FlutterQueryExecutor constructor, will moor re-run the select query after the update is called?

Nevermind, the problem was that I was creating a new instance of AppDatabase by mistake, and the one updating was the new one.
Now it's working fine, but thanks for the help anyway! 😄


Update with new GeoUtils (casting single-band to 2D, implicit loading, new module names)

To fix all issues that arose with https://github.com/GlacioHack/geoutils/pull/360 (including https://github.com/GlacioHack/geoutils/pull/351).
Resolves #342
Might resolve #323, as #186 only tested errors in BlockwiseCoreg.fit() and not in BlockwiseCoreg.apply()
This PR only fixes what is in xdem/ and tests/, what is in doc/ will be done in #303 to avoid a merging nightmare.

Merging now to continue fixes linked to doc in #303.


[Question] Suggested machine and GPUs to run the training

In the https://github.com/fe1ixxu/ALMA#training- section, there's some instructions to run the

monolingual fine-tuning
full fine-tuning parallel data
LORA fine-tuning

Is there some guidance on what machines and GPUs that the various fine-tuning step should run on? E.g. with

single machine, 8xA100
single machine, 8xV100
single machine, A10g (e.g. on colab.google.com)

Does the training/fine-tuning scripts also work on multi-node instances?

Hi, thanks for the interest!
How many steps should run on?
Our 7B and 13B models are trained on 20B and 12B tokens, respectively. However, as indicated in the paper, fine-tuning 1B tokens should boost the performance substantially. The steps required to fine-tune 1 billion tokens also vary based on your batch size. In our case, the batch size is calculated as follows: 16 GPUs * 4 (batch size per GPU) * 4 (gradient accumulation steps) = 256. With a sequence length of 512, we need approximately 8,000 steps to train on 1 billion tokens, calculated as 10^9 / (256*512) ≈8000 steps. However, you may choose to fine-tune more steps to get better performance.
8 * A100

monolingual fine-tuning: a single machine of 8*A100 is good enough to train with the same settings as ours, but it needs 16K steps to finish fine-tuning 1B tokens.
stage 2 fine-tuning: one can directly use the bash files under the runs folder, where settings are the same. It will save the best model checkpoint based on the lowest loss on the dev set.

8 * V100 (32G), but only for 7B models (not recommended for full-weight fine-tuning)

monolingual fine-tuning: The batch size per device should be reduced to 1, and the user also needs to enable CPU offloading in DeepSpeed configs.
stage 2, full-weight fine-tuning: The same above.
stage 2, LoRA fine-tuning: One may train the model without CPU offloading.

A10g
We haven't tested ALMA on the A10g yet, but we'll update the information once we do!
Do the training/fine-tuning scripts also work on multi-node instances?
Yes, our code is compatible with cross-node training! Just ensure you've configured the DeepSpeed settings correctly for cross-node functionality and have installed the required launcher, such as pdsh, openmpi, or mvapich.

Hi @fe1ixxu ,
Sorry for disturbing you. My local machine is 8*A100, each of them has 40GB. I pretrained LLaMA-2 7B using the same settings as your settings but it got error overflow as below:

Exception: Current loss scale already at minimum - cannot decrease scale anymore. Exiting run.
Traceback (most recent call last):
File “run_llmmt.py”, line 227, in

Exception: Current loss scale already at minimum - cannot decrease scale anymore. Exiting run.
Traceback (most recent call last):
  File "run_llmmt.py", line 227, in <module>
    main()
  File "run_llmmt.py", line 176, in main
    train_result = trainer.train(resume_from_checkpoint=checkpoint)

How can I solve this problem? Thank you very much!


[islands] refreshed page content for 4.0

Description (required)
This refreshes the page content of /concepts/islands.mdx to go along with concept changes for Why Astro and Getting Started.
Mostly Fred's content changes, with Sarah's light editing.

LOVE this post!


Export version fixes

Features added:

Handle plugin installation on startup via file argument
Create file association during install
Suppress repeated notification titles in a single Herald voice queue

Issues addressed:

Installer now asks for install location
Filter fast spinning stellar remnants from "fast rotation" check
Refactor new "high value" check and change wording
Complete overhaul of herald cache handling to remove race condition
SSML re-ordered and namespace handling improved; fixes selected voice styles not working
Maybe have finally put to bed issue where second voice line doesn't always play
Fix explorer ssml titles placing numeric ordinals in spell-out elements.

This needs further testing to ensure it's all good, but wanted to get some eyes on it ahead of time.


Invalid vimscript expression: single-argument printf

After signing in and running :LeetCodeList:
Error detected while processing function leetcode#ListProblems:
E119: Not enough arguments for function: printf
E15: Invalid expression: printf('leetcode.get_problems(["all"])')

and, from :help printf:
printf({fmt}, {expr1} ...)				*printf()*
		Return a String with {fmt}, where "%" items are replaced by
		the formatted form of their respective arguments.

so that {expr1} seems to be required.
This is isolated and shown when running :echo printf("x"):
E119: Not enough arguments for function: printf
E15: Invalid expression: printf("x")

vim --version (Ubuntu 18.04.3 LTS):
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Jun 06 2019 17:31:41)
Included patches: 1-1453
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version with GTK2 GUI.

Thanks! It's merged.


Relations does not work when using select

Laravel Version: 8.38.0
PHP Version: 7.4.7
Database Driver & Version: mysql8.0.18

Description:
if you use ->select([]) in ORM and get data of relations with that Entity it's always null unless You remove ->select() from the ORM query chain
Steps To Reproduce:
Create models like Article and Comment and add some data using factories and database seeders and then try to query data in this manner:
$events = Article::query()             ->select(['id', 'name'])             ->with('comments')             ->get();
dd($events[0]->comments);
the result is null
after you remove the ->select() statement its working fine!

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Try :
$events = Article::query()->with('comments')->get(['id', 'name']);


Provide PHP normal wasm build

Is your feature request related to a problem? Please describe.
Hi, I tried the wasm build of python, ruby and php with my wasmer runtime, python and ruby are good, but the PHP only provide wasmedge version which could not run under the latest wasmer v4.2.3.
> wasmer python-3.11.4.wasm --dir . hello.py
Hello World

> wasmer ruby-3.2.2.wasm --dir . hello.rb
Hello World

> wasmer php-cgi-8.2.6.wasm --dir . hello.php
Status: 404 Not Found
X-Powered-By: PHP/8.2.6
Content-type: text/html; charset=UTF-8

No input file specified.

> wasmer php-8.2.6-wasmedge.wasm --dir . hello.php
error: Instantiation failed
╰─▶ 1: Error while importing "wasi_snapshot_preview1"."sock_open": unknown import. Expected Function(FunctionType { params: [I32, I32, I32], results: [I32] })

> wasmedge.exe --dir . php-8.2.6-wasmedge.wasm hello.php
Hello World
>

Can we build the normal or general PHP wasm?
Describe the solution you'd like
Build normal or general PHP wasm like python and ruby.
Describe alternatives you've considered
N.A.
Additional context
N.A.

The php-cgi CLI will need the -f option before the filename. This should work.
wasmer php-cgi-8.2.6.wasm --dir . -f hello.php

We only offer the full PHP interpreter flavored for WasmEdge as we've based it's socket implementation on top of WasmEdge sockets.


What does Pomade actually do?

I found this package by searching for python / selenium / page objects. I see references to page objects throughout the repo, but I don't see anything telling me what exactly Pomade does to help me create page objects or use them. In fact I can't really tell much at all, except from the readme, apparently this package makes it easy to run my tests on Sauce via Nose. Is there a base class here I can inherit to create a my page objects?
The only references to this package I can find are on github itself, so far. I had hoped to find a blog post or tutorial or something explaining things in a little more detail than the readme.

I think Berni is your man to go to. As far as I remember Pomade basically
added the custom functionality we now have in our
SeleniumTestCase/Selenium2TestCase objects to a page object.
On Tue, Mar 8, 2016 at 8:02 AM, Kyle Pittman<EMAIL_ADDRESS>wrote:

I found this package by searching for python / selenium / page objects. I
see references to page objects throughout the repo, but I don't see
anything telling me what exactly Pomade does to help me create page objects
or use them. In fact I can't really tell much at all, except from the
readme, apparently this package makes it easy to run my tests on Sauce via
Nose. Is there a base class here I can inherit to create a my page objects?
The only references to this package I can find are on github itself, so
far. I had hoped to find a blog post or tutorial or something explaining
things in a little more detail than the readme.
—
Reply to this email directly or view it on GitHub
https://github.com/saucelabs/pomade/issues/3.

--
Kuba Siemiątkowski Software Engineer
Twitter: @sasklacz
GitHub: @siemiatj
Sauce Labs http://saucelabs.com/ | Sauce blog http://sauceio.com/
Talk to us anytime: https://twitter.com/saucelabs
https://www.linkedin.com/company/sauce-labs
https://plus.google.com/+SauceLabs_Official/posts
https://www.facebook.com/saucelabs

However I don't think this project is maintained anymore.

@monkpit yeah this project is not actively maintained anymore. Pomade is basically a set of extracted selenium good practices (spinning asserts, page objects, helpers, custom assertions) that makes it easy to write well abstracted tests and run them on Sauce too. Unfortunately we do not have any blog posts or tutorials available at this time but I think it might be informative to go through the source code too se how abstractions are defined.


🔄 synced file(s) with worgarside/github-config-files

synced local file(s) with worgarside/github-config-files.

This PR was created automatically by the repo-file-sync-action workflow run #6922010146

This PR is empty, closing it. :no_good:


Parse failure when CSV has trailing newline

Returns {"details":"parse error (not enough input) at \"\"","message":"Cannot parse request payload"}
Lots of tools that generate CSV files add a trailing newline, so this behavior tends to be frustrating.
I can work around it by using head -c -1, but it'd be nice if I could just curl a CSV file without having to worry about it.

Yeah that is annoying. I'll make a test case, check if there's an option in our CSV parsing library to make it more forgiving, and look into a fix for that library if there isn't.

@pib sorry for the delay on this, the upstream package cassava seems like it might be unmaintained or something. They aren't responding to my issue. I bumped it one more time. If we get no answer then it might be time to look for a replacement library.

@begriffs Thanks for the update! I've managed to work around this issue any time I've run into it, but it does seem to come up at least once every time someone new writes some code that pushes data into one of our PostgREST services :D

Note to self: we need to switch from cassava to something maintained like https://hackage.haskell.org/package/csv-conduit

This seems to be resolved by now. Couldn't reproduce, csv insert with trailing new line worked just fine.


splat render looks weird on vr

I use OpenXr and when i tried to use vr with single pass it didn't work , multi pass did work but when i tried out the scene was looking weird  this only happening in splat rendering mode and not any other modes , also this does not happen with non vr camera so somethings happening with the target eye setting in the camera

I have zero VR devices around, and have not spent any time whatsoever in trying to make it work in VR/XR/AR. It might work, ir might not. There are other reports about this, closing as duplicate: #17, #26


Account switching

Description
This change includes an event listener on the ethereum provider that updates the address and JWT token when the account is changed. This event is triggered if there is more than one account connected to the Dapp, or if the user signs out and signs in with a different account.
Type

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Diligence Checklist
(Please delete options that are not relevant)

[ ] This change requires a documentation update
[ ] I have included unit tests
[ ] I have updated and/or included new integration tests
[ ] I have updated and/or included new end-to-end tests
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

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

Codecov Report
Patch and project coverage have no change.

Comparison is base (4cef197) 90.34% compared to head (c000a05) 90.34%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage   90.34%   90.34%
=======================================
  Files          25       25
  Lines        3336     3336
  Branches      270      270
=======================================
  Hits         3014     3014
  Misses        322      322

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update broke JavaScript syntax Highlighting

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I just updated my Atom editor to version 1.32.2 on osx 10.13.6 and the highlighting of my javascript code broke.
The Todo's are not highlighted like before. All JsDoc comments are just gray. And my variables have no color highlighting.
When I checked the developer tools the classes for variables are missing:

This is just a line a code like this:
const componentName = componentSplit[3]
I tried running the newest beta version of Atom, the bug still is there.
I also tried downgrading to an old version, but then I can't open the Projects because of an Error

Thanks for the report - this is related to Tree-sitter getting enabled by default in 1.32.0:
http://blog.atom.io/2018/10/23/atom-1-32.html#tree-sitter
And I believe the 3 issues you mentioned have been reported:

The Todo's are not highlighted like before.

https://github.com/atom/language-todo/issues/82 if you want to subscribe.

All JsDoc comments are just gray.

https://github.com/atom/language-javascript/issues/600 - the fix will be in 1.33.0 /cc https://github.com/atom/language-javascript/issues/600#issuecomment-439229717.

And my variables have no color highlighting.
...
This is just a line a code like this:
const componentName = componentSplit[3]

I believe this is a duplicate of https://github.com/atom/language-javascript/issues/597, see Max's comment /cc https://github.com/atom/language-javascript/issues/597#issuecomment-423629661 in particular.  If I'm mis-understanding and you're talking about a different issue, feel free to open a new issue in https://github.com/atom/language-javascript with the template filled out and we'll take another look.


Add ability to add alt text retroactively

Is your feature request related to a problem? Please describe.
Once posted, you cannot add alt text to pictures or edit the existing one.
Describe the solution you'd like
It would be nice if alt text could be added or edited retroactively.
Original idea here: https://staging.bsky.app/profile/ruben.codes/post/3jvsx5anwad26

i would appreciate a feature where others can contribute to the alt text in a way that the creator of the original post can approve, in posts where alt text is often missing.
this feature is specially helpful for people who posts a ton of images but none of which with alt text. in those cases i assume it would help more people start writing those texts on their own when posting new images


lisa.tests.base: Add df_noisy_tasks

This new interface allows to get the list of noisy tasks for a test.

part of ignore_threaded_irqs

part of ignore_threaded_irqs


@user is a scary default for user.current

Version info

intercom-rails version: 0.3.2
Rails version: 4.2.0

Expected behavior
Just wanted to highlight what, in my opinion, is a risky default for this library. I do not expect Intercom to treat a user assigned to @user in a controller as the currently logged in user.
I've remedied this by explicitly setting user.current to current_user in the initializer config, but it doesn't seem like developers should have to do that. I expect assigning something to @user to have no effect on Intercom's behavior, especially since related conversations can be a big security risk. Ultimately a miss on my part, but it seems safest to not include @user as a default for user.current.
Expected: If current_user is nil and @user is defined, Intercom will not use @user as the current user.
Actual behavior
Until today I wasn't even aware that was a default in addition to current_user. It was an issue for me when displaying a profile view for a user. When viewing a user's profile (assigned to @user) without being logged in (current_user is nil), the conversations that showed up in the Intercom widget were those of that user.
Actual: If current_user is nil and @user is not, @user is used to convey currently logged in user.
Steps to reproduce

Log out of whatever (I'm using devise)
Visit an action that assigns @user to the user you want to view the profile of and assume that it's one person viewing their friend Fred's profile
Open Intercom widget, see Fred's message history about how he reset his password and update his credit card information

I have the same issue.
User1 started a conversation with a team, than visited the profile page of User2 and wrote to team again. All old conversation of User1 passed to User2.

@kira4ka if you haven't already, you can fix this by adding
config.user.current = Proc.new { current_user } # or whatever your current user method is

to your Intercom initializer

@albatrocity, yes, I did it some hours ago and just finished to test it on different environments. Thanks.

We also missed that tidbit when setting up Intercom for a new customer. We were quite surprised to learn about this behaviour and the fact that we'd been leaking conversations to random people. I very much agree that it is too much of a stretch to assume that @user is the currently logged in user.

This should be higher priority, and is 100% a potential security risk. If you don't use current_user like @albatrocity fix above, then @user SILL overrides whatever you set in the config.user.current Proc.
Our users have the option of setting pages public page. And setting an @user ANYWHERE in that code causes intercom to leak @user messages out to the public.  I could go in and change all our @user object names, but this library shouldn't force me to do that.

This just bit us and wanted to +1 to change this behaviour or clearly warn about it in documentation and/or logs.

Also got bit by this -- +1


Implement trash for forget

Output of restic version
restic 0.12.0 compiled with go1.15.7 on darwin/amd64
What are you trying to do? What problem would this solve?
restic recover does an awesome job at finding forgotten data and adding it to a new snapshot. However, (meta)data related to the snapshot is still lost, which means that accidentally forgetting a snapshot is still a bit costly. --dry-run is great and should be best practice, but with a simple trash for snapshots, one could recover accidentally forgotten data completely with a restic recover.
What should restic do differently? Which functionality do you think we should add?

restic forget: Instead deleting a snapshot file, move it from repo/snapshots to repo/trash/snapshots(create folder if not present).
restic recover: Before doing anything else, move any files found in repo/trash/snapshots to repo/snapshots.
restic prune: Before doing anything else, delete any files found in repo/trash/snapshots

With this feature, restic recover would be able to quickly recover all forgotten snapshots with full metadata until restic prune is run.
I've tested this by manually moving snapshots around, and all commands seem to work totally fine.
Some thoughts:

This would change the repository format, which should be done only reluctantly. However, my tests suggest that the current version is completely fine with having the extra trash.
Why trash/snapshots? I'm thinking that when examining the repo, it would be obvious to a user what a trash folder would do. And although the proposal is only to allow snapshots in trash, having a subfolder for snapshots seems prudent,  clear and future-compatible.
No further changes are explicitly suggested in this feature request, but I do wonder if this would allow the default behavior of restic recover to be changed so that it would not do a full search for trees unless specified with a switch. It seems to me that forgotten snapshots would always be available and easily recoverable from trash (unless restic prune had been run, but by then the underlying data is gone anyway). The full might of restic recover could perhaps then be uncovered with a switch.

Did restic help you today? Did it make you happy in any way?
Yes, I love the forget command. I've never accidentally forgotten an (important) snapshot and I hope I never will :-)

A regular/not specifically large snapshot takes a few hundred bytes, maybe a few kBs if it contains a bunch of changes. Instead of adding a bunch of code which comes with more complexity and maintenance costs (and possibly a new repo feature), can't you just increase your retention period to solve the problem?
I'm not sure the benefit is worth the cost in additional code, when there's --dry-run and tags to use instead. To use tags, you can do the following to effectively create a trash bin of your own, without needing to modify restic:
# Mark snapshots according to the specified retention policy with a "trash" tag:
./restic -r foo tag --add trash $( ./restic -r foo forget --dry-run --group-by paths --keep-daily 3 --keep-tag trash --json | jq -r '.[].remove | select(. != null) | .[].short_id' )

# When you feel like it, "empty" the trash by removing all (but the most recent) snapshots having the "trash" tag on them:
./restic -r foo forget --group-by tags --tag trash --keep-last 1

Notice I group by paths in the forget example above, you'll of course have to modify the policy to your liking. But this should indeed give you what you're looking for, and it's really not that complex either, just one or two commands. Put it in a script :)

As I mentioned in the initial report, "--dry-run is great and should be best practice", and I agree that changing repo specification should be done in a conservative manner.
The custom trash tag is neat, but this feature request was not about the size of snapshots, or really about my specific needs. The idea was to reduce the cost of accidentaly running a forget without --dry-run. So implementing a trash in a script would not really solve this.
That said, this would definitely add some extra code. Probably at least a TrashFiles() and TrashFilesChecked() to complement the current DeleteFiles() and DeleteFilesChecked(), and they would probably have to be implemented by loading and writing, because the backends don't have any Move() functions. And then something like UnTrashFiles() (and perhaps UnTrashFilesChecked()?) and an EmptyTrash() function. These would then be called from forget, recover, and prune, respectively.
This is not something that is of maximum priority, so I'm not rushing to implement it, but I wanted to put it out there because I think it could make repository maintenance safer for users.

Totally not worth it in my personal opinion.
We can't prevent people from shooting themselves in the foot if they really want to, and any sane person should know perfectly well that they must test and/or make sure that their forget policies are actually correctly configured before they run them definitely.
Also, restic has a safety net in that it never deletes the very latest snapshot unless explicitly told to, so you will never accidentally lose your most recent backup.
And as you said, worst case there's the recover command. So there's so many things that a user must do completely wrong to lose all or part of their backups. Too much for this suggestion to be worth it IMO :)

All good points. I would be grateful if we didn't close this just yet. I'm still holding out hope that the idea will grow on y'all :-)
But if not, that's OK as well.

IMO this functionality is better implemented at the storage layer. E.g. S3 versioning, or BTRFS snapshots.

And as you said, worst case there's the recover command.

This doesn’t seem to be very useful except if you deleted just one snapshot.
I just had a failed forget where I didn’t do a dry run because reasons, but having added --group host before.
Well, now, for hosts where I backed up files and SQL dumps separately, I now have good as no backup any more.
Great. (Not.)
I still have the IDs of all pruned snapshots; with this, I could undelete them. I thought that this was already a possibility.


Add unit tests to backend source code

Add unit test to the below functionalities :

backends
health checks
http-servers

Hi , You can assign this to me ? :)


Change incorrect LdAddr to LdConst in disassembler #273_154

Motivation
Somehow LdConst is written as LdAddr in the disassembler output. This PR corrects it for clarity.

/canary

:sunny: Canary successful

/land


ISSUE-1117 Fix issue while retrieving Kafka offset metrics

topology configuration has been changed after metrics feature is implemented

pull kafka topic information from source component configuration

This closes #1117

Thanks @arunmahadevan for finding out reason of issue #1117. Really appreciated.

Thanks @HeartSaVioR , merged to master.


Does anyone can tell me how to build tesseract.framework?

I am new to this, can not follow the instruction.
Thanks.

Are you searching for the info to build it for MacOS, like you've requested in #265.
If so, pls, refer to that ticket.
I'm closing this one as a duplicate.


Upgrade from Node.js 16 to 20

When I run this I get:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

@parsley72 This is not fixed with v0.1.0, thanks for raising the issue


"Not Applicable" section does not cover some rules

In the Security Center we are seeing "Not Applicable" results that don't fall into the categories listed in the "Why does my VM show as 'not applicable' in the recommendation" section:

It's not protected by Azure Defender
It's an image in an AKS cluster or part of a virtual machine scale set
It's not running on one of the supported operating systems

Specifically, there are rules like the one in the below screenshot that list some resources as 'Not Applicable' because they don't satisfy some requirement of the rule (in this case, that the VM is internet-facing).

Our security team is bringing up the argument that results like the above that are 'Not Applicable' still need to be fixed because they don't fall into the 3 bullet points above. But even if we add a network security group above, this will still show as Not Applicable.
I think that this section should be expanded to cover results like this.

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

ID: 4fdbbf80-92cd-d6b2-85fd-a830589a8ed9
Version Independent ID: d89d9eef-daed-636d-7414-0b15f3195734
Content: Security Center's integrated vulnerability assessment solution for Azure virtual machines
Content Source: articles/security-center/deploy-vulnerability-assessment-vm.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

Thanks for the feedback.
The section you referenced is specifically about ONE recommendation: the one the whole page is discussing, namely "A vulnerability assessment solution should be enabled on your virtual machines."
It's not a blanket discussion about every not applicable tab, for every recommendation details page.
Hope that helps clear things up.
Feel free to reopen this issue or create a new one of we can be of further assistance.
#please-close


Fixing typo

@JSeb225 : Thanks for your contribution! The author, @mamccrea, has been notified to review your proposed change.


Not working with 0.219.14 - Bog Witch

Not working for me under the new Bog Witch update, Game is unplayable, and I get this exception in the BepInEx window:
[Info   :   BepInEx] Loading [More Gates Extended 1.0.2]
[Info   :Jotunn.Main] Initializing LocalizationManager
[Info   :Jotunn.Main] Initializing PieceManager
[Error  : Unity Log] MissingFieldException: Field not found: System.Collections.Generic.List1<string> .Hud.m_buildCategoryNames Due to: Could not find field in class Stack trace: Jotunn.Managers.PieceManager.AddPieceCategory (System.String name) (at D:/Projects/Jotunn/JotunnLib/Managers/PieceManager.cs:294) Jotunn.Entities.CustomPiece.set_Category (System.String value) (at D:/Projects/Jotunn/JotunnLib/Entities/CustomPiece.cs:44) Jotunn.Entities.CustomPiece..ctor (UnityEngine.AssetBundle assetBundle, System.String assetName, System.Boolean fixReference, Jotunn.Configs.PieceConfig pieceConfig) (at D:/Projects/Jotunn/JotunnLib/Entities/CustomPiece.cs:201) MoreGatesExtended.MoreGatesExtended.LoadAsset (System.String name, Jotunn.Configs.RequirementConfig[] requirements) (at <b96e2637a5074fbab9c53fabb84f0117>:0) MoreGatesExtended.MoreGatesExtended.RegisterPrefabs () (at <b96e2637a5074fbab9c53fabb84f0117>:0) MoreGatesExtended.MoreGatesExtended.Awake () (at <b96e2637a5074fbab9c53fabb84f0117>:0) UnityEngine.GameObject:AddComponent(Type) BepInEx.Bootstrap.Chainloader:Start() UnityEngine.GameObject:.cctor() PlatformTools.Common.SingletonMonoBehaviour1:get_Instance()
PlatformTools.PlatformManagerInitializer:OnRuntimeMethodLoad()
Let me know if you need more

What's the version of your Jotunn?

Jotun is <IP_ADDRESS>

Don't worry, just continue using the mod and enjoy.


feat: feature flags: Add feature flag config for Octuple

Add a feature flag config for octuple which can provide a mechanism to provide default behavior for Octuple components.
SUMMARY:
This adds a feature flag config, and creates one flag that can control the way that Panels are rendered
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[ ] Bugfix Pull Request
[X] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[X] I have added unittests for this change

TEST PLAN:
Open the panel in storybook, and toggle the lazyLoadPanelContent flag in the settings. You should be able to see that the panel contents are rendered or not rendered even when the panel is not visible

I'm curious if this is the direction we want to go in for shipping Octuple changes faster.
I can see how feature flags can help land breaking changes into Octuple without massive efforts on regression testing, but I think we're also increasing our surface area for testing in the future.
Also, it might be a bit more complex to debug if component-level props are being ignored for context-level props.
If we decide we want to continue with feature flags though, I think we should:

Separating it out a bit more by component name might make it easier to follow & make it clear which feature flags pertain to look for if debugging is necessary.
Add JSDoc comments to each feature flag to indicate what it does / when to use it.
e.g.

export interface FeatureFlags {
  panel?: {
    /**
     * If `true`, the panel content will render ONLY when `visible` is `true`.
     * Otherwise, the panel content will default to the `renderContentAlways` value.
     *
     * < some information about when/why this should be used >
     */
    lazyLoadContent?: boolean;
  }
}

:shipit:


Add the ability to Steps.IO.Files renames the file named the value in PipelineContext.Items["SOURCE_FILE_NAME"] to the name contained in PipelineContext.Items["TARGET_FILE_NAME"]

MoveFileStep can handle the rename function.


style(GUI): improve unsafe mode dialog button labels

When a user attempts to enable unsafe mode, the dialog offers two
options: "YES, CONTINUE", and "CANCEL".
In order to improve the user experience, we changed those labels to
"ENABLE UNSAFE MODE", and "STAY IN SAFE MODE".

WarningModalService.display() now takes an object including
message, confirmButtonMessage, and rejectButtonMessage
properties.

Fixes: https://github.com/resin-io/etcher/issues/729
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

I think "stay in safe mode" still looks too much like it is disabled. There needs to be more contrast between background and text.

Exactly, the original request in #729 was to change the colours, not necessarily change the button labels ;-)

P.S. Now that #821 has been merged, it's actually impossible to exactly replicate the screenshot from the first comment in this thread ;-)

https://github.com/resin-io/etcher/issues/729#issuecomment-258049364

The screenshot has been updated!


Opening .ipynb from Git Viewlet Fails on Latest Insiders Build

Opening an .ipynb file from the Git viewlet results in the following exception being thrown:
Unable to open 'sop003-view-bdc-pod-status.ipynb (Untracked)': this.textEditorModel.onDidStateChange is not a function.

Fixed by #7403.


Enter key did not insert new line

On Android emulator it works. But not on device. RN 0.43.4. Probably this issue?
https://github.com/facebook/react-native/issues/12717

Until they fix it, add this as solution. From renehauck:
onSubmitEditing={() => {
if (!this.state.postText.endsWith("\n")) {
let postText = this.state.postText;
postText = postText + "\n";
this.setState({ postText: postText })
}
}}

I tried the other solution (on the same post) that add new line between words, and it works "ok".
But another problem arises now: the autogrow doesn't work. I noticed that the onChange events isn't dispatching, so i forced it:
<AutoGrowingTextInput
    ref="descriptionTextInput"
    style={styles.textInput}
    blurOnSubmit={false}
    onChangeText={(text) => {
        this.setState({ text });
    }}
    onChange={(event) => {this.nativeEvent = event.nativeEvent}} //save the nativeEvent to pass as parameter later
    value={this.state.text}
    onSelectionChange={(event) => this.setState({ cursorPosition: event.nativeEvent.selection.start })}
    onSubmitEditing={this.onSubmitEditingDescription.bind(this)}
    />

On constructor:
constructor(props) {
        super(props);

        this.state = { text: ''};

        this.onSubmitEditingDescription = debounce(this._onSubmitEditingDescription, 100, true); //prevent android to dispatch the event twice (google it)
    }

Now the function that add new line and force the new height:
_onSubmitEditingDescription(event) {
            const { text, cursorPosition } = this.state;

            let newText = text;
            const ar = newText.split('');
            ar.splice(cursorPosition, 0, '\n');
            newText = ar.join('');

            this.setState({ text: newText });
            this.refs.descriptionTextInput._handleNativeEvent({contentSize: {height: this.nativeEvent.contentSize.height + 30}});
    }

I used +30 because this size fits to my needs.
What do u think? Is there some other smart way to solve this?

hey guys, thanks for reporting about it.
I pushed a fix that I hope will solve it for your case as well.
available in version 4.0.0

artald, thanks for your push!
I tested here and i've got exactly the same behavior described above. I removed the line that I'm forcing the _handleNativeEvent expecting the textinput's height to be updated correctly, but nothing happens.
I still have to force the event. Am I doing something wrong?
My test scenario was: a clean  with the workaround to add new line.

"react": "16.0.0-alpha.6"
"react-native": "0.44.0"

@vendramini that's strange, maybe something in your layout is breaking it. We saw different results between iOS and Android where a certain layout was fine for iOS but broke Android.
Can you try to run the example project and see if it works for you?

Well. I updated autogrow to the latest. Still the same issue on my users' android device. I gave up. I just tell the users to download and use Gboard (Google Keyboard). Insert new line for multiline textInput just works with Gboard on android.

@manrashid, it's not an autogrow-textinput issue, this new line's problem came from RN, for whatever reason.
@artald, I ran the example and it works as expected without the workaround for new line. With the workaround, the component isnt updated. Try yourself, it's exactly your example plus the workaround:
import React, {Component} from 'react';
import {AppRegistry, StyleSheet, Text, View, TouchableOpacity, Platform} from 'react-native';

import {AutoGrowingTextInput} from 'react-native-autogrow-textinput';
import debounce from 'debounce';

class example extends Component {
    constructor(props) {
        super(props);
        this.state = {textValue: 'My initial\nText', shortProductDescription: ''};
        this.onSubmitEditingDescription = debounce(this._onSubmitEditingDescription, 100, true);
    }

    _onSubmitEditingDescription(event) {
        const { shortProductDescription, cursorPosition } = this.state;

        let newText = shortProductDescription;
        const ar = newText.split('');
        ar.splice(cursorPosition, 0, '\n');
        newText = ar.join('');

        this.setState({ shortProductDescription: newText });
        this.refs.descriptionTextInput._handleNativeEvent({contentSize: {height: this.nativeEvent.contentSize.height + 30}});
    }

    render() {
        return (
            <View style={styles.container}>
                <Text style={styles.welcome}>
                    Auto Growing TextInput Example
                </Text>
                <View style={styles.textInputContainer}>
                    <AutoGrowingTextInput
                        style={styles.textInput}
                        ref="descriptionTextInput"
                        blurOnSubmit={false}
                        onChangeText={(text) => {
                            this.setState({ shortProductDescription: text });
                        }}
                        onChange={(event) => {this.nativeEvent = event.nativeEvent}}
                        value={this.state.shortProductDescription}
                        onSelectionChange={(event) => this.setState({ cursorPosition: event.nativeEvent.selection.start })}
                        onSubmitEditing={this.onSubmitEditingDescription.bind(this)}
                    />
                    <TouchableOpacity style={styles.button} onPress={() => this._resetTextInput()}>
                        <Text>Clear</Text>
                    </TouchableOpacity>
                </View>
            </View>
        );
    }

    _onChange(event) {
        this.setState({ textValue: event.nativeEvent.text || '' });
    }

    _resetTextInput() {
        this.refs.descriptionTextInput.clear();
        this.refs.descriptionTextInput.resetHeightToMin();
    }
}

const IsIOS = Platform.OS === 'ios';
const styles = StyleSheet.create({
    container: {
        flex: 1,
        alignItems: 'center',
        backgroundColor: '#76c6ff'
    },
    textInputContainer: {
        flexDirection: 'row',
        paddingLeft: 8,
        paddingRight: 8
    },
    welcome: {
        marginTop: 100,
        fontSize: 20,
        textAlign: 'center',
        margin: 10
    },
    textInput: {
        paddingLeft: 10,
        fontSize: 17,
        flex: 1,
        backgroundColor: 'white',
        borderWidth: 0,
        borderRadius: IsIOS ? 4 : 0,
    },
    button: {
        paddingLeft: 5,
        alignItems: 'center',
        justifyContent: 'center',
    }
});

AppRegistry.registerComponent('example', () => example);

Type something, then add breaks between words. The height is not updated, so i've to force the event as I said above.
Thanks!

@vendramini

it works as expected without the workaround

So I'm not sure i completely understand - if it works, then why add the workaround? :)
Am I missing something here?

@artald, maybe is ambiguous the way I put. When I said "it works" I mean: the text input is auto growing. But the problem still: RN isn't adding new line when press . So we need to add the workaround. When added and we press break, the height is not updated.
See: https://github.com/facebook/react-native/issues/12717
How are you dealing with breaks in multiline text input?

@vendramini okay, now I got what you're saying. Yes, I guess it's a new bug in RN.
We've just started an upgrade process from RN38 to RN44, so don't know how to handle this just yet.

@artald ok, for now I'm going to use the workaround as suggested on that link plus forcing your updated event, It's working well, but not coded as I would like.
Some news about that I back here.
Thanks!


How to prevent rebase of images in css?

I'm facing the following problem:
We have a big angular project with a lot of lazy loading, including CSS for each module. When gulp-clean-css is executed, it adds a crazy path to the images. We want to prevent it without having to edit all the CSS (it's a log) to set absolute positioning.
How can I prevent clean-css to rebase my images using gulp-clean-css?

@phackwer thank you for the interest, but I'd recommend asking a question on StackOverflow. If you include an isolated working example you'll surely get an answer the same day. Closing for now as I'm not sure the solution lies within modifying the repo itself, but please reopen should you suspect otherwise.


fix: Styling issues with the buttons on the platform

Description

[x] Update statbox widget icon (It was using the default container widget icon)
[x] Fix button styling in the platform (Datasource edit page)

Type of change

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

How Has This Been Tested?
Visually confirmed
Checklist:

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

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/button-style-issues

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.91 (0)
36.65 (-0.17)
34.32 (0)
55.43 (0)

:red_circle:
app/client/src/components/designSystems/appsmith/BaseButton.tsx
28.89 (1.86)
1.44 (-8.82)
0 (0)
35.14 (2.88)

:sparkles: :new:
app/client/src/widgets/FilePickerWidgetV2/constants.ts
100
100
100
100

:sparkles: :new:
app/client/src/widgets/FilePickerWidgetV2/index.ts
100
100
100
100

:sparkles: :new:
app/client/src/widgets/FilePickerWidgetV2/component/index.tsx
52.38
0
20
50

:sparkles: :new:
app/client/src/widgets/FilePickerWidgetV2/widget/index.tsx
34.65
0
20.69
35.05

:x:
app/client/src/widgets/FilePickerWidgetV2Widget/constants.ts
100
100
100
100

:x:
app/client/src/widgets/FilePickerWidgetV2Widget/index.ts
100
100
100
100

:x:
app/client/src/widgets/FilePickerWidgetV2Widget/component/index.tsx
52.38
0
20
50

:x:
app/client/src/widgets/FilePickerWidgetV2Widget/widget/index.tsx
34.65
0
20.69
35.05

@riodeuno Verified the changes and they are working as expected now


 For implementing Attribute.flags(), to clarify, are you talking about:

Moving BitFlags type to core/src/lib and importing that into db/src/db
Provide an Attribute.flags() function that returns a bitfield containing all of the flags (index_vaet, etc) that we want in the bitfield?

Provide an Attribute.flags() function that returns a bitfield containing all of the flags (index_vaet, etc) that we want in the bitfield?

@jsantell, I was thinking of this.

@ncalexan  Ok, made the type changes and flags() helper -- good for review

Also, check the tests of all the crates -- Travis is angry :)


Duplicate folders in Folder Explorer

Type: Bug
In my Folder Explorer for the dev container I have two folders named SEEM
If I try to rename one of them it shows a red error that the folder already exists and to choose another location.

One of the folders started as empty as part of the repo. Then as part of the repo setup, we clone the [SEEM (Segment Everything Everywhere Model)](https://github.com/UX-Decoder/Segment-Everything-Everywhere-All-At-Once_ into the SEEM folder.
I didn't think this state should be possible.
 Do you have a simple script?

how can I end up with two folders with the same name

This is the bug. It should not be possible. I don't know how it occurred either.

Do you have a simple script?

No

This is the bug. It should not be possible. I don't know how it occurred either.

Yes I understand. If you repeat the same steps do you end up in the same state?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


implementation Classification ZOI

Pull Request Template
Types of changes
What types of changes does your code introduce to Woosmap Geofencing?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Before you submit a pull request, please make sure you have to following:

[X ] I've read contributing guidelines
[X] My code contain tests relevant for the problem I'm solving
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation (if appropriate)

Il serait bien de rajouter un effet visuel et un blocage lorsqu'on appui sur le bouton de test pour éviter un double appui (qui régénère les zones) et l'incompréhension de l'utilisateur

Oky pour moi, fonctionne sur simulateur


Set up coveralls for code coverage collection and reporting

@akosthekiss I'd like to add test coverage reporting to this repo. I've used https://coveralls.io/ in other JS/TS projects. It's free for open source projects. Someone that is an admin on this github org will have to log in with https://coveralls.io/ and enable it.
I can take care of the rest (adding the relevant pieces to .travis.yml, package.json, ...)

@martijnthe I've enabled coveralls for the repo.

Thanks, will integrate it once the first feature PR lands (https://github.com/jerryscript-project/jerryscript-debugger-ts/pull/4)


User guide - sample code fixes (misleading indentation, copy&pase errors...)

TextField's variable used for regexp pattern was named mail, renamed to regExpPatternTxt for consistency.
Edit: did some sample code formatting

Just to clarify, I am not simply converting spaces to tabs or vice-versa to my liking. I fix indentation when a code in one scope has different levels of indentation, which makes it hard to read.

Thanks!
I've merged the PR but I had to squash it to be able to merge it to 6.x/7.x and now GitHub doesn't detect it as merged.
Please close it because we don't have permissions!

Changes have been merged, closing the PR as requested.
Thanks.


Allow User to set output property of Current State node

Thinking some more about the Current State node, I think it might be useful to allow the user to specify where the output of this node should be stored.
I have removed the pervious "Override Payload" approach and now allow the use to store the output in either the message, the flow context, or the global context.

In the current version of this, only the status attribute is passed to the msg.payload. In this version the entire obiect is passed to the output.
Would it be useful to allow the user to specify which (or all) attributes of the object to return?


Qt 6 support

Hi!
Because qdoc now ships with the latest Qt beta, and is vastly better the the 5.x version, I wanted to install Qt6 via aqt but was unable to do so.
`

PS C:\aqt> aqt install --outputdir c:\aqt 6.0 windows desktop win64_msvc2019_64
2020-11-14 19:11:49,190 - aqt - INFO - aqtinstall(aqt) v0.9.8 on Python 3.9.0 [CPython MSC v.1927 64 bit (AMD64)]
2020-11-14 19:11:49,402 - aqt - ERROR - Download error when access to https://download.qt.io/online/qtsdkrepository/windows_x86/desktop/qt6_60/Updates.xml
Server response code: 404, reason: Not Found

`

The Qt Company changes Qt6 version scheme to 6.0.0 from 6.0 in 13, Nov.

Yes this does work, thanks! Maybe add this to the readme? 👏👍

@kelteseth You are welcome to add it to README through Pull-Request as usual.

Repository is updated


Simplify semantics for "no-op" master arbitration updates

See issue #361.
This text seems to have originally come from a large commit from two years ago fixing master arbitration updates: https://github.com/p4lang/p4runtime/commit/c3f6bdc3499910f4ba85c24215e7ba6fe579e414

Apologies for the delay, I will take a look tomorrow.


Proposal: homepage field in user profile

Hi, at shopkick we'd like our users to have a personalized homepage. At the bare minimum we would like each team to have the list of their dashboards only (while now all users see all dashboards, but we have too many).
We were thinking to build a generic homepage field in the user profile, that can be used in different ways. For example, we will fill it with the link to the search results of "owner=team". But this solution is more flexible, other people may prefer to specify a dashboard, anther search result, or even leave the current welcome (default). Also, based on the permission system, this field can or can't edited directly by the user herself, or only by admins.
If you think this is relevant and/or there could be interest, I can try working on that and contribute a PR within the next 10 days or so. Or if you have other ideas/requirements we can discuss the here.

We've been talking about a customizable Welcome page here too! Some of it is about:

curation: favorites dashboards & slices
surfacing popular/relevant/recent content
for content creators: showing usage and spinoffs of the content they created
an activity log, maybe similar to github's user profile page, maybe more relevant when looking at other people's page. I guess the "User Profile Page" is different to the "User Welcome Page", but somewhat related

There may be a need to summarize key usage information from Caravel's internal logs to summary table that we can use to build these pages. Things like "7day views", "28 days views", live to date views, distinct users views.
We've gotten pretty peculiar about writing/rewriting everything in React/ES6 with pretty high standard (test coverage, strict linter, ...), a great opportunity to learn if you're ready to comply to our new high standards :).
@ascott had some thoughts on this. Let's start the brainstorm / mocks right here!

BTW, somewhat related is the fact that if we'd have a way to have and cache thumbnails of all slices and dashboard we could have some beautiful gallery-type views to surface faves and suggested content. The whole headless-browser and image rendering thing is a headacke that would be nice to sort out.

Oh that's super cool!
Let me specify a bit better our use case, so we can gather all requirements.
We know that Flask App Builder doesn't support groups -- however we work in teams. We have created a "fake" user for each team, with emails like<EMAIL_ADDRESS>Then, each user that creates a new dashboard also adds her team to the owners. This way, we can easily list all dashboard of a team X, by filtering owner in<EMAIL_ADDRESS>What typically happens is that a user (think of manager/C-level) requests a dashboard, someone else in the company creates it, adding the right team as an owner, and the person who requested can see the dashboard without any further action, no favorite, no activity, etc.
Right now, however, people still have to enter the right search url, and we would like to make that url their default homepage.
In some sense the 2 things seem orthogonal to me. What I was proposing is an homepage field in the user profile, such that one can write any url there. What @mistercrunch is proposing is a more personalized homepage, but let say same url for everybody.
I think the 2 solutions can also co-exists. Clearly if the personalized homepage works well, there's no need to change the url, but I can still see use cases, e.g. for external users/partners/or specific people inside the company where you just want a different behavior, thus a different url. Make sense?

@mistercrunch
These are features that our team is actively thinking of extending!  It would be very helpful to know when these will be made generally available?

I landed a profile page recently which tackles some of this. We were thinking about making the current profile page the Welcome page, but it could also be its own thing and re-use many of the react components from the profile page.

@ecesena , I recently became a maintainer of Flask App Builder, we could add support for groups there.


[Bug]: Using both image: and build: in a Docker Compose service breaks rolling updates

Description
If an image: "image" is specified in a docker compose service together with a build: instruction (meaning "build the container from a dockerfile and store it as "image""), rollback images are not created
Background: We use a docker compose setup, where services reuse images built in previous steps. Thus we do need to use build: and image: tags in one and the same service to be able to refer to the image in a later service.
Note: This might (!) be related to https://github.com/coollabsio/coolify/discussions/2824, but my point here is we are using/setting image (!) names, and not container names. IMHO the conclusion in the mentioned discussion might be erroneous.
Minimal Reproduction (if possible, example repository)
Create a resource of type Private Repository with the Docker Compose build pack.
This docker-compose.yml generates a rollback image correctly:
services:
    shell:
        build:
            context: .
            dockerfile: docker/Dockerfile

This docker-compose.yml does not generate a rollback image:
services:
    shell:
        image: local/shell:latest
        build:
            context: .
            dockerfile: docker/Dockerfile

Exception or Error
There is no rollback image stored.
Version
v4.0.0-beta.332
Cloud?

[ ] Yes
[X] No

im having the same issue!


Provide JWT_ALGO/KEY to the API

Great to issue custom tokens that doesn't cover the normal user registration/login flow.

Available as env vars:
NHOST_JWT_ALGORITHM
HOST_JWT_KEY

Available as env vars:
NHOST_JWT_ALGORITHM
HOST_JWT_KEY


AMBARI-24612 Host Level Maintenance mode is not working through Ambari UI

How was this patch tested?
21817 passing (31s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3896/
Test PASSed.


Update defaults for modelmesh

Ref: https://issues.redhat.com/browse/RHOAIENG-126

https://github.com/red-hat-data-services/rhods-operator/blob/6eb6d4a768fa9c48b26c351e72a063c663e72c10/bundle/manifests/rhods-operator.clusterserviceversion.yaml#L128 should need updated too ?

Added another commit to set kserve defaults explicitly.
Also csv is already updated - https://github.com/red-hat-data-services/rhods-operator/pull/143/files#diff-8ec76fb27d43278dfd517742bfaedb5c3b57ff3f64965cb6a97c35266316878eR35


Apache default index.html with ssl vhost

OSX 10.11.5
Vagrant 1.8.4
Ansible <IP_ADDRESS>
Trying to provision a site using the latest master 85f602 and everything appears to go fine until I try to add an ssl vhost.
Without the ssl vhost in the config.yml the dashboard and adminer pages work as expected. When I add the following to my config.yml immediately above the non-ssl vhosts I get the Apache2 Ubuntu Default Page for every domain.:
apache_vhosts_ssl:
  - servername: "{{ drupal_domain }}"
    documentroot: "{{ drupal_core_path }}"
    certificate_file: "/etc/ssl/certs/ssl-cert-snakeoil.pem"
    certificate_key_file: "/etc/ssl/private/ssl-cert-snakeoil.key"
    extra_parameters: |
          ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000{{ drupal_core_path }}"

I get the default /var/www/html/index.html page for all domains.
Heres my working non-ssl sites-enabled.vhosts
DirectoryIndex index.php index.html

<VirtualHost *:80>
  ServerName example.dev
  ServerAlias www.example.dev
  DocumentRoot /var/www/example/docroot

  <Directory "/var/www/example/docroot">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/var/www/example/docroot"

</VirtualHost>

<VirtualHost *:80>
  ServerName adminer.example.dev
  DocumentRoot /opt/adminer

  <Directory "/opt/adminer">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/opt/adminer"

</VirtualHost>

<VirtualHost *:80>
  ServerName xhprof.example.dev
  DocumentRoot /usr/share/php/xhprof_html

  <Directory "/usr/share/php/xhprof_html">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/usr/share/php/xhprof_html"

</VirtualHost>

<VirtualHost *:80>
  ServerName pimpmylog.example.dev
  DocumentRoot /usr/share/php/pimpmylog

  <Directory "/usr/share/php/pimpmylog">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/usr/share/php/pimpmylog"

</VirtualHost>

<VirtualHost *:80>
  ServerName <IP_ADDRESS>
  ServerAlias dashboard.example.dev
  DocumentRoot /var/www/dashboard

  <Directory "/var/www/dashboard">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/var/www/dashboard"
DirectoryIndex index.html

</VirtualHost>

And here is the non-working sites.enabled.conf after I add the ssl vhost
DirectoryIndex index.php index.html

<VirtualHost *:80>
  ServerName example.dev
  ServerAlias www.example.dev
  DocumentRoot /var/www/example/docroot

  <Directory "/var/www/example/docroot">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/var/www/example/docroot"

</VirtualHost>

<VirtualHost *:80>
  ServerName adminer.example.dev
  DocumentRoot /opt/adminer

  <Directory "/opt/adminer">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/opt/adminer"

</VirtualHost>

<VirtualHost *:80>
  ServerName xhprof.example.dev
  DocumentRoot /usr/share/php/xhprof_html

  <Directory "/usr/share/php/xhprof_html">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/usr/share/php/xhprof_html"

</VirtualHost>

<VirtualHost *:80>
  ServerName pimpmylog.example.dev
  DocumentRoot /usr/share/php/pimpmylog

  <Directory "/usr/share/php/pimpmylog">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/usr/share/php/pimpmylog"

</VirtualHost>

<VirtualHost *:80>
  ServerName <IP_ADDRESS>
  ServerAlias dashboard.example.dev
  DocumentRoot /var/www/dashboard

  <Directory "/var/www/dashboard">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/var/www/dashboard"
DirectoryIndex index.html

</VirtualHost>

<VirtualHost *:443>
  ServerName example.dev
  DocumentRoot /var/www/example/docroot

  SSLEngine on
  SSLCipherSuite AES256+EECDH:AES256+EDH
  SSLProtocol All -SSLv2 -SSLv3
  SSLHonorCipherOrder On
  SSLCompression off
  SSLCertificateFile /etc/ssl/certs/ssl-cert-snakeoil.pem
  SSLCertificateKeyFile /etc/ssl/private/ssl-cert-snakeoil.key

  <Directory "/var/www/example/docroot">
    AllowOverride All
    Options -Indexes +FollowSymLinks
    Require all granted
  </Directory>
  ProxyPassMatch ^/(.*\.php(/.*)?)$ "fcgi://<IP_ADDRESS>:9000/var/www/example/docroot"

</VirtualHost>

Any idea where I am going wrong here?  I had this working on an older vm.

Issue is fixed with an apache2 restart.
sudo service apache2 restart

vagrant halt and vagrant up also fixes the issue.  Looks like ssl adds the need for an extra apache restart.

@jamesrward - Interesting! Any time a vhost is modified (even on the initial generation of the VM), it should trigger the 'restart apache' handler: https://github.com/geerlingguy/ansible-role-apache/blob/master/tasks/configure-Debian.yml#L38
I'd be interested to know if you vagrant destroy the VM to delete it entirely, then rebuild again with vagrant up, if the same thing happens. If the initial provision fails midway through, it can sometimes lead to Apache not being restarted the first time, so the config change isn't necessarily picked up.

Bingo.  I'm getting an error on the first provisioning attempt with selenium enabled:
FAILED! => {"changed": false, "failed": true, "msg": "Error when trying to enable selenium: rc=1 selenium.service is not a native service, redirecting to systemd-sysv-install\nExecuting /lib/systemd/systemd-sysv-install enable selenium\ninsserv: warning: script 'selenium' missing LSB tags and overrides\nupdate-rc.d: error: selenium Default-Start contains no runlevels, aborting.\n"}

Subsequent provisions on the same vm appear to finish cleanly.
Destroyed and commented out selenium and the next provision worked fine as did ssl.  Looks like this is actually an issue with selenium.

It looks like this is upstream issue https://github.com/arknoll/ansible-role-selenium/issues/7
Closing this issue in favor of that one.

@arknoll updated the Selenium role and tagged 1.3.0, so I've updated the requirements so it will be the default version in future Drupal VM releases.

I got failed on first time vagrant up command
After the successful attempt of second and third time I got a Apache2 default index page this issue is fixed by restarting apache service using this command sudo service apache2 restart.


tests:drivers_can_api: mimxrt1060 can api test meet assert failure

Describe the bug
test_filter_attach ASSERTION FAIL [!FLEXCAN_IsMbOccupied(base, mbIdx)] @ WEST_TOPDIR/modules/hal/nxp/mcux/drivers/imx/fsl_flexcan.c:1734 E: r0/a1: 0x00000004 r1/a2: 0x000006c6 r2/a3: 0x00800000 E: r3/a4: 0x60004121 r12/ip: 0x00000000 r14/lr: 0x600064a3 E: xpsr: 0x41000000 E: Faulting instruction address (r15/pc): 0x6000a876 E: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0 E: Current thread: 0x80000218 (unknown) E: Halting system
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -DBOARD=mimxrt1060_evk ..
make flash --runner=pyocd
See error

Expected behavior
test sucessfully
Impact
can operation
Logs and console output
test_filter_attach ASSERTION FAIL [!FLEXCAN_IsMbOccupied(base, mbIdx)] @ WEST_TOPDIR/modules/hal/nxp/mcux/drivers/imx/fsl_flexcan.c:1734 E: r0/a1: 0x00000004 r1/a2: 0x000006c6 r2/a3: 0x00800000 E: r3/a4: 0x60004121 r12/ip: 0x00000000 r14/lr: 0x600064a3 E: xpsr: 0x41000000 E: Faulting instruction address (r15/pc): 0x6000a876 E: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0 E: Current thread: 0x80000218 (unknown) E: Halting system </error>

Environment (please complete the following information):

OS: (e.g. Linux, )
Toolchain (Zephyr SDK, ...)
Commit SHA or Version used:
zephyr-v2.4.0-3224-gc77ae15c9967

@MaureenHelm  this looks like a test application settings issue, I will try to fix it in test side

@MaureenHelm  this looks like a test application settings issue, I will try to fix it in test side

@henrikbrixandersen  do you test this driver with can_api before?

@henrikbrixandersen  do you test this driver with can_api before?

@hakehuang If you are referring to the FlexCAN shim driver, then yes. I have just retested it along with the Kinetis FlexCAN driver on frdm_k64f:
*** Booting Zephyr OS build zephyr-v2.4.0-3549-g4ac2c420f9e6  ***
Running test suite can_driver
===================================================================
START - test_set_loopback
 PASS - test_set_loopback
===================================================================
START - test_send_and_forget
 PASS - test_send_and_forget
===================================================================
START - test_filter_attach
 PASS - test_filter_attach
===================================================================
START - test_receive_timeout
 PASS - test_receive_timeout
===================================================================
START - test_send_callback
 PASS - test_send_callback
===================================================================
START - test_send_receive_std
 PASS - test_send_receive_std
===================================================================
START - test_send_receive_ext
 PASS - test_send_receive_ext
===================================================================
START - test_send_receive_std_masked
 PASS - test_send_receive_std_masked
===================================================================
START - test_send_receive_ext_masked
 PASS - test_send_receive_ext_masked
===================================================================
START - test_send_receive_buffer
 PASS - test_send_receive_buffer
===================================================================
START - test_send_receive_wrong_id
 PASS - test_send_receive_wrong_id
===================================================================
START - test_send_invalid_dlc
E: DLC of 9 exceeds maximum (8)
 PASS - test_send_invalid_dlc
===================================================================
Test suite can_driver succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

I have not tested the i.MX FlexCAN driver variant as I do not have access to any i.MX boards.

@hakehuang If you are referring to the FlexCAN shim driver, then yes. I have just retested it along with the Kinetis FlexCAN driver on frdm_k64f:
*** Booting Zephyr OS build zephyr-v2.4.0-3549-g4ac2c420f9e6  ***
Running test suite can_driver
===================================================================
START - test_set_loopback
 PASS - test_set_loopback
===================================================================
START - test_send_and_forget
 PASS - test_send_and_forget
===================================================================
START - test_filter_attach
 PASS - test_filter_attach
===================================================================
START - test_receive_timeout
 PASS - test_receive_timeout
===================================================================
START - test_send_callback
 PASS - test_send_callback
===================================================================
START - test_send_receive_std
 PASS - test_send_receive_std
===================================================================
START - test_send_receive_ext
 PASS - test_send_receive_ext
===================================================================
START - test_send_receive_std_masked
 PASS - test_send_receive_std_masked
===================================================================
START - test_send_receive_ext_masked
 PASS - test_send_receive_ext_masked
===================================================================
START - test_send_receive_buffer
 PASS - test_send_receive_buffer
===================================================================
START - test_send_receive_wrong_id
 PASS - test_send_receive_wrong_id
===================================================================
START - test_send_invalid_dlc
E: DLC of 9 exceeds maximum (8)
 PASS - test_send_invalid_dlc
===================================================================
Test suite can_driver succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

I have not tested the i.MX FlexCAN driver variant as I do not have access to any i.MX boards.

@henrikbrixandersen , Thanks this issue only happens on RT1xxx series boards, let me check it further

@henrikbrixandersen , Thanks this issue only happens on RT1xxx series boards, let me check it further

the problem is due to RT SOC ERRATA
#if ((defined(FSL_FEATURE_FLEXCAN_HAS_ERRATA_5641) && FSL_FEATURE_FLEXCAN_HAS_ERRATA_5641) || \
     (defined(FSL_FEATURE_FLEXCAN_HAS_ERRATA_5829) && FSL_FEATURE_FLEXCAN_HAS_ERRATA_5829))
        if (0U == mbIdx)
        {
            fgRet = true;
        }
        else
#endif
        {
            fgRet = false;
        }
    }

so when the test application use id 0 then it will return true which mean 0 is engaged.

the problem is due to RT SOC ERRATA
#if ((defined(FSL_FEATURE_FLEXCAN_HAS_ERRATA_5641) && FSL_FEATURE_FLEXCAN_HAS_ERRATA_5641) || \
     (defined(FSL_FEATURE_FLEXCAN_HAS_ERRATA_5829) && FSL_FEATURE_FLEXCAN_HAS_ERRATA_5829))
        if (0U == mbIdx)
        {
            fgRet = true;
        }
        else
#endif
        {
            fgRet = false;
        }
    }

so when the test application use id 0 then it will return true which mean 0 is engaged.


Implement compiler-supplied ids for java compilation

Our current approach to Java compilation diagnostic IDs and their respective label was a bit simple: we derived an ID and a message from the kind of diagnostic, which limited us to 5 categories overall.
The compiler provides us with separate keys for various problems, which we could use on our end to also create finer-grained reports.
Fixes #30514

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.

The merge queue build has failed. Click here to see all failures.


Updating the build script to add RPM to dist directory and not

@mgreene @jlongstaf
What issues does this address?
Fixes #202
What's this change do?
Makes corrections in the build and deploy of the rpm so that it can be loaded to GitHub.
Where should the reviewer start?
travis file.
Any background context?
Need to add the hyphens before the file and fix the location of the rpm install.

You've go the wrong @mgreene here. :)