Uncaught TypeError: packages[Symbol.iterator] is not a function

[Enter steps to reproduce:]

Open Atom. ( Doesn't matter if first launch or not.. )
Edit > Preferences > ( Attempt to ) Open Install Menu

Atom: 1.21.0 x64
Electron: 1.6.9
OS: Ubuntu 17.04
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: packages[Symbol.iterator] is not a function
At /usr/share/atom/resources/app/node_modules/settings-view/lib/install-panel.js:390

TypeError: packages[Symbol.iterator] is not a function
    at InstallPanel.addPackageViews (/usr/share/atom/resources/app/node_modules/settings-view/lib/install-panel.js:390:30)
    at /usr/share/atom/resources/app/node_modules/settings-view/lib/install-panel.js:461:26
    at /usr/share/atom/resources/app/node_modules/settings-view/lib/atom-io-client.js:68:26
    at AtomIoClient.module.exports.AtomIoClient.fetchFromCache (/usr/share/atom/resources/app/node_modules/settings-view/lib/atom-io-client.js:160:22)
    at AtomIoClient.module.exports.AtomIoClient.featuredPackages (/usr/share/atom/resources/app/node_modules/settings-view/lib/atom-io-client.js:65:25)
    at InstallPanel.loadFeaturedPackages (/usr/share/atom/resources/app/node_modules/settings-view/lib/install-panel.js:455:27)
    at new InstallPanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/install-panel.js:128:16)
    at /usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:237:22
    at SettingsView.getOrCreatePanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:361:25)
    at SettingsView.showPanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:419:30)
    at SettingsView.showDeferredPanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:406:20)
    at SettingsView.addPanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:335:20)
    at SettingsView.addCorePanel (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:325:18)
    at SettingsView.initializePanels (/usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:236:18)
    at /usr/share/atom/resources/app/node_modules/settings-view/lib/settings-view.js:118:26
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Packages
I have not installed any packages aside from any that are installed by default. Nor have I been able to..

Thanks for the report - I think this looks like a duplicate of https://github.com/atom/settings-view/issues/962, so we'll close in favor of that issue.  Looks like we haven't been able to repro it - if you can let us know in that issue if the error is reproducible in safe mode (atom --safe) and any other details that you think might be relevant, please do.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

:+1:  it looks like you commented in #962 where this is being tracked and there's a potential fix in https://github.com/atom/settings-view/pull/1034 if you want to subscribe for updates.


Entry not found in cache on Create New Project (VSCode Extension)

Does this occur consistently?  No
Repro steps:

Change subscription (Azure Sign In command from VS Code)
Click Create New Project in Functions extension

Action: azureFunctions.createNewProject
Error type: Error
Error Message: Entry not found in cache.
Version: 0.21.0
OS: win32
OS Release: 10.0.19569
Product: Visual Studio Code
Product Version: 1.42.1
Language: en

Call Stack
a.createError extension.js:1:48963
b.<anonymous> extension.js:229:179463
extension.js:229:174078extension.js:229:174078
extension.js:229:46222extension.js:229:46222
extension.js:229:44624extension.js:229:44624
extension.js:229:43776extension.js:229:43776
i.find extension.js:229:78269
extension.js:229:25219extension.js:229:25219

Thanks for filing - as a workaround you could try signing out and signing back in to your Azure Account using the command palette.
Unfortunately there's not much we can do in this extension, but this is being tracked upstream here: microsoft/vscode-azure-account#53


The Massachusetts legislator phone #s and offices are out of date because they were just reassigned

Massachusetts state reps & state senators just got new offices over the past couple weeks, so the listed phone numbers and office numbers are (mostly) out-of-date.

thanks for the heads up, we could use some help with this one if anyone has the time!

Hey James, I have all the info logged in here: https://www.progressivemass.com/ma_legislators_contact. If there's a way for me to help update the site directly, just let me know!


Update Command Path

Problem
Need to align command path with latest IM spec
Change overview
--Rename CommandPath to CommandPathIB and remove groupdId inside(spec remove that from commandPathIB)
--Add Type struct inside CommandPathIB that can replace the existing CommandPathParams and ConcretePathParams so that we can manage command path in one place.
--Update all related code location
Testing
The existing tests covers.

Need to align command path with latest IM spec

But that's not what this PR is doing....

--Rename CommandPath to CommandPathIB and remove groupdId inside(spec remove that from commandPathIB)

The IM spec has a group id inside CommandPathIB.

--Add Type struct inside CommandPathIB

Why?  This is, in my opinion, a bad pattern that we should not have more of.  There's no reason CommandPathIB can't be the actual object type representing command paths, without the extra ::Type bit.

that can replace the existing CommandPathParams and ConcretePathParams so that we can manage command path in one place.

This seems to be based on a misunderstanding of what sorts of command paths we have in the spec.
In the spec, there are at least three concepts of "command path":

The thing that gets encoded into the TLV.  This may or may not have an endpoint id but always has a cluster id and command id, it it's valid.
The IM CommandPathIB.  This can be reconstructed from the TLV bits and session information (the latter for node ids and group ids, plus determining whether missing endpoint id means "wildcard" or "group invoke").  This has either a node id and a (possibly wildcard) endpoint id or a group id.  In practice, the node id is always the node id of the recipient, hence for our purposes we just need to keep track of whether the path is targeting a particular endpoint, all endpoints, or a particular group, plus
A concrete path.  See "<IP_ADDRESS>. Incoming Invoke Request Action" in the spec, the bits about concrete paths vs things that require "Request Path to Concrete Path Expansion".

What I think we should be doing is to have a CommandPathIB type that represents the IM CommandPathIB.  This type can be properly deserialized from a message only based on a combination of the message TLV and message metadata.  It cannot be reconstructed just from the TLV.
We should also have a separate ConcreteCommandPath.  We could reuse CommandPathIB to represent that, in theory, since obviosly it can represent a concrete path.  But it seems clearer to me to keep track of where our paths might be group/wildcard and where they're guaranteed to be concrete, such that you don't have to worry about the wildcarding.  The right boundary for that is probably about where we have the boundary right now: DispatchSingleClusterCommand.  The "Single" means we need a concrete path at that point.
Once we have group (and maybe wildcard) support, CommandHandler will need to desugar group/wildcard paths into concrete paths and call DispatchSingleClusterCommand one by one on the resulting set of concrete paths.


Stacked bar graph

Produce a stacked bar graph component, which should be able to produce results like https://www.datarevelations.com/wp-content/uploads/2012/01/E_Likert1.png.

It should be a horizontal bar
Each axis should be a different question / category
Answers should be the various datasets represented within the column
They should be ordered within the column based on their numeric value
Colours used for colouring answers should not infer good or bad
X axis should ideally be a percentage
On mouse over, show the actual counts for each answer

 Single point report

Example:
            <StackedBarChart data={{
              labels:['Question 1', 'Question 2', 'Question 3'],
              series:[{
                data: [1,2,3],
                label: '1',
              }, {
                data: [2,2,3],
                label: '2',
              }, {
                data: [5,2,4],
                label: '3',
              }, {
                data: [3,2,3],
                label: '4',
              }, {
                data: [3,6,1],
                label: '5',
              }],
            }} xAxisLabel="Percentage" showPercentage={true}/>

 Question Distribution Reports


renderElement TypeError when backend doesn't exist

I'm getting the following error.

I'm using URLSearchParams like that visible in the error, ?TS=T1&DS=, and it seems this might be where the trouble lies. Interestingly, the algorithm still renders properly.
Any ideas?

Sorry, the obscured part reads read properties of undefined (reading 'name') at Object.renderElement

Update: only the first algorithm to appear on the page renders properly, which makes sense because it looks like it's an error in a for loop.

The PR above fixes the issue, which in the end had nothing to do with the url search parameters. Rather, no renderer.backend existed, so renderer.backend.name gave the error.

Many thanks for debugging this issue and for the PR!

My pleasure!


Using hypothesis as a binary classifier

Question Description
For a given learning problem, we can return the quality of found best hypothesis/DL concept  as such
for desc in model.best_hypotheses(1):
    print('The result:', renderer.render(desc.concept), 'has quality', desc.quality)

We should use a learned DL concept as a binary classifier of an individual.
Ideally, this DL concept maps

an individual existing in the input KB or
a set of triples and an individual not existing in the input KB
into a binary value.

I think we already have that functionality in Ontolearn. The predict method (check it here) provides a binary classification for some given instances and returns a data frame with ones and zeros.

Could you please add a test for this ?

I just noticed that we also have some test cases for this. They are located in the script: tests/test_base_concept_learner.py

That is great!


[nrf fromtree] tests: bluetooth: tester: add nrf54h20 configs

Added:
nrf54h20dk_nrf54h20_cpuapp.overlay: Choose uart136 hwfc 115200 nrf54h20dk_nrf54h20_cpuapp.conf

I'd suggest adding more reviewers as well. I'm not familiar with the change nor this process

@hermabe can you help adding the right reviewers for this? I don't think I nor @asbjornsabo are the right ones

I missed the Zephyr PR but I see that it's pretty much the same as the 53 overlay/config. But why CONFIG_BT_TINYCRYPT_ECC is not selected here? Is it because 54h20 use the PSA Crypto backend?

