Develop

fix #79
update packages

Thanks @Huuums !
One question. I updated the plugin from VS Code and it wiped out all content for .fttemplates folder. Is it normal behaviour? Not sure if it's related to the plugin or VS Code settings.

Hey @green-anger ,
I'm sorry that happened. This is the first time I've heard of this happening.
My .fttemplates folder is still there and untouched after the update. So I am close to 100% sure that this has nothing to do with the extension update.

No problem at all, my mistake of not making any backup copy. I'm still experimenting with the setup and learning the plugin, so no big deal.
Thanks again for the quick update!


Is there a way where we can call API from another API and pass access token and authorize it in Identity Server

Hi,
Is there a way where we can call API from another API and pass access token and authenticate the same  in Identity Server

http://docs.identityserver.io/en/release/topics/extension_grants.html


Failed Test PR #15 - Error running algorand endpoints

Summary and Screenshots
Getting this error when trying to execute the curl tests for algorand (network status, adding key, balances, poll, etc.)

Steps To Reproduce

Clone gateway and checkout on this branch
On the gateway directory run yarn and yarn build afterwards
Run yarn start
On a separate terminal, export needed variables for curl test
Run curl test for Algorand

Test Result Comparison
N/A, new chain
Attachments
N/A

This looks to be cause by not copying the new algorand.yml file from the src/template folder into the config folder


 Or since it renders the animation first then change the url route later which creates a smooth experience. Currently trying out react router native stack for animation. But the rendering is a little bit delayed.
I also tested that when without animation. There's still a tiny delay for switch. Hopefully, helping out! Thanks


UI should implement search on _id using an id

UI implements search by _id as an identifier search using system/value:
http://yellow.testlab.nhs.uk/careconnect-ri/resource?serverId=home&pretty=true&resource=Condition__
_id is one of the common parameters and should be an id field. See: http://hl7.org/fhir/stu3/search.html#all

Can you explain a little more? Is this just a UI issue.

It looks like a UI issue only.

Incidentally, 'Use the following code snippet to execute this action in your own client.' doesn't work for my client :)


Getting gradle issue

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugAssets'.

Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
Could not resolve com.mapbox.navigation:android:2.4.1.
Required by:
project :app > project :flutter_mapbox_nav
Could not resolve com.mapbox.navigation:android:2.4.1.
Could not get resource 'https://api.mapbox.com/downloads/v2/releases/maven/com/mapbox/navigation/android/2.4.1/android-2.4.1.pom'.
Could not GET 'https://api.mapbox.com/downloads/v2/releases/maven/com/mapbox/navigation/android/2.4.1/android-2.4.1.pom'. Received status code 401 from server: Unauthorized

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1m 1s
Running Gradle task 'assembleDebug'...
[!] Gradle threw an error while downloading artifacts from the network. Retrying to download...

how to fix this


chore: bump zint to latest and remove patch

fix #10

Haven't figure out how to use the new mem file yet 😅

I might have figured out how it works. The code128 example is working now

For some reason rust code is still buggy

Letting you know, I'm working on adding xtask to build the project. This will replace build.sh with a bunch of rust code that handles installing build requirements (WASI SDK, wasi-stub, wasm-opt) and packaging.
Noting this because it changes /zint-wasm-sys/build.rs but only the env::set_var("CC", ...) & AR lines. There should be minimal conflicts with this.

New wasm size:
-rwxr-xr-x 1 mgt mgt 850K Oct 17 23:42 typst-package/zint_typst_plugin.wasm*

I just checked the new zint.h. Besides the memory file option, there is not new constants and macros


Made a market, it appears successfully, but still pending above

Does this still happen for you on http://development.augur.divshot.io/?  I made a few different market combinations and the "pending" pane always seems to get cleared ok for me.

No it seems to be fixed after latest update


iPhone 5C iOS 8.1 issue

Hello,
I just updated estimote SDK from 1.3.0 to 2.2.1 to be able to communicate with beacons in iOS 8. Followed your articles to set up location update and I noticed funny behavior.
I can monitore in background/foregroud, but when I range I always get empty list of beacons. My UUID is of course changed, not the original estimote UUID. Why is that?

There was a bug in how the region parameter was being passed into the didEnterRegion delegate. Since you were using that "buggy" region to start ranging, this could've been the reason why ranging was returning an empty list of beacons. This has since been fixed—try the newest version of the SDK.


Code Quality: Renamed DirectoryPropertiesViewModel to StatusBarViewModel

Summary

Reanmed the view model
Formatted codebase around it

PR Checklist

[ ] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
[ ] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?
None

Screenshots
None

I found two issues

the folder item count isn't displayed
the git actions are missing


node16/nodenext support for fastify plugin

What happens?
When I import fastifyApiReference, the type is any :
Unsafe argument of type any assigned to a parameter of type FastifyPluginCallback<{ routePrefix: string; }, RawServerDefault, FastifyTypeProvider, FastifyBaseLogger>

tsconfig.json with node16 :
{
  "compilerOptions": {
    "outDir": "dist",
    "declaration": false

    "module": "node16",
    "moduleResolution": "node16",
    "moduleDetection": "force",
    "target": "es2022",

    "resolveJsonModule": false,
    "declaration": true,
    "sourceMap": true,
    "stripInternal": true,
    "strict": true,
    "noImplicitReturns": true,
    "noImplicitOverride": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noFallthroughCasesInSwitch": true,
    "noUncheckedIndexedAccess": true,
    "noPropertyAccessFromIndexSignature": true,
    "noEmitOnError": true,
    "forceConsistentCasingInFileNames": true,
    "skipLibCheck": true
  },
  "include": [
    "src"
  ]
}

I came across this issue on the fastify repo https://github.com/fastify/fastify/issues/4349, with an explanation on how to fix it in javascript plugins, but I do not know how to fix it here in this typescript plugin 🤔
I would need some help 🙏
What did you expect to happen?
I was expecting the type of fastifyApiReference to be defined
OpenAPI Document
No response

Node 16 isn’t officially supported anymore, so we don’t support it either. Any chance you could update your target to Node18? 🤔
Source: https://nodejs.org/en/about/previous-releases

I do use Node 20, I meant when I set node16  in the module and moduleResolution property of my tsconfig.json 😅
I think I will create a PR to update fastify to the new major version  (v5.0.0) and I will try again.


Export libpq functions

Codecov Report

Merging #2 (bc9df8c) into master (fb52a5a) will increase coverage by 0.82%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   74.54%   75.36%   +0.82%
==========================================
  Files           5        5
  Lines         330      341      +11
==========================================
+ Hits          246      257      +11
  Misses         84       84

Flag
Coverage Δ

unittests
75.36% <100.00%> (+0.82%)
:arrow_up:

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

Impacted Files
Coverage Δ

postgres.lua
94.44% <100.00%> (+2.44%)
:arrow_up:

Continue to review full report at Codecov.

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


No mention of Imports via SSO through an Enterprise Data Gateway

No mention of Imports via SSO through an Enterprise Data Gateway. Is that coming? Soon? Currently imports only work via Personal Data Gateways using windows credentials. Keep in mind we only allow SSO, so direct DB passwords won't work for us.

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

ID: fa7ca8d8-0f83-2cd6-57b1-04fc8e04981b
Version Independent ID: 196ee3c3-aa88-f8ca-c725-33bf77be0db5
Content: Use SAML for single sign-on (SSO) to on-premises data sources - Power BI
Content Source: powerbi-docs/service-gateway-sso-saml.md
Service: powerbi
Sub-service: powerbi-gateways
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Thanks for your question @pcgeekus. Adding @arthiriyer to comment.

@pcgeekus Currently SSO is available for DQ. We may consider it for the upcoming semester starting October- please upvote at ideas.PowerBI.com so ew can prioritize.

Thanks @arthiriyer, closing this out.

I think product development is already working on it, please check and let
me know
On Wed, Jun 26, 2019 at 10:27 AM Arthi Ramasubramanian Iyer <
<EMAIL_ADDRESS>wrote:

@pcgeekus https://github.com/pcgeekus Currently SSO is available for
DQ. We may consider it for the upcoming semester starting October- please
upvote at ideas.PowerBI.com so ew can prioritize.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/powerbi-docs/issues/1156?email_source=notifications&email_token=ALG3YRLX3TVYWP67OGHT72DP4ODFRA5CNFSM4HZNT7T2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYT46QA#issuecomment-505925440,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALG3YRPU5EG6UXN6B6OKHALP4ODFRANCNFSM4HZNT7TQ
.


Generate Entity Diagram - Move_excludedEntities to config

In the power apps sample to generate a Entity Diagram from the Dataverse metadata, can I suggested that the _excludedEntities,
static string[] _excludedEntities = new string[] {
"activityparty" ,  "annotation" ,  "appointment" ,   "asyncoperation" , "attributemap" ,
"bulkdeletefailure" ,  "bulkimport" ,  "bulkoperationlog" ,   "businessunit" ,  "businessunitmap" ,
"commitment" ,  "connection" ,
"displaystringmap" ,  "documentindex" ,  "duplicaterecord"  ,
"email" ,   "entitymap" ,
"fax" ,
"imagedescriptor" ,  "importconfig" ,  "integrationstatus" ,   "internaladdress" ,
"letter" ,
"msdyn_wallsavedquery" ,  "msdyn_wallsavedqueryusersettings" ,
"owner" ,
"partnerapplication" ,  "phonecall" ,   "plugintype" ,  "postfollow" ,  "postregarding" ,   "postrole" ,
"principalobjectattributeaccess" ,  "privilegeobjecttypecodes" ,   "processsession" ,  "processstage" ,
"recurringappointmentmaster" ,    "roletemplate" ,  "roletemplateprivileges" ,
"serviceappointment" ,  "sharepointdocumentlocation" , "statusmap" ,  "stringmap" ,  "stringmapbit" ,
"subscription" ,  "systemuser" ,
"task" ,   "tracelog" ,  "traceregarding" ,
"userentityinstancedata"  };
and _excludedRelationships entries
static string[] _excludedRelations = new string[] { "owningteam", "organizationid" };
in DiagramBuilder are instead added to app.config and read from config rather than in the code. It would make generating the diagrams easier.

