Databricks Unity Catalog Terraform Setup Guide Misconfigurations/Additions

Following and setting up a test workspace of the Unity Catalog with Terraform, I noticed a few misconfigurations and additions that may be helpful within the guide:

The referenced admins group within the .tfvars files must be created within the Databricks Account Console. The Local Workspace admins group cannot be used with the Unity Catalog.
The storage privileges within the .tfvars files must now be ALL_PRIVILEGES instead of ALL PRIVILEGES.
User deploying must have rights to the external Storage Account for the databricks_grants in external-storage-grants.tf to deploy. I used the Storage Account Data Owner permission for simplicity since I was learning how things worked, but there may be a better, least permissive role to use instead.
The databricks_external_location resource within external-storage.tf has the url backwards. It should have azurerm_storage_container.ext_storage.name prior to azurerm_storage_account.ext_storage.name for the formatted url string to be correct. The end result would be<EMAIL_ADDRESS>
For reference, these are the Terraform provider versions I was using during deployment:

azure/azapi: v1.6.0
databricks/databricks: v1.19.0
hashicorp/azurerm: v3.61.0

I'd offer to create a PR to help update these, but so far, I haven't been able to find the referenced article within the azure-docs repo. The Content Source link below also leads to a 404 page.

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

ID: 1750d9e5-33cf-fe64-cecb-29567c2e79ed
Version Independent ID: 1750d9e5-33cf-fe64-cecb-29567c2e79ed
Content: Automate Unity Catalog setup using Terraform - Azure Databricks
Content Source: databricks/data-governance/unity-catalog/automate.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@btreich
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for providing feedback that helps improve our documentation. We've created an internal work item (DOC-9640) to address your feedback. The timeline for resolution varies based on resourcing.
#please-close


Check passwordless capacity for date strictly aganst the date command.

This fix https://github.com/SignalK/set-system-time/issues/4
Previously,  the ability of using the date command without password was checked aganst the echo command.
Witch would fail in restrictive context where only  sudo date command is allowed password less and not sudo echo
Sudoers entry for reference :
signalk ALL=(ALL) NOPASSWD: /bin/date

Note: The command have been tested the a terminal only.

Please add the sudoers entry to README.

2nd commit added

Thanks! Published as<EMAIL_ADDRESS>


Fixed minor dependencies for fresh deployment

Here are some changes and dependencies that needs to be added to make the 03_segmentation_app and 04_mis_tutorial notebooks work.

@vikashg Please commit with signature to pass DCO requirement.
See https://github.com/Project-MONAI/monai-deploy-app-sdk/pull/208/checks?check_run_id=4317364632

Thanks @vikashg for the change!


Handle different kinds of problems differently!

https://github.com/smucclaw/baby-l4/blob/247752a5db143ae05bb60bbafc34d7d104921e95/lsp/Main.hs#L210-L212
This function should be replaced with one that allows us to capture different types of errors. Later, where we handle these errors, we need to handle them in different ways, including:

Just ignoring some minor issues and just not displaying any hover information
Showing diagnostic information when we have a location for the issues (and making sure that the diagnostics are preserved as long as they should)
Showing a different kind of error, e.g. a pop-up-box for more serious errors and bugs that doesn't have a relevant location.

This issue depends on #66

Example of how to send messages: https://github.com/haskell/lsp/blob/master/lsp/example/Reactor.hs#L179


Add 'std::' for cmath functions

As discussed in #2691 we can't rely on the cmath functions working without an explicit 'std::'. I added 'std::'s for every cmath function, which wasn't already present in foundation::.

I now added all of the cmath functions, plus min and max. Also, I included the functions which had a ( in front (instead of a space).


AzureOpenAIClient with DefaultOpenAIClient() throws intermittent 401s

I have the following setup as a Singleton using dependency injection:
new AzureOpenAIClient(
    new Uri(endpoint),
    new DefaultAzureCredential(),
    new AzureOpenAIClientOptions(ServiceVersion.V2024_06_01)
);

I then inject this into my application and use this code to chat:
var chatClient = _openAIClient.GetChatClient(deploymentName);
var response = await chatClient.CompleteChatAsync(messages, options);

Intermittently, this call returns a 401. It seems to happen every 30 seconds or so and lasts for 30 seconds, then it goes away. Is there an issue with token management? FYI, this code worked fine before v2.0.

Thank you for reaching out, @cdwaddell  ! This repo corresponds to the OpenAI library. Since you're using the AzureOpenAIClient with the DefaultAzureCredential from the Azure.AI.OpenAI library, could you file this issue in the Azure repo instead? Here's a link: 🔗 https://github.com/Azure/azure-sdk-for-net
Thank you!

Thanks, here is the bug if anyone needs it:
https://github.com/Azure/azure-sdk-for-net/issues/44910


Fix staticPlugin check

The truthiness if staticPlugin is checked to determine if Happo should
use the remoteRunner, but since this is always assigned to an object
now, the logic was broken and Happo would never end up using the
domRunner. I believe we can fix this by being a little more careful
about how we assign the staticPlugin variable.

There is coverage, I just hadn't been running them on the branch (triggered on PR, not push).


Rename EMULATOR to avoid confusion and separate to other repository

Really it's not an Emulator.
Here are implementation details:

high-level Playstation API reimplementation which translates Playstation calls into compatible APIs
SDK that allowing to compile originally targeted Playstation games on other platforms
Psy-Q compatible headers
Implements Geometry Transformation Engine (GTE) in software and adapts it's calls
Implements LibSPU with ADPCM decoding on OpenAL
Implements LibGPU with Playstation-style polygon and image handling

Suggested names:

PsyXtend
PsyPort
PsyQRe
RePsyQ

dun.

will separate later


fix: Skip temporary multi component tests due an error on application-service cdq

Description
Multi-component is not fully supported yet due an error in HAS service.
Please see jira for more context: https://issues.redhat.com/browse/DEVHAS-225
Issue ticket number and link
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("[test_id:01] DEVHAS-62 devfile source")
[ ] I have updated labels (if needed)

/retest

let's pickup another lotter ticket for claims!
/retest

/retest

/test redhat-appstudio-e2e


Remove remaining apex-net s3 bucket references from the code

Fix binaries artifacts name in build job and scripts
Fix namespace name in makefile for run-sql-apiserver target
Fix apexd references in ansible vars/readme

@russellb @nerdalert YOLOing this to trigger the deploy job with latest code and debug the failures.


Update jsonnet vendor folder in Makefile to root vendor/

For some unknown reason we started using the wrong jsonnet vendor folder. This fixes it to use the top-level vendor/ again.

This is not needed anymore, as we move the operator out of this repository into its separate.


Issues in training and loss/accuracy computation

In the file model_train.py, in the training loop, I see below code in L175-179:
output = model.tmodel(input_data)
_, preds = torch.max(output, 1)
running_corrects += torch.sum(preds == labels.data)
loss = model_criterion(output, labels)
What I understood is model.tmodel is the feature extractor, but yet we are comparing the outputs with target labels. Shouldn't we compare it with the last layer (classifier) output?

I think I got now. tmodelis not feature extractor actually but a full network.
Apologies for the confusion. :(


⚠ Processors has degraded performance

In 94291f0, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in f1d65b3.


Implement Serialize and Deserialize for RangeInclusive

Copied implementation from Range, is it OK?
Edit: I will fix the failing tests but RangeInclusive structure was modified in #51622, should we wait for the next Rust release?

@dtolnay Tests are fixed. Not sure about the additional field since every new instance of RangeInclusive starts with the same value is_empty: None.

I published these impls in serde 1.0.71.


woodpecker-{agent,cli,server}: init at 0.15.3

Description of changes
Supersedes #169470.
Closes #178332.
Some notes :

[x] We probably want to prefix the different binaries with woodpecker-, instead of just cli, agent, and server.
[x] Address/remove the FIXME about musl.
[x] Perhaps add an update script.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Do you plan to add a module?

Not sure why ofborg is unhappy about (what seems to be) the mkYarnPackage derivation.

they all just copy in package.json, its a simple wget or curl in the update script. Again I'll see if I can tweak yarn2nix to stop the IFD issues when I get some time

I made some changes https://github.com/NixOS/nixpkgs/commit/e243a8c9ec4043cb436ed2d570b6fb975cb9d47c

adds prefixes to bins
removes the need for libexec/xyz etc by just keeping the dist folder

before: λ du -h -d 0 ./result/ -> 359M
after: λ du -h -d 0 ./result/ -> 664K

seperate the frontend to its own input, this is important so people can fix it if it breaks like jellyfin and all other node stuff did recently https://github.com/NixOS/nixpkgs/issues/176288
drop the extra static stuff, it seems to cause more issues than it solves, we don't need to exactly copy the upstream builds

LMK if this resolves the glibc musl stuff, not sure what the actual issue was in order to test it

server started and ui shows fine but that's all I did:
WOODPECKER_GITHUB=true ./result/bin/woodpecker-server --server-host http://<IP_ADDRESS>

@06kellyjac I think we should only keep the front-end as a passthru attribute, I don't think it makes much sense to have it as an actual package in nixpkgs but it would still allow overriding thusly (approximately? not tested):
self: super:
{
  woodpecker-server = super.woodpecker-server.override {
    woodpecker-frontend = super.woodpecker-server.passthru.woodpecker-frontend.override {
      # Override here
    };
  };
}

Pushed some fixups (which will need to be rebased/squashed).

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

3 packages built:

woodpecker-agent
woodpecker-cli
woodpecker-server

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/531


How to restore the local environment when i remove it carelessly?

After updating to Portainer 2.15 version, I can't enter the local environment, and  I remove it ...
I delete all Portainer container and images, rollback to old version,  but no effect

@CcCcaidan
The Portainer Backup is only for Portainer configurations and stacks. We do not include containers or volumes in the backup.  You will need to manually restore.
Thanks!


Does not start with node.js 7.3.0

I get the following error:
% sudo /usr/bin/node /srv/http/node-sonos-web-controller/server.js
module.js:472
    throw err;
    ^

Error: Cannot find module 'node-static'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/srv/http/node-sonos-web-controller/server.js:3:22)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)

Are you sure you have installed the dependencies correctly with npm
install?
Does it work if you start it from the actual web controller directory?
On 3 Jan 2017 7:56 AM, "hexadecagram"<EMAIL_ADDRESS>wrote:

I get the following error:
% sudo /usr/bin/node /srv/http/node-sonos-web-controller/server.js
module.js:472
throw err;
^
Error: Cannot find module 'node-static'
at Function.Module._resolveFilename (module.js:470:15)
at Function.Module._load (module.js:418:25)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at Object. (/srv/http/node-sonos-web-controller/server.js:3:22)
at Module._compile (module.js:571:32)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jishi/node-sonos-web-controller/issues/58, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADBNI9OhSTMY1yTzOGN11Ug2PbRhWtzEks5rOfEVgaJpZM4LZYNd
.

I'm installing this on a new machine and forgot the npm install step. When I attempt to run it in the controller directory, it appears to work but I get the following:
% sudo npm install
npm WARN<EMAIL_ADDRESS>No license field.
% sudo npm update
npm ERR! Linux 4.4.39-1-lts
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "update"
npm ERR! node v7.3.0
npm ERR! npm  v4.0.5
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/sonos-discovery
npm ERR! 404
npm ERR! 404  'sonos-discovery' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /srv/http/node-sonos-web-controller/npm-debug.log

Weird. Did you get any error during the npm install step? That package is
referenced via a github URL, and should have been installed correctly. You
could try and install it manually, using the URL in package.json, as a
test.
On 3 Jan 2017 8:32 AM, "hexadecagram"<EMAIL_ADDRESS>wrote:

I'm installing this on a new machine and forgot the npm install step. When
I attempt to run it in the controller directory, it appears to work but I
get the following:
% sudo npm install
npm WARN<EMAIL_ADDRESS>No license field.
% sudo npm update
npm ERR! Linux 4.4.39-1-lts
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "update"
npm ERR! node v7.3.0
npm ERR! npm  v4.0.5
npm ERR! code E404
npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/sonos-discovery
npm ERR! 404
npm ERR! 404  'sonos-discovery' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.
npm ERR! Please include the following file with any support request:
npm ERR!     /srv/http/node-sonos-web-controller/npm-debug.log
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jishi/node-sonos-web-controller/issues/58#issuecomment-270062583,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBNI8ULFTEEgKvwnW3T1Q_gC720MW6wks5rOfmKgaJpZM4LZYNd
.

No errors. npm install just spat out that warning after displaying a tree. Do you want to see npm-debug.log generated by npm update?
Where should I install sonos-discovery? And how would I go about doing it manually?

Aha, you ran npm update. That one tries to update to the latest available version of a dependency, that might not work for that depencency. The install ran without outputting anything, which means it thinks that all deps are filled. Do you still get the same error when trying to start it?

For manually installing the package, just use the url from package.json and use with npm install:
npm install https://github.com/jishi/node-sonos-discovery/archive/v1.1.3.tar.gz
But as I said, it shouldn't be necessary unless npm 4+ has started with some weird behavior, haven't tested that version yet.

Aha, you ran npm update.

Not exactly. Re-read the output here and then this comment more closely. I ran npm install first, which gave me the tree (which I edited out of the output for brevity) followed by the warning and then I ran npm update.
I do see sonos-discovery when I npm ls. It turns out that it was installed with my first run of npm install and I did not need to install it using the URL you provided after all.
% npm ls
└─┬<EMAIL_ADDRESS>
At any rate, the service now starts but there are no Sonos players listed in the left hand pane, which I was experiencing with my other machine. Is there any way to see debug logs?
The warning about the license field also seems to be interfering with my ability to npm uninstall sonos-web-controller.

All logging is sent to stdout. You can increase logging level by starting it with env variable NODE_LOG_LEVEL set to "debug":
NODE_LOG_LEVEL=debug npm start
What are you running this on? If it doesn't find a system, it's usually because of network restrictions.

Duh! This Linux machine is on a different subnet than Sonos. So it's back to the other machine.
As I mentioned, though, the license warning seems to be preventing me from uninstalling or updating with npm, which means both machines are dead in the water, one running node-sonos-web-controller 0.13.0, which is on the correct subnet but fails to start and the other 0.14.0, which starts but is on the wrong subnet.

Okay, I think my newness with node.js is showing. ;-)
I just ran git pull on the old machine, which updated it to 0.14.0, which is reflected in npm ls. I thought npm would install dependencies in some subdirectory off the root but no, they go in ./node_modules apparently.
Everything is working now, and look at that! I see my Sonos devices too!

This software would actually traverse one routing jump if everything is forwarded correctly. However, it's not trivial to get running, depending on your setup. Basically it would require:
multicast forwarding (multiplexing) of UDP 1900 to <IP_ADDRESS> (or broadcast <IP_ADDRESS>), also allow unicast return traffic on UDP port 1905 (dynamically selected, but 1905 is the first attempt)
port forward of TCP 1400 to Sonos network
port forward of TCP 3500 from Sonos network (dynamically selected, but 3500 is first attempt, then 3501 if occupied etc)


Update eslint-config

Details
Updates the eslint-config repo.
Related Issues
https://github.com/Expensify/react-native-onyx/issues/136
Automated Tests
Covered by lint and automated tests
Linked PRs

Oh dang, lotsa changes but from skimming they looks good. I need just a little bit longer to review this.

Most changes are harmless (just adding commas, line breaks, etc). But there are a few that demand more attention 😅

@luacmartins Do we need to bump the version of react-native-onyx on NewDot/wherever else too?


NextGen BMI integration changes

These commits are a cleaned up presentation of the substantive changes I've made to the SCHISM BMI code as a result of my efforts to integrate it with the NextGen model engine.
Each commit should be reviewed on its own. I figured it would be less work for everyone to just say this, rather than making a PR per commit.

There's at least one more commit coming here. I just noticed a bug in itemsize

The problem with itemsize was that it was returning the value from sizeof applied to arrays, no individual items therein. As documented for both GNU and Intel, and tested on GNU with this code, that would give the memory footprint of the entire array:
program foo
  integer(8) :: test_array(10)
  print*, 'sizeof(test_array): ', sizeof(test_array)
end program foo

All the changes to the SCHISM BMI look great to me and are straightforward with the reasoning and improvements to the coastal model BMI implementation under the framework. Thanks @PhilMiller for all your work on this! Once @hellkite500 gives the green light, I'll go ahead and merge this into the SCHISM BMI repo.


Clarify mesh conventions

pmesh paint() and readout() seem to project particles in [0, boxsize] to [0, nmesh].
Yet, as defined in pmesh, x-coordinates larger than nmesh//2 are flipped to [- nmesh//2, 0].
Hence this function: https://github.com/adematti/mockfactory/blob/7df2a96bee83db561a64188a2c5491777852c2ce/mockfactory/gaussian_mock.py#L49
To keep current pmesh x-coordinates, one could also paint positions in [-boxsize/2, boxsize/2] (then the [-boxsize/2,0] part is shifted to [boxsize/2, boxsize] in the mesh array), e.g. https://github.com/bccp/nbodykit/issues/659, but this is somewhat counter-intuitive (to me).
It'd be great to stick with one convention in the clustering pipeline.
@pmcdonal, any suggestion?

This is the kind of thing... I'm not smart enough to stop right now and figure it out out of context... I'm sure I'd have an opinion if I got to using it, but I'd try not to freeze in any uncertain choice unnecessarily...


How to add/register custom JsonProvider?

Could you give me a hint, how to add/register newly created JsonProvider with logback?
TBH I wasn't able to find doc on that.
I want to use my provider with net.logstash.logback.encoder.LoggingEventCompositeJsonEncoder.
Thanks.

<encoder class="net.logstash.logback.encoder.LoggingEventCompositeJsonEncoder">
  <providers>
    <provider class="your.provider.class.name">
        <!-- Any properties exposed by your provider can be set here -->
    </provider>
  </providers>
</encoder>

Will update the docs to make this more clear.

Ok, but then there is no chance to have DI here, right?

Logback is not aware of any dependency injection frameworks.
However, if you really want to use DI to assemble your logback configuration, you can configure logback programmatically, rather than using xml configuration.   #252 has an example of someone using spring to configure logback.
There's a little bit of an ordering problem here though.  Since logback is generally initialized prior to DI frameworks.


Add industries to marker pop-up

Markup should include industries.

#65 does this; currently assuming each user belongs to exactly 1 industry.


datetimes are read with time zone offset

I'm not sure if this is a question or a bug yet, but let me explain my problem.
When I specify a dateFormat like "MM/dd/yyyy HH:mm:ss" and read in a date like
"07/09/2015 00:00:00"
It interprets it as having the local (PST) timezone. I really want it to have UTC time zone.
If I print the date with UTC timezone, it report something like 2015-07-09 00:00:00-08:00.
How can I read a date and have it interpreted as UTC?
One way might be to change the date format in the file so that dates end with Z or -00:00 and use a corresponding dateFormat option. But what if I cannot change my datafile date format? Can we have a way to specify the timezone used when reading dates (if it is not specified)? Another timeZone option would allow for this, but I'm not sure if its the best solution.

In next version of the package you will be able to specify format. See this PR: https://github.com/databricks/spark-csv/pull/280

Yes. I am aware that the next version will allow specifying the format. In fact, we are already using that using a SNAPSHOT. This question/request is about specifying the time zone when reading, instead of using the local time zone. In my case I want to use GMT/UTC time zone. Can we have an additional option for that?

If your data has timezone, then you can use format to read them correctly. If they don't I think timezone transformations are better handled with DataFrame transformations (e.g., adding a new column with fixed date).

I could add a DataFrame transformation to shift the timezone after the data has been loaded, but I  thought it would be more convenient and efficient to specify it when loading.


[rfr][preprints] Add custom signup/ campaign page for OSF Preprints service

Purpose
Add a new campaign, so that users of the preprints service (through the OSF) will see a custom signup page and receive unique signup emails.
Sample URL: http://staging.osf.io/login?campaign=osf-preprints
If they click "use an existing account", then when they are done with the login process, they will be taken to staging.osf.io/preprints (the preprints landing page)
Upon confirming an account, they should be taken to the landing page for the campaign (eg staging.osf.io/preprints)

Changes
Add new osf-preprints campaign: custom signup page, and track users who came in this way via metrics.
Side effects
None expected.
Ticket
https://openscience.atlassian.net/browse/PREP-131
TODO

[x] Language approval for email and landing page

This seems like it'll be fine. If it works, it's good. 🎉 🐧 🎉

LGTM. Tested and confirmed :shipit:


Gradient Buttons

I'm making a new issue for each feature I added to discuss it because I added too many features at once in my fork. Here are the gradient buttons I added:

To use a gradient button, you just set the attribute "gradient" to true in xml. There are gradient square buttons as well, just not pictured

how to make "onClic and open new activity" on https://github.com/theDazzler/Android-Bootstrap.??


CodeCop rule AS0016 goes off too often and does not follow the documentation

Describe the bug
This rule goes off with every field that does not have DataClassification. In our software the DataClassification is often specified on table level and only for non-standard fields on field level.
According to the documentation this should be possible. However, the codecop will go off on all fields where nothing is currently specified.
Documentation:

When the DataClassification property is set on a table level, the fields on the table will inherit the property value. If the DataClassification property is set on both a table and field level, then the data classification specified on a field level will override the one from the table.

To Reproduce
Steps and to reproduce the behavior:

Make sure CodeCop AS0016 generates errors.
Create a table as following:

table 50000 "Demo Table"
{
    DataClassification = CustomerContent;
    fields
    {
        field(1; "My Field"; Code[20])
        {
            Caption = 'My Field';
        }
    }
}

Compile the solution and see in the output of the compilation Error AS0016: Field with name 'My Field' must use the DataClassification property and its value should be different from ToBeClassified.

Expected behavior
The codeCop should see if a DataClassification is specified at table level. If specified, it should not proceed at the field level.
Screenshots
None
5. Versions:

AL Language: v6.1.381323

Business Central: 17.0.16993.0-w1

Hi @sybrenwille, thank you for reporting this issue and for the good repro.
This issue has already been fixed in our latest developer previews (AL extension version 7.*).
The field data classification is now inherited from the table.

Hi @sybrenwille, thank you for reporting this issue and for the good repro.
This issue has already been fixed in our latest developer previews (AL extension version 7.*).
The field data classification is now inherited from the table.

Hi @qutreson that's exactly what is needed.
I'm looking forward to the availability of that version

Hi @qutreson that's exactly what is needed.
I'm looking forward to the availability of that version

I am closing this issue because it appears that it has been resolved. Please open a new issue if you believe this has not been resolved and reference the current issue.


Occasional flicker of black screen when zooming (or moving camera?)

Not related to culling is it? Can I make it happen reliably somehow to test?


Konga on Docker

Hi, I got an error

Failed to connect to kong_test.
Make sure your active connection is valid and Kong is up and running.

I got an error after docker-compose down then "docker-compose up" again

version: '2.2'
services:
kong-database:
image: postgres:9.4-alpine
container_name: kong-database
environment:
- POSTGRES_USER=kong
- POSTGRES_DB=kong
healthcheck:
test: "pg_isready -U kong && psql -d kong -U kong -c "SELECT 1=1""
interval: 10s
timeout: 5s
retries: 5
kong-migration:
image: kong
container_name: kong-migration
depends_on:
kong-database:
condition: service_healthy
environment:
- KONG_DATABASE=postgres
- KONG_PG_HOST=kong-database
command: sh -c "kong migrations up && touch migrations_run && sleep 30"
healthcheck:
test: "if [[ -f migrations_run ]] ; then exit 0; else exit 1; fi"
interval: 10s
timeout: 5s
retries: 5
kong:
image: kong
container_name: kong
depends_on:
kong-migration:
condition: service_healthy
healthcheck:
test: "curl localhost:8001"
interval: 10s
timeout: 5s
retries: 5
environment:
- KONG_DATABASE=postgres
- KONG_PG_HOST=kong-database
- KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001
ports:
- 8001:8001
- 8000:8000
konga:
image: pantsel/konga
container_name: konga
ports:
- 1337:1337
environment:
- NODE_ENV=development

after that cannot create new connection,
different version when first create connection(0.12.x) on dashboard and /connections > new connection (0.10.x)

@sulthonzh,
what version of Kong and Konga are using?

@pantsel latest version from docker hub like docker compose, but (on first create at dashboard i see 0.2.x and when create from /connection (0.10.x))

Are you sure the the Kongs Admin api is accessible from the machine Konga is running?

It's probably me but I have the same issue - Container can ping to kong:8001, but can't connect. Am I missing something?

@tomkerkhove , can you curl from konga container to kong:8001? ]
What is the response?

@pantsel Can't use Curl nor apt-get since it's not in the image but you can ignore my comment. I'll just use the API directly. Thank you anyway!

Closing this due to incactivity

I also suffered from this pain.
But I guess, I just needed to logout and login again on Konga.
Any thoughts on this?
Thank you for this nice dashboard.

I also suffered from this pain.
But I guess, I just needed to logout and login again on Konga.
Any thoughts on this?
Thank you for this nice dashboard.

Thanks @AlexandreLage.


Add Generate CSRF PoC workflow

This pull request adds the CSRF PoC generator workflow, which is useful when creating the PoC for CSRF.
Currently, only application/x-www-form-urlencoded and application/json (with the text/plain technique) through the form submission are supported.
While the parsers included in this workflow are not spec-compliant, it should be able to handle requests from the browser.

