Gareth add sort command testing

SortCommandTesting

test sorting list by names in ascending order
test sorting list by names in descending order
test sorting list of professors by name in ascending order
test sorting list of students by name in ascending order
test sorting list of teaching assistants by name in ascending order

OrderTesting

test creating increasing order from string
test creating deceasing order from string
test equals

Any suggestions for ways to test the sort function?

Codecov Report

Merging #61 (d531cc6) into master (92b235e) will decrease coverage by 0.63%.
The diff coverage is 43.33%.

@@             Coverage Diff              @@
##             master      #61      +/-   ##
============================================
- Coverage     62.14%   61.51%   -0.64%
- Complexity      464      500      +36
============================================
  Files            87       91       +4
  Lines          1749     1972     +223
  Branches        208      250      +42
============================================
+ Hits           1087     1213     +126
- Misses          566      633      +67
- Partials         96      126      +30

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/ProfCommand.java
66.66% <ø> (ø)

...a/seedu/address/logic/commands/StudentCommand.java
100.00% <ø> (ø)

...n/java/seedu/address/logic/commands/TaCommand.java
66.66% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
68.18% <0.00%> (-3.25%)
:arrow_down:

...eedu/address/logic/parser/GithubCommandParser.java
0.00% <0.00%> (ø)

...edu/address/logic/parser/StudentCommandParser.java
5.26% <0.00%> (-1.41%)
:arrow_down:

...va/seedu/address/logic/parser/TaCommandParser.java
5.00% <0.00%> (-1.25%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
100.00% <ø> (+4.00%)
:arrow_up:

.../java/seedu/address/model/util/SampleDataUtil.java
13.63% <0.00%> (-6.37%)
:arrow_down:

...address/storage/JsonAdaptedPersonDeserializer.java
77.77% <ø> (ø)

... and 45 more

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


small little fix

I think this is still not going to work because it splits by word, so multiple-word keywords won't work 🤔

yep you're right

We can solve this with a regex that searches for the keyword with beginning line/newline/end line/space. I don't know how to do it but it's definitely possible. Will look into it


wg-ui-and-a11y Status Update 2020-12-02

What was accomplished

Design doc for amp-render completed, design review on 12/02
Code complete for Serving NPA ads (publishing in Q1 2021
amp-ima-video bugfixes
amp-fit-text accessibility fix in review

What's next

Begin implementing amp-render after getting feedback from design review
Begin design for amp-list v2
Working on finalizing design for granular consent
Working on e2e test for amp-fit-text a11y update
Update from GA4 team coming 12/4

Any chance to influence amp-list v2? We have a previously reported issue that might be a candidate for resolution in V2.

We have a previously reported issue that might be a candidate for resolution in V2.

Which issue? Can you link to it? Is it a candidate for being fixed in V1 as well?

@samouri Here it is: https://github.com/ampproject/amphtml/issues/23675
The WaPo link is broken. But the idea is simple. Think this way: a dictionary of terms, one endpoint for each term. Then, for a given page, use only the term needed for (a paragraph). We might use one structure on many pages. The WaPo implementation was an image and the player performance stats. Rather than managing one structure of all players, and then using a hop operator to localize the specific player for display (via amp-mustache), the WaPo design was one player : one end-point. This approach is similar to the PubChem approach for chemicals. I can fetch the specific chemical i need; do not have to load the entire PubChem endpoint and then hop to the target chemical. But amp-list cannot be used inline (e.g. in a span) as it forces a line break.
Our request: remove the line-break caused by amp-list. Then we can build dictionaries of terms, not have to have a dictionary of all terms.

@samouri Here is another (suspect) issue with amp/state/list/mustache: https://github.com/ampproject/amphtml/issues/31532
It appears that some CSS can break amp/state/list/mustache injection and formatting. Do you see a user error in the third example?


add getter for FluidBoundaryFeatures and StratigraphicUnitFeatures

(Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.)
What does this implement/fix? Explain your changes.
implement getter for FluidBoundaryFeatures and StratigraphicUnitFeatures
Does this close any currently open issues?
no
Any relevant logs, error output, etc?
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Any other comments?
…
Where has this been tested?
Operating System: macos with gcc-8.3
Platform: …

ok,
It would be great to have some kind of roadmap to know what to do or not to do in order to avoid spending time on a wrong design.
I have another PR on RockFluidOrganizationInterpretation.
I hope I didn't work for nothing!

I prefer to be honest : I have no clear view on a roadmap except a very high level one (example : ETP1.2 is in the roadmap, tracked github issues are on the roadmap, WITSML2 is on the roadmap). And the roadmap change often according to paying members requirements.
I think the best is:

to raise an issue which I would confirm or not.
work after on this issue.
If you would have raised the issue first, I would not have confirmed it and I would have told you my above message preventing you to spend work for nothing.
So please, do not work on something before to ask it. Or, you will always take this kind of risk to have your PR simply rejected.

In case of RockFluidOrganizationInterpretation, I have already confirmed the need. A review will be obviously done (maybe with request for change) but the need at least is confirmed.
Sorry not to be better organized but keep in mind there is only one person (sometimes 1.2 people) behind fesapi who has also to provide other works to keep Fesapi free.

I made wrong assumptions for this one :)
The next needs obviously a review process :D

FYI, the last commit contains the following change linked to this PR discussion:

Rename EpcDcoument::getResqml2_0Objects into EpcDcoument::getDataObjects in order to get WITSML objects as well.


Add a clipboard history feature to the editor

Qt Creator has a nice feature where pressing Command + Shift + V (on macOS) brings up a clipboard history menu:

So the most recently things you've copied in the editor will show up there, making it really easy to quickly paste in variables etc. you've recently been working with.

Quick note: there are exisiting solutions for clipboard history in KDE (builtin), Gnome (extension), Windows 10 (builtin), and Mac (builtin and apps). Of course, not having to switch context to the desktop shell in order to view the history might still be useful.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


BUILD_ISMRMRD (=OFF)

It would be good to have an option to disable ismrmrd. This should then remove all of ISMRMRD's dependencies, e.g., HDF. (SIRF currently depends on HDF, but it should probably be ISMRMRD that depends on it. That way, when ISMRMRD is removed, HDF is also removed).
I could imagine this getting tricky. We don't want to compile MR-based SIRF code if ISMRMRD is disabled.

@evgueni-ovtchinnikov I gave this a go here, but I haven't tested it at all.


add clojure to euclidean algorithm in fundamental algorithms

not much but it's a start :)
sorry I didn't know how to preview the the book so expect things to break

Looks great! Thanks for the submission!


Updating dev-spaces-preview extension to honor azds commands update

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/saurabsa/azure-cli-extensions.git@saurabsa/UpdateDevSpaces#subdirectory=src/$EXT&egg=$EXT"


India WRIS: Water Quality and Rainfall dataset

Creating a draft PR since the place names for water quality datasets are not resolved yet.
@pradh @ajaits
Please find the data imports for India WRIS datasets - Ground Water quality, Surface Water quality, Rainfall data.

@beets I tried rectifying lint errors multiple times but the checks keep failing. Would you please tell me which files are throwing the error? I don't have access to the build logs.

hm, the error seems to be in an unrelated file.. and the fix shows what's in the repo (below). it seems the version of the file in your commits is fairly old (from oct 14) while what's at HEAD is more recent
try doing git pull upstream master. there are also some conflicts to be resolved.
Step #2 - "python_format_check": --- scripts/google_research/open-covid-19/generate_csv_mcf.py	(original)
Step #2 - "python_format_check": +++ scripts/google_research/open-covid-19/generate_csv_mcf.py	(reformatted)
Step #2 - "python_format_check": @@ -425,7 +425,9 @@
Step #2 - "python_format_check":      """Generates node MCFs for geos that do not come with DCIDs
Step #2 - "python_format_check":      for local resolution."""
Step #2 - "python_format_check":      mcfs = []
Step #2 - "python_format_check": -    template = ('Node: {iso}\n' 'typeOf: schema:Place\n' 'isoCode: "{iso}"\n')
Step #2 - "python_format_check": +    template = ('Node: {iso}\n'
Step #2 - "python_format_check": +                'typeOf: schema:Place\n'
Step #2 - "python_format_check": +                'isoCode: "{iso}"\n')
Step #2 - "python_format_check":      for value in observation_abouts.unique():
Step #2 - "python_format_check":          if value.startswith('l:'):
Step #2 - "python_format_check":              mcfs.append(template.format_map({'iso': value[2:]}))
Step #2 - "python_format_check": ERROR: Fix lint errors by running ./run_test.sh -f

Thank you @beets. I pulled upstream/master and it resolved the errors. All checks are passed now.

@beets I have implemented the changes you suggested. Also added an exception clause in base.py to catch district to lgd code mapping errors.
Also, there are some files from US census in this latest commit. Those were created or modified when I ran scripts/ level python tests.

Thanks for the updates!
Regarding the extra files -- we should fix up the other tests so they don't output files (or we ignore them in git). Before we fix those, could you remove them from this PR? After running the tests, don't run git add . but rather add only files that are relevant for merging. I find it's easier to do this from github desktop

Thanks for the fix in the new PR, @beets. I will use that after the PR's been merged.

I just merged the PR -- please try. You might need to manually revert the extra added files from the PR even after they've been added to gitignore.

@beets So the modified US census files in my previous commits are now deleted in datacommons master branch. So I have deleted those files from this PR, too. There are other files showing up in Files Changed now. But they appeared after I pulled the datacommons master branch today.

@beets I changed the StatVar names in this import to make it as similar to Superfund import. My latest commit fails the lint checks. But, my local tests are running successfully (screenshot attached). Can you please check the logs and share the files that are failing the lint checks?

Hm, it seems to be lint errors. Do you have local changes that haven't been pushed?
Already have image: python:3.7
### Testing lint
Installing Python requirements
#### Testing Python lint
--- scripts/google_research/open-covid-19/generate_csv_mcf.py	(original)
+++ scripts/google_research/open-covid-19/generate_csv_mcf.py	(reformatted)
@@ -425,7 +425,9 @@
     """Generates node MCFs for geos that do not come with DCIDs
     for local resolution."""
     mcfs = []
-    template = ('Node: {iso}\n' 'typeOf: schema:Place\n' 'isoCode: "{iso}"\n')
+    template = ('Node: {iso}\n'
+                'typeOf: schema:Place\n'
+                'isoCode: "{iso}"\n')
     for value in observation_abouts.unique():
         if value.startswith('l:'):
             mcfs.append(template.format_map({'iso': value[2:]}))
ERROR: Fix lint errors by running ./run_test.sh -f

That one file raises lint issues whenever I pull upstream. Not sure why. I just copied the code of that file from master branch and pasted locally and pushed it. The issue is resolved now. Thank you @beets!


Relational Database Basics

[x] 12%

https://learning.edx.org/course/course-v1:IBM+DB0101EN+1T2021/home


Removed dashboard references to latencyTotal_mean

Addresses #1098

NetflixOSS » Hystrix » Hystrix-pull-requests #358 SUCCESS
This pull request looks good


Display protein modifications for genes identified via UniProt ids

Example:

http://localhost:3000/gene/P37173 should display 18 modifications

Complete once KarrLab/datanator_rest_api#106 is addressed.

Done


Cannot use buffered I/O or constants

Cannot open the buffered I/O php://input and php://output constants.
Also, the STDIN, STDOUT and STDERR constants that should set to be already-opened php://stdin, php://stdout, php://stderr streams are just strings.
Code:
<?php

var_dump(\STDIN);
var_dump(\STDOUT);
var_dump(\STDERR);

var_dump(fopen('php://input', 'r'));
var_dump(fopen('php://output', 'w'));

var_dump(fopen('php://stdin', 'r'));
var_dump(fopen('php://stdout', 'w'));
var_dump(fopen('php://stderr', 'w'));

Result:
[daniil@daniil-arch jav]$ jppm start
-> linux
-> app:run
-> install
string(5) "STDIN"
string(6) "STDOUT"
string(6) "STDERR"
Warning: fopen(): failed to open stream, null in 'res://7.php' on line 6 at pos 10
bool(false)
Warning: fopen(): failed to open stream, null in 'res://7.php' on line 7 at pos 10
bool(false)
object(php\io\MiscStream)#1031061344 (4) {
  ["path":"php\io\MiscStream":private]=>
  string(5) "stdin"
  ["mode":"php\io\MiscStream":private]=>
  string(1) "r"
  ["path":"php\io\Stream":private]=>
  NULL
  ["mode":"php\io\Stream":private]=>
  NULL
}
object(php\io\MiscStream)#686466458 (4) {
  ["path":"php\io\MiscStream":private]=>
  string(6) "stdout"
  ["mode":"php\io\MiscStream":private]=>
  string(1) "w"
  ["path":"php\io\Stream":private]=>
  NULL
  ["mode":"php\io\Stream":private]=>
  NULL
}
object(php\io\MiscStream)#875016237 (4) {
  ["path":"php\io\MiscStream":private]=>
  string(6) "stderr"
  ["mode":"php\io\MiscStream":private]=>
  string(1) "w"
  ["path":"php\io\Stream":private]=>
  NULL
  ["mode":"php\io\Stream":private]=>
  NULL
}