@edfenergy-richardcannock How does this make it easier? Are you modifying the list of excluded entities/relationships, and so moving those lists to the config file would not require you to recompile?

@phecke  - yes, that's exactly it. I am modifying the list of excluded entities and relationships to provide a more succinct view of the key business domain entities that we primarily use from our perspective. Having to recompile is a bit cumbersome, particularly for those users who don't have access to VS or the source code, but who may wish to generate the ERD. Many thanks.

@edfenergy-richardcannock  We give you the source code so that you can understand how the program works and can modify it as needed. If you want to make the code changes and submit a PR, I will review it and consider merging the PR.
The upcoming change for the program is to update the code from using the CrmServiceClient to ServiceClient class when establishing the web service connection. This will remove the reliance on the ADAL.NET library which has been deprecated. We keep getting pings from customers about this.
I will consider your request as an enhancement. However, considering our small team and workload, it would be some time before we could ever get around to making such code changes.
Thanks for your feedback.


Zebra view

Hi,
Thanks for this great library.
Is it possible you to add zebra view feature, so that odd/even rows or columns have different background colors?
I managed to do it by checking cells' positions and setting their background colors. At first it seemed ok but if I sort the rows  everything becomes messed up because row's background does not get updated by their new order.
Thanks again.

hi again,
i did it as you mentioned on issue  #69 but still same. Could it be that i set the background on wrong place? i do it in the onBindCellViewHolder.

Hi @joegillian,
You need to update each of the background value of cell item.
Assume that you have an tableView like this.
1.cellA : blue
2.cellB : white
3.cellC : blue
after sorting process, each cell item shows its own background color.
1.cellC: blue
2.cellA: blue
3.cellB: white
So, what is the solution? You just need to update each of cell item after the sorting process.
just update your cell model data with right color and run the notifyDataSetChanged function of the TableViewAdapter

Thanks, I did it as you say and I works perfect.

Great!
I think, the easiest way setting background color at binding process.
@Override
     public void onBindCellViewHolder(AbstractViewHolder holder, Object cellItemModel, int
             columnPosition, int rowPosition) {
          Cell cell = (Cell) cellItemModel;

          // Determine the right color considering to the row position;
          int bgColor = rowPosition % 2 == 0 ? myFirstColor : mySecondColor;

          // Set the background color to model
           cell.setBackgroundColor(bgColor);

           // Get the holder to update cell item value
           MyCellViewHolder viewHolder = (MyCellViewHolder) holder;
           viewHolder.setCell(cell);
}

With this way, It will set the right color every time, not just for once. However, after sorting process, you need to call notifyDataSetChanged as well. Because, the cell items whose positions are unchanged after the sorting process, won't be notified.

I tried both this solution and the one at 69, it didn't work, also I have to change bg color of cell like this
app:unselected_color="@color/grayBlack", it doesn't work when I change it in cell layout. What to do?


Elif without preceding If causes Syntax error when Elif has a complex condition

I'm getting a syntax error when using a function call that creates a signal inside an Elif.
I can understand why this kind of function call might be problematic in a Python DSL, however I'm raising a bug because this code looks like it ought to be valid. If it's breaking a rule, I'm not sure what the rule is.
from amaranth import *
from amaranth.back import verilog

class Stepper(Elaboratable):
  def __init__(self):
    self.output = Signal(3)
    self.up = Signal()
    self.down = Signal()
    self.ports = [self.output, self.up, self.output]

  def edge_detect(self, m, input):
    last = Signal()
    m.d.sync += last.eq(input)
    detected = input & ~last
    return detected

  def elaborate(self, platform):
    m = Module()
    with m.If(self.edge_detect(m, self.up)):
      m.d.sync += self.output.eq(self.output+1)
    with m.Elif(self.edge_detect(m, self.down)):
      m.d.sync += self.output.eq(self.output-1)
    return m

s = Stepper()
verilog.convert(s, name='Top', ports=s.ports)