I missed the Zephyr PR but I see that it's pretty much the same as the 53 overlay/config. But why CONFIG_BT_TINYCRYPT_ECC is not selected here? Is it because 54h20 use the PSA Crypto backend?

CONFIG_BT_TINYCRYPT_ECC is selected as part of default prj.conf here https://github.com/nrfconnect/sdk-zephyr/blob/main/tests/bluetooth/tester/prj.conf#L39


Fix Failing Proxy LiveTests

@samvaity if I don't want these to explode in runtime, so it's a good motivator to fix the race condition :)

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests


Log the XP in the console.

I think it would be great if our level and XP is logged to the console along with the other stuff, (or just logged when the player dies).
I'll see if I can read some documentation on this for agario-client, but I'm not sure they even support it.
@pulviscriptor is it possible to call the XP and player level through agario-client?

Yes!!!! This is the #1 in priority right now I think.

+1 but then also with the level

Also add a log file..  so that we have a log even if it crashes :+1:

@hassanila97 I'm going to try to add that xD

@MastaCoder
Try to use event on.experienceUpdate(level, current_exp, need_exp) if it will not work, then its in another server https://github.com/pulviscriptor/agario-client/issues/78 which is not supported by agario-client.

@pulviscriptor Thanks alot for your help!
You saved a lot of time and research in moniclip's protocol.

@Cr4xy Don't do an update yet, I'm working on something great and sorta big :+1:

You guys are awesome I wish I was millionaire so I could donate millions to you.

@MastaCoder Yep, im out for today.
gn8, everyone :)

gn8

Awwww, gn8 @all :3 xD

Hello! could make a autoreclamador of coins in agar.io of free coins?
I think it would be a great idea
when one is not in the pc all day :)

I don't think agario-client has support for that @Jhair2015

but I think it's a great idea, other methods could make the same purpose or
What is what you could do to be claimed automatically every hour?

I think the thing for gathering coins is on another server. We don't mess with the other server. If you want to do that, an autoclicker is your only option.

@pulviscriptor  i used this for the level and xp but no output....
maybe packet in agar-client has changed..
	myClient.on('experienceUpdate', function(level, current_exp, need_exp) { //if facebook key used and server sent exp info
	if (config.logging) fs.appendFileSync('log.txt', 'Experience update: Current level is ' + level + ' and experience is ' + current_exp + '/' + need_exp + "\r\n", encoding = 'utf8');
	});

@Salvationdk
I think its now in other server with other protocol https://github.com/pulviscriptor/agario-client/issues/78
I probably will remove experienceUpdate event since its not working anymore.


contextual commands with interactive tty

Contextual commands currently do not accept user input, so they can't be used for stuff like ssh where user interaction is required.

this actually works!


text with None top_node

 Issue by stefancut
Wed Oct  2 14:29:13 2019
Originally opened as https://github.com/codelucas/newspaper/issues/739

Hi,
I am new to newspaper and I am trying it to get the text from the below article and somehow article.text is an empty string. I discovered that the .calculate_best_node function is returning a None node so when article is being parsed no text is extracted (and there is text to extract).
Would anyone have any opinion on this please? Thank you
http://www.ansa.it/sito/notizie/flash/2019/10/02/-siri-ok-giunta-senato-a-sequestro-computer-_abc46d6f-f4d3-40a7-bd74-6296f2d44cb6.html

works in 0.9.1


Deprecate query parameters

Coverage decreased (-0.5%) to 92.427% when pulling 9915f10668fbdaabef7ca0a4f4cbe87b1afab397 on feature into 53d24ba3f89fd9ad3764f239c7f8e5b9f2b3d56f on master.


[AMA]

Ask me anything...
@pmarsceill,
write your question here
First I want to compliment you on the Just the Docs theme. It is awesome🥇
I have two questions for you.
First a quick context for you. We have a new product that is promoting Manual Tests as Code. It is called Testspace.  It is built on GitHub Repos, issues, etc.  It uses markdown files as test specs, also supports a subset of Liquid.  It is brand new (meaning very limited users).
Anyway, it works perfectly with your theme. I have some help documentation on it - https://help.testspace.com/docs/manual/implementation-spec#spec-preview.  Note, that I am still working on improving the steps based on your documentation/examples.
Now for my questions:

When using the source:  setting in _config.yml the _include directly processed by Jekyll for pages is not using the relative path. It works locally, but not when generating a page. Getting the File is a symlink issue.
This question is really for GitHub, but I would like to get your thoughts (and you said "AMA" 😄). I want to promote test spec documentation for projects using GitHub pages, and specifically the Just the Docs theme.  IMHO all works perfectly and really promotes good "test documentation". But for a private repo, the site is public, even for team accounts.  I guess you have to have an Enterprise Account.

Sorry for the long and probably confusing question.

Mark

Most likely question not applicable, thus closing the issue.


withFile -> writeFile suggestion is wrong in case of AppendMode

Warning introduced in #311 can easily suggest a wrong substitution:
src/Nerv/Preferences.hs:203:5: Warning: Use writeFile
Found:
  withFile (prefs </> logFileName) AppendMode $
    \ h -> hPutStr h (statement <> "\n")
Perhaps:
  writeFile file (statement <> "\n")

In case we're using AppendMode, it should suggest appendFile instead but it ignores the mode, always suggesting writeFile.

Closing as duplicate of #564, sorry for the double noise.


Refactoring and CleanUp Codes

I think these codes need to be refactored a bit, for example :
1) some methods which have only one line that are return a await module could be changed to returning only task instead.
2) We could remove some unused namespaces and sort them out.
3) For some cleaning some codes we can use following example
return _current ?? (_current= new DefaultJsonContentFormatter());

instead of this way has been implemented so far
if (_current == null)
    _current = new DefaultJsonContentFormatter();
return _current;

4) We can use Length/Count for Array/List instead of Count() and Any() for handling better performance
5) For optimizing performance we could change Codes to Find() instead of FirstOrDefault() on lists
6) We can use Method-Groups instead of calling methods directly like below code
"string".Any(char.IsDigit)

7) Use this way
_context = context ?? throw new ArgumentNullException(nameof(context));

instead of this way
if (context == null)
    throw new ArgumentNullException(context);

8) We can use null condation ?. for checking a chain of conditional member access instead of check with null
9) And finally format all codes (Ctrl+K+D)
Thanks in advance.

I'm fine with recommended changes. If you want to do more, please let me know before submitting your PR.


TypeError: option.includes is not a function

Stacktrace : TypeError: option.includes is not a function     at convertOrderByObject (/nfs/2015/v/vmarchau/hypertube/node_modules/thepiratebay/lib/Torrent.js:103:20)     at Object.search (/nfs/2015/v/vmarchau/hypertube/node_modules/thepiratebay/lib/Torrent.js:193:26)
Code : piratebay.search(name, {                 category: 'all',                 filter: {                     verified: false                   },                 orderBy: 'seeds',                 sortBy: 'desc'              });
The stacktrace is called when i made the research with any string value, i obviously have the promise after but i dont do anything apart log the result.
Node : 5.9.0
Npm : 3.7.3
OS : macOS El capitan
thepiratebay version : 1.1.8
Dunno what i can add, maybe a problem with the backport of babel ?

Okay so that was my environment, nodejs was not installed but i was able to run my sails app anyway, didnt know that, sorry for the issue

Hey how did you fix this? Im also getting this.

That was with a weird install bug, i just re-installed nodejs and it worked again

@alexanimal what are your node and npm versions?

i ended up just search results it into a python service layer with the django pirate bay api. I did re-install node and try again but it didnt work - then after i had the functionality built i figured out that i actually didnt reinstall it and point it at the right path. So maybe later I will try again and update, but right now i have some working functionality that im going to leave alone.

@alexanimal I would recommend using node 6 and running these steps in the repo that you installed thepiratebay in:
npm cache clean
npm rebuild
npm update

It should work after this.

@amilajack if you refactor/polyfill the .includes call at https://github.com/t3chnoboy/thepiratebay/blob/master/src/PirateBay.js#L81 (see https://developer.mozilla.org/nl/docs/Web/JavaScript/Reference/Global_Objects/Array/includes or https://lodash.com/docs/4.17.4#includes) or use a lodash variant your package works perfectly on node 4.4.5 as well.


Removing Reader dependency in Results module

As the title.
The key is to replace the class ansys-mapdl-reader.rst.Result by some class which provides the same interface. The complete lists of functions in that class are:
Show list of Result class functions

