Expand compat features with a lock file

Referencing compat features will be cumbersome for very large feature groups (e.g., the Temporal API). We ought to have a way to both a) reference many features with a small notation (e.g., Temporal.*) and b) ensure the stability of the features selected (i.e., do not automatically vacuum up all future features).
Philip suggested a lockfile approach, which sounds quite manageable:

I think we'll need wildcards or some way to list an interface with all of its members, but still notice when we update BCD if the list of matched features changes. One way to think about that is like a lockfile, where we put both the simple-ish input and the expanded output under source control, and ensure they stay in sync through CI.

Originally posted by @foolip in https://github.com/web-platform-dx/feature-set/pull/93#pullrequestreview-1335679692

Since https://github.com/web-platform-dx/web-features/pull/787, we have dist files which take in tags from BCD (the lockfile approach). Since https://github.com/web-platform-dx/web-features/pull/1227, we have compute_from which allows us to take in new items without necessarily accepting them for the headline status. I think this is done.


Fix decode

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.

Please check out how to sign the CLA, @Jamaika1

thanks for the patch, @Jamaika1. The CLA bot is still complaining
https://github.com/libjxl/libjxl/pull/1878/checks?check_run_id=9711389211

I changed the email address associated to the commits so they match the email address you added to the AUTHORS file. That fixed that part of the authors check, however the CLA part is not working with that email address:
https://github.com/libjxl/libjxl/pull/1878/checks?check_run_id=9741969958

Could you take a look at the CLA, @Jamaika1 ?


[BEAM-8473] Fix generator send_type / return_type warning

This fixes the generator warning when None is used -- it was checking for None, while the typehint actually returns NoneType.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aaltay @charlesccychen

Run Python PreCommit

Run Python PreCommit

Run Python PreCommit


Rails/ActionControllerFlashBeforeRender reports false positives with if/else and redirect_to

RuboCop reports a false positive with if/else and redirect_to
Follow up from https://github.com/rubocop/rubocop-rails/pull/812#issuecomment-1287651233

Expected behavior
Rubocop does not report errors
Actual behavior
Inspecting 1 file
C

Offenses:

pages_controller.rb:6:7: C: [Correctable] Rails/ActionControllerFlashBeforeRender: Use flash.now before render.
      flash[:success] = t('.flash.success') # false positive
      ^^^^^
pages_controller.rb:17:7: C: [Correctable] Rails/ActionControllerFlashBeforeRender: Use flash.now before render.
      flash[:notice] = t('.flash.notice') # false positive
      ^^^^^
pages_controller.rb:25:7: C: [Correctable] Rails/ActionControllerFlashBeforeRender: Use flash.now before render.
      flash[:success] = t('.flash.success') # false positive
      ^^^^^
pages_controller.rb:27:7: C: [Correctable] Rails/ActionControllerFlashBeforeRender: Use flash.now before render.
      flash[:alert] = t('.flash.fail') # false positive
      ^^^^^

1 file inspected, 4 offenses detected, 4 offenses autocorrectable

Steps to reproduce the problem

Create the following pages_controller.rb file

# frozen_string_literal: true

class PagesController < ApplicationController
  def update
    if @page.save
      flash[:success] = t('.flash.success') # false positive
    else
      flash.now[:alert] = t('.flash.fail')
      render 'edit'
    end

    redirect_to @page
  end

  def dismiss
    if @something
      flash[:notice] = t('.flash.notice') # false positive
    end

    redirect_to pages_path
  end

  def destroy
    if @page.destroy
      flash[:success] = t('.flash.success') # false positive
    else
      flash[:alert] = t('.flash.fail') # false positive
    end

    redirect_to pages_path
  end
end

Run rubocop --require rubocop-rails --only Rails/ActionControllerFlashBeforeRender pages_controller.rb

RuboCop version
$ rubocop --require rubocop-rails -V
1.37.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.23.0, running on ruby 3.1.2) [x86_64-darwin21]
  - rubocop-rails 2.17.0

Also tested against rubocop-rails main

Hi, I've tested 2.17.2 but there are still false positives. As far as I can tell, some of them are not easy to detect and also providing a reduced test case is tricky
I've started with #843


Implement RBAC: KC: restart permissions #3720

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
all 'Action.EDIT' replaced with 'Action.RESTART'
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] 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
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@Haarolean  http://localhost:8080/api/clusters/local/connects//connectors
this is because we do not get any connect and therefore the link does not have the connect_name of the connection.
this is where we get the name of the connect_name
const { data: connects } = useConnects(clusterName);
If there is only one connect, The selection connect form is not shown in the form and the name of the connect is set automatically.

so if there is no connect at all, then the form will not show at all

we need to know where to get the name of the connect  if this code returns 0 connect

this is because we do not get any connect and therefore the link does not have the connect_name of the connection.

@David-DB88 the problem is introduced within this branch, it works fine on master.


Questions about uncertainty based implementation

Hi, Chengcheng Guo and Bo Zhao:
Thanks for your thorough research and clean codes. However, I have some questions about uncertainty based implementation.
As mentioned in the DeepCore paper, samples with lower confidence may have a greater impact on model optimization than those with higher confidence, and should therefore be included in the coreset. But the implementation here actually calculate the inverse scores of uncertainty.
Take entropy as an example, np.log(preds + 1e-6) * preds is the negative of the entropy, so np.argsort(scores)[::-1][:self.coreset_size] select the samples with low entropy (uncertainty). This confused me a lot, which shows inconsistant implementation with the statement in the paper. Is there some bugs in the implementation?
Data-reindeer

The code selects samples with large entropy. Code np.argsort(scores)[::-1][:self.coreset_size]select samples with smaller scores, where scores are negative of the entropy. Smaller score means larger entropy.
For example, there are two samples with predicted probability [0.1, 0.9] and [0.4, 0.6], respectively. We can calculate their entropies, which are $e_1=-0.1ln(0.1)-0.9ln(0.9)=0.325$ and $e_2=-0.4ln(0.4)-0.6ln(0.6)=0.673$. The algorithm prefers sample 2 rather than sample 1 (because of larger entropy). I think this is consistent with what is stated in the paper.

Thanks for your reply. I think there are some misunderstanding about function np.argsort()
I have tried the following codes
scores = np.array(range(10))
result = np.argsort(scores)[::-1]
result2 = np.argsort(scores)
and print the results
result = [9 8 7 6 5 4 3 2 1 0]
result2 = [0 1 2 3 4 5 6 7 8 9]
The np.argsort sorts the values in ascending order by default, and [::-1] returns the values in descending order. So it seems that the code np.argsort(scores)[::-1][:self.coreset_size] actually select samples with larger scores.

Hi, may I ask if this issue has been resolved? I agree with @Data-reindeer. It seems we are selecting "more confident" samples instead of "less confident" ones as mentioned in the report.


Add AppPolice.app Latest

Thank you for the contribution. It was merged directly as cd8c9a2d15fa720c8120cf97bad217480b0784dd to keep commit history cleaner. Your contribution is still credited to you.


fix: Adding optional dependencies for Snowflake

Summary of Changes
Bunch of fixes related to Snowflake integration:

Dependencies for Snowflake aren't listed. This adds them but comments them out since we don't want these to be installed for all users even when they aren't using Snowflake. Updates the documentation. This is inspired by #139 which never made it in.
If the Snowflake schema is lower case, the metadata extraction query fails. Quoting the name of the schema fixes it.
Some minor docs changes in a few places.

Tests
Helps with dev environment so no test changes. Existing tests pass.
Documentation
Updated how to use Snowflake extractor
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test

The dependency is listed optional in setup.py.

Thanks @feng-tao
Probably a dumb question: How do you decide what goes in requirements.txt vs. setup.py
I read this and took away the conclusion that these dependencies should be in requirements.txt in addition to setup.py since they were needed for dev, not just packaging.
Do you think about that differently?

We don't need the requirements.txt file. You could generate one based on setup.py. I will delete it later once remove the CI dependency.

Got it, let me fix and update this PR, thanks!