fails with:
[/usr/local/lib/python3.7/dist-packages/amaranth/back/verilog.py](https://localhost:8080/#) in convert(elaboratable, name, platform, ports, emit_src, strip_internal_attrs, **kwargs)
     48         warnings.warn("Implicit port determination is deprecated, specify ports explictly",
     49                       DeprecationWarning, stacklevel=2)
---> 50     fragment = ir.Fragment.get(elaboratable, platform).prepare(ports=ports, **kwargs)
     51     verilog_text, name_map = convert_fragment(fragment, name, emit_src=emit_src)
     52     return verilog_text

[/usr/local/lib/python3.7/dist-packages/amaranth/hdl/ir.py](https://localhost:8080/#) in get(obj, platform)
     35                 code = obj.elaborate.__code__
     36                 obj._MustUse__used = True
---> 37                 new_obj = obj.elaborate(platform)
     38             elif hasattr(obj, "elaborate"):
     39                 warnings.warn(

[<ipython-input-20-74600c1d41a4>](https://localhost:8080/#) in elaborate(self, platform)
     19     with m.If(self.edge_detect(m, self.up)):
     20       m.d.sync += self.output.eq(self.output+1)
---> 21     with m.Elif(self.edge_detect(m, self.down)):
     22       m.d.sync += self.output.eq(self.output-1)
     23     return m

[/usr/lib/python3.7/contextlib.py](https://localhost:8080/#) in __enter__(self)
    110         del self.argElif s, self.kwds, self.func
    111         try:
--> 112             return next(self.gen)
    113         except StopIteration:
    114             raise RuntimeError("generator didn't yield") from None

[/usr/local/lib/python3.7/dist-packages/amaranth/hdl/dsl.py](https://localhost:8080/#) in Elif(self, cond)
    251         if_data = self._get_ctrl("If")
    252         if if_data is None or if_data["depth"] != self.domain._depth:
--> 253             raise SyntaxError("Elif without preceding If")
    254         try:
    255             _outer_case, self._statements = self._statements, []

SyntaxError: Elif without preceding If

However, when the condition is calculated outside the Elif, it works
from amaranth import *
from amaranth.back import verilog

class Stepper(Elaboratable):
  def __init__(self):
    self.output = Signal(3)
    self.up = Signal()
    self.down = Signal()
    self.ports = [self.output, self.up, self.output]

  def edge_detect(self, m, input):
    last = Signal()
    m.d.sync += last.eq(input)
    detected = input & ~last
    return detected

  def elaborate(self, platform):
    m = Module()
    is_down = self.edge_detect(m, self.down)
    with m.If(self.edge_detect(m, self.up)):
      m.d.sync += self.output.eq(self.output+1)
    with m.Elif(is_down):
      m.d.sync += self.output.eq(self.output-1)
    return m

s = Stepper()
verilog.convert(s, name='Top', ports=s.ports)

with m.If(self.edge_detect(m, self.up)):

The use of m both in a calling function and in a callee in this way is not supported. This is true even if you are not using self.edge_detect(m, ...) inside of a conditional expression (see below), but when used in a conditional expression it just breaks.
The reason it's not supported is that the following code (still using your edge_detect function):
    is_down = self.edge_detect(m, self.down)
    with m.If(...):
      m.d.sync += self.output.eq(is_down)

has very different semantics from the code below:
    with m.If(...):
      is_down = self.edge_detect(m, self.down)
      m.d.sync += self.output.eq(is_down)

which does not match how Python functions, variables, and scoping normally work, and is very likely to be a source of obscure bugs.

That said, the error it's currently raising is nonsensical, and the rule I'm talking about should be a warning (or an error), rather than something that is silently accepted.
I also have some ideas on letting people use functions without the scoping hazard I described, though that would need an RFC.

Thanks for taking the time to write such a complete response. I learned something new.

The use of m both in a calling function and in a callee in this way is not supported.

I think I see the problem I created: by calling edge_detect() inside the Elif condition, perhaps the inverse condition of the If applies to it. If that is so, then it the code snippet created by edge_detect() has an extra enable input, and last created by edge_detect() is only updated when the If condition is false. In short, that code is ambiguous and hard to understand, and I ought to write it differently.
I tend to lean into using Python as a macro language for Amaranth, which is convenient, but m gets passed around a lot. Some guidance (as an error message or extra documentation) around when functions that use m can be called and when not would be very helpful.

Some guidance (as an error message or extra documentation) around when functions that use m can be called and when not would be very helpful.

I agree. Let's keep ths open, since at the very least the error you're hitting is wrong.


use RPC handle marshalling when running on win8.1+

this was compile time check before and we would use RPC handle marshalling when target platform was win8.1+. Now it will be used if running on 8.1+ regardless of target platform.

I was hoping the system_handle attribute would be ignored on older versions of Windows. Unfortunately it appears that not only are they not ignored, but they cause compile errors. I need to find a new solution.


A more detailed description of the privacy rules of accepted apps

For example, it is clear for an f-droid user that any app downloaded from their repo contains no trackers or propriatry libraries and dependancies.
What is Accrescent's policy in that respect? Do we have one? I couldn't find one in the app description.
What can I as an end user be guaranteed regarding the privacy of the apps in your repos?
While I have no reason to suspect any wrong-doing, I believe detailed information would not hurt.

I agree this needs to be more explicitly documented. To answer your immediate questions, Accrescent has no additional restrictions regarding proprietary code. Analytics/tracking are allowed "within reason and with reasonable consent." Since Accrescent is currently in alpha and only a select few developers can upload, there isn't yet a clear definition on what this entails.


opendkim howto ?

I mounted the /etc/opendkim folder on the host to ensure persistence of the DKIM signature, I set the corresponding public key in the DNS using content from /etc/opendkim/keys/<my domain>/mail.txt. But while testing it I noticed the mail weren't signed.
In fact, I noticed this in my logs:
opendkim: /etc/opendkim.conf: refile:/etc/opendkim/TrustedHosts: dkimf_db_open(): No such file or directory
It coming from:
/etc/init.d/opendkim start (L292 of start-mailserver.sh)
Is there anything specific to do to enable DKIM ?
EDIT: BTW, I'm not familiar with DKIM but shouldn't there be an /etc/opendkim/TrustedHosts file ? It seems to be missing.

Ok, I got it to work by manually adding the /etc/opendkim/TrustedHosts with the correct content. It was easier than I thought.
I noticed it's not part of the opendkim initialization process of the script I guess it should be. I will make a fix and a PR soon.

@auguster sorry for the late reply... anyway...
I've that same setup.
I've copied the content from a first run of the container and made it persistent, that way all the needed file are ok :)
I've added a commit to the v2 branch that handles host based opendkim configurations.
If interested I can merge it on master, take a look and let me know :)

@00angus  Don't worry, I was supposed to make a PR soon anyway ^^
I think the file should be generated by the start script if missing. It should just contain <IP_ADDRESS> and localhost anyway. Manually copying files feels manual and error proned if not well documented (or if the documentation gets ignored).
I still hope on making a quick fix soon if my schedule allows.

Here is the patch take a look if you want.
The only manual part is the initial creation of the config that should be made persistent. It's sufficient to add a /usr/local/bin/ script and call it like we call the script for the generation of ssl certs.

From what I read that script doesn't create /etc/opendkim/TrustedHosts if the file is missing.
By the way, this script doesn't add a new DKIM for a new domain while /etc/opendkim/TrustedHosts exists. So I would say the current version of the script is more powerfull in term of flexibility. It's just missing a check for /etc/opendkim/TrustedHosts's existence.

Yep, and this has been fixed in v2.
https://github.com/tomav/docker-mailserver/blob/v2/Dockerfile#L39

Nop, that doesn't change a thing. If I mount an empty folder from the host to /etc/opendkim/, to have it populated with persistent DKIM stuff. Then the TrustedHosts file is not there and won't be recreated by start-mailserver.sh. That's what this issue is all about.

Maybe I'm getting confused :)
Let me clarify:

I want to generate - one time - opendkim data/config ? Then I cannot mount a host volume on /etc/opendkim to generate that data with that image (v2 or master in this case is the same).
By mounting an empty host volume into /etc/opendkim you are overwriting the image current content, and that includes the TrustedHosts file. That file will be missing. Currently you have to manual copy that data out and that manual thing is what we want to remove (I think you would agree on that - at least I hope :) );
I want to use an already generated data/config for opendkim (like I already have and - I hope - you already have): In that case you can safely use v2 branch because of the patch already merged, that would copy the data from /tmp/docker-mailserver/opendkim (if I remember correctly)

I know that the current setup for opendkim could be reworked as initially was NOT intended for importing external data/config. My current patch on v2 is my temporary workaround but a more complete and integrated solution would be needed.
We could - as an example - generate all the needed config/data during start-mailserver.sh startup (included the TrustedHosts file) and then at each startup if a config is already available do not generate anything else ... or something like that. That way our host volume could safely be mounted on /etc/opendkim and everything should work.
What do you think about it?

No your not not getting confused, that's exactly the behavior I'm looking for.

@tomav what do you think about it? Could you add a "feature" label to the issue so to not forget about it?

Just added enhancement and help wanted if someone can get time to PR something.

I think I'm gonna close this issue and open another one to get a generator (script) to get files for opendkim.
We do the same for accounts management in v2, so it makes sense.
start-mailserver.sh is just a start script, and I think that config generation should be out of it.
Let me know what you think about it.

That's a good idea, it makes perfect sense.


NEW README.MD WOOOOOOOO

Gonna say all of it:
Fixed grammar, fixed credits, and added some kewl image's from gab engine (From my POV).

Added BBPanzu's credit, please do so gaberial!!!!
Revamped the features while giving an explanation for some stuff.
Add's comments on how to do it.
Please accept this pr, it took me like 3 hours to make

yeah


Error 5002 has a go link

Input
Navigate to https://sorbet.org/docs/error-reference#5002
Observed output
There's a link to http://go/types/rbi
Expected behavior
I'm guessing that should link to https://sorbet.org/docs/rbi

Apologies if this is the wrong place for this sort of feedback! Feel free to redirect me if this is the incorrect place.

Stripe uses different RBI generation tooling, which is why the doc prefaces that link with "If you are at Stripe"
Thanks for the report but this is written as expected.


Explicitly programmatically fail test

Is your feature request related to a problem or a nice-to-have?? Please describe.
For testing asynchronous network processing code, I wrote a small test framework. The code to be tested uses a so-called Handle interface. On that interface, there are .write(data) and .read() -> Data methods. During tests, I use a mock Handle which contains some assertions. For example, when the code to be tested calls .read() when we don't expect a read, an assertion is triggered.
If this assertion bubbles up, then the Mocha test fails, as intended.
However, if the code being tested has a try-catch block around this call, then the assertion error is inadvertently caught and the test does not fail.
To solve this, I'd like to have a mechanism to explicitly fail a test programmatically.
Describe the solution you'd like
Ideally there is a method on the test's Context. For example:
it('tests something', async function() {
    this.fail('The test failed');
});

I could then pass a reference to this.fail to the mocked handle, so that the test actually fails, no matter how deep down in the stack the assertion is triggered.
Describe alternatives you've considered
At first I thought I could use the done parameter:
it('tests something', async function(done: Mocha.Done) {
    done(new Error('The test failed'));
});

This does fail the test. However, when I add the done parameter to an async test function, then I always must call done() at the end or the test fails. Without the parameter, that is not required (the async test function simply results in a promise, which is awaited by Mocha).
Additional context
n/a

@dbrgn why not simply return a rejected Promise, which is propagated up the call stack? Without a donecallback.
I don't understand your scenario where you catch an error, but actually you intend the opposite.

I could then pass a reference to this.fail to the mocked handle, so that the test actually fails, no matter how deep down in the stack the assertion is triggered.

I doubt this would work. The call stack isn't cleared when you fail a function on a higher level in the stack. All the pending async tasks/Promises will hang around and could lead to weird results on later tests.

The problem is that I'm mocking/injecting a certain component in a more complex system, so the failure happens deep down in the stack and there is no failed promise that I could return easily.
What I'm doing right now is setting a flag on that injected component, saying that "the assertion failed". In every test, I check for that flag at the end of the test, and explicitly fail the test if the flag is set. However, it would be nice if there were an option in mocha to fail the test even if the async test function itself resolved without an error.
(The workaround does work, so it would be fine to close this feature request. However, it would allow for more ergonomic project-specific test frameworks.)

Ah, and an example where propagating an exception (i.e. a failed rejection) through the stack is where errors are catched inside the code under test with a try-catch block and logged (and ignored). The code under test cannot differentiate between errors and test failures. In this case the error never reaches the top level.

(The workaround does work, so it would be fine to close this feature request. However, it would allow for more ergonomic project-specific test frameworks.)

👍
I think the fact that this hasn't been asked for a lot (only one upvote in a year) and doesn't exist in other popular frameworks that I can find (e.g. https://github.com/jestjs/jest/issues/11780#issuecomment-1056386158) is a good indication that it's out of scope for now. Closing out as wontfix.
But if someone has a strong desire to see something like this.fail happen, please do post back with your reasoning. We'd be happy to take another look. Cheers! 🤎


SCC: Allow a single sc broker connect to multiple scc-broker worker processes

https://github.com/SocketCluster/scc-broker-client/issues/4

Done since SC v11.3.1.


Properly handle SynchronizatonContext

:)

Thank you! I'll release as beta until this new functionality is covered by tests.


feat(tooltip): add formatter cache to tooltip

move PR 12504 to next branch

I'm not sure if we need to put this cache logic inside echarts. The caching strategy heavily depends on the business logic on the top. And it's not so complex to be implemented.

I'm not sure if we need to put this cache logic inside echarts. The caching strategy(for example how to compose the key) heavily depends on the business logic on the top. And it's not so complex to be implemented.

Maybe you are right.


when consumer close and create frequently, delay message will be duplicated

Fixes #14722
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #14722
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

Can we add a units test to cover this case?

see https://github.com/apache/pulsar/pull/14740


Fix #1 and add support for ISBNs starting with 979

I updated the lib/index.js file to correctly handle ISBNs starting with 979.
I also added a npm script to easily get new groups from the ISBN International Agency website. It can be run with:
$ npm run update-groups

The script will fetch an XML file from the server and update the lib/groups.js file accordingly.

Thanks :+1: I'll release a new version


Fix some problems

Fix the broken build in windows because of wrong reference to winapi
Fix the problem that device_query could be panicked with following message when trying to get new display because Drop trait is not defined for DeviceState
Maximum number of clients reached...

Thanks for the PR! Looks good to me.


Improved pointer support on iOS 13.4

Since iOS 13.4 has improved cursor support, perhaps we should rethink some of the interactions if the user has a mouse or trackpad connected. For example, hiding the cursor and disabling some of the gestures (as they are now handled by the hardware) might be a good idea.

Is there any API to detect if a mouse is connected?

I don't think it's dropped yet. I wonder when the next Xcode beta will be out…

I know that it's supposed to snap to buttons, so theoretically somebody could make a program to detect the Windows cursor status, and report it to the app, and the app places a control there. Of course, we could just wait for an API...

FYI #176 will track work supporting cursor inside the VM. This issue is for better handling for trackpad to navigate UTM.

I implemented cursor handling in the VM list view, VM metal view toolbar and (for #176) the actual cursor movement. :)


Question about the performance on SIDD dataset

Hi, thanks so much for sharing the source code first! I tested the provided pre-trained model on SIDD validation dataset and only get  34.43/0.851 (PSNR/SSIM) on average. I am wondering if my versions of python and pytorch match yours. I am using python 3.9 and pytorch 1.10.0

Hi, I have checked the pre-trained model (DnCNN-LUDVAE-SIDD.pth), and the result is

I am using python 3.8.3 and pytorch 1.6.0
Can you try again and use the pre-trained model DnCNN-LUDVAE-SIDD.pth from https://drive.google.com/drive/folders/1XKIt5WBPh0Fc3AFEcWxZphfs_N_5YnYw
and the code from
https://github.com/zhengdharia/LUD-VAE/tree/main/LUD_VAE_sidd/DnCNN

Thanks for the reply and kind reminder. It seems that I misused the test code. The result now is the same as yours.


[Question] ModuleNotFoundError: No module named 'kashgari.embeddings.bert_embedding_v2'

Getting below error Please advise...
Using >>kashgari-2.0.0a1 and TF > 2.2.0
Following this tutorial>>>>https://kashgari.readthedocs.io/en/v1.1.1/embeddings/bert-embedding_v2.html

There is some API diff between 1.x and 2.x. Please read the version-specific docs from https://kashgari.readthedocs.io/en/latest/.
You can change the version by clinking the latest button.


CSS of the mifes are missing on the deployment

Hello, Jose.
First and foremost, I would like to thank you for creating this plugin. You saved me from migrating to webpack :D
And now, onto the issue:
Root app is deployed on example.com and mife on example.com:4101.
importMap.json of the mife:
{
  "imports": {
    "@mife/navigation": "http://example.com:4101/spa.js",
  }
}

vite.config.ts:
plugins: [
  react(),
  viteTsconfigPaths(),
  vitePluginSingleSpa({
    type: 'mife',
    serverPort: 4101,
    projectId: 'navigation',
  }),
],

The problem is that the CSS of the mife is being requested from the deployment of the root application instead of the deployment of the mife itself.
In my case in browser devtools I can see that app trying to get mife's CSS from the http://example.com/assets/vpss(navigation)spa-240be869.css instead of http://example.com:4101/assets/vpss(navigation)spa-240be869.css where it actually is.
Also, in version 0.1.0 of the plugin, there was an option called deployedBase, which allowed to solve this issue by specifying in my case:
vitePluginSingleSpa({
  type: 'mife',
  serverPort: 4101,
  deployedBase: 'http://example.com:4101',
 }),

Hello!

You saved me from migrating to webpack

I was in a much worse position:  Create React App (with webpack) and wanted out badly!
You are on the right track:  The base.  Remember that Vite provides the base property.  Not me.  Just set it.  I recommend doing it like this:
export default function ({ command }) {
    return defineConfig({
        plugins: [
            react(),
            viteTsconfigPaths(),
            vitePluginSingleSpa({
                type: 'mife',
                serverPort: 4101,
                projectId: 'navigation',
            }),
        ],
        base: command === 'build' ? 'http://example.com:4101' : undefined
    });
};

I used to have deployedBase because I was using it for two things.  When I discovered server.origin, then I figured my need was reduced drastically, so I dropped it.
In retrospective, seeing that I am forced to make the export as a function and only set the base when building, I might reconsider reviving deployedBase, as my plug-in would only apply it as the name says:  When deploying (a. k. a., building).
On the other hand, I just know that I cannot free every developer out there from having to use the function version of Vite config, so my brain tells me that I shouldn't overextend.
I'll leave the issue open for now.  Feel free to close it if this solves your problem.

Solved.
Thanks a lot, for the plugin and for the help!


issue with push() after webpack minification

We are developing an app with a logout button that logs a user out and takes them back to a login page. This is the function we are using:
export function logoutCurrentUser() {
  return dispatch => {
    setTimeout(() => {
      dispatch(resetState())
      dispatch(push(LOGIN_URL))
    }, 0)
  }
}

This works fine in development, but after the js is minified with webpack the login page is not rendered. The user is logged out but a blank screen is shown. Has anyone encountered issues with push() after minification? Any help would be greatly appreciated.

What kind of mangling are you doing? What is your minifier?

We're using webpack -p.

Do you have any specific UglifyJS options you're using?

@timdorr We are using the HtmlwebpackPlugin and jquery with the ProvidePlugin for webpack. Otherwise just default minification settings.

Is the URL changing when you log out? It sounds like something is getting mangled that shouldn't be.

Small suggestion but try add:
/*global LOGIN_URL:true*/

At the top of the file. I assume you are using the define plugin? It creates a global that you need to notify the minifier of. If you can post this Webpack config it might help debug the issue.

Ultimately, this is a tooling problem, not something with this library.

@lvela did you fix  this problem in the end...I'm encountering a similar issue

We found a work around but I can't remember the details. Maybe @antondomratchev remembers?

@ns6482 Our solution was to use push in browserHistory from react-router instead of react-router-redux. My guess is something wasnt being minified properly.
import { browserHistory } from 'react-router'
browserHistory.push(LOGIN_URL)


Adds test about DCE decoding

Redmine ticket: https://redmine.openinfosecfoundation.org/issues/3637

Merged in #1321, thanks!


Generated files are not being moved when submission is posted

Describe the issue
As noted in #8010 and #8015, the generated files for newly posted XML submissions are not being reliably moved into place. The result is that only the directly uploaded XML file is available.
Current theory is that this is due to a race between posting the submitted draft and the generated files appearing on the shared filesystem where they're stored. If the draft is posted before they are visible, the routine that does the copying will ignore them. There's some poking to do to confirm this as the cause, but if so, we need to ensure that such a race is accounted for.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Lowering from critical to major because we appear to have mitigated the issue by disabling negative lookup caching on the nfs mounts.


Prompt: does this power pole also serve as a street lamp? and vice-versa

Many power poles also serve as street lamp poles.
Or one could say many street lamps are mounted on power poles.
Therefore, in iD, no matter if the user types "Street Lamp" or "Power
pole", there should be a box at the end
[ ] This pole also serves as a street lamp, or
[ ] This pole also serves as a power pole.
Which upon clicking/adding a check mark,
would open up appropriate additional entry fields.
And in the editor, there needs to be a new icon, for such combined poles.

Vast majority of power poles are not used as street lamps, the same for street lamps - vast majority are not a power poles

FWIW, see the ongoing Utility poles proposal on the OSM wiki.

Vast majority of power poles are not used as street lamps, the same for street lamps - vast majority are not a power poles

Where I live 100% of street lamps are mounted on power poles.
Why (would the utility company) waste a whole new pole when there is a ready made one nearby?
Yes if the power lines are underground then separate light poles would be needed.
Where I live only telephone poles never have street lamps mounted on them,
as the government rules say the systems must be separate (so they don't fail at the same time due to a pole falling down, etc.)

Hmm this is a tricky situation because it's one structure that's functionally two different things. OSM and iD aren't great about features tagged with multiple types. Closing this since the tagging isn't really settled yet and the iD repo isn't the place to discuss it.


How to use it with Storyboard?

it really confused me that I don't know how to create a BFPaperButton using storyboard

Hi @terminatorlkf !
Thank you for using this control :]
Below is a quick guide to getting started using BFPaperButton with Storyboards:
1) Drag and drop a UIButton onto a the Storyboard and change it's class to BFPaperButton:

2) Change the button type to Custom (optional, but recommended):

3.a) Import
<UIColor+BFPaperColors/UIColor+BFPaperColors.h>
and "BFPaperButton.h"   // or <BFPaperButton/BFPaperButton.h>.
3.b) Drag and drop an IBOutlet into your ViewController for the button.
3.c) In viewDidLoad (or viewWillAppear, etc.) customize your BFPaperButton!