The buffered I/O streams can just be linked to stdin and stdout, like they normally are on PHP CLI.

Regarding 'php: // input', I don’t really understand how to integrate this into the engine, because jphp does not implement web functionality as in the original php.

Yeah, it's a bit tricky.
You could link it to php://stdin, or to an empty stream like zend PHP does.


bug with build.sh

### Preparing USB device
GPT fdisk (gdisk) version 1.0.9

Partition table scan:
  MBR: protective
  BSD: not present
  APM: not present
  GPT: present

Found valid GPT with protective MBR; using GPT.

Command (? for help): This option deletes all partitions and creates a new protective MBR.
Proceed? (Y/N):
Command (? for help): Partition number (1-128, default 1): First sector (34-3907029134, default = 2048) or {+-}size{KMGTP}: Last sector (2048-3907029134, default =<PHONE_NUMBER>) or {+-}size{KMGTP}: Current type is 8300 (Linux filesystem)
Hex code or GUID (L to show codes, Enter = 8300): Changed type of partition to 'EFI system partition'

Command (? for help): Partition number (2-128, default 2): First sector (34-3907029134, default = 1230848) or {+-}size{KMGTP}: Last sector (1230848-3907029134, default =<PHONE_NUMBER>) or {+-}size{KMGTP}: Current type is 8300 (Linux filesystem)
Hex code or GUID (L to show codes, Enter = 8300): Changed type of partition to 'Linux filesystem'

Command (? for help): Partition number (3-128, default 3): First sector (34-3907029134, default = 3328000) or {+-}size{KMGTP}: Last sector (3328000-3907029134, default =<PHONE_NUMBER>) or {+-}size{KMGTP}: Current type is 8300 (Linux filesystem)
Hex code or GUID (L to show codes, Enter = 8300): Changed type of partition to 'Linux filesystem'

Command (? for help):
Final checks complete. About to write GPT data. THIS WILL OVERWRITE EXISTING
PARTITIONS!!

Do you want to proceed? (Y/N): OK; writing new GUID partition table (GPT) to /dev/sda.
The operation has completed successfully.
mkfs.fat 4.2 (2021-01-31)
mke2fs 1.46.5 (30-Dec-2021)
/dev/sda2 contains a ext4 file system labelled 'fedora-usb-boot'
        created on Mon Dec 18 22:07:34 2023
Discarding device blocks: done
Creating filesystem with 262144 4k blocks and 65536 inodes
Filesystem UUID: a1492762-3fe2-4908-a8b9-118439becd26
Superblock backups stored on blocks:
        32768, 98304, 163840, 229376
Allocating group tables: done
Writing inode tables: done
Creating journal (8192 blocks): done
Writing superblocks and filesystem accounting information: done

mke2fs 1.46.5 (30-Dec-2021)
/dev/sda3 contains a ext4 file system labelled 'fedora-usb-root'
        created on Mon Dec 18 22:07:35 2023
Discarding device blocks: done
Creating filesystem with 487962624 4k blocks and 121995264 inodes
Filesystem UUID: d747cb2a-aff1-4e47-8a33-c4d9b7475df9
Superblock backups stored on blocks:
        32768, 98304, 163840, 229376, 294912, 819200, 884736, 1605632, 2654208,
        4096000, 7962624, 11239424, 20480000, 23887872, 71663616, 78675968,
        102400000, 214990848

Allocating group tables: done
Writing inode tables: done
Creating journal (262144 blocks): 262143
done
Writing superblocks and filesystem accounting information: done

mkosi: error: unrecognized arguments: --root-password=fedora

I just pushed out an update that will check the version of mkosi before attempting an install
can you do a git pull and trying running the script again?
It'll now show you the path and version of mkosi you're using. Only mkosi version 19 is supported at this time.
Every version of mkosi is so different -- that you need to build a project around a specific version.
If you're on F38, you'll need to install mkosi version 19 via pip.
If you're on F39, just install mkosi via dnf.
Let me know if you need additional help.


New standard napari screenshot.  contest?

this https://github.com/napari/napari/pull/942#issuecomment-582158682 from @jni, (suggesting the astronaut be retired as the representative napari screenshot) got me flippantly suggesting a contest... but now I think I might be serious :)
should we put out a twitter post or something to request submissions for a screenshot highlighting napari?  preferably 4D image with some analysis layers?  I know we've seen some lovely examples out there already.
... or should we just make our own?

It would be super awesome to have people submit their screenshots!!! Maybe have them include what they love about napari too so we can put them in presentations lol ;)

I'm +/- 1 here. #BoatyMcBoatFace. =P

I'm +/- 1 here. #BoatyMcBoatFace. =P

I think @tlambert03 is suggesting: "you all submit screenshots and we will choose a winner", not "you all submit screenshots and you all get to choose the winner".
@kne42 makes a great point about being able to use them for presentations. For that reason you might also suggest people submit gifs or short videos.

ha!  I had to google "boatymcboatface" to understand that.  that's hilarious. and yes, I meant we pick.  and I also like requesting both stills and gifs.

Did you see the original? My wife and I got the giggles for several days when we first saw it 😂😂😂😂😂😂 https://i.kym-cdn.com/photos/images/newsfeed/001/110/982/f62.jpg
I'd like to minimise gifs as much as possible in our site and use autoplay mp4/ogg instead. gifs are bandwidth hogs that furthermore look terrible compared to how buttery-smooth napari is in normal use.

Anyway, assuming we go ahead with this, which I'm coming around to, I'd like to lay a minimum of three rules:

full code needs to be available and BSD/MIT licensed
data needs to be available, and CC-BY or CC0+BY licensed.
bare viewer only, no plugin-y things like PyQtGraph integration (as cool as that is, this is for the face of napari, so it should be what users see by default)

I'm supportive. I think it would be nice to think about how to show case all submissions (that fall within any area of science) in a public gallery rather than it be purely about picking a winner. I'd also want to communicate that up front, so people know if they go through the effort of making a submission it will see the light of day somewhere. I think it's also important that we are looking for scientific images.
It would be nice if they provided some contact info with an indicator if they would be willing to participate in some user research - CZI has a new user researcher that will be working broadly in the imaging space, and this could serve as a great way for her to get in contact with some folks that are really excited about napari - but we want to think carefully about how this is done.
I also don't think there is a rush on this, so would be happy to let everyone have more time to think about the best way to engage the community on this one as it will be really exciting to see.
Would be really cool actually if the script also took the screenshot using the viewer.screenshot(with_viewer=True, path='name_of_their_choosing.png') method. Note that the path arg still needs to be added, as discussed in #944