[x] Follow the same folder structure as other workflows (see https://github.com/caido/workflows/tree/main/convert)
[x] Has a proper author name and workflow description

Eyyyy, this is sick! Nice work.

I've modified my local copy to only encode the required characters for a more easy-to-read CSRF PoC:
function encodeHTML(str) {
  let result = "";
  for(const char of str) {
    if (!char.match(/["'<>]/)){ result+=char}
    else{result += `&#${char.charCodeAt()};`;}
  }
  return result;
}

Thanks for the submission!


Disable neon for apple targets

Workaround for #230.
Don't really have time to look at this more in depth, so this just disables neon intrinsics for apple targets leaving the rest of them untouched. There's probably just a missing flag or something, but someone who actually has the will to compile for aarch64-apple-darwin can maybe figure that out.

Not aarch64-apple-darwin, https://github.com/rust-lang/libz-sys/blob/223f5650c75fb4c0fec5e00fc47f2fd6b1a95cc2/.github/workflows/ci.yml#L79C9-L80C32


External Search for existing annotations

This feature lets you quickly do an "external search" to Golr server for existing annotations for that particular GP and aspect. It works just like the Component Companion.
In Brief
Step 1 - Enter Gene Product

Step 2 - Select 1 or more rows and Click Save

Step 3 - Your fields are populated

@thomaspd  Can you explain in more details.
@kltm @cmungall

is this scheduled for 1.0?

I don’t think it’s in the requirements but it’s one of the first things curators ask for when they see Noctua.  So as long as it doesn’t delay the 1.0 release date I think it would be good to include it.
From: Chris Mungall<EMAIL_ADDRESS>Reply-To: geneontology/simple-annoton-editor<EMAIL_ADDRESS>Date: Monday, March 12, 2018 at 2:59 PM
To: geneontology/simple-annoton-editor<EMAIL_ADDRESS>Cc: Paul Denis Thomas<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [geneontology/simple-annoton-editor] External Search for existing annotations (#41)
Resent-From<EMAIL_ADDRESS>is this scheduled for 1.0?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_geneontology_simple-2Dannoton-2Deditor_issues_41-23issuecomment-2D372476909&d=DwMCaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=33hhNKh7WpvJRiaXHf366Q&m=7qza5HEeWGPzTjU0rszp_vxrWrLbp0CGa9JQKv0re9c&s=G5QugsCMvK7yEuzeYsqT9th9SzqvIBP2AnmitNqSU84&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ALaXeMfRlkgJ39VrD-2DXt59rAy-5FsE45iEks5tdu-2DkgaJpZM4SnvXy&d=DwMCaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=33hhNKh7WpvJRiaXHf366Q&m=7qza5HEeWGPzTjU0rszp_vxrWrLbp0CGa9JQKv0re9c&s=zqyyDdbz76qHHPu65pMqGoXrfnQHzjgW4qjVFOEWkb0&e=.

The major use case is for curators trying to create a larger "pathway" or other model, using annotations that are already in the GO database.  This was in the original spec Huaiyu and I drew up years ago, and multiple curation groups asked for this functionality during the Noctua demos at EBI and the Corvallis meeting.  UCL was waiting for this functionality before creating larger models for SynGO.

@cmungall @kltm from the earlier conversation about groups,

Is there a way to attribute (providedBy) on each evidence node when a user is using annotations that are already in the GO database?
If(1) then how do I easily get the uri from the group shorthand?

This sounds like:
https://github.com/geneontology/noctua/issues/539#issue-297183549
https://github.com/geneontology/noctua/issues/458#issuecomment-369333576
I believe the current function companion technically accomplishes your 1 (and maybe 2), but I think we want to revisit if that is really what we want to do. As well, we should not have independent implementations of this translation.

Also see: https://github.com/geneontology/simple-annoton-editor/issues/78


Introducing 2 Step Approach to Allow Game Selection

Hey,
I've added a menu to pick the game you want to launch as I wanted to use that also with Emulators for Wii/Switch or non-Steam Games. Furthermore I also made some docker adaptations, i.e. specifying a name of the container and I've added --rm to remove the container after exiting to cleanup the ressources again after launch.
The game name is passed to the Launch Script by an ENV Variable. I would probably later add config options in the add-on for framerate and resolution and pass this also by ENV Variables to the launch Script.

Thanks for this.  I ended up tweaking the environment variable setup a bit to simplify the passing of the resolution/fps/bitrate options.  Let me know if you think there is a better way to do that.

Thanks for this.  I ended up tweaking the environment variable setup a bit to simplify the passing of the resolution/fps/bitrate options.  Let me know if you think there is a better way to do that.


GADMAdapterMobFox duplicated files

Hi,
I'm trying to integrate MobFox Admob adapter but inside the project the files GADMAdapterMobFox.h/.m are duplicated inside the Adapters folder and also inside the Samples/AdMobAdapterSample/AdMobAdapterSample.
Since they are different which version is the correct one ?
BR,

Hi,
Please use this folder: https://github.com/mobfox/MobFox-iOS-SDK/tree/master/Adapters/AdMob
We will fix this duplication soon.

Hi,
Thanks.
Another error I'm getting when using the pod version is that MFEventsHandler.h is not exposed in the public headers.
Is the pod deprecated ?
BR,

As a side note, I decided to use the latest release from this repo.
The module could not be compiled becaused MobFoxAdTest.h is not added to MobFoxCore.h
I've added the missing import and it works.
Still, I'm not sure if the MobFoxAdTest.h is supposed to be there.
Could you please check this ?
Thanks,

Hi,
Have you integrated the SDK manually or using cocoa pods?

Hi,
I've tried using cocoapods but got the MFEventsHandler error.
Now I'm trying it manually but got the missing MobFoxAdTest import error.
Just trying to make this work :)

ok, we're trying to reproduce and fix this. Will update this issue.

Uploaded a fix and also updated cocoapods. Sorry, this is what we get for rushing things this week.


Bug: Revision of KeptnAppVersion is bumped on every new version

Description
The revision of KeptnAppVersion is bumped always when KeptnApp is updated without taking the version of KeptnApp into notice.
How to reproduce
Applying 3x KeptnApp with versions 0.1.0, 0.1.1, 0.1.2 one after another
Result:
List of KeptnAppVersion resources looks like:
app-0.1.0-1
app-0.1.1-2
app-0.1.2-3
It should look like the following:
app-0.1.0-1
app-0.1.1-1
app-0.1.2-1
Problem
The problem is that the .metadata.generation field of KeptnApp get's bumped with every update of the resource, This field is used to generate the name of the KeptnAppVersion.
Generating the name here
Generating the KeptnAppVersion here

This is something between bug and enhancement 😓
The original application changed, hence we have an identifier for that. For your suggestion, we require a K8s API call to fetch the revision/generation of KAV, if it exists. I don't know if it is worth the extra complexity for this cosmetic change 🤔

The maintainers agreed to use a random hash using the following format ${appName}-${appVersion}-${hash} to avoid giving semantics to the final number that is used only as a unique identifier. This is also for following the pod nomenclature


usvg ignores style="mask-type:alpha" produced by figma

It looks like usvg correctly passes through the mask-type="alpha" attribute, but strips the equivalent style="mask-type:alpha" from mask tags. Unfortunately, figma svg exports use style="mask-type:alpha", so usvg ends up mangling them.
Ideal behavior: usvg converts from style="mask-type:alpha" to mask-type="alpha"
Samples (these are svg's):
using mask-type="alpha"
Before usvg

After usvg

using style="mask-type:alpha"
Before usvg

After usvg

Yes, this is known to not be supported yet, see also https://github.com/docs/svg2-changelog.md

Thanks! Easy fix. The SVG2 spec is a complete mess and I didn't noticed that mask-type is a presentation attribute.

Thanks for the quick fix!


Dawn Integration Part 2

Description
This PR is the second portion of the integration with the Dawn compiler and includes implementations of the various Dawn backends for CPU and GPU.

I still want to take a deeper look to the changes in dawn_backends but my main concerns with the PR have been raised already raised by @jdahm (and fixed), so it looks closer to be accepted.
One important thing that is missing is to verify that all the external people contributing to this PR have signed a contributor assignment agreement and to update the AUTHORS.rst file with their names.

daw4py dependency in setup.cfg should be pinned to a specific version by adding @tag in the url before #subdirectory=dawn. For example: dawn4py<EMAIL_ADDRESS>

I added a comment to your last commit but it is mostly informative, just take a look. The current state of the PR is good for me so if no one else has complaints, I will merge the PR in 24 hours.

Ooops! I really should've caught this one! 😥

src/gt4py/backend/dawn_backends.py:376:76: E711 comparison to None should be 'if cond is None:'


chore(master): release 1.1.58

:robot: I have created a release beep boop
1.1.58 (2023-09-04)
Miscellaneous

deps: update dependency phpstan/phpstan to v1.10.33 (d91d354)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-webhook-client/releases/tag/v1.1.58 :sunflower:


Align damage animation with newer KH games

Kingdom Hearts II, which the gauge is based on, uses a "fade out" effect for the damaged portion of the bar.  Newer games in the series instead have the damaged section "drain" out instead, which is overall more aesthetically pleasing.
The fade-out effect also has the issue that if the gauge receives damage during a fade, the partially faded-out portion will snap back to full brightness, which is jarring.  The drain effect isn't susceptible to that glitch at all.

One thing I'm trying to decide is whether the draining animation should be constant-speed or constant-time.  The former will look "smoother"  when individual damage events vary in magnitude, but has the disadvantage that large amounts of instantaneous damage--say, multiple bars' worth--will take a while to drain out, which may be distracting.

So I studied the behavior of the enemy HP gauge in KH 0.2 (the newest game in the series).  It seems to behave very much like the original KH2 gauge:

Damaged portions are converted from green to red instantaneously.
When a combo ends, the red portion fades out.
If another combo is started before the fade-out finishes, it gets canceled and the red portion disappears immediately. (kh2Bar differs here, as noted above)

The enemy HP gauge in Dream Drop Distance (both the 3DS and PS4 versions) behaves a bit differently:

The red portion drains out gradually after a combo ends.
If a new combo is started, the draining animation is paused and damage continues to accumulate.  It then resumes draining from the point it last paused at.

Overall I would say I prefer the DDD behavior.  The KH2 behavior ends up looking sloppy in a fast-paced battle since the fading animation keeps getting canceled.

I checked KH 0.2 again, it seems I was wrong.  It behaves pretty much identically to the DDD gauge.  That'll teach me to try to use a blurry YouTube video to make out fine details like health gauge animations. :o)
Anyway, the canonical behavior ultimately seems to be:

Damage during a combo accumulates.

When a combo ends, the damaged portion drains out.  This happens fairly quickly, but appears to be linear (constant speed).  Any reserve slots included in the damage fade out--only the visible bar is subject to the draining animation.

This was hard to ascertain because the bar drains out so quickly, but it appears that starting a new combo doesn't cancel the animation--it continues to drain up to the start of the current combo's damage.

Implementing the canonical behavior will require some refactoring to the rendering logic, but it shouldn't be too difficult.  Mainly it's just a matter of tracking draining animations individually for each combo.


Add react-tooltip to enhance icons hovering

Current dependencies on/for this PR:

main

PR #43

PR #45   👈

This comment was auto-generated by Graphite.


Add help link field in addition to help text on application form stream fields

In to this field it should be possible to enter an internal or external link and has it be rendered as a link on the form.
The link text should be "More information" and the link tag should open in a new window/tab.

Would this work for OTF?

Staff adds help links to each field that needs it.
They get rendered as shown below. The link text is always the same, as an example I opted for "See help guide for more information.".
When users click the link it's open in a new tab.

See guide for more information

I will include instructions in the introduction like "Further detail is available in the OTF Guidebook and in the Admissions Frequently Asked Questions page" or "Please click on the link 'Learn More' if you would like more information"
Other options instead of "See guide for more information":

"Guidebook"
"Learn More"
"Instructions to Applicants"


Outline not working on built-in scripts

Not sure if this is intentional, but as for now the outline is not working if you are editing on a built-in scripts (The one that isn't a stand alone .gd file)

Never tested it with built-in scripts. I will check it out.


Image to ascii

Added functionality to turn images and video into ASCII art.

All tests passing and looks good.


[GH-411] Dungeon supercampaign scene

Part of #411
Motivation
Connect the backend features that implement the dungeon supercampaign, its campaign and levels with our client.
Summary of changes

Dungeon supercampaign prefab
Dungeon campaign prefab and levels

How has this been tested?

Start the game, go to the Dungeon scene and click on the Battle button.
Select the dungeon supercampaign
Select the first level and start the battle.
Check that the battle finishes properly. If you win, the second level should be unlocked.

In order to win the dungeon levels and test all the available levels in the campaign, you may need to get some upgrades for your units. Follow these steps to do so:

Get your user: [user] = GameBackend.Repo.all(GameBackend.Users.User).
Give some pearls to your user, so you can afford the upgrades: GameBackend.Users.Currencies.add_currency_by_name_and_game(user.id, "Pearls", 2, 1000)
Get the available upgrades: GameBackend.Repo.all(GameBackend.Users.Upgrade)
Look for the "HP Upgrade 1" id in the previous response and buy it: Champions.Users.purchase_dungeon_upgrade(user.id, upgrade1_id)
Repeat for the "HP Upgrade 2": Champions.Users.purchase_dungeon_upgrade(user.id, upgrade2_id)

With this, you should be able to play and win the Dungeon levels.
Checklist

[X] I have tested the changes locally.
[X] I self-reviewed the changes on GitHub, line by line.
[ ] Tests have been added/updated.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] I have tested the changes in another devices.

[ ] Tested in iOS.
[ ] Tested in Android.

Here's some easier code for fetching the user without needing to know the name!
[user] = GameBackend.Repo.all(GameBackend.Users.User)
assuming you didn't crate any other users of course

Also I think it would be nice that if a level has a max number of units, it automatically deselects the leftover units, but I believe that is a backend change.

Also I think it would be nice that if a level has a max number of units, it automatically deselects the leftover units, but I believe that is a backend change.

Yes, we'll need a PR in the backend for that to be made properly.


[#132718533] CouchDB Handler - Add CouchDB Plugin

This handler should store cippus in CouchDB and returns Handler Status

COMPLETE : add CouchDB in kuss

Current coverage is 96.06% (diff: 100%)

Merging #20 into master will not change coverage

@@             master        #20   diff @@
==========================================
  Files             8          8
  Lines           178        178
  Methods           5          5
  Messages          0          0
  Branches         11         11
==========================================
  Hits            171        171
  Misses            7          7
  Partials          0          0

Powered by Codecov. Last update 66d731b...2f48d45


Final Project Report

Missing notebooks:

[ ] EDA
[ ] XGBoost

[ ] Fix paths in Jupyter notebooks

[ ] Fix latex


Link to my repo for Windows ?

https://github.com/majkinetor/postgrest-test
I linked here: https://github.com/majkinetor/postgrest-test/blob/master/README.md#notes

Sure! I've added your project on https://github.com/steve-chavez/postgrest-benchmark#notes.


Parse and distribute feature gates in devel script

This PR adds the feature gates flag to cainjector (currently empty).
All feature gates passed to the cert-manager install scripts (via FEATURE_GATES env var), are parsed and given to each component that supports each particular feature. This simplifies setting feature gates during an install for both CI and developers, as only the component that supports that feature gate will have it applied, and not cause the component to error during start.
/assign @munnerz
/kind cleanup
/milestone v1.8
NONE

/lgtm
/approve


undefined method `options_for_through_record'

After upgrade I got:
protected_attributes-1.1.1/lib/active_record/mass_assignment_security/associations.rb:96:in <class:HasManyThroughAssociation>': undefined method options_for_through_record' for class `ActiveRecord::Associations::HasManyThroughAssociation' (NameError)

Could you give more information? Which version of Rails are you using? What are you trying to do?

Thanks for the quick response. It's working ok now (updated to version 1.1.3)
I did a bundle update and the gem updated from 1.0.8 to 1.1.1:
gems/protected_attributes-1.0.8/lib/active_record/mass_assignment_security/associations.rb
gems/protected_attributes-1.1.1/lib/active_record/mass_assignment_security/associations.rb
Gems included by the bundle after upgrade are:

actionmailer (4.1.0)
actionpack (4.1.0)
actionview (4.1.0)
activeadmin (1.0.0.pre1 539fa4b)
activemodel (4.1.0)
activerecord (4.1.0)
activesupport (4.1.0)
arbre (1.0.3 239e811)
arel (<IP_ADDRESS>40414130214)
autoprefixer-rails (5.2.1)
bcrypt (3.1.10)
bootstrap-sass (3.3.5)
bootstrap_form (2.3.0)
bourbon (3.2.4)
builder (3.2.2)
bundler (1.10.5)
coffee-rails (4.0.1)
coffee-script (2.4.1)
coffee-script-source (<IP_ADDRESS>)
devise (3.5.1)
erubis (2.7.0)
execjs (2.5.2)
font-awesome-rails (<IP_ADDRESS>)
formtastic (3.1.3)
formtastic_i18n (0.4.1)
friendly_id (5.1.0)
has_scope (0.6.0)
hike (1.2.3)
i18n (0.7.0)
inherited_resources (1.6.0)
jbuilder (2.3.1)
jquery-rails (3.1.3)
jquery-turbolinks (2.1.0)
jquery-ui-rails (5.0.5)
json (1.8.3)
kaminari (0.16.3)
mail (2.5.4)
metamagic (3.1.7)
mime-types (1.25.1)
minitest (5.7.0)
multi_json (1.11.2)
mysql2 (0.3.18)
orm_adapter (0.5.0)
pivot_table (0.5.0)
polyamorous (1.2.0)
polyglot (0.3.5)
protected_attributes (1.1.1)
rack (1.5.5)
rack-test (0.6.3)
rails (4.1.0)
railties (4.1.0)
rake (10.4.2)
ransack (1.6.6)
rdoc (4.2.0)
responders (1.1.2)
sass (3.2.19)
sass-rails (4.0.5)
sdoc (0.4.1)
sprockets (2.11.0)
sprockets-rails (2.3.2)
sqlite3 (1.3.10)
thor (0.19.1)
thread_safe (0.3.5)
tilt (1.4.1)
treetop (1.4.15)
turbolinks (2.5.3)
tzinfo (1.2.2)
uglifier (2.7.1)
warden (1.2.3)


Fsoc 164

Description
Isolated tags are now supported in fsoc solution subscribe. This is similar to how fsoc solution push handles tagging
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[x] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report

Merging #147 (5640cc5) into main (113e56b) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #147   +/-   ##
=======================================
  Coverage   32.08%   32.08%
=======================================
  Files          27       27
  Lines        3597     3597
=======================================
  Hits         1154     1154
  Misses       2352     2352
  Partials       91       91


Mathematical operators perform in arcane order.

Calculation of '4 + 6 * 3 - 20' returns -98, NOT 2, as expected from BODMAS order ( 4 + (6*3) - 20 ). Order appears to be strictly right to left, will attempt to fix this. Not sure which stage this error occurs at.

Found the issue. Grammar featured various "prim_expr OP exp" statements designed to avoid left recursion. As it turns out, these were destroying operator precedence since Bison/Jison expects left recursive grammars. Whoops. Pushing change.


changed file to represent actual auto exposure ROI in python

fixed this code to not be duplicate of camera control file. Fixes issue I opened: #522.

Commenting for stale.


specification (v1): chunk sync markers

Problem:
Today if a chunk record is written corrupted such that a reader loses track of its place relative to records, and a correct EOF index is not written (because of crash, bugs, or configuration), a reader will not be able to parse any subsequent parts of the file, even if subsequent chunks are written uncorrupted.
Potential solution:
For each MCAP recording, generate a random sync marker of sufficient length that it is more-or-less guaranteed never to occur in file content. Put the sync marker in the file header and as the first field of every chunk. This means if your chunk is corrupted, but subsequent uncorrupted chunks are written, a reader may re-synchronize and still extract useful data.
Questions:
We could incorporate this at a regular byte interval as well or in addition, to protect un-chunked files.
Prior art:

Avro incorporates this feature, but documents it mainly in terms of splittability: https://avro.apache.org/docs/current/api/java/org/apache/avro/file/DataFileWriter.html. MCAP files today are splittable as long as the end of file index gets written - however various recording modes or bug conditions could preclude this. Avro files do not contain an index so this is their only mechanism for ready splittability, aside from parsing the file.
Discussion of this strategy as beneficial on deep space links: https://ipnpr.jpl.nasa.gov/progress_report/42-136/136H.pdf

Downside: same inputs will no longer produce the same output file hash. Work around this by allowing writers to specify a sync marker if they wish to override the random choice.


Add a Mockito Testcase

There are two OpenJ9 user issues where a crash with Mockito is reported:

https://github.com/eclipse-openj9/openj9/issues/18750
https://github.com/eclipse-openj9/openj9/issues/18781

The Mockito test case will help us identify the above crashes in our builds.
Test case variant 1: https://github.com/eclipse-openj9/openj9/files/13920987/reproApp.tar.gz
Test case variant 2: https://github.com/eclipse-openj9/openj9/files/13989201/segmentationerror.zip
Dependencies:

Mockito jar: https://repo1.maven.org/maven2/org/mockito/mockito-core
Bytebuddy
Maven

Not all dependencies might be needed; and test case variant 2 is simpler and relies on fewer dependencies.
Guidance based on the discussions with the test team:

The test should be added in https://github.com/adoptium/aqa-tests/tree/master/functional since it can be run with both OpenJ9 and OpenJDK.
We can create an folder MockitoTests.
We do not need testng in this case.
We need build.xml, playlist.xml, and the test case.
Sample example: https://github.com/eclipse-openj9/openj9/tree/master/test/functional/TestExample.
3rd party jars should to be added in adoptium/TKG/scripts/getDependencies.pl#L66-L176.
Licences of 3rd party jars should be verified before including them into TKG:

Mockito: https://github.com/mockito/mockito/blob/main/LICENSE
Byte-buddy: https://github.com/raphw/byte-buddy/blob/master/LICENSE

fyi @llxia

@sophiaxu0424 Please take a look. Thanks
The test should be added under https://github.com/adoptium/aqa-tests/tree/master/functional
The example can be found at https://github.com/adoptium/aqa-tests/tree/master/functional/OpenJcePlusTests
You will need to modify both aqa-tests repo and TKG repo.

mockito test case is attached in the above issue description.

@sophiaxu0424 any progress on this issue?

https://github.com/eclipse-openj9/openj9/issues/18781 is a dup of https://github.com/eclipse-openj9/openj9/issues/18750. The two test case variant are almost the same. The only difference is that one is directly invoke Mockito.mock(ArrayList.class); in main; and the other using junit to call Mockito.mock(Some.class);.
@babsingh is it necessary to add both cases?

No. Adding one should provide the required coverage.

Test case variant 1 is added. We can close this issue. Thanks @sophiaxu0424


Improved login page

Issue #40
Improvements made :

Aligned the button content with proper margins and padding.
Both the buttons are aligned in line with the title.
Added a simple border to the page.
Edited the title content to 'Login into Blogweet'

hi @KeyurGK

@AKD-01
Changes made as advised :
1)Removed the border.
2)Edited back to the original title content.

@AKD-01 Changes made as advised!

Have you checked the deployment?

It is showing something like this.


qnap arm base

can it work there?
i'm tryng but the docker desn't start
it give me this message
azure-sql-edge-arm64
exec /opt/mssql/bin/permissions_check.sh: exec format error

I'm afraid base image of mcr.microsoft.com/azure-sql-edge doesn't work on Qnap ARM either.
Following Microsoft's announcement, Azure SQL Edge based on ARM64 architecture is retired. See: https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827

Do you know where can i find an older version befire of the retire?
Inviato da Outlook per Androidhttps://aka.ms/AAb9ysg

From: Krzysztof Strugiński @.>
Sent: Wednesday, November 8, 2023 10:10:10 AM
To: cagrin/azure-sql-edge-arm64 @.>
Cc: Davide Sposato @.>; Author @.>
Subject: Re: [cagrin/azure-sql-edge-arm64] qnap arm base (Issue #3)
I'm afraid base image of mcr.microsoft.com/azure-sql-edge doesn't work on Qnap ARM either.
Following Microsoft's announcement, Azure SQL Edge based on ARM64 architecture is retired. See: https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827https://urldefense.com/v3/__https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827__;!!LFrK7YXB0Q!qGJO9ILGRxOlaw9tHcq-zN6v_eAI2PLORvCrO50zgH6PgtjYrbzcXBkOXyLDjju9KFn1IoKhhexnKIEnIP9yRnYD3g$
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/cagrin/azure-sql-edge-arm64/issues/3*issuecomment-1801375901__;Iw!!LFrK7YXB0Q!qGJO9ILGRxOlaw9tHcq-zN6v_eAI2PLORvCrO50zgH6PgtjYrbzcXBkOXyLDjju9KFn1IoKhhexnKIEnIP8bh9a7Zw$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/BDRF3YEYE3BYB7AKCTL3KV3YDND7FAVCNFSM6AAAAAA62VYP46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBRGM3TKOJQGE__;!!LFrK7YXB0Q!qGJO9ILGRxOlaw9tHcq-zN6v_eAI2PLORvCrO50zgH6PgtjYrbzcXBkOXyLDjju9KFn1IoKhhexnKIEnIP-w-bCzTQ$.
You are receiving this because you authored the thread.Message ID: @.***>

You can try some old tags: https://mcr.microsoft.com/v2/azure-sql-edge/tags/list

thanks, but i'm not so skilled, how should i use those tag for pull the image for container station ?

Da: Krzysztof Strugiski @.>
Inviato: mercoledì 8 novembre 2023 11:20
A: cagrin/azure-sql-edge-arm64 @.>
Cc: Davide Sposato @.>; Author @.>
Oggetto: Re: [cagrin/azure-sql-edge-arm64] qnap arm base (Issue #3)
You can try some old tags: https://mcr.microsoft.com/v2/azure-sql-edge/tags/listhttps://urldefense.com/v3/__https://mcr.microsoft.com/v2/azure-sql-edge/tags/list__;!!LFrK7YXB0Q!usxx5pZMXSX7Uhjai_Qtgf4hjcIguUZZLzv_z5QJdm3f4KGJEU7WEcu-E4zupg8n7upkf2TcQ75LMzIAAKGuFFZOzw$
―
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/cagrin/azure-sql-edge-arm64/issues/3*issuecomment-1801543990__;Iw!!LFrK7YXB0Q!usxx5pZMXSX7Uhjai_Qtgf4hjcIguUZZLzv_z5QJdm3f4KGJEU7WEcu-E4zupg8n7upkf2TcQ75LMzIAAKF7VcKUWw$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/BDRF3YD63I62JBW4TAEAQODYDNMHXAVCNFSM6AAAAAA62VYP46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBRGU2DGOJZGA__;!!LFrK7YXB0Q!usxx5pZMXSX7Uhjai_Qtgf4hjcIguUZZLzv_z5QJdm3f4KGJEU7WEcu-E4zupg8n7upkf2TcQ75LMzIAAKGfXYkdFg$.
You are receiving this because you authored the thread.Message ID: @.***>


Check exception when finalizing output parameters

If an error occurs or an exception is thrown during an execution, the values of any output parameters or columns are undefined. Therefore, do not depend on them having any specific values, because the ODBC driver may or may not have modified them.

Coverage increased (+0.3%) to 75.049% when pulling 78ba40f3a23bd3185bd87aa6b50d81283160ba6b on yitam:errOutParam into f45f25745f12c08594c031c5b49dbc5930334740 on microsoft:dev.


Wordpress: Error establishing a database connection

1.16 Running a WordPress Blog Using Two Linked Containers
7.1 Using Docker Compose to Create a WordPress Site
Both examples ends up with "Error establishing a database connection" when opening http://localhost
Test environment:

Ubuntu 14.0.3, 64bits
Docker 1.91
Docker Compose 1.51

mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)'
Check that mysqld is running and that the socket: '/var/run/mysqld/mysqld.sock'
exists!
Getting the following error


Adds notebooks for built-in vision algorithms.

@googlebot I signed it!

@jys-gg can you please address comments by @dizcology ?

@jys-gg PTAL

@jys-gg is this PR still relevant ?

@jys-gg I think makes more sense to migrate it to our new repo ai-platform-samples here: https://github.com/GoogleCloudPlatform/ai-platform-samples/tree/master/notebooks/samples

@dizcology no longer relevant. request to close this.


use dotnet core version can't login

when i use the last dotnet version, it's couldn't show login page, error message as below, also i try to use typescript2.2.1\2.1.5\2.1.4 and can not solve this error, do you have any ideas?
Uncaught TypeError: Cannot read property 'getValue' of undefined
at TMS_ASP.ScriptInitialization.TMS_ASP.ScriptInitialization (ScriptInitialization.ts:4)
at TMS_ASP (ScriptInitialization.ts:5)
at ScriptInitialization.ts:5
jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3846 jQuery.Deferred exception: TMS_ASP.Membership.LoginPanel is not a constructor TypeError: TMS_ASP.Membership.LoginPanel is not a constructor
at HTMLDocument. (http://localhost:59952/Account/Login/?ReturnUrl=%2F:140:5)
at mightThrow (http://localhost:59952/Scripts/jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3570:29)
at process (http://localhost:59952/Scripts/jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3638:12) undefined
jQuery.Deferred.exceptionHook @ jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3846
jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3855 Uncaught TypeError: TMS_ASP.Membership.LoginPanel is not a constructor
at HTMLDocument. (VM70 ?ReturnUrl=%2F:140)
at mightThrow (jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3570)
at process (jquery-3.1.1.js?v=RoNrvGA8lWW1zAYRAMy6yA:3638)

Just throwing it out there, try running sergen g. I had the same issue and that resolved it.

Same error here

See latest change in ScriptInitialization


All processes are terminated when last bash window closed

When the last Bash window is closed, all the linux processes are terminated, including daemons, like sshd, or ssh-agent. I think background processes should be kept alive across sessions.
Windows versions: Microsoft Windows [Version 10.0.14393]
Reproduction: launch ssh-agent, close all the bash windows, start a new one, look for the already started daemon.
OR: start sshd, ssh into the linux subsystem, close all bash windows, and your ssh session will terminate.

By design for now.  Once they open up the COM interface, you could probably program it to control an instance with something like a tray icon.

Thanks for reporting the issue. There's a few related posts on this so closing it out as a duplicate. Please see #834 and #1007 for more details.


Remove duplicate delete_by_query_rethrottle method definition

Description
Silences a Ruby warning when using this gem. Also see https://github.com/opensearch-project/opensearch-ruby/issues/230#issuecomment-2047853874
Issues Resolved
#230
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thank you!


Additional action on certain log levels?

I'm using logger console and logger mongo and both work perfectly.  I'd like to send an email with the log message on error and critical levels.  Rather than making a new logging interface, is there a way to execute a function when the log function is called, so that I can inspect the level and perform actions based on the level (like send an email in some cases).
thanks,
Cliff

@meecect one of the options I see right away — subscribe to the changes in MongoDB collection and trigger actions based on new entries.
Another one — implement .on() hook.
P.S. it's been 12 days, have you found a solution so far?

I ended up doing something kinda hacky:
log._error_orig = log.error;

log.error = function(message, data, userId) {
  var body;
  if (data) {
    if(typeof data == 'string') {
      body = data;
    } else {
      body = JSON.stringify(data,null, "  ");
    }
  } else {
    body = message;
  }
  Meteor.call(
    'sendEmail',
    message,
    body,
  );
  return log._error_orig(message, data, userId);
}

I'm not sure what .on() hook you are referring to.  Is there an .on() hook in this project?

@meecect I meant implement .on() hook is another option to your issue

Feel free to close it in case if the issue is solved on your end.


[PicalaBridge] Add new bridge

Picala is an independant media which communicates only about electrically assisted cycles in french.
This bridge retrieve articles from this site by category :

News
Economy
Tests
Usage

Hi and thanks for your comment.
I don't know where to add the description. Maybe the constant?

Picala is an independant media which communicates only about electrically assisted cycles in french.

I removed all comments in the bridge and fixed the content.
About the TYPES constant, I disagree. If it was only used in the PARAMETERS constant, I wouldn't have done this but it's also used in 2 array_search to build the name and description. array_search($this->getInput('type'), self::PARAMETERS[0][0]['type']['values']) is not readable.

Ah, I see about the types. Then it's okay.
As for the description, I meant the PR description. Just a few lines on what the page is/does and so on.
The content looks better, but I think you're grabbing the wrong image link for articles 2-4. The image you're pulling is some weird, super low pixel preview. You want the one from the srcset, not the src.
Example:
                         <img src="https://picala-static.s3.amazonaws.com/media/CACHE/images/article/2022-03-24/velos-a-paris/b0c4c3bf8c5b3dd54d34c6045ae826f9.jpg"
                              srcset="https://picala-static.s3.amazonaws.com/media/CACHE/images/article/2022-03-24/velos-a-paris/26923ce767b6138cc8d39cbf4b85eb63.jpg 320w"
                              data-src="https://picala-static.s3.amazonaws.com/media/CACHE/images/article/2022-03-24/velos-a-paris/26923ce767b6138cc8d39cbf4b85eb63.jpg"
                              data-srcset="https://picala-static.s3.amazonaws.com/media/CACHE/images/article/2022-03-24/velos-a-paris/26923ce767b6138cc8d39cbf4b85eb63.jpg 320w,
                              https://picala-static.s3.amazonaws.com/media/CACHE/images/article/2022-03-24/velos-a-paris/39d90d848e9260b5faad83c7c57ab5fd.jpg 640w"
                              sizes="(max-width:812px) 90vw, 250px"
                              alt="un vélo à paris sur le boulevard sébastopol"
                              class="lazy article__picture teaser__img"
                         >

You're pulling the src but the srcset is way better.

Now it's not pulling the image for the latest :) I think it's pulling the webp file, not the jpg.

I took data-src instead. With srcset, you must parse the last part of the string.
I updated the description here.

Oh the latest in feed but the first on the website. For some reason, some doesn't have data-src and other have multiples srcset because why not?
OK, time for parsing.

LGTM. Thanks


graph from readme

Hi,
Thank you for sharing this work.
Could you detail what exactly is plotted on the dynamic histogram in the main readme ?
Best

Sure thing, Jonathan.
In the main.py code, I ran 64 total trials, each one for 2,000 episodes.
The way I calculated evaluation returns is I would train the agent on each step through a single episode and then run an entire evaluation episode without any training steps using deterministic actions from the policy -> I would log the total reward of that evaluation episode. This is also made clear in the main.py file.
I now had a 64 x 2,000 matrix of data with dimensions (number of trials, number of episodes per trial). Taking the mean and standard deviation along the first dimension gave me two different 1 x 2,000 vectors that I used to create the episode vs. reward graph on the README.
In the dynamic histogram, what you're seeing at each time-step is a 64 x 1 slice in the second dimension of that matrix of data. So, for a single episode (Episode X in the title of the graph), what does a histogram of all 64 trials look like for their evaluation performance just on that specific episode? That's what's being plotted.
Hope this helps,
Momin

Hey,
Thanks for the comment ;)
Best


Allow custom transform smoothing logic in ObjectManipulator

Overview
This change creates a TransformSmoothingLogic base class that can be implemented by client code, and allows an instance of that class to be specified for ObjectManipulator.


[aws-sdk-cpp] Make aws-sdk-cpp rpath relocatable

This is important since vcpkg offers binary caching which could be
shared between different machines (as long as both machines use the same
"toolchains" [more technically, as long as the vcpkg hash is the same])
Describe the pull request

What does your PR fix? Fixes #14544

Which triplets are supported/not supported? Have you updated the CI baseline? No changes related to triplets support or to CI baseline.

Does your PR follow the maintainer guide? Yes.

@klalumiere, thanks for the PR!
Could you also update the Port-Version from 0 to 1 in CONTROL file?

I always forget about this :flushed: :laughing:. Done :heavy_check_mark:.

All features test passed with x64-windows.

Thanks @klalumiere 👍


Parse block test fails with uninformative message

Aim
The parse_all_failing tests in parse_block include a test case with a message where the expected and found tokens are both empty, defaulting to EOF.
By modifying the following, it's possible to see that it
fails with an opaque error. (i.e. the 'expected' and 'found' are both empty)
// src/parser/errors.rs
impl chumsky::Error<Token> for ParserError {
    fn expected_input_found<Iter>(span: Self::Span, expected: Iter, found: Option<Token>) -> Self

Expected Behavior
Return an error that explains what went wrong
Bug
Error is appx. expected token EOF while found EOF. Inspecting the expected/found tokens shows both are empty. Likely due to
To Reproduce
Run the mentioned test and modify expected_input_found to panic when both expected and found are empty.
Project Impact
Nice-to-have
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
No response
Installation Method
None
Nargo Version
No response
NoirJS Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Do you know what specific code triggers that error message? (that is, a full program that triggers that error message)

@asterite IIRC approximately:
let expected: Vec<_> = expected.iter().collect();
assert!(!(expected.empty() && found.is_none()));
let expected = expected.iter();

Ah, my link went out of date. These: https://github.com/noir-lang/noir/blob/bd861f282144056ecb52954fa9f6fd8db918e093/compiler/noirc_frontend/src/parser/parser.rs#L1438

Ah, I see. I'll try to reproduce it. The problem is that parsing a block in isolation never happens in a real program, it always happens in the context of something else and so the error mentioned above might not happen in a real program.

This can be closed as the old parser doesn't exist anymore :-)


Please, add a parameter to display the branch and release of the program

If we have different branches and releases of the program installed it would be very good that b2fuse.py reported what its branch and release is with something like "b2fuse --version" or similar. Right now, the only way I can devise is to check the name of the folder b2fuse is placed in, which is not optimal IMO.
I have almost finished a bash script that will conveniently allow the use of different branches and releases that could do a good use of this. This would allow people to use the latest stable release to safely store their important files in a bucket and use another bucket to test the latest released master branch or other alternative branches.

This new enhancement in the master branch reports: "B2Fuse version 1.3". But there is already another version that is also 1.3; this one: https://github.com/sondree/b2_fuse/releases/tag/v1.3 Both versions of the program are different despite being named "1.3", not approriate IMO. In fact, what we have in the master branch is the version 1.3 plus some enhancements and/or bug fixes, which I suppose it will become some day 1.4 or something. I would call it a developement version.
This distinction may be important for the actual use of the program. For real backups of important data one might want to use the 1.3 version in the aforementioned link since it has been more tested and its problems and limitations are (hopefully) known, and use the master branch for testing prposes, since its enhancements might have lead to not yet known bugs, being too risky for real backups.
Anyway, both are different versions and IMO should have a different version name. The master might be named "1,3 master" or just "master" or perhaps "developement".


Implement changes to accomodate addition of data request byte

Also removes the recursive functionality of the serial reader and replaces it with returning -1 for all values addressing #7.

@agupta231 At the moment, we just have the basic testing node serial_test.py. As far as I can tell using that testing node, everything is working as expected. With that said, yes having more in-depth tests would be a good idea and ROS has a structure in place to create tests. However, and I believe this was mentioned in a previous PR, doing testing on the com_serial node could prove to be a complex undertaking considering that the node deals with outgoing and incoming serial data. You couldn't unit test the joint angles output without having some sort of serial port snooper and you can't unit test the sensor data request without having a snooper and trusting that the Arduino runs properly and provides the correct response.
With that said, we should make an issue and trello card for unit testing this node to later come back to. For now, this PR and the arduino PR are meant as hotfix patches for an existing issue rather than a feature update.

Yo @mahajanrevant @agupta231 I just realized this PR hasn't been merged in yet. Can this get approved real quick so that it can be merged into master? It's critical to the master branch working.


(possible?) Windows compile issue

I get this error while trying to compile firrtl in sbt under Windows on two different machines. However, firrtl compiles fine under Linux, even using WSL on the same machine. I tried blowing away the entire firrtl repository as well as the entire .ivy2 cache and versions of firrtl in the .ivy2 local.
Anyone else have issues building firrtl under Windows?
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLBaseVisitor.java:13:14: FIRRTLBaseVisitor is already defined as object FIRRTLBaseVisitor
[error] public class FIRRTLBaseVisitor<T> extends AbstractParseTreeVisitor<T> implements FIRRTLVisitor<T> {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLLexer.java:16:14: FIRRTLLexer is already defined as object FIRRTLLexer
[error] public class FIRRTLLexer extends Lexer {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLParser.java:13:14: FIRRTLParser is already defined as object FIRRTLParser
[error] public class FIRRTLParser extends Parser {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLVisitor.java:12:18: FIRRTLVisitor is already defined as object FIRRTLVisitor
[error] public interface FIRRTLVisitor<T> extends ParseTreeVisitor<T> {
[error]                  ^
[error] four errors found
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLBaseVisitor.java:13:14: FIRRTLBaseVisitor is already defined as object FIRRTLBaseVisitor
[error] public class FIRRTLBaseVisitor<T> extends AbstractParseTreeVisitor<T> implements FIRRTLVisitor<T> {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLLexer.java:16:14: FIRRTLLexer is already defined as object FIRRTLLexer
[error] public class FIRRTLLexer extends Lexer {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLParser.java:13:14: FIRRTLParser is already defined as object FIRRTLParser
[error] public class FIRRTLParser extends Parser {
[error]              ^
[error] C:\GitHub\firrtl\target\scala-2.12\src_managed\main\firrtl\antlr\FIRRTLVisitor.java:12:18: FIRRTLVisitor is already defined as object FIRRTLVisitor
[error] public interface FIRRTLVisitor<T> extends ParseTreeVisitor<T> {
[error]                  ^
[info] No documentation generated with unsuccessful compiler run
[warn] two warnings found
[error] four errors found
[error] (Compile / compileIncremental) Compilation failed
[error] (Scalaunidoc / doc) Scaladoc generation failed
[error] Total time: 30 s, completed Oct 5, 2018 12:43:30 PM

Type of issue: bug report
If the current behavior is a bug, please provide the steps to reproduce the problem:

If possible, please directly provide the FIRRTL file or a relevant fragment if the file is large
What is the current behavior? It don't build
What is the expected behavior? It should build
Please tell us about your environment:
(examples)

version: master
OS: Windows 10

What is the use case for changing the behavior?
Impact: unknown
Development Phase: something is broken and I don't know what
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. Stack Overflow, gitter, etc)
See above for stack trace.

Are you using an IDE? We've previously had similar issues on MacOS using IntelliJ

No, this is with just sbt.

I can reproduce it on my machine. All it takes is a fresh clone followed by a sbt assembly.

I have the same problem on my Windows 10 machine with fresh installed Oracle JDK8 and SBT tools.

I have the same problem ,anyone know how to solve it

I haven't found a good solution, but here are some workarounds:

Use WSL, which compiles FIRRTL fine. It's still a Linux system in Windows, but with better integration compared to a virtual machine.
Use a published jarfile from Maven, by using managed dependencies. Note that we've recently started publishing snapshots, so this is viable if you're developing something that depends on FIRRTL (for example, if you only need FIRRTL as a dependency to run the latest Chisel), but not if you're developing FIRRTL.
You can also take a jarfile compiled in a non-Windows environment (like WSL) and drop it in the .ivy2 directory where publishLocal normally drops them.

I have the same problem on win7 and fedora29.  and get a temporary solution from a kind-hearted guy, and I also share it with you.
When the error comes up compiling firrtl, remove other files except FirrtlProtos.java in firrtl\target\streams\protobuf\protobufGenerate$global\streams\protobuf_2.12\out-cache, then re-compile, it will pass and you will notice the java files to be compiled are reduced from 9 to 5.
Hope anyone can find the root cause for this problem.

I have the same problem on win7 and fedora29. and get a temporary solution from a kind-hearted guy, and I also share it with you.
When the error comes up compiling firrtl, remove other files except FirrtlProtos.java in firrtl\target\streams\protobuf\protobufGenerate$global\streams\protobuf_2.12\out-cache, then re-compile, it will pass and you will notice the java files to be compiled are reduced from 9 to 5.
Hope anyone can find the root cause for this problem.

It may compile but it doesn't pass the tests.

sbt -mem 2048 ++2.11.12 publishLocal


Fix annotation dependency

Description
These plugins don't explicitly depend on androidx.annotation. In some cases, they relied on their transitive dependencies to include the dependency.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@tagesholly1 I'm curious why you reacted 👎
It looks like this might fix flutter/flutter#35239, I've added that to the description


Tweaks to cursorless csv machinery

[ ] Add comment at the top saying it's machine updated etc and add link
[ ] For each new entry we add, put a link in the comment above it

Probably won't bother because we'll migrate to talon lists soon anyway


Add an un-ignorable severity above "error".

Currently the analyzer can configure the severity of messages as "info", "warning" or "error".
It's also possible to ignore messages using // ignore: problem_name.
Would it make sense to have a severity above "error" which is an "error which cannot be ignored"?
This is hypothetical, I don't actually have a need for it myself (right now), so this is mainly to see if anyone is interested in the idea.
I could imagine teams where some (probably style-related) lints would be considered mandatory, and not allowed to be ignored. Or framework-related lints, which are really for your own protection.
Rather than having to check that in code-reviews, they could just set the severity to "fatal" (strawman name) in analysis_options.yaml, and then an // ignore: ... would not hide the error, and maybe the // ignore: ... itself would be an error (e.g., for // ignore_for_file: comments).
The existing workaround is to just educate your developers to not add such ignores, and to catch any mistakes in code-review.
But people like automation!

Love the idea of atomic un-ignorable rules, but what do you think about the ESLint approach to have a config option, which ignores all "disable-like" comments https://eslint.org/docs/latest/user-guide/configuring/rules#disabling-inline-comments?

Or it's too strict and less flexible?

I'm not sure whether it solves the problem, but analyzer supports a cannot-ignore key in the analysis options file whose value is a list of either the names of specific diagnostics or severities.

That works too. It's even more general, because it can cause hints and warnings that cannot be ignored, not just errors.
It does require more typing (name the lint once to enable it, once to add it to the cannot-ignore list), but this would mostly make sense for large setups where a lot of code shares the same analysis options.
So, accepted as solved.


Use parent DHT host

@riccardobl proposed this some time ago! This time I used the host from dht.io
Reference: https://github.com/riccardobl/hyperswarm-dht/commit/df9aa0d81bc88b08163ce263580e6603305ab44e

This doesn't necessarily depends on https://github.com/mafintosh/dht-rpc/pull/63 but they go together

@mafintosh Ready!

Reverse flow of this implemented in https://github.com/holepunchto/hyperswarm-dht/commit/ebef0db6024f5fba0be6e7c9c344624681e5ca06 for now


As a dyslexic user I want an app with easy to read font

so that I am able to use this app comfortable without being hindered by my disabilities

fixed by #137


Make sure we arent passing null to any converters

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #support.

Fixed, branch: zach/319/non-null-converters


Stack empty exception

Since version 1.0.0.x on Nuget (last known working version is <IP_ADDRESS> from Nuget) there is a stack empty exception for some of my RTF strings.
System.InvalidOperationException: Stack empty.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.Stack`1.Peek()
   at RtfPipe.Parser.<Tokens>d__16.MoveNext()
   at RtfPipe.Parser.Parse()
   at RtfPipe.Rtf.ToHtml(RtfSource source, XmlWriter writer, RtfHtmlSettings settings)
   at RtfPipe.Rtf.ToHtml(RtfSource source, RtfHtmlSettings settings)

Here is an example:
{\rtf1\ansi\ansicpg1252\deff0\deflang1033\fs20{\fonttbl{\f0\fswiss\fprq2\fcharset0 Arial;}
{\f99\froman\fcharset0\fprq2{\*\panose<PHONE_NUMBER>0405020304}Arial;}{\f100\fnil\fcharset2 Symbol;}{\f101\fnil\fcharset2 Wingdings;}{\f102\fcharset204{\*\fname Courier New;}Courier New CYR;}{\f103\fcharset0 Arial;}}
{\colortbl ;\red0\green0\blue0;\red51\green102\blue255;}
\paperw12240\paperh15840\margl1417\margr1134\margt1134\margb1134
\pard\sb100\sa100\sbauto1\saauto1\fs20\lang1033
ist Ersatzabruf für<PHONE_NUMBER> warAN 68595 bez 12.261,20- Re<PHONE_NUMBER>658060-ok !!\pard}}

The problem appears to be an extra closing brace at the end of the RTF document. This should be addressed in the latest release.


Fix break due to dotnet API change

Yeah. It's good. But can you make sure the CI passes?

Looks like CI timeout. I restarted the build. Code change is ok. You can merge whenever CI passed.
Thank you.

retest continuous-integration/travis-ci/pr please
// Previous failure: https://travis-ci.org/OmniSharp/omnisharp-roslyn/builds/111373187
// Retest reason:

@panopticoncentral I'll close this since I fixed the build break in my latest commit:
https://github.com/OmniSharp/omnisharp-roslyn/commit/d92d6b5e85cbdd67452bea6ef73e2d76efe32058
Thanks for looking into it.


Infra: Rename rill-developer repo to rill

We already publish packages as rill

@himadrisingh : Can you help take this one ?
Few things to verify -

Git history, issues remains intact
Able to redirect users from current git URL to new one
Release publishing works fine after deployment
Github actions for tests etc, do not break

Git history remains
Git takes care of the redirection
Will need few changes thr to make it work with new naming convention
GA should work, might require some minor changes
Github Apps should work as they don't depend on rill-developer repo

This is now completed


Resource 'MaterialDesignFloatingActionButton' is not found even if I added Resource

Hi,
I have weird issue with my project (VS 2015 v: 14.0.25123.00 Update 2). I have added one style.
<ResourceDictionary xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
                    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" >
    <ResourceDictionary.MergedDictionaries>

              <ResourceDictionary Source="pack://application:,,,/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.Light.xaml" />
        <ResourceDictionary Source="pack://application:,,,/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.Defaults.xaml" />
        <ResourceDictionary Source="pack://application:,,,/MaterialDesignColors;component/Themes/Recommended/Primary/MaterialDesignColor.Indigo.xaml" />
        <ResourceDictionary Source="pack://application:,,,/MaterialDesignColors;component/Themes/Recommended/Accent/MaterialDesignColor.Lime.xaml" />
    </ResourceDictionary.MergedDictionaries>

    <Style x:Key="MyStyle.Buttons.Circle.Red"
           BasedOn="{StaticResource MaterialDesignFloatingActionButton}"
           TargetType="Button">
        <Setter Property="Background" Value="{DynamicResource  RedPrimary900Brush}" />
        <Setter Property="BorderBrush" Value="{DynamicResource  RedPrimary900Brush}" />
    </Style>

</ResourceDictionary>

But Visual Studio always shows "Object. Resource MaterialDesignFloatingActionButton is not found" even if it'd defined in the Resource Dictionary.
Only when I've added directly "MaterialDesignTheme.Button.xaml" the warning disappear.
<ResourceDictionary Source="pack://application:,,,/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.Button.xaml" />

I've tried rebuild/clean/remove/add files but nothing helps.
What could be a problem ?
Thank for your advice,
Alexiej

Hi,
This is not a bug, and you are not doing anything wrong.  MaterialDesignFloatingActionButton is a style for PopupBox.  PopupBox has a different, default style.  So this is effectively a non-standard style, so in this case you have to include the extra resource dictionary.


Reorganized tests by module

Codecov Report

Merging #320 into master will increase coverage by 0.1%.
The diff coverage is 99.35%.

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


Could not find StbConfig when using find_package(Darknet)

Hey Guys,
when trying to use Darknet (aquired via vcpkg) in my cmake Project there is an error while generating the Makefile:
CMake Error at C:/Dev/vcpkg/scripts/buildsystems/vcpkg.cmake:233 (_find_package):
Could not find a package configuration file provided by "Stb" with any of
the following names:
StbConfig.cmake
stb-config.cmake
Add the installation prefix of "Stb" to CMAKE_PREFIX_PATH or set "Stb_DIR"
to a directory containing one of the above files.  If "Stb" provides a
separate development package or SDK, be sure it has been installed.
Stb[core] was installed correctly with the darknet library.
I think the problem here is that the stb package doesnt deliver a stb-config.cmake in the share folder.
Any suggestions?

I prepared a FindStb.cmake script for darknet, but maybe i left a bug so that it is not correctly propagated for downstream projects. Let me check and I will let you know asap.
Sorry... since I am using darknet mostly outside vcpkg (unfortunately, but that will change soon) I still had no time to use it fully from here and do some well-deserve dogfooding

@mschmidbauer the latest PR for darknet #6047 should fix your issue. Please let me know if you encounter any problem

it is now merged in master. I hope that just updating fixes your problems. If not, again, please let me know

Thank you cenit for the fix, @mschmidbauer, please reopen it if it still be an issue for you.


[#267] Parse trial statuses into "status" and "recruitment_status"

Depends on https://github.com/opentrials/api/pull/71
This gives us more granularity to know the status of a trial. The mapping of the statuses comes from https://github.com/opentrials/opentrials/issues/267#issuecomment-239802828.
opentrials/opentrials#267

In writers (record, trial) we use explicit approach so status also should be added there.
Also do you think manage this mappings as one merged in helpers is a bad idea?

In writers (record, trial) we use explicit approach so status also should be added there.
Also do you think manage this mappings as one merged in helpers is a bad idea?

@roll You mean moving this logic to processors/base/writers instead of it being in each separate processor? If so, I don't think it's a good move, as it breaks encapsulation. The identifiers might make sense, as it's the same for any processor (right?).


Is this the correct way to handle nested arrays/objects?

Looking into groqd, wanted to test it first on a navbar. The syntax looks a bit messy with the way I've done it, curious if this was the way to go about it.
const navbarSelection = {
        mainNavigationGroups: q.array(q.object({
            navItem: q.object({
                text: q.string(),
                navigationItemUrl: q.object({
                    externalUrl: q.string()
                })
            }),
            subNavigationItems: q.array(q.object({
                    text: q.string()
                }
            ))
        }))
    }

const query = q("*", {isArray: true})
        .filter("_type == 'navbar'")
        .grab$({
            _id: q.string(),
            ...navbarSelection
        });

There aren't a lot of in-depth examples in the docs, so I figured I'd ask here, thanks!

Yeah I think a common misconception is how q.object and q.array work. They are basic zod types and won't actually affect how the query is built. Instead you should use/chain the .grab and .filter functions respectively.
I put together an example here if you want to take a look.
https://github.com/FormidableLabs/groqd/issues/88#issuecomment-1463021786
Hope this helps!

This is a fantastic question. What you have works, but might be over-fetching data (which might be fine based on your scenario).
By using the q.array/q.object approach, you're doing a somewhat coarse query and then letting Zod filter out the stuff you want once the data is fetched. Like @JohnGemstone mentioned, using .grab and .filter will allow you to be more explicit about what you're fetching.
Here's a quick example. Let's say you have pokemon with some type information on them. You could do something like this:
q("*").filterByType("pokemon").grab({
  types: q.array(q.object({ name: q.string() }))
})

This results in a query of *[_type == "pokemon"]{types}, and if the types array entities end up with a lot of information besides name, you'll be getting all of that data from your network request – and then it'll get filtered out.
You could modify that to something like this:
q("*").filterByType("pokemon").grab({
  types: q("types").filter().grab({ name: q.string() })
})

which results in a query like *[_type == "pokemon"]{"types": types[]{name}} which only fetches the name property.
Both will result in an output of the same shape – there's just a difference in granularity of the groq query/fetch.


Provisioning Tab and Start and stop provisioning settings in the new UI in Enterprise applications UI has been changed.

Provisioning Tab and Start and stop provisioning settings in the new UI in Enterprise applications UI has been changed.It doesn't match with the video uploaded in the document.I dont see the option of mappings and synchronize Azure AD users.

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

ID: 77fa544c-e028-c8da-a86c-0dfa357852fc
Version Independent ID: 574a6ca7-fb2a-19c3-8cf0-8541b5164dec
Content: What is automated SaaS app user provisioning in Azure Active Directory
Content Source: articles/active-directory/app-provisioning/user-provisioning.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@priyankareddy20
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.