Result:

Let me know if you have anymore questions. Thanks!

thanks！I sincerely appreciate your help
发自我的 iPhone

在 2016年3月23日，上午8:54，Bence Feher<EMAIL_ADDRESS>写道：
Hi @terminatorlkf !
Thank you for using this control :]
Below is a quick guide to getting started using BFPaperButton with Storyboards:

Drag and drop a UIButton onto a the Storyboard and change it's class to BFPaperButton:

Change the button type to Custom (optional, but recommended):

3.a) Import
<UIColor+BFPaperColors/UIColor+BFPaperColors.h>
and "BFPaperButton.h" // or <BFPaperButton/BFPaperButton.h>.
3.b) Drag and drop an IBOutlet into your ViewController for the button.
3.c) In viewDidLoad (or viewWillAppear, etc.) customize your BFPaperButton!
Result:
Let me know if you have anymore questions. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


chore: use sofa-node instead of sofa-node-demo

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Affected core subsystem(s)

Description of change

1.0.0


Remove nested folders

Remove two levels of nested folder that served no purpose. From cmd/plugin/cli/version.go → cmd/version.go.
Update the linting workflow and config slightly:

Removed step that installed dependencies that no longer are needed for any of the following steps of that action
Update the golangci-lint to the latest version due to newer issues that this picks up vs the current version in use (these were mostly due to newer version of gofmt)

I tried utilising golangci-lint within superlinter but the version it uses is even older
Already updated the code issues picked up by the new gofmt

Updated the github actions in use (mostly from V2 to V3 with the exception of golangci-lint which already had V3.2)
Superlinter now only lints the files that were updated with the push action that triggered the run
Superlinter uses the slim image which is 2gb smaller further reducing the time the action takes to run
Added more comments directing to docs of the linters and why certain steps are in place

The newer gofmt will format a few comments where we have unusual identation (trying to list a file system directory, etc) and that is expected with this new version (see release notes for GO1.19). To work around this for now, I have surrounded these comments with /* */ as to preserve the formatting.
This is a temporary fix and we should, at some point, address these types of comments to follow newer go version guidelines.

I ran into the same issue here and never got around to sorting it out #208

This was a rather annoying finding. I don't have an answer as to why this is failing for us yet but I plan to have a look at some point in the near future 👍
If you do find anything before that then pls let me know.

Rebasing...


feat: useAzureMapFeature tests

Can you add a screen of current coverage of our tests?

After merge latest master it look like:

Let's add tests for AzureMapFeature.tsx and we can merge it


Make tests pass with latest master of CoffeeScript

Tested with jashkenas/coffeescript commit d7e752b.
They also still pass with latest stable CoffeeScript (1.10.0).

Run npm test as usual to verify that it works with CoffeeScript 0.10.0.
Install latest master of CoffeeScript and run npm test.

NOTE: The most common change in this PR is adding checks for the new 'PROPERTY' around code that looks for the 'IDENTIFIER' token. There are more occurances of 'IDENTIFIER' in the code that I have NOT touched. I simply worked until all tests passed. I'm not sure if these cases need modifying or not. If they do, the tests need to be updated to cover it. I'd like some help from a more experienced coffeelint developer here.

Hey thanks for doing this. Looks like the tests didn't pass because of the length limit, but that seems trivial to fix.
When did this property token get added? Would like to take a look at the CoffeeScript to see what changed. It looks like you got the tests passing regardless.

https://github.com/jashkenas/coffeescript/commit/e42fc477478889bbdeb182f3dc7a2d9e9ea2803b
https://github.com/jashkenas/coffeescript/commit/021d2e437630a60ef4ea53556c2f64cecb762b8c
https://github.com/jashkenas/coffeescript/commit/585932cf5bda03d16cd772beb67c1f76f39fc09a
https://github.com/jashkenas/coffeescript/commit/4d8cd032987f49b84484d194305ef7f71f3434cc
https://github.com/jashkenas/coffeescript/commit/9a0babf5b1fe7a7e1b8d47653b84c6a7919dc5cf