@sofroniewn I would prefer not to mix the contest up with the user research. I think we can do a separate bit of outreach for that.

@sofroniewn I would prefer not to mix the contest up with the user research. I think we can do a separate bit of outreach for that.

Yup, that's fine - probably wise to keep it separate, was just an idea, and maybe good to put on the napari teams radar that we'll likely be able to get some help with that from CZI in the future :-)

@jni - maybe you could ask David if his users would be happy to enter with some of the neutrophil data we saw earlier? A small time-series of one neutrophil could be fun

@tlambert03 We still don't have our standard screenshot... =)


Documentation that describe file target archiveNumbering="DateAndSequence" behavior

It will be great to have documentation in https://github.com/nlog/nlog/wiki/File-target that describe file target archiveNumbering="DateAndSequence" behavior. It was added at https://github.com/NLog/NLog/pull/449 PR and still not documented. I found it accidentally with nlog API code.

Agree.  Can you do that? We're busy with preparing the upcoming release.

This is done

Also some examples are added:
https://github.com/NLog/NLog/wiki/File-target#archive-numbering-examples


When will Connect-IPPSSession get -Certificate* parameter support

I'm hoping to find out if/when Microsoft will add support for app-only, certificate/client credential authentication for Connect-IPPSSession, like has already been added for Connect-ExchangeOnline.  Any ideas on ETA that can be shared here?
Thanks in advance.

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

ID: ebcb7e5d-c0aa-f4f3-e6d5-914219d16dff
Version Independent ID: 36d9a36f-fbdf-e4a1-45d7-51cc1a65643b
Content: Connect to Security & Compliance Center PowerShell using the EXO V2 module
Content Source: exchange/docs-conceptual/connect-to-scc-powershell.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@JeremyTBradshaw, nothing to share. Your best bet is to open a support case about this.

Thanks, it was a shot in the dark.

Putting it here.
MC316448 (Published Jan 21, 2022)

Certificate based Authentication for Security and Compliance Center PowerShell
With this update, which will improve the security of your tenant, Security & Compliance PowerShell will enable configuration of certificate-based authentication to allow unattended App-only access.
When this will happen:
Rollout will begin in mid-February and is expected to be complete by early March.

And re-closing.

@chrisda This worked!  Thanks very much for the tip.  Note, and this maybe obvious to many, with the Office 365 Exchange Online API's application permission Exchange.ManageAsApp + Exchange Administrator role (assigned to the Enterprise App / ServicePrincipal) I only got access to these commands:
CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        Delete-QuarantineMessage                           1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceDetailsReport                  1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceDetailsReportFilter            1.0        tmp_2jjokwj4.gen
Function        Get-DeviceCompliancePolicyInventory                1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceReportDate                     1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceSummaryReport                  1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceUserInventory                  1.0        tmp_2jjokwj4.gen
Function        Get-DeviceComplianceUserReport                     1.0        tmp_2jjokwj4.gen
Function        Get-Group                                          1.0        tmp_2jjokwj4.gen
Function        Get-QuarantineMessage                              1.0        tmp_2jjokwj4.gen
Function        Get-QuarantineMessageHeader                        1.0        tmp_2jjokwj4.gen
Function        Get-Recipient                                      1.0        tmp_2jjokwj4.gen
Function        Get-User                                           1.0        tmp_2jjokwj4.gen
Function        Preview-QuarantineMessage                          1.0        tmp_2jjokwj4.gen
Function        Release-QuarantineMessage                          1.0        tmp_2jjokwj4.gen

But with the Compliance Administrator AAD role assigned, I then got access to most of the other commands.  I'm not sure if AAD roles alone cover off the requirements for all of the possible SCC cmdlets, but checked/confirmed that in https://compliance.micrososoft.com, I can't assign roles in there to Service Principals.  I'm thinking there could be some gaps here with the export-enabled role entries, but that's just a hunch and no biggie as this functionality covers more than I've been needing so far.
Thanks again.  This is great.

@chrisda are you familiar with Message Center message MC316448?  It is completely lacking any information about HOW Microsoft plans to rollout app-only certificate authentication for the Security and Compliance Center.  It simply links to these two pages which show nothing of what you shared with me:
Learn more:
Connect to Security & Compliance PowerShell
App-only authentication for unattended scripts in the EXO V2 module
Additional information
I would like to know - is Connect-IPPSSession being updated, or will the new functionality just be advertised with Connect-ExchangeOnline's documentation.  Any idea?
PS.  That Message Center announcement is driving me nuts! 🍻

@JeremyTBradshaw, due to the upcoming demise of Basic auth and the MC post, it only makes sense that Connect-IPPSSession will be updated to support CBA.
You'll note that this issue (and others) are tagged with the "CBA SCC" label, so it's easy for me to come back and add any new information.

Thank you.  I will try to be patient and wait for the updates to come.  The MC post says to Act by Feb 20th, but no info for what that means.

Version 2.0.6-Preview5 of the EXO V2 Module is now available, so the various Credential parameters are now available on Connect-IPPSSession, so certibicate-based/app-oly authentication is now fully available in Security & Compliance Center PowerShell.


Split rules into their own files?

https://github.com/DavidAnson/markdownlint/blob/master/lib/rules.js is huge. It would be easier to develop if the rules were in their own files.

See https://github.com/DavidAnson/markdownlint/issues/42 for some context. This has not been an issue yet, but will be naturally fixed when that’s supported.

Would you be upset if I sent a PR?

I learn/remember a lot by revisiting the complete set of rules regularly in the process of maintaining things and find it convenient to search across all rules at once. Similarly, it’s easy to share helper methods within the same file.  Neither is a deal breaker, but I worry that a change like this would make my life harder without significant benefit to offset that.

I worry that a change like this would make my life harder without significant benefit to offset that.

Fun story: one of my early projects was something like 500 global constructs (functions, variables) across maybe a dozen files. It felt nice having things all together... but in actuality it became a huge unorganized mess of helper utilities thrown all over the place. Eventually it became impossible to work on because you'd need three tabs open of the same file, lint/compile rebuilds were horrendously slow, and I'd repeatedly get lost. markdownlint certainly isn't there (phew!) but I would be scared to slip down that path.
I'll send a PR for this and if you hate it, no worries. 😄


Changes for ROS Kinetic, fix build warnings and verbosity.

Minor changes to make build and runtime less verbose. I added a call to avutil log_level to set the log level to error instead of warnings to avoid warnings about deprecated pixel formats that aren't meant for end users.

Looks like this breaks the build for earlier versions.
https://travis-ci.org/bosch-ros-pkg/usb_cam/builds/130302572


TypeError: zip argument #1 must support iteration

my torch version is 1.4
Hello, when I execute this line of code：
python -m pcdet.datasets.kitti.kitti_dataset create_kitti_infos tools/cfgs/dataset_configs/kitti_dataset.yaml
I encountered this report error：
TypeError: zip argument #1 must support iteration.
the details is follows:
/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py:125: RuntimeWarning: 'pcdet.datasets.kitti.kitti_dataset' found in sys.modules after import of package 'pcdet.datasets.kitti', but prior to execution of 'pcdet.datasets.kitti.kitti_dataset'; this may result in unpredictable behaviour
warn(RuntimeWarning(msg))
/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/kitti/kitti_dataset.py:430: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
dataset_cfg = EasyDict(yaml.load(open(sys.argv[2])))
---------------Start to generate data infos---------------
Traceback (most recent call last):
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/kitti/kitti_dataset.py", line 436, in
save_path=ROOT_DIR / 'data' / 'kitti'
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/kitti/kitti_dataset.py", line 396, in create_kitti_infos
kitti_infos_train = dataset.get_infos(num_workers=workers, has_label=True, count_inside_pts=True)
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/kitti/kitti_dataset.py", line 188, in get_infos
infos = executor.map(process_single_scene, sample_id_list)
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/concurrent/futures/_base.py", line 548, in map
fs = [self.submit(fn, *args) for args in zip(*iterables)]
TypeError: zip argument #1 must support iteration

my torch version is 1.4

You could try to use PyTorch 1.1 since I only tested it with PyTorch 1.1.

You could try to use PyTorch 1.1 since I only tested it with PyTorch 1.1.

thank you for your reply.And I tried to use pytorch 1.1 and torchvision 0.3 on cuda 9.0 and cudnn 7.6.5 and python 3.6,but there was a new error:
(pytorch_gpu) zzqjh@zzqjh-X8Ti-G-Series-GK7CP6R:~/3D-OT/OpenPCDet/PCDet$ python -m pcdet.datasets.kitti.kitti_dataset create_kitti_infos tools/cfgs/dataset_configs/kitti_dataset.yaml
Traceback (most recent call last):
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/init.py", line 3, in
from .dataset import DatasetTemplate
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/dataset.py", line 5, in
from .augmentor.data_augmentor import DataAugmentor
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/data_augmentor.py", line 3, in
from . import augmentor_utils, database_sampler
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/database_sampler.py", line 3, in
from ...utils import box_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/utils/box_utils.py", line 4, in
from ..ops.roiaware_pool3d import roiaware_pool3d_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_utils.py", line 5, in
from . import roiaware_pool3d_cuda
ImportError: /home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_cuda.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceIN3c108BFloat16EEEPKNS_6detail12TypeMetaDataEv