We don't need the requirements.txt file. You could generate one based on setup.py. I will delete it later once remove the CI dependency.
There are a bunch of places in the documentation that refer to the requirements.txt as well (like #4 under https://github.com/amundsen-io/amundsen/blob/master/docs/installation.md#bootstrap-a-default-version-of-amundsen-using-docker). Should we change all references to requirements.txt docs (including the link) above too then?


QLever missing features or unexpected behavior

https://qlever.cs.uni-freiburg.de/wikidata/QhTtmx

It does respond, with an error message. The SERVICE keyword is not yet supported, but it's on our list.
It's valuable for us if you report features that are not yet implemented. But it's not so helpful when you open a separate issue for each one of them. Instead, I would suggest to open one issue "QLever missing features or unexpected behavior" and collect queries there.

Moved to #615

In my case there was no response on first attempt and than later i saw the same error message as you are mentioning. Unfortunately the "no response" effect seems not to be reproducible.


Version 4.0 add command FIND a line in the filename

Command FIND with FIND_MODE : a string with SUBSTRING (Default) or EQUALS and a regular expression with REGEX_FIND (contains) and REGEX_MATCH (entire region the pattern)

I'll do release into Maven, then will re-start the GH Action.
Ideally, we should have separate PR for changing the descriptor, for better coordination


Cross site scripting [self xss] on simplemde-markdown-editor

Hi,
there is vulnerability XSS on simplemde editor ,
steps :
1 - Go to https://simplemde.com/
2 - copy this '>  or  or click me! and etc
3 - paste on simplemde-markdown-editor
4 - click to preview
5 - you see execute javascript code,
Thanks,
Ramin

Correct me if I'm wrong, but this is due to Markdown allowing HTML to be used within it, and the problem is just that their demo doesn't disable that. It's not an inherent error/danger in SimpleMDE itself?

i don't know on SimpleMDE safe mode is enable or no in README, if you have test with  you will filter javascript tag in SimpleMDE, on mardown html tag should filter, like img , svg and ...

This is due to Markdown allowing HTML to be used alongside the Markdown syntax. SimpleMDE makes no effort to filter this client side. Your own application code should worry about handling these types of attacks.

I would be super nice to add this as a recommendation in the docs for previewRender.


[#release] prepare for 2.1.1

2.1.1-SNAPSHOT -> 2.1.1

Codecov Report

Merging #7426 (9bab1b7) into master (b353ef9) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7426      +/-   ##
==========================================
+ Coverage   38.74%   38.76%   +0.02%
==========================================
  Files        3332     3332
  Lines       96197    96197
  Branches    11901    11901
==========================================
+ Hits        37269    37292      +23
+ Misses      56012    55990      -22
+ Partials     2916     2915       -1

Impacted Files
Coverage Δ

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70.00% <0.00%> (-3.34%)
:arrow_down:

...ercorp/pinpoint/profiler/util/ThreadDumpUtils.java
62.12% <0.00%> (+1.51%)
:arrow_up:

...itor/metric/deadlock/ThreadDumpMetricSnapshot.java
70.58% <0.00%> (+2.35%)
:arrow_up:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
80.48% <0.00%> (+2.43%)
:arrow_up:

...ntext/thrift/ThreadDumpThriftMessageConverter.java
92.85% <0.00%> (+3.57%)
:arrow_up:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
88.74% <0.00%> (+4.63%)
:arrow_up:

...m/navercorp/pinpoint/web/util/ThreadDumpUtils.java
39.04% <0.00%> (+5.71%)
:arrow_up:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
85.00% <0.00%> (+10.00%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
68.00% <0.00%> (+16.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Interactive Tutorial - Configuring a Java Microservice

Hi,
the maven package command lines aren't working. here are the traces.
also, please provide with the source code. the Interactive Tutorial is not practical to see a microservice. And we could run it locally.
It would be nice to have a tutorial on the manifest before this one.
Thanks,
remaining at your disposal,
V
1st app
[ERROR] Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.10:build (default) on project system: Could not build image: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
$
2nd app
[ERROR] Unknown lifecycle phase "mvn". You must specify a valid lifecycle phase or a goal in the format : or :[:]:. Available lifecycle phases are: validate, initialize, generate-sources, process-sources, generate-resources, process-resources, compile, process-classes, generate-test-sources, process-test-sources, generate-test-resources, process-test-resources, test-compile, process-test-classes, test, prepare-package, package, pre-integration-test, integration-test, post-integration-test, verify, install, deploy, pre-clean, clean, post-clean, pre-site, site, post-site, site-deploy. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/LifecyclePhaseNotFoundException
$

/triage duplicate

There's no solution really; I updated https://github.com/kubernetes/website/issues/37994 to reflect my new understanding of the bug.
/kind bug
/language en

Per this comment, I am closing this issue. Thanks!
/close


merge from template-coveralls

package.json

chore(deps): add ^7.23.8 remove ^7.23.7 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
GitHub release
v5.0.44

Your semantic-release bot :package::rocket:


Bug: Play spam then leave results in crash

Needs repro, but pushing multiple play commands then leaving can crash the bot.

Able to reproduce - seem to be an issue with how the leave command was implemented.
Fixed in 1.5.7 by using Queue#leave


Event cretion not working with voice audio

Interesting project like it. tested in local:
Start voice audio and give some task put it in calendar events  but not working. In the screen always this error:
"TypeError Cannot read properties of undefined (reading 'tool_calls')"
In the  console log
chat {
id: 'chatcmpl-90MvTAiVlyVJ52ckbG1oPoMhO8wvY',
object: 'chat.completion',
created:<PHONE_NUMBER>,
model: 'gpt-3.5-turbo-1106',
choices: [
{
index: 0,
message: [Object],
logprobs: null,
finish_reason: 'tool_calls'
}
],
usage: { prompt_tokens: 1352, completion_tokens: 20, total_tokens: 1372 },
system_fingerprint: 'fp_f93e21ed76'
}
assistant1 {
role: 'assistant',
content: null,
tool_calls: [
{
id: 'call_VFklkG0uRwqcanqJF3jvH16n',
type: 'function',
function: [Object]
}
]
}
LoopCount: 1
data-response undefined
isArray false
Exactly throwing this error :
console.log("data-response", data.response.tool_calls)

console.log("isArray", Array.isArray(data.response.tool_calls))

Can you please check?
Thanks.

Exactly function calls are not working :
tool_calls:[{
used : model: 'gpt-3.5-turbo-1106',
Can you able to validate this please.

Do you have a mongodb installed locally? It is used in saving the context and for the calendar and memory data.


Patrick:  Do you want to change your eMail on the Wiki/Contact Us page...

... from your hawaii.edu to the gmail.com account.
Also, I checked the gmail account my calendar program is using to send invites your way... it's just what you wrote down.  You should have received 3x invites by now.  Let me know if you didn't get `em.

Have updated email on the wiki page, now set to gmail.com.
Still not receiving calendar invites, no clue why.  Try sending a direct email?

Sent a direct eMail and 2x calendar invites (one to each account).  The calendar invites do not show up in my outbound mailbox.

Direct email received no issues.

Finally got the calendar invites to my hawaii.edu account, still haven't received them to my gmail.com account.
I'm not going to worry about it; we usually are good about checking in on meetings as they come up.


Image displayed is not matching with the index parameter

Describe the bug
I'm giving to the carousel the index parameter since I want to totally control the carousel. The bug is if I slide the carousel but for some external reason in my program I dont want the index to change, the image will still change but the arrows and dot buttons will still be corrects and remains at the right index.
To Reproduce
Steps to reproduce the behavior:

Declare a variable to store the index and give it to the carousel
Do not update this variable in the callback onIndexChange
See error : the image is still changing while the arrows and buttons are not following the same behavior and remains at the index passed in the component.

Expected behavior
Since the props index is passed, the displayed image should follow the index and shouldn't have it's own behavior.
Desktop:

OS: Windows 10
Browser: Opera GX

Hi @BarthelemyPassinCauneau ! Thank you for raising this issue!
I will check it tomorrow!

Hi @BarthelemyPassinCauneau !
I think I understood this issue. The index prop for specifying the initial index. I am sorry that I did not make this clear in the documentation. I think it is not possible to totally control the carousel as of now.

Hi!
@BarthelemyPassinCauneau I assume you could instead use the goToIndex(index) on ref instead of passing the index as a prop. As an example:
 const carouselRef = useRef(null);

  const openGalleryAtIndex = (index) => {
      console.log('index is..', index);
      if (carouselRef.current !== null) {
          carouselRef.current.goToIndex(index);
          carouselRef.current.maximize();
      }
  };

Hi @acewebsolutions ! Thanks for your suggestions! 😀


bugs found by @ykawashima for v1.4.1

See #404, but I made a new issue for the record (with adding numbering).

[ ] 1. I would like to confirm whether the change of the file path in the retrieval tutorial (0af60abd305a04aed50889d158c9f9e77c337c70) is correct or not. I think it is more straightforward to mention the tutorial notebook itself. Or did you intend to mention this python file? (In the latter case, some parts need to be fixed to be compatible with the current version.)
[x] 2. I got the following error for tests/integration/premodit/checindex.py due to the change of the input parameters of init_premodit.

  File "/home/kawashima/exojax/tests/integration/premodit/checkindex.py", line 28, in <module>
    ngamma_ref_grid_H2O, n_Texp_grid_H2O, R_H2O, pmarray_H2O = initspec.init_premodit(
TypeError: init_premodit() got an unexpected keyword argument 'interval_contrast'

[x] 3. I got the following error for tests/integration/premodit/line_strength_comparison_exomol.py due to the change of the input parameters of unbiased_ngamma_grid. Could you add Tref_broadening parameter? The same thing applies to tests/integration/premodit/line_strength_comparison_hitemp.py. Thank you.

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/premodit/line_strength_comparison_exomol.py", line 78, in <module>
    ngamma_grid = unbiased_ngamma_grid(Ttest, P, ngamma_ref_grid, n_Texp_grid,
TypeError: unbiased_ngamma_grid() missing 1 required positional argument: 'Tref_broadening'

[x] 4. I got the following errors for tests/integration/moldb/comparison_api.py and tests/integration/moldb/comparison_api_xs.py. I think these two tests are no more needed given that MdbExomol and MdbHit have been discarded in exojax.spec.moldb.

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/moldb/comparison_api.py", line 8, in <module>
    from exojax.spec.moldb import MdbExomol as MdbExomol_orig
ImportError: cannot import name 'MdbExomol' from 'exojax.spec.moldb' (/home/kawashima/anaconda3/lib/python3.9/site-packages/ExoJAX-1.4-py3.9.egg/exojax/spec/moldb.py)

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/moldb/comparison_api_xs.py", line 28, in <module>
    mdbCO_moldb = moldb.MdbHit('~/exojax/data/CO/05_hit12.par', nus)
AttributeError: module 'exojax.spec.moldb' has no attribute 'MdbHit'

[ ] 5. I got the following error for tests/integration/moldb/quantum_states_filter_exomol.py. Maybe v_u and v_l should be replaced by j_upper and j_lower?

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/moldb/quantum_states_filter_exomol.py", line 31, in <module>
    mask = mdb.df["v_u"] - mdb.df["v_l"] == dv
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/vaex/dataframe.py", line 5395, in __getitem__
    self.validate_expression(item)
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/vaex/dataframe.py", line 3446, in validate_expression
    raise NameError(str(e)) from None
NameError: Column or variable 'v_u' does not exist. Did you mean: 'v_l'

[ ] 6. I got the following errors, saying that the lengeth of mdb.Sij0 is just 1,  for tests/integration/moldb/quantum_states_filter_hitemp.py and tests/integration/moldb/quantum_states_filter_hitran_co.py.
I cannot recognize the line where the line strength is assigned when the activate function is used. Or do I misunderstand?

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/moldb/quantum_states_filter_hitemp.py", line 32, in <module>
    plt.plot(1.e4 / mdb.nu_lines,
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/pyplot.py", line 2740, in plot
    return gca().plot(
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_axes.py", line 1662, in plot
    lines = [*self._get_lines(*args, data=data, **kwargs)]
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_base.py", line 311, in __call__
    yield from self._plot_args(
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_base.py", line 504, in _plot_args
    raise ValueError(f"x and y must have same first dimension, but "
ValueError: x and y must have same first dimension, but have shapes (1300,) and (1,)

Traceback (most recent call last):
  File "/home/kawashima/exojax/tests/integration/moldb/quantum_states_filter_hitran_co.py", line 28, in <module>
    plt.plot(1.e4 / mdb.nu_lines,
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/pyplot.py", line 2740, in plot
    return gca().plot(
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_axes.py", line 1662, in plot
    lines = [*self._get_lines(*args, data=data, **kwargs)]
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_base.py", line 311, in __call__
    yield from self._plot_args(
  File "/home/kawashima/anaconda3/lib/python3.9/site-packages/matplotlib/axes/_base.py", line 504, in _plot_args
    raise ValueError(f"x and y must have same first dimension, but "
ValueError: x and y must have same first dimension, but have shapes (634,) and (1,)

is really bug. It should have v_l and v_u and it had, but now is not available...

Sij0 is now deprecated. Use line_strength_ref.

tests/integration/moldb/quantum_states_filter_hitran_co.py). requires develop branch of radis.

Thanks. The former one is correct. I think all of the bugs have been fixed.


Remove local clock feature

Description
This PR removes the local clock feature (node['ntp']['use_cmos']) because this is no longer recommended practice
See here for reference.
Issues Resolved
List any existing issues this PR resolves
#161
Check List

[x] A summary of changes made is included in the CHANGELOG under ## Unreleased
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable.

Released as: 5.0.0


python311Packages.cliff: 4.2.0 -> 4.3.0

Description of changes

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.

Result of nixpkgs-review pr 241637 run on x86_64-linux 1

10 packages marked as broken and skipped:

python310Packages.optuna
python310Packages.optuna.dist
python310Packages.oslo-db
python310Packages.oslo-db.dist
python310Packages.subunit2sql
python310Packages.subunit2sql.dist
python311Packages.oslo-db
python311Packages.oslo-db.dist
python311Packages.subunit2sql
python311Packages.subunit2sql.dist

6 packages failed to build:

python311Packages.oslo-log
python311Packages.oslo-log.dist
python311Packages.python-manilaclient
python311Packages.python-manilaclient.dist
python311Packages.tempest
python311Packages.tempest.dist

94 packages built:

glanceclient (python310Packages.python-glanceclient)
glanceclient.dist (python310Packages.python-glanceclient.dist)
heatclient (python310Packages.python-heatclient)
heatclient.dist (python310Packages.python-heatclient.dist)
ironicclient (python310Packages.python-ironicclient)
ironicclient.dist (python310Packages.python-ironicclient.dist)
jenkins-job-builder (python310Packages.jenkins-job-builder)
jenkins-job-builder.dist (python310Packages.jenkins-job-builder.dist)
manilaclient (python310Packages.python-manilaclient)
manilaclient.dist (python310Packages.python-manilaclient.dist)
openstackclient (python310Packages.python-openstackclient)
openstackclient.dist (python310Packages.python-openstackclient.dist)
python310Packages.cliff
python310Packages.cliff.dist
python310Packages.hacking
python310Packages.hacking.dist
python310Packages.keystoneauth1
python310Packages.keystoneauth1.dist
python310Packages.openstacksdk
python310Packages.openstacksdk.dist
python310Packages.osc-lib
python310Packages.osc-lib.dist
python310Packages.oslo-concurrency
python310Packages.oslo-concurrency.dist
python310Packages.oslo-config
python310Packages.oslo-config.dist
python310Packages.oslo-context
python310Packages.oslo-context.dist
python310Packages.oslo-i18n
python310Packages.oslo-i18n.dist
python310Packages.oslo-log
python310Packages.oslo-log.dist
python310Packages.oslo-serialization
python310Packages.oslo-serialization.dist
python310Packages.oslo-utils
python310Packages.oslo-utils.dist
python310Packages.python-cinderclient
python310Packages.python-cinderclient.dist
python310Packages.python-jenkins
python310Packages.python-jenkins.dist
python310Packages.python-keystoneclient
python310Packages.python-keystoneclient.dist
python310Packages.python-novaclient
python310Packages.python-novaclient.dist
swiftclient (python310Packages.python-swiftclient)
swiftclient.dist (python310Packages.python-swiftclient.dist)
python310Packages.stestr
python310Packages.stestr.dist
python310Packages.tempest
python310Packages.tempest.dist
python311Packages.cliff
python311Packages.cliff.dist
python311Packages.hacking
python311Packages.hacking.dist
python311Packages.jenkins-job-builder
python311Packages.jenkins-job-builder.dist
python311Packages.keystoneauth1
python311Packages.keystoneauth1.dist
python311Packages.openstacksdk
python311Packages.openstacksdk.dist
python311Packages.osc-lib
python311Packages.osc-lib.dist
python311Packages.oslo-concurrency
python311Packages.oslo-concurrency.dist
python311Packages.oslo-config
python311Packages.oslo-config.dist
python311Packages.oslo-context
python311Packages.oslo-context.dist
python311Packages.oslo-i18n
python311Packages.oslo-i18n.dist
python311Packages.oslo-serialization
python311Packages.oslo-serialization.dist
python311Packages.oslo-utils
python311Packages.oslo-utils.dist
python311Packages.python-cinderclient
python311Packages.python-cinderclient.dist
python311Packages.python-glanceclient
python311Packages.python-glanceclient.dist
python311Packages.python-heatclient
python311Packages.python-heatclient.dist
python311Packages.python-ironicclient
python311Packages.python-ironicclient.dist
python311Packages.python-jenkins
python311Packages.python-jenkins.dist
python311Packages.python-keystoneclient
python311Packages.python-keystoneclient.dist
python311Packages.python-novaclient
python311Packages.python-novaclient.dist
python311Packages.python-openstackclient
python311Packages.python-openstackclient.dist
python311Packages.python-swiftclient
python311Packages.python-swiftclient.dist
python311Packages.stestr
python311Packages.stestr.dist

python311Packages.oslo-log
python311Packages.oslo-log.dist
python311Packages.python-manilaclient
python311Packages.python-manilaclient.dist
python311Packages.tempest
python311Packages.tempest.dist

openstack has bad python 3.11 support. I am going to take a look at them in the next days and maybe mark some things broken.

Result of nixpkgs-review pr 241637 run on x86_64-linux 1

10 packages marked as broken and skipped:

python310Packages.optuna
python310Packages.optuna.dist
python310Packages.oslo-db
python310Packages.oslo-db.dist
python310Packages.subunit2sql
python310Packages.subunit2sql.dist
python311Packages.oslo-db
python311Packages.oslo-db.dist
python311Packages.subunit2sql
python311Packages.subunit2sql.dist

100 packages built:

glanceclient (python310Packages.python-glanceclient)
glanceclient.dist (python310Packages.python-glanceclient.dist)
heatclient (python310Packages.python-heatclient)
heatclient.dist (python310Packages.python-heatclient.dist)
ironicclient (python310Packages.python-ironicclient)
ironicclient.dist (python310Packages.python-ironicclient.dist)
jenkins-job-builder (python310Packages.jenkins-job-builder)
jenkins-job-builder.dist (python310Packages.jenkins-job-builder.dist)
manilaclient (python310Packages.python-manilaclient)
manilaclient.dist (python310Packages.python-manilaclient.dist)
openstackclient (python310Packages.python-openstackclient)
openstackclient.dist (python310Packages.python-openstackclient.dist)
python310Packages.cliff
python310Packages.cliff.dist
python310Packages.hacking
python310Packages.hacking.dist
python310Packages.keystoneauth1
python310Packages.keystoneauth1.dist
python310Packages.openstacksdk
python310Packages.openstacksdk.dist
python310Packages.osc-lib
python310Packages.osc-lib.dist
python310Packages.oslo-concurrency
python310Packages.oslo-concurrency.dist
python310Packages.oslo-config
python310Packages.oslo-config.dist
python310Packages.oslo-context
python310Packages.oslo-context.dist
python310Packages.oslo-i18n
python310Packages.oslo-i18n.dist
python310Packages.oslo-log
python310Packages.oslo-log.dist
python310Packages.oslo-serialization
python310Packages.oslo-serialization.dist
python310Packages.oslo-utils
python310Packages.oslo-utils.dist
python310Packages.python-cinderclient
python310Packages.python-cinderclient.dist
python310Packages.python-jenkins
python310Packages.python-jenkins.dist
python310Packages.python-keystoneclient
python310Packages.python-keystoneclient.dist
python310Packages.python-novaclient
python310Packages.python-novaclient.dist
swiftclient (python310Packages.python-swiftclient)
swiftclient.dist (python310Packages.python-swiftclient.dist)
python310Packages.stestr
python310Packages.stestr.dist
python310Packages.tempest
python310Packages.tempest.dist
python311Packages.cliff
python311Packages.cliff.dist
python311Packages.hacking
python311Packages.hacking.dist
python311Packages.jenkins-job-builder
python311Packages.jenkins-job-builder.dist
python311Packages.keystoneauth1
python311Packages.keystoneauth1.dist
python311Packages.openstacksdk
python311Packages.openstacksdk.dist
python311Packages.osc-lib
python311Packages.osc-lib.dist
python311Packages.oslo-concurrency
python311Packages.oslo-concurrency.dist
python311Packages.oslo-config
python311Packages.oslo-config.dist
python311Packages.oslo-context
python311Packages.oslo-context.dist
python311Packages.oslo-i18n
python311Packages.oslo-i18n.dist
python311Packages.oslo-log
python311Packages.oslo-log.dist
python311Packages.oslo-serialization
python311Packages.oslo-serialization.dist
python311Packages.oslo-utils
python311Packages.oslo-utils.dist
python311Packages.python-cinderclient
python311Packages.python-cinderclient.dist
python311Packages.python-glanceclient
python311Packages.python-glanceclient.dist
python311Packages.python-heatclient
python311Packages.python-heatclient.dist
python311Packages.python-ironicclient
python311Packages.python-ironicclient.dist
python311Packages.python-jenkins
python311Packages.python-jenkins.dist
python311Packages.python-keystoneclient
python311Packages.python-keystoneclient.dist
python311Packages.python-manilaclient
python311Packages.python-manilaclient.dist
python311Packages.python-novaclient
python311Packages.python-novaclient.dist
python311Packages.python-openstackclient
python311Packages.python-openstackclient.dist
python311Packages.python-swiftclient
python311Packages.python-swiftclient.dist
python311Packages.stestr
python311Packages.stestr.dist
python311Packages.tempest
python311Packages.tempest.dist

Update is part of #251878.


v9.309 Lose all pipe/power connections when outside 'chunk' area even if area chunkloaded

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
Whether or not you are running a chunk loader (FTBUtils for example) .309 still loses all connections
and all my machines shut down.  Including my reactor.
Steps to reproduce:

Chunk load using FTBUtils (or any other chunk loader
Make sure area is showing chunkloaded
Go outside what would be the normal distance for the chunk to be unloaded

Version (make sure you are on the latest version before reporting):
Forge: <IP_ADDRESS>
Mekanism: <IP_ADDRESS>9
Other relevant version:  Minecraft 1.11.2
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

Sorry.  Forge <IP_ADDRESS>3.  My bad

Did you verify it still happens with another chunkloader? I could be completely mistaken but I remember reading something about FTBLib lets the chunk unload and then reloads it chunkloaded.

Yes.  I tried using one called "achunkloader" and it does the same thing.  But thanx for the suggestion!
-Rick

hopefully this will give some additional insight to this problem.
MC 1.11.2
Forge Version 2393
Mek Versions 311
I wasn't using a chunkloader, logged in to find all mek. pipes not working. All pipes had to be busted and replaced. This includes pressurized, Mechanical and universal. RF readers would show universals running at anywhere from -21 to 4RF. Any universal that was already in place had to be broken and replaced, not even connecting them  to newly placed working universals would fix them. The RF flow would stop  at the ones that have not yet been replaced.
Things to note:
I was running the digital miner a few chunks away using an entangelporter. So maybe something went wrong with that interaction while not logged in (??). But, I have done something similar a day or so ago with no problems either.

Can anyone provide a strict reproduction scenario which will lead to this issue? I'd really like to get this solved soon but I've been unable to reproduce anything similar.

Forge: <IP_ADDRESS>0
Mekanism: Mekanism-1.11.2-<IP_ADDRESS>9.jar
I am using Silent's Gems, which provide an item that can teleport you.  If I set it up so that I can get from one place to another using them, and the chunks are far enough apart that they unload when I move to the other one, I can pretty reliably cause it to happen by teleporting between the two places.  If I then teleport away and back, often everything will start working again.
Notably, going between dimensions seems to make it happen far more regularly.  For example, since I have Galacticraft, teleporting between say the moon and the overworld base will trigger it in either location, but then teleporting back and forth will get things going again. Also, Galacticraft pipes don't seem to be affected if that helps.  It's .. Interesting to return to your moon base and the oxygen system has failed! :)

Forge: 2426
Mekanism: 1.12-9.3.3.homebaked
Server: yes
Digital Miner lost all upgrades and setting after leaving dimension it was in/ just being unloaded even with an anchor upgrade. I feel those are somewhat related.

Digital miner is a totally separated issue, this is about transmitters (fluid, energy, heat, gasses) losing connections. The digital miner has several other issues ;)

@thommy101 i think it actually is related being that the issue is the failure to recover from a chunk load/unload

If that would be the case, a lot more than only transmitters and digital miners would be affected. Digital miner has some known bugs with chunkloading not working and missing settings which are unrelated to this issue.

I've spotted the issue; definitely wasn't an easy task. I'm experimenting with fixes right now and should have this addressed in a public release by the end of the week.

Fixed in 1.11 and 1.12 development branches!

@aidancbrady do you mind pointing/referencing the commit and/or PR for educational purposes?

9801b12b62d93a21b8ed5774fcb6f38fd847980a

Here's a quick explanation I wrote up when I found the issue:
Basically, every time TransmitterNetworkRegistry is ticked, orphan transmitters are handled: either a network is created for them, they are added to an existing network, or a new network is formed that merges several and also adds new transmitters
Transmitters are not immediately added to the actual transmitter set of a DynamicNetwork, however. TransmitterNetworkRegistry needs to be ticked again for the changes to be “committed.”
Now, let’s say multiple network merges happen in a single tick to try and form a single large network. If a network has “pending” transmitters when it is merging with a larger network, those are going to be lost in the change.
This results in transmitters not being added to the complete network, and the existence of “ghost” networks of unadded transmitters that are not registered with TransmitterNetworkRegistry.
Because they’re technically in individual networks, they’re not considered orphans. This is why you still have to break every single one to get the network to work again.

@aidancbrady this is still happening in 1.12, allthemods 3 v5.8
If i leave my chunk for too long and come back every single cable/pipe/etc has to be replaced in order to work.

A) this issue was created for 1.11
B) did you bother searching? #4676

@thiakil no i manually typed this GitHub url guessing the issue #4507, what are the chances.


2024.04.03 Sorting on number in stock is LSCS assignment window not working

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Press assign LCSC
Press column to sort on stock
Sorting is not correct

Expected behavior
A clear and concise description of what you expected to happen.
KiCad Version

Application: KiCad PCB Editor x86_64 on x86_64

Version: 8.0.2-unknown-202405031934~57c8fbc9fe~ubuntu20.04.1, release build

Libraries:
	wxWidgets 3.2.1
	FreeType 2.10.1
	HarfBuzz 2.6.4
	FontConfig 2.13.1
	libcurl/7.68.0 OpenSSL/1.1.1f zlib/1.2.11 brotli/1.0.7 libidn2/2.2.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh/0.9.3/openssl/zlib nghttp2/1.40.0 librtmp/2.3

Platform: Ubuntu 20.04.6 LTS, 64 bit, Little endian, wxGTK, X11, ubuntu, x11

Build Info:
	Date: May  3 2024 19:34:19
	wxWidgets: 3.2.1 (wchar_t,wx containers) GTK+ 3.24
	Boost: 1.71.0
	OCC: 7.6.3
	Curl: 7.68.0
	ngspice: 42
	Compiler: GCC 9.4.0 with C++ ABI 1013

Build settings:

Can confirm that sorting by stock does not work.

I think this can be closed as fixed by https://github.com/Bouni/kicad-jlcpcb-tools/commit/42782481804cd0b89eedcd7fbbbf8283a038d6b6 , which is in release https://github.com/Bouni/kicad-jlcpcb-tools/releases/tag/2024.07.01 .


[FEATURE] Sticky header

Description
If we provide the sticky header then it will be helpful for users to navigate
Expected Behavior
when the user scrolls down,the header will stick to the top, so if the user wants to navigate to another link then he/she does not need to scroll the page.
Actual Behavior
Here the header will disappear when the user is scrolling

We will do this once the homepage is ready

@Dharmik3 please work on it!

Can i work on it?

@Vijaykv5 I work on it

I've made an PR. Can you review it?

@Dharmik3 please work on it

I made the PR #163 please review it.

Please see those reviews


Enforce Control Plane RBAC settings

User story
As an admin, I want the RBAC settings from the Control Plane enforced on the data plane.
Acceptance criteria

[ ] Only tokens from the configured IDPs are accepted
[ ] Only tokens containing the configured audiences are accepted
[ ] Only roles assigned to a user are enforced for that user
[ ] Permissions not assigned to the user are not allowed
[ ] When values are updated, the changes are reflected

Moving work item tracking to Azure DevOps.


Ploting the n_counts/per cell on umap

Hi,
Is there a way to visualize the n_counts or n_genes for each cells in a tsne or umap plot much like you do in cellranger html outputs (see attachment)?

Thanks for your help.

Never mind, that was silly easy sorry.

Good you found it! :)


Restore sanity

Description

Update information on how to use local yaml to speed up makefile builds
Add v3.6 to list of ignored dirs for _config_dev.yml

The lines I replaced are as follows:
# Note in order to install local (faster) yaml: "go get github.com/mikefarah/yaml"
YAML_CMD:=$(shell which yaml || echo docker run --rm -i calico/yaml)

When I run go get github.com/mikefarah/yaml, it does not install a go binary called yaml. When I run go get mikefarah/yq.v2, it installs a binary called yq.v2. The output of yq.v2 --help closely resembles that of  docker run --rm -i calico/yaml --help which leads me to believe that mikefarah split the binary into a separate repository from the library code.
You can validate this change by running any make step and noticing it takes ~4 seconds before executing. Then when you run go get mikefarah/yq.v2, your makefile steps will now take <1 second to start.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 6889aab4251290aeccbe17265720ba9d9e7622f4
https://deploy-preview-2573--calico.netlify.com

@neiljerram I updated the description of the PR with additional information.

Ahhhh my sanity is restored. Thanks @neiljerram


UG: Reorganize the content page for better presentation

Now we have 20 features all nested under one section, we should split them into more sections.

done


FancyZones not working with multiple displays in combination with portable apps

Microsoft PowerToys version
0.67.1
Installation method
GitHub, PowerToys auto-update
Running as admin
None
Area(s) with issue?
FancyZones
Steps to reproduce
Preinfo:

The FancyZone feature is working on the Main Display the described Bug appears only at the additional displays in my case the right one or the left one from the main display
According to my observation the error only occurs with portable programs (in my Case with (putty/kitty=<IP_ADDRESS> or keepass portable=2.53.1)
Already tried as Administrator

Precondition:

Use minimum 2 Displays

Create FancyZones with diffrent sizes
Download the Portable Version of Keepass oder Putty or Kitty
Open the Portable Applications
Try to Snap Keepass Putty oder Kitty to a Fancy Zone

Screencast:
https://user-images.githubusercontent.com/22096341/222428178-f0f7bc64-e9c7-4bb0-b056-cb514b0a03a8.mp4
✔ Expected Behavior
Window is adjusting and snapping to the selected Fancy Zone area/zone
❌ Actual Behavior
Window is sizing to the choosen fancy zone but shifts to the left
Other Software
kitty <IP_ADDRESS>
keepass portable 2.53.1

PowerToysReport_2023-03-02-10-38-26.zip

the same behavior described in my screencast also happens with notepad++
(notepad++ v8.4.4 is installed and not a portable version)
so its probably not only limited to portable apps

my current version PowerToys Version is now 0.68.0

If i deactivate the checkbox for transparency while dragging windows, the behavior described above doesnt applies to notepad and keepass, but putty/kitty is still not snapping into the fancyzone

small update on the topic
I now have 4 screens of which 3 are UHD screens and 1 is a FHD screen when turning off the FHD screen the scaling of the windows works properly even on the non-main screens.
I think it is a general problem of position calculation. Which is not directly noticeable in the other components.
For example, it is obvious to see when the window is kept in the foreground.
This is visible in the frame of the window, which shows an enormous shift in relation to the window itself.
After moving the window, I took the screenshots one by one and put them together.

My current Version is v0.72.0

I noticed that when the screen scaling is set to 100%, the problematic windows of the applications can be arranged normally again. maybe this is another interesting piece of information


Return moniker information

Currently the Keel plugin does not return expected moniker information. This is needed for frontend to render properly.
Specifically, the moniker object is missing.
{
	"applicationPaused": false,
	"hasManagedResources": false,
	"currentEnvironmentConstraints": [],
	"resources": [{
		"id": "k8s:resource:keel-dev-service-keeltest",
		"kind": "k8s/resource@v1",
		"status": "ACTUATING",
		"displayName": "keel-dev-service-keeltest",
		"locations": {
			"account": "deploy-keel-dev",
			"vpc": "vpc0",
			"regions": []
		}
	}, {
		"id": "ec2:security-group:mccloman-1:keeltest-dev-sgs",
		"kind": "ec2/security-group@v1",
		"status": "HAPPY",
		"moniker": {
			"app": "keeltest",
			"stack": "dev",
			"detail": "sgs"
		},
		"displayName": "keeltest-dev-sgs",
		"locations": {
			"account": "mccloman-1",
			"vpc": "Test",
			"regions": [{
				"name": "us-west-2"
			}]
		}
	}]
}

Closing this for now. If moniker is specified for each resource it does work. It's not a good solution but limitation comes from MD UI interface.


Boolean default not taken if written after help

Hi,
I recently encountered an error when trying to setup a default value for boolean options, whenever I also include a help instruction within the menu.
The following works in setting the default value correctly
menu "General options"
	config Kconf_Options_Foo
		bool "This is the foo option"
		default y
endmenu

This, in turn, does not work:
menu "General options"
	config Kconf_Options_Foo
		bool "This is the foo option"
                help "This setups foo, which causes baz"
		default y
endmenu

Inverting default and help prevents parsing of KConfig file
menu "General options"
	config Kconf_Options_Foo
		bool "This is the foo option"
                default y
                help "This setups foo, which causes baz"
endmenu

produces

menuconfig.py: error: expected 'endmenu' at end of 'Kconfig'

I'm using the latest pip package (14.1.0).
Is this expected behavior? Am I doing something wrong?
Btw I've seen issue #121 however it's not clear to me whether a takeover is already taking place or not; if so, I'd like to report this issue over to the new repo, if any exist.
Thanks!

I know this is bit of an archaeology to reply to this, but maybe it will help somebody in the future. :)
The true issue is that the syntax is incorrect; help text should be at a new line with indentation:
  menu "General options"

      config IDF_FOO
          bool "This is the foo option"
          help
              "Some help text"
          default y

  endmenu

Hey @Honza0297 thanks for the explanation! Incidentally I started using the right syntax months after and forgot about this issue. Much appreciated!


[Yunikorn 2001] The link to Download the configuration file is Wrong

What is this PR for?
for fixing the wrong link in Deploy to Kubernetes page.
I find same queues.yaml in 1.1.0 version. And Recently it's move to master/deployments/scheduler/ folder
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

What is the Jira issue?
[Yunikorn 2001] The link to Download the configuration file is Wrong
How should this be tested?
./local_build run (make sure all related json files are created in .docusaurus)
./local_build run zh-cn (This step will create additional translated json files in .docusaurus)
Screenshots (if appropriate)

Closed in favor of #338.


Decode payload with URLEncoding per the JWT spec

Please provide a description of this PR:
Per the JWT spec, all three sections of the token are to be base64url encoded.  The current implementation uses the raw base64 decoder for the payload which causes problems if the encoded payload contains underscore or dash characters.  This PR changes the payload decoding to use URLEncoding.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ X] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ X] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @kellinwood. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/ok-to-test

@kellinwood: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

integ-helm_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-helm

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

integ-helm_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-helm

integ-security_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

integ-helm_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-helm

integ-security_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security

integ-security-istiodremote_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security-istiodremote

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests-arm64

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

integ-helm_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-helm

integ-security_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security

integ-security-istiodremote_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security-istiodremote

integ-security-multicluster_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security-multicluster

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test unit-tests

integ-helm_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-helm

integ-security_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security

integ-security-istiodremote_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security-istiodremote

integ-security-multicluster_istio
1d3d45dc90cf5107476bae0690908a8d7b4f5500
link
true
/test integ-security-multicluster

gencheck_istio
50cff01fedbf048b1d1e1b9061b7ac114dd547c2
link
true
/test gencheck

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

/retest

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
50cff01fedbf048b1d1e1b9061b7ac114dd547c2
link
true
/test gencheck

lint_istio
50cff01fedbf048b1d1e1b9061b7ac114dd547c2
link
true
/test lint

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

@kellinwood: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
50cff01fedbf048b1d1e1b9061b7ac114dd547c2
link
true
/test lint

gencheck_istio
50cff01fedbf048b1d1e1b9061b7ac114dd547c2
link
true
/test gencheck

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

@howardjohn Hi.  I'm looking for approval from the 'WG - Networking Maintainers' group.  It looks like you're not only a member, but a previous contributor to jwtutil.go, so I thought you might be a good person to contact regarding getting this PR approved and mergable.  Thanks


Update SPIRV-Cross to expose argument buffer binding

Includes #100 (to fix the failed test)

Thank you for the test!


Expensify Card - Empty card details page can be opened after deactivating card

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.71-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Expensify Card feature is enabled.
User has assigned themselves a virtual card.

Go to staging.new.expensify.com
Go to workspace settings > Expensify Card.
Click on the virtual card.
Click Deactivate card.
Deactivate the card.
Click browser forward button.

Expected Result:
Not here page should open because the card is deactivated.
Actual Result:
Empty card details page opens. It can be edited, which leads to "Hidden" profile in Expensify Card list.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/af6e1e72-3718-48df-bfca-0c79290f3463
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Expensify Card - Empty card details page can be opened after deactivating card
What is the root cause of that problem?
We haven't restricted access to the detail card page if a user tries to view an invalid card
What changes do you think we should make in order to solve the problem?
In WorkspaceExpensifyCardDetailsPage we need to implement the same logic as we did in WorkspaceCompanyCardDetailsPage
https://github.com/Expensify/App/blob/d839c4dd34e81b6d5d19f1a5708d0d256cdcfa1c/src/pages/workspace/companyCards/WorkspaceCompanyCardDetailsPage.tsx#L75-L77
    if (!card && !isLoadingOnyxValue(cardsListMetadata)) {
        return <NotFoundPage />;
    }

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

@mountiny This seems like a lack of when implementing this feature. Could I take over this one?

yeah if the cardId is not locally we should show not found page

@thelullabyy's proposal looks good to me
@twilight2294 I noticed that you mentioned another bug in your proposal: "When deactivating the card, the card is removed before the modal is dismissed. It caused the empty value to be displayed in a moment". But I believe that we shouldn't add pendingAction to the deactivating card action because this action follows Offline Pattern C
@thelullabyy Let's also resolve this additional bug in your PR. One potential approach that we already used widely in our codebase is to deactivate the card after the modal is closed completely by using runAfterInteractions.
🎀 👀 🎀 C+ Reviewed

@twilight2294 I just checked again

Company Card: Applying offline pattern B
Expensify Card: Applying offline pattern C

It isn't the same.

@twilight2294 No, the company cards is a simple update in our database hence pattern B but for anything related to Expensify card we also need to communication with thirdparty api and that can fail for unknown reasons hence we only want users to be able to do take these critical actions online

@thelullabyy Any update?

@DylanDylann I created a draft PR, I just need to test and finish all videos before opening to review.

@thelullabyy - I'm having trouble finding you in Upwork. Please apply for the job in Upwork, and I can complete the payment process. Thanks!

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

@thelullabyy - I'm having trouble finding you in Upwork. Please apply for the job in Upwork, and I can complete the payment process. Thanks!

@alexpensify Applied!

Thanks, I've sent offers to @thelullabyy and @DylanDylann in Upwork. Please accept and I can complete the payment process.

@alexpensify Accepted, ty!

All set and closing, everyone has been paid via Upwork. Since I'm OOO until January 3, I'm going to close and work on the QA step when I'm back online.


[Drag and drop] Questions

Hello,
I use the drag and drop for a project, and I would like get the position X and Y of my object, is possible ?
I would like also move my object with TypeScript, how can I put my object in X 25px and Y 32px just with TS ?
Thank's in advance.
Jérémy

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.


Incorrect buffer clear circumstances

Keys in that range ignore (*mods & ~MOD_MASK_SHIFT) != 0 condition after the switch so ctrl + backspace didnt clear the buffer because KC_1 ... KC_SLASH contains KC_BACKSPACE
It works in this case, but im not sure if we should actually just return true there ignoring other mods, have to think about it more probably, like ctrl + 1 also wont clear the buffer
^ irrelevant after the second commit

It should just fallback to the default processing, it returns true is shift is the only mode either

I have merged this PR, because, even though this might not be a complete fix to all of our key interaction issues, it does fix a real and present problem without introducing any new problems. And it doesn't complicate the code for when a more thorough refactoring of this code is done in the future


fatal error: set identifier: file FoldingCell.swift

I am getting the above mentioned error, seems like a method, configureDefaultState(), within FoldingCell.swift, is checking for "ForegroundViewTop" and "ContainerViewTop".
Maybe clearer documentation on how to add those 2 in the storyboard.

Adding screenshot for reference

add video https://vimeo.com/154954299 to readme.md
I hope this help

I doing this in a programatically way. In fact I'm also using easypeasy like the example, but I always getting the error set ForegroundViewTop or ContainerViewTop constraint.
I update the code:
extension DemoFoldingCell {
private func createForegroundView() -> RotatedView {
    let foregroundView =  RotatedView(frame: .zero)
    foregroundView.backgroundColor = UIColor.red
    foregroundView.translatesAutoresizingMaskIntoConstraints = false
    contentView.addSubview(foregroundView)

    foregroundView.easy.layout(
        Height(75),
        Left(8),
        Right(8)
    )
    let constraint = foregroundView.easy.layout( Top(8).to(self, .top) ).first
    constraint?.identifier = "ForegroundViewTop"
    foregroundView.layoutIfNeeded()

    return foregroundView
}

private func createContainerView() -> UIView {
    let containerView = UIView(frame: .zero)
    containerView.backgroundColor = UIColor.green
    containerView.translatesAutoresizingMaskIntoConstraints = false
    contentView.addSubview(containerView)

    containerView.easy.layout(
        Height(CGFloat(75 * itemCount)),
        Left(8),
        Right(8)
    )

    let constraint = containerView.easy.layout( Top(8).to(self, .top)  ).first
    constraint?.identifier = "ContainerViewTop"
    containerView.layoutIfNeeded()

    return containerView
}

}
Thanks
Francisco

@fbarraganef  I rewrite code like you but have an issue on this line:
fatalError("set foregroundViewTop or containerViewTop outlets in storyboard")
How you solve it?


New interface for networking standard version 4

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Get Inputs metadata from 'pachctl inspect datum' during a job runtime

What is the goal / desired outcome?
For cases where Pachyderm is required to run long running jobs, some datums are bound to fail and/or recover. From an incident response perspective, logs and metrics can spot pretty quickly the problematic datum ids. However, to effectively investigate the problematic datums, It is desirable to see datum related input metadata as it allow for reproduction.
Currently, 'pachctl inspect datum' only returns such metadata when the job finishes, even though the inputs metadata is available internally in Pachyderm at job start.
If there is a way to accomplish this today via workaround, what does that require?
Wait for a job to finish, which can be quite long. Alternatively, add specific logging in the user code and parse it in the logs.
(Optional) What is your proposal for a feature to solve this?
'pachctl inspect datum' should return the partially available metadata, with emphasis on the datum Inputs even before the job finishes.
Environment?:

Pachyderm CLI version (use pachctl version):
Pachyderm Hub, 1.11.0

@RaananHadar what metadata are you looking for here? I think the inspect datum response might already include the information you're looking for it just doesn't get rendered in the CLI pretty printer. If you add --raw to your command it will print out the raw response object which includes some more information. If what you want is in there then it's just a question of adding it to the pretty printer which we can do pretty easily.

Greetings @jdoliner. The metadata I am looking for is already available in inspect datum: its the datum inputs (which files were yielded by the glob).  Even when attempting a --raw didn't solve this. I am betting that the inputs to the datum are one of the first things that are available when runtime starts and it would be extremely beneficial to print them even before the commit finishes.

@RaananHadar that makes sense, I just looked at the code and the issue here is that we use the stats commit, which gets created during the job merge as the lookup table for datums, without it we have no lookup table so can't return a response for inspect datum we should be able to get this to work though, in the worst case we can traverse the datums looking for the right one but we could also probably do something smarter. In the meantime I think you can get the information you want by doing list datum --raw and searching for the ID you want, sorry I realize that's inconvenient.

Thanks @jdoliner i'll definitely look into that.


fatal: Value receiver for function call is dead.

Got this Artichoke bug / fatal output when running the class specs:
An exception occurred during: loading core/class/new_spec.rb
fatal: Value receiver for function call is dead. This indicates a bug in the mruby garbage collector. Please leave a comment at https://github.com/artichoke/artichoke/issues/1336.

(eval):590:in each
/artichoke/virtual_root/src/lib/spec_runner.rb:74:in run_specs
(eval):1

This was the spec config:
# This config file lists the largest set of ruby/specs from the `core` group
# that will let Artichoke Ruby run the spec-runner harness to completion.
#
# Valid values for `include` are:
#
# - all - run all specs. When `include` is set to `all`, the optional `skip`
#   field may list specs to skip.
# - none - run no specs, equivalent to the section not being present in this
#   config file.
# - `set` - run an enumerated set of specs. When `include` is set to `set`, the
#   set of specs must be listed in the required `specs` field, which is a list
#   of strings.

## Ruby Core

[specs.core.argf]
include = "all"

[specs.core.array]
include = "set"
specs = [
  "any",
  "append",
  "array",
  "assoc",
  "at",
  "clear",
  "collect",
  "combination",
  "compact",
  "concat",
  "constructor",
  "count",
  "cycle",
  "delete",
  "delete_at",
  "delete_if",
  "drop",
  "drop_while",
  "each",
  "each_index",
  "empty",
  "first",
  "frozen",
  "include",
  "last",
  "length",
  "map",
  "multiply",
  "plus",
  "prepend",
  "push",
  "rassoc",
  "replace",
  "reverse",
  "reverse_each",
  "shift",
  "size",
  "sort_by",
  "to_ary",
  "try_convert",
  "unshift",
]

[specs.core.basicobject]
include = "all"

[specs.core.binding]
include = "none"

[specs.core.builtin_constants]
include = "all"

[specs.core.class]
include = "all"

[specs.core.comparable]
include = "none"

[specs.core.complex]
include = "none"

[specs.core.dir]
include = "none"

[specs.core.encoding]
include = "none"

[specs.core.enumerable]
include = "none"

[specs.core.enumerator]
include = "none"

[specs.core.env]
include = "none"

[specs.core.exception]
include = "none"

[specs.core.false]
include = "none"

[specs.core.fiber]
include = "none"

[specs.core.file]
include = "none"

[specs.core.filetest]
include = "none"

[specs.core.float]
include = "none"

[specs.core.gc]
include = "none"

[specs.core.hash]
include = "none"
skip = [
  # This generates a `SystemStackError` on recursive hashes that takes an
  # eternity to print out when running with the `artichok` or `yaml` formatter.
  "eql",
  # This generates a `SystemStackError` on recursive hashes that takes an
  # eternity to print out when running with the `artichok` or `yaml` formatter.
  "equal_value",
]

[specs.core.integer]
include = "none"

[specs.core.io]
include = "none"

[specs.core.kernel]
include = "none"

[specs.core.main]
include = "none"

[specs.core.marshal]
include = "none"

[specs.core.matchdata]
include = "none"

[specs.core.math]
include = "none"

[specs.core.method]
include = "none"

[specs.core.module]
include = "none"

[specs.core.mutex]
include = "none"

[specs.core.nil]
include = "none"

[specs.core.numeric]
include = "none"

[specs.core.objectspace]
include = "none"

[specs.core.proc]
include = "none"

[specs.core.process]
include = "none"

[specs.core.queue]
include = "none"

[specs.core.random]
include = "none"

[specs.core.range]
include = "none"

[specs.core.rational]
include = "none"

[specs.core.regexp]
include = "none"

[specs.core.signal]
include = "none"

[specs.core.sizedqueue]
include = "none"

[specs.core.string]
include = "none"

[specs.core.struct]
include = "none"

[specs.core.symbol]
include = "none"

[specs.core.systemexit]
include = "none"

[specs.core.thread]
include = "none"

[specs.core.threadgroup]
include = "none"

[specs.core.time]
include = "none"

[specs.core.tracepoint]
include = "none"

[specs.core.true]
include = "none"

[specs.core.unboundmethod]
include = "none"

[specs.core.warning]
include = "none"

Also see this in ARGF when creating fixtures from text files.
Array#first when doing implicit conversion to int.
I think we might be missing an arena in the implicit conversion functions.

Some of these appear to be from codegen errors that are not really catchable.
See below for one example:

https://github.com/mruby/mruby/issues/5752

Backports for codegen and syntax error fixes is tracked here:

https://github.com/artichoke/mruby/issues/14


DBBranches: Split up unpause from setCheckpoint

DBBranches: Split up unpause from setCheckpoint
Fixes issue:
https://github.com/METR/vivaria/issues/267
The situation before this PR:
unpause would also get a new checkpoint to set.
Bugs it caused:

Issue 267, when the checkpoint was null
Sometimes the call to unpause didn't take this into account (see how the call to unpaused, after the fix, doesn't call setCheckpoint)

The situation was also bad because
These are different functionalities, the client can just call them one after the other.
What was tested

Added one test so far (for this functionality exactly)
Made sure the integration tests didn't break (more than they were already broken (another PR coming up for that))

Missing in the PR

The new test might be duplicating an existing test
Some endpoints were tested (a theoretical bug was fixed in them), but no test was added to make sure that bug doesn't exist anymore
(By default I'll fix these things, but if reviewers feel the PR is good enough then I'll just merge. I felt strongly that I better open a PR already instead of trying to improve everything. Meanwhile at least people can comment)

This code was mostly pair programmed
with @tbroadley

@mtaran, I'm also adding a test that @tbroadley wanted, that makes sure a bug we found is now fixed (you can't see the code yet because I'm on a plane and git push isn't working)

@tbroadley, we spoke about the new test maybe being a duplicate (or potential extension) of an existing test: I think it isn't: Searching general_routes.test.ts for unpauseAgentBranch (which is the interesting method we changed), I only find it in tests on whether certain pause/unpause reasons are allowed, which I think is meaningfully different (and I wouldn't personally merge those tests). (unless you disagree)

Hey, could we merge this? I don't have permissions to:

(Note there were more commits added since the first review)

It's officially co-authored with you now, @mtaran

May I merge? @mtaran

If you got the permissions 😂. Otherwise I can push the button.


Response headers when using +Model

I'm using +Model to define what my resource is and then in the actions I reference my model. Here is an example of my resource
## Store [/stores]

+ Model (application/json)

    + Body

            {
                "id": "1234",
                "name": "My Store"

            }

    + Schema

            {
                Some JSON Schema
            }

Here we see an example of a PUT operation:
### Update Store [PUT /stores/{id}]

+ Parameters
    + id (string, required) - The identifiers for the schema .

+ Request

    [Store][]

+ Response 200

    [Store][]

The PUT request will include If-None-Match header and the response will include ETag header. Is it possible to specify request headers and response headers when using models?
Thanks,
Gaston

I think you can use Attributes if you want to make the request and responses DRY but still have the capacity to add Headers.
Resource Model is a concept which is limited about what it can do.

Thanks for the response. I will explore the option of using 'includes' then. Not as strong as expressing what is the model of a resource but at least I can define the model in one place.


When I tried to train the code for smacv2, I encountered this error message——AssertionError: check recurrent policy!

The detailed errors are as follows：
Traceback (most recent call last):
File "../train/train_smac.py", line 260, in
main(sys.argv[1:])
File "../train/train_smac.py", line 138, in main
"check recurrent policy!")
AssertionError: check recurrent policy!
What should I do？

just update the code. Try again~btw: just delete assert line


fix: aligns rendering of hunks with gitsigns

Introduction
nvim-scrollbar doesn't render the hunks in the same way as gitsigns. This becomes obvious when you put them side-to-side.
Discrepancy between gitsigns and nvim-scrollbar
Here you can see the difference between gitsigns (colors on the left) and nvim-scrollbar (colors on the right).

gitsigns and nvim-scrollbar in-sync
To align rendering, we need to split the changes in a "change" hunk to separate added rows from deleted rows.

This is great, love your theme!

Thanks! My terminal is WezTerm with "Tokyo Day" as its theme. In Neovim I use "Catppuccin Latte".

Thank you for the PR

You're welcome, and thanks for merging!


Small JcloudsLocation refactor

I took the opportunity to move a number of the supporting classes in JcloudsLocation to their own top-level classes. I particular I touched RebindToMachinePredicate, the logic for creating users, and all of the CustomizeTemplateOptions and CustomizeTemplateBuilders.
It's a fairly large PR but I haven't introduced much that's new or changed any existing functionality. Most of it is ASF license headers.
Suggest reviewing it with whitespace changes disabled (append ?w=1 to GitHub's URL).

P.S. to preempt Jenkins: tests pass locally.

Thanks @sjcorbett - great to separate out this huge class, and improve this code!
I haven't had a proper chance to review yet. Does it change the names of any classes that might be referenced in persisted state? Do we need to add anything to deserializingClassRenames.properties?

@aledsage My intention was to avoid anything that was persisted. Of the classes touched:

RebindToMachinePredicate was a private static class that is created in getRebindToMachinePredicate rather than injected through config.
UserCreation shouldn't have been persisted unless someone has done something strange, but still exists and is usable as before.
Nobody should have persisted SUPPORTED_TEMPLATE_OPTIONS_PROPERTIES and SUPPORTED_TEMPLATE_BUILDER_PROPERTIES. All related classes were anonymous inner classes.

Thanks @sjcorbett - looks great. Unfortunately there are merge conflicts again (probably due to https://github.com/apache/brooklyn-server/pull/496).
Once those are resolved, and you've taken a look through the comments to see what you think is worth addressing, then this is good to merge. Happy for you to do that as soon as you're ready, before any more conflicts come in!

Looks good @sjcorbett, a useful tidy-up to this big class; have had a look through PR but nothing to add to above comments.

Thanks @aledsage and @geomacy. I will merge this now.


feat: add buildin loaders for simplifying usage

I noticed you add a feature about using react/typescript loader to transform code. Nice feature.
Maybe could let loader more simplifier by removing babel or loaders script.
like this:
<!--
<script<EMAIL_ADDRESS><script src="https://unpkg.com/inline-module/dist/loaders.js"></script>
-->
<script src="https://unpkg.com/inline-module/dist/core.js" setup></script>

core.js can scan type="inline-script" script automatically and can check whether import babel/react/typescript loader or not.
I pull a request to review, it's counting on you.

The idea is pretty well but users may prefer other CDN files such as https://cdnjs.com/libraries/babel-standalone. I think we should give more choices to users.


Rewrite CLI with argparse

I got frustrated with the current CLI & rewrote it...
Right now you need to separate the headers passed via --headers with whatever is passed as --separator, so it needs to be a tab delimited list if you're working with a TSV, or a pipe delimited list if you're working with any of the BitTorrent tracker site dumps. This doesn't match the --help text, which says it's a comma separated list, and none of these match the convention set by the GNU coreutils (space separated lists of args).
Also, it seemed odd that the filename was supposed to be passed before the flag args - usually positional args come last. I ended up getting rid of the filename arg entirely, so rather than passing a file to the CLI, we read everything from STDIN & write to STDOUT. This way we don't need to do require('fs') or handle any errors associated with the filesystem.
This PR also fixes handling of EPIPE (when the output is piped into something like head), automatically generates the --help output, correctly passes the escape arg, and improves the validation of args passed to the CLI.

@slang800 thanks for putting this PR together all those years back. I've stepped in to help maintain the project and am just catching up on the outstanding pull requests. A few notes about yours:

Looks like stdin support in the CLI was added a while back, but after you opened your PR here.
At a point the CLI moved to use minimist, which doesn't have the same positional requirements you cited.

If you'd like to open a separate PR:

exit-on-epipe is  still a good idea, and we'd be happy to use that

I'm also doing a lot of re-org and re-documentation in the v2.0 branch that addresses some of the doc points you made.
Going to close this PR for now, as it seems the bulk of the changes have already been addressed.


SAMZA-1327: create zk namespace if does not exist

Closing the request, because we decided not to create the name space for now.


Update the libraries in JSFiddle

So now angular is at 1.3.15, leaflet at 7.3, this lib at whatever is hosted at http://tombatossals.github.io/angular-leaflet-directive/dist/angular-leaflet-directive.min.js
=)

Perfect @Maistho, thanks for noticing it.


runner-axe not provide all the axe core results

Hi,
after i have installed the runner-axe and run some tests i have noticed that i get not all the results that axe core provide.
for example, i have installed firefox axe add-on and i have test the same page. in the firefox add-on i get many issues. but when i run  runner-axe i get only related to "contrast". am i missing something?
this is my code:
		const CrawlKit = require('crawlkit');
		const AxeRunner = require('crawlkit-runner-axe');
		const crawler = new CrawlKit(website);
		// You could add a finder here in order to audit a whole network of pages
		//crawler.addRunner('aXe', new AxeRunner());
		crawler.addRunner('aXe', new AxeRunner(), {
		  runOnly: {
			  type: 'tag',
			  values: ['wcag2aa']
			}
		});
		crawler.crawl()
			.then((data) => {
				res.send(JSON.stringify(data.results, true, 2));
			}, (err) => console.error(err));

can you paste the firefox output here? I think it might be a case of configuring the runOnly parameter properly. Have you tried omitting that setting completely?

yes i have tried with:
crawler.addRunner('aXe', new AxeRunner());
here is an example:
you can test https://www.elegantthemes.com/
if firefox add-on you get:
37 violations found:
Results:
Buttons must have discernible text - 2 violations
Elements must have sufficient color contrast - 25 violations
Frames must have title attribute - 2 violations
 element must have a lang attribute - 4 violations
Images must have alternate text - 3 violations
Form elements must have labels 1 violation

HI @joscha ,
did you check it?

I had a short look, didn't check out the FF plugin itself though, all should come from https://github.com/dequelabs/axe-core/ -  I think the difference might be the version - runner-axe uses https://github.com/dequelabs/axe-core/blob/master/doc/API.md#api-name-axea11ycheck - have you tried to update the code in index.js to use the new .run API? As you can see in the runner code, it is very simple, there is no special logic other than a direct call to .a11ycheck and passing along any options you might have given.

Also, can you have a look at running your code with debugging enabled as described here: https://github.com/crawlkit/crawlkit#debugging please? Just to make sure its not opening a different website or gets redirected, obstructed by a firewall, etc. and that's why the results are different.


SQL Permissions needed

This page should indicate that MABS when backing up SQL workloads requires the SYSADMIN role.
This is documented under SharePoint for MABS, and under the separate DPM documentation, but not here.

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

ID: 97341fbf-735c-2e5a-258f-223faa6b4724
Version Independent ID: 87e233b3-f1b7-89a6-b1a8-19c46c969730
Content: Back up SQL Server by using Azure Backup Server - Azure Backup
Content Source: articles/backup/backup-azure-sql-mabs.md
Service: backup
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@esparks-ipl Thanks for reaching out to us. The documentation - Back up SQL Server by using Azure Backup Server - Azure Backup doesn't explicitly talk about the permissions required because the same is covered in "Before you start" section - Before you begin, ensure that you have installed and prepared Azure Backup Server. adding a similar section would reflect as duplication of content.
Hope that clarifies!

@SadiqhAhmed-MSFT
The "Install and prepared Azure Backup Server" page does not reference the need for these permissions anywhere that I can see?
Additionally why is it specifically called out in the SharePoint instructions page, but not on the SQL instructions page?
Seems counter-intuitive.

@dcurwin FYI.

@SujayJDesai - can you give your insights into this?
#reassign:SujayJDesai

@esparks-ipl - Thanks for the feedback. We will add the required details. For now please refer DPM article referenced below. We will add this information to MABS as well.
https://docs.microsoft.com/en-us/system-center/dpm/back-up-sql-server?view=sc-dpm-2019#prerequisites-and-limitations

Thanks for your comment.
The document has been updated.
#please-close


[Question] how to put log for debug?

Hi~
I try to put some log message for debugging but it doesn't show up. :(
I hope ktrain has slack ch. for asking someone to this general question.
Here is my trial steps:

put warnings message
warnings.warn('I want to debug')
OR
print('I want to debug')

build and install
$ python setup.py build -f
$ python setup.py install -f

import ktrain

run code

But I can't see any printed message.
Could you someone please advise how to debug?
PS: Ktrain has Slack ch?

There are print and warning statements throughout the library, so not sure why the ones you added are not showing up.  That said, ktrain does suppress some TF warning messages by default.  To see them, you can add this to the top of your script or notebook:
import os
os.environ['SUPPRESS_TF_WARNINGS'] = '0'

Don't see how this would affect your print/warning statements, though.
We're using GitHub Issues as a help forum for right now, so you can continue posting questions here for now. Thanks.


Decide on highest priority EDA functions

...or QA/QC, whatever is most useful to start with!
When I break up tasks into package functions I think of what outputs I want - usually either a table resulting from a specific data summary/analysis, or a figure (map, graph, etc) - and I write one function per output. If I find myself copy-pasting code from one function into another, that's usually a sign that those tasks should be combined into a single function (or the copy-pasted code needs its own function). Often, one function will feed into another - e.g. the tabular output of a summary function can feed into a function that plots that output.
One-function tasks might look like:
Map plot locations
Calculate sample size
Calculate understory density
Calculate understory density % change
Plot understory density % change
General function structure:
SomeUnderstoryAnalysis <- function(park, sample_frame, certified) {
    raw_data <- FilterPACNVeg("Understory", park, sample_frame, certified)

    # Do a single summary or analysis task here - this is often a chain of calls to dplyr functions

    return(summarized_data)
}

Feel free to look through the MOJN spring veg code for some examples. https://github.com/nationalparkservice/mojn-sv-rpackage/tree/analysis/R


Suggested Addition for "How to: Find an Element by Its Name - WPF .NET Framework"

The example's implementation of the Find() method will only find a child element for the specified name that has been added to the Window's or Page's Content as a non-dynamic element. By default,  a dynamic UI Element (e.g., the WPF Framework UIElement was instantiated and and added to the Content of the main view programmatically and not through a XAML declaration)  is not added as a NameScoped mapped object, and any calls to FindName() using a dynamic object's Name will return a null object reference.
If a FrameworkElement has been added to the Content dynamically, and you would still need to get a reference to the named element, the elements name should be registered  to the NameScoped object map by calling the FrameworkElement.RegisterName() method before the element is added to the parent container for a Window or Page's Content.
See Also:
FrameworkElement.RegisterName(String, Object) Method
https://docs.microsoft.com/en-us/dotnet/api/system.windows.frameworkelement.registername?view=windowsdesktop-5.0
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 68b97c0b-22eb-e996-3c07-e0ba81e3e7ef
Version Independent ID: f000a744-c058-153a-f140-3921038e7184
Content: How to: Find an Element by Its Name - WPF .NET Framework
Content Source: dotnet-desktop-guide/framework/wpf/advanced/how-to-find-an-element-by-its-name.md
Product: dotnet-framework
Technology: dotnet-wpf
GitHub Login: @adegeo
Microsoft Alias: adegeo

Thank you for this information. I'll flag this for updating.


Inaccurate UI for child span

Background
I have a java server that is making an RPC call to a go server. The java rpc client and go rpc server are instrumented with lightstep. Everything about the trace looks normal except for where in the lightstep UI, the go rpc server span is placed.
The java span has ts<PHONE_NUMBER>325 which is right before the request is sent to the go server. The go rpc server has 2 timestamps:<PHONE_NUMBER>326 is when it received the request and started the span,<PHONE_NUMBER>336 is after it responded and finished the span.
Problem
I would expect the UI to have the go span horizontally to the immediate right of the java span. Instead, it is far to the right.
The only possible cause I can think of is an incompatibility between v0.10.1 which java code is using and v0.9.1 which go is using. Is this a possibility? Do you have any thoughts on a possible cause?
The go code is essentially:
import (
    lightstep "github.com/lightstep/lightstep-tracer-go"
    opentracing "github.com/opentracing/opentracing-go"
)

tracer := lightstep.NewTracer(lightstep.Options{
    AccessToken: ls.AccessToken,
    Collector:   lightstep.Endpoint{ls.Host, ls.Port, true},
    Tags:        map[string]interface{}{lightstep.ComponentNameKey: component},
})

spanContext, err := tracer.Extract(opentracing.TextMap, opentracing.TextMapCarrier(req.GetLightstepData()))
span = tracer.StartSpan(
    endpoint,
    opentracing.ChildOf(spanContext))
}

// handle the request

span.Finish()

I am using these libraries:
github.com/opentracing/opentracing-go at 6edb48674bd9467b8e91fda004f2bd7202d60ce4
github.com/lightstep/lightstep-tracer-go at 3f04c904839c54177e85df579d910718b09c42a3

Hi @meirf. This sounds like the clock correction code in java may be perturbing the span.
Do you mind going to the  help center (via the doc's link in the top right nav menu in LightStep) and submit a request? That way we can communicate directly and get this resolved.
Thanks,

Ted

Hey Ted,
We have instrumentation in java and go. I've confirmed that of the four permutations, only java->go has this problem.
It turns out java->go isn't a real concern for us.
I have to prioritize real concerns so unfortunately I don't have the bandwidth to communicate further on this right now :(
Meir

Ok thanks! We're aware of this issue - basically some code that helps with bad clocks on android can sometimes mildly mess with good clocks on the server.