npm test failed for me even before I started working on this, because of lint errors, but the test suite passed. I don't think I've introduced any new lint errors.

Merged into a5babd23aa094132c0bdf97d47e639650aec72a5

Thanks! :+1:


 Am using a custom prompt, will try again with a clean one.
Expected behavior
The installation to run and not error out.
Environment

OS: MacOS Catalina 10.15.3
OS version: 10.15.3
gopass Version: Latest, installation didn't finish.
Installation method: go get github.com/gopasspw/gopass

I have had the same issue. Looks like gopass does not support Go 1.14 yet.

This git repository haven't been updated for decade there are many bugs. I make a pull request here to fix the problem but i don't think they will push or fix it soon: #1242 . For the temporary solution checkout my answer at this issue: #1248

Should be fixed by #1273.


Cancel clearEditing on link, at least temporarily

Why? Which issue is this related to?

This PR is canceled. I linked the new pull request. But in a nutshell, the linking completely broke and needed an urgent fix.


test(report): Add tests for report components

Current coverage is 47.17% (diff: 100%)

Merging #168 into dev will increase coverage by 10.88%

@@                dev       #168   diff @@
==========================================
  Files           142        142
  Lines          2747       2747
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            997       1296   +299
+ Misses         1750       1451   -299
  Partials          0          0

Powered by Codecov. Last update a7901fe...9da6ff0


LibreSSL download is broken?

The error appears when trying to build mavsdk_server with the following command:
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_BACKEND=ON -Bbuild/default -H
It looks like either the download server is down

Was just temporary, closing.


Add notebooks to documentation

closes #100
This PR adds notebooks, and recipes to doc. It also reorganizes the doc. Preview the documentation here.

All notebooks work except one, see inline suggestion.
The collapsed recipes at https://springtime--105.org.readthedocs.build/en/105/recipes/ look nice.
Why are we rendering the datasets notebooks, but downloading the example use case notebooks? Why not render them both? As you can add download link to rendered notebooks with https://github.com/danielfrg/mkdocs-jupyter#download-notebook-link

Download links are added, see an example.

It would be nice to add some explanation about the various datasets and their sources.
@Peter9192 each notebook has a link to the data source. See an example. Are more explanations about datasets needed?


Will there be a mac version/port

Hey I am wondering if there will be a mac port of this plugin.
I have found this plugin however, it appears to me that its only a keyboard and it's hard for me to install.

Closed because duplicate of #89.
Personally I don't know how to use MacOS, or have any previous experience with it, so it's best if someone knows how to port it :(
I believe you can build this from scratch just like Linux, but I need someone using Mac to test this out.


GCAM-USA to CERF: Fuel prices by technology, vintage, and state xml query

GCAM-USA to CERF: Fuel prices by technology, vintage, and state xml query
GCAM-USA version: 5.3 (IM3 version)
Python script: get_fuel_prices.py
Data Description: Fuel Prices by Technology
Meta-Issue: https://github.com/IMMM-SFA/gcam-usa_to_downstream-models/issues/1

This looks correct as far as I can tell. The fuels costs all seem to be at the appropriate level.
Note that the biomass prices will include carbon subsidies. Any consumer of that would have to pay for emissions, should they occur, in the RCP 4.5. From that perspective, do we need to query for CO2 prices? (I notice you already have queries for carbon contents and CCS capture rates).

I will have to think about how best to incorporate this in the downstream models.

The "CO2 prices" query as below should work just fine.  The way the policy is structured in IM3 no matter which / how many regions you query there will only be results for a single "global" market.  It will give results for "globalCO2", "globalCO2_Constraint", and "CO2_LUC".  You can filter for just "globalCO2".
<marketQuery title="CO2 prices">
    <axis1 name="market">market</axis1>
    <axis2 name="Year">market</axis2>
    <xPath buildList="true" dataName="price" group="false" sumAll="false">Marketplace/market[true() and contains(@name,'CO2')]/price/node()</xPath>
    <comments/>
</marketQuery>


Fixed responsiveness of Home/Cart page

Issue reference:#53
Proposed changes:Fixed responsiveness
Type of change:CSS changes
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

At 500px the the image is sticking to the right:

Fix this!

Both done do check!!

@Prikshit-anthal
Works nicely 👍
Thanks for your contribution 🚀


Add dedicated build machine

@mversic

The suggestion here is to have a machine (e.g. ec2 instance) where our team member would be able to SSH into and build the code remotely, or run tests. The reason is that build (especially with release profile) takes a long time, and consumes a lot of disk space, on a local machine. This is especially true when building whole of Iroha with cargo test. Main premise here is that the build on a dedicated machine would perform significantly faster compared to it being done locally. I also think that user separation (in linux space) on the machine would be preferable, but we can make a quasi separation with one folder assigned to each team member

@Cre-eD

@Cre-eD Should we use the kubernetes cluster for this?

What's the resource requirement?


Enhancement: Integrate a spinner to track time taken for long running operations.

What happened:
Multiple logs with the same message is printed on screen which is redundant by nature, a spinner can be used to achieve the objective.
I0927 03:24:02.531498  835123 import.go:258] Importing image rhel-94-04242024-tier0. Please wait...
I0927 03:24:06.596504  835123 import.go:277] Image import is in-progress, current state: queued
I0927 03:26:06.680951  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:28:06.623011  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:30:06.843204  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:32:06.727223  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:34:06.769074  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:36:06.920080  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:38:06.861958  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:40:06.951969  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:42:06.826810  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:44:06.873676  835123 import.go:277] Image import is in-progress, current state: running
I0927 03:46:06.901292  835123 import.go:271] Image imported successfully, took 22m0.670579377s

Rather a spinner can be set-up lo track the current status and the time elapsed to improve user experience.

What you expected to happen:
A single line of log to track the status.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

pvsadm version (use pvsadm --version):
Environment - On-Prem or IBM PowerVS:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Others:

/remove-kind bug
/kind feature
/good-first-issue

cc: @priyanshikhetwani

/assign

Fixed via #680.
/close


expired pokemon not removed from map?

Clicking on a pokemon shows it as being expired, but doesn't remove it form the map.
Refreshing removes expired pokemon from the map, but the auto-refresh from the web.py doesnt seem to be auto-refreshing at all

Yeah, good amount of interactivity (like auto refresh) is broken on the frontend right now. I'm intending on fixing it soon.

There is a typo in web.py:
@app.route('/config') def config():     """ Gets the settings for the Google Maps via REST"""     center = {         'lat': DOUBLE_LAT,         'lng': DOUBLE_LON,         'zoom': 15,         'identifier': "fullmap"     }     return json.dumps(center)
Should be original_lat and original_lon instead of DOUBLE_LAT and DOUBLE_LON respectively.
After it was fixed autorefresh and old pokemons removal worked fine.


Selected nodes break after updating filetree.

After FiletreeService.updateFiletree is called, previously selected nodes cannot be unselected. This is likely caused by accessing nodes outside of user interaction with the filetree.

getDirContents() was producing duplicate nodes with different object IDs. This method now edits the node children array in-place, fixing this issue.


Bug Fix:show/hide Statusbar issue

Issue: https://github.com/chrizuuu/react-native-gallery-preview/issues/11
Fix: Now It’s translucent and customizable with backgroundColor

Hi, your code is fine as far as android is concerned but on ios I need a slightly different approach.
I'm closing your PR but thanks a lot for the contribution :)


receiving '503' / resource not found from maven

Howdy,
I am seeing a '503' / resource not found from maven install from edu.cmu.lti.oaqa.
Would it be possible for the developers to bundle up the github for the open source question answering github repo to which I believe the oaqa in the pom.xml is pointing?
Thanks!
Scott Malec, CMU '10

Sorry for the late response. Actually that legacy repository is not used. I have just removed that repo from the pom and updated the README file. Please check it out.


The exec dispatcher fails to respect rule.

Hyprland Version
 System/Version info
Hyprland, built from branch  at commit 7587cadd0a4e94c9ba42c00307854052798cf9b2  ().
Date: 2024-03-18
Tag:

flags: (if any)

System Information:
System name: Linux
Node name: burque
Release: 6.8.1
Version: #1-NixOS SMP PREEMPT_DYNAMIC Fri Mar 15 18:19:29 UTC 2024

GPU information:
05:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Picasso/Raven 2 [Radeon Vega Series / Radeon Vega Mobile Series] [1002:15d8] (rev c4) (prog-if 00 [VGA controller])

os-release: ANSI_COLOR="1;34"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.05.20240316.c75037b"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
IMAGE_ID=""
IMAGE_VERSION=""
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.05 (Uakari)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.05 (Uakari)"
VERSION_CODENAME=uakari
VERSION_ID="24.05"

plugins:

Bug or Regression?
Bug
Description
The bind only works on an empty workspace.
How to reproduce
Add this bind to your config file.
bind=$mainMod, K, exec, [float;center] kitty
Open with mod K on an empty workspace and kitty behaves as expected
Open with mod k with a tiled window present and kitty is forced down to the bottom 10% of the screen.
Crash reports, logs, images, videos
No response

working on my end, try updating.

Indeed after an update it works beautifully. Thank you kindly.


[SPARK-45096][INFRA] Optimize apt-get install in Dockerfile

What changes were proposed in this pull request?
follow the Best practices for writing Dockerfiles :

Always combine RUN apt-get update with apt-get install in the same RUN statement.

Why are the changes needed?
1, to address https://github.com/apache/spark/pull/42253#discussion_r1280479837
2, when I attempted to change the apt-get install in https://github.com/apache/spark/pull/41918, the behavior was confusing. By following the best practices, further changes should work immediately.
Does this PR introduce any user-facing change?
NO, dev-only
How was this patch tested?
CI
Was this patch authored or co-authored using generative AI tooling?
NO

cc @Yikun @dongjoon-hyun

CI link: https://github.com/zhengruifeng/spark/actions/runs/6093404505/job/16533661403

LGTM, but I think you should know (or maybe already know):

Always combine RUN apt-get update with apt-get install in the same RUN statement.