I suggest you to re-install the pytorch 1.1 and re-build the spconv (remember to delete the spconv/build directory before you re-build spconv).

You could try to use PyTorch 1.1 since I only tested it with PyTorch 1.1.

thank you for your reply.And I tried to use pytorch 1.1 and torchvision 0.3 on cuda 9.0 and cudnn 7.6.5 and python 3.6,but there was a new error:
(pytorch_gpu) zzqjh@zzqjh-X8Ti-G-Series-GK7CP6R:~/3D-OT/OpenPCDet/PCDet$ python -m pcdet.datasets.kitti.kitti_dataset create_kitti_infos tools/cfgs/dataset_configs/kitti_dataset.yaml
Traceback (most recent call last):
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/init.py", line 3, in
from .dataset import DatasetTemplate
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/dataset.py", line 5, in
from .augmentor.data_augmentor import DataAugmentor
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/data_augmentor.py", line 3, in
from . import augmentor_utils, database_sampler
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/database_sampler.py", line 3, in
from ...utils import box_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/utils/box_utils.py", line 4, in
from ..ops.roiaware_pool3d import roiaware_pool3d_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_utils.py", line 5, in
from . import roiaware_pool3d_cuda
ImportError: /home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_cuda.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceIN3c108BFloat16EEEPKNS_6detail12TypeMetaDataEv

Have you solved the problem? I have got the same error.
CUDA10.1, PyTorch1.5

You could try to use PyTorch 1.1 since I only tested it with PyTorch 1.1.

thank you for your reply.And I tried to use pytorch 1.1 and torchvision 0.3 on cuda 9.0 and cudnn 7.6.5 and python 3.6,but there was a new error:
(pytorch_gpu) zzqjh@zzqjh-X8Ti-G-Series-GK7CP6R:~/3D-OT/OpenPCDet/PCDet$ python -m pcdet.datasets.kitti.kitti_dataset create_kitti_infos tools/cfgs/dataset_configs/kitti_dataset.yaml
Traceback (most recent call last):
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/zzqjh/.conda/envs/pytorch_gpu/lib/python3.6/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/init.py", line 3, in
from .dataset import DatasetTemplate
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/dataset.py", line 5, in
from .augmentor.data_augmentor import DataAugmentor
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/data_augmentor.py", line 3, in
from . import augmentor_utils, database_sampler
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/datasets/augmentor/database_sampler.py", line 3, in
from ...utils import box_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/utils/box_utils.py", line 4, in
from ..ops.roiaware_pool3d import roiaware_pool3d_utils
File "/home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_utils.py", line 5, in
from . import roiaware_pool3d_cuda
ImportError: /home/zzqjh/3D-OT/OpenPCDet/PCDet/pcdet/ops/roiaware_pool3d/roiaware_pool3d_cuda.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceIN3c108BFloat16EEEPKNS_6detail12TypeMetaDataEv

Have you solved the problem? I have got the same error.
CUDA10.1, PyTorch1.5

I turned out it was a path error. The path was None. Check the path carefully can avoid this error. Note that there should be a ImageSets folder within the KITTI dataset. Please organise the files as the author suggests.
OpenPCDet
├── data
│   ├── kitti
│   │   │── ImageSets
│   │   │── training
│   │   │   ├──calib & velodyne & label_2 & image_2 & (optional: planes)
│   │   │── testing
│   │   │   ├──calib & velodyne & image_2
├── pcdet
├── tools

This problem is solved by copy ImageSets/ to the Real kitti folder

Be careful of Path. The 'training/' and 'testing/' folders are not in 'ImageSets/' but directly in 'kitti/'.

omg... it works. thanks a lot


Add new vendor - Slido

What is the vendor's name?
Slido
What is the vendor's pricing site?
https://www.slido.com/pricing?plan=annual
What is the base pricing? Use the lowest tier that looks sane for a small business customer, not free or personal tiers.
$15 per user/month
What is the minimum pricing for SSO support?
$50 per user/month
Does this pricing info come from a quote or other non-public source?
Used public information only
Are there any caveats we should list in the footnotes?

Hey @voidlily ,
as this repository was not maintained for a long time, I decided to start a new initiative under https://ssotax.org/. The data is already way more up2date than in this repository! Moreover I integrated feedback from the community like a Friends of SSO page.
Feel free to contribute there. I will merge changes quickly, I promise!


CLI Interactive mode Cellery modify UX improvement

Description:
Currently when user modifies runtimes s/he is supposed to come back and select "DONE" for the changes to be updated. This may confuse the user that the changes are already applied when they select enable/disable.
Suggested flow

can remove the "BACK" option in the last step.


json with assets

When i use json with asset i get:
2017-06-19 18:58:23.108 Oklive[27198:7781275] -[LOTLayerView _setImageForAsset]: Warn: image not found: img_0
the animation working but without the image parts
(v1.1.1)

+1 ,same problem

Same problem, how to add the folder(of images) to the animations in react native....

i find a way
when you look on lottie ios code in LOTLayerView.m:
in the lottie ios repo the function  look like this and
- (void)_setImageForAsset {
    if (_layerModel.imageAsset.imageName) {
        UIImage *image;
        if (_layerModel.imageAsset.rootDirectory.length > 0) {
            NSString *rootDirectory  = _layerModel.imageAsset.rootDirectory;
            if (_layerModel.imageAsset.imageDirectory.length > 0) {
                rootDirectory = [rootDirectory stringByAppendingPathComponent:_layerModel.imageAsset.imageDirectory];
            }
            NSString *imagePath = [rootDirectory stringByAppendingPathComponent:_layerModel.imageAsset.imageName];
            image = [UIImage imageWithContentsOfFile:imagePath];
        }else{
            NSArray *components = [_layerModel.imageAsset.imageName<EMAIL_ADDRESS>            image = [UIImage imageNamed:components.firstObject inBundle:_bundle compatibleWithTraitCollection:nil];
        }

        if (image) {
            _childSolid.contents = (__bridge id _Nullable)(image.CGImage);
        } else {
            NSLog(@"%s: Warn: image not found: %@", __PRETTY_FUNCTION__, _layerModel.imageAsset.imageName);
        }
    }
}

and in the version i install with react native its look like this
(void)_setImageForAsset {
  if (_layerModel.imageAsset.imageName) {
    NSArray *components = [_layerModel.imageAsset.imageName<EMAIL_ADDRESS>    UIImage *image = [UIImage imageNamed:components.firstObject];
    if (image) {
      _childSolid.contents = (__bridge id _Nullable)(image.CGImage);
    } else {
      NSLog(@"%s: Warn: image not found: %@", __PRETTY_FUNCTION__, components.firstObject);
    }
  }
}

you can see that in the ios repo they handle folder and in react version just image name
so the for me to fix it with out change is to get all the image from bodymovin image folder
(ex: img_0,img_1 etc)
and add them to bundle resources
if this dont work try to had with the folder (you can search _setImageForAsset in xcode and see the function version )
hope this help =]

+1 - the same problem

Same issue over here, on iOS the image doesn't get loaded.
Any update anyone?
In another issue, #80 (of which imho this is a duplicate) they were considering the imageAssetFolder prop as possible solution but it's Android only :(

Quick update: in iOS to have the animation pick up the image you need to - basically - open XCode, right click on the Resources folder, "Add file to [Project]" and add the image (and then rebuild the project) - so my suggestion is that you give that image a "recognizable name" when you import it;)

@kelset is it also required to update the lottie json to remove the 'image/' path ref's?


WIP: Add wait command to diagnostics

This add a wait command that will wait for a diagnostics bundle to finish.
I also added a spinner while the process waits but this is a little un-unixy so maybe we'll want to remove that before merging the PR.
Addresses: https://jira.mesosphere.com/browse/DCOS_OSS-5488

I'd prefer to keep it since I think it adds a bit of responsiveness that you wouldn't otherwise get, the --quiet flag exists if you don't want to see it and I could modify that to print the bundle ID (instead of nothing, currently).
It's already a bit of an odd command anyway in terms of unix conventions. I think the more "proper" way to do this under unix conventions would be to add a diagnostics status <bundle> command then set up a few commands to periodically query the bundle and pipe it in jq to check the status until it's finished.

Squashed commits and fixed merge conflicts


You can no longer right click things inside of a container to select them

Description

Put item in locker
Put yourself in the locker
Close the locker
Right click the locker
Be unable to select anything

Reproduction

Screenshots

Additional context

https://github.com/space-wizards/space-station-14/pull/31792 possibly related?

At a glance, yeah that's probably it. I think it also prevents being able to use the context menu debug commands to examine contained entities (i.e., right click player to see their organs).
It should probable be reverted. And we need to figure out whatever engine change caused this? https://github.com/space-wizards/space-station-14/issues/31756#issuecomment-2324990035

If it does get reverted then remove AI job in the interim as it will break.

Why does the AI job the context menu to exclude sensors?

should just list contents of a container if you are inside it, regardless of where you right clicked


Travis: Build against more python versions

Landing this PR now~

This has been merged/released in 0.12.4. Thanks for the PR!

thanks for merging, would you have anything against us adding windows
builds for you? We're already testing your tool against windows in coala
and it works, it would probably be better if you catch your windows
problems before we hit them.
2015-11-16 9:05 GMT+01:00 Todd Wolfson<EMAIL_ADDRESS>
This has been merged/released in 0.12.4. Thanks for the PR!
—
Reply to this email directly or view it on GitHub
https://github.com/twolfson/restructuredtext-lint/pull/24#issuecomment-156951281
.

I'm open to it but I typically avoid it unless the package is doing something OS specific (e.g. running a program, reading a hardcoded filepath from disk).
The reasoning is that the more versions/platforms that we test, the more likely something is to break in those external systems (e.g. what happens if Appveyor has a regression in their CI platform) which leads to more maintenance.