```text
ELEMENT_INDEX_TABLE_KEYS
ELEMENT_RESULT_NCOMP
__annotations__
__class__
__del__
__delattr__
__dict__
__dir__
__doc__
__eq__
__format__
__ge__
__getattribute__
__gt__
__hash__
__init__
__init_subclass__
__le__
__lt__
__module__
__ne__
__new__
__reduce__
__reduce_ex__
__repr__
__setattr__
__sizeof__
__str__
__subclasshook__
__weakref__
_animate_point_scalars
_animate_time_solution
_bc_header
_cfile
_element_map
_element_solution_header
_extract_element_components
_extract_node_components
_extract_surface_element_result
_is_cyclic
_is_distributed
_is_main
_is_thermal
_load_element_table
_load_materials
_load_section_data
_map_flag
_nodal_result
_nodal_solution_result
_nodfor
_nodstr
_plot_nodal_result
_plot_point_scalars
_read_components
_read_result_header
_result_nitem
_result_pointers
_result_solution_header
_result_solution_header_ext
_solution_header
_store_mesh
animate_nodal_displacement
animate_nodal_solution
animate_nodal_solution_set
available_results
cs_4x4
cylindrical_nodal_stress
element_components
element_lookup
element_solution_data
element_stress
filename
materials
mesh
n_results
n_sector
nodal_acceleration
nodal_boundary_conditions
nodal_displacement
nodal_elastic_strain
nodal_input_force
nodal_plastic_strain
nodal_reaction_forces
nodal_solution
nodal_static_forces
nodal_stress
nodal_temperature
nodal_thermal_strain
nodal_time_history
nodal_velocity
node_components
overwrite_element_solution_record
overwrite_element_solution_records
parse_coordinate_system
parse_step_substep
pathlib_filename
plot
plot_cylindrical_nodal_stress
plot_element_result
plot_nodal_displacement
plot_nodal_elastic_strain
plot_nodal_plastic_strain
plot_nodal_solution
plot_nodal_stress
plot_nodal_temperature
plot_nodal_thermal_strain
plot_principal_nodal_stress
principal_nodal_stress
read_record
result_dof
save_as_vtk
section_data
solution_info
text_result_table
time_values
version
write_tables
```

However, I'm planning to migrate only the public ones:
To migrate

[x] available_results
[ ] cylindrical_nodal_stress
[x] element_components
[ ] element_lookup
[ ] element_solution_data
This requires a mapping sort of: - EMS: misc. data
 - ENF: nodal forces
 - ENS: nodal stresses
 - ENG: volume and energies

[x] element_stress
[x] filename
[ ] materials
[x] mesh
[x] n_results
[x] n_sector
[x] nodal_acceleration
[ ] nodal_boundary_conditions
[x] nodal_displacement
[x] nodal_elastic_strain
[ ] nodal_input_force
[x] nodal_plastic_strain
[x] nodal_reaction_forces
[x] nodal_solution
[x] nodal_static_forces
[x] nodal_stress
[x] nodal_temperature
[x] nodal_thermal_strain
[x] nodal_time_history
[x] nodal_velocity
[ ] node_components
[ ] overwrite_element_solution_record
[ ] overwrite_element_solution_records
[ ] parse_coordinate_system
[x] parse_step_substep
[x] pathlib_filename
[x] principal_nodal_stress
[ ] read_record
[ ] result_dof
[ ] section_data
[x] time_values
[x] version
[ ] write_tables

Next PR
Plotting PR

[ ] animate_nodal_displacement
[ ] animate_nodal_solution
[ ] animate_nodal_solution_set
[ ] plot
[ ] plot_cylindrical_nodal_stress
[ ] plot_element_result
[ ] plot_nodal_displacement
[ ] plot_nodal_elastic_strain
[ ] plot_nodal_plastic_strain
[ ] plot_nodal_solution
[ ] plot_nodal_stress
[ ] plot_nodal_temperature
[ ] plot_nodal_thermal_strain
[ ] plot_principal_nodal_stress

VTK PR

[ ] save_as_vtk
This is going to require a separate package.

To not migrate:

[ ] cs_4x4 - I couldn't make it work locally, so I'm not porting it. It will raise a not implemented error.
[ ] solution_info
It doesn't make sense to be ported, because of this type of information is hidden in DPF. >>> rst.solution_info(0)
        {'cgcent': [],
         'fatjack': [],
         'timfrq': 44.85185724963714,
         'lfacto': 1.0,
         'lfactn': 1.0,
         'cptime': 3586.4873046875,
         'tref': 71.6,

[ ] text_result_table
Used for the text in the plots. Not porting it.

There are many methods which are duplicate from mapdl.post_processing, I will probably redirect one to another (I don't know which one should prevail, probably post-processing).

Fundamentally, this all needs to be moved to DPF-Post and DPF post needs to have an even simpler API.
Right now we have:

DPF-Core
DPF-Post
MAPDL-Post
Legacy reader

I'd rather not make a 5th. There's a potential new hire that will work for the PyAnsys team on DPF-Post and I'd like him to work on this along with you.

On-hold for the moment until I see how pydpf-post develops.

Closing because I do not have the capacity for this. I think eventually we should use dpf-post for this.


az aks get-credentials writes to stderr on successful acquisition starting in 2.27.1

Describe the bug
az aks get-credentials writes to stderr when merging kubernetes context into file starting in Azure CLI version 2.27.1
To Reproduce
az aks get-credentials --subscription ***** --resource-group ***** --name ***** --file 'test.kubeconfig' --admin 1> 'test.kubeconfig.stdout' 2> 'test.kubeconfig.stderr'

The test.kubeconfig.stdout file is empty but the test.kubeconfig.stderr file contains WARNING: Merged "*****" as current context in *****\test.kubeconfig. The test.kubeconfig file did not exist before executing the get-credentials command.
Expected behavior
The command should not write to stderr when no issue occurs. This message used to be written to stdout (Merged "*****" as current context in *****\test.kubeconfig) instead of stderr in 2.27.0 and prior.
Environment summary
Install method: MSI
CLI Version: 2.27.2
OS Version: Windows 10
Shell Type: pwsh.exe on Windows
Additional context
This started to cause Azure DevOps deployments to fail on August 18th because our tasks by default fail when stderr lines are detected. Tested that this issue reproduces in both Azure DevOps Azure CLI Task 2.1.1 as well as on a local machine.

route to service team

According to https://www.gnu.org/software/libc/manual/html_node/Standard-Streams.html

Variable: FILE * stderr
The standard error stream, which is used for error messages and diagnostics issued by the program.

Therefore, stderr is the right place for Azure CLI to write WARNING messages.
If the pipeline has failOnStderr set to true and the WARNING message is causing failure, you may either

In the pipeline, set failOnStderr to false
Disable WARNING messages in Azure CLI commands using the only_show_errors configuration.

I would contend that:

This is a breaking change. A behavioral change was made (writing out additional messages that were not written in prior same major release) that caused this issue to occur. Without any changes to our pipeline, it started to fail because of the change between 2.27.0 and 2.27.1
The usage of the command as described does not request to write out diagnostic information, thus it should not write diagnostic information. Thus, the diagnostics issued by the program portion of the explanation is not acceptable. What remains is which is used for error messages; the message is not not an error. Since it does not meet either explanation, it should not be written to stderr under these conditions.
The CLI should not be writing this warning message under the explained conditions. Merged "*****" as current context in *****\test.kubeconfig should only reasonably be a written when credentials were actually merged into an existing file, not when writing into a new file. This warning is being written out when the file doesn't exist in the first place.

I've already mitigated this issue by using --only-show-errors. I will not set failOnStderr: false so that we can catch other actual errors when they occur.

@jander-msft, as for whether az aks get-credentials should print a WARNING message, I will leave it to @andyliuliming to decide.
I still would like mention that there are other CLI commands that will print WARNING messages to stderr, such as az ad sp create-for-rbac which even prints several WARNING messages:
> az ad sp create-for-rbac > out.txt
WARNING: In a future release, this command will NOT create a 'Contributor' role assignment by default. If needed, use the --role argument to explicitly create a role assignment.
WARNING: Creating 'Contributor' role assignment under scope '/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590'
WARNING: The output includes credentials that you must protect. Be sure that you do not include these credentials in your code or check the credentials into your source control. For more information, see https://aka.ms/azadsp-cli
WARNING: 'name' property in the output is deprecated and will be removed in the future. Use 'appId' instead.

With failOnStderr: true but no --only-show-errors, az ad sp create-for-rbac will cause the pipeline to fail for sure. Correct way is to determine whether a command successful is to check the exit code (https://github.com/Azure/azure-cli/issues/18372#issuecomment-918741485).
You may disable all WARNING messages with
# This command itself shows a warning which should be disabled
az config set core.only_show_errors=true --only-show-errors

@jander-msft I hope you had a chance to look at the pointers suggested by @jiasli. I see that this github issue has been open for quite sometime now. I just wanted to do a follow-up to check if you still require assistance on this. Awaiting your reply.

I've already mitigated this as stated in https://github.com/Azure/azure-cli/issues/19314#issuecomment-919558448
I've already mitigated this issue by using --only-show-errors. I will not set failOnStderr: false so that we can catch other actual errors when they occur.

@jander-msft Thanks for your reply. If you need any further assistance on this, please reopen this thread. We would be happy to help you.


Enable more type checks

Pyright settings were a bit lax. Tighten them.

Codecov Report

Merging #78 (ecc6cf0) into master (3892245) will not change coverage.
Report is 5 commits behind head on master.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   92.18%   92.18%
=======================================
  Files           6        6
  Lines         614      614
=======================================
  Hits          566      566
  Misses         48       48

Flag
Coverage Δ

unittests
92.18% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

pygitguardian/__init__.py
100.00% <100.00%> (ø)

pygitguardian/client.py
79.82% <ø> (ø)

pygitguardian/models.py
98.92% <100.00%> (ø)


Program to get keyboard input using Go

Simple program to get keyboard input and display in the terminal.
source : https://golang.org/

@ChethiyaNishanath
Wohoo!! Congrarts🥳 your PR has been merged.

@ChethiyaNishanath
By the way I like your beard 😂


[Bulk Download] Add aegea script and download status update endpoints.

Description

Added aegea kickoff script to Rails back-end, which accepts the correct values for source files for the "Original Input File" bulk download type.
Added public controller endpoints that accept a per-download access token, allowing the ecs task to ping the Rails server with success, error, and progress updates.
Added UI for various statuses in bulk download list.

Factored out a couple of components (LoadingBar, StatusLabel)

Notes

How the bulk download works is:

When a bulk download is created, we kick off an ECS task with aegea.
We give the ECS task a "success-url", an "error-url" and a "progress-url", which it can ping with updates. Each of these URLs has a form like "/bulk_downloads/5/success/fbb546f2930fff6", where "fbb546f2930fff6" is an access token assigned to that particular download.
The per-download access token removes the need for any kind of user authentication.
After the ECS task completes, it will either ping the success-url or the error-url and the BulkDownload status field will update.

This all assumes that the ECS task completes without failing in the middle. The ECS task (see idseq-services) DOES have an error block that surrounds the entire task so it should catch all normal errors. But we will probably also need an occasional lambda function that checks if any BulkDownloads have been "running" for a long time, in case tasks hang. (Thanks for David for suggesting)
In order to test BulkDownloads locally, you will need to use something like ngrok. After running ngrok http 3000, you can restart your web service with DEVELOPMENT_TUNNEL_URL=https://eb9417b1.ngrok.io docker-compose up web. This allows the ECS task to ping your local web server. Make sure to bring down your tunnel when you are done.

Tests

Manually tested the bulk downloads locally, and verified that the UI works properly.
Wrote rspec tests for all model and controller functions.

@davidrissato I'm going to go ahead and submit this PR, and work on a follow-up PR today that handles the file naming. Since the file naming seems a little tricky, and the rest of this PR is already reviewed. None of this is visible to users yet, so we can continue to iterate.


feat(core): support ESM Forge module loading

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[ ] The changes have sufficient test coverage (if applicable).
[X] The testsuite passes successfully on my local machine (if applicable).

Summary
This PR focuses on bringing the initial support loading Forge modules (plugins, makers, publishers etc.) that use ESM for their module format. It should also contain some cosmetic changes like rename of require-search to import-search (to reflect it isn't around the require API only anymore), improvements around the helper so it won't resolve path for modules passed by their names and introduces new function, dynamicImportMaybe, that will also try doing require first and falling back to ESM when it fails (I chose to require first so I won't have to resolve default in CJS import).
The main motivation on that is to allow third-parties as well as official Forge modules to be able to effortlessly utilise the ESM syntax entirely. While this is possible for now with the Forge as well, it requires of already importing the class to the config on your own.
There are currently a few things that could be considered to be worked on in the future, but are not really necessary from what I've been testing:

Resolving directory paths for import in order to load them properly. This could be useful when makers are installed in unusual paths that even make Node not to be able to find modules when they are placed that way. Plus, there could be some issue with actually finding what to import given how complex it became in Node to resolve imports, plus resolving internal imports (starting in #) would probably also have to be supported as well. It might not be worth of the efforts for now.

(Maybe) dropping dynamicImport or making it private, since it doesn't seem to be used outside of its own module scope anymore.

Improving some tests, so they have simpler logic. Currently, I focused more on preserving on the logic that tests actually uses and only changing them by making them async or use some wrapped stuff. This should make them work essentially similar way while fixing false negatives introduced by some drastic changes, like making some synchronous functions async or renaming a module. Done in 9229bff.

I should also mention, the changes in this repo were tested by me (locally) with my own implementation of ESM maker.
Changes around the tests
Of course, I had to modify tests for require-search and publish-spec, due to changes in module names and switching some previously synchronous code to be async. For now, testing only the workspace @electron-forge/core, I've seen similar number of tests failing on both official implementation and this fork, and I think they were caused because Forge expected to run tests from a root project than a workspace, as I saw those were mostly caused due to missing scripts in workspace package (so it's nothing serious). I guess it's out of the scope to fix that tho.
 This is my first time to contribute to Forge, so I'm definitely learning how to do stuff right while trying to avoid mistakes as much as I could 😄.

Looks like template tests are failing with the following because this.baseDir is not instantiated on time. I'm not sure what the root cause is exactly, but https://github.com/electron/forge/pull/3582/commits/d65e18ceea4485da72e6de2327f7db908f2f5012 seems to be the trigger.
I can't reproduce locally either.
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
    at Object.rm (node:fs:1248:10)
    at Object.remove (node_modules/fs-extra/lib/remove/index.js:9:24)
    at /Users/distiller/project/node_modules/universalify/index.js:8:12
    at new Promise (<anonymous>)
    at Object.remove (node_modules/universalify/index.js:7:14)
    at /Users/distiller/project/packages/plugin/webpack/src/WebpackPlugin.ts:168:20
    at _TaskWrapper.namedHookWithTaskInner (packages/plugin/base/src/Plugin.ts:59:13)
    at /Users/distiller/project/packages/api/core/src/util/plugin-interface.ts:117:48
    at _Task.taskFn (packages/utils/tracer/src/index.ts:51:36)
    at _Task.run (node_modules/listr2/dist/index.cjs:2063:35)

Is this PR complete to be merged?

@rtritto I think so, I guess I'm awaiting more reviews / @MarshallOfSound comment on that (if he didn't noticed the changes within the PR). I personally don't think I want to change anything, unless some other dev has some idea or want to discuss some stuff here I'd say it's ready to be merged.
I might periodically update the branch tho to ensure there's nothing breaking when merging it on top the latest code, or resolve conflicts if I'll encounter any.

Can this PR still be merged?


chore: Disable all caching during dev time

Caching causes more problems than it solves, during development time. This PR explicitly disables all cache during development.

@sharat87 , is this script used only during dev?

@SatishGandham, yes, only during development. Thanks for the approve! 🙂


SuppressedError docs are out of date

The readme gives an options argument to SuppressedError, taking a cause.
class SuppressedError extends Error {
  /**
   * Wraps an error that suppresses another error, and the error that was suppressed.
   * @param {*} error The error that resulted in a suppression.
   * @param {*} suppressed The error that was suppressed.
   * @param {string} message The message for the error.
   * @param {{ cause?: * }} [options] Options for the error.
   */
  constructor(error, suppressed, message, options);
..

Happened to notice #147 that removed the cause option. Wouldn't have even realized if that issue didn't cross my path. I think the docs need updating.
(Really glad the new error is coming, I've tripped over how to surface suppressed errors for years in so many places!)

I see there's a PR #230 to update the docs. Closing this to delegate to the patch.


Evaluate expanding group-based access control rules with source/destination port and protocol

Is your feature request related to a problem? Please describe.

I'd like to be able to allow only specific traffic on a netbird group to expose only specific services (e.g. a group for FTP, one for RDP / VNC and so on). Currently, this needs to be configured locally per-peer, but a centralized solution manageable from netbird webui could be handy.

Describe the solution you'd like

As per title, the access control rules could be expanded with source/destination ports and protocol.

Describe alternatives you've considered

I have not investigated how they are implemented so far, so I cannot estimate the effort required or if this could be the correct way to implement this feature. Peer-based firewall configuration could also be a valid alternative / appreciated feature, but the initial idea was to expand rather than writing an entirely new feature, which could be more cost-effective if the changes are not too many.

Additional context

Firewall rules should only be related to netbird tunnel.
Rules should not be able to overwrite the peer's rules, otherwise hosts firewall could be compromised by malicious rules.

PS: if this feature is available already I deeply apologized, I must have missed it entirely

We've added support with version 0.21.0. see https://github.com/netbirdio/netbird/releases  and https://docs.netbird.io/how-to/manage-network-access for more details.

Thanks a lot, it's exactly the feature I was looking for. Really appreaciate that you managed to address also my less visible issue among all the others, I'm closing the issue.


Laravel 5.5 problem when i migrate require Doctrine DBAL

hey hello, i hope you be good...
I have a problem when make a php artisan migrate the problem star when return this message:
In ChangeColumn.php line 30:
Changing columns for table "roles" requires Doctrine DBAL; install "doctrine/dbal".
i install the doctrine/dbal and
later migrate again, and later return a line with a message,
php artisan migrate
Migration table created successfully.
In AbstractPlatform.php line 423:
Unknown database type enum requested, Doctrine\DBAL\Platforms\MySqlPlatform may not support it.
i find the problem and in the file 2015_12_04_003040_add_special_role_column.php have this line
public function up()
{
Schema::table('roles', function ($table) {
$table->enum('special', ['all-access', 'no-access'])->nullable();
});
}
late change this line for a solution...
public function up()
{
Schema::table('roles', function ($table) {
$table->string('special', 10)->nullable();
});
}
and migrate again and now have other problem with this line.
php artisan migrate
Migration table created successfully.
In Connection.php line 664:
SQLSTATE[HY000]: General error: 1833 Cannot change column 'id': used in a foreign key constraint 'permission_role_role_id_foreign' of table 'commonexpe
nses.permission_role' (SQL: ALTER TABLE roles CHANGE id id BIGINT UNSIGNED AUTO_INCREMENT NOT NULL)
In PDOStatement.php line 107:
SQLSTATE[HY000]: General error: 1833 Cannot change column 'id': used in a foreign key constraint 'permission_role_role_id_foreign' of table 'commonexpe
nses.permission_role'
In PDOStatement.php line 105:
SQLSTATE[HY000]: General error: 1833 Cannot change column 'id': used in a foreign key constraint 'permission_role_role_id_foreign' of table 'commonexpe
nses.permission_role'
i don't understand how can fix this problem now... please i need help....
i use Laravel 5.5
php 7.0.30
Thank you!

Sorry I can only support the latest version of the package, which requires Laravel 5.7+ and PHP 7.1

understand but the solution in my project is reinstall the package...
composer require caffeinated/sinobi version for my Laravel 5.5
composer require caffeinated/sinobi 3.1 or 3.2 or 3.4.
now is working the migration and migrate.
Thank you @kaidesu

Interesting! Glad it all works for you now :v:


$ rosrun jetbot_ros jetbot_motors.py doesn't work

Installing jetbot-ros on the L4T r32.2 image after successfully installing jetbot_ros and ROS Melodic. Testing command rosrun jetbot_ros jetbot_motors.py, the error message below occurs after  processing. I would appreciate your assistance.
i had run : sudo usermod -aG i2c $USER(user name)
error is below
Traceback (most recent call last):
File "/home/jetbot/workspace/catkin_ws/src/jetbot_ros/scripts/jetbot_motors.py", line 75, in
motor_driver = Adafruit_MotorHAT(i2c_bus=1)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_MotorHAT/Adafruit_MotorHAT_Motors.py", line 231, in init
self._pwm = PWM(addr, debug=False, i2c=i2c, i2c_bus=i2c_bus)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_MotorHAT/Adafruit_PWM_Servo_Driver.py", line 57, in init
self.i2c = get_i2c_device(address, i2c, i2c_bus)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_MotorHAT/Adafruit_PWM_Servo_Driver.py", line 23, in get_i2c_device
return I2C.get_i2c_device(address, busnum=i2c_bus)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_GPIO/I2C.py", line 64, in get_i2c_device
return Device(address, busnum, i2c_interface, **kwargs)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_GPIO/I2C.py", line 97, in init
self._bus = Adafruit_PureIO.smbus.SMBus(busnum)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_PureIO/smbus.py", line 105, in init
self.open(bus)
File "/home/jetbot/.local/lib/python2.7/site-packages/Adafruit_PureIO/smbus.py", line 130, in open
self._device = open('/dev/i2c-{0}'.format(bus), 'r+b', buffering=0)
IOError: [Errno 13] Permission denied: '/dev/i2c-1'

it was solved after reboot


Coalesce the guidance on 'Avoid using a loop variable directly in a lambda expression' into one spot

Place in one spot and cross-link to it.

Actually, it's already in one spot in the Event Handler doc (and cross-linked). There's only one other full example for it in the Images and Documents article, and I like the content there. However, it's a NOTE ... I'm going to use this issue to promote it to regular content.


Access to revisions of an external control plane

Describe the feature request
When using an external control plane, the remote istiod location is configured in several places, three of which (discoveryAddress, caAddress, and injectionURL) should, but currently do not, take revisions into account:
spec:
  meshConfig:
    defaultConfig:
      discoveryAddress: $EXTERNAL_ISTIOD_ADDR:15012
  values:
    global:
      caAddress: $EXTERNAL_ISTIOD_ADDR:15012
    istiodRemote:
      injectionURL: https://$EXTERNAL_ISTIOD_ADDR:15017/inject

For example, when the the injectionURL value is set, it will be used in the MutatingWebhookConfiguration as is, with no regard to the webhooks associated revision. Only the not-set ({{- else }}) uses the revision to select the correct istiod service.
If instead, we added a new variable (istiodRemote.istiodURL), we could pass the revision as a path prefix, which the external control plane could use to access the appropriate istiod revision:
    {{- if .Values.istiodRemote.injectionURL }}
    url: {{ .Values.istiodRemote.injectionURL }}
    {{- else if .Values.istiodRemote.istiodURL }}
    url: {{ .Values.istiodRemote.istiodURL }}:15017{{- if not (eq .Values.revision "") }}/{{ .Values.revision }}{{- end }}/inject
    {{- else }}
    service:
      name: istiod{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
      namespace: {{ .Release.Namespace }}
      path: "/inject"
    {{- end }}

If we make a similar change for all three variables, this would also simplify the external control plane configuration. Only one variable (istiodRemote.istiodURL) would need to be configured, instead of the current three:
spec:
  values:
    istiodRemote:
      istiodURL: https://$EXTERNAL_ISTIOD_ADDR

Describe alternatives you've considered
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Additional context

I think the path prefix would not work for CA/discovery address due to grpc

@howardjohn, good point, my suggestion will only work for the https one (injectionURL).
What about using wildcard subdomains instead, something like this:
    {{- if .Values.istiodRemote.injectionURL }}
    url: {{ .Values.istiodRemote.injectionURL }}
    {{- else if .Values.istiodRemote.istiodHost }}
    url: https://{{- if not (eq .Values.revision "") }}{{ .Values.revision }}.{{- end }}{{ .Values.istiodRemote.istiodHost }}:15017/inject
    {{- else }}
    service:
      name: istiod{{- if not (eq .Values.revision "") }}-{{ .Values.revision }}{{- end }}
      namespace: {{ .Release.Namespace }}
      path: "/inject"
    {{- end }}

Would that work?
Any other suggestions? @esnible, I'm not sure I understand how #31342 fixes this.
Seems that currently revisions can't be used with external control planes.

That could work, but there is not one way that this can/should be implemented. Users will have their own routing requirements. Some examples of other ideas that may work better for other users (note: these are pretty old): https://gist.github.com/howardjohn/15618fd3a1d2ed54b95ae6fb50548433 https://gist.github.com/howardjohn/41e5c30816b82b6abc8c89f943393d96
I don't think we should enforce the https://revision.url format - but we can document that its one way to configure things.

Another option is to set XDS_HEADER_REVISION env var, then you can do standard virtual service header routing

not stale


rpm does not work inside toolbox with selinux failure

Fedora SilverBlue 29, updated yesterday:
rpm -ivh foo.rpm
Error: Plugin selinux: hook tsm_pre failed" when using rpm -ivh

It seems to be testing for:
openat(AT_FDCWD, "/etc/selinux/targeted/contexts/files/file_contexts.subs_dist", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/etc/selinux/targeted/contexts/files/file_contexts.subs", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
stat("/etc/selinux/targeted/contexts/files/file_contexts", 0x7ffe6d43ab28) = -1 ENOENT (No such file or directory)
stat("/etc/selinux/targeted/contexts/files/file_contexts.bin", 0x7ffe6d43abc0) = -1 ENOENT (No such file or directory)

i.e. /etc/selinux/targeted/ exists outside the toolbox, but not inside
🔹[hughsie@toolbox build]$ getenforce
Enforcing

So perhaps rpm thinks that the policy is enforcing (and so the plugin runs) but then the plugin can't find anything it needs? Although, using setenfoce 0 outside the toolbox makes the toolbos show as "Permissive", and the rpm inside the toolbox still fails.

A workaround is sudo mv /usr/lib64/rpm-plugins/selinux.so /usr/lib64/rpm-plugins/selinux.so.old

Additional, value-add failure:
Traceback (most recent call last):
  File "/usr/sbin/semanage", line 28, in <module>
    import seobject
  File "/usr/lib/python3.7/site-packages/seobject.py", line 34, in <module>
    import sepolicy
  File "/usr/lib/python3.7/site-packages/sepolicy/__init__.py", line 188, in <module>
    raise e
  File "/usr/lib/python3.7/site-packages/sepolicy/__init__.py", line 184, in <module>
    policy_file = get_installed_policy()
  File "/usr/lib/python3.7/site-packages/sepolicy/__init__.py", line 139, in get_installed_policy
    raise ValueError(_("No SELinux Policy installed"))
ValueError: No SELinux Policy installed

  Running scriptlet: tpm2-abrmd-selinux-2.0.0-2.fc29.noarch                                             267/315
cp: cannot stat '/etc/selinux/targeted/contexts/files/file_contexts': No such file or directory
error: %prein(tpm2-abrmd-selinux-2.0.0-2.fc29.noarch) scriptlet failed, exit status 1

Error in PREIN scriptlet in rpm package tpm2-abrmd-selinux

does your rpm work if you install selinux-policy-targeted inside the container? you will see a permission denied error when you install that rpm, but imho this can be ignored (seems like the package tries to load the policies which obviously does not work inside the container).

Thanks for digging that up, @tosmi !

The toolbox containers are created with --security-opt label=disable, which turns off SELinux labelling. I wonder if removing that can help.
Labelling was turned off because some time ago, it would make container creation (or was it something else? I can't remember) really slow. However, I took that option out to enable labelling, and it didn't seem noticeably slow. Maybe Podman got better at this?

@debarshiray The issue is /sys/fs/selinux is being mounted into the container.  We saw this yesterday.
This is causing SELinux labeling to be turned on IFF, you have selinux-policy installed in an image, and running with --net=host.
podman run IMAGE id -Z
Should tell you SELinux is disabled.

This means tools inside the container like RPM attempt to do labeling.  Which is the wrong thing to do. Whether or not SELinux separation is enabled.

https://github.com/containers/libpod/pull/2613 fixes this problem.

@mheon we might want to consider a new release of podman for this issue.

@rhatdan I'd like to wait a week for things to settle a bit more, and for Brent to do more work on healthchecks. Once he lands his healthchecks v3 PR, I think we're good for a 1.2 release

Thanks for the explanation and the fix, @rhatdan
I realized that I didn't have selinux-policy in my toolboxes, which is why I wasn't able to reproduce this. @hughsie probably managed to pull it in as a side-effect of something else.

probably managed to pull it in as a side-effect of something else

Indeed I did.

Ok, great! Thanks everybody. Closing.

I Basically think we all don't have to face all these deceit and lies from our spouse...in a case of mine wen i got sick and tired of all the lies and deceit i had to contact a friend of mine to get me the contact of one of the best hackers in the states ..then i met Gary.He saved me from the lies of my cheating husband by hacking his phone..Incase you need help with hacking any phone or account or other jobs..Tell him i reffered you.He will help you
<EMAIL_ADDRESS>or you can text him too +1<PHONE_NUMBER>


Typechecker can't weaken comparable to number without explicit constraints

A minimal example, based on this thread
Consider the following code:
badMaxNumber : List number -> Maybe number
badMaxNumber l =
  case l of
    [] ->
      Nothing

    h :: t ->
      Just <| List.foldl max h t

goodMaxNumber : List number -> Maybe number
goodMaxNumber l =
  case l of
    [] ->
      Nothing

    h :: t ->
      Just <| List.foldl max (h+0) t

The first function fails with a mismatch between List number -> Maybe number and List comparable -> Maybe comparable.
The second function typechecks just fine, since the (+0) constraint tells it that we're looking at numbers, not comparables.
Should the typechecker be able to tell that number is a subtype of comparable, just like it knows Int is a subtype of number? Or is the price of having hacky (+0)s throughout the code worth the simplicity of avoiding a type heirarchy?

Why report this here? Isn't that the wrong repo? And in any case, the issue looks just a duplicate of https://github.com/elm-lang/elm-compiler/issues/1281.

Bah, definitely the wrong repo, and I wasn't familiar with the previous issue, probably because I was looking in the wrong repo. Sorry!.


Timing function

Feature request is related to

[x] embla-carousel (core package)

Is your feature request related to an issue?
I there any way to change timing function of slides transition?

Hi @Filip1139,
Do you mean applying custom easing? Or just changing speed?
Best,
David

Duplicate of:

#184


Publishing documentation

This PR is the sibling of #72. It does the following:

Push release documentation on GH pages under version/x.y.z where x.y.z is the release number
Push latest master documentation on GH pages (intermediate master documentations can be found as artefacts in their respective build workflows)

Checks do not run, your workflow files have errors.

Checks do not run, your workflow files have errors.

Indeed fixed

Please squash your commits into 1 or 2 commits


Problem: some activities only return error

Describe the bug
A number of activities in this project are not returning a struct. This can lead to backward-compatibility issues if we add one in the future.  If we use a struct, the activity remains compatible even if the fields change. This means we can add, remove, or change fields in our struct without breaking the function signature.
Some examples:

CopyToPermanentLocationActivity,
CleanUpActivity,
DeleteOriginalActivity,
DisposeOriginalActivity,
MoveToPermanentStorageActivity,
PollMoveToPermanentStorageActivity,
RejectPackageActivity,
UploadActivity, or
DeleteTransferActivity.

Expected behavior
Always return a struct even if empty. Refer to the code example in https://docs.temporal.io/dev-guide/go/foundations#activity-return-values:
// YourActivityResultObject is the struct returned from your Activity.
// Use a struct so that you can return multiple values of different types.
// Additionally, your function signature remains compatible if the fields change.
type YourActivityResultObject struct {
    ResultFieldX string
    ResultFieldY int
}

Additional context
I believe this is a future-proofing effort that will become more important when we hit production and as upgrades are rolled out progressively the system will have both old and new versions of workers running concurrently.
Related: https://github.com/artefactual-sdps/enduro/pull/787#discussion_r1409683586.

Should we consider doing this for local activities as well? Some local activities use a uint instead of a struct as well which seems non-standard. Also for reference, we tend to name result structs like:
type OurActivityResult struct {
    FieldX string
    FieldY int
}

@Diogenesoftoronto it makes sense to me to do this for local activities as well.  I think our naming convention is fine, I don't think adding a "Object" suffix is necessary or adds any valuable information.

@Diogenesoftoronto it makes sense to me to do this for local activities as well.

Agree. In practice, I don't think local activity inputs/outputs are encoded because they're local to the worker, but it sounds like it's a good idea for consistency purposes.


parameter for squidpy connectivities key

Added a connectivities_key parameter to the AnnData2Data classes. This allow you to pass your own parameter key for the connectivities if you have calculated the squidgy spatial graph up front (for some more flexible graph initialisation).

sorry a bit late with this but the new merge should provide this


feat/Updated Binance Spot api Request weight rate limits

Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
The Request Weights for Binance was updated
The RAW_REQUESTS for SPOT API has been adjusted to 61,000 every 5 minutes.
The weights for the SPOT API have been upadated
For the mention:
Binance changelog from 2023-08-25
https://binance-docs.github.io/apidocs/spot/en/#change-log
Tests performed by the developer:
Tips for QA testing:

Test performed:

Review changes

Review REQUEST_WEIGHT was has been adjusted to 6,000 every minute.
Review The RAW_REQUESTS for SPOT API has been adjusted to 61,000 every 5 minutes.
Review The weights to the following requests for the SPOT API have been adjusted according to https://binance-docs.github.io/apidocs/spot/en/#change-log

Cloned and installed feature branch
connected binance successfully
started script v2_market-making_dman_v3_multiple_pairs.py using binance spot successfully, no rate limit related errors showed
pending tests: longrun on multiple pairs/order levels


Add check for contiguous data in BilinearSampler

I noticed the output of the BilinearSampler is not consistent across channels (D dimension), when the inputImages are not contiguous. In my case I permuted my input images from BDHW to BHWD, which gave strange issues unless I clone after permuting.
This PR adds a check for contiguity (is that a word?) in the check function.

Thanks, I should have done that before !

Also I like to use the word "contiguity" a lot :)


route generated not https

what I've done:

forceSchema on provider
env set APP_URL
clear all cache

It's very weird and frustrating, when I debug at console, I saw "template" not holding https domain.

i dunno how its somehow fixed :(

The solution is set the APP_URL to an https url

@tiaxter the solution is actually after you do all of these:
-forceSchema on provider
-env set APP_URL
-clear all cache
finally, you need to logout n login back

@tiaxter the solution is actually after you do all of these:
-forceSchema on provider
-env set APP_URL
-clear all cache
finally, you need to logout n login back

I only set APP URL to an HTTPS url and clear cache
And i solved


How to integrate ESLint plugins?

I was just wondering the best way to integrate ESLint plugins to XO.
Let's say I want to use eslint-plugin-no-use-extend-native along with XO, what should I do? Does XO simply invokes ESLint and things work as expected or is there any trick that I need to do?

I don't have a story for that yet unfortunately, but I'm planning to include that plugin by default in the next release.

Maybe having a 'plugins' key on the configuration could be a good idea?
Anyway, thanks for the answer.
--
Rafael Rinaldi
http://rinaldi.io

On Sep 25, 2015, at 08:17, Sindre Sorhus<EMAIL_ADDRESS>wrote:
I don't have a story for that yet unfortunately, but I'm planning to include that plugin by default in the next release.
—
Reply to this email directly or view it on GitHub.


Scrolling in browser does not work correctly when Avalonia content does not start at the top-left corner

Describe the bug
When Avalonia content is rendered in a part of the page instead of the full page, scrolling within a ScrollViewer does not work correctly. It appears that the mouse coordinates are being considered relative to the page instead of relative to the top left corner of Avalonia content. There is a "dead area" at the right and bottom of the scrollviewer whose width and height match the offset of the top left corner of the Avalonia content w.r.t the page.
To Reproduce
Steps to reproduce the behavior:

Create an Avalonia browser application targeting .NET 8 with Avalonia 11.0.7
Replace the contents of index.html with

<!DOCTYPE html>

    <title>AvaTestNet8.Browser</title>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <base href="/" />
    <link rel="modulepreload" href="./main.js" />
    <link rel="modulepreload" href="./dotnet.js" />
    <link rel="modulepreload" href="./avalonia.js" />
    <link rel="stylesheet" href="./app.css" />

Loading...

Replace the content in MainView.xaml with

<ScrollViewer HorizontalScrollBarVisibility="Auto" VerticalScrollBarVisibility="Auto">
  <Rectangle Height="4000" Width="4000">
    <Rectangle.Fill>
      <LinearGradientBrush>
        <LinearGradientBrush.GradientStops>
          <GradientStop Color="Yellow" Offset="0"/>
          <GradientStop Color="Green" Offset="1"/>
        </LinearGradientBrush.GradientStops>
      </LinearGradientBrush>
    </Rectangle.Fill>
  </Rectangle>
</ScrollViewer>

Run the application in the browser and try to scroll with the mouse wheel when the cursor is near the bottom or right edges of the Avalonia content. Scrolling works around the top edges of the content but not near the bottom edges. The width / height of the "dead areas" match the offset of the content.

Expected behavior
Scrolling should work correctly.
Environment

OS: Browser
Avalonia-Version: 11.0.7

Not commenting on if this is a bug or by-design.
I've been able to use iframes instead of div to create a page with multiple Avalonia WASM views side-by-side and the mouse inputs were correct. You may be able to do the same, but my scenario was a bit different. This was a separate webpage that embedded already web-deployed WASM apps as a test (the Solitaire demo). Not sure how to integrate iframe into the WASM app itself for mixed content.

Duplicate of #12780?

Yeah, it’s not a bug. Only a confusion with how CSS works.

Please note that the "out" div does have position relative. Also, click events and all other mouse interactions work correctly.
Looking at the source code in AvaloniaView.cs, I see that the OnWheel handler uses clientX and clientY. Other handlers use offsetX and offsetY.
https://github.com/AvaloniaUI/Avalonia/blob/2a9e7e9dcfc3e590bdd50b1e874ce70c0fafcc24/src/Browser/Avalonia.Browser/AvaloniaView.cs#L259

It looks like a very safe fix because OnWheel is the only handler that uses clientX and clientY. All other handlers use a utility method ExtractRawPointerFromJSArgs which uses offsetX and offsetY.
I will be able to open a PR only after a few weeks since this is not a showstopper for me right now.

Haven't paid attention that issue is mouse wheel specific. Yes, changing to offsetX/offsetY makes sense.

@koustubhmoharir a PR would be very welcome


Added additional IFTTT script for Afrezza

For Afrezza or can also be used for insulin pen correction bolus.

Following discussion with Dana and Scott - I was made aware that this action would not be put into the calculation for OpenAPS, rather it creates a note in Nightscout.
The suggested path to solving this was to put a temp target for 30-45 minutes of 200. This would allow OpenAPS sufficient time to see what is happening with the extra insulin IOB, while reduce dosing.
Scott also highlighted:
if you don't have profile.exercise_mode or profile.high_temptarget_raises_sensitivity set, then you might need a high temp target of 160 or more. if you do, then something like 140 would probably be sufficient
allowSMB_with_high_temptarget is also relevant: if that's not set, you won't get any SMBs as long as a >= 111 high target is set


TDLib: Ask questions about user avatars or profile_photo

When I use the TDIb getUser to get Userinfo, the profile_photo information should be the data belonging to the avatar information, but how can I make this data appear in my application?Please give a method。

If you want to download the photos, you need to use downloadFile method.

如果要下载照片，则需要使用downloadFile方法。

If I use the downloadFile method, this is to generate the corresponding file in the photos directory, but this downloaded file is not the file I want to display the user's avatar. I consulted the TDRib document, the document display will return a file, I Look at the file returned in my log does not have any information attached, and my purpose is not how to download a picture from the chat, but how to let my application display the avatar of the object I chat with on the interface, after all, Chatting with a black frame makes people feel less comfortable.

@AquariusHunters Yes, downloadFile will make the file available in the photos directory. After that you can read it and show to the user. Profile picture is a file, and it needs to be downloaded first via downloadFile.

@AquariusHunters是的，downloadFile将使文件在照片目录中可用。之后，您可以阅读并向用户显示。个人资料图片是一个文件，需要先通过下载downloadFile。

@AquariusHunters Yes, downloadFile will make the file available in the photos directory. After that you can read it and show to the user. Profile picture is a file, and it needs to be downloaded first via downloadFile.

Sorry, my expression may be misunderstood. I mean how to get the profile photo of the chat object instead of downloading the profile photo to the local directory, so I don't think it is convenient.  I can get a remote URL to download the corresponding profile photo into my program's memory, and then display it. It's not that I want to download it to the local directory. I don't think this is a good way. I should Where is he going to deal with him?

A chat photo must be saved to the local directory in order to not be redownloaded on every application restart and to be available without Internet access. There is no way in TDLib you can get a chat photo only in memory without saving it on the drive, Why you can't read a local file and show a photo from the file?

必须将聊天照片保存到本地目录，以便在每次重启应用程序时都不会重新加载，并且无需Internet访问即可使用。在TDLib中你无法在内存中获取聊天照片而不将其保存在驱动器上，为什么你无法读取本地文件并显示文件中的照片？

How do I use these downloads to save local images? Did I look at the 10,000 members of the Super Group members and I have to save the profile photos of these 10,000 members locally? If so, how do you map these photos stored locally to these 10,000 members?

A chat photo must be saved to the local directory in order to not be redownloaded on every application restart and to be available without Internet access. There is no way in TDLib you can get a chat photo only in memory without saving it on the drive, Why you can't read a local file and show a photo from the file?

When I use the getRemoteFile api, I get a json about the file. Can I match the file according to the data value of the remote, then when I use the downloadFile, the ID is the short int in the file. So how do I put him in the directory I want instead of the photos directory? After all, there are some other weird files in this directory.

@AquariusHunters Each file has a unique ID, which is used to download the file. You need to listen to updateFile updates to follow download progress. Since TDLib 1.4.0 you can also use parameter synchronous in downloadFile request to receive the response from downloadFile only after the download is finished.

@AquariusHunters Each file has a unique ID, which is used to download the file. You need to listen to updateFile updates to follow download progress. Since TDLib 1.4.0 you can also use parameter synchronous in downloadFile request to receive the response from downloadFile only after the download is finished.

Do I need to go to the updatefile to determine and download the user's avatar? Isn't it going to use the download in the file as I thought? But I tested it, I used download under file but didn't see the user avatar file I downloaded in the profile_photo directory.

@levlam
Json::Value photoInfoJsonx;
photoInfoJsonx["@type"] = "downloadFile";
photoInfoJsonx["file_id"] = root["id"].asString();
photoInfoJsonx["priority"] = 1;
td_json_client_send(client, photoInfoJsonx.toStyledString().c_str());
This is how I use downloadFile. Do you have any problems with this? I am using it under the type of file.

@AquariusHunters Does root["id"].asString(); is an identifier of the file, i.e. user['profile_photo']['small']['id'] or user['profile_photo']['big']['id']?
updateFile is an update. It will be sent whenever anything about a file, including number of downloaded bytes, is changed.

@levlam
I use FileTypeJson["@type"] = "fileTypeProfilePhoto" ;
getRemoteFileJson["@type"] = "getRemoteFile";
getRemoteFileJson["remote_file_id"] =user_remote_profile_photo["id"].asString();
getRemoteFileJson["file_type"] = FileTypeJson;
Td_json_client_send(client, getRemoteFileJson.toStyledString().c_str());
under Userinfo Come to get it file
{"@type":"file","id":876,"size":0,"expected_size":0,"local":{"@type":"localFile","path":"","can_be_downloaded":true,"can_be_deleted":false,"is_downloading_active":false,"is_downloading_completed":false,"downloaded_prefix_size":0,"downloaded_size":0},"remote":{"@type":"remoteFile","id":"AQADBQADq6cxG23h-RwACJ0n1TIABAb6UKZzz8qSzzYCAAEC","is_uploading_active":false,"is_uploading_completed":true,"uploaded_size":0}}
Then use the download according to the id of the file Such as 876

@AquariusHunters The ID is correct. The download of the file begins after call to downloadFile. After the call you need to wait for updateFile for the file with file["local"]["is_downloading_active"] == false. If after the downloading is finished file["local"]["is_downloading_completed"] == true, then in file["local"]["path"] there will be a path to the file. If you use TDLib 1.4.0 and pass "synchronous" == true parameter to downloadFile, then you can handle only downloadFile response instead of updateFile updates.

@AquariusHunters The ID is correct. The download of the file begins after call to downloadFile. After the call you need to wait for updateFile for the file with file["local"]["is_downloading_active"] == false. If after the downloading is finished file["local"]["is_downloading_completed"] == true, then in file["local"]["path"] there will be a path to the file. If you use TDLib 1.4.0 and pass "synchronous" == true parameter to downloadFile, then you can handle only downloadFile response instead of updateFile updates.

Thanks for your help, I have already got profile_photo, but I found a problem. Some of the Userinfo I got from UpdateUser have no profile_photo information. I think this is not normal. Every user should not have one. Profile_photo? Even this avatar is the default.

No. A lot of user has no profile photo. In that case it is generated client-side from first letters of user's name.

thank you very much ，I will be helpless without your help.


[Data Discovery] Use new metadata fields instead of sample model fields.

Use only metadata fields (in particular, for location and sample tissue/type) in projects and samples routes.

Cool!
(1) What is the test plan for this PR?
(2) Will https://github.com/chanzuckerberg/idseq-web/blob/2f0026b12800246af9c9cbe81bc3db6076486d39/app/models/sample.rb#L147 be edited/deprecated as well?

Cool!
(1) What is the test plan for this PR?
(2) Will
idseq-web/app/models/sample.rb
Line 147 in 2f0026b
     OR samples.sample_tissue LIKE :search

be edited/deprecated as well?

No new tests, because nothing changed regarding access control.
This PR changes endpoints used in DD. Added a prefix on title to make that more clear.
So, test plan was just checking projects and samples list return the new fields.
There are two other places where sample_tissue appears: that sample search endpoint which not being used right now by DD, and the phylo tree creation flow (for extra info). These two will eventually be fixed after the DD sprint.

For my education, is sample type intended to be more general than sample tissue?

Yeah, I believe so. Definitely is a superset of sample tissue.


Shorter X-Axis labels on bar graphs produce wider bars, longer labels= narrow bars

Expected behavior
Datalabel length  for X-axis labels should have no effect on bar width
Current Behavior
Shortening the length of the text in an X-Axis label widens the bar above it.
https://jsfiddle.net/pqs6te04/2/
Steps to Reproduce

Click on the "Relabel" button and notice how the bars widen and shrink based on label size.

Context
This issue becomes particularly pronounced in a small graph with only two bars occupying a small area of the screen.

Fiddle updated. Confirmed issue still exists.

Thanks! Very weird indeed, though I think it's fixed in master (maybe by #5884, @nagix)

Its fixed in master: https://jsfiddle.net/pqs6te04/2/

Closing as duplication of #5818 and fixed by #5884


Artificer: fix design in "All actions" block on profile view pages

we can use this block when we disable page covers but it looks awful


Timestamp changelog

Timestamps makes it easier to interpret / read the changelog, to understand e.g. release cadence.

Great idea. Thank you


[Star tree] Star tree file formats refactoring and fixing offset bug

Description
This PR removes the exposure of DocIdSetIterator in the file formats flow.
So, in order to prevent exposing DISI directly as part of star tree file format readers , this change contains a simple wrapper iterator that abstracts the DISI , and exposes relevant methods tied to star tree to retrieve the values.
Apart from this also fixing a bug where file size greater than Integer.MaxValue is not handled correctly.
Related Issues
https://github.com/opensearch-project/OpenSearch/issues/15974 - issue for the bug found with offsets

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks good, thanks!
I'm not a huge fan of how many of these classes are public.
In a future PR, can we think about how to organize the classes into common packages and make most of these classes package-private? That would make the "public API" a lot clearer.

Thanks Froh for the comments. Yes, makes sense to make most of these classes package private, will take this up as a followup.

Flaky test: https://github.com/opensearch-project/OpenSearch/issues/15944

The PR branch also has this 2.18.0 tag in bwcversion: https://github.com/bharath-techie/OpenSearch/blob/starrefactor/.ci/bwcVersions
Still bwc is failing:
Execution failed for task ':distribution:bwc:staged:buildBwcLinuxTar'.
> Building 2.17.0 didn't generate expected file /var/jenkins/workspace/gradle-check/search/distribution/bwc/staged/build/bwc/checkout-2.17/distribution/archives/linux-tar/build/distributions/opensearch-min-2.17.0-SNAPSHOT-linux-x64.tar.gz

https://github.com/opensearch-project/OpenSearch/issues/14324 flaky test


feat(mobile): add separate config flags to toggle build

This suggestion was brought up by @kara.
During development, compiling the App Shell can be a burden: It takes a while to bundle files into a single concatenated script, the compilation often fails because of a missing provider or other non-prerender-friendly code. However, compiling during development also has advantages, like making sure the transition from shell to real app is seamless, making sure compilation works, etc.
So it would be nice to be able to easily disable app shell compilation during dev build.
Right now, the only option in angular-cli.json for mobile projects is a single mobile: true setting. This could be expanded to be an object with more granular configuration, one of which could modify what mobile pieces are built during dev and prod:
angular-cli.json
mobile: {
  dev: {
    buildAppShell: true,
    buildServiceWorker: false,
    buildAppManifest: true
  },
  prod: {
    buildAppShell: true,
    buildServiceWorker: false,
    buildAppManifest: true
  }
}

The downside to this design is that the build becomes more complicated. For example, right now, Service Worker assumes that App Shell is being built, and therefore only caches the bundled JS plus static non-JS assets. With more configuration, the Service Worker would have to figure out which JS tree it should be caching.

@jeffbcross is this still an issue on the webpack branch?

@filipesilva this would still be a nice feature to have.

I can't wait for this feature especially after the Angular Universal integration as well. Creating progressive web app will be so easy and integrated nicely. So excited!!

Closing as it this isn't being acted on anymore. See https://github.com/angular/angular-cli/issues/2228 for context.

Haha. I caused this issue to close. Sort of by saying something. But I understand @filipesilva. Only so much time :) Thanks for the reply.

Kinda, yeah :D. When you commented I got an email that prompted me to review it.
Glad you're enjoying the CLI!


Replace wood tools with flint tools

Make sure you are not opening a duplicate.

[X] I have searched the issue tracker and did not find an issue describing my bug/feature.

Description of your suggestion.
It would be cool if there was and option to have flint tools completely replace wood tools, having the same stats as wood tools. Just a nice touch that would make thing make a bit more sense as well as making flint tools a bit more useful.

Mh you would need to disable those with a mod like cubejs.
not a bad idea but not in my vision.


Fix test containers

This patch fixes the test containers, which were failing since git is now required for building/running the admin interface.

Tested the docker image with:
docker run --rm -it -p <IP_ADDRESS>:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-945
The git dependency is found now, but it can't find the git repo, it seems.
fatal: not a git repository (or any of the parent directories): .git
failed to load config from /admin-interface/vite.config.ts
error when starting dev server:
Error: Command failed: git rev-parse HEAD
fatal: not a git repository (or any of the parent directories): .git

    at genericNodeError (node:internal/errors:984:15)
...

Should be fixed now

Though it works it's using the default:
/bin/sh: git: not found
cat: can't open 'commit': No such file or directory


Added dbengine build and runtime dependencies libuv, liblz4, Judy, and openssl.

Per the latest documents at https://docs.netdata.cloud/packaging/installer/ netdata requires the packages libuv, liblz4, Judy and openssl to support the new dbengine memory mode. The dev versions of those libraries are required when building netdata and then only the normal versions required thereafter to run it. Addresses #88

To configure Netdata to use the dbengine see the documentation here

@titpetric

@titpetric Any chance you would have time to review this PR?

Thanks for the contribution. I muted the repo so I only have time on occasion to check it, I appreciate the effort however, so please don't be discouraged :)


Preselect "create index in background" option when creating indexes i…

Scope & Purpose
Preselect "create index in background" option when creating indexes in the web
UI. The "create index in background" option can be less intrusive because it
allows other write operations on the collection to proceed.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[ ] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

Tests passed except chaos suite which don't exist in 3.6.13.


QBean doesn't work with superclass' fields

If i create a QBean using new QBean(myType, true, Map<String, ? extends Expression<?>> bindings);, it doesn't see the fields of the superclass of myType.
The prblem is in initFields():

Class beanType = type;
while (!beanType.equals(Object.class)){
    try {
       Field field = type.getDeclaredField(property);  // Should be "Field field = beanType..."
       ...
    } catch (NoSuchFieldException e) {
        beanType = beanType.getSuperclass();
    }
}

Released in 2.2.4


Add dataVolume on delete validation

Description:
currently, the user is able to delete the dataVolume from the UI when it is owned by a VM, and it will cause an API error on the VM deletion.
Solution:

prevent the user to delete data volumes that are currently owned by the VM.
fix VM dataVolume not-found error on VM deleting.

Related Issue:
https://github.com/rancher/harvester/issues/221

@gitlawr updated, thanks.


Fix typos in OverpassGatewayTimeout class

Coverage remained the same at 94.301% when pulling cf57466a35e0578b32e2822bfac667491d6bd546 on gregrs-uk:fix-typo into 969889826a85ea0c70bcd6a844c0badf62a447e0 on DinoTools:master.

👍

@phibos Please merge :)

Thanks for your contribution.


Set the default group size to 10,000

Adjusted to be in line with the README

Fix for #6

@washingtoneg thanks for the PR! We actually changed it from 10k to 2k for performance reasons that we were noticing. We'd prefer to keep it at 2k, could you instead update the README to match the code?

@sgringwe, thanks for reviewing! I made the change you requested in de32d36. Could you please elaborate on the performance issues you noticed? I'm curious and I'd like to use this code with as much context as you are willing to share.
Thanks for open sourcing this!

@washingtoneg thanks! I don't remember the exacts on the performance, unfortunately. can you look into the test failures?

@washingtoneg disreard the tests - your changes wouldn't have caused that


Initial commit for containerized Apache reverse proxy

This PR defines an Apache2 reverse proxy suitable for use with RStudio or Jupyter notebooks. It includes GitHub workflows to run tests and build and publish the proxy as a Docker container to GHCR.

would you mind adding/updating this PR's title and description to reflect it's purpose?

Done.

Also why does the old repo still exists at https://github.com/Sage-Bionetworks/notebook-reverse-proxy ?

I just deleted it.

Here's an example of a workflow that incorporates my suggestions #2

I incorporated your suggestions in to the PR.


[WFLY-15375] Fix unstable text

https://issues.redhat.com/browse/WFLY-15375
Increase ryuk timeout
Modify assume() to check for Docker in setup
Remove "always pull" image policy
Move test container setup to separate class

Nit:
s/Fix unstable text/Fix unstable test/g
;)

Nit:
s/Fix unstable text/Fix unstable test/g
;)

Just noticed that as well. Fixed in the commit. :)


nixos/cfdyndns: add option to use CF token

Description of changes
Adds an option to set a file path containing a cloudflare API token, instead of using an api key/email combo

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

LGTM, I'll merge soon unless someone else protests and thinks the assert is necessary.

hey, it looked like that PR was ready to merge, so I went ahead, I hope it's fine by everyone.

also small thing I just noticed looking at this module. Perhaps we should run it as a DynamicUser ?
(I didn't see the explicit need to define a user and a group, maybe I missed something. Unless someone is against it, I'll make a PR at some point).