This best practices is for to help reduce redundat layer between update and install, but for spark infra, there are a tradeoff between image size download cost and image rebuild cost.
Move frequently modified dependencies to front means: once the Dockerfile head front dependencies are modified, the following content will also be rebuild (due to infra cache invalidated). Finally, it impacts PRs which change Dockerfile head lines or haven't rebase in time CI will cost more time (about 1h).
TL;DR: move stable dependencies to top, keep frequently modified dependencies to end.

LGTM, but I think you should know (or maybe already know):

Always combine RUN apt-get update with apt-get install in the same RUN statement.

This best practices is for to help reduce redundat layer between update and install, but for spark infra, there are a tradeoff between image size download cost and image rebuild cost.
Move frequently modified dependencies to front means: once the Dockerfile head front dependencies are modified, the following content will also be rebuild (due to infra cache invalidated). Finally, it impacts PRs which change Dockerfile head lines or haven't rebase in time CI will cost more time (about 1h).
TL;DR: move stable dependencies to top, keep frequently modified dependencies to end.

thanks for the detailed explaination, since we rarely modify the apt-install (we frequently changes the pip-install), so I think it would be fine.

thanks @Yikun and @HyukjinKwon , merged to master


Examples of using this and/or generated summaries?

I'd love to give this repo a try, but there are no instructions or examples of generated summaries.

Hi @Hellisotherpeople Thanks for showing interest in my work, I am updating the working towards updating my GitHub repositories. I will complete it by soon. Please spare me some time. I will publish the readme with instructions soon.
Thanks
Arunav


DRILL-6540: Upgrade to HADOOP-3.x libraries

Set in order changes from this PR: https://github.com/apache/drill/pull/1732
Updated Hadoop and HBase libraries to the latest versions:
Hadoop: 3.2.1
HBase: 2.2.2
Jira - DRILL-6540.

By @vvysotskyi:
Here is the list of things to be done before merging this PR:

[ ] Cleanup poms:

[ ] Remove commented out lines, revise new dependencies to reduce JDBC driver size

[ ] Revert removing excluding commons-logging, it will be better to add an absent mock class from commons-logging to Drill

[ ] Remove change with logging test error in ITTestShadedJar.java
[ ] Test Drill on Hadoop 3 cluster (secure, non-secure modes)
[ ] Check with Drill-on-YARN
[ ] Test Drill in embedded mode on Windows (test Hadoop win-utils)
[ ] Test JDBC client with new Hadoop version
[ ] Check whether new Hadoop version API is compatible with the current one, in the case if not, decide whether to introduce new profile, use a property to specify the version during building or document supported versions

As it appeared, the new Hadoop library version required updating of hadoop-winutils, though there is  only an artifact for the version 2.7.0 in maven repository. So I created the needed artifact using jitpack.io.

@agozhiy, since winutils binaries are hosted in your repository, could you please add README.md file with the link to the repository and steps on how to publish new versions. As an example, may be used Calcite.md.

There are a lot of unit tests failures with these changes, most of the errors are the following:
java.lang.NoSuchMethodError: com.google.common.base.Preconditions.checkState(ZLjava/lang/String;I)V

It looks like some tests which don't patch guava loads Preconditions class and then it cannot be patched in other classes where it is required. All unit tests classes should be revised and patching should be added where it was absent, so I propose to fix it in the scope of another Jira, but before merging this PR.


Exception handler able to catch getter and setter exceptions

Exception handler able to catch getter and setter exceptions (2nd try). Should solve issue #58.

Changes Unknown when pulling 2a9d96f493542156cc6281b114d5757cfe7da280 on Maros-1984:master into ** on SQiShER:master**.


Last bugfix

Final bugfix (solves both the "RocksDbException: no locks available" and "NPE on dbAccessor" issues)
In AssignedTasks (the abstract class from which the standby and active task variations extend) a commit failure (even due to broker down/unavailable) is treated as a TaskMigratedException after which the failed task is closed as a zombie and removed from running -- the remaining tasks (ie those still in running are then also closed as zombies in the subsequent onPartitionsLost
However we do not remove the closed task from runningByPartition nor do we remove the corresponding changelogs, if restoring, from the StoreChangelogReader since that applies only to active tasks, and AssignedTasks is generic/abstract. The changelog reader then retains a mapping from the closed task's changelog partition to its CompositeRestoreListener (and does not replace this when the new one comes along after the rebalance). The restore listener has a reference to a specific RocksDBStore instance, one which was closed when the task was closed as a zombie, so it accidentally tries to restore to the "old" RocksDBStore instance rather than the new one that was just opened.

Passed with --repeat 4 on my debug branch (basically this + logging) : https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3308/
Passed with --repeat 8 on debug branch: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3309/
Still running (--repeat 15) on this branch: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/3310


Radio Page Improvements

📝 Changed routes:

/radio/[slug]
/radio

(view full size)
(view full size)

And 1 other route:

/api/revalidate/radio

Commit efc070cf1cce5a53f492e0a4bc5ab27be1165401 (https://refuge-worldwide-idjfr8vx3-refugeworldwide.vercel.app).


Shellcheck *.sh files in the CI

This PR adds a step to the CI that runs shellcheck on all *.sh scripts of the repo. Shellcheck is the standard tool for ensuring the quality of shell scripts and there are docker images readily available for running it in the CI, making this a low-hanging fruit for keeping the quality of scripts high. The new CI step automatically applies to newly added scripts.
To test this MR locally:

Install shellcheck
Break an existing script, for example replace echo "shellcheck $script" by echo $script in scripts/shellcheck_all.sh
Run ./scripts/shellcheck_all.sh, witness it shows an error and returns a non-zero error code.

Check that I didn't break check_licence_headers.sh:

Break a license header, for example remove the first line of src/main/scala/dicom/ScalaVR.scala
Run ./scripts/check_licence_headers.sh, witness it fails, as expected.

I have tested this MR on the CI, see this run where I had broken the script ./scripts/check_license_headers.sh.

@GuillaumeDesforges> what is the workflow here? Do you expect me to merge? Or will you do it?

Please merge it :+1:  thanks !


Make required fields required; to-one relations don’t need arguments

This makes the types generated more accurate:
type Review {
  id: Int
  title: String
  stars: Int
  text: String
  reviewerId: Int!
  movieId: Int!
  reviewer: Person!
  movie: Movie!
}

Does this make any sense? If a property is needed whe creating or updating a model it doesn't necessarily mean that it needs to be selected when querying? Or is that what required means in graphql (I have never used it)?

@koskimas As the spec explains:

Fields are always optional within the context of a query, a field may be omitted and the query is still valid. However fields that return Non‐Null types will never return the value null if queried.

So, I'm not quite using the correct term. This should be called Make required fields non-null.


Can't use some commands

When I try to use a command such as hack -place mini_market and press enter, it doesn't do anything at all. If I add any other character to it, or make it 1 character less it gives me an error. Something is wrong with the redirection of the function

Dev is working on it hardly.... please be patient till bugfix is released ;)

0.08 is totally bugged, I'm really sorry.
However, it's currently fixed in 0.09, the branch have not been merged on stable yet, but you can still play the game from the 0.09 branch with all bugs fixed.
I will close it once merge 0.09 to stable.

It's good, beta-0.09 have been merged to stable branch. Just waiting for the host of the alpha-tester server to update the repo.


Instrument React Router into the project

Description
Linked to Issue: #19
Instrument a Router with a single route / into the project.
Changes

Add devDependencies to package.json:

@testing-library/react - unit testing utilities for React
@testing-library/jest-dom - dom element matchers for React component unit testing

Add dependency to package.json:

react-router-dom

Change filename app/index.tsx to app/App.tsx in preparation for convention changes

File names should match the component name

Wrap App component in React.StrictMode component
Create app/AppRoutes.tsx component using react-router

Create a / root route to render the Home component

Update client/entry.tsx to wrap the App in a BrowserRouter for client-side hydration
Update server/index.tsx to defer routing to the React App, wrapped in a StaticRouter for server-side rendering
Configure jest.config.json to set testEnvironment to jsdom for component test cases

Steps to QA

Pull down and switch to this branch
Run npm run build:dev and ensure the build runs without issue

Run node dist/index.js to ensure the server runs without issue. Verify in browser.

Run npm start to ensure watcher builds and runs the server without issue

Verify in browser

TODOs have been added in #19 (in a comment) based off of additional tasks found in the development of this PR.


430 picard metrics for bam files

coverage: 85.586% (+0.009%) from 85.577% when pulling 1d309f2c7bef2ae50fa88e5633e070f922119ce7 on 430-picard-metrics-for-bam-files into c5f9dde2d2ce8570779030c1917b3132918ad33b on main.


Update README.md

added a space on line 4 to separate the HTML tag and markdown code!

Bohot tez ho rhe ho 😂😂

Bhai mujhe hacking sikhna he


allow passing in libyaml cflags by env var

Heya,
I'm just learning rust, so i'm not sure if this is the best way. On my system (OSX 10.10) the package manager homebrew installs libyaml under /usr/local/
which isn't picked up by the codegen compile so this just allows me to pass an env var to get it to compile.
LIBYAML_CFLAGS=-I/usr/local/include cargo build

Thank you for the commit.
While your commit works OK on my macbook, it looks like travis-ci build fails. I'll try again in my linux box later.


mc_att_control: output zero throttle in manual mode when landed

MPC_MANTHR_MIN is used as minimum throttle in attitude control mode when
the vehicle is in air. This is useful to retain some control around roll and
pitch axis if airmode is not enabled and the user demands zero throttle.
However, when the vehicle is landed there is not need to keep the throttle
at a higher value than zero.
SITL testing:

Log:
https://logs.px4.io/plot_app?log=d6e6fc13-9e50-44aa-888a-47a9f66801d2

This is also how the system behaved in PX4 1.8.

@Stifael could you review this as well?

wouldn't know what negative impact this PR possibly could have


can't install

I tried the same method:
**npm install linkedin-quizzes-kodyfire**
but those resulting kind of vulnerabilities, here is how it looks like:
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '>=14.16' },
npm WARN EBADENGINE   current: { node: 'v12.22.9', npm: '8.5.1' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '>=14.16' },
npm WARN EBADENGINE   current: { node: 'v12.22.9', npm: '8.5.1' }
npm WARN EBADENGINE }

up to date, audited 585 packages in 5s

146 packages are looking for funding
  run _npm fund_ for details

23 vulnerabilities (2 low, 6 moderate, 14 high, 1 critical)

To address issues that do not require attention, run:
  npm audit fix

Some issues need review, and may require choosing
a different dependency.

Run _npm audit_ for details.

I tried both, (npm audit, npm fund) commands, still the same message!
I'm looking forward, guys
Thanks in advance

Thank you @anis-marrouchi
I installed the nodejs  v16.17.0 version, and that's helped to install it, you are a problem solver.

You're welcome @mohammedouahman.
I'm going to close this issue.


Update documentation for investment tax

This PR to @jdebacker's branch updates the documentation to reflect the new investment tax credit. This PR changes profit, interest rate, and government revenue equations in firms.md, government.md, stationarization.md, and equilibrium.md. Also, the equations for K_m/Y_m and K^{r*}_{m} are updated in equilibrium.md.

Thank you for the doc updates @rickecon!


Internet-facing load balancer creation flow allowing individual VMs from an availability set to be added

for example:
azure.loadBalancer().define(newName)
                .withRegion(region)
                .withExistingResourceGroup(groupName)
                .withExistingVirtualMachines(vm1, vm2)
                .withExistingPublicIpAddresses(pip1, pip2)
                .create();

Can one of the admins verify this patch?


Fix bench of issues with new lint tool

https://github.com/uber/cadence/issues/2961
When using better lint tool, it throws hundreds of violations.
This PR address half of them, and fixes some bugs majorly around:

error check
bugs caused by copy paste
remove unused and dead code

(canary is marked as nolint for now, since it is not fully functioning)
There will be following PR address other issues.

Coverage increased (+0.09%) to 67.843% when pulling 34d7ce4628854cfa39876bf30f7236e0bfc1a025 on flint into 8554d88ffaea91e6b614b3dde8f1e0ce3f6efb6f on master.


is GTP-4 api supported？

GTP-4 has been released, is GTP-4 api supported?

GTP-4 has been released, is GTP-4 api supported?

@snow8261 GPT-4 uses the exact same API structure as gpt-3.5-turbo, so it's only a matter of changing the model name from gpt-3.5-turbo to gpt-4.

We plan to introduce a feature that will enable users to select the model to be used for their current chat.


Add explicit support for C++14 and newer

Currently the library cannot be compiled with C++14 and newer unless UAVCAN_CPP_VERSION is overridden.

Bumping this as we are facing this issue when trying to build our project using C++14. Is there anything blocking support for C++14 or is it just a matter of adding correct detection logic to build_config.hpp?

Yes the change would be very trivial as you described. Or you could just define UAVCAN_CPP_VERSION=UAVCAN_CPP11.

I think we need to fix up the top level CMakeLists.txt. I'll post a PR to see what we think of my plan.

Closing as we allow it now. Issue #228 tracks remaining issues.


add TaskGroup here or in another library

asyncio could really use a TaskGroup implementation, as provided by Trio (aka nurseries), Curio, aiotools, and EdgeDB.
TaskGroup was originally slated to be added Python as long ago as 3.8, but was held up because exception group handling was not available. However, exception groups are now being added: PEP 654, PR). Once that PEP is implemented, TaskGroup will probably follow. But the timeline for PEP 654 to go into the MicroPython core is unclear. So any TaskGroup we add will not be quite compatible, and we probably want to add it in an ancillary library (adafruit_asyncio or asynciox or xasyncio or ??).

TaskGroups going in to CPython 3.11:
https://bugs.python.org/issue46752
https://github.com/python/cpython/pull/31270
Quattro version (not using multi-except):
https://github.com/Tinche/quattro

I agree it would be a good idea to support structured concurrency, as going head-on with the older asyncio primitives can get pretty messy.


add voltage level editor

Add a container that displays and manipulates the voltage level element in the substation section of the SCL file.
The following user interaction shall be included

add bay element
add power transformer element
edit name, desc, nomFreq, numPhases attributes as well as voltage element  of the voltage level element
voltage element attributes shall be multiplier="k" and unit="V"
delete the voltage level element
add LNode to the voltage level element - opens UI to connect and save the bay element in the destination filter
copy bay structure (excluding the LNode)

Closed by #40


ORC-1432: Add MacOS 13 GitHub Action Job

What changes were proposed in this pull request?
This PR aims to add MacOS 13 GitHub Action Job.
Why are the changes needed?
MacOS 13 Ventura is the latest MacOS.

https://en.wikipedia.org/wiki/MacOS_Ventura

How was this patch tested?
Pass the CIs.

+1, LGTM (Pending CIs).

Thank you, @guiyanakuang ! Newly added MacOS 13 jobs passed.


Update Markdown Tables on Notebooks

https://github.com/executablebooks/MyST-Parser/issues/430

Current version: myst-parser==0.15.2

Updated to version: myst-parser==0.17.0 and I did not get the same error:
in render_table assert token.children and len(token.children) > 1 AssertionError

Another error
myst-nb 0.13.2 requires myst-parser~=0.15.2, but you have myst-parser 0.17.0 which is incompatible

With mys-parser I get the following error:
  File "c:\users\user\appdata\local\programs\python\python39\lib\site-packages\myst_parser\main.py", line 235, in default_parser
    raise NotImplementedError(
NotImplementedError: default_parser has been deprecated and replaced by create_md_parser.You must also supply the renderer class directly to create_md_parser.

I had to revert to
myst-nb 0.13.2 requires myst-parser~=0.15.2
and simply removed empty tables


Apple/Android: migrate connectivity checks to Go

Migrates connectivity checks to Go, through the go-tun2socks library. Deprecates native connectivity checks (21cb6a2, 18a827e).
Removes CocoaAsyncSocket from third party dependencies.
Updates Cordova go-tun2socks libraries.

Thanks for the review @bemasc. This PR only affects Android, iOS, and macOS clients (aka Cordova). I updated the title to reflect the affected platforms.

Unsurscibe me
Misty n A.j.
On Fri, Sep 13, 2019, 4:31 PM alalamav<EMAIL_ADDRESS>wrote:

Merged #664 https://github.com/Jigsaw-Code/outline-client/pull/664 into
alalama-go-tun2socks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jigsaw-Code/outline-client/pull/664?email_source=notifications&email_token=AI6HQJVHP4I2VSEESB42WGLQJQBD7A5CNFSM4IVBFAIKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTT44DCA#event-2633613704,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI6HQJTRSMO4AZOLV7UMFETQJQBD7ANCNFSM4IVBFAIA
.


Marker: onPositionChanged does not provide event

The onPositionChanged event does not provide the triggered event.
See StackOverflow
class FindLocationMarker extends Component {

  onMarkerPositionChange = (event) => {
    let target = event.target; // event is undefined
    this.props.handleMarkerPositionChange(target);
  }

  render() {
    let that = this;
    return (
      <div>
        <Marker
          position={this.props.position}
          draggable
          onPositionChanged={this.onMarkerPositionChange}
        />
      </div>
      )
  }
}

Yes, no args from vanilla.

https://developers.google.com/maps/documentation/javascript/3.exp/reference#Marker
You may use ref with public methods to get the location.

a use ref and call get Postion. but null value :(

It's a function. Call it .lat()

For everybody whos looking for a solution.

Thanks @daniellobrunello ! Would you mind open a PR containing the demo code for this feature?


Migrate creates unnecessary migration with SQL syntax error

Reproduction repository: https://github.com/janpio/prisma-sqlite-bug
See README for how this was created.

Check out https://github.com/janpio/prisma-sqlite-bug/tree/353667e9b4bd8b813d84546178f648fe505bd825
Run npm install
Run npx prisma db pull to introspect the schema so it definitely matches the database
Run npx prisma migrate dev --create-only
Observe invalid SQL on line 7 of the migration.sql: INSERT INTO "new_USER" ("email", "id") SELECT "email", coalesce("id", ) AS "id" FROM "USER";
Complete file:-- RedefineTables
PRAGMA foreign_keys=OFF;
CREATE TABLE "new_USER" (
    "id" INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT,
    "email" TEXT
);
INSERT INTO "new_USER" ("email", "id") SELECT "email", coalesce("id", ) AS "id" FROM "USER";
DROP TABLE "USER";
ALTER TABLE "new_USER" RENAME TO "USER";
Pragma writable_schema=1;
CREATE UNIQUE INDEX "sqlite_autoindex_USER_1" ON "USER"("email");
Pragma writable_schema=0;
PRAGMA foreign_key_check;
PRAGMA foreign_keys=ON;

Just to see the error message, run npx prisma migrate deploy:λ npx prisma migrate deploy
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": SQLite database "dev.db" at "file:./dev.db"

2 migrations found in prisma/migrations
Error: P3018

A migration failed to apply. New migrations cannot be applied before the error is recovered from. Read more about how to resolve migration issues in a production database: https://pris.ly/d/migrate-resolve

Migration name:<PHONE_NUMBER>2119_should_be_empty

Database error code: 1

Database error:
near ")": syntax error
   0: sql_migration_connector::sql_database_step_applier::apply_script
           with migration_name="20210910212119_should_be_empty"
             at migration-engine\connectors\sql-migration-connector\src\sql_database_step_applier.rs:99
   1: migration_core::commands::apply_migrations::Applying migration
           with migration_name="20210910212119_should_be_empty"
             at migration-engine\core\src\commands\apply_migrations.rs:68
   2: migration_core::api::ApplyMigrations
             at migration-engine\core\src\api.rs:69

If you want you can replace steps 4 to 6 with running npx prisma db push instead and get this error message directly:
λ npx prisma db push
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": SQLite database "dev.db" at "file:./dev.db"
Error: near ")": syntax error
   0: sql_migration_connector::sql_database_step_applier::apply_migration
             at migration-engine\connectors\sql-migration-connector\src\sql_database_step_applier.rs:11
   1: migration_core::api::SchemaPush
             at migration-engine\core\src\api.rs:161

Seems the initial migrated SQL from the repository https://github.com/janpio/prisma-sqlite-bug/blob/main/prisma/migrations/init/migration.sql creates a SQLite database, that when introspected, creates a schema that Migrate will try to migrate again (although it should of course not) and then create invalid SQL to do that.

So this issue is interesting. Your "mistake" here was to not define the PRIMARY KEY column as NOT NULL. What the current version does in this case is the schema says the column is NOT NULL, but this is only due compatibility reasons. Actually without setting it NOT NULL and not defining a default value, we get an autoincrement column and if you explicitly try to insert a null value to the column, you'll get the next rowid integer instead.
My change now is to just consider even NULL columns as required, if the column is part of a primary key and the key holds only one column.


Fix misspelled "starred" and added better instructions

Changed 'stared' to 'starred' and added more clear instructions for copying the initialization snippet

I signed it!
On Sat, May 21, 2016 at 8:47 AM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/firebase/quickstart-js/pull/1#issuecomment-220764440

Thanks @lorint ! :+1:


Add backend to test matrix

Perfect. There are some duplicate tests that are triggered, but this can be fixed. If tests pass then LGTM.

Which duplicate tests are you referring to?

Also, I'm almost certain test/pytest (ray, 3.7) is stalling because of an issue related to #263. If it's okay with you, I'll merge this PR despite test/pytest (ray, 3.7) timing out

Also, I'm almost certain test/pytest (ray, 3.7) is stalling because of an issue related to #263. If it's okay with you, I'll merge this PR despite test/pytest (ray, 3.7) timing out

Can we wait/rerun just to be certain?

Perfect. There are some duplicate tests that are triggered, but this can be fixed. If tests pass then LGTM.

Which duplicate tests are you referring to?

Anything which doesn't rely on an ArrayApplication fixture, e.g. tests.core.test_app_manager

Perfect. There are some duplicate tests that are triggered, but this can be fixed. If tests pass then LGTM.

Which duplicate tests are you referring to?

Anything which doesn't rely on an ArrayApplication fixture, e.g. tests.core.test_app_manager

Gotcha. Yeah, this should be pretty straightforward to do in a follow-up PR

Also, I'm almost certain test/pytest (ray, 3.7) is stalling because of an issue related to #263. If it's okay with you, I'll merge this PR despite test/pytest (ray, 3.7) timing out

Can we wait/rerun just to be certain?

Yeah, that's fine. Re-running now


Add nominee: SisuID

Automatic addition of a new nominee submitted through the online form available at https://digitalpublicgoods.net/submission

@lacabra from the information provided the solution is yet to be released as open source. Have reached out to the project contact person and informed them that we are unable to complete the nomination until that indicator is ticked. Also requested them once they have open sourced the solution to also provide the details below:

explicit links to the licenses for software, data and standard such GitHub pages.
evidence links to support SDGs 8 and 9 if available.
explicit link to repository.

Moving this PR to archive due to slow response as the per SOP archiving policy.


fix: sanitize file names before downloading

Description

Strip out url and filename unfriendly characters from content disposition header when downloading
Should fix https://oak-national-academy.sentry.io/issues/6482803
Potentially fixes https://oaknationalacademy.slack.com/archives/C05RYE3JL4S/p1727855976518189

:tada: This PR is included in version 1.9.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Actions Features

In doing the initial prototype, here are a few things that I thought of that we should consider for actions:

do() returns a promise with redo() and undo().  It is up the method implementor to figure out what it actually needs to do to undo something.  A potential improvement is that the action.state is cloned after .do() and passed into undo()/redo().    Another way of managing this is that whatever is returned/resolved from the do() (or previous method in the chain) is passed into undo()/redo(). (Actually writing that down, I liked the second option)
When no undo() method is specified upon instantiation, it results in a noop, which is resolved with no value.  Maybe it makes sense to resolve with the previous value in the chain.
Not sure what the best behaviour is for rejection/throws in a long chain.  I realised I didn't actually code a behaviour, but I don't know if a long chain of .do().undo().redo() if each subsequent method should just reject with the previous value.
Post instantiation, there is no designed way to replace the do()/undo()/redo() implementation methods.  Don't know if that is desired/logical.
There is no way of customising the returned promise further.  Again, I don't know if this just adds arbitrary complexity.
It isn't based on Evented. Therefore there are no built in event mechanisms.  Again, I don't know if this is logical/expected.
At run-time, calling do(), undo(), or redo() with different arity of arguments then specified in the interface does not pass these onto the implementation method.  This is sort of "mean" in the sense that if you decided you want to pass something to undo(), your method won't receive it.  Conceptually it would be anti-pattern to supply arguments to undo() but is it fair to actually discard them?

For implementors to be able to implement undo() they are generally going to need to know precisely what do() did, which means knowing the state prior to the execution of do() and the state immediately after the execution of do(). Right now state only exists on the action factory; no state history is maintained by the trail of actions. Passing the result of do() to undo() does not address this - undo() still doesn't have the information it needs to be able to undo whatever do() did. So I think we may need to clone state before executing do() and store it on the action. And then pass the result of do() to undo().
Yeah, I think to keep the chain valid noops should probably return the result of the most recent successful op.
I think a rejection or error should cancel the rest of the chain.
I think we'll have to put actions into practice to see. It does look like destroying an action will currently enable you to redefine it.
No opinion on customizing returned promise.
Can't think of why it would need Evented.
Abide by the method signature or suffer the consequences!

I'm not quite sure how redo() should work, but it might not make sense to have it default to do() when undefined. I would think like undo(), redo() would take no parameters. I think it would be clearer in client code to simply re-execute do() (along with the relevant parameters) than to call redo() to achieve the same effect.
redo() should only be executable when undo() has been called.

It seems like we might not want to decorate errors. Or decorate them in a way that undo() and redo() simply propagate the error.

no idea what the purpose is of this class, and no offense but: i tried this model and it didn't work out well for me, especially on the performance side. Consider a few grid instances, having each around 100 actions easily (60% are standard grid actions) and each of those can have an own internal version of its own per visibility, containing a state for label, enabled, icon, widget-class, widget options but also optional processing options (action value from/to its reference) . Thats usually around 3-5+ of them: context menu, local toolbar, global toolbar, main - menu, and property views,...). Of course not all need that but rich actions like sort, group, option toggles do indeed need an override for these widgets.
And, there isn't only do/undo/redo btw., I had to introduce 'preview' as well (do/undo isn't really an option).  I optimized this with dstore backed ring buffer cache which avoids re-creation of the action class(usually also a dmodel object).
However, I am amazed where this goes, working on a good and flexible, and re-usable 'Action' class turned out as bigger project that thought over the years, keeps me busy almost every day.
just saying, have fun

no idea what the purpose is of this class, and no offense but: i tried this model and it didn't work out well for me, especially on the performance side.

To be honest, it seems like you do, because you expound up the reasons why it doesn't work... so I assume you understand what it is for.

Thanks for your thoughts on where you struggled with something similar though...  Avoiding many of those issues will be a consideration.

Thanks for the links, very interesting!  Btw. and just to round this up: you actually need at least 3 classes for user-actions:

Action. Btw. Instead of the type property, its 'command' (dmodel collides here). I am using this property for the action - store id but also to extract the label from the last command segment (Clipboard/Copy). I also use it to build menu trees, using the overrides to restructure the rendering.

ActionProvider. A mixin and interface: createAction, runAction(action,args), onBeforeAction, onAfterAction(restores usually focus and a new or old selection). This is applied to grids and will be the default owner of the action created. You actually also have multiple Action-Stores here. A local, and sometimes global (plugins) action - stores.

ActionConsumer. A mixin for Action-Renderers like Context-Menu, Ribbons, Main-Menu. It will use the local property 'visibility' to find overrides for each action when rendered. This class also should support multiple ActionProviders. (Main-Menu for instance). This interface has setProvider, addProvider, removeProvider.

'do' is btw. implemented most of the time by the owner, not the action itself.
After all you want this:

grid.runAction(action|commandString,args|null). Most of the time this will be applied to the current selection
grid.setActionProperty(action|command-string,key,value). For instance: setActionProperty('Clipboard/Paste','disabled',true) should update all referencing widgets (MenuItem, Toolbar-Button,...) and disable this action visually.
Create compound actions out of existing actions, as you did for Maqetta.
Highlight actions visually (wizards, intro tours).
Having easier unit-tests and create a sort of macro system to compose new actions more easy.

ok, just wanted to add my 2 cents (after fighting with Maqetta for years on this ;-)

This,one of my mess pretty much illustrates the problem. All versions of an action are linked to a another target widget with different mappings per binding. They beaty I figured was that most actions could be displayed with a grid, its all there already.

Closing for now... we have revised Actions for the time being to simplify them and may revisit features in the future.


Allow custom headers in nginx config.

So that operators can set custom headers for each service in nginx.

@jmcarp What's the use case for this?

@frodenas: our use case is compliance--we're required to set security-related headers like X-Frame-Options, X-XSS-Protection, etc, for everything we run.

Understood, thanks @jmcarp!


[FEATURE REQUEST] Support for S3 compatible services

Is your feature request related to a problem? Please describe.
Support for S3 compatible services like min.io, cloudflare r2 and etc.
Describe the solution you'd like
Allow other services compatible with S3 instead of just AWS.
Describe alternatives you've considered
Only options are use one of the supported storage.
Additional context
You might want to try do add a generic implementation to handle all S3 compatibles generic services, but will not be possible to have specific credentials per principal.
Or try to keep different implementations, per example use cloudflare R2 api token to generate new access_key and secrets for each principal with a limited scoped.
I am using cloudflare, thus I have interest in implement this feature. Just wanna know if do you have plans to create a generic S3 support or I should focus on a specific implementation (R2) to allow principal credentials revocation.

Isn't this a duplicate of #32?

@snazy Yes it is. Sorry about that.


feature: log the events to help debugging issues

The goal of this change is to let us know when certain events weren't raised. This should help us debug issues when an application is reporting that they didn't receive some key events.
Also, the debugging line should help us spot how many events have been raised for certain resource types.

Looks good to me!
One thing I might watch out for...how big are the events (resource mode.Event) going to be? It looks like this will print out the entire event if Debug is enabled. So it might be an issue if the events are "big".

Thank you for the tip, yeah. I have ended up reverting the change because these logs didn't provide much information: the payload had some references, instead of the dereferenced structs I was expecting. Also, the errors were being logged on the caller functions so the logs were being duplicated. Thanks again!