sure then I'll not do it, as long as nothing bad happens with file paths we
all should be fine.
2015-11-16 9:31 GMT+01:00 Todd Wolfson<EMAIL_ADDRESS>
I'm open to it but I typically avoid it unless the package is doing
something OS specific (e.g. running a program, reading a hardcoded filepath
from disk).
The reasoning is that the more versions/platforms that we test, the more
likely something is to break in those external systems (e.g. what happens
if Appveyor has a regression in their CI platform) which leads to more
maintenance.
—
Reply to this email directly or view it on GitHub
https://github.com/twolfson/restructuredtext-lint/pull/24#issuecomment-156954853
.


Error on kmmlu evaluation set with vLLM: KeyError: 'exact_match,none'

Hello, Thank you for this amazing repo.
I trying out this new benchmark K-MMLU, and I got this error at the end of the evaluation.
My command is:
$ lm_eval --model vllm \
--model_args pretrained=beomi/Yi-Ko-6B,dtype=auto,gpu_memory_utilization=0.9 \
 --tasks kmmlu \
--batch_size auto

and I got this error message at the end of the evaluation.
100%|___________________________________________________________________________________________________________| 43238/43238 [10:37<00:00, 67.82it/s]
2024-02-23:15:47:12,103 INFO     [evaluator.py:369] Running loglikelihood requests
Processed prompts: 100%|_______________________________________________________________________________________| 16416/16416 [01:05<00:00, 249.75it/s]
100%|__________________________________________________________________________________________________________| 16416/16416 [01:06<00:00, 246.17it/s]
Traceback (most recent call last):
  File "/opt/conda/bin/lm_eval", line 8, in <module>
    sys.exit(cli_evaluate())                                                                                File "/root/lm-evaluation-harness/lm_eval/__main__.py", line 288, in cli_evaluate
    results = evaluator.simple_evaluate(
  File "/root/lm-evaluation-harness/lm_eval/utils.py", line 288, in _wrapper                                                                              return fn(*args, **kwargs)
  File "/root/lm-evaluation-harness/lm_eval/evaluator.py", line 190, in simple_evaluate
    results = evaluate(
  File "/root/lm-evaluation-harness/lm_eval/utils.py", line 288, in _wrapper
    return fn(*args, **kwargs)
  File "/root/lm-evaluation-harness/lm_eval/evaluator.py", line 535, in evaluate
    metrics = [                                                                                File "/root/lm-evaluation-harness/lm_eval/evaluator.py", line 536, in <listcomp>                                                                        results[task][metric] for task in task_list
KeyError: 'exact_match,none'

Please help me with this

In addition, when evaluating the subset of it, It works well.
lm_eval --model vllm     --model_args pretrained=beomi/Yi-Ko-6B,tensor_parallel_size=1,dtype=auto,gpu_memor
y_utilization=0.9,data_parallel_size=1     --tasks kmmlu_direct_machine_design_and_manufacturing     --batch_size auto                                2024-02-23:16:37:52,784 INFO     [__main__.py:209] Verbosity set to INFO
2024-02-23:16:37:52,785 INFO     [__init__.py:358] lm_eval.tasks.initialize_tasks() is deprecated and no longer necessary. It will be removed in v0.4.2 release. TaskManager will instead be used.
2024-02-23:16:37:56,687 INFO     [__main__.py:285] Selected Tasks: ['kmmlu_direct_machine_design_and_manufacturing']                                  2024-02-23:16:37:56,687 INFO     [__main__.py:286] Loading selected tasks...
2024-02-23:16:37:56,687 INFO     [evaluator.py:95] Setting random seed to 0                                                                           2024-02-23:16:37:56,687 INFO     [evaluator.py:99] Setting numpy seed to 1234
2024-02-23:16:37:56,687 INFO     [evaluator.py:103] Setting torch manual seed to 1234                                                                 INFO 02-23 16:37:57 llm_engine.py:73] Initializing an LLM engine with config: model='beomi/Yi-Ko-6B', tokenizer='beomi/Yi-Ko-6B', tokenizer_mode=auto,
 revision=None, tokenizer_revision=None, trust_remote_code=False, dtype=torch.bfloat16, max_seq_len=2048, download_dir=None, load_format=auto, tensor_parallel_size=1, quantization=None, seed=1234)
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.                                 INFO 02-23 16:38:03 llm_engine.py:222] # GPU blocks: 58851, # CPU blocks: 4096
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.                                 2024-02-23:16:38:05,756 INFO     [evaluator.py:150] get_task_dict has been updated to accept an optional argument, `task_manager`Read more here:https:
//github.com/EleutherAI/lm-evaluation-harness/blob/main/docs/interface.md#external-library-usage                                                      2024-02-23:16:38:12,878 INFO     [task.py:361] Building contexts for kmmlu_direct_machine_design_and_manufacturing on rank 0...
2024-02-23:16:38:13,745 INFO     [evaluator.py:369] Running generate_until requests                                                                   Processed prompts: 100%|__________________________________________________________________________________________| 1000/1000 [00:12<00:00, 77.88it/s]
100%|_____________________________________________________________________________________________________________| 1000/1000 [00:12<00:00, 77.76it/s]vllm (pretrained=beomi/Yi-Ko-6B,tensor_parallel_size=1,dtype=auto,gpu_memory_utilization=0.9,data_parallel_size=1), gen_kwargs: (None), limit: None, n
um_fewshot: None, batch_size: auto                                                                                |                    Tasks                    |Version|Filter|n-shot|  Metric   |Value|   |Stderr|
|---------------------------------------------|------:|------|------|-----------|----:|---|-----:|                                                    |kmmlu_direct_machine_design_and_manufacturing|      2|none  |None  |exact_match|    0|_  |     0|

Thanks for reporting this! @h-albert-lee did you observe this occurring? if not, or if you don't have the time to check this out, I can investigate myself.

@haileyschoelkopf I don't think I've seen that issue in my implementation, but I've reproduced it on my personal laptop and I get that error. It's probably a difference between the forked version and the current version, or it's an operating system issue. The implementation was worked on and tested on ubuntu, but the reproduction is on Windows, so there might be a difference.

May be closed by #1441 ?

I think so

Confirmed no longer occurring on most recent code version!

@h-albert-lee Thank you for sharing this benchmark.
If I want to reproduce results table2 in the paper is it correct to use kmmlu_direct for the task, not kmmlu?
It seems if I set task to kmmlu , it evaluates on all 4 groups(direct, hard, hard_direct, hard_cot). I think the error @caisarl76 shared occurs because 4 groups have different evaluation metrics(e.g. exact_match,none, acc_norm,none, ...)

@yoonniverse  If you want to reproduce what is presented in the paper, it is correct to use only kmmlu_direct. Also, as stated in the readme, you can reproduce the exact values by taking the macro average.


[BUG] The set-output command is deprecated.

This is not really a bug, but neither it is a new feature. But it might become a bug.
Using your tool I get this warning/info:
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Maybe you can update your tool before the set-output command will be disabled.

Hi @Doxoh,
Sure, I'll check it out this week.
thank you

Looking forward as well, Github will disable it by the end of the month

We are monitoring telemetry for the usage of these commands and plan to fully disable them on 31st May 2023

Hi, can someone test version v3.1.0-preview3 please? if you confirm that it's working I'll release v3.1.0
I don't have much time but looks ok to me (tested once and it worked). just want to make sure before releasing it.
thank you

Sadly there are no examples for Javascript in the references. Maybe this is an approach? https://github.com/actions/toolkit/issues/1218

@alirezanet did this help you? GerardSmit@47f6a3c

This looks a good implementation, 👌
Can you create a PR?

@alirezanet done #10

Thank you, I think we can test version v3.1.0-test1 now. let me know if you did it. if not I'll try to test it till tomorrow

Fixed in v3.1.0.
thanks to @Doxoh


histogram function breaks when passing a string column and using category

when calling the histogram function with a column that's a string and then passing the category argument, the function breaks, this is because histogram internally calls _histogram, which returns None  as bin_size, breaking the execution.
here's how to reproduce the error:
from sql.plot import histogram, _histogram

%load_ext sql

%sql duckdb://

%%sql
-- Create the 'fruits' table
CREATE TABLE fruits (
    fruit_id INT,
    fruit_name VARCHAR(50),
    fruit_color VARCHAR(50)
);

-- Insert rows with fruit names and colors
INSERT INTO fruits (fruit_id, fruit_name, fruit_color) VALUES
    (1, 'Apple', 'Red'),
    (2, 'Banana', 'Yellow'),
    (3, 'Orange', 'Orange'),
    (4, 'Grapes', 'Purple'),
    (5, 'Strawberry', 'Red'),
    (6, 'Kiwi', 'Green'),
    (7, 'Mango', 'Yellow'),
    (8, 'Blueberry', 'Blue'),
    (1, 'Apple', 'Red'),
    (2, 'Banana', 'Yellow'),
    (3, 'Orange', 'Orange'),
    (4, 'Grapes', 'Purple');

Running query in 'duckdb://'

Count

ResultSet : to convert to pandas, call .DataFrame() or to polars, call .PolarsDataFrame()
_, _, bin_size = _histogram("fruits", "fruit_name", bins=10)
bin_size is None

True

histogram("fruits", "fruit_name", bins=10, category="fruit_color")

---------------------------------------------------------------------------

TypeError                                 Traceback (most recent call last)

Cell In[6], line 1
----> 1 histogram("fruits", "fruit_name", bins=10, category="fruit_color")

File ~/miniconda3/envs/r/lib/python3.10/site-packages/ploomber_core/dependencies.py:40, in requires.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
     32 @wraps(f)
     33 def wrapper(*args, **kwargs):
     34     check_installed(
     35         pkgs=pkgs,
     36         name=name or f.__name__,
     37         extra_msg=extra_msg,
     38         pip_names=pip_names,
     39     )
---> 40     return f(*args, **kwargs)

File ~/miniconda3/envs/r/lib/python3.10/site-packages/ploomber_core/telemetry/telemetry.py:752, in Telemetry.log_call.<locals>._log_call.<locals>.wrapper(*args, **kwargs)
    750         result = func(*injected_args, **kwargs)
    751     else:
--> 752         result = func(_payload, *args, **kwargs)
    753 else:
    754     result = func(*args, **kwargs)

File ~/dev/jupysql/src/sql/plot.py:418, in histogram(payload, table, column, bins, with_, conn, category, cmap, color, edgecolor, ax, facet)
    416 bin_, height, bin_size = _histogram(table, column, bins, with_=with_, conn=conn)
    417 width = _get_bar_width(ax, bin_, bin_size)
--> 418 data = _histogram_stacked(
    419     table, column, category, bin_, bin_size, with_=with_, conn=conn, facet=facet
    420 )
    421 cmap = plt.get_cmap(cmap or "viridis")
    422 norm = Normalize(vmin=0, vmax=len(data))

File ~/miniconda3/envs/r/lib/python3.10/site-packages/ploomber_core/exceptions.py:109, in modify_exceptions.<locals>.wrapper(*args, **kwargs)
    106 @wraps(fn)
    107 def wrapper(*args, **kwargs):
    108     try:
--> 109         return fn(*args, **kwargs)
    110     except (ValueError, TypeError) as e:
    111         _add_community_link(e)

File ~/dev/jupysql/src/sql/plot.py:601, in _histogram_stacked(table, column, category, bins, bin_size, with_, conn, facet)
    598     conn = sql.connection.ConnectionManager.current
    600 cases = []
--> 601 tolerance = bin_size / 1000  # Use to avoid floating point error
    602 for bin in bins:
    603     # Use round half down instead of floor to mimic
    604     # right-closed histogram intervals in R ggplot
    605     case = (
    606         f"SUM(CASE WHEN ABS(CEILING({column}/{bin_size} - 0.5)*{bin_size} "
    607         f"- {bin}) <= {tolerance} THEN 1 ELSE 0 END) AS '{bin}',"
    608     )

TypeError: unsupported operand type(s) for /: 'NoneType' and 'int'
If you need help solving this issue, send us a message: https://ploomber.io/community

@mehtamohit013 any updates?

AC Criteria:

Fix the return type and function calls
Testcases
Changelog

@edublancas
Problem
In the case of string based column or a categorical column, the histogram function fails, because various SQL statements used in histogram functions are written with numerical columns in mind
This includes the error of bin_size being None. But even after setting it to 1(Hardcoding it) in the case of categorical columns, various functions of the histogram are not working as intended.
Solution

Convert categorical columns to numerical columns by assigning them rank, something like ('Apple', 'Orange', 'Banana') to (0,1,2) (Preferable)
OR
Add a separate case for every functionality of the histogram like stacking, bin_size, etc. to accommodate categorical columns.

Doubts

What does the category argument of histogram do? Does it stacks the histogram like this? I am unable to find any documentation on the argument in docs as well as in the code.
In the case of the categorical histogram, shouldn't the bin size be equal to the number of categories(Because extra bins do nothing here)? So, In the example above, bins should be equal to 8 ('Apple', 'Banana', 'Blueberry', 'Grapes', 'Kiwi', 'Mango', 'Orange', 'Strawberry'), rather than 10. So should we raise a ValueError here, something like For the categorical histogram, number of bins ({bins}) must match the number of categories ({len(bin_)}).

What does the category argument of histogram do? Does it stacks the histogram like this? I am unable to find any documentation on the argument in docs as well as in the code.

I think so (check existing image tests to confirm this, as I didn't author the code)

In the case of the categorical histogram, shouldn't the bin size be equal to the number of categories(Because extra bins do nothing here)? So, In the example above, bins should be equal to 8 ('Apple', 'Banana', 'Blueberry', 'Grapes', 'Kiwi', 'Mango', 'Orange', 'Strawberry'), rather than 10. So should we raise a ValueError here, something like For the categorical histogram, number of bins ({bins}) must match the number of categories ({len(bin_)}).

I don't think we should raise an error, we can just show a warning saying "the column passed is categorical, bins are ignored". I think what happens if you pass a categorical column is that the histogram function returns a bar chart (strictly speaking, this isn't a histogram)
if I recall correctly, this works:
# i think bins is ignored (?)
histogram("fruits", "fruit_name", bins=10)

but this doesn't, and it should:
histogram("fruits", "fruit_name", bins=10, category="fruit_color")

if I recall correctly, this works:

# i think bins is ignored (?)
histogram("fruits", "fruit_name", bins=10)

This is mainly due to the SQL command used works for both strings as well as the integer column type.
While for this:

but this doesn't, and it should:

histogram("fruits", "fruit_name", bins=10, category="fruit_color")

THE SQL command used is:
SELECT fruit_color,
        SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Apple) <= 0.001 THEN 1 ELSE 0 END) AS 'Apple', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Banana) <= 0.001 THEN 1 ELSE 0 END) AS 'Banana', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Blueberry) <= 0.001 THEN 1 ELSE 0 END) AS 'Blueberry', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Grapes) <= 0.001 THEN 1 ELSE 0 END) AS 'Grapes', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Kiwi) <= 0.001 THEN 1 ELSE 0 END) AS 'Kiwi', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Mango) <= 0.001 THEN 1 ELSE 0 END) AS 'Mango', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Orange) <= 0.001 THEN 1 ELSE 0 END) AS 'Orange', SUM(CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Strawberry) <= 0.001 THEN 1 ELSE 0 END) AS 'Strawberry',
        FROM "fruits"
        WHERE "fruit_name" IS NOT NULL
        GROUP BY fruit_color
        ORDER BY fruit_color DESC;

As you can clearly see in the command above, the syntax will only work in the case of numeric columns.
For Example :
CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - Apple) will work if I replace Apple with 0 resulting in CASE WHEN ABS(CEILING(fruit_name/1 - 0.5)*1 - 0). Same with other fruit names.
So, my current approach would be to convert the string column to an integer and assign them rank @edublancas

@edublancas Created a PR with the above logic: #816


ConfigMap generated by Helm contains duplicates

Running helm template with the parameters used for GKE creates a configmap which contains duplicate keys. This is likely overall a bug in the Helm charts and might not be GKE specific. @twpayne discovered this while generating YAML files for GKE.
How to reproduce:
$ helm template cilium cilium/cilium --version 1.9.0 \
  --namespace cilium \
  --set nodeinit.enabled=true \
  --set nodeinit.reconfigureKubelet=true \
  --set nodeinit.removeCbrBridge=true \
  --set cni.binPath=/home/kubernetes/bin \
  --set gke.enabled=true \
  --set ipam.mode=kubernetes \
  --set nativeRoutingCIDR=$NATIVE_CIDR > cilium-gke.yaml
$ pip install --user yamllint
...
$ yamllint -d relaxed --no-warnings cilium-gke.yaml
cilium-gke.yaml
  151:3     error    duplication of key "tunnel" in mapping  (key-duplicates)
  185:3     error    duplication of key "ipam" in mapping  (key-duplicates)

The tunnel key also contains conflicting values, which makes this potentially more dangerous:
$ egrep 'tunnel|ipam' cilium-gke.yaml
  tunnel: vxlan
  ipam: "kubernetes"
  tunnel: "disabled"
  ipam: "kubernetes"

This also happens on Cilium 1.8.5.

I think there was exsiting issues or slack discussion before, saying that gke.enabled and ipam.mode should not be set together.

I identified the lines where the keys are getting duplicated, and so I would like to take a shot at creating the patch ✌🏼
The tunnel key gets duplicated because there's no condition around line 303 and when gke is enabled the control goes through lines 314 to 320, where tunnel key is set again at line 317
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L303-L320
And here variable $ipam is compared against "hostscope",
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L615-L617
which was set here to kubernetes,  so that's getting duplicated as well.
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L34

Although, I understand that we can put the tunnel key at line 303 around a condition which checks if .Values.gke.enabled is true or not, I’m not sure what “hostscope” defines here.
Should we put the ipam key at line 616 within a condition that checks for .Values.gke.enabled as well or are there any other checks involved?

@PranaviRoy Thanks for digging into this. We discussed this in today's Cilium Community meeting, here is what I think we should do:

We want to add a check for .Values.gke.enabled for the tunnel key as you suggested.
The old hostscope ipam mode has been removed in Cilium 1.9.0: https://github.com/cilium/cilium/issues/12974 and I believe what you found is a leftover which did not get removed by accident. Therefore you may safely remove that check for hostscope and replace it with a check for .Values.gke.enabled instead.

For the ipam duplicate, we actually want to remove the ipam key from the gke.enabled section here and require the user to set it explicitly. So all that is needed is to remove the ipam line here:
https://github.com/cilium/cilium/blob/f66515219c21041f17d424cb8f2ca2b1b81c4687/install/kubernetes/cilium/templates/cilium-configmap.yaml#L316

If we remove ipam from this block
https://github.com/cilium/cilium/blob/f66515219c21041f17d424cb8f2ca2b1b81c4687/install/kubernetes/cilium/templates/cilium-configmap.yaml#L314-L320
and we replace this check with gke.enabled i.e. if its true then we won't be setting ipam anywhere if gke.enabled: false. Similarly if the check is for gke.enabled: false then we won't be setting ipam if gke.enabled: true
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L615-L617

In my opinion we can either,

add the check for gke.enabled: false in here, so that ipam is set here if gke is not enabled
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L615-L617
and let ipam be set here if gke is enabled
https://github.com/cilium/cilium/blob/f66515219c21041f17d424cb8f2ca2b1b81c4687/install/kubernetes/cilium/templates/cilium-configmap.yaml#L314-L320
OR
remove ipam from here
https://github.com/cilium/cilium/blob/f66515219c21041f17d424cb8f2ca2b1b81c4687/install/kubernetes/cilium/templates/cilium-configmap.yaml#L314-L320
and remove the check from here altogether and let ipam be set explicitly, always
https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L615-L617

Ah, apologies, I think I mixed up tunnel and ipam above when talking about the hostscope check above.

In my opinion we can either,
  [...]

   **OR**

2. remove `ipam` from here
   https://github.com/cilium/cilium/blob/f66515219c21041f17d424cb8f2ca2b1b81c4687/install/kubernetes/cilium/templates/cilium-configmap.yaml#L314-L320

   and remove the check from here altogether and let `ipam` be set explicitly, always
   https://github.com/cilium/cilium/blob/ddb2423fae063ec050bd0332981c5d78f7a39396/install/kubernetes/cilium/templates/cilium-configmap.yaml#L615-L617

Yes, I think we should go with option 2 here. From what I gathered during the Cilium Community meeting, it is intentional that we ask users to explicitly set the IPAM mode.

Yes, I think we should go with option 2 here. From what I gathered during the Cilium Community meeting, it is intentional that we ask users to explicitly set the IPAM mode.

Okaay!!


Why not create one project include all samples?

I want to review some samples, but I must build many project.
Why you not create one project include all samples?
Example: Have a main view include list of buttons link to samples.

I had done that before, and it becomes a lot for a person to find what they are looking for in the code. Also, some samples will conflict with others based on their design. For example, there are many RootController types, how would I show that all can be the window!.rootViewController in one sample. Also the samples are going to become more robust as I put more of them together to show advanced topics. For example: the CardTableView sample is literally a base application one can start from. I hope this helps clear the reasons up for you.


Use Helm global.externalIstiod for pilot env variable

Please provide a description of this PR:
For multi-cluster primary-remote installation with Helm, externalIstiod needs to be set both for .Values.global and .Values.pilot.env. This PR consolidates the installation setup to just use .Values.global to set the pilot env variable. However, to prevent duplication, if EXTERNAL_ISTIOD is set via the pilot env variable, then respect that instead.
Resolves: https://github.com/istio/istio/issues/51595

@nshankar13: 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

gencheck_istio
1ed0023d679548a571ae321f049760924e038f62
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-sigs/prow repository. I understand the commands that are listed here.

@nshankar13: 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
1ed0023d679548a571ae321f049760924e038f62
link
true
/test gencheck

release-notes_istio
1ed0023d679548a571ae321f049760924e038f62
link
true
/test release-notes

unit-tests-arm64_istio
1ed0023d679548a571ae321f049760924e038f62
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-sigs/prow repository. I understand the commands that are listed here.

@nshankar13: 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
1ed0023d679548a571ae321f049760924e038f62
link
true
/test gencheck

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

release-notes_istio
464c64cfca4ad3730e810b2cbfb08734125aefdc
link
true
/test release-notes

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-sigs/prow repository. I understand the commands that are listed here.

@nshankar13: 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
1ed0023d679548a571ae321f049760924e038f62
link
true
/test gencheck

release-notes_istio
464c64cfca4ad3730e810b2cbfb08734125aefdc
link
true
/test release-notes

unit-tests_istio
464c64cfca4ad3730e810b2cbfb08734125aefdc
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-sigs/prow repository. I understand the commands that are listed here.

@nshankar13: 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
464c64cfca4ad3730e810b2cbfb08734125aefdc
link
true
/test unit-tests

release-notes_istio
bcf9f4926f1a3eea3b08de2c275cf159b212f587
link
true
/test release-notes

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-sigs/prow repository. I understand the commands that are listed here.

@nshankar13: 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

release-notes_istio
5894ef1bfd99f5a06cfc2538283ecbe502a5db60
link
true
/test release-notes

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-sigs/prow repository. I understand the commands that are listed here.


🐛 Tiago's name displays random unicode characters on over 600+ rules

Cc: @christianmorfordwaitessw @adamcogan @pierssinclairssw @bradystroud

Hi All,
Describe the Bug

As shown below in the image, Tiago's last name isn't displaying correctly.

Figure: Tiago's last name displaying the weird characters on Production

Figure: The same rule on Staging is working
To Reproduce
Steps to reproduce the behavior:

Go to https://www.ssw.com.au/rules/the-goal-of-devops
See error

Expected Behavior

Tiago's name should be displaying the same as on staging, see image below

Figure: How Tiago's last name should be displayed
Tasks

[x] Investigate
[x] Fix

More Information

This seems to only be an issue on production.
From a quick search on Cosmos we can see that there are over 600 occurrences where Tiago's last name is displaying incorrectly.
Environment

Browser: chrome, Edge
OS: Windows

Thanks!
AB#65627

Hey guys,
FYI - It is working fine for me in Production:

Some thoughts:

I believe it's a recent issue, otherwise someone else would have reported before... it's a very obvious bug.
Because it's not happening for all, it might have something to do with the users' system (language, charsets, fonts installed etc)?
I'm happy to have my surname written without the accent, as per:

SSW.People https://www.ssw.com.au/people/tiago-araujo

SSW CRM https://ssw.crm6.dynamics.com/main.aspx?appid=05daa2a9-8768-446e-b9d3-580c8a6f9b8b&pagetype=entityrecord&etn=systemuser&id=fe0b5b92-f8aa-dd11-a7e0-00155d01221b
BTW - I though SSW.Rules took this data from either of those. Where does it come from?

Let me know if I can help with anything else ;)

Done - I've updated the existing entries in the CosmosDB as new updates are displaying the correct encoding.
See https://www.ssw.com.au/rules/the-goal-of-devops

Figure: Tiago's name is now displaying correctly


[SYM-4738] Add slack_user and slack_user_list prompt field types

Current dependencies on/for this PR:

main

PR #144   👈

This comment was auto-generated by Graphite.


animated submit button

can u please assign me this issue?

Hey, @BinilTomJose1278 I`m inquisitive about this. Please are you capable of assign it to me?


Test stale close action

This is an issue to test automatic close action of github

This issue has been automatically marked as stale due to inactivity. If no further activity occurs, it will be closed.

This issue has been closed due to inactivity.


single.html not render if exist a comment in his first line.

$ hugo version
hugo v0.121.2+extended linux/amd64 BuildDate=unknown

Does this issue reproduce with the latest release?
Yes.
How to reproduce:
If I add a comment in the first line of the file layouts/_default/single.html.

{{ define "main" }}

{{ .Title }}

        {{ .Content }}

{{ end }}

Will render an empty page.

If this comment is inside the "define" block will works correctly.

See documentation:
https://gohugo.io/templates/base/#override-the-base-template

Code that you put outside the block definitions can break your layout. This even includes HTML comments. For example...


Detail countryspecific resources for VAT statements

There are documentation missing for multiple countries in this list; Sweden and United Kingdom for example. Both have special VAT reports developed with a required and recommended setup which should be included here as implementation guidelines. Thanks.

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

ID: a3b93833-d8a1-acca-deab-b09919a9cd05
Version Independent ID: 309dd00f-9a84-f3ef-4445-836940030f93
Content: VAT reporting for Europe - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/financials/localizations/emea-vat-reporting.md
Service: dynamics-ax-applications
GitHub Login: @ShylaThompson
Microsoft Alias: v-elgolu

@LizaGolub  Please review this issue and update the topic if necessary.

Hello,
Thank you for your comment. The article mentioned is dedicated to the process of VAT statement generation when data collection is based on Reporting codes and Electronic reporting format. United Kingdom VAT statement cannot be included into this list as this report doesn't use Reporting codes setup and such association could introduce a confusion in understanding of how to setup system for it. For Sweden and Finland there is no specific VAT statement electronic reporting format supported for now. These countries are in the list at the top of the article because Reporting codes are used by these countries in setup of sales tax but there is no specific articles because of absence of specific electronic reporting format.


[mealie] Fix quoting of mealie environment variables

Description of the change

Pipe environment variables to quote to ensure that values such as "False" are not interpreted as boolean.
Benefits

All environment variables are correctly interpreted as strings, avoiding the need for additional quotation.
Possible drawbacks

N/A
Applicable issues

fixes an issue raised in #1591

Additional information

Checklist

[x] Title of the PR starts with chart name (e.g. [home-assistant])
[x] Chart version bumped in Chart.yaml according to semver.
[x] Chart artifacthub.io/changes changelog annotation has been updated in Chart.yaml. See Artifact Hub documentation for more info.
[x] Variables have been documented in the values.yaml file.

Awesome! Thanks for the PR I was going to create one :laughing:


Adding GoogleServicesJson json to Android is breaking the XamariniOS Build Task.

Environment

Server - VSTS
Account Name: RobVoisey
Team Account Name: biotaware
Watchie-Xamarin.iOS-Dev-CI
Build number 1160

Issue Description
We have an application build with Xamarin Forms with two projects for Droid and iOS. Recently we added Firebase + push notifications to the Android project which meant including the GoogleServicesJson json tag in the project file which now looks like:
<ItemGroup>
    <GoogleServicesJson Include="google-services.json" />
</ItemGroup>

Both iOS and Android build on our local machines however the iOS build does not build on VSTS using the XamariniOS Build task (Android does).
If we commented out the above tag it builds without problem.
<ItemGroup>
    <!-- <GoogleServicesJson Include="google-services.json" /> -->
</ItemGroup>

Error logs
Xamarin.iOS task failed with error Error: /Library/Frameworks/Mono.framework/Versions/Current/Commands/xbuild failed with return code: 1. For guidance on setting up the build definition, see https://go.microsoft.com/fwlink/?LinkId=760847.

Errors:
/Users/vsts/agent/2.133.3/work/1/s/Watchie.sln (default targets) ->
(Build target) ->
/Users/vsts/agent/2.133.3/work/1/s/Droid/Watchie.Droid.csproj (default targets) ->
/Users/vsts/agent/2.133.3/work/1/s/packages/Xamarin.GooglePlayServices.Basement.60.1142.1/build/MonoAndroid80/Xamarin.GooglePlayServices.Basement.targets (SetupGoogleServicesJson target) ->
    : error : Error initializing task ConvertToAbsolutePath: Not registered task ConvertToAbsolutePath.
    8 Warning(s)
    1 Error(s)

I'm seeing a similar issue when building from the command line via xbuild. I switch from GCM to FCM, and that brought along some other nuget packages needed.
NugetPackages/Xamarin.GooglePlayServices.Basement.60.1142.1/build/MonoAndroid80/Xamarin.GooglePlayServices.Basement.targets (SetupGoogleServicesJson target) ->
: error : Error initializing task ConvertToAbsolutePath: Not registered task ConvertToAbsolutePath.

I just cleaned my project and used msbuild in place of xbuild and everything built without errors. Installed the app on things are working so far. Any reason why xbuild wouldn't work, whereas msbuild does? I know it's going to be deprecated, but that might mean qualifying a new tool for some industries. :(

@RVoiseyFirstApp: Can you please try switching to "msbuild"?
xbuild is the build tool for XamarinStudio which is being replaced by "Visual Studio for Mac". If you are using "VS for Mac" in your development environment, you should switch to "msbuild". We left xbuild as the default for backcompat but in the latest version of Xamarin iOS task we have removed "xbuild".
@megraham: Thank you for posting your solution. The Xamarin team and community will be better equipped to answer your question on why your build fails with xbuild. You can post it here if you want to follow up: https://forums.xamarin.com/
Thanks,
Madhuri

Closing this issue to clean up the backlog. Reopen this if the issue presents itself again.


lost cookies in request.post

i request a web page with cookies successfully,
then i request.post another one and find that there is no cookies in request packet.why was that?what have i missed?

There are different reasons, but rarely the request.js itself.
Did you set the jar option to true?

Cookies are disabled by default (else, they would be used in subsequent requests). To enable cookies, set jar to true (either in defaults or options).

var request = request.defaults({jar: true})
request('http://www.google.com', function () {
request('http://images.google.com')
})

yeah..I think so.Here is my code:
var request=require('request').defaults({jar:true});
var j=request.jar();
var cookie_obj=request.cookie(cookie_string);
j.setCookie(...,function(err,cookie){
    var options={.....,jar:j,....};
    request.post(options,....);
})

You're right.Thanks for your help.I got the domain wrong.


Unroll Untraced Execution

PR description
Add a second run-to-halt path used when there is no tracing.  This path calls static final versions of select operations with the aim of avoiding the megamorphic dispatch of the original loop.
Fixed Issue(s)

Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[ ] I thought about the changelog and included a changelog update if required.

~20-30% from prior PR,
~41-48% from 22.7.x
The hitch is that these are only for executions that do not involve tracing.  This is standard for a mainnet production, but hedera depends on a tracer for some execution.  I think I can re-work the trace interface to do what hedera needs.
Old (#4533)
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"211.153","gasUsed":"0x96cd6dc2","time":11982032500,"test":"loopMul","fork":"Berlin","d":0,"g":0,"v":0,"postHash":"0x65134195701ec3aa354731a26d9533e946f11e11e60247cc96f57b305778fb12","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"235.737","gasUsed":"0x1778312c4","time":26724899042,"test":"loopMul","fork":"Berlin","d":1,"g":0,"v":0,"postHash":"0xa419240a3f3b2a7858212be7e9d6418d8cffd1673dca395582f191621555993d","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"147.324","gasUsed":"0x212496c6","time":3774310042,"test":"loopMul","fork":"Berlin","d":2,"g":0,"v":0,"postHash":"0x0ed14414e692f2ed97cef2b5a500863e2de358b42351d93b963ef4efb3d13d72","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"212.748","gasUsed":"0x96cd658e","time":11892231375,"test":"loopMul","fork":"Istanbul","d":0,"g":0,"v":0,"postHash":"0xee92d0c049225c53aa9b156986ad4e7cc9ec2524395c60876a17259fadc8aeae","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"235.210","gasUsed":"0x177830a90","time":26784780875,"test":"loopMul","fork":"Istanbul","d":1,"g":0,"v":0,"postHash":"0xe9bca424874ebc2457c9822f6c2f6d2b81ca9194b767c62f9b3c1e554d0a58f3","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"150.844","gasUsed":"0x21248e92","time":3686206708,"test":"loopMul","fork":"Istanbul","d":2,"g":0,"v":0,"postHash":"0xdf31131823f2d621cf2c2d980084681c7ac75a3991480c7d05a7031df34e21c5","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"214.435","gasUsed":"0x96cd6dc2","time":11798662084,"test":"loopMul","fork":"London","d":0,"g":0,"v":0,"postHash":"0x662588feffcd4391bc440510de21cbfa8f6c59ceafb2384ed706eccb55ec3a92","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"237.000","gasUsed":"0x1778312c4","time":26582431333,"test":"loopMul","fork":"London","d":1,"g":0,"v":0,"postHash":"0x9a8c08120c2517fd2f5c96d377b401258121e3e44b9e5086ac236c6438bfac8e","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"151.160","gasUsed":"0x212496c6","time":3678536750,"test":"loopMul","fork":"London","d":2,"g":0,"v":0,"postHash":"0x221d03f92d6384bd7f947f156244f75262164d483dd1e11ee3f4c32e4030d9fe","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}```

New
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"255.379","gasUsed":"0x96cd6dc2","time":9907033375,"test":"loopMul","fork":"Berlin","d":0,"g":0,"v":0,"postHash":"0x65134195701ec3aa354731a26d9533e946f11e11e60247cc96f57b305778fb12","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"336.091","gasUsed":"0x1778312c4","time":18745051292,"test":"loopMul","fork":"Berlin","d":1,"g":0,"v":0,"postHash":"0xa419240a3f3b2a7858212be7e9d6418d8cffd1673dca395582f191621555993d","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"191.400","gasUsed":"0x212496c6","time":2905156125,"test":"loopMul","fork":"Berlin","d":2,"g":0,"v":0,"postHash":"0x0ed14414e692f2ed97cef2b5a500863e2de358b42351d93b963ef4efb3d13d72","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"293.888","gasUsed":"0x96cd658e","time":8608883125,"test":"loopMul","fork":"Istanbul","d":0,"g":0,"v":0,"postHash":"0xee92d0c049225c53aa9b156986ad4e7cc9ec2524395c60876a17259fadc8aeae","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"334.783","gasUsed":"0x177830a90","time":18818286917,"test":"loopMul","fork":"Istanbul","d":1,"g":0,"v":0,"postHash":"0xe9bca424874ebc2457c9822f6c2f6d2b81ca9194b767c62f9b3c1e554d0a58f3","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"186.399","gasUsed":"0x21248e92","time":2983077875,"test":"loopMul","fork":"Istanbul","d":2,"g":0,"v":0,"postHash":"0xdf31131823f2d621cf2c2d980084681c7ac75a3991480c7d05a7031df34e21c5","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"e2ef8b895ee03259ee57850b1a1ed4aa7689bec352659200e63440303f9d0b81","Mgps":"293.697","gasUsed":"0x96cd6dc2","time":8614488584,"test":"loopMul","fork":"London","d":0,"g":0,"v":0,"postHash":"0x662588feffcd4391bc440510de21cbfa8f6c59ceafb2384ed706eccb55ec3a92","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"ffffffffffffffffffffffffffaaffffffffffffffffbbffffffffffffff0009","Mgps":"331.839","gasUsed":"0x1778312c4","time":18985231666,"test":"loopMul","fork":"London","d":1,"g":0,"v":0,"postHash":"0x9a8c08120c2517fd2f5c96d377b401258121e3e44b9e5086ac236c6438bfac8e","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}
{"output":"0e1c6aac6663c379a52d9ccc7ba4757131020772d41447dfcf478cf9fb0c2bbf","Mgps":"186.066","gasUsed":"0x212496c6","time":2988433875,"test":"loopMul","fork":"London","d":2,"g":0,"v":0,"postHash":"0x221d03f92d6384bd7f947f156244f75262164d483dd1e11ee3f4c32e4030d9fe","postLogsHash":"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347","pass":true}

Work isn't done, I should finish unrolling the basic arithmetic that hasn't changed since Frontier.  Keeping the operations for complex operations won't have too bad an impact and preserves the customizability of the Besu EVM.


Unpin Flatbuffer version dependency

Combining DataFusion 7.1.0 and Arrow 12.0.0 pose a challenge due to pinned flat buffer version, or at least that is what error message tells me. From what I have seen version pinning was a temporary fix to workaround incompatible changes in flatbuffers, but that doesn't apply anymore. Because these versions of flatbuffers dep were dropped from crates.io and anyway right now Arrow uses newest version of FlatBuffer dependency.

FWIW Datafusion 7.1.0 has a version pin on arrow 9.0.0, and so cannot be used with arrow 12.0.0 regardless... What will likely end up happening is cargo will link in both versions of arrow and complain if you try to mix types from one version